Re: [netcdf-java] Mavenizing NetCDF: Mostly done, feedback needed

• To: Martin Desruisseaux <martin.desruisseaux@xxxxxxxxxxx>
• Subject: Re: [netcdf-java] Mavenizing NetCDF: Mostly done, feedback needed
• From: Brian Schlining <bschlining@xxxxxxxxx>
• Date: Mon, 13 Dec 2010 09:54:17 -0800

Martin,

Very nice work. I do have one comment though, the pom's are missing the
version numbers for dependencies. For example, in
http://hg.geotoolkit.org/netcdf-deploy/file/3f88cc956b9f/opendap/pom.xml you
have:

64<http://hg.geotoolkit.org/netcdf-deploy/file/3f88cc956b9f/opendap/pom.xml#l64>
<dependency>
65<http://hg.geotoolkit.org/netcdf-deploy/file/3f88cc956b9f/opendap/pom.xml#l65>
<groupId>commons-httpclient</groupId>
66<http://hg.geotoolkit.org/netcdf-deploy/file/3f88cc956b9f/opendap/pom.xml#l66>
<artifactId>commons-httpclient</artifactId>
67<http://hg.geotoolkit.org/netcdf-deploy/file/3f88cc956b9f/opendap/pom.xml#l67>
</dependency>

That really should be:
<dependency>
    <groupId>commons-httpclient</groupId>
    <artifactId>commons-httpclient</groupId>
    <version>3.1</version>
</dependency>

I know is sounds pretty anal; But it will save users a lot of headaches down
the road.

Cheers and thanks for all your efforts!

I'm mostly done in mavenizing the NetCDF library. The pom.xml files are at
> the usual location: http://hg.geotoolkit.org/netcdf-deploy/file/tip/.
> Maybe it would be useful to have them copied on the SVN?
>
> A couple of notes:
>
> * I used the following version numbers, please update them as needed:
>  - NetCDF: 4.2.5 (because I noticed that the last tag was 4.2.4)
>  - UnidataCommon: 4.2.5
>  - OpenDap: 2.2
>  - Grib: 8.0.27
>  - bufrTables: 2.0.1
>  - visad: 2010-08-09 (seems to be the last release date)
>

-- 
~ ~ ~ ~ ~ ~ ~ ~ ~ ~ ~ ~ ~ ~ ~ ~ ~ ~ ~ ~ ~ ~ ~
Brian Schlining
bschlining@xxxxxxxxx

• Follow-Ups:
  • Re: [netcdf-java] Mavenizing NetCDF: Mostly done, feedback needed
    • From: Martin Desruisseaux

• References:
  • [netcdf-java] Mavenizing NetCDF: Mostly done, feedback needed
    • From: Martin Desruisseaux