Date: Sun, 29 Jul 2018 15:48:10 -0700 (PDT) From: Don Lewis <truckman@FreeBSD.org> To: Ian Lepore <ian@freebsd.org> Cc: src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org Subject: Re: svn commit: r336859 - head/sbin/init/rc.d Message-ID: <tkrat.e7ca6dcd996ca389@FreeBSD.org> In-Reply-To: <1532885755.61594.122.camel@freebsd.org> References: <201807290542.w6T5g8rt018138@repo.freebsd.org> <1532873787.61594.107.camel@freebsd.org> <tkrat.ba85a2c19f614432@FreeBSD.org> <1532885755.61594.122.camel@freebsd.org>
index | next in thread | previous in thread | raw e-mail
On 29 Jul, Ian Lepore wrote: > On Sun, 2018-07-29 at 09:59 -0700, Don Lewis wrote: >> On 29 Jul, Ian Lepore wrote: >> > On Sun, 2018-07-29 at 05:42 +0000, Don Lewis wrote: >> >> Author: truckman >> >> Date: Sun Jul 29 05:42:07 2018 >> >> New Revision: 336859 >> >> URL: https://svnweb.freebsd.org/changeset/base/336859 >> >> >> >> Log: >> >> Fix a variable name typo in r336845 that prevented the rc.d scripts >> >> from being installed in the correct directory. >> >> >> >> Resurrect a few rc.d scripts that were prematurely deleted from the >> >> Makefile by r336845. >> >> >> >> Reviewed by: brd >> >> >> >> Modified: >> >> head/sbin/init/rc.d/Makefile >> >> >> >> Modified: head/sbin/init/rc.d/Makefile >> >> ===================================================================== >> >> ========= >> >> --- head/sbin/init/rc.d/Makefile Sun Jul 29 05:14:26 2018 >> >> (r336858) >> >> +++ head/sbin/init/rc.d/Makefile Sun Jul 29 05:42:07 2018 >> >> (r336859) >> >> @@ -2,7 +2,7 @@ >> >> >> >> .include >> >> >> >> -CONFSDIR= /etc/rc.d >> >> +CONFDIR= /etc/rc.d >> >> CONFGROUPS= CONFS >> >> >> > >> > I'm pretty sure CONFSDIR was right here. You are now trapped in a maze >> > of small twisty variable names, all alike. >> >> Without this change, the rc.d scripts get installed directly under >> /var/tmp/temproot/etc/ and mergemaster wants to delete the scripts under >> /etc/rc.d. Answering yes to that mergemaster question results in an >> unbootable system. >> >> I see the following in /usr/share/mk/bsd.confs.mk: >> ${group}DIR?= ${CONFDIR} >> but I don't see ${CONFSDIR} anywhere. >> > > You don't see CONFSDIR because it appears as ${group}DIR and with > CONFGROUPS?= CONFS that turns into CONFSDIR. So now the question is why > doesn't setting CONFSDIR work in this makefile, but it works in others? I don't know, but with undoing my change with this: Index: sbin/init/rc.d/Makefile =================================================================== --- sbin/init/rc.d/Makefile (revision 336859) +++ sbin/init/rc.d/Makefile (working copy) @@ -2,7 +2,7 @@ .include <src.opts.mk> -CONFDIR= /etc/rc.d +CONFSDIR= /etc/rc.d CONFGROUPS= CONFS CONFS= DAEMON \ gives me this in mergemaster: *** Beginning comparison *** Checking /etc/rc.d for stale files *** The following files exist in /etc/rc.d but not in /var/tmp/temproot/etc/rc.d/: accounting amd apm bsnmpd hastd jail local_unbound power_profile sendmail sshd virecover zfs zfsbe zfsd zvol The presence of stale files in this directory can cause the dreaded unpredictable results, and therefore it is highly recommended that you delete them. *** Delete them now? [n]help
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?tkrat.e7ca6dcd996ca389>