From mboxrd@z Thu Jan 1 00:00:00 1970 From: Arnout Vandecappelle Date: Fri, 14 Nov 2014 22:33:44 +0100 Subject: [Buildroot] [PATCH 01/10] skeleton/S40network: tweak for debian ifupdown In-Reply-To: <1415969096-3032-2-git-send-email-gustavo@zacarias.com.ar> References: <1415969096-3032-1-git-send-email-gustavo@zacarias.com.ar> <1415969096-3032-2-git-send-email-gustavo@zacarias.com.ar> Message-ID: <54667538.6050700@mind.be> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: buildroot@busybox.net On 14/11/14 13:44, Gustavo Zacarias wrote: > Make the S40network script create the /run/network directory for the > debian variant of ifupdown which uses it as a lock directory. > > Signed-off-by: Gustavo Zacarias > --- > system/skeleton/etc/init.d/S40network | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/system/skeleton/etc/init.d/S40network > b/system/skeleton/etc/init.d/S40network > index e69e613..4317d14 100755 > --- a/system/skeleton/etc/init.d/S40network > +++ b/system/skeleton/etc/init.d/S40network > @@ -3,6 +3,9 @@ > # Start the network.... > # > > +# Debian ifupdown needs the /run/network lock directory > +[ -h /sbin/ifup ] || mkdir -p /run/network I think checking for a symlink is a pretty fragile way for distinguishing between Debian and busybox ifupdown... Why not just create the directory unconditionally? People who care about the 4K tmpfs wastage will probably want to get rid of this script entirely... Regards, Arnout > + > case "$1" in > start) > echo "Starting network..." > -- Arnout Vandecappelle arnout at mind be Senior Embedded Software Architect +32-16-286500 Essensium/Mind http://www.mind.be G.Geenslaan 9, 3001 Leuven, Belgium BE 872 984 063 RPR Leuven LinkedIn profile: http://www.linkedin.com/in/arnoutvandecappelle GPG fingerprint: 7CB5 E4CC 6C2E EFD4 6E3D A754 F963 ECAB 2450 2F1F