Home | About | Sematext search-lucene.com search-hadoop.com
 Search Hadoop and all its subprojects:

Switch to Threaded View
Hadoop >> mail # general >> getting started building Mavenized hadoop common

Copy link to this message
Re: getting started building Mavenized hadoop common
On Tue, Aug 2, 2011 at 3:47 PM, Jeffrey Naisbitt <[EMAIL PROTECTED]> wrote:
> On 8/2/11 5:21 PM, "Alejandro Abdelnur" <[EMAIL PROTECTED]> wrote:
>> Regarding adding the 'target/generated-src/test/java' dir to the build path.
>> You are correct, you have to add it manually to your IDE (I use IntelliJ and
>> it is the same story). But unless you need to debug through the generated
>> code you don't need to do so (doing a 'mvn test -DskipTests' will
>> generate/compile the class and the .class file will be in the IDE project
>> classpath).
> I like to debug through the code :)  It would be nice if there were an
> automated way to handle that folder, but in the meantime, it would probably
> be useful to document that along with the eclipse instructions.

I had to do this step too. I've added it to the instructions on
http://wiki.apache.org/hadoop/EclipseEnvironment, but I agree it would
be nice to automate this if anyone knows the relevant setting.

>> Regarding MAVEN_HOME, I don't have it in my environment and the build works.
> I was referring to running test-patch.sh.  (test-patch.sh now requires the
> MAVEN_HOME setting).  I'm ok if this is a requirement to run it now (like
> ANT_HOME was required before), but it should probably be mentioned somewhere
> since I didn't think about it :)

I've updated http://wiki.apache.org/hadoop/HowToContribute to mention


>> Regarding the test-patch.sh issues, was it working for you prior to
>> HADOOP-6671? Because we didn't change that line. Also, keep in mind that the
>> injection fault tests are not wired yet.
> I don't remember seeing those before, but yeah, it doesn't look like your
> patch touched anything to do with them.  They seem to be ignored anyway.
> Thanks again...great work :)
> -Jeff
>> On Tue, Aug 2, 2011 at 2:55 PM, Jeffrey Naisbitt
>> <[EMAIL PROTECTED]>wrote:
>>> Thanks for all your work and updates on this, Alejandro!  It's much
>>> better/easier to work with :)
>>> I did have a few issues/questions:
>>> First, I still had to manually add 'target/generated-src/test/java' to my
>>> build path sources in eclipse.  I don't know if this is due to something I
>>> did wrong, but I would think this should be automatic.
>>> Also, I ran into a few issues with the test-patch.sh script:
>>> First, it will fail if MAVEN_HOME is not set, and I didn't see anything
>>> about that in documentation.
>>> Also, it gives a couple of non-critical errors:
>>> ./dev-support/test-patch.sh: line 578: auxwww: command not found
>>> ./dev-support/test-patch.sh: line 578: /usr/bin/nawk: No such file or
>>> directory
>>> The first is because $PS is not set (and was previously passed in for the
>>> HUDSON version), and the second is just because my box doesn¹t have nawk on
>>> it.
>>> Thanks again.
>>> -Jeff
>>> On 8/2/11 4:13 PM, "Alejandro Abdelnur" <[EMAIL PROTECTED]> wrote:
>>>> With the HADOOP-6671 commit the way of building hadoop common has changed
>>>> significantly.
>>>> While the wiki explains these changes, and there is a BUILDING.txt
>>>> directory, still I guess things will hit many of of you.
>>>> Because of this I've put together some brief notes.
>>>> Thanks.
>>>> Alejandro
>>>> ----------------------
>>>> After updating the trunk you'll the the following directory changes at
>>> top
>>>> level
>>>>  Removed: common/
>>>>  New: hadoop-common/, hadoop-project/, hadoop-annotations/,
>>>> hadoop-assemblies/
>>>> * hadoop-common/ is the new common/ and its sub-dirs are organized
>>> following
>>>> Maven standard project layout.
>>>> * hadoop-project/ contains Hadoop project root POM, all dependency
>>> versions
>>>> are defined there
>>>> * hadoop-annotations/ contains the Hadoop public/private annotation
>>> classes
>>>> * hadoop-assemblies/ contains the assembly files that create the
>>>> distribution directories layout
>>>> ----------------------