[arch-projects] [PATCH] initscripts: load of sysctl files at startup

[Dave Reisner]

On Sun, Aug 28, 2011 at 11:44:17PM +0200, Earendil wrote:
> Hi 
> 
> Just a little patch to correct the load of the sysctl configuration file
> at startup.

And what needs to be corrected? Ignoring the additional directory you're
pulling from), your patch exhibits no change in behavior for loading the
default /etc/sysctl.conf.

> I have also added the capability to stip th sysctl.conf files in many
> files in the sysctl.d directory.
> This idea was inspired by Debian architecture.
> 

Sure, systemd uses this as well. Do we have any cases where distributing
a sysctl file with a package is needed? I'm really not sure this is
something that's wanted/needed in Arch.

d

> 
> -- 
> keys.gnupg.net: 0xEBCBE59D
> 
> "J'ai toujours rêvé d'un ordinateur qui soit aussi facile à utiliser 
>  qu'un téléphone. Mon rêve s'est réalisé : je ne sais plus comment 
>  utiliser mon téléphone."
> 
>  Bjarne Stroustrup
> 
> ------------ PATCH -----------------------
> --- rc.multi.orig   2011-07-28 14:21:07.000000000 +0200
> +++ rc.multi    2011-08-12 23:48:23.914340475 +0200
> @@ -9,7 +9,10 @@
>  run_hook multi_start
> 
>   # Load sysctl variables if sysctl.conf is present
>   -[[ -r /etc/sysctl.conf ]] && sysctl -q -p &>/dev/null
>   +for file in /etc/sysctl.conf /etc/sysctl.d/*.conf ; do
>   +    [[ -r "${file}" ]] && sysctl -q -p "${file}" &> /dev/null
>   +done
>   +stat_done
> 
>    # Start daemons
>     for daemon in "${DAEMONS[@]}"; do
> 
> ------------------------------------------

> --- rc.multi.orig	2011-07-28 14:21:07.000000000 +0200
> +++ rc.multi	2011-08-12 23:48:23.914340475 +0200
> @@ -9,7 +9,10 @@
>  run_hook multi_start
>  
>  # Load sysctl variables if sysctl.conf is present
> -[[ -r /etc/sysctl.conf ]] && sysctl -q -p &>/dev/null
> +for file in /etc/sysctl.conf /etc/sysctl.d/*.conf ; do
> +    [[ -r "${file}" ]] && sysctl -q -p "${file}" &> /dev/null
> +done
> +stat_done
>  
>  # Start daemons
>  for daemon in "${DAEMONS[@]}"; do