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

packaging, bloader, github: restore cleanliness of snapd info file; check in GA workflow #11211

Conversation

anonymouse64
Copy link
Member

The Copyright year for the generated bootloader asset files was out of date since the
year is now 2022, so building the snapd deb/snap resulted in this being changed.

The old_seccomp.go file is only needed for trusty, build tags already exclude this file
from being built, so it's unnecessary to do this for xenial et al.

This should produce clean snapd versions again in the info/version.go files.

Also add to the GitHub Actions snap build a check that the info file has a clean version.

Signed-off-by: Ian Johnson <ian.johnson@canonical.com>
This is only needed for trusty, build tags already exclude this file from being
built, so it's unnecessary to do this for xenial et al.

This should produce clean snapd versions again in the info/version.go files.

Signed-off-by: Ian Johnson <ian.johnson@canonical.com>
…eanliness

This is to prevent committing code to PR's which result in snapd internal
version being dirty, like we currently have on master.

Signed-off-by: Ian Johnson <ian.johnson@canonical.com>
@anonymouse64 anonymouse64 added the Simple 😃 A small PR which can be reviewed quickly label Jan 6, 2022
@anonymouse64 anonymouse64 requested a review from mvo5 January 6, 2022 15:29
Copy link
Contributor

@mvo5 mvo5 left a comment

Choose a reason for hiding this comment

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

Thank you so much!

@mvo5 mvo5 added this to the 2.54 milestone Jan 6, 2022
@mvo5 mvo5 added the ⚠ Critical High-priority stuff (e.g. to fix master) label Jan 6, 2022
@anonymouse64 anonymouse64 merged commit 8b7a11a into snapcore:master Jan 6, 2022
@anonymouse64 anonymouse64 deleted the bugfix/alt-debian-dirty-clean-approach branch January 6, 2022 19:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cherry-picked ⚠ Critical High-priority stuff (e.g. to fix master) Simple 😃 A small PR which can be reviewed quickly
Projects
None yet
2 participants