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

V4 gpg v1 #96

Closed
wants to merge 3 commits into from
Closed

V4 gpg v1 #96

wants to merge 3 commits into from

Conversation

cstamas
Copy link
Member

@cstamas cstamas commented Mar 2, 2024

This PR is just here to not have it forgotten. This fixes gpg use on Windows that plagued maven-gpg-plugin since quite a long time.

@cstamas cstamas self-assigned this Mar 2, 2024
@cstamas
Copy link
Member Author

cstamas commented Mar 2, 2024

Is used successfully in PR apache/maven-gpg-plugin#65. Whole project had Windows IT failing for quite some long time. Last successful CI run was 2 years ago!

@@ -152,6 +152,12 @@ on:
default: false
type: boolean

install-gpg:
description: Ensure all build images have GnuPG installed
Copy link
Member Author

@cstamas cstamas Mar 2, 2024

Choose a reason for hiding this comment

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

Description is somewhat in disparity with what it actually does:

  • installs gpg4win (on Windows)
  • tests is gpg installed (on all OSes)

Copy link
Member Author

Choose a reason for hiding this comment

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

After last commit this is actually fine: on Win it installs (as others does not need it), and on all OS runs gpg --version.

@cstamas
Copy link
Member Author

cstamas commented Mar 3, 2024

Pushed as v4

@cstamas cstamas closed this Mar 3, 2024
@cstamas cstamas deleted the v4-gpg-v1 branch March 3, 2024 15:47
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.

1 participant