Skip site navigation (1)Skip section navigation (2)
Date:      Sun, 29 Jul 2018 08:16:27 -0600
From:      Ian Lepore <ian@freebsd.org>
To:        Don Lewis <truckman@FreeBSD.org>, 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:  <1532873787.61594.107.camel@freebsd.org>
In-Reply-To: <201807290542.w6T5g8rt018138@repo.freebsd.org>
References:  <201807290542.w6T5g8rt018138@repo.freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
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 <src.opts.mk>
>  
> -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.

CONFGROUPS=CONFS is the default set by bsd.conf.mk, and it's the only
one that needs CONF to be singular. Perhaps if we establish the idiom
of not re-specifying the default value in all the individual makefiles,
that'll leave everything as CONFS* and it'll be a bit less confusing?

-- Ian


>  CONFS=	DAEMON \
> @@ -75,6 +75,7 @@ CONFS=	DAEMON \
>  	netif \
>  	netoptions \
>  	netwait \
> +	newsyslog \
>  	nfsclient \
>  	nfscbd \
>  	nfsd \
> @@ -112,6 +113,8 @@ CONFS=	DAEMON \
>  	stf \
>  	swap \
>  	swaplate \
> +	sysctl \
> +	syslogd \
>  	tmp \
>  	${_ubthidhci} \
>  	ugidfw \
> 



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?1532873787.61594.107.camel>