Skip to content

Update sysvinit script to fix a few inconsistencies and be more useful/correct downstream#2365

Merged
tianon merged 1 commit intomoby:masterfrom
tianon:sysvinit-consistency
Nov 4, 2013
Merged

Update sysvinit script to fix a few inconsistencies and be more useful/correct downstream#2365
tianon merged 1 commit intomoby:masterfrom
tianon:sysvinit-consistency

Conversation

@tianon
Copy link
Member

@tianon tianon commented Oct 23, 2013

(including sourcing /etc/default/docker instead of /etc/default/lxc)

This is after some discussion with @paultag regarding how to make sure they can use this exact same init script in downstream Debian directly without modification. 👍

@ghost ghost assigned jpetazzo Oct 24, 2013
@tianon
Copy link
Member Author

tianon commented Oct 24, 2013

Ping @jpetazzo

@jpetazzo
Copy link
Contributor

+0

@tianon
Copy link
Member Author

tianon commented Oct 24, 2013

Closing this while I coopt it to adapt our sysvinit be more useful for downstream (aka, for @mzdaniel and @paultag).

@tianon tianon closed this Oct 24, 2013
@tianon tianon reopened this Oct 24, 2013
@tianon
Copy link
Member Author

tianon commented Oct 28, 2013

Just as an update, I'll merge this as soon as @paultag has a moment to give it a proper once-over. 👍

@paultag
Copy link

paultag commented Nov 4, 2013

LGTM! 👍

tianon added a commit that referenced this pull request Nov 4, 2013
Update sysvinit script to fix a few inconsistencies and be more useful/correct downstream
@tianon tianon merged commit 8ad46ef into moby:master Nov 4, 2013
@tianon tianon deleted the sysvinit-consistency branch November 4, 2013 23:34
cpuguy83 pushed a commit to cpuguy83/docker that referenced this pull request May 25, 2021
…handling

controller.loadIPAMDriver: Unwrap error type returned by PluginGetter
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants