intrigeri writes:

> Hi,
>
> Olaf Meeuwissen wrote (31 Oct 2014 04:13:11 GMT) :
>> Actually, I take that partially back.  It turns out to be dependent on
>> what init system you are using.  For the traditional SysV init and
>> Upstart the `export` seems to be needed but for systemd it causes the
>> variable to be ignored, as per the following from my syslog:
>
> Apart of improving the comments, given we want to support both init
> systems in Debian, and to have a single "default" file configure both,
> it seems that we should remove the `export' from
> contrib/init/sysvinit-debian/docker.default, and have the initscript
> export these TMPDIR and http_proxy if they are set, e.g.
> something like the attached (untested) patch.
>
> Thoughts?

If you want to support both, the attached patch looks fine to me.  Note,
I have neither tested it, nor do I know how and where the source files
end up in the binary package.  The logic looks OK to me, that's all.

Hope this helps,
-- 
Olaf Meeuwissen, LPIC-2           FLOSS Engineer -- AVASYS CORPORATION
FSF Associate Member #1962               Help support software freedom
                 http://www.fsf.org/jf?referrer=1962


-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org

Reply via email to