New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Multivalued deb init/upstart/default options #487

Merged
merged 1 commit into from Jul 12, 2013

Conversation

Projects
None yet
2 participants
@doismellburning
Contributor

doismellburning commented Jul 11, 2013

I misinterpreted
http://www.debian.org/doc/manuals/maint-guide/dother.en.html#initd - it
shouldn't take foo.init and turn it into /etc/init.d/packagename, it
should become /etc/init.d/foo, so packages can have multiple init
scripts (c.f. nis providing ypbind, yppasswdd etc.)

So, now, these are multivalued options, and they work independently of
the package name, as (I believe) should have been the case originally

Multivalued deb init/upstart/default options
I misinterpreted
http://www.debian.org/doc/manuals/maint-guide/dother.en.html#initd - it
shouldn't take `foo.init` and turn it into `/etc/init.d/packagename`, it
should become `/etc/init.d/foo`, so packages can have multiple init
scripts (c.f. `nis` providing `ypbind`, `yppasswdd` etc.)

So, now, these are multivalued options, and they work independently of
the package name, as (I believe) should have been the case originally
@doismellburning

This comment has been minimized.

Contributor

doismellburning commented Jul 11, 2013

Example old behaviour:

$ fpm -s dir -t deb --deb-init foo -n bar . # -> /etc/init.d/bar

New:

$ fpm -s dir -t deb --deb-init foo -n bar . # -> /etc/init.d/foo

jordansissel added a commit that referenced this pull request Jul 12, 2013

Merge pull request #487 from doismellburning/deb-init-fixes
Multivalued deb init/upstart/default options

@jordansissel jordansissel merged commit c0c9401 into jordansissel:master Jul 12, 2013

prof-milki pushed a commit to prof-milki/xpm that referenced this pull request Dec 18, 2014

jls
Merge pull request jordansissel#487 from doismellburning/deb-init-fixes
Multivalued deb init/upstart/default options

prof-milki pushed a commit to prof-milki/xpm that referenced this pull request Dec 27, 2014

Merge pull request jordansissel#487 from doismellburning/deb-init-fixes
Multivalued deb init/upstart/default options

jordansissel added a commit that referenced this pull request Apr 24, 2015

Merge pull request #487 from doismellburning/deb-init-fixes
Multivalued deb init/upstart/default options

jordansissel added a commit that referenced this pull request Jun 20, 2016

Merge pull request #487 from doismellburning/deb-init-fixes
Multivalued deb init/upstart/default options
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment