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

fix: Use /usr/lib/ rather than /lib in packaging code #5970

Merged
merged 1 commit into from
Jan 14, 2025

Conversation

TheRealFalcon
Copy link
Member

Proposed Commit Message

fix: Use /usr/lib/ rather than /lib in packaging code

Support for unmerged /usr was removed from systemd a while ago:
https://lists.freedesktop.org/archives/systemd-devel/2022-September/048352.html

Furthermore, Debian tooling will now throw lintian errors if files
continue to reference /lib. Since it is no longer supported in upstream
systemd, there is no reason for it to be supported in upstream
cloud-init code that uses systemd.

Additional Context

Test Steps

Merge type

  • Squash merge using "Proposed Commit Message"
  • Rebase and merge unique commits. Requires commit messages per-commit each referencing the pull request number (#<PR_NUM>)

Support for unmerged /usr was removed from systemd a while ago:
https://lists.freedesktop.org/archives/systemd-devel/2022-September/048352.html

Furthermore, Debian tooling will now throw lintian errors if files
continue to reference /lib. Since it is no longer supported in upstream
systemd, there is no reason for it to be supported in upstream
cloud-init code that uses systemd.
Copy link
Member

@holmanb holmanb left a comment

Choose a reason for hiding this comment

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

Thanks @TheRealFalcon! I tested this locally and it resolves the lintian error.

@TheRealFalcon TheRealFalcon merged commit 0547349 into canonical:main Jan 14, 2025
22 checks passed
@TheRealFalcon TheRealFalcon deleted the usr-merge branch January 14, 2025 18:11
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.

2 participants