-
Notifications
You must be signed in to change notification settings - Fork 9
Fix rabbitmq/rabbitmq-packaging#49 #50
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
Fix rabbitmq/rabbitmq-packaging#49 #50
Conversation
Fixes unwanted service startup after installation when the systemd service unit is disabled by a systemd preset. With compat 12 `--skip-systemd-native` is now passed to invoke-rc.d. Removed `--with systemd` since it was removed since compat 11 - see https://manpages.debian.org/bookworm/debhelper/debhelper-compat-upgrade-checklist.7.en.html Tested by building the package on Debian 12 with debhelper version 13.11.4. Should theoretically work with debhelper 11.4 or later, according to debhelper changelog at https://launchpad.net/debian/+source/debhelper/11.4
a9a975a to
c601a5b
Compare
|
I'm testing this out in my own Debian 12 env. Thanks. |
lukebakken
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This worked perfectly in my environment 👍
|
@kaistierl thank you for contributing I suspect that setting was added a while ago when systemd was a fairly new development in Debian. Can you think of any risks around adopting this change? |
I guess #49 mostly answers my questions. |
|
@Mergifyio backport v3.13.x |
✅ Backports have been created
|
Fixes #49
Fixes unwanted service startup after installation when the systemd service unit is disabled by a systemd preset. With compat 12
--skip-systemd-nativeis now passed to update-rc.d.Removed
--with systemdsince it was removed since compat 11 - see https://manpages.debian.org/bookworm/debhelper/debhelper-compat-upgrade-checklist.7.en.htmlTested by building the package on Debian 12 with debhelper version 13.11.4. Should theoretically work with debhelper 11.4 or later, according to debhelper changelog at https://launchpad.net/debian/+source/debhelper/11.4