You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@maven.apache.org by jd...@apache.org on 2009/04/29 00:17:38 UTC
svn commit: r769567 - in /maven/components/branches/maven-2.2.x:
maven-artifact-manager/pom.xml
maven-artifact-manager/src/test/java/org/apache/maven/artifact/manager/DefaultWagonManagerTest.java
maven-core/pom.xml pom.xml
Author: jdcasey
Date: Tue Apr 28 22:17:38 2009
New Revision: 769567
URL: http://svn.apache.org/viewvc?rev=769567&view=rev
Log:
[MNG-4147] Use the httpclient-based http wagon instead of http-lightweight in order to avoid long-password Base64 problems in the HTTPURLConnection implementation from Sun.
Modified:
maven/components/branches/maven-2.2.x/maven-artifact-manager/pom.xml
maven/components/branches/maven-2.2.x/maven-artifact-manager/src/test/java/org/apache/maven/artifact/manager/DefaultWagonManagerTest.java
maven/components/branches/maven-2.2.x/maven-core/pom.xml
maven/components/branches/maven-2.2.x/pom.xml
Modified: maven/components/branches/maven-2.2.x/maven-artifact-manager/pom.xml
URL: http://svn.apache.org/viewvc/maven/components/branches/maven-2.2.x/maven-artifact-manager/pom.xml?rev=769567&r1=769566&r2=769567&view=diff
==============================================================================
--- maven/components/branches/maven-2.2.x/maven-artifact-manager/pom.xml (original)
+++ maven/components/branches/maven-2.2.x/maven-artifact-manager/pom.xml Tue Apr 28 22:17:38 2009
@@ -41,7 +41,7 @@
</dependency>
<dependency>
<groupId>org.apache.maven.wagon</groupId>
- <artifactId>wagon-http-lightweight</artifactId>
+ <artifactId>wagon-http</artifactId>
<scope>test</scope>
</dependency>
<dependency>
Modified: maven/components/branches/maven-2.2.x/maven-artifact-manager/src/test/java/org/apache/maven/artifact/manager/DefaultWagonManagerTest.java
URL: http://svn.apache.org/viewvc/maven/components/branches/maven-2.2.x/maven-artifact-manager/src/test/java/org/apache/maven/artifact/manager/DefaultWagonManagerTest.java?rev=769567&r1=769566&r2=769567&view=diff
==============================================================================
--- maven/components/branches/maven-2.2.x/maven-artifact-manager/src/test/java/org/apache/maven/artifact/manager/DefaultWagonManagerTest.java (original)
+++ maven/components/branches/maven-2.2.x/maven-artifact-manager/src/test/java/org/apache/maven/artifact/manager/DefaultWagonManagerTest.java Tue Apr 28 22:17:38 2009
@@ -34,7 +34,7 @@
import org.apache.maven.wagon.TransferFailedException;
import org.apache.maven.wagon.UnsupportedProtocolException;
import org.apache.maven.wagon.Wagon;
-import org.apache.maven.wagon.providers.http.LightweightHttpWagon;
+import org.apache.maven.wagon.providers.http.HttpWagon;
import org.apache.maven.wagon.repository.Repository;
import org.codehaus.plexus.PlexusTestCase;
import org.codehaus.plexus.util.FileUtils;
@@ -420,7 +420,7 @@
private void runThread()
throws Exception
{
- LightweightHttpWagon wagon = (LightweightHttpWagon) wagonManager.getWagon( repository );
+ HttpWagon wagon = (HttpWagon) wagonManager.getWagon( repository );
assertEquals( TEST_USER_AGENT, wagon.getHttpHeaders().getProperty( "User-Agent" ) );
container.release( wagon );
}
Modified: maven/components/branches/maven-2.2.x/maven-core/pom.xml
URL: http://svn.apache.org/viewvc/maven/components/branches/maven-2.2.x/maven-core/pom.xml?rev=769567&r1=769566&r2=769567&view=diff
==============================================================================
--- maven/components/branches/maven-2.2.x/maven-core/pom.xml (original)
+++ maven/components/branches/maven-2.2.x/maven-core/pom.xml Tue Apr 28 22:17:38 2009
@@ -50,7 +50,7 @@
</dependency>
<dependency>
<groupId>org.apache.maven.wagon</groupId>
- <artifactId>wagon-http-lightweight</artifactId>
+ <artifactId>wagon-http</artifactId>
<scope>runtime</scope>
</dependency>
<dependency>
Modified: maven/components/branches/maven-2.2.x/pom.xml
URL: http://svn.apache.org/viewvc/maven/components/branches/maven-2.2.x/pom.xml?rev=769567&r1=769566&r2=769567&view=diff
==============================================================================
--- maven/components/branches/maven-2.2.x/pom.xml (original)
+++ maven/components/branches/maven-2.2.x/pom.xml Tue Apr 28 22:17:38 2009
@@ -409,7 +409,7 @@
<dependency>
<groupId>commons-cli</groupId>
<artifactId>commons-cli</artifactId>
- <version>1.0</version>
+ <version>1.2</version>
</dependency>
<dependency>
<groupId>org.apache.maven.doxia</groupId>
@@ -469,7 +469,7 @@
</dependency>
<dependency>
<groupId>org.apache.maven.wagon</groupId>
- <artifactId>wagon-http-lightweight</artifactId>
+ <artifactId>wagon-http</artifactId>
<version>${wagonVersion}</version>
</dependency>
<dependency>
Re: svn commit: r769567 - in /maven/components/branches/maven-2.2.x: maven-artifact-manager/pom.xml maven-artifact-manager/src/test/java/org/apache/maven/artifact/manager/DefaultWagonManagerTest.java maven-core/pom.xml pom.xml
Posted by Jörg Schaible <jo...@gmx.de>.
Hi Brett,
Brett Porter wrote at Mittwoch, 29. April 2009 01:24:
[snip]
> We don't have many ITs for the CLI options themselves, and I know that
> CLI upgrades have not been all that smooth in the past, so I'm a bit
> overly cautious :)
In fact 1.2 is more compatible to 1.0 than 1.1 ever was ;-)
- Jörg
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
For additional commands, e-mail: dev-help@maven.apache.org
Re: svn commit: r769567 - in /maven/components/branches/maven-2.2.x:
maven-artifact-manager/pom.xml maven-artifact-manager/src/test/java/org/apache/maven/artifact/manager/DefaultWagonManagerTest.java
maven-core/pom.xml pom.xml
Posted by John Casey <ca...@gmail.com>.
Sorry. I'm learning the git-svn workflow. I'll revise.
On Tue, Apr 28, 2009 at 7:24 PM, Brett Porter <br...@apache.org> wrote:
>
> On 29/04/2009, at 8:17 AM, jdcasey@apache.org wrote:
>
>>
>> Modified: maven/components/branches/maven-2.2.x/pom.xml
>> URL:
>> http://svn.apache.org/viewvc/maven/components/branches/maven-2.2.x/pom.xml?rev=769567&r1=769566&r2=769567&view=diff
>> =
>> =
>> =
>> =
>> =
>> =
>> =
>> =
>> ======================================================================
>> --- maven/components/branches/maven-2.2.x/pom.xml (original)
>> +++ maven/components/branches/maven-2.2.x/pom.xml Tue Apr 28 22:17:38 2009
>> @@ -409,7 +409,7 @@
>> <dependency>
>> <groupId>commons-cli</groupId>
>> <artifactId>commons-cli</artifactId>
>> - <version>1.0</version>
>> + <version>1.2</version>
>> </dependency>
>
> This crept in again. Is it really required, or a left over of testing the
> -XOXOXOXOX problem?
>
> We don't have many ITs for the CLI options themselves, and I know that CLI
> upgrades have not been all that smooth in the past, so I'm a bit overly
> cautious :)
>
> - Brett
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
> For additional commands, e-mail: dev-help@maven.apache.org
>
>
--
John Casey
---
Maven Developer (http://maven.apache.org)
---
Blog: http://www.ejlife.net/blogs/buildchimp
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
For additional commands, e-mail: dev-help@maven.apache.org
Re: svn commit: r769567 - in /maven/components/branches/maven-2.2.x: maven-artifact-manager/pom.xml maven-artifact-manager/src/test/java/org/apache/maven/artifact/manager/DefaultWagonManagerTest.java maven-core/pom.xml pom.xml
Posted by Brett Porter <br...@apache.org>.
On 29/04/2009, at 8:17 AM, jdcasey@apache.org wrote:
>
> Modified: maven/components/branches/maven-2.2.x/pom.xml
> URL: http://svn.apache.org/viewvc/maven/components/branches/maven-2.2.x/pom.xml?rev=769567&r1=769566&r2=769567&view=diff
> =
> =
> =
> =
> =
> =
> =
> =
> ======================================================================
> --- maven/components/branches/maven-2.2.x/pom.xml (original)
> +++ maven/components/branches/maven-2.2.x/pom.xml Tue Apr 28
> 22:17:38 2009
> @@ -409,7 +409,7 @@
> <dependency>
> <groupId>commons-cli</groupId>
> <artifactId>commons-cli</artifactId>
> - <version>1.0</version>
> + <version>1.2</version>
> </dependency>
This crept in again. Is it really required, or a left over of testing
the -XOXOXOXOX problem?
We don't have many ITs for the CLI options themselves, and I know that
CLI upgrades have not been all that smooth in the past, so I'm a bit
overly cautious :)
- Brett
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
For additional commands, e-mail: dev-help@maven.apache.org