Re: [PATCH] Rump on GNU/Hurd (4): Userspace PCI I/O

  • From: Robert Millan <rmh@xxxxxxx>
  • To: Antti Kantee <pooka@xxxxxx>, rumpkernel-users@xxxxxxxxxxxxx
  • Date: Mon, 31 Aug 2015 16:04:17 +0200

El 31/08/15 a les 13:07, Antti Kantee ha escrit:

On 30/08/15 15:10, Robert Millan wrote:
But that's not what you were asking for. I don't know what's wrong
based on the above. Can you paste the entire Makefile and command line?

Makefile is attached (in my tree, this is pci-userspace/src-gnu/Makefile)

Command-line is:

../../buildrump.sh/obj/tooldir/rumpmake dependall

Ah. That happens because the make which needs experimentalUser.c does not see
the rule in the current Makefile (everything in SUBDIR is run in another make).

The easy way to fix it is to add the following line to the Makefile containing
the rule and SUBDIRs:

.BEGIN: experimentalUser.c

It's a slightly weird way to use make, though.

I wonder if rump/dev/lib/libpci should look at a variable to decide if it needs to
include some further definitions, e.g. RUMPCOMP_MAKEFILEINC.compname. That would solve
both LDFLAGS and this issue. "leave it to the client"

I had some trouble with the .BEGIN approach, but the MAKEFILEINC one works
perfectly. I'm attaching a patch.

--
Robert Millan
Index: rumpkernel-0~20150715/buildrump.sh/src/sys/rump/dev/lib/libpci/Makefile
===================================================================
--- rumpkernel-0~20150715.orig/buildrump.sh/src/sys/rump/dev/lib/libpci/Makefile
+++ rumpkernel-0~20150715/buildrump.sh/src/sys/rump/dev/lib/libpci/Makefile
@@ -41,6 +41,8 @@ CPPFLAGS+= ${RUMPCOMP_CPPFLAGS.rumpdev_
# XXX: messy
.undef RUMPKERN_ONLY

+.include "${RUMPCOMP_MAKEFILEINC.rumpdev_pci}"
+
.include "${RUMPTOP}/Makefile.rump"
.include <bsd.lib.mk>
.include <bsd.klinks.mk>

Other related posts: