On Thu, 6 Dec 2018, Serge Belyshev wrote:
> Indeed your version is better. (Although as a minor nitpick
> bootclean.sh uses [ and -L in other places, so it will look inconsistent)
I’d recommend replacing at least the -L “while here anyway”.
Using test over [ is personal preference, but makes it
> This is *WRONG*, it will also return 0 if /tmp is not a symlink.
Wow, my bad, sorry for the noise. Should not have really mixed C and
shell code in my head.
> (Also, use test -h instead of test -L for symlinks, cf. the
> GNU autoconf texinfo manual, *Limitations of Builtins::)
>
> What about t
On Wed, 5 Dec 2018, Serge Belyshev wrote:
> clean_tmp() {
> + # Will be created by mount_tmp()
> + [ -L /tmp ] && [ -d /tmp ] || return 0
This is *WRONG*, it will also return 0 if /tmp is not a symlink.
(Also, use test -h instead of test -L for symlinks, cf. the
GNU autoconf texinfo man
Package: initscripts
Version: 2.93-1
In /usr/share/doc/initscripts/README.Debian it is documented that /tmp
could be a symlink to (for example) /run/tmp. This feature was
implemented in version 2.88dsf-13.3 (see
https://salsa.debian.org/debian/sysvinit/commit/3bec14850 ).
But currently it does no
4 matches
Mail list logo