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

Maintenance work #27

Merged
merged 5 commits into from
Jul 16, 2024
Merged

Maintenance work #27

merged 5 commits into from
Jul 16, 2024

Conversation

fmoessbauer
Copy link
Member

Just CI and release creation cleanup. No functional changes to any of the extension parts.

The apt frontent should not be used in non-interactive jobs.
Instead, apt-get shall be used.

Signed-off-by: Felix Moessbauer <[email protected]>
By that, the release tag can be set from the outside when building the
package, solving issues around shallow CI clones and non-git copies.

Signed-off-by: Felix Moessbauer <[email protected]>
With the existing git checkout configuration, the current tag was not
correctly detected by the versioning logic in the Makefile. This led to
incorrect version numbers of the debian package.

Instead, we now explicitly set the RELEASE_TAG to the information we get
from GitHub, when building the release package in CI.

Signed-off-by: Felix Moessbauer <[email protected]>
Signed-off-by: Felix Moessbauer <[email protected]>
The native messaging manifest of chrome and chromium contains two
allowed extension ids: A placeholder for a local (non-signed) install
and the offical extension id (signed version). When installing globally,
the placeholder for the local version needs to be removed. Due to a typo
in the Makefile, this happened only for chrome, but not for chromium.

Reported-by: Dr. Johann Pfefferl <[email protected]>
Signed-off-by: Felix Moessbauer <[email protected]>
@fmoessbauer fmoessbauer self-assigned this Jul 16, 2024
Makefile Show resolved Hide resolved
Copy link
Collaborator

@jan-kiszka jan-kiszka left a comment

Choose a reason for hiding this comment

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

Looks good. How you customize variables in makefiles might also be a matter of taste - up to you if you change or not.

@fmoessbauer fmoessbauer merged commit 2213ec2 into main Jul 16, 2024
3 checks passed
@fmoessbauer fmoessbauer deleted the fm/maint-jobs branch July 16, 2024 17:10
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