Skip to content
This repository has been archived by the owner on Apr 26, 2024. It is now read-only.

Update debhelper #13594

Merged
merged 13 commits into from
Aug 26, 2022
Merged

Update debhelper #13594

merged 13 commits into from
Aug 26, 2022

Conversation

behrmann
Copy link
Contributor

Pull Request Checklist

This PR updates the Debian packaging to a more recent version of debhelper (within the limits of current Debian targets, judging from scripts-dev/build_debian_packages.py). It drops stuff that is related to init scripts, since the package doesn't ship any anymore.

This is spun out off #13582.

I left out the changelog entry and the Debian changelog entry so far, because I was unsure what version to add to the latter.

  • Pull request is based on the develop branch
  • Pull request includes a changelog file. The entry should:
    • Be a short description of your change which makes sense to users. "Fixed a bug that prevented receiving messages from other servers." instead of "Moved X method from EventStore to EventWorkerStore.".
    • Use markdown where necessary, mostly for code blocks.
    • End with either a period (.) or an exclamation mark (!).
    • Start with a capital letter.
    • Feel free to credit yourself, by adding a sentence "Contributed by @github_username." or "Contributed by [Your Name]." to the end of the entry.
  • Pull request includes a sign off
  • Code style is correct
    (run the linters)

Don't call dh_installinit anymore, because it has been deprecated, and use
dh_installsystemd instead of dh_systemd_enable for the same reason.

Signed-off-by: Jörg Behrmann <[email protected]>
It was used for reasons of interactions of dh_systemd_start and dh_installinit,
which have both be deprecated

Signed-off-by: Jörg Behrmann <[email protected]>
@behrmann behrmann requested a review from a team as a code owner August 23, 2022 09:19
@behrmann behrmann mentioned this pull request Aug 23, 2022
4 tasks
@richvdh richvdh self-requested a review August 24, 2022 10:45
Copy link
Member

@richvdh richvdh left a comment

Choose a reason for hiding this comment

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

Thanks for bearing with this! All it needs now is an entry in debian/changelog.

@richvdh
Copy link
Member

richvdh commented Aug 25, 2022

(which might mean it'll be easiest to wait for #13593 to land)

@behrmann
Copy link
Contributor Author

Yeah, this is why I was waiting with adding that snippet.

@behrmann
Copy link
Contributor Author

Added the changelog entry.

@richvdh richvdh enabled auto-merge (squash) August 26, 2022 07:42
@richvdh
Copy link
Member

richvdh commented Aug 26, 2022

Thanks for working through all this!

@richvdh richvdh merged commit 998e211 into matrix-org:develop Aug 26, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants