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/07/24 17:30:06 UTC

[GitHub] [maven-script-interpreter] slawekjaranowski opened a new pull request #16: [MSHARED-939] Remove Apache Ivy from dependency

slawekjaranowski opened a new pull request #16:
URL: https://github.com/apache/maven-script-interpreter/pull/16


   


----------------------------------------------------------------
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-script-interpreter] slawekjaranowski commented on pull request #16: [MSHARED-939] Remove Apache Ivy from dependency

Posted by GitBox <gi...@apache.org>.
slawekjaranowski commented on pull request #16:
URL: https://github.com/apache/maven-script-interpreter/pull/16#issuecomment-664340012


   I hope - we can, As I wrote in in issue all test in maven-invoker-plugin pass without it.
   
   I think that we can add another dependency to test by adding dependency in scope test and use addTestClassPath at maven-invoker-plugin or add dependency to plugin configuration.
   
   Mixed dependency resolving system may not be good way. 
   So we shouldn't provided such functionality by default for all users.
   
   I'm not sure if it works - we don't any tests for confirmation.


----------------------------------------------------------------
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-script-interpreter] slachiewicz commented on pull request #16: [MSHARED-939] Remove Apache Ivy from dependency

Posted by GitBox <gi...@apache.org>.
slachiewicz commented on pull request #16:
URL: https://github.com/apache/maven-script-interpreter/pull/16#issuecomment-664345761


   @hboutemy what is your opinion?


----------------------------------------------------------------
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-script-interpreter] elharo commented on pull request #16: [MSHARED-939] Remove Apache Ivy from dependency

Posted by GitBox <gi...@apache.org>.
elharo commented on pull request #16:
URL: https://github.com/apache/maven-script-interpreter/pull/16#issuecomment-664331205


   checking ci here: https://builds.apache.org/job/maven-box/job/maven-script-interpreter/job/ty/
   
   I'd love to get rid of this if we can. 


----------------------------------------------------------------
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-script-interpreter] asfgit closed pull request #16: [MSHARED-939] Remove Apache Ivy from dependency

Posted by GitBox <gi...@apache.org>.
asfgit closed pull request #16:
URL: https://github.com/apache/maven-script-interpreter/pull/16


   


----------------------------------------------------------------
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