From: chris...@astron.com (Christos Zoulas), Date: Wed, 10 Jul 2013 15:08:39 +0000 (UTC)
> In article <20130710.232445.202823964487691297.ryo...@yk.rim.or.jp>, > Ryo ONODERA <ryo...@yk.rim.or.jp> wrote: >>From: chris...@astron.com (Christos Zoulas), Date: Wed, 10 Jul 2013 >>13:08:49 +0000 (UTC) >> >>> In article <krbu04$2j1$1...@ger.gmane.org>, >>> Christos Zoulas <chris...@astron.com> wrote: >>>>>Module Name: src >>>>>Committed By: sjg >>>>>Date: Sat Jul 6 18:19:17 UTC 2013 >>>>> >>>>>Modified Files: >>>>> src/usr.bin/make: main.c var.c >>>>> >>>>>Log Message: >>>>>If using gmake's MAKELEVEL; use it the same way >>>> >>>>Now you put it back the way it was before which is wrong. Gmake does not >>>>behave this way. Before your change the following Makefile printed: >>> >>> This is still broken. Renaming the variable is not a fix either. >>> Consider the case where we switched from bmake to gmake as the >>> pkgsrc wrapper. The packages that depend on $MAKELEVEL being 0 or >>> unset would still break. So the proper fix for the 2 packages that >>> broke is to make pkgsrc unset MAKELEVEL before invoking gmake. >>> >>> So to fix those: >>> - revert the changes so that MAKELEVEL again works the same way as in gmake. >>> - add glue to invoke gmake with MAKELEVEL unset. >> >>Hi, >> >>I feel second idea is not so good. >>Because pkgsrc creates symlink, work/.tools/bin/make, for our make and gmake. >>So with gmake in USE_TOOLS, it is symlink for /usr/pkg/bin/gmake, >>and without gmake in USE_TOOLS, it is for /usr/bin/make. >>I cannot distinguish them with filename. > > It is simple: > > 1. You unset it for all (easiest+safest) > 2. You create a shell script instead of a symlink that does: > unset MAKELEVEL && exec /usr/pkg/bin/gmake "$@" Thanks for your explanation. In pkgsrc case, shell script may work. Outside pkgsrc, some program may be broken by this make(1) change. I vote to "revert the changes so that MAKELEVEL again works the same way as in gmake". Thank you. -- Ryo ONODERA // ryo...@yk.rim.or.jp PGP fingerprint = 82A2 DC91 76E0 A10A 8ABB FD1B F404 27FA C7D1 15F3