Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

improve Reproducible Builds #836

Merged
merged 1 commit into from
Nov 15, 2022
Merged

improve Reproducible Builds #836

merged 1 commit into from
Nov 15, 2022

Conversation

hboutemy
Copy link
Member

@mattrpav
Copy link
Contributor

mattrpav commented May 4, 2022

@hboutemy what is the problem you are experiencing w/ builds not being reproducible?

@mattrpav mattrpav self-assigned this May 4, 2022
@jbonofre jbonofre self-requested a review May 4, 2022 15:34
@hboutemy
Copy link
Member Author

hboutemy commented May 4, 2022

in the reproducible-central page, there are link to diffoscope results that show the different problems found, for example https://github.com/jvm-repo-rebuild/reproducible-central/blob/master/content/org/apache/activemq/activemq-parent-5.17.0.diffoscope

@hboutemy
Copy link
Member Author

hboutemy commented May 5, 2022

to me, once this PR is merged, the only non-reproducible bits are a stupid timestamp, I'll need to chase in Spring later

├── META-INF/spring.handlers
│ @@ -1,3 +1,3 @@
│  #Generated by xbean-spring
│ -#Thu May 05 08:24:00 CEST 2022
│ +#Thu May 05 08:25:21 CEST 2022

@mattrpav
Copy link
Contributor

mattrpav commented Sep 3, 2022

to me, once this PR is merged, the only non-reproducible bits are a stupid timestamp, I'll need to chase in Spring later

├── META-INF/spring.handlers
│ @@ -1,3 +1,3 @@
│  #Generated by xbean-spring
│ -#Thu May 05 08:24:00 CEST 2022
│ +#Thu May 05 08:25:21 CEST 2022

FYI00 You might need to run this one down via xbean project, and not spring.

@mattrpav
Copy link
Contributor

mattrpav commented Sep 3, 2022

@hboutemy would you please rebase against main and squash your commits? I'd think this would be good to get going in 5.18.x

@hboutemy
Copy link
Member Author

hboutemy commented Sep 3, 2022

FYI00 You might need to run this one down via xbean project, and not spring.

yes, https://issues.apache.org/jira/browse/XBEAN-335 created

@hboutemy
Copy link
Member Author

hboutemy commented Sep 3, 2022

@hboutemy would you please rebase against main and squash your commits? I'd think this would be good to get going in 5.18.x

@mattrpav done, with workarounds for plugins injecting timestamps: we should be ok for full Reproducible Builds with 5.18.0

you can easily check reproducibility befaore and after the change by running

mvn clean install -DskipTests -Papache-release -Dgpg.skip -Dmaven.javadoc.skip && mvn clean package artifact:compare -DskipTests -Papache-release -Dgpg.skip -Dmaven.javadoc.skip

and checking artifact:compare (= the last plugin) output

@hboutemy
Copy link
Member Author

@mattrpav PR rebased

@jbonofre jbonofre merged commit cc28773 into apache:main Nov 15, 2022
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants