Skip to content
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

MDEV-34191: Make sure that Debian respects systemd disabled #3268

Open
wants to merge 1 commit into
base: 10.11
Choose a base branch
from

Conversation

illuusio
Copy link
Contributor

@illuusio illuusio commented May 20, 2024

  • The Jira issue number for this PR is: MDEV-34191

Description

Make sure that Debian respects systemd disabled by bumping to deb compat 11 which is available in Debian 10 and Ubuntu 20.04 and it provides better integration with systemd.

Start using dh_installsystemd which is new recommended way in compat 11

Release Notes

Update Debian deb compat to 12

How can this PR be tested?

Basing the PR against the correct MariaDB version

  • This is a bug fix and the PR is based against the earliest maintained branch in which the bug can be reproduced.

PR quality check

  • I checked the CODING_STANDARDS.md file and my PR conforms to this where appropriate.
  • For any trivial modifications to the PR, I am ok with the reviewer making the changes themselves.

@illuusio illuusio force-pushed the 10.11-MDEV-34191-systemd-disable-fix branch 2 times, most recently from fc74665 to 3509b42 Compare May 27, 2024 05:58
debian/rules Outdated
Comment on lines 194 to 188
dh $@ --parallel --with systemd --fail-missing
dh $@ --parallel
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why was --fail-missing removed?

Per https://manpages.debian.org/testing/debhelper/dh_missing.1.en.html

Please note that in compat 11 and earlier without either of these options, dh_missing will silently do nothing. In compat 12, --list-missing is the default. In compat 13 and later, --fail-missing is the default.

From my understanding, this change will make it so that the build process will no longer fail on missing files. Is this intended and safe?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'll add it back as we are using compat 11 not compat 12 which is not building on Debian 10 proberly.

@illuusio illuusio force-pushed the 10.11-MDEV-34191-systemd-disable-fix branch 2 times, most recently from f3bdfc8 to 489d7c5 Compare June 4, 2024 09:31
@illuusio
Copy link
Contributor Author

illuusio commented Jun 4, 2024

Testing should or should not we have this restart thing but noted that we should not as it was.. so revert to that as this test has passed

@illuusio illuusio force-pushed the 10.11-MDEV-34191-systemd-disable-fix branch from 489d7c5 to 332c93e Compare June 4, 2024 10:55
@tonychen2001
Copy link
Contributor

Testing should or should not we have this restart thing but noted that we should not as it was.. so revert to that as this test has passed

I assume your referring to the --restart-after-upgrade option that was removed?

According to https://man7.org/linux/man-pages/man1/dh_installsystemd.1.html, the --restart-after-upgrade option is the default behaviour as of compat 10 for dh_installsystemd. So specifying the option is unnecessary since you are also bumping to compat 11.


The test failures are unrelated. Changes look good but I will leave it to someone with more experience with Debian packaging for a further review.

@illuusio
Copy link
Contributor Author

illuusio commented Jun 5, 2024

Thank you having read documentation more than me! Yes failures are unrelated to this. I'll update the bb branch to get the Deb packages for testing

Make sure that Debian respects systemd disabled by bumping
to deb compat 11 which is available from Debian 10 and Ubuntu 20.04
and it provides better integration with systemd.

Start using dh_installsystemd which is new recommended way
in compat 11
@illuusio illuusio force-pushed the 10.11-MDEV-34191-systemd-disable-fix branch from 332c93e to e891fce Compare June 10, 2024 10:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
2 participants