Hey Martin,

On Aug 15, 2012, at 9:20 AM, Martin Desruisseaux wrote:

> I saw that you already committed the patch, thanks :-)

Anytime!

> 
> Do I create a series of patch proposals for configuring the Maven build? My 
> proposals would be as below (note: I'm not going to start this task before 
> tomorrow, so you have time to tell me if some point are bad ideas):

Up to you in terms of the way you organize it in JIRA. I'm always a fan of 
smaller, more
easily revertible patches, but if you made one I wouldn't complain :) I 
appreciate the help.

> 
> 1. Eventually merge the sis-parent/pom.xml file with the root pom.xml
>   file, for simplifying a little bit the system.

Fine by me if you can make it work out the same way it currently does.

> 2. Upgrade the org.apache.apache parent from version 7 to version 10.

+1.

> 3. Add a <pluginManagement> section for the missing declaration of
>   plugins version, namely maven-javadoc-plugin, maven-jxr-plugin,
>   maven-surefire-report-plugin and maven-pmd-plugin. This would get
>   ride of the warnings at building time.

+1.

> 4. Move the version declaration of maven-bundle-plugin (currently
>   repeated in every modules) to the above cited <pluginManagement>
>   section.

+1.

> 5. Move the <reporting> section, which is now deprecated, to the
>   <configuration> section of the maven-site-plugin plugin.

+1.

> 6. Configure the javadoc plugin.

+1.

Awesomeness.

Cheers,
Chris

++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Chris Mattmann, Ph.D.
Senior Computer Scientist
NASA Jet Propulsion Laboratory Pasadena, CA 91109 USA
Office: 171-266B, Mailstop: 171-246
Email: [email protected]
WWW:   http://sunset.usc.edu/~mattmann/
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Adjunct Assistant Professor, Computer Science Department
University of Southern California, Los Angeles, CA 90089 USA
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++

Reply via email to