search

apache / maven-mvnd

Apache Maven Daemon
https://maven.apache.org/
Apache License 2.0
2.85k stars 205 forks source link

[1.x] mixup in case of DISPLAY messages type #1037

Closed cstamas closed 2 weeks ago

cstamas commented 2 weeks ago

ProjectEvent message mixup fixed.

Fixes #1036

cstamas commented 2 weeks ago

Locally verified: mvnd 1.0.0 does this https://gist.github.com/cstamas/7090de741c6c7df393198d6b7962dc4e This PR fixed mvnd does successfully executes command from the gist above.

The message that fails with CCEx:

[standalone-pom] Choose archetype:
cstamas commented 2 weeks ago

Hm, so there is some mixup, as: https://github.com/apache/maven-mvnd/blob/6150ae34c842d909b3ca1a0e8034dc82ca4e2b9e/common/src/main/java/org/mvndaemon/mvnd/common/Message.java#L1115-L1118

but there is this as well:

https://github.com/apache/maven-mvnd/blob/mvnd-1.x/common/src/main/java/org/mvndaemon/mvnd/common/Message.java#L1171-L1173