On Wed, Jun 15, 2011 at 03:19:13PM +0200, David Martin wrote: > > On Thu, Jun 9, 2011 at 10:49 PM, Diego Biurrun <diego@xxxxxxxxxx> wrote: > > On Mon, May 30, 2011 at 09:38:29AM +0200, David Martin wrote: > >> > >> >> restart() { > >> >> echo -n "Restarting hipd" > >> >> - killall hipd > >> >> + # if hipd is running, give it 5 seconds to exit gracefully > >> >> + killall hipd && env sleep 5 > >> > > >> > Why "env"? > >> > >> from 'info coreutils 'sleep invocation' ': > >> > Due to shell aliases and built-in `sleep' command, using an > >> > unadorned `sleep' interactively or in a script may get you different > >> > functionality than that described here. Invoke it via `env' (i.e., > >> > `env sleep ...') to avoid interference from the shell. > > > > OpenWrt does not use GNU coreutils, it uses BusyBox, so this is the > > wrong manual to look in. > > Oh, you are right. It won't happen again. You may wish to directly refer to the standard in the future: http://pubs.opengroup.org/onlinepubs/9699919799/ Diego