-
Notifications
You must be signed in to change notification settings - Fork 108
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
bundle a version of gpg for Windows #36
Comments
Honestly, it makes a sense for all three platforms. Bundling a known version of gpg would definitely make some of the more arcane bits of gpg support more reliable. There are many differences between gpg 1 and gpg 2 in terms of command-line arguments, daemon management, pinentry protocols and so on that we need to account for presently, and at some point we just have to throw our hands in the air and give up. A few of the downsides that come to mind:
|
Yeah, but I think it's worth it if we can avoid the hassles of setting up and enabling commit signing for a user.
I wouldn't like to override a config file the user already has defined one, but then I hadn't thought of the potential compatibility issues between gpg 1 and gpg 2 if we focus on supporting gpg 2, for example.
Excellent points. I guess we would already have to do this if Git receives a CVE, but formalizing a process behind this would be 💎 in general.
Probably, especially if we find ourselves building cryptography libraries from source. Will investigate.
Yeah, Git's configuration system is definitely more resilient in this regard. But interoperating with an existing setup is something I'd like to support - particularly if it's the same major version of GPG as what we have in-the-box. |
MinGit does not contain a version of
gpg
to use for managing commit signing.I think we can live without this for macOS and Linux (cc @smashwilson for feels).
The text was updated successfully, but these errors were encountered: