From dev-return-8661-apmail-continuum-dev-archive=continuum.apache.org@continuum.apache.org Tue Mar 02 10:44:29 2010 Return-Path: Delivered-To: apmail-continuum-dev-archive@www.apache.org Received: (qmail 37204 invoked from network); 2 Mar 2010 10:44:29 -0000 Received: from unknown (HELO mail.apache.org) (140.211.11.3) by 140.211.11.9 with SMTP; 2 Mar 2010 10:44:29 -0000 Received: (qmail 86977 invoked by uid 500); 2 Mar 2010 10:44:25 -0000 Delivered-To: apmail-continuum-dev-archive@continuum.apache.org Received: (qmail 86929 invoked by uid 500); 2 Mar 2010 10:44:25 -0000 Mailing-List: contact dev-help@continuum.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@continuum.apache.org Delivered-To: mailing list dev@continuum.apache.org Received: (qmail 86916 invoked by uid 99); 2 Mar 2010 10:44:25 -0000 Received: from nike.apache.org (HELO nike.apache.org) (192.87.106.230) by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 02 Mar 2010 10:44:25 +0000 X-ASF-Spam-Status: No, hits=2.2 required=10.0 tests=HTML_MESSAGE,SPF_PASS X-Spam-Check-By: apache.org Received-SPF: pass (nike.apache.org: domain of odeaching@gmail.com designates 209.85.216.200 as permitted sender) Received: from [209.85.216.200] (HELO mail-px0-f200.google.com) (209.85.216.200) by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 02 Mar 2010 10:44:16 +0000 Received: by pxi38 with SMTP id 38so31796pxi.27 for ; Tue, 02 Mar 2010 02:43:54 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma; h=domainkey-signature:mime-version:sender:received:in-reply-to :references:date:x-google-sender-auth:message-id:subject:from:to :content-type; bh=Em0N26bCFLG4iRWIkQvNHp1EKP85Amq1LVxK7Khmkik=; b=EWpkDtq6SW3tydseEyD5byM9gHUXDAbaM3lqZZroHN28gikxUzGLgyYcfDmZTGt6kn zvsGjQ3k81xHP0cZHhhCAX0XOBMkB7WD+F5Si8mk1SUJ0D1iqDfDLkfZy8hnlOAeU8Tu A8S4YXfSl2Fh+zKCjw158424czqvxxLOgnDWw= DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=mime-version:sender:in-reply-to:references:date :x-google-sender-auth:message-id:subject:from:to:content-type; b=NpZVHPAY4TsYGo5lv02ktgCqVRQxsyKGFDi3GLFw9oytaxmm/bd/SY9X31LK0Eu0DX tdo3Kb2qEIRVZa5ZO0edpMnoXyeLYfg09rtSufS3HDM3v9jyqGQG92bta61t4oxuWLqo QE9B52r6Bbdb2A6GKG6PvSpIDRJ6JaVNeTZko= MIME-Version: 1.0 Sender: odeaching@gmail.com Received: by 10.115.87.4 with SMTP id p4mr3289357wal.202.1267526631749; Tue, 02 Mar 2010 02:43:51 -0800 (PST) In-Reply-To: References: <20100302084817.A812423888A2@eris.apache.org> <8667b1bd1003020054h1d7df53hab4c587e570e6415@mail.gmail.com> Date: Tue, 2 Mar 2010 18:43:51 +0800 X-Google-Sender-Auth: 0fced255ab15b656 Message-ID: <8667b1bd1003020243g354a9894i3679c51fb987f5a3@mail.gmail.com> Subject: Re: svn commit: r917918 - in /continuum/trunk/continuum-release/src: main/java/org/apache/maven/continuum/release/phase/ test/java/org/apache/maven/continuum/release/executors/ test/scm/db/ test/scm/db/revprops/ test/scm/db/revs/ From: Deng Ching To: dev@continuum.apache.org Content-Type: multipart/alternative; boundary=0016e64aef46a0f9840480cf0a76 X-Virus-Checked: Checked by ClamAV on apache.org --0016e64aef46a0f9840480cf0a76 Content-Type: text/plain; charset=ISO-8859-1 Ok, I'll merge it to 1.3.x branch :) Thanks for the quick reply Emm! On Tue, Mar 2, 2010 at 5:25 PM, Emmanuel Venisse wrote: > Sounds good to me. > > Emmanuel > > On Tue, Mar 2, 2010 at 9:54 AM, Deng Ching wrote: > > > Since this is not a new feature and it has a handful of votes, can we > > include this in 1.3.6? > > > > Thanks, > > Deng > > > > On Tue, Mar 2, 2010 at 4:48 PM, wrote: > > > > > Author: oching > > > Date: Tue Mar 2 08:48:16 2010 > > > New Revision: 917918 > > > > > > URL: http://svn.apache.org/viewvc?rev=917918&view=rev > > > Log: > > > [CONTINUUM-1814] release prepare : when generate-reactor-projects can't > > > resolve transitively dependency > > > submitted by Brent Atkinson > > > o use MavenProjectBuilder#build(...) instead of > > > MavenProjectBuilder#buildWithDependencies(..), we only need to get the > > > projects to be added in the reactor and not the transitive deps > > > > > > additional changes made aside from the applied patch: > > > o added test for releasing a project which has a dependency with a > custom > > > packaging type > > > o removed no longer used import statements > > > > > > Added: > > > continuum/trunk/continuum-release/src/test/scm/db/revprops/3 > > > continuum/trunk/continuum-release/src/test/scm/db/revprops/4 > > > continuum/trunk/continuum-release/src/test/scm/db/revprops/5 > > > continuum/trunk/continuum-release/src/test/scm/db/revs/3 (with > > props) > > > continuum/trunk/continuum-release/src/test/scm/db/revs/4 > > > continuum/trunk/continuum-release/src/test/scm/db/revs/5 (with > > props) > > > Modified: > > > > > > > > > continuum/trunk/continuum-release/src/main/java/org/apache/maven/continuum/release/phase/GenerateReactorProjectsPhase.java > > > > > > > > > continuum/trunk/continuum-release/src/test/java/org/apache/maven/continuum/release/executors/ReleaseTaskExecutorTest.java > > > continuum/trunk/continuum-release/src/test/scm/db/current > > > > > > Modified: > > > > > > continuum/trunk/continuum-release/src/main/java/org/apache/maven/continuum/release/phase/GenerateReactorProjectsPhase.java > > > URL: > > > > > > http://svn.apache.org/viewvc/continuum/trunk/continuum-release/src/main/java/org/apache/maven/continuum/release/phase/GenerateReactorProjectsPhase.java?rev=917918&r1=917917&r2=917918&view=diff > > > > > > > > > ============================================================================== > > > --- > > > > > > continuum/trunk/continuum-release/src/main/java/org/apache/maven/continuum/release/phase/GenerateReactorProjectsPhase.java > > > (original) > > > +++ > > > > > > continuum/trunk/continuum-release/src/main/java/org/apache/maven/continuum/release/phase/GenerateReactorProjectsPhase.java > > > Tue Mar 2 08:48:16 2010 > > > @@ -27,8 +27,6 @@ > > > import org.apache.maven.artifact.repository.ArtifactRepository; > > > import org.apache.maven.artifact.repository.DefaultArtifactRepository; > > > import > > > org.apache.maven.artifact.repository.layout.DefaultRepositoryLayout; > > > -import org.apache.maven.artifact.resolver.ArtifactNotFoundException; > > > -import org.apache.maven.artifact.resolver.ArtifactResolutionException; > > > import org.apache.maven.continuum.release.ContinuumReleaseException; > > > import org.apache.maven.profiles.DefaultProfileManager; > > > import org.apache.maven.profiles.ProfileManager; > > > @@ -119,7 +117,7 @@ > > > { > > > ArtifactRepository repository = getLocalRepository( > > > descriptor.getAdditionalArguments() ); > > > > > > - project = projectBuilder.buildWithDependencies( > > > getProjectDescriptorFile( descriptor ), repository, > > > + project = projectBuilder.build( getProjectDescriptorFile( > > > descriptor ), repository, > > > > > > getProfileManager( getSettings() ) ); > > > > > > reactorProjects.add( project ); > > > @@ -130,14 +128,6 @@ > > > { > > > throw new ContinuumReleaseException( "Failed to build > > > project.", e ); > > > } > > > - catch ( ArtifactNotFoundException e ) > > > - { > > > - throw new ContinuumReleaseException( "Failed to build > > > project.", e ); > > > - } > > > - catch ( ArtifactResolutionException e ) > > > - { > > > - throw new ContinuumReleaseException( "Failed to build > > > project.", e ); > > > - } > > > > > > try > > > { > > > @@ -167,7 +157,7 @@ > > > try > > > { > > > MavenProject reactorProject = > > > - projectBuilder.buildWithDependencies( pomFile, > > > repository, getProfileManager( getSettings() ) ); > > > + projectBuilder.build( pomFile, repository, > > > getProfileManager( getSettings() ) ); > > > > > > reactorProjects.add( reactorProject ); > > > > > > @@ -177,14 +167,6 @@ > > > { > > > throw new ContinuumReleaseException( "Failed to build > > > project.", e ); > > > } > > > - catch ( ArtifactNotFoundException e ) > > > - { > > > - throw new ContinuumReleaseException( "Failed to build > > > project.", e ); > > > - } > > > - catch ( ArtifactResolutionException e ) > > > - { > > > - throw new ContinuumReleaseException( "Failed to build > > > project.", e ); > > > - } > > > } > > > } > > > > > > > > > Modified: > > > > > > continuum/trunk/continuum-release/src/test/java/org/apache/maven/continuum/release/executors/ReleaseTaskExecutorTest.java > > > URL: > > > > > > http://svn.apache.org/viewvc/continuum/trunk/continuum-release/src/test/java/org/apache/maven/continuum/release/executors/ReleaseTaskExecutorTest.java?rev=917918&r1=917917&r2=917918&view=diff > > > > > > > > > ============================================================================== > > > --- > > > > > > continuum/trunk/continuum-release/src/test/java/org/apache/maven/continuum/release/executors/ReleaseTaskExecutorTest.java > > > (original) > > > +++ > > > > > > continuum/trunk/continuum-release/src/test/java/org/apache/maven/continuum/release/executors/ReleaseTaskExecutorTest.java > > > Tue Mar 2 08:48:16 2010 > > > @@ -98,7 +98,7 @@ > > > File scmTargetPath = new File( getBasedir(), "target/scm-test" > > > ).getAbsoluteFile(); > > > FileUtils.copyDirectoryStructure( scmPath, scmTargetPath ); > > > } > > > - > > > + > > > public void releaseSimpleProject() > > > throws Exception > > > { > > > @@ -140,6 +140,7 @@ > > > releaseAndRollbackProject(); > > > releaseSimpleProjectWithNextVersion(); > > > releasePerformWithExecutableInDescriptor(); > > > + releaseProjectWithDependencyOfCustomPackagingType(); > > > } > > > > > > public void releaseSimpleProjectWithNextVersion() > > > @@ -300,6 +301,50 @@ > > > fail( "Error in release:perform. Missing executable" ); > > > } > > > } > > > + > > > + // CONTINUUM-1814 > > > + public void releaseProjectWithDependencyOfCustomPackagingType() > > > + throws Exception > > > + { > > > + String scmPath = new File( getBasedir(), > > > "target/scm-test/continuum-1814" ).getAbsolutePath().replace( '\\', '/' > > ); > > > + File workDir = new File( getBasedir(), > > > "target/test-classes/continuum-1814" ); > > > + FileUtils.deleteDirectory( workDir ); > > > + File testDir = new File( getBasedir(), > > > "target/test-classes/test-dir" ); > > > + FileUtils.deleteDirectory( testDir ); > > > + > > > + ContinuumReleaseDescriptor descriptor = new > > > ContinuumReleaseDescriptor(); > > > + descriptor.setInteractive( false ); > > > + descriptor.setScmSourceUrl( "scm:svn:file://localhost/" + > > scmPath > > > + "/trunk" ); > > > + descriptor.setWorkingDirectory( workDir.getAbsolutePath() ); > > > + > > > + ScmRepository repository = getScmRepositorty( > > > descriptor.getScmSourceUrl() ); > > > + ScmFileSet fileSet = new ScmFileSet( workDir ); > > > + scmManager.getProviderByRepository( repository ).checkOut( > > > repository, fileSet, (ScmVersion) null ); > > > + > > > + String pom = FileUtils.fileRead( new File( workDir, "pom.xml" > ) > > ); > > > + assertTrue( "Test dev version", pom.indexOf( > > > "1.6-SNAPSHOT" ) > 0 ); > > > + > > > + doPrepareWithNoError( descriptor ); > > > + > > > + pom = FileUtils.fileRead( new File( workDir, "pom.xml" ) ); > > > + assertTrue( "Test version increment", pom.indexOf( > > > "1.7-SNAPSHOT" ) > 0 ); > > > + > > > + repository = getScmRepositorty( "scm:svn:file://localhost/" + > > > scmPath + "/tags/continuum-1814-1.6" ); > > > + fileSet = new ScmFileSet( testDir ); > > > + scmManager.getProviderByRepository( repository ).checkOut( > > > repository, fileSet, (ScmVersion) null ); > > > + > > > + pom = FileUtils.fileRead( new File( testDir, "pom.xml" ) ); > > > + assertTrue( "Test released version", pom.indexOf( > > > "1.6" ) > 0 ); > > > + > > > + performExec.executeTask( > > > + getPerformTask( "testRelease", descriptor, new File( > > > getBasedir(), "target/test-classes/build-dir" ) ) ); > > > + > > > + ReleaseResult result = (ReleaseResult) > > > releaseManager.getReleaseResults().get( "testRelease" ); > > > + if ( result.getResultCode() != ReleaseResult.SUCCESS ) > > > + { > > > + fail( "Error in release:perform. Release output > follows:\n" > > + > > > result.getOutput() ); > > > + } > > > + } > > > > > > private void doPrepareWithNoError( ReleaseDescriptor descriptor ) > > > throws TaskExecutionException > > > > > > Modified: continuum/trunk/continuum-release/src/test/scm/db/current > > > URL: > > > > > > http://svn.apache.org/viewvc/continuum/trunk/continuum-release/src/test/scm/db/current?rev=917918&r1=917917&r2=917918&view=diff > > > > > > > > > ============================================================================== > > > --- continuum/trunk/continuum-release/src/test/scm/db/current > (original) > > > +++ continuum/trunk/continuum-release/src/test/scm/db/current Tue Mar > 2 > > > 08:48:16 2010 > > > @@ -1 +1 @@ > > > -2 5 1 > > > +5 i 1 > > > > > > Added: continuum/trunk/continuum-release/src/test/scm/db/revprops/3 > > > URL: > > > > > > http://svn.apache.org/viewvc/continuum/trunk/continuum-release/src/test/scm/db/revprops/3?rev=917918&view=auto > > > > > > > > > ============================================================================== > > > --- continuum/trunk/continuum-release/src/test/scm/db/revprops/3 > (added) > > > +++ continuum/trunk/continuum-release/src/test/scm/db/revprops/3 Tue > Mar > > 2 > > > 08:48:16 2010 > > > @@ -0,0 +1,14 @@ > > > +K 10 > > > +svn:author > > > +V 4 > > > +deng > > > +K 8 > > > +svn:date > > > +V 27 > > > +2010-03-01T10:12:34.122774Z > > > +K 7 > > > +svn:log > > > +V 39 > > > +import test project for continuum-1814 > > > + > > > +END > > > > > > Added: continuum/trunk/continuum-release/src/test/scm/db/revprops/4 > > > URL: > > > > > > http://svn.apache.org/viewvc/continuum/trunk/continuum-release/src/test/scm/db/revprops/4?rev=917918&view=auto > > > > > > > > > ============================================================================== > > > --- continuum/trunk/continuum-release/src/test/scm/db/revprops/4 > (added) > > > +++ continuum/trunk/continuum-release/src/test/scm/db/revprops/4 Tue > Mar > > 2 > > > 08:48:16 2010 > > > @@ -0,0 +1,14 @@ > > > +K 10 > > > +svn:author > > > +V 4 > > > +deng > > > +K 8 > > > +svn:date > > > +V 27 > > > +2010-03-01T10:13:02.056710Z > > > +K 7 > > > +svn:log > > > +V 48 > > > +create tags dir for continuum-1814 test project > > > + > > > +END > > > > > > Added: continuum/trunk/continuum-release/src/test/scm/db/revprops/5 > > > URL: > > > > > > http://svn.apache.org/viewvc/continuum/trunk/continuum-release/src/test/scm/db/revprops/5?rev=917918&view=auto > > > > > > > > > ============================================================================== > > > --- continuum/trunk/continuum-release/src/test/scm/db/revprops/5 > (added) > > > +++ continuum/trunk/continuum-release/src/test/scm/db/revprops/5 Tue > Mar > > 2 > > > 08:48:16 2010 > > > @@ -0,0 +1,14 @@ > > > +K 10 > > > +svn:author > > > +V 4 > > > +deng > > > +K 8 > > > +svn:date > > > +V 27 > > > +2010-03-01T10:43:19.644938Z > > > +K 7 > > > +svn:log > > > +V 20 > > > +set elements > > > + > > > +END > > > > > > Added: continuum/trunk/continuum-release/src/test/scm/db/revs/3 > > > URL: > > > > > > http://svn.apache.org/viewvc/continuum/trunk/continuum-release/src/test/scm/db/revs/3?rev=917918&view=auto > > > > > > > > > ============================================================================== > > > Binary file - no diff available. > > > > > > Propchange: continuum/trunk/continuum-release/src/test/scm/db/revs/3 > > > > > > > > > ------------------------------------------------------------------------------ > > > svn:mime-type = application/octet-stream > > > > > > Added: continuum/trunk/continuum-release/src/test/scm/db/revs/4 > > > URL: > > > > > > http://svn.apache.org/viewvc/continuum/trunk/continuum-release/src/test/scm/db/revs/4?rev=917918&view=auto > > > > > > > > > ============================================================================== > > > --- continuum/trunk/continuum-release/src/test/scm/db/revs/4 (added) > > > +++ continuum/trunk/continuum-release/src/test/scm/db/revs/4 Tue Mar 2 > > > 08:48:16 2010 > > > @@ -0,0 +1,52 @@ > > > +id: h.0.r4/0 > > > +type: dir > > > +count: 0 > > > +cpath: /continuum-1814/tags > > > +copyroot: 0 / > > > + > > > +PLAIN > > > +K 4 > > > +tags > > > +V 12 > > > +dir h.0.r4/0 > > > +K 5 > > > +trunk > > > +V 15 > > > +dir 6.0.r3/3840 > > > +END > > > +ENDREP > > > +id: 5.0.r4/150 > > > +type: dir > > > +pred: 5.0.r3/4021 > > > +count: 1 > > > +text: 4 75 62 62 0f542496a7d441a9d7a22151515328c8 > > > +cpath: /continuum-1814 > > > +copyroot: 0 / > > > + > > > +PLAIN > > > +K 14 > > > +continuum-1814 > > > +V 14 > > > +dir 5.0.r4/150 > > > +K 4 > > > +tags > > > +V 14 > > > +dir 2.0.r2/712 > > > +K 5 > > > +trunk > > > +V 14 > > > +dir 3.0.r2/598 > > > +END > > > +ENDREP > > > +id: 0.0.r4/406 > > > +type: dir > > > +pred: 0.0.r3/4263 > > > +count: 4 > > > +text: 4 290 103 103 6b3929e554925dc771dfde26a278dda6 > > > +cpath: / > > > +copyroot: 0 / > > > + > > > +_0.0.t3-1 add false false /continuum-1814/tags > > > + > > > + > > > +406 535 > > > > > > Added: continuum/trunk/continuum-release/src/test/scm/db/revs/5 > > > URL: > > > > > > http://svn.apache.org/viewvc/continuum/trunk/continuum-release/src/test/scm/db/revs/5?rev=917918&view=auto > > > > > > > > > ============================================================================== > > > Binary file - no diff available. > > > > > > Propchange: continuum/trunk/continuum-release/src/test/scm/db/revs/5 > > > > > > > > > ------------------------------------------------------------------------------ > > > svn:mime-type = application/octet-stream > > > > > > > > > > > > --0016e64aef46a0f9840480cf0a76--