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

Reproducible build #189

Merged
merged 2 commits into from May 3, 2023
Merged

Reproducible build #189

merged 2 commits into from May 3, 2023

Conversation

aalmiray
Copy link
Contributor

Updates parent pom to make build reproducible.
As a side effect the code-formatter was also updated hence why so many source files have also been updated.

@aalmiray
Copy link
Contributor Author

Supersedes #177

@hboutemy
Copy link
Contributor

@aalmiray any blocker before merging and releasing?

@aalmiray
Copy link
Contributor Author

@hboutemy waiting for a review from @gunnarmorling as the changes posted are more extensive that I thought

Copy link
Member

@gunnarmorling gunnarmorling left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. One question inline, but it's not a blocker. Feel free to merge one way or the other. Thanks!

description = "Path to the module-info.java descriptor",
converter = PathConverter.class
)
@Parameter(names = "--module-info", required = true, description = "Path to the module-info.java descriptor", converter = PathConverter.class)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this actually more readable? Not that I'm interested in having a formatting style discussion, but could we configure it so it doesn't join wrapped lines? At least there's such an option in the Eclipse formatter. If it's not doable here for some reason, let's go with the proposed change.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I took the formatting configuration from the moditect-org-parent project

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok, can we then perhaps add the "don't join wrapped lines" config there?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As far as I can tell it's already there

<setting id="org.eclipse.jdt.core.formatter.join_wrapped_lines" value="false"/>

@hboutemy
Copy link
Contributor

hboutemy commented May 3, 2023

@aalmiray time for merge and release?

@aalmiray aalmiray merged commit 7432c99 into moditect:main May 3, 2023
1 check passed
@aalmiray aalmiray deleted the reproducible-build branch May 3, 2023 06:39
@aalmiray aalmiray mentioned this pull request May 3, 2023
@hboutemy
Copy link
Contributor

hboutemy commented May 4, 2023

FYI: i just checked RB for this release and all went well!!!
https://github.com/jvm-repo-rebuild/reproducible-central/tree/master/content/org/moditect/moditect

the plugin itself is reproducible
now let's check that plugins users now get a reproducible output from this release also = the most expected benefit :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants