Skip to content

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

Closed
hboutemy opened this issue Jan 12, 2021 · 14 comments
Closed

release 1.0.6 is not reproducible #59

hboutemy opened this issue Jan 12, 2021 · 14 comments
Labels
bug dependencies Pull requests that update a dependency file

Comments

@hboutemy
Copy link
Member

timestamp of META-INF/versions/9/module-info.class is not reproducible

https://github.com/jvm-repo-rebuild/reproducible-central/blob/master/content/org/codehaus/plexus/plexus-languages/plexus-languages-1.0.6.diffoscope

@hboutemy hboutemy added the bug label Jan 12, 2021
@rfscholte
Copy link
Member

I guess a bug in plexus archiver, right?

@hboutemy
Copy link
Member Author

I guess Maven JAR Plugin, on its calls to archive version-specific classes
but I need to investigate to be sure

@hboutemy
Copy link
Member Author

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?
IIUC https://github.com/codehaus-plexus/plexus-archiver/blob/master/src/main/java/org/codehaus/plexus/archiver/jar/JarToolModularJarArchiver.java#L162 , it uses jar in-process CLI tool, and I don't see any parameter to define timestamp: would that mean that the only option is to modify timestamp on disk? (I don't like that idea...)

@hboutemy
Copy link
Member Author

notice: this discussion would probably be better done in plexus-archiver...
need to create an issue: I'll do it tonight if you don't beat me at it

@olamy
Copy link
Member

olamy commented Jul 11, 2021

@hboutemy I guess we can close this one?

@hboutemy
Copy link
Member Author

@olamy sadly I'm not sure at all: this one is tricky given how it is done

@rfscholte
Copy link
Member

This one needs to be crosslinked with the ticket of plexus-archiver

@slachiewicz
Copy link
Member

@rfscholte
Copy link
Member

Let's keep it open until it is reproducible, otherwise it looks it is fixed (even though the fix is somewhere else)

@andpab
Copy link

andpab commented Dec 31, 2022

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.

@slachiewicz slachiewicz added dependencies Pull requests that update a dependency file and removed depdendency upgrade labels Dec 31, 2022
@hboutemy
Copy link
Member Author

hboutemy commented Jan 1, 2023

requires maven-jar-plugin 3.3.0 https://issues.apache.org/jira/browse/MJAR-275, apparently done in 26f6fd4

@slachiewicz
Copy link
Member

Thx for check ✅

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug dependencies Pull requests that update a dependency file
Projects
None yet
Development

No branches or pull requests

6 participants