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

Creating Pom leaves ":" in artifactId leading to Maven issues #357

Open
jakestebbins opened this issue Sep 5, 2023 · 3 comments
Open

Creating Pom leaves ":" in artifactId leading to Maven issues #357

jakestebbins opened this issue Sep 5, 2023 · 3 comments

Comments

@jakestebbins
Copy link
Contributor

In DepsModel the packaging is concatenated to the artifactID which causes issues with the maven pom spec "no strange symbols" seems to include colons.

I'm specifically hitting this issue when I'm using mvn dependency:tree e.g. [ERROR] 'dependencies.dependency.artifactId' for com.github.jnr:jffi:jar:native:jar with value 'jffi:jar:native' does not match a valid id pattern.

Is there a way to either concat this differently or possibly use the label in CreatePom.scala? https://maven.apache.org/ref/3.9.4/maven-model/maven.html#class_dependency

@johnynek
Copy link
Collaborator

johnynek commented Sep 5, 2023

I think you are seeing issues with encoding classifiers which may be not correctly supported.

If we research how classifiers should be encoded in XML then a change should be pretty easy (I can help you make a PR).

@jakestebbins
Copy link
Contributor Author

That would be great, thanks so much!

Yeah was thinking of the "< classifier >" tag which looks like that info could already be contained in the MavenArtifactId? (either classifier or packaging)

@jakestebbins
Copy link
Contributor Author

#358 is my attempt at fixing, idk why I couldn't get them to link

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

No branches or pull requests

2 participants