[GitHub] [maven] rmannibucau commented on pull request #408: [MNG-7045] - Drop useless and outdated cdi-api from maven
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
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
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
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
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