Skip to content

[MCOMPILER-397] Use relative paths in jpms.args to make the builds reproducible #193

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

Open
wants to merge 1 commit into
base: maven-compiler-plugin-3.x
Choose a base branch
from

Conversation

ebourg
Copy link
Member

@ebourg ebourg commented Jun 17, 2023

@ebourg
Copy link
Member Author

ebourg commented Jun 17, 2023

This is incomplete, the path separators have to be normalized too.

@slawekjaranowski slawekjaranowski added the waiting-for-feedback Waiting for 90 days until issues or pull request will be closed label Feb 18, 2025
@desruisseaux desruisseaux self-assigned this Mar 15, 2025
desruisseaux added a commit to Geomatys/maven-compiler-plugin that referenced this pull request Mar 15, 2025
@desruisseaux
Copy link
Contributor

Done on a clone of the master branch (not yet merged). The fix in this pull request could not be applied on master, but could be applied on the 3.x branch. Should we redirect the target branch to 3.x, or close as "fixed" (after merge of aa4509b)?

@github-actions github-actions bot removed the waiting-for-feedback Waiting for 90 days until issues or pull request will be closed label Mar 15, 2025
@slawekjaranowski
Copy link
Member

We can change target branch by edit PR ...

@desruisseaux desruisseaux removed their assignment Mar 15, 2025
@desruisseaux desruisseaux changed the base branch from master to maven-compiler-plugin-3.x March 15, 2025 18:48
desruisseaux added a commit to Geomatys/maven-compiler-plugin that referenced this pull request Mar 24, 2025
desruisseaux added a commit to Geomatys/maven-compiler-plugin that referenced this pull request Mar 26, 2025
desruisseaux added a commit to Geomatys/maven-compiler-plugin that referenced this pull request Apr 2, 2025
desruisseaux added a commit to Geomatys/maven-compiler-plugin that referenced this pull request Apr 2, 2025
desruisseaux added a commit to Geomatys/maven-compiler-plugin that referenced this pull request Apr 6, 2025
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.

3 participants