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

Update jcommander to a version which supports Java 8 (release redeploy?) #186

Merged
merged 1 commit into from
Sep 5, 2019

Conversation

fcojfernandez
Copy link
Contributor

In this commit, dependabot updated jcommander from version 1.71 to version 1.75. This version seems to be corrupted somehow.

Local execution

[INFO] Restricted to JDK 1.8 yet com.beust:jcommander:jar:1.75:compile contains com/beust/jcommander/converters/BaseConverter.class targeted to 55.0
[WARNING] Rule 4: org.apache.maven.plugins.enforcer.EnforceBytecodeVersion failed with message:
Found Banned Dependency: com.beust:jcommander:jar:1.75
Use 'mvn dependency:tree' to locate the source of the banned dependencies.
[INFO] ------------------------------------------------------------------------
[INFO] Reactor Summary:
[INFO] 
[INFO] Plugins compatibility tester Aggregator 0.1.1-SNAPSHOT SUCCESS [  1.173 s]
[INFO] Plugins compatibility tester model layer ........... SUCCESS [  4.354 s]
[INFO] Plugins compatibility tester for Google App Engine client side API SUCCESS [  1.863 s]
[INFO] Plugins compatibility tester ....................... SUCCESS [  3.824 s]
[INFO] Plugins compatibility tester CLI ................... FAILURE [  0.522 s]
[INFO] Plugins compatibility tester for Google App Engine 0.1.1-SNAPSHOT SKIPPED
[INFO] ------------------------------------------------------------------------
[INFO] BUILD FAILURE
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 12.010 s
[INFO] Finished at: 2019-09-05T16:51:20+02:00
[INFO] ------------------------------------------------------------------------

Reverting to jcommander-1.71 and updating to jcommander-1.78 work.

Checking with @bmunozm, using an old local maven repository, jcommander-1.75 worked for her, but after removing that version from her repository, she could reproduce the same issue, so our guess is that the jar file has become corrupted somehow at some moment.

@oleg-nenashev @raul-arabaolaza

@oleg-nenashev
Copy link
Member

Are you running PCT with Java 11?

@fcojfernandez
Copy link
Contributor Author

@oleg-nenashev with java 8

@oleg-nenashev
Copy link
Member

There is eve no releases in https://github.com/cbeust/jcommander/releases . I;d guess YOLO.
If you need an emergency release to fix the stuff, please ping me tomorrow

@oleg-nenashev oleg-nenashev changed the title Update jcommander to non-break version Update jcommander to a version which supports Java 8 (release redeploy?) Sep 5, 2019
@oleg-nenashev oleg-nenashev merged commit cb7aca8 into jenkinsci:master Sep 5, 2019
@fcojfernandez fcojfernandez deleted the fix-jcommander branch September 5, 2019 17:10
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.

4 participants