From owner-svn-src-head@freebsd.org Wed Mar 8 06:12:20 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 E2375D02E33; Wed, 8 Mar 2017 06:12:20 +0000 (UTC) (envelope-from freebsd@pdx.rh.CN85.dnsmgr.net) Received: from pdx.rh.CN85.dnsmgr.net (br1.CN84in.dnsmgr.net [69.59.192.140]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (Client did not present a certificate) by mx1.freebsd.org (Postfix) with ESMTPS id B19881A95; Wed, 8 Mar 2017 06:12:20 +0000 (UTC) (envelope-from freebsd@pdx.rh.CN85.dnsmgr.net) Received: from pdx.rh.CN85.dnsmgr.net (localhost [127.0.0.1]) by pdx.rh.CN85.dnsmgr.net (8.13.3/8.13.3) with ESMTP id v286CJDt098842; Tue, 7 Mar 2017 22:12:19 -0800 (PST) (envelope-from freebsd@pdx.rh.CN85.dnsmgr.net) Received: (from freebsd@localhost) by pdx.rh.CN85.dnsmgr.net (8.13.3/8.13.3/Submit) id v286CJHZ098841; Tue, 7 Mar 2017 22:12:19 -0800 (PST) (envelope-from freebsd) From: "Rodney W. Grimes" Message-Id: <201703080612.v286CJHZ098841@pdx.rh.CN85.dnsmgr.net> Subject: Re: svn commit: r314893 - in head/usr.bin/fortune: . datfiles fortune unstr In-Reply-To: <201703080531.v285Vs1Y065051@repo.freebsd.org> To: Ngie Cooper Date: Tue, 7 Mar 2017 22:12:19 -0800 (PST) CC: src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org Reply-To: rgrimes@freebsd.org X-Mailer: ELM [version 2.4ME+ PL121h (25)] MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Type: text/plain; charset=US-ASCII 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: Wed, 08 Mar 2017 06:12:21 -0000 > Author: ngie > Date: Wed Mar 8 05:31:54 2017 > New Revision: 314893 > URL: https://svnweb.freebsd.org/changeset/base/314893 > > Log: > usr.bin/fortune: convert to OBJTOP/SRCTOP idioms > > - Use OBJTOP/SRCTOP-relative paths when looking for include files and > strfile. > - Add FORTUNES_OBJ and FORTUNES_SRC to abbreviate usr.bin/fortune > pathing. Is this a one off change, or are you planning to propogate this to FOO_OBJ and FOO_SRC through out the tree? > This is being done to simplify make output/idioms. An hornorable goal, but I think, given the several different attempts I have seen attempted at this, a step be taken back to really engineer a solution. > MFC after: 1 week Please dont > Reviewed by: bdrewery I had asked that a larger discussion occur on -hackers regarding the SRCTOP changes. I seen you post that, and thank you for doing it, yesterday, however to proceed with a Differential and commit without allowing that discussion to occur is counter productive. > Sponsored by: Dell EMC Isilon > Differential Revision: D9916 > > Modified: > head/usr.bin/fortune/Makefile.inc > head/usr.bin/fortune/datfiles/Makefile > head/usr.bin/fortune/fortune/Makefile > head/usr.bin/fortune/unstr/Makefile > > Modified: head/usr.bin/fortune/Makefile.inc > ============================================================================== > --- head/usr.bin/fortune/Makefile.inc Wed Mar 8 05:27:04 2017 (r314892) > +++ head/usr.bin/fortune/Makefile.inc Wed Mar 8 05:31:54 2017 (r314893) > @@ -1,3 +1,6 @@ > -.if exists(${.CURDIR}/../../Makefile.inc) > -.include "${.CURDIR}/../../Makefile.inc" > -.endif > +# $FreeBSD$ When did this line get lost? > + > +FORTUNE_SRC= ${SRCTOP}/usr.bin/fortune > +FORTUNE_OBJ= ${OBJTOP}/usr.bin/fortune > + > +.include "${SRCTOP}/usr.bin/Makefile.inc" You have now added 3 hard codings of usr.bin into a Makefile which makes it requre an edit to move this part of the src tree to any other part. Is that REALLY the direction we want to go? > > Modified: head/usr.bin/fortune/datfiles/Makefile > ============================================================================== > --- head/usr.bin/fortune/datfiles/Makefile Wed Mar 8 05:27:04 2017 (r314892) > +++ head/usr.bin/fortune/datfiles/Makefile Wed Mar 8 05:31:54 2017 (r314893) > @@ -15,7 +15,7 @@ FILESDIR= ${SHAREDIR}/games/fortune > > .for f in ${DB} > $f.dat: $f > - PATH=$$PATH:/usr/bin:${.OBJDIR}/../strfile \ > + PATH=$$PATH:/usr/bin:${FORTUNE_OBJ}/strfile \ Are we seriously talking about introducing ~6000 variables into the .mk system? > strfile -Cs ${.ALLSRC} ${.TARGET} > .endfor > > > Modified: head/usr.bin/fortune/fortune/Makefile > ============================================================================== > --- head/usr.bin/fortune/fortune/Makefile Wed Mar 8 05:27:04 2017 (r314892) > +++ head/usr.bin/fortune/fortune/Makefile Wed Mar 8 05:31:54 2017 (r314893) > @@ -3,7 +3,7 @@ > > PROG= fortune > MAN= fortune.6 > -CFLAGS+=-DDEBUG -I${.CURDIR}/../strfile > +CFLAGS+=-DDEBUG -I${FORTUNE_SRC}/strfile > > .include > > > Modified: head/usr.bin/fortune/unstr/Makefile > ============================================================================== > --- head/usr.bin/fortune/unstr/Makefile Wed Mar 8 05:27:04 2017 (r314892) > +++ head/usr.bin/fortune/unstr/Makefile Wed Mar 8 05:31:54 2017 (r314893) > @@ -3,6 +3,6 @@ > > PROG= unstr > MAN= > -CFLAGS+= -I${.CURDIR}/../strfile > +CFLAGS+= -I${FORTUNE_SRC}/strfile > > .include > > -- Rod Grimes rgrimes@freebsd.org