Skip to content
This repository has been archived by the owner on Jun 10, 2021. It is now read-only.

GPG CMD Signing #79

Closed
wants to merge 1 commit into from
Closed

GPG CMD Signing #79

wants to merge 1 commit into from

Conversation

weickmanna
Copy link

@weickmanna weickmanna commented Feb 15, 2021

The current version of the plugin can only sign if the private key passphrase is stored somewhere on the machine.
Although there is a readConsole mechanism in place, this does not work at least not with current Gradle versions, because the Console object is always null.

The GPG implementation will already prompt for the passphrase though. To enable this feature, I added 2 configs:

  1. whether to use gpg cmd instead of the default Java implementation
  2. whether to read the passphrase from the console (if false, it won't try and instead rely on the gpg implementation to ask for it)

@bmuschko
Copy link
Owner

Although there is a readConsole mechanism in place, this does not work at least not with current Gradle versions, because the Console object is always null.

This is mainly an issue with the Gradle daemon process I believe. If you run Gradle without the daemon, then it should work. I am definitely not making an argument for turning off the daemon here. Gradle should expose a public API for capturing input. At this time, it does only have something internally.

@bmuschko
Copy link
Owner

FYI: You probably shouldn't use this plugin anymore as it uses the "old" Maven plugin which will be removed with Gradle 7.

@weickmanna
Copy link
Author

Is there an alternative to accomplish all the steps needed for publishing to Maven Central?

I tried to turn off the Gradle daemon, but it didn't help - Console object is still null.

All that aside, if people are not encouraged to use this plugin anymore, would you mind merging my PR and pushing out another version? I only added two configs that are off by default, so existing builds are not affected.

@bmuschko
Copy link
Owner

This plugin reached EOL and will add no more new features.

@bmuschko bmuschko closed this Mar 21, 2021
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