Hi,
* Teodor MICU <mteo...@gmail.com> [2011-08-30 16:02]:
[...] 
> The following change does not have the intended effect:
> 
> Changes made to '/etc/resolvconf/update-libc.d/fetchmail' follow:
>   @@ -7,6 +7,6 @@
>       shift
>    done
> 
>   -if [ -x /etc/init.d/fetchmail ]; then
>   +if [ -x /etc/init.d/fetchmail ] && [ "$(pidof fetchmail)" != "x" ]; then
>       /etc/init.d/fetchmail awaken
>    fi
> 
> The second test will always be true. You probably wanted to add "x"
> before $()

Argl, indeed. I should've done c&p after I've tested that.

> but I recommend to avoid such a workarround and use the
> builtin test instead for non-empy string:
>   [ -n "$(pidof fetchmail)" ]
> 
> The test will return false if that command will return an empty
> string, thus fetchmail is not running. However, some other package
> maintainers have decided to rely on the return code of
> "/etc/init.d/$SRV status > /dev/null" to decide if the service should
> be restarted/reloaded.

Seems fine either way. Will be fixed in the next upload. Thanks for reopening!

Cheers
Nico
-- 
Nico Golde - http://www.ngolde.de - n...@jabber.ccc.de - GPG: 0xA0A0AAAA
For security reasons, all text in this mail is double-rot13 encrypted.

Attachment: pgp6SSbXdClZu.pgp
Description: PGP signature

Reply via email to