Github user Tibor17 commented on the pull request:

    https://github.com/apache/maven-surefire/pull/107#issuecomment-164236954
  
    I was about to push your commit and start the release Vote.
    Please run the ITs. I found a problem in IT
    Tests in error: 
      testNgListenerReporter[4: TestNG 
5.14.4](org.apache.maven.surefire.its.CheckTestNgListenerReporterIT)
    `java.lang.NoClassDefFoundError: com/beust/jcommander/ParameterException`
    The exception class is really missing in JAR and `TestNG` and 
`RemoteTestNG` class are referencing the exception.
    
    Do not hesitate to express the Warning and message in 
`MojoExecutionException` in `AbstractSurefireMojo` in detail, e.g. testng pom 
contains system scope artifact `org.testng:guice:2.0`. Other reason like you 
already mentioned in `CheckTestNgListenerReporterIT` comments.
    
    According to the comment in your IT Version 5.14.5 is wrong? Treatment was 
to exclude guice artifact?
    



---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to