From owner-svn-src-head@freebsd.org Fri Mar 10 20:24:39 2017 Return-Path: Delivered-To: svn-src-head@mailman.ysv.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1]) by mailman.ysv.freebsd.org (Postfix) with ESMTP id 16C3BD06D78; Fri, 10 Mar 2017 20:24:39 +0000 (UTC) (envelope-from bdrewery@FreeBSD.org) Received: from freefall.freebsd.org (freefall.freebsd.org [IPv6:2610:1c1:1:6074::16:84]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (Client CN "freefall.freebsd.org", Issuer "Let's Encrypt Authority X3" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id DB1081C41; Fri, 10 Mar 2017 20:24:38 +0000 (UTC) (envelope-from bdrewery@FreeBSD.org) Received: from mail.xzibition.com (freefall.freebsd.org [IPv6:2610:1c1:1:6074::16:84]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (Client did not present a certificate) by freefall.freebsd.org (Postfix) with ESMTPS id D5DDBF65; Fri, 10 Mar 2017 20:24:37 +0000 (UTC) (envelope-from bdrewery@FreeBSD.org) Received: from mail.xzibition.com (localhost [172.31.3.2]) by mail.xzibition.com (Postfix) with ESMTP id 52A2835A13; Fri, 10 Mar 2017 20:24:30 +0000 (UTC) X-Virus-Scanned: amavisd-new at mail.xzibition.com Received: from mail.xzibition.com ([172.31.3.2]) by mail.xzibition.com (mail.xzibition.com [172.31.3.2]) (amavisd-new, port 10026) with LMTP id PDh9dhUJQXH7; Fri, 10 Mar 2017 20:24:18 +0000 (UTC) Subject: Re: svn commit: r314808 - in head: contrib/bmake contrib/bmake/PSD.doc contrib/bmake/mk contrib/bmake/unit-tests usr.bin/bmake DKIM-Filter: OpenDKIM Filter v2.9.2 mail.xzibition.com 8428E35A0A To: "Simon J. Gerraty" , src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org References: <201703062302.v26N2Xt6009979@repo.freebsd.org> From: Bryan Drewery Organization: FreeBSD Message-ID: <796c4d4e-db18-d50d-0811-4a8a15f569e1@FreeBSD.org> Date: Fri, 10 Mar 2017 12:24:14 -0800 User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.12; rv:45.0) Gecko/20100101 Thunderbird/45.7.1 MIME-Version: 1.0 In-Reply-To: <201703062302.v26N2Xt6009979@repo.freebsd.org> Content-Type: multipart/signed; micalg=pgp-sha512; protocol="application/pgp-signature"; boundary="eKjju3IAj3dcJhI85bNfFslTAGThwSesq" X-BeenThere: svn-src-head@freebsd.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: SVN commit messages for the src tree for head/-current List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 10 Mar 2017 20:24:39 -0000 This is an OpenPGP/MIME signed message (RFC 4880 and 3156) --eKjju3IAj3dcJhI85bNfFslTAGThwSesq Content-Type: multipart/mixed; boundary="1qPsnRGRruhs51H01jasOWHN6gnXXg3Hw"; protected-headers="v1" From: Bryan Drewery To: "Simon J. Gerraty" , src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org Message-ID: <796c4d4e-db18-d50d-0811-4a8a15f569e1@FreeBSD.org> Subject: Re: svn commit: r314808 - in head: contrib/bmake contrib/bmake/PSD.doc contrib/bmake/mk contrib/bmake/unit-tests usr.bin/bmake References: <201703062302.v26N2Xt6009979@repo.freebsd.org> In-Reply-To: <201703062302.v26N2Xt6009979@repo.freebsd.org> --1qPsnRGRruhs51H01jasOWHN6gnXXg3Hw Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable On 3/6/17 3:02 PM, Simon J. Gerraty wrote: > Author: sjg > Date: Mon Mar 6 23:02:33 2017 > New Revision: 314808 > URL: https://svnweb.freebsd.org/changeset/base/314808 >=20 > Log: > Merge bmake-20170301 >=20 > Modified: > head/contrib/bmake/ChangeLog > head/contrib/bmake/Makefile > head/contrib/bmake/PSD.doc/tutorial.ms > head/contrib/bmake/bmake.1 > head/contrib/bmake/bmake.cat1 > head/contrib/bmake/dir.c > head/contrib/bmake/main.c > head/contrib/bmake/make.1 > head/contrib/bmake/meta.c > head/contrib/bmake/mk/ChangeLog > head/contrib/bmake/mk/dirdeps.mk > head/contrib/bmake/mk/dpadd.mk > head/contrib/bmake/mk/install-mk > head/contrib/bmake/mk/meta.stage.mk > head/contrib/bmake/mk/meta2deps.py > head/contrib/bmake/mk/prog.mk > head/contrib/bmake/mk/subdir.mk > head/contrib/bmake/mk/sys.mk > head/contrib/bmake/mk/sys.vars.mk > head/contrib/bmake/os.sh > head/contrib/bmake/parse.c > head/contrib/bmake/unit-tests/varmisc.exp > head/contrib/bmake/unit-tests/varmisc.mk > head/contrib/bmake/var.c > head/usr.bin/bmake/Makefile > Directory Properties: > head/contrib/bmake/ (props changed) >=20 > Modified: head/contrib/bmake/ChangeLog > =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D > --- head/contrib/bmake/ChangeLog Mon Mar 6 22:46:49 2017 (r314807) > +++ head/contrib/bmake/ChangeLog Mon Mar 6 23:02:33 2017 (r314808) > @@ -1,3 +1,32 @@ > +2017-03-01 Simon J. Gerraty > + > + * Makefile (_MAKE_VERSION): 20170301 > + Merge with NetBSD make, pick up > + o main.c: use -C arg as is rather than getcwd() > + if they identify the same directory. > + This change is broken for relative paths specified to -C. I had just convinced someone that -C was flawless last week and now it's actually broken! > ~/git/freebsd/lib/libthr # make -C ../libnetbsd obj > /usr/obj../libnetbsd created for ../libnetbsd > ~/git/freebsd/lib/libthr # rm -rf /usr/obj../libnetbsd > ~/git/freebsd/lib/libthr # make -C ../libthr -V .OBJDIR > make: "../libthr/../libthr/Makefile" line 14: Could not find src.opts.m= k > make: "../libthr/../libthr/Makefile" line 55: Malformed conditional (${= MK_INSTALLLIB} !=3D "no") > make: "../libthr/../libthr/Makefile" line 61: Malformed conditional (${= MK_PROFILE} !=3D "no") > make: "../libthr/../libthr/Makefile" line 65: Malformed conditional (${= MK_TESTS} !=3D "no") > make: Fatal errors encountered -- cannot continue > make: stopped in ../libthr > ~/git/freebsd/lib/libthr # make -C ../libnetbsd -V .OBJDIR > make: "/usr/share/mk/bsd.obj.mk" line 1: here > ../libnetbsd/../libnetbsd > ~/git/freebsd/lib/libthr # make -m .../share/mk -C ../libnetbsd -V .OB= JDIR > make: "/root/git/freebsd/share/mk/bsd.obj.mk" line 45: OBJDIR: ../libne= tbsd/../libnetbsd > make: "/root/git/freebsd/share/mk/bsd.obj.mk" line 46: CURDIR: ../libne= tbsd > ../libnetbsd/../libnetbsd 2 problems: 1. .OBJDIR is somehow duplicate ${.CURDIR}${.CURDIR} 2. The default MAKESYSPATH of .../share/mk doesn't work with -C .., it ends up reaching out to /usr/share/mk. An explicit -m works though. Reverting it: > ~/git/freebsd/lib/libthr # make -C ../libnetbsd -V .OBJDIR > make: "/root/git/freebsd/share/mk/bsd.obj.mk" line 45: OBJDIR: /root/gi= t/freebsd/lib/libnetbsd > make: "/root/git/freebsd/share/mk/bsd.obj.mk" line 46: CURDIR: /root/gi= t/freebsd/lib/libnetbsd > /root/git/freebsd/lib/libnetbsd Can this piece please be reverted for now while relative paths get more testing? > diff --git contrib/bmake/main.c contrib/bmake/main.c > index 087438927355..870eefd6e591 100644 > --- contrib/bmake/main.c > +++ contrib/bmake/main.c > @@ -389,6 +389,7 @@ MainParseArgs(int argc, char **argv) > int arginc; > char *argvalue; > const char *getopt_def; > + struct stat sa, sb; > char *optscan; > Boolean inOption, dashDash =3D FALSE; > char found_path[MAXPATHLEN + 1]; /* for searching for sy= s.mk */ > @@ -457,6 +458,11 @@ rearg: > (void)fprintf(stderr, "%s: %s.\n", prog= name, strerror(errno)); > exit(2); > } > + if (stat(argvalue, &sa) !=3D -1 && > + stat(curdir, &sb) !=3D -1 && > + sa.st_ino =3D=3D sb.st_ino && > + sa.st_dev =3D=3D sb.st_dev) > + strncpy(curdir, argvalue, MAXPATHLEN); > ignorePWD =3D TRUE; > break; > case 'D': --=20 Regards, Bryan Drewery --1qPsnRGRruhs51H01jasOWHN6gnXXg3Hw-- --eKjju3IAj3dcJhI85bNfFslTAGThwSesq Content-Type: application/pgp-signature; name="signature.asc" Content-Description: OpenPGP digital signature Content-Disposition: attachment; filename="signature.asc" -----BEGIN PGP SIGNATURE----- Comment: GPGTools - https://gpgtools.org iQEcBAEBCgAGBQJYwwtvAAoJEDXXcbtuRpfPzJYIAK4zFFzhdyPtYKcxe7yzjl+X ba0zNhZKIyOlf5Z73jNPeZASAlwTN8cPcawnG3w53smsWuLkykEqKcmNyc+fwrCf pnyTkMVt8+cJDavD5B15ARpI17mr956taKuEC2OXPuDqBFHxjz+5ajjY10nKvTxL v8Hq1Sbk5U/0IXzRKI3gSO5KKlcQ7fr5zI355Ih5YF27PfwmlhmYQBD1j+GVxXxK EA0xbudvJVDqRwT2oeb0zWmzAg+dE6z+NH+VEpjUBo1I4pKDvS/2fnrUiSFvXpTp n7M6lS65IfWYOBqMF3Sn2r2mYdwQAC+gIEvbE7yz5vBRzirW+6ytg1VyQxPpCMM= =aMJi -----END PGP SIGNATURE----- --eKjju3IAj3dcJhI85bNfFslTAGThwSesq--