[Pkg-salt-team] Bug#825578: Support overriding salt binary locations on Debian 7

[Robert Hernandez]

Package: salt
Version: 2015.8.8+ds-1
Severity: minor

I believe I've ran into a small bug in the Debian 7 init.d script. I'm
currently working on a feature for the salt-bootstrap project to enable
installing salt into a virtualenv on Debian (
https://github.com/l2ol33rt/salt-bootstrap/tree/debian_venv_support). This
moves the daemon binary to a location other than the default:
/usr/bin/salt-<daemon>. I've attempted to leverage the
/etc/default/salt-<daemon> file to point to the new binary location. For
example, the file /etc/default/salt-master would contain:
DAEMON=/usr/lib/virtualenv/salt/bin/salt-master

The salt-bootstrap script pulls directly from the debian git repository (
http://anonscm.debian.org/cgit/pkg-salt/salt.git/) for its init.d scripts
since they are not bundled with the salt project itself. However, the
init.d scripts test for the daemon binary's existence right before the
loading of /etc/default/salt-<daemon> which then fails causing the service
to exit immediately.

Attached is my patch for the 4 salt init.d scripts, I hope this would be
something worth including into the salt debian repo.

System Versions:
         dist: debian 7.10
        machine: x86_64
        release: 3.2.0-4-amd64
         system: Linux
        version: debian 7.10

Thanks for your time,

~Rob
https://github.com/l2ol33rt
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.alioth.debian.org/pipermail/pkg-salt-team/attachments/20160527/ea2b4213/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: move_daemon_check.patch
Type: text/x-patch
Size: 2584 bytes
Desc: not available
URL: <http://lists.alioth.debian.org/pipermail/pkg-salt-team/attachments/20160527/ea2b4213/attachment.bin>