I have reverted the patch in branch-2. Colin we may need a separate branch-2 patch instead of merging the patch.
On Thu, Oct 18, 2012 at 7:19 PM, Suresh Srinivas <sur...@hortonworks.com>wrote: > I meant revert the patch and merge the right patch :-) > > > On Thu, Oct 18, 2012 at 7:18 PM, Suresh Srinivas > <sur...@hortonworks.com>wrote: > >> Lets revert the job and merge the right patch. >> >> >> On Thu, Oct 18, 2012 at 6:49 PM, Colin McCabe <cmcc...@alumni.cmu.edu>wrote: >> >>> Hmm. It did build for me (in the sense of completing successfully, etc). >>> >>> You've definitely found an issue, though. The version should be >>> 2.0.3-SNAPSHOT for those projects, not 3.0.0-SNAPSHOT. >>> >>> Do you want to file a JIRA and post this patch, or should I? >>> >>> cheers. >>> Colin >>> >>> >>> On Thu, Oct 18, 2012 at 6:46 PM, Andrew Purtell <apurt...@apache.org> >>> wrote: >>> > Sorry, I always forget that about the mailing list software. >>> > >>> > From a6efe4883f3f7694ec6e8da04de982fb21eed8a9 Mon Sep 17 00:00:00 2001 >>> > From: Andrew Purtell <apurt...@apache.org> >>> > Date: Thu, 18 Oct 2012 17:22:17 -0700 >>> > Subject: [PATCH] Fix incorrect versions and missing module parent >>> introduced >>> > by HADOOP-8887 >>> > >>> > --- >>> > dev-support/cmake-maven-ng-plugin/pom.xml | 7 ++++++- >>> > dev-support/pom.xml | 4 ++-- >>> > hadoop-project/pom.xml | 2 +- >>> > 3 files changed, 9 insertions(+), 4 deletions(-) >>> > >>> > diff --git a/dev-support/cmake-maven-ng-plugin/pom.xml >>> > b/dev-support/cmake-maven-ng-plugin/pom.xml >>> > index e3d54bc..8652214 100644 >>> > --- a/dev-support/cmake-maven-ng-plugin/pom.xml >>> > +++ b/dev-support/cmake-maven-ng-plugin/pom.xml >>> > @@ -17,10 +17,15 @@ >>> > xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 >>> > http://maven.apache.org/maven-v4_0_0.xsd";> >>> > <modelVersion>4.0.0</modelVersion> >>> > + <parent> >>> > + <groupId>org.apache.hadoop</groupId> >>> > + <artifactId>dev-support</artifactId> >>> > + <version>2.0.3-SNAPSHOT</version> >>> > + </parent> >>> > <groupId>org.apache.hadoop.cmake.maven.ng</groupId> >>> > <artifactId>cmake-ng</artifactId> >>> > <packaging>maven-plugin</packaging> >>> > - <version>3.0.0-SNAPSHOT</version> >>> > + <version>2.0.3-SNAPSHOT</version> >>> > <name>cmake-ng Maven Mojo</name> >>> > <url>http://maven.apache.org</url> >>> > <properties> >>> > diff --git a/dev-support/pom.xml b/dev-support/pom.xml >>> > index 721e231..11a96cc 100644 >>> > --- a/dev-support/pom.xml >>> > +++ b/dev-support/pom.xml >>> > @@ -20,12 +20,12 @@ >>> > <parent> >>> > <groupId>org.apache.hadoop</groupId> >>> > <artifactId>hadoop-project</artifactId> >>> > - <version>3.0.0-SNAPSHOT</version> >>> > + <version>2.0.3-SNAPSHOT</version> >>> > <relativePath>../hadoop-project</relativePath> >>> > </parent> >>> > <groupId>org.apache.hadoop</groupId> >>> > <artifactId>dev-support</artifactId> >>> > - <version>3.0.0-SNAPSHOT</version> >>> > + <version>2.0.3-SNAPSHOT</version> >>> > <description>Apache Hadoop Development Support</description> >>> > <name>Apache Hadoop Development Support</name> >>> > <packaging>pom</packaging> >>> > diff --git a/hadoop-project/pom.xml b/hadoop-project/pom.xml >>> > index 568a59d..88d4676 100644 >>> > --- a/hadoop-project/pom.xml >>> > +++ b/hadoop-project/pom.xml >>> > @@ -704,7 +704,7 @@ >>> > <plugin> >>> > <groupId>org.apache.hadoop.cmake.maven.ng</groupId> >>> > <artifactId>cmake-ng</artifactId> >>> > - <version>3.0.0-SNAPSHOT</version> >>> > + <version>2.0.3-SNAPSHOT</version> >>> > </plugin> >>> > <plugin> >>> > <groupId>org.apache.maven.plugins</groupId> >>> > -- >>> > 1.7.9.5 >>> > >>> > >>> > On Thu, Oct 18, 2012 at 6:35 PM, Colin McCabe <cmcc...@alumni.cmu.edu >>> >wrote: >>> > >>> >> Hi Andrew, >>> >> >>> >> It seems that your attachment did not appear on the mailing list. >>> >> >>> >> I'm taking a look at the branch-2 build presently. >>> >> >>> >> Colin >>> >> >>> >> >>> >> On Thu, Oct 18, 2012 at 6:20 PM, Andrew Purtell <apurt...@apache.org> >>> >> wrote: >>> >> > The port of HADOOP-8887 to branch-2 fails the build. Please kindly >>> see >>> >> > attached. >>> >> > >>> >> > -- >>> >> > Best regards, >>> >> > >>> >> > - Andy >>> >> > >>> >> > Problems worthy of attack prove their worth by hitting back. - Piet >>> Hein >>> >> > (via Tom White) >>> >> > >>> >> >>> > >>> > >>> > >>> > -- >>> > Best regards, >>> > >>> > - Andy >>> > >>> > Problems worthy of attack prove their worth by hitting back. - Piet >>> Hein >>> > (via Tom White) >>> >> >> >> >> -- >> http://hortonworks.com/download/ >> >> > > > -- > http://hortonworks.com/download/ > > -- http://hortonworks.com/download/
