Re: [PR] [MNG-8535] Full std stream support for executor [maven]

2025-02-08 Thread via GitHub
cstamas merged PR #2082: URL: https://github.com/apache/maven/pull/2082 -- 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: issues-unsubscr...@maven.apache.

Re: [PR] [MNG-8535] Full std stream support for executor [maven]

2025-02-03 Thread via GitHub
gnodet commented on code in PR #2082: URL: https://github.com/apache/maven/pull/2082#discussion_r1939643819 ## impl/maven-cli/src/main/java/org/apache/maven/cling/invoker/LookupInvoker.java: ## @@ -308,13 +309,13 @@ protected void createTerminal(C context) { if (context

Re: [PR] [MNG-8535] Full std stream support for executor [maven]

2025-02-03 Thread via GitHub
gnodet commented on code in PR #2082: URL: https://github.com/apache/maven/pull/2082#discussion_r1939451296 ## impl/maven-executor/src/main/java/org/apache/maven/cling/executor/forked/ForkedMavenExecutor.java: ## @@ -87,12 +87,16 @@ protected void validate(ExecutorRequest execut

Re: [PR] [MNG-8535] Full std stream support for executor [maven]

2025-02-03 Thread via GitHub
gnodet commented on code in PR #2082: URL: https://github.com/apache/maven/pull/2082#discussion_r1939322955 ## impl/maven-cli/src/main/java/org/apache/maven/cling/invoker/LookupInvoker.java: ## @@ -305,16 +307,29 @@ protected BuildEventListener doDetermineBuildEventListener(C c

Re: [PR] [MNG-8535] Full std stream support for executor [maven]

2025-02-03 Thread via GitHub
gnodet commented on code in PR #2082: URL: https://github.com/apache/maven/pull/2082#discussion_r1939321451 ## impl/maven-cli/src/main/java/org/apache/maven/cling/invoker/LookupInvoker.java: ## @@ -305,16 +307,29 @@ protected BuildEventListener doDetermineBuildEventListener(C c

Re: [PR] [MNG-8535] Full std stream support for executor [maven]

2025-02-03 Thread via GitHub
gnodet commented on code in PR #2082: URL: https://github.com/apache/maven/pull/2082#discussion_r1939321129 ## impl/maven-cli/src/main/java/org/apache/maven/cling/invoker/LookupInvoker.java: ## @@ -305,16 +307,29 @@ protected BuildEventListener doDetermineBuildEventListener(C c

[PR] [MNG-8535] Full std stream support for executor [maven]

2025-02-03 Thread via GitHub
cstamas opened a new pull request, #2082: URL: https://github.com/apache/maven/pull/2082 The std stream support was incomplete in maven-executor --- https://issues.apache.org/jira/browse/MNG-8535 -- This is an automated message from the Apache Git Service. To respond to the m