Re: CVS commit: src/sys/arch/evbarm/conf

2013-07-16 Thread Havard Eidnes
 Module Name:  src
 Committed By: matt
 Date: Wed Jul 10 20:25:21 UTC 2013

 Modified Files:
   src/sys/arch/evbarm/conf: Makefile.evbarm.inc

 Log Message:
 If using mdsetimage to install a ramdisk install, add the
 ramdisk to SYSTEM_DEP so the kernel will be relinked and the
 new ramdisk installed.


 To generate a diff of this commit:
 cvs rdiff -u -r1.24 -r1.25 src/sys/arch/evbarm/conf/Makefile.evbarm.inc

With this change, SYSTEM_DEP ends up not including any object
files for the evbarm kernels, causing build failure due to
missing files when we get to the link the kernel step.

The reason is that sys/conf/Makefile.kern.inc has:

SYSTEM_DEP?=Makefile ${SYSTEM_OBJ:O} .gdbinit

So if SYSTEM_DEP is already assinged (it is by this change at
this point), SYSTEM_OBJ will not be added to SYSTEM_DEP.

Now, I don't quite understand why that SYSTEM_DEP?= should not be
SYSTEM_DEP+= instead...

Regards,

- HÃ¥vard


Re: CVS commit: src/usr.bin/make

2013-07-16 Thread Ryo ONODERA
Hi,

From: Christos Zoulas chris...@netbsd.org, Date: Tue, 16 Jul 2013 10:00:53 
-0400

 Module Name:  src
 Committed By: christos
 Date: Tue Jul 16 14:00:53 UTC 2013
 
 Modified Files:
   src/usr.bin/make: main.c make.1 var.c
 
 Log Message:
 More gmake compatibility:
 1. add -w flag to print Entering and Leaving directory name the the beginning
and the end of processing.
 2. export MAKELEVEL=$((MAKELEVEL + 1)) only in the child environment.
 3. when printing error messages, prefix them with the program name 
 [$MAKELEVEL]
for $MAKELEVEL  0
 4. if $MAKEFLAGS consists only of letters assume it is a set of flags (as
allowed by posix), convert them to -f -l -a -g -s, so that they get parsed
properly.
 With those fixes gmake - bmake - gmake - bmake etc. works as expected.
 
 
 To generate a diff of this commit:
 cvs rdiff -u -r1.219 -r1.220 src/usr.bin/make/main.c
 cvs rdiff -u -r1.218 -r1.219 src/usr.bin/make/make.1
 cvs rdiff -u -r1.181 -r1.182 src/usr.bin/make/var.c
 
 Please note that diffs are not public domain; they are subject to the
 copyright notices on the relevant files.
 

With this change, I have gotten following errors.
I think -lutil is needed.

cc  -O -DDEFSHELL_CUSTOM=/bin/sh -DHAVE_SETENV=1 -DHAVE_STRDUP=1 
-DHAVE_STRERROR=1 -DHAVE_STRFTIME=1 -DHAVE_VSNPRINTF=1  -c 
/usr/src/tools/make/../../usr.bin/make/lst.lib/lstReplace.c
cc  -O -DDEFSHELL_CUSTOM=/bin/sh -DHAVE_SETENV=1 -DHAVE_STRDUP=1 
-DHAVE_STRERROR=1 -DHAVE_STRFTIME=1 -DHAVE_VSNPRINTF=1  -c 
/usr/src/tools/make/../../usr.bin/make/lst.lib/lstSucc.c
cc  -O -o nbmake *.o
main.o: In function `main':
main.c:(.text+0x2362): undefined reference to `estrdup'
main.c:(.text+0x23ca): undefined reference to `estrdup'
main.c:(.text+0x2d6d): undefined reference to `emalloc'

ERROR: Build of nbmake failed
*** BUILD ABORTED ***
r

--
Ryo ONODERA // ryo...@yk.rim.or.jp
PGP fingerprint = 82A2 DC91 76E0 A10A 8ABB  FD1B F404 27FA C7D1 15F3


Re: CVS commit: src/usr.bin/make

2013-07-16 Thread Christos Zoulas
On Jul 16, 11:20pm, ryo...@yk.rim.or.jp (Ryo ONODERA) wrote:
-- Subject: Re: CVS commit: src/usr.bin/make

| With this change, I have gotten following errors.
| I think -lutil is needed.

No, I fixed it... I was supposed to use bmake_foo() not efoo().

christos


Re: CVS commit: src/usr.bin/make

2013-07-16 Thread Ryo ONODERA
From: chris...@zoulas.com (Christos Zoulas), Date: Tue, 16 Jul 2013 10:23:04 
-0400

 On Jul 16, 11:20pm, ryo...@yk.rim.or.jp (Ryo ONODERA) wrote:
 -- Subject: Re: CVS commit: src/usr.bin/make
 
 | With this change, I have gotten following errors.
 | I think -lutil is needed.
 
 No, I fixed it... I was supposed to use bmake_foo() not efoo().

Thank you very much!

--
Ryo ONODERA // ryo...@yk.rim.or.jp
PGP fingerprint = 82A2 DC91 76E0 A10A 8ABB  FD1B F404 27FA C7D1 15F3


Re: CVS commit: src/usr.bin/make

2013-07-16 Thread David Holland
On Tue, Jul 16, 2013 at 08:00:56PM +, Simon J. Gerraty wrote:
  Modified Files:
   src/usr.bin/make: var.c
  
  Log Message:
  When a var is set in the CMD context, it prevents the same name
  being set in GLOBAL context.  We should also delete any such
  variable in GLOBAL context, else make -V will show the wrong value.

Example?

-- 
David A. Holland
dholl...@netbsd.org