From 4abc34570ab59e78e6953b1c9a5f010534ec7c99 Mon Sep 17 00:00:00 2001 From: Elliotte Rusty Harold Date: Fri, 28 Nov 2025 07:28:46 -0600 Subject: [PATCH 1/2] sign deprecation --- .../org/apache/maven/shared/release/phase/ScmTagPhase.java | 6 +++++- .../shared/release/phase/IsScmTagParametersEquals.java | 4 ++-- .../apache/maven/shared/release/phase/ScmTagPhaseTest.java | 3 ++- 3 files changed, 9 insertions(+), 4 deletions(-) diff --git a/maven-release-manager/src/main/java/org/apache/maven/shared/release/phase/ScmTagPhase.java b/maven-release-manager/src/main/java/org/apache/maven/shared/release/phase/ScmTagPhase.java index 9ec739f9..6113e7da 100644 --- a/maven-release-manager/src/main/java/org/apache/maven/shared/release/phase/ScmTagPhase.java +++ b/maven-release-manager/src/main/java/org/apache/maven/shared/release/phase/ScmTagPhase.java @@ -26,6 +26,7 @@ import java.util.List; import org.apache.maven.project.MavenProject; +import org.apache.maven.scm.CommandParameters; import org.apache.maven.scm.ScmException; import org.apache.maven.scm.ScmFileSet; import org.apache.maven.scm.ScmTagParameters; @@ -115,7 +116,10 @@ public ReleaseResult execute( ScmTagParameters scmTagParameters = new ScmTagParameters(releaseDescriptor.getScmCommentPrefix() + "copy for tag " + tagName); scmTagParameters.setRemoteTagging(releaseDescriptor.isRemoteTagging()); - scmTagParameters.setSign(releaseDescriptor.isScmSignTags()); + scmTagParameters.setSignOption( + releaseDescriptor.isScmSignTags() + ? CommandParameters.SignOption.FORCE_SIGN + : CommandParameters.SignOption.DEFAULT); scmTagParameters.setScmRevision(releaseDescriptor.getScmReleasedPomRevision()); scmTagParameters.setPinExternals(releaseDescriptor.isPinExternals()); if (getLogger().isDebugEnabled()) { diff --git a/maven-release-manager/src/test/java/org/apache/maven/shared/release/phase/IsScmTagParametersEquals.java b/maven-release-manager/src/test/java/org/apache/maven/shared/release/phase/IsScmTagParametersEquals.java index 267e7493..1904b38c 100644 --- a/maven-release-manager/src/test/java/org/apache/maven/shared/release/phase/IsScmTagParametersEquals.java +++ b/maven-release-manager/src/test/java/org/apache/maven/shared/release/phase/IsScmTagParametersEquals.java @@ -22,7 +22,7 @@ import org.mockito.ArgumentMatcher; /** - * Mockito constraint to compare tags since it has no equals method. + * Mockito constraint to compare tags since ScmTagParameters has no equals method. * * @author olamy */ @@ -39,6 +39,6 @@ public boolean matches(ScmTagParameters argument) { return stp.getMessage().equals(this.scmTagParameters.getMessage()) && stp.isRemoteTagging() == this.scmTagParameters.isRemoteTagging() && stp.isPinExternals() == this.scmTagParameters.isPinExternals() - && stp.isSign() == this.scmTagParameters.isSign(); + && stp.getSignOption() == this.scmTagParameters.getSignOption(); } } diff --git a/maven-release-manager/src/test/java/org/apache/maven/shared/release/phase/ScmTagPhaseTest.java b/maven-release-manager/src/test/java/org/apache/maven/shared/release/phase/ScmTagPhaseTest.java index 4c33ac0e..fec4e3ea 100644 --- a/maven-release-manager/src/test/java/org/apache/maven/shared/release/phase/ScmTagPhaseTest.java +++ b/maven-release-manager/src/test/java/org/apache/maven/shared/release/phase/ScmTagPhaseTest.java @@ -28,6 +28,7 @@ import java.util.List; import org.apache.maven.project.MavenProject; +import org.apache.maven.scm.CommandParameters; import org.apache.maven.scm.ScmException; import org.apache.maven.scm.ScmFile; import org.apache.maven.scm.ScmFileSet; @@ -102,7 +103,7 @@ void testTag() throws Exception { builder.setScmSignTags(true); ScmTagParameters expectedTagParameters = new ScmTagParameters("[my prefix] copy for tag release-label"); - expectedTagParameters.setSign(true); + expectedTagParameters.setSignOption(CommandParameters.SignOption.FORCE_SIGN); ScmFileSet fileSet = new ScmFileSet(rootProject.getFile().getParentFile()); From 9380f010b7604dc9cc2e377b4eaba5a4072762c6 Mon Sep 17 00:00:00 2001 From: Elliotte Rusty Harold Date: Sat, 29 Nov 2025 19:51:50 -0600 Subject: [PATCH 2/2] revert --- .../org/apache/maven/shared/release/phase/ScmTagPhase.java | 6 +----- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/maven-release-manager/src/main/java/org/apache/maven/shared/release/phase/ScmTagPhase.java b/maven-release-manager/src/main/java/org/apache/maven/shared/release/phase/ScmTagPhase.java index 6113e7da..9ec739f9 100644 --- a/maven-release-manager/src/main/java/org/apache/maven/shared/release/phase/ScmTagPhase.java +++ b/maven-release-manager/src/main/java/org/apache/maven/shared/release/phase/ScmTagPhase.java @@ -26,7 +26,6 @@ import java.util.List; import org.apache.maven.project.MavenProject; -import org.apache.maven.scm.CommandParameters; import org.apache.maven.scm.ScmException; import org.apache.maven.scm.ScmFileSet; import org.apache.maven.scm.ScmTagParameters; @@ -116,10 +115,7 @@ public ReleaseResult execute( ScmTagParameters scmTagParameters = new ScmTagParameters(releaseDescriptor.getScmCommentPrefix() + "copy for tag " + tagName); scmTagParameters.setRemoteTagging(releaseDescriptor.isRemoteTagging()); - scmTagParameters.setSignOption( - releaseDescriptor.isScmSignTags() - ? CommandParameters.SignOption.FORCE_SIGN - : CommandParameters.SignOption.DEFAULT); + scmTagParameters.setSign(releaseDescriptor.isScmSignTags()); scmTagParameters.setScmRevision(releaseDescriptor.getScmReleasedPomRevision()); scmTagParameters.setPinExternals(releaseDescriptor.isPinExternals()); if (getLogger().isDebugEnabled()) {