elharo opened a new issue, #404: URL: https://github.com/apache/maven-shared-utils/issues/404
`AbstractStreamHandler.waitUntilDone()` (line 33) is `synchronized` and calls `wait()` in a `while (!isDone())` loop. But `setDone()` (line 47) only sets the `done` flag — it never calls `notifyAll()`. If a subclass calls `setDone()` without also calling `notifyAll()`, any thread blocked in `waitUntilDone()` waits forever. `StreamPumper` works around this by calling `notifyAll()` in its own `finally` block (line 98), but the base class should be correct. Fix: make `setDone()` synchronized and add `this.notifyAll()`. -- 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]
