Comment 6 for bug 1023329

Revision history for this message
Javier López (javier-lopez) wrote :

Hi Marco,

Thank you for your grib2 file and full explanation about how to test it.

I was able to pass the two testcases (grib1 - grib2) with cdo in precise/quantal/raring with the ppa version. Now let me give you my opinion about some of your questions.

> cdo 1.5.4 (since this is no more available on the developers' site, I downloaded it from http://archive.ubuntu.com/ubuntu/pool/universe/c/cdo/cdo_1.5.4+dfsg.1.orig.tar.gz ; I hope this is is conformal to the developers' original one).

Yes, the orig.* files are the upstream unmodified sources

> There is in fact one difference between the cdo available in the Ubuntu repositories and the ones I tested to write this post.

This is the 'cdo -V' output of the ppa versions:

raring:
cdo -V
Climate Data Operators version 1.5.6.1 (http://code.zmaw.de/projects/cdo)
Compiler: gcc -std=gnu99 -g -O2 -fstack-protector --param=ssp-buffer-size=4 -Wformat -Werror=format-security -fPIC -pthread
 version: gcc (Ubuntu/Linaro 4.7.2-5ubuntu5) 4.7.2
    with: PTHREADS Z JASPER PROJ.4
Compiled: by buildd on mercury (i686-pc-linux-gnu) Nov 14 2012 01:16:49
     CDI library version : 1.5.6 of Nov 14 2012 01:16:37
 CGRIBEX library version : 1.5.3 of Jun 14 2012 13:22:45
GRIB_API library version : 1.9.16
  netCDF library version : 4.1.3 of May 1 2012 04:15:00 $
    HDF5 library version : 1.8.8
 SERVICE library version : 1.3.0 of Nov 14 2012 01:16:12
   EXTRA library version : 1.3.0 of Nov 14 2012 01:16:05
     IEG library version : 1.3.0 of Nov 14 2012 01:16:10
    FILE library version : 1.8.0 of Nov 14 2012 01:16:05

quantal:
cdo -V
Climate Data Operators version 1.5.4 (http://code.zmaw.de/projects/cdo)
Compiler: mpicc -std=gnu99 -g -O2 -fstack-protector --param=ssp-buffer-size=4 -Wformat -Werror=format-security -fPIC
 version: gcc: error: unrecognized command line option '-V'
    with: PTHREADS Z JASPER PROJ.4
Compiled: by buildd on actinium (i686-pc-linux-gnu) Nov 14 2012 06:51:52
     CDI library version : 1.5.4 of Nov 14 2012 06:51:44
 CGRIBEX library version : 1.5.1 of Aug 29 2011 20:30:27
GRIB_API library version : 1.9.16
  netCDF library version : 4.1.3 of May 1 2012 04:15:00 $
    HDF5 library version : 1.8.8
 SERVICE library version : 1.3.0 of Nov 14 2012 06:51:23
   EXTRA library version : 1.3.0 of Nov 14 2012 06:51:17
     IEG library version : 1.3.0 of Nov 14 2012 06:51:22
    FILE library version : 1.7.1 of Nov 14 2012 06:51:1

precise:
cdo -V
Climate Data Operators version 1.5.3 (http://code.zmaw.de/projects/cdo)
Compiler: mpicc -std=gnu99 -g -O2 -fstack-protector --param=ssp-buffer-size=4 -Wformat -Wformat-security -Werror=format-security -fPIC
 version: gcc: error: unrecognized option '-V'
    with: PTHREADS Z JASPER PROJ.4
Compiled: by buildd on platinum (i686-pc-linux-gnu) Nov 14 2012 04:36:09
     CDI library version : 1.5.3 of Nov 14 2012 04:35:59
 CGRIBEX library version : 1.5.1 of Aug 29 2011 20:30:27
GRIB_API library version : 1.9.9
  netCDF library version : 4.1.1 of Nov 7 2011 11:30:05 $
    HDF5 library version : 1.8.4
 SERVICE library version : 1.3.0 of Nov 14 2012 04:35:34
   EXTRA library version : 1.3.0 of Nov 14 2012 04:35:28
     IEG library version : 1.3.0 of Nov 14 2012 04:35:32
    FILE library version : 1.7.1 of Nov 14 2012 04:35:2

As you can see there is no more differences between your version and the ones I'm planning to request to merge.

> Now I cannot understand why, for my case, the --enable-cgribex option was not necessary to make things go well, while in
your case it was necessary to specify it explicitely at configuration time.

Looking at the upstream report it seems like --enable-cgribex is a default option, that explain why you didn't have to configure it, however in the ubuntu package it was disabled on purpose, so I enable it on purpose too and wrote about it on the changelog file. I could had just delete --disable-cgribex and the package would had enable cgribex by default.

> I will let you know of my forthcoming double check on your ppa. However, if you have any further questions or comments, let me know.

It would be great to test them against netCDF too, so we don't have any doubt that the program is full working. I'll request a merge as fast as I can see it's working not just for me.

Greetings
---
Ubuntu Bug Squad volunteer triager
http://wiki.ubuntu.com/BugSquad