From owner-svn-src-head@freebsd.org Wed May 22 21:06:28 2019 Return-Path: Delivered-To: svn-src-head@mailman.ysv.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2610:1c1:1:606c::19:1]) by mailman.ysv.freebsd.org (Postfix) with ESMTP id D392415B779E; Wed, 22 May 2019 21:06:27 +0000 (UTC) (envelope-from freebsd@gndrsh.dnsmgr.net) Received: from gndrsh.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 2E2A36A8A7; Wed, 22 May 2019 21:06:26 +0000 (UTC) (envelope-from freebsd@gndrsh.dnsmgr.net) Received: from gndrsh.dnsmgr.net (localhost [127.0.0.1]) by gndrsh.dnsmgr.net (8.13.3/8.13.3) with ESMTP id x4ML6IH4086720; Wed, 22 May 2019 14:06:18 -0700 (PDT) (envelope-from freebsd@gndrsh.dnsmgr.net) Received: (from freebsd@localhost) by gndrsh.dnsmgr.net (8.13.3/8.13.3/Submit) id x4ML6Idr086719; Wed, 22 May 2019 14:06:18 -0700 (PDT) (envelope-from freebsd) From: "Rodney W. Grimes" Message-Id: <201905222106.x4ML6Idr086719@gndrsh.dnsmgr.net> Subject: Re: svn commit: r347961 - head In-Reply-To: To: John Baldwin Date: Wed, 22 May 2019 14:06:18 -0700 (PDT) CC: Alexey Dokuchaev , Renato Botelho , Brad Davis , 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-Rspamd-Queue-Id: 2E2A36A8A7 X-Spamd-Bar: ------ Authentication-Results: mx1.freebsd.org X-Spamd-Result: default: False [-6.95 / 15.00]; NEURAL_HAM_MEDIUM(-1.00)[-1.000,0]; NEURAL_HAM_SHORT(-0.95)[-0.950,0]; REPLY(-4.00)[]; NEURAL_HAM_LONG(-1.00)[-1.000,0] X-BeenThere: svn-src-head@freebsd.org X-Mailman-Version: 2.1.29 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, 22 May 2019 21:06:28 -0000 > On 5/22/19 5:48 AM, Alexey Dokuchaev wrote: > > On Wed, May 22, 2019 at 09:42:29AM -0300, Renato Botelho wrote: > >> On 18/05/19 16:36, Brad Davis wrote: > >>> New Revision: 347961 > >>> URL: https://svnweb.freebsd.org/changeset/base/347961 > >>> > >>> Log: > >>> Add note to UPDATING for users of mergemaster after the move of > >>> master.passwd and group in r347638. > >>> > >>> Approved by: allanjude (mentor) > >>> > >>> Modified: > >>> head/UPDATING > >>> > >>> +20190517: > >>> + For users of mergemaster, after recent changes to locations of > >>> + master.passwd and group in the source tree mergemaster needs to be > >>> + updated. Before running `mergemaster -p', cd to usr.sbin/mergemaster > >>> + and run `make install'. > >>> + > >> > >> etcupdate is installing group and master.passwd to /lib/libc/gen. It > >> seems wrong. > >> > >> root@x230:/usr/src # etcupdate -p > >> A /lib/libc/gen/group > >> A /lib/libc/gen/master.passwd > > > > *facepalm* > > > > Let me ask again: why these files were moved out of their perfectly > > (and likely the only) reasonable location (etc/) under some cryptic > > libc/gen/ which not only makes little sense per se, but also breaks > > mergemaster(8) and similar tools? > > Because bmake doesn't support .PATH? Oh wait... I spoke long ago when this whole "rip etc/ apart" started, and objected repeatedly and was shot down. I even asked for a "complete plan" and was more or less told this is by design of pkg base. Let me speak to "by design", the BSD source tree is, by design, to reflect the installed location of a file with very rare exception to that design. The .mk system by design is written with this assumption in mind as well. The above artifact is by that design. Attempts to alter such by design behavior has problems, some of which are now, repeatedly showing there ugly head. .PATH could of easily been used, as well as cd ${SRCTOP}/etc && make FOOCONF to implement exactly what was done without ANY file moves. Oh, wait that was pointed out, I belive twice, maybe 3 times, and by some experts on the .mk system as well. sjg (bmake maintainr) and bdrewery (one of our .mk system experts) have both pushed back on these changes. Good luck with supporting this so not by design package system if it requires this to be operable. Also good luck maintaining any of the ${DESTDIR}/etc files as now no one knows how to find them in the source tree. > John Baldwin -- Rod Grimes rgrimes@freebsd.org