-
Notifications
You must be signed in to change notification settings - Fork 3k
Upgrade maven-shade-plugin #44671
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
Upgrade maven-shade-plugin #44671
Conversation
Thanks for your pull request! Your pull request does not follow our editorial rules. Could you have a look?
This message is automatically generated by a bot. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@hboutemy Interesting, thanks, I wasn't aware of this effort and it's greatly appreciated. I'm extremely surprised this is not a lot worse as IIRC our code generation wasn't reproducible-friendly.
This comment has been minimized.
This comment has been minimized.
We have a test failure:
I'm not entirely sure it was caused by this update but I don't remember it failing before so it looks suspicious. |
Given this config in <build>
<plugins>
<plugin>
<groupId>org.apache.maven.plugins</groupId>
<artifactId>maven-shade-plugin</artifactId>
<executions>
<execution>
<phase>package</phase>
<goals>
<goal>shade</goal>
</goals>
<configuration>
<createDependencyReducedPom>true</createDependencyReducedPom>
<createSourcesJar>true</createSourcesJar>
<shadedClassifierName>shaded</shadedClassifierName>
<shadedArtifactAttached>true</shadedArtifactAttached>
<filters>
<filter>
<artifact>com.salesforce.servicelibs:jprotoc</artifact>
<excludes>
<exclude>com/google/protobuf/*</exclude>
<exclude>com/google/protobuf/compiler/*</exclude>
</excludes>
</filter>
</filters>
</configuration>
</execution>
</executions>
</plugin>
</plugins>
</build> |
This comment has been minimized.
This comment has been minimized.
The other tests are not related to this change but I suppose it will require a closer look. |
This update solves our reproducible build issue. We can't go to 3.6.0 due to some weird issues: java.lang.NoClassDefFoundError: com/salesforce/jprotoc/Generator See quarkusio#44671
Closing in favor of #48870 . |
will fix a few remaining Reproducible Builds issues https://github.com/jvm-repo-rebuild/reproducible-central/blob/master/content/io/quarkus/README.md