Hello,

> I successfully tested the following patch to the .service file:
> 
> @@ -5,7 +5,8 @@
>  [Service]
>  Type=forking
>  PIDFile=/run/mini_httpd.pid
> -ExecStart=/usr/sbin/mini_httpd -C /etc/mini-httpd.conf
> +EnvironmentFile=-/etc/default/mini-httpd
> +ExecStart=/usr/sbin/mini_httpd -C /etc/mini-httpd.conf $DAEMON_OPTS
> -i /run/mini_httpd.pid
>  
>  [Install]
>  WantedBy=multi-user.target
> 
Thanks for your input and suggestions :)
> The EnvironmentFile uses "=-" to support a non-existant
> /etc/default/mini-httpd. The ExecStart line also adds a -i option to
> make sure
> that neither /etc/mini-httpd.conf nor $DAEMON_OPTS can set -i to
> something that
> is different from the path in PIDFile.
> 
> What do you think?
This is great, I successfully tested your proposed service file changes
and everything appears to work great !
We're currently on mini-httpd/unstable,now 1.30-4;
The next release (1.30-5) will incorporate your fix, closing this bug.

> 
> Thanks!
> 
> cheers, josch
Thanks to you too !
Best,
Alexandru 

Attachment: signature.asc
Description: This is a digitally signed message part

Reply via email to