[GitHub] [maven-release] olamy commented on a diff in pull request #181: [MRELEASE-1103] Wrong file path used for master pw file

["olamy (via GitHub)" <gi...@apache.org>]

olamy commented on code in PR #181:
URL: https://github.com/apache/maven-release/pull/181#discussion_r1167797158


##########
maven-release-manager/src/main/java/org/apache/maven/shared/release/util/MavenCrypto.java:
##########
@@ -63,6 +63,9 @@ private MavenCryptoException(String message, Throwable cause) {
     public MavenCrypto(DefaultSecDispatcher secDispatcher, PlexusCipher plexusCipher) {
         this.secDispatcher = secDispatcher;
         this.plexusCipher = plexusCipher;
+
+        // Adjust the default path (def path != maven path)
+        this.secDispatcher.setConfigurationFile("~/.m2/settings-security.xml");

Review Comment:
   Does this work? I'm not sure. Windows and even on unix system?
   It would be better to use something such `System.getProperty("user.home")+"/.m2/settings-security.xml"` 



-- 
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-unsubscribe@maven.apache.org

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