svn commit: r738732 - /maven/components/branches/maven-2.1.x/maven-core/src/main/java/org/apache/maven/DefaultMaven.java

[br...@apache.org]

Author: brett
Date: Thu Jan 29 03:41:20 2009
New Revision: 738732

URL: http://svn.apache.org/viewvc?rev=738732&view=rev
Log:
[MNG-3600] reverted the defaults for deployment. In 2.0.x, they didn't work due to bugs in the Wagon implementation using Jsch and were hit and miss depending on existing permissions in scpexe. These were resolved in 2.1.0-M1, however the use of defaults prevents the user setting it up in the environment itself (eg using umasks), and problems with file modes causing errors when they didn't need to be set. In conjunction, more deployment has moved to HTTP reducing the impact. This is best left at "system default" and the way to configure it documented (as it is on the site)

Modified:
    maven/components/branches/maven-2.1.x/maven-core/src/main/java/org/apache/maven/DefaultMaven.java

Modified: maven/components/branches/maven-2.1.x/maven-core/src/main/java/org/apache/maven/DefaultMaven.java
URL: http://svn.apache.org/viewvc/maven/components/branches/maven-2.1.x/maven-core/src/main/java/org/apache/maven/DefaultMaven.java?rev=738732&r1=738731&r2=738732&view=diff
==============================================================================
--- maven/components/branches/maven-2.1.x/maven-core/src/main/java/org/apache/maven/DefaultMaven.java (original)
+++ maven/components/branches/maven-2.1.x/maven-core/src/main/java/org/apache/maven/DefaultMaven.java Thu Jan 29 03:41:20 2009
@@ -750,14 +750,6 @@
                 }
             }
 
-            RepositoryPermissions defaultPermissions = new RepositoryPermissions();
-
-            defaultPermissions.setDirectoryMode( "775" );
-
-            defaultPermissions.setFileMode( "664" );
-
-            wagonManager.setDefaultRepositoryPermissions( defaultPermissions );
-
             for ( Iterator i = settings.getMirrors().iterator(); i.hasNext(); )
             {
                 Mirror mirror = (Mirror) i.next();