You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@maven.apache.org by jv...@apache.org on 2007/03/22 17:49:58 UTC
svn commit: r521340 - in /maven/components/trunk:
maven-artifact-manager/src/main/java/org/apache/maven/artifact/manager/
maven-embedder/src/main/java/org/apache/maven/embedder/execution/
Author: jvanzyl
Date: Thu Mar 22 09:49:57 2007
New Revision: 521340
URL: http://svn.apache.org/viewvc?view=rev&rev=521340
Log:
o fixing default perms for deployment
Modified:
maven/components/trunk/maven-artifact-manager/src/main/java/org/apache/maven/artifact/manager/DefaultWagonManager.java
maven/components/trunk/maven-artifact-manager/src/main/java/org/apache/maven/artifact/manager/WagonManager.java
maven/components/trunk/maven-embedder/src/main/java/org/apache/maven/embedder/execution/DefaultMavenExecutionRequestDefaultsPopulator.java
Modified: maven/components/trunk/maven-artifact-manager/src/main/java/org/apache/maven/artifact/manager/DefaultWagonManager.java
URL: http://svn.apache.org/viewvc/maven/components/trunk/maven-artifact-manager/src/main/java/org/apache/maven/artifact/manager/DefaultWagonManager.java?view=diff&rev=521340&r1=521339&r2=521340
==============================================================================
--- maven/components/trunk/maven-artifact-manager/src/main/java/org/apache/maven/artifact/manager/DefaultWagonManager.java (original)
+++ maven/components/trunk/maven-artifact-manager/src/main/java/org/apache/maven/artifact/manager/DefaultWagonManager.java Thu Mar 22 09:49:57 2007
@@ -1,4 +1,4 @@
-package org.apache.maven.artifact.manager;
+ package org.apache.maven.artifact.manager;
/*
* Licensed to the Apache Software Foundation (ASF) under one
@@ -74,7 +74,7 @@
// TODO: proxies, authentication and mirrors are via settings, and should come in via an alternate method - perhaps
// attached to ArtifactRepository before the method is called (so AR would be composed of WR, not inherit it)
private Map proxies = new HashMap();
-
+
private Map authenticationInfoMap = new HashMap();
private Map serverPermissionsMap = new HashMap();
@@ -96,6 +96,8 @@
private Map availableWagons = new HashMap();
+ private RepositoryPermissions defaultRepositoryPermissions;
+
// TODO: this leaks the component in the public api - it is never released back to the container
public Wagon getWagon( Repository repository )
throws UnsupportedProtocolException, WagonConfigurationException
@@ -209,8 +211,10 @@
if ( serverPermissionsMap.containsKey( repository.getId() ) )
{
RepositoryPermissions perms = (RepositoryPermissions) serverPermissionsMap.get( repository.getId() );
+
getLogger().debug(
"adding permissions to wagon connection: " + perms.getFileMode() + " " + perms.getDirectoryMode() );
+
artifactRepository.setPermissions( perms );
}
else
@@ -744,8 +748,8 @@
public void addPermissionInfo( String repositoryId, String filePermissions, String directoryPermissions )
{
-
RepositoryPermissions permissions = new RepositoryPermissions();
+
boolean addPermissions = false;
if ( filePermissions != null )
@@ -871,4 +875,8 @@
serverConfigurationMap.put( repositoryId, xmlConf );
}
+ public void setDefaultRepositoryPermissions( RepositoryPermissions defaultRepositoryPermissions )
+ {
+ this.defaultRepositoryPermissions = defaultRepositoryPermissions;
+ }
}
Modified: maven/components/trunk/maven-artifact-manager/src/main/java/org/apache/maven/artifact/manager/WagonManager.java
URL: http://svn.apache.org/viewvc/maven/components/trunk/maven-artifact-manager/src/main/java/org/apache/maven/artifact/manager/WagonManager.java?view=diff&rev=521340&r1=521339&r2=521340
==============================================================================
--- maven/components/trunk/maven-artifact-manager/src/main/java/org/apache/maven/artifact/manager/WagonManager.java (original)
+++ maven/components/trunk/maven-artifact-manager/src/main/java/org/apache/maven/artifact/manager/WagonManager.java Thu Mar 22 09:49:57 2007
@@ -30,6 +30,7 @@
import org.apache.maven.wagon.events.TransferListener;
import org.apache.maven.wagon.proxy.ProxyInfo;
import org.apache.maven.wagon.repository.Repository;
+import org.apache.maven.wagon.repository.RepositoryPermissions;
import org.codehaus.plexus.PlexusContainer;
import org.codehaus.plexus.util.xml.Xpp3Dom;
@@ -118,5 +119,7 @@
void registerWagons( Collection wagons, PlexusContainer extensionContainer );
- void findAndRegisterWagons( PlexusContainer container );
+ void findAndRegisterWagons( PlexusContainer container );
+
+ void setDefaultRepositoryPermissions( RepositoryPermissions permissions );
}
Modified: maven/components/trunk/maven-embedder/src/main/java/org/apache/maven/embedder/execution/DefaultMavenExecutionRequestDefaultsPopulator.java
URL: http://svn.apache.org/viewvc/maven/components/trunk/maven-embedder/src/main/java/org/apache/maven/embedder/execution/DefaultMavenExecutionRequestDefaultsPopulator.java?view=diff&rev=521340&r1=521339&r2=521340
==============================================================================
--- maven/components/trunk/maven-embedder/src/main/java/org/apache/maven/embedder/execution/DefaultMavenExecutionRequestDefaultsPopulator.java (original)
+++ maven/components/trunk/maven-embedder/src/main/java/org/apache/maven/embedder/execution/DefaultMavenExecutionRequestDefaultsPopulator.java Thu Mar 22 09:49:57 2007
@@ -20,6 +20,7 @@
*/
import org.apache.maven.SettingsConfigurationException;
+import org.apache.maven.wagon.repository.RepositoryPermissions;
import org.apache.maven.artifact.manager.WagonManager;
import org.apache.maven.artifact.repository.ArtifactRepositoryFactory;
import org.apache.maven.artifact.repository.ArtifactRepositoryPolicy;
@@ -179,14 +180,22 @@
wagonManager.addAuthenticationInfo( server.getId(), server.getUsername(), server.getPassword(),
server.getPrivateKey(), server.getPassphrase() );
- wagonManager.addPermissionInfo( server.getId(), server.getFilePermissions(),
- server.getDirectoryPermissions() );
+ wagonManager.addPermissionInfo( server.getId(), server.getFilePermissions(), server.getDirectoryPermissions() );
if ( server.getConfiguration() != null )
{
wagonManager.addConfiguration( server.getId(), (Xpp3Dom) server.getConfiguration() );
}
}
+
+ RepositoryPermissions defaultPermissions = new RepositoryPermissions();
+
+
+ defaultPermissions.setDirectoryMode( "775" );
+
+ defaultPermissions.setFileMode( "664" );
+
+ wagonManager.setDefaultRepositoryPermissions( defaultPermissions );
for ( Iterator i = settings.getMirrors().iterator(); i.hasNext(); )
{