basil commented on code in PR #201:
URL: https://github.com/apache/maven-release/pull/201#discussion_r1663233072


##########
maven-release-manager/src/main/java/org/apache/maven/shared/release/phase/AbstractRewritePomsPhase.java:
##########
@@ -96,7 +96,7 @@ public abstract class AbstractRewritePomsPhase extends 
AbstractReleasePhase impl
      * Regular expression pattern matching Maven expressions (i.e. references 
to Maven properties).
      * The first group selects the property name the expression refers to.
      */
-    private static final Pattern EXPRESSION_PATTERN = 
Pattern.compile("\\$\\{(.+)\\}");
+    private static final Pattern EXPRESSION_PATTERN = 
Pattern.compile("\\$\\{(.+?)\\}");

Review Comment:
   Causes [MRELEASE-1151](https://issues.apache.org/jira/browse/MRELEASE-1151). 
This caused the weekly Jenkins core release to fail on July 2, 2024. We have 
downgraded Maven Release Plugin to 3.0.1 in order to restore stability to our 
releases.



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

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to