On 7 March 2015 at 05:58, intrigeri <intrig...@debian.org> wrote: > a) Drop the EnvironmentFile= support, on the grounds that it's > incomplete and misleading. Then document in README.Debian that > under systemd: > - /etc/default/docker is ignored (and maybe make that file point > to README.Debian); > - one can customize the daemon startup options and runtime > environment via the standard systemd drop-in override mechanism. > > b) Make it clear, in /etc/default/docker, which options take effect > when running under systemd, and which ones don't. And add a note > about the Environment= directive that allows to set the latter > (http_proxy, TMPDIR); this note could live in /etc/default/docker > and/or in README.Debian. > > Plan (b) seems quite acceptable, and perhaps easier to implement. > Plan (a) feels cleaner, more consistent, and easier to grasp by the > user IMO.
I like (a), but the EnvironmentFile= patch is Paul's baby, so I'm going to leave when it disappears up to him. If he still wants to keep it, I'm happy to work on implementing (b), but I'm not sure how RT would feel about trying to get either fix into Jessie. ♥, - Tianon 4096R / B42F 6819 007F 00F8 8E36 4FD4 036A 9C25 BF35 7DD4 -- To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org
