-
Notifications
You must be signed in to change notification settings - Fork 5
release 1.0.6 is not reproducible #59
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
Comments
I guess a bug in plexus archiver, right? |
I guess Maven JAR Plugin, on its calls to archive version-specific classes |
oh nice: yes, this looks like the right location. Thank you for pointing out before I loose time looking at wrong location :) is there a chance that we inject a timestamp to this process? |
notice: this discussion would probably be better done in plexus-archiver... |
@hboutemy I guess we can close this one? |
@olamy sadly I'm not sure at all: this one is tricky given how it is done |
This one needs to be crosslinked with the ticket of plexus-archiver |
Let's keep it open until it is reproducible, otherwise it looks it is fixed (even though the fix is somewhere else) |
The crosslinked issue codehaus-plexus/plexus-archiver#164 was apparently fixed by #205 and closed a few months ago. I think that means this one can be closed as well. On a side note: How about spinning a new release of plexus-languages? The asm version updates would bring Java 19 and Java 20 bytecode support. On another side note: The label "depdendency upgrade" has a typo in it. |
requires maven-jar-plugin 3.3.0 https://issues.apache.org/jira/browse/MJAR-275, apparently done in 26f6fd4 |
Thx for check ✅ |
timestamp of
META-INF/versions/9/module-info.class
is not reproduciblehttps://github.com/jvm-repo-rebuild/reproducible-central/blob/master/content/org/codehaus/plexus/plexus-languages/plexus-languages-1.0.6.diffoscope
The text was updated successfully, but these errors were encountered: