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

Relocate maven extensions #81

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

headius
Copy link
Collaborator

@headius headius commented Jun 26, 2024

Maven was fixed in apache/maven#94 to only search the target pom's directory structure for .mvn when passing -f to indicate a specific pom file. Unfortunately that breaks logic in ruby-maven intended to temporarily copy the .mvn/extensions.xml file to the current directory to enable Polyglot Ruby poms. This in turn breaks jar-dependencies when running with newer ruby-maven-libs at the root of the filesystem, for reasons described in jruby/jruby#7059:

jruby/jruby#7059 (comment)

This is a possible workaround, by putting the extension config in the same dir as the files we want to treat as Polyglot Ruby pom scripts. It does not fix the underlying problem with ruby-maven.

Maven was fixed in apache/maven#94 to only
search the target pom's directory structure for .mvn when passing
-f to indicate a specific pom file. Unfortunately that breaks
logic in ruby-maven intended to temporarily copy the
.mvn/extensions.xml file to the current directory to enable
Polyglot Ruby poms. This in turn breaks jar-dependencies when
running with newer ruby-maven-libs at the root of the filesystem,
for reasons described in jruby/jruby#7059:

jruby/jruby#7059 (comment)

This is a possible workaround, by putting the extension config
in the same dir as the files we want to treat as Polyglot Ruby pom
scripts. It does not fix the underlying problem with ruby-maven.
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.

1 participant