Date: Thu, 16 May 2019 08:05:29 -0700 From: Conrad Meyer <cem@freebsd.org> To: Brad Davis <brd@freebsd.org> Cc: src-committers <src-committers@freebsd.org>, svn-src-all <svn-src-all@freebsd.org>, svn-src-head <svn-src-head@freebsd.org> Subject: Re: svn commit: r347638 - in head: . etc lib/libc/gen Message-ID: <CAG6CVpXDbdwhmx99A3sg25fOQO8BRiHqpvfrhu=jtnexYAFNFQ@mail.gmail.com> In-Reply-To: <201905160109.x4G19DhY080690@repo.freebsd.org> References: <201905160109.x4G19DhY080690@repo.freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
Hi Brad, Can you revert this? It seems to break every config-management tool we have (mergemaster, etcupdate, beinstall ...). Additionally, I don't believe this move has been justified =E2=80=94 the co= ns seem to outweigh the pros, if there are any. I skimmed the differential and it was not immediately obvious what the benefit was. The commit message describes what was changed, but not *why* you thought it was a good idea. Thanks, Conrad On Wed, May 15, 2019 at 6:09 PM Brad Davis <brd@freebsd.org> wrote: > > Author: brd > Date: Thu May 16 01:09:13 2019 > New Revision: 347638 > URL: https://svnweb.freebsd.org/changeset/base/347638 > > Log: > Move master.passwd and group to lib/libc/gen/ > > libc was picked as the destination location for these because of the sy= scalls > that use these files as the lowest level place they are referenced. > > Approved by: will (mentor), rgrimes, manu > Differential Revision: https://reviews.freebsd.org/D16728 > > Added: > head/lib/libc/gen/group > - copied unchanged from r347637, head/etc/group > head/lib/libc/gen/master.passwd > - copied unchanged from r347637, head/etc/master.passwd > Deleted: > head/etc/group > head/etc/master.passwd > Modified: > head/Makefile.inc1 > head/etc/Makefile > head/lib/libc/gen/Makefile.inc > > Modified: head/Makefile.inc1 > =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/Makefile.inc1 Thu May 16 00:53:54 2019 (r347637) > +++ head/Makefile.inc1 Thu May 16 01:09:13 2019 (r347638) > @@ -871,8 +871,8 @@ DB_FROM_SRC=3D yes > .endif > > .if defined(DB_FROM_SRC) > -INSTALLFLAGS+=3D -N ${.CURDIR}/etc > -MTREEFLAGS+=3D -N ${.CURDIR}/etc > +INSTALLFLAGS+=3D -N ${.CURDIR}/lib/libc/gen > +MTREEFLAGS+=3D -N ${.CURDIR}/lib/libc/gen > .endif > _INSTALL_DDIR=3D ${DESTDIR}/${DISTDIR} > INSTALL_DDIR=3D ${_INSTALL_DDIR:S://:/:g:C:/$::} > > Modified: head/etc/Makefile > =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/etc/Makefile Thu May 16 00:53:54 2019 (r347637) > +++ head/etc/Makefile Thu May 16 01:09:13 2019 (r347638) > @@ -15,7 +15,6 @@ SUBDIR+=3Dsendmail > .endif > > BIN1=3D \ > - group \ > login.access \ > rc.bsdextended \ > rc.firewall \ > @@ -65,21 +64,7 @@ distribution: > .endif > cd ${.CURDIR}; \ > ${INSTALL} -o ${BINOWN} -g ${BINGRP} -m 644 \ > - ${BIN1} ${DESTDIR}/etc; \ > - ${INSTALL} -o ${BINOWN} -g ${BINGRP} -m 600 \ > - master.passwd ${DESTDIR}/etc; > - > -.if ${MK_TCSH} =3D=3D "no" > - sed -i "" -e 's;/bin/csh;/bin/sh;' ${DESTDIR}/etc/master.passwd > -.endif > - pwd_mkdb -i -p -d ${DESTDIR}/etc ${DESTDIR}/etc/master.passwd > -.if defined(NO_ROOT) > - ( \ > - echo "./etc/passwd type=3Dfile mode=3D0644 uname=3Droot g= name=3Dwheel"; \ > - echo "./etc/pwd.db type=3Dfile mode=3D0644 uname=3Droot g= name=3Dwheel"; \ > - echo "./etc/spwd.db type=3Dfile mode=3D0600 uname=3Droot = gname=3Dwheel"; \ > - ) | ${METALOG.add} > -.endif > + ${BIN1} ${DESTDIR}/etc > ${_+_}cd ${.CURDIR}/gss; ${MAKE} install > ${_+_}cd ${.CURDIR}/mtree; ${MAKE} install > ${_+_}cd ${SRCTOP}/share/termcap; ${MAKE} etc-termcap > > Modified: head/lib/libc/gen/Makefile.inc > =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/lib/libc/gen/Makefile.inc Thu May 16 00:53:54 2019 (= r347637) > +++ head/lib/libc/gen/Makefile.inc Thu May 16 01:09:13 2019 (= r347638) > @@ -4,7 +4,8 @@ > # machine-independent gen sources > .PATH: ${LIBC_SRCTOP}/${LIBC_ARCH}/gen ${LIBC_SRCTOP}/gen > > -CONFS=3D shells > +CONFS+=3D group master.passwd shells > +CONFSMODE_master.passwd=3D 600 > > SRCS+=3D __getosreldate.c \ > __pthread_mutex_init_calloc_cb_stub.c \ > @@ -543,3 +544,16 @@ MLINKS+=3Dvis.3 nvis.3 \ > vis.3 svis.3 > > MLINKS+=3Dwordexp.3 wordfree.3 > + > +afterinstallconfig: > +.if ${MK_TCSH} =3D=3D "no" > + sed -i "" -e 's;/bin/csh;/bin/sh;' ${DESTDIR}/etc/master.passwd > +.endif > + pwd_mkdb -i -p -d ${DESTDIR}/etc ${DESTDIR}/etc/master.passwd > +.if defined(NO_ROOT) && defined(METALOG) > + ( \ > + echo "./etc/pwd.db type=3Dfile mode=3D0644 uname=3Droot g= name=3Dwheel"; \ > + echo "./etc/spwd.db type=3Dfile mode=3D0600 uname=3Droot = gname=3Dwheel"; \ > + echo "./etc/passwd type=3Dfile mode=3D0644 uname=3Droot g= name=3Dwheel"; \ > + ) | cat -l >> ${METALOG} > +.endif > > Copied: head/lib/libc/gen/group (from r347637, head/etc/group) > =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 > --- /dev/null 00:00:00 1970 (empty, because file is newly added) > +++ head/lib/libc/gen/group Thu May 16 01:09:13 2019 (r347638,= copy of r347637, head/etc/group) > @@ -0,0 +1,36 @@ > +# $FreeBSD$ > +# > +wheel:*:0:root > +daemon:*:1: > +kmem:*:2: > +sys:*:3: > +tty:*:4: > +operator:*:5:root > +mail:*:6: > +bin:*:7: > +news:*:8: > +man:*:9: > +games:*:13: > +ftp:*:14: > +staff:*:20: > +sshd:*:22: > +smmsp:*:25: > +mailnull:*:26: > +guest:*:31: > +video:*:44: > +bind:*:53: > +unbound:*:59: > +proxy:*:62: > +authpf:*:63: > +_pflogd:*:64: > +_dhcp:*:65: > +uucp:*:66: > +dialer:*:68: > +network:*:69: > +audit:*:77: > +www:*:80: > +ntpd:*:123: > +_ypldap:*:160: > +hast:*:845: > +nogroup:*:65533: > +nobody:*:65534: > > Copied: head/lib/libc/gen/master.passwd (from r347637, head/etc/master.pa= sswd) > =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 > --- /dev/null 00:00:00 1970 (empty, because file is newly added) > +++ head/lib/libc/gen/master.passwd Thu May 16 01:09:13 2019 (= r347638, copy of r347637, head/etc/master.passwd) > @@ -0,0 +1,28 @@ > +# $FreeBSD$ > +# > +root::0:0::0:0:Charlie &:/root:/bin/csh > +toor:*:0:0::0:0:Bourne-again Superuser:/root: > +daemon:*:1:1::0:0:Owner of many system processes:/root:/usr/sbin/nologin > +operator:*:2:5::0:0:System &:/:/usr/sbin/nologin > +bin:*:3:7::0:0:Binaries Commands and Source:/:/usr/sbin/nologin > +tty:*:4:65533::0:0:Tty Sandbox:/:/usr/sbin/nologin > +kmem:*:5:65533::0:0:KMem Sandbox:/:/usr/sbin/nologin > +games:*:7:13::0:0:Games pseudo-user:/:/usr/sbin/nologin > +news:*:8:8::0:0:News Subsystem:/:/usr/sbin/nologin > +man:*:9:9::0:0:Mister Man Pages:/usr/share/man:/usr/sbin/nologin > +sshd:*:22:22::0:0:Secure Shell Daemon:/var/empty:/usr/sbin/nologin > +smmsp:*:25:25::0:0:Sendmail Submission User:/var/spool/clientmqueue:/usr= /sbin/nologin > +mailnull:*:26:26::0:0:Sendmail Default User:/var/spool/mqueue:/usr/sbin/= nologin > +bind:*:53:53::0:0:Bind Sandbox:/:/usr/sbin/nologin > +unbound:*:59:59::0:0:Unbound DNS Resolver:/var/unbound:/usr/sbin/nologin > +proxy:*:62:62::0:0:Packet Filter pseudo-user:/nonexistent:/usr/sbin/nolo= gin > +_pflogd:*:64:64::0:0:pflogd privsep user:/var/empty:/usr/sbin/nologin > +_dhcp:*:65:65::0:0:dhcp programs:/var/empty:/usr/sbin/nologin > +uucp:*:66:66::0:0:UUCP pseudo-user:/var/spool/uucppublic:/usr/local/libe= xec/uucp/uucico > +pop:*:68:6::0:0:Post Office Owner:/nonexistent:/usr/sbin/nologin > +auditdistd:*:78:77::0:0:Auditdistd unprivileged user:/var/empty:/usr/sbi= n/nologin > +www:*:80:80::0:0:World Wide Web Owner:/nonexistent:/usr/sbin/nologin > +ntpd:*:123:123::0:0:NTP Daemon:/var/db/ntp:/usr/sbin/nologin > +_ypldap:*:160:160::0:0:YP LDAP unprivileged user:/var/empty:/usr/sbin/no= login > +hast:*:845:845::0:0:HAST unprivileged user:/var/empty:/usr/sbin/nologin > +nobody:*:65534:65534::0:0:Unprivileged user:/nonexistent:/usr/sbin/nolog= in >
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CAG6CVpXDbdwhmx99A3sg25fOQO8BRiHqpvfrhu=jtnexYAFNFQ>