slachiewicz commented on issue #49:
URL: https://github.com/apache/maven-executor/issues/49#issuecomment-5858619211

   Additions from eclipse-tycho/tycho#6383, which moves Tycho's 226 integration 
tests from maven-verifier 1.8.0 to maven-executor 1.0.0 with an in-repo 
Verifier (seventh suite for the sources table; it also needed 1.8.0's splitting 
of `-s <path>` into two arguments, relevant to A1 and T1):
   
   - **E9. Timeout discards grabbed output (#58).** With 
`grabOutputAsString(true)`, a timeout throws a bare `ExecutorException`; the 
output so far is lost. Target 1.1, next to E4.
   - **A9. Diagnostics on timeout.** Before destroying a timed-out build, dump 
the threads of the Maven JVM and its descendants (`jcmd <pid> Thread.print` 
over `ProcessHandle.descendants()`) into the failure. Tycho's first CI run 
stalled a test for 6 hours on macOS with nothing showing where. Target 1.x, 
optional.
   - **R1. ClearlyDefined metadata.** ClearlyDefined has no license data for 
`org.apache.maven.executor:maven-executor:1.0.0`, so Eclipse projects' license 
checks fail and need a manual IP review. Requesting a harvest (and a curation 
if the parent-inherited license is not picked up) after each release unblocks 
them.
   - `ExecutorException` extends `RuntimeException` and can be thrown from 
`close()`; worth a Javadoc line, or `close()` should not throw.
   


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

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to