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

ST4-testjar not found #300

Closed
mamtananavati opened this issue Apr 6, 2022 · 9 comments · Fixed by #301
Closed

ST4-testjar not found #300

mamtananavati opened this issue Apr 6, 2022 · 9 comments · Fixed by #301

Comments

@mamtananavati
Copy link

mamtananavati commented Apr 6, 2022

When including ST4 V 4.3.2 in project pom it isnt pulling in ST4-testjar
Running mvn dependency:tree throws the below warning message

[WARNING] The POM for org.antlr:ST4:jar:4.3.2 is invalid, transitive dependencies (if any) will not be available, enable debug logging for more details

@parrt
Copy link
Member

parrt commented Apr 6, 2022

hmm...it's part of the testing stuff. I wonder if this can just be excluded from deployments? I couldn't figure out another way to get a test.jar into the mvn ecosystem.

@pzygielo
Copy link
Contributor

transitive dependencies (if any) will not be available

Above of course makes 4.3.2 hardly usable without explicitly defining dependencies. I'd rather avoid that.

@parrt
Copy link
Member

parrt commented Apr 12, 2022

Would love to fix! Could you tell me how to fix the poms? I literally don't understand maven. I can release today. @pzygielo

@parrt
Copy link
Member

parrt commented Apr 12, 2022

I think this is better:

https://github.com/antlr/stringtemplate4/blob/master/pom.xml

@parrt
Copy link
Member

parrt commented Apr 12, 2022

I deployed 4.3.3-SNAPSHOT. Can you use that and see if it works? thanks!

@mamtananavati
Copy link
Author

Thank you , will give it a try.

@pzygielo
Copy link
Contributor

pzygielo commented Apr 12, 2022

I think this is better:

https://github.com/antlr/stringtemplate4/blob/master/pom.xml

I think I have slightly better solution even than that. But I will need few more minutes to fork and prepare PR.

@parrt
Copy link
Member

parrt commented Apr 13, 2022

I have released 4.3.3 :)

@parrt
Copy link
Member

parrt commented Apr 13, 2022

Fixed by #301

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

Successfully merging a pull request may close this issue.

3 participants