[GitHub] [maven] rmannibucau commented on pull request #408: [MNG-7045] - Drop useless and outdated cdi-api from maven

2020-12-22 Thread GitBox


rmannibucau commented on pull request #408:
URL: https://github.com/apache/maven/pull/408#issuecomment-749438205


   @michael-o I got a lot of issues with the env (some missing maven version 
for ex) then with the git setup I guess (EOL). For ex, using maven 3.6.3 I get:
   
   
   mng6772 NestedImportScopeRepositoryOverride.itInDependency(). FAILURE 
(1.5 s)
   mng6772 NestedImportScopeRepositoryOverride.itInProject() FAILURE 
(1.4 s)
   mng6759 
TransitiveDependencyRepositories.TransitiveDependenciesAccountForRepositoriesListedByDependencyTrailPredecessor()
 FAILURE (1.7 s)
   mng6720 FailFast.ItShouldWaitForConcurrentModulesToFinish().. OK (4.7 s)
   
   I suspect mng6772 can be a wrong maven version rule but mvn6759 should have 
passed with 3.6.3.
   



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:
us...@infra.apache.org




[GitHub] [maven] rmannibucau commented on pull request #408: [MNG-7045] - Drop useless and outdated cdi-api from maven

2020-12-21 Thread GitBox


rmannibucau commented on pull request #408:
URL: https://github.com/apache/maven/pull/408#issuecomment-749390858


   I also have some stability issues with testing suite - but even without 
these PR - but from a first review it looks really unrelated so I would move 
forward with this PR and maybe open a track about IT to mayve use 
docker/testcontainers - to have the expected setup which is not trivial and was 
causing some issues with my local build for ex.



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:
us...@infra.apache.org




[GitHub] [maven] rmannibucau commented on pull request #408: [MNG-7045] - Drop useless and outdated cdi-api from maven

2020-12-20 Thread GitBox


rmannibucau commented on pull request #408:
URL: https://github.com/apache/maven/pull/408#issuecomment-748665179


   @michael-o have to admit i dont see the link there :s, anything could help 
to reproduce?



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:
us...@infra.apache.org




[GitHub] [maven] rmannibucau commented on pull request #408: [MNG-7045] - Drop useless and outdated cdi-api from maven

2020-12-20 Thread GitBox


rmannibucau commented on pull request #408:
URL: https://github.com/apache/maven/pull/408#issuecomment-748652591


   @michael-o don't you see the same without the fix? 



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:
us...@infra.apache.org




[GitHub] [maven] rmannibucau commented on pull request #408: [MNG-7045] - Drop useless and outdated cdi-api from maven

2020-12-14 Thread GitBox


rmannibucau commented on pull request #408:
URL: https://github.com/apache/maven/pull/408#issuecomment-744291605


   @michael-o late ping ;)



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:
us...@infra.apache.org