You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@maven.apache.org by GitBox <gi...@apache.org> on 2020/06/30 22:15:08 UTC

[GitHub] [maven-shared-io] elharo opened a new pull request #5: limit maven-compat to test scope

elharo opened a new pull request #5:
URL: https://github.com/apache/maven-shared-io/pull/5


   @hboutemy 


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [maven-shared-io] elharo commented on pull request #5: limit maven-compat to test scope

Posted by GitBox <gi...@apache.org>.
elharo commented on pull request #5:
URL: https://github.com/apache/maven-shared-io/pull/5#issuecomment-652375598


   Some of the tests use it. I'm trying to figure out how to eliminate it from tests here and elsewhere, but at least this will keep this artifact from slipping compat into the transitive dependency tree of everything that depends on shared-io. 
   
   ```
   [ERROR] testShouldNotResolveWhenArtifactResolutionExceptionThrown(org.apache.maven.shared.io.location.ArtifactLocatorStrategyTest)  Time elapsed: 0.001 s  <<< ERROR!
   java.lang.NoClassDefFoundError: org/apache/maven/artifact/factory/ArtifactFactory
   	at org.apache.maven.shared.io.location.ArtifactLocatorStrategyTest.setUp(ArtifactLocatorStrategyTest.java:52)
   
   [ERROR] testShouldFailToResolveSpecWithTwoTokens(org.apache.maven.shared.io.location.ArtifactLocatorStrategyTest)  Time elapsed: 0 s  <<< ERROR!
   java.lang.NoClassDefFoundError: org/apache/maven/artifact/factory/ArtifactFactory
   	at org.apache.maven.shared.io.location.ArtifactLocatorStrategyTest.setUp(ArtifactLocatorStrategyTest.java:52)
   ```


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [maven-shared-io] elharo commented on pull request #5: limit maven-compat to test scope

Posted by GitBox <gi...@apache.org>.
elharo commented on pull request #5:
URL: https://github.com/apache/maven-shared-io/pull/5#issuecomment-652376772


   or not. This one fails CI and needs more work. 


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [maven-shared-io] michael-o commented on pull request #5: limit maven-compat to test scope

Posted by GitBox <gi...@apache.org>.
michael-o commented on pull request #5:
URL: https://github.com/apache/maven-shared-io/pull/5#issuecomment-652329966


   Interesting, why does it require compat at all?


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [maven-shared-io] elharo edited a comment on pull request #5: limit maven-compat to test scope

Posted by GitBox <gi...@apache.org>.
elharo edited a comment on pull request #5:
URL: https://github.com/apache/maven-shared-io/pull/5#issuecomment-652375598


   Some of the tests use it. I'm trying to figure out how to eliminate it from tests here and elsewhere, but at least this PR will keep compat from slipping into the transitive dependency tree of everything that depends on shared-io. 
   
   ```
   [ERROR] testShouldNotResolveWhenArtifactResolutionExceptionThrown(org.apache.maven.shared.io.location.ArtifactLocatorStrategyTest)  Time elapsed: 0.001 s  <<< ERROR!
   java.lang.NoClassDefFoundError: org/apache/maven/artifact/factory/ArtifactFactory
   	at org.apache.maven.shared.io.location.ArtifactLocatorStrategyTest.setUp(ArtifactLocatorStrategyTest.java:52)
   
   [ERROR] testShouldFailToResolveSpecWithTwoTokens(org.apache.maven.shared.io.location.ArtifactLocatorStrategyTest)  Time elapsed: 0 s  <<< ERROR!
   java.lang.NoClassDefFoundError: org/apache/maven/artifact/factory/ArtifactFactory
   	at org.apache.maven.shared.io.location.ArtifactLocatorStrategyTest.setUp(ArtifactLocatorStrategyTest.java:52)
   ```


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [maven-shared-io] elharo closed pull request #5: limit maven-compat to test scope

Posted by GitBox <gi...@apache.org>.
elharo closed pull request #5:
URL: https://github.com/apache/maven-shared-io/pull/5


   


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org