From 8058ef787e82504d0b6fab9384f1b74f72393f1a Mon Sep 17 00:00:00 2001 From: Flavio Cruz Date: Thu, 31 Dec 2015 16:45:51 +0100 Subject: Use DEVICE_IMPORTS to include libports/ports.h On Thu, Dec 31, 2015 at 04:28:14PM +0100, Justus Winter wrote: > Quoting Samuel Thibault (2015-12-31 16:11:30) > > Flavio Cruz, on Thu 31 Dec 2015 15:56:31 +0100, wrote: > > > This fixes three compiler warnings about ports_payload_get_name. > > > > > > * devnode/Makefile: Define DEVICE_IMPORTS. > > > * term/Makefile: Likewise. > > > * trans/Makefile: Likewise. > > > > Why not adding them to the corresponding mig-mutate.h? > > Sometimes we don't have a mig-mutate.h, e.g. for most (all?) > translators in trans/. I prefer stuffing that in a .h as well, but I > wouldn't be opposed to just applying the patch, as we use both ways of > mutating the mig types. Agreed. I think only devnode should include it in mig-mutate.h. For term, only some stubs are including mig-mutate.h. For trans, there's no mig-mutate.h at all. Here's the modified patch. --- trans/Makefile | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) (limited to 'trans') diff --git a/trans/Makefile b/trans/Makefile index 2fdcd38b..65b51d12 100644 --- a/trans/Makefile +++ b/trans/Makefile @@ -45,7 +45,9 @@ ifsock-MIGSFLAGS=\ "-DIFSOCK_IMPORTS=import \"../libtrivfs/mig-decls.h\";" # device_replyServer is used by the streamio translator. -device_reply-MIGSFLAGS="-DMACH_PAYLOAD_TO_PORT=ports_payload_get_name" +device_reply-MIGSFLAGS=\ + "-DMACH_PAYLOAD_TO_PORT=ports_payload_get_name" \ + "-DDEVICE_IMPORTS=import \"$(srcdir)/../libports/ports.h\";" # fsysServer is only used by the symlink translator which does not use # libports. Disable the default payload to port conversion. -- cgit v1.2.3