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

Version number script does not handle GPG-signed commits #41

Closed
nmaggioni opened this issue May 11, 2018 · 2 comments
Closed

Version number script does not handle GPG-signed commits #41

nmaggioni opened this issue May 11, 2018 · 2 comments

Comments

@nmaggioni
Copy link
Contributor

The version number generation script fails during installation if the latest commit was signed with a GPG key (e.g.: git commit -S).

Strings with quotes inside them are probably imported unescaped and the rest of the code gets wrapped inside them until EOF, causing unparseable output.

@stronnag
Copy link
Owner

I cannot reproduce this. Signed and unsigned commits behave the same. The only git information used by mwp is the output of git log -1 --format="%h" which does not change with signing (at least here).

Can you please provide an example of the output you get from from git log -1 --format="%h" where a commit is signed, and the git log -1 --format="%h" output contains quotes?

@nmaggioni
Copy link
Contributor Author

I tend to forget that I've enabled showing signatures in logs with my config (log.showsignature=true)...
With that option the output of git log -1 --format="%h" includes verbose GPG signature details and breaks C strings.

I've found that git rev-parse --short HEAD does not have this problem, being it a plumbing command.

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

No branches or pull requests

2 participants