Skip to content

Conversation

matevarga
Copy link
Contributor

Context

This fixes #548

Contributor Checklist

  • Added relevant integration or unit tests to verify the changes
  • Update the Readme or any other documentation (including relevant Javadoc)
  • Ensured that tests pass locally: mvn clean package
  • Ensured that the code meets the current checkstyle coding style definition: mvn clean verify -Pcheckstyle -Dmaven.test.skip=true -B

@matevarga matevarga changed the title Explicitly hides commit sigs when fetching #548 Explicitly hides commit sigs #548 Mar 19, 2021
@TheSnoozer
Copy link
Collaborator

Wow, nice find! Thank you for creating an issue and the MR to fix it! This command line option seems to have been added with git/git@aa37999 (v2.10.0-rc0) which was released in ~ 2016. Based on that I hope that no one uses an older version of git. Besides that the failing tests seems to be actually green. I still need to understand why the java 16 tests are failing, but that's not relevant to this fix.

Thanks for your contribution!

@TheSnoozer TheSnoozer merged commit 5130b38 into git-commit-id:master Mar 19, 2021
@TheSnoozer TheSnoozer added this to the 4.0.5 milestone Mar 19, 2021
@TheSnoozer TheSnoozer added the bug label Mar 19, 2021
thomasdarimont added a commit to thomasdarimont/embedded-spring-boot-keycloak-server that referenced this pull request Sep 17, 2021
As a Workaround for a build error on jitpack.
```
[ERROR] Failed to execute goal pl.project13.maven:git-commit-id-plugin:4.9.10:revision (git-commit-id-on-validate-revision) on project embedded-keycloak-server-spring-boot-parent: Could not complete Mojo execution... pl.project13.core.NativeGitProvider$NativeCommandException: Git command exited with invalid status [128]: directory: `/home/jitpack/build`, command: `git log -1 --pretty=format:%an --no-show-signature HEAD`, stdout: ``, stderr: `fatal: unrecognized argument: --no-show-signature` -> [Help 1]
```

This was introduced in version 4.0.5 via:
git-commit-id/git-commit-id-maven-plugin#549

Signed-off-by: Thomas Darimont <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Plugin breaks if git config log.showSignature is 'true'

2 participants