Re: [Plplot-devel] Poor RC1 test results on Debian woody

[Rafael L. <lab...@ps...>]

* Alan W. Irwin <ir...@be...> [2003-04-15 14:03]:

> *** Simple wish and tclsh tests following the directions in README.tcldemos
> and README.tkdemos don't work in RC1 because there is a problem for
> bindings/*/pkgIndex.tcl.in.  Those scripts assume the library suffix is the
> same as VERSION (true at the time I introduced those scripts, but not any
> longer).  To solve this I have introduced LIBRARY_VERSION = numerical
> suffix of the library in configure.ac which must be adjusted to be
> consistent with SOVERSION. Rafael, can you find an automatic way of keeping
> LIBRARY_VERSION consistent with SOVERSION?  That's important because
> otherwise we have a tcl/tk maintenance issue that will come back to haunt
> us every time SOVERSION is changed.  I did look for any variable that was
> already set to the numerical library suffix, but I couldn't find any.

I dislike this solution for two reason reason.  the first, as you mention
above, is the maintenance burden that it introduces.  The second (and more
important) reason is that your assumption of library suffix generation does
not work cross-platform.  If I remember correctly, in HPUX (for instance)
when SOVERSION=8:0:3, then the library would be something like
libplplot.sl.8.3.  There are other system with even more exotic rule and the
assumption that it will be libplplot.5.3.0 is far from being the rule.

> Rafael and Joao, what are your results for
> ./plplot-test.sh --device=png
> ( or ./plplot-test.sh --front-end=octave --device=png) for RC1.

$ ./plplot-test.sh --front-end=octave --device=png
[removed warnings about automatic_replot]Opened p1.png
Opened p2.png
Opened p3.png
Opened p4.png
Opened p5.png
Opened p6.png
Opened p7.png
Opened p8.png
Opened p9.png
Opened p13.png
Opened p15.png
Opened p16.png
Plplot library version: 5.2.0.cvs.20030415
Opened x01o.png
[removed mouse click message]
Opened x02o.png
Opened x03o.png
Opened x04o.png
Opened x05o.png
Opened x06o.png
Opened x07o.png
Opened x08o.png
Opened x09o.png
Opened x10o.png
Opened x11o.png
Opened x12o.png
Opened x13o.png
Opened x15o.png
Opened x16o.png
Opened x18o.png

> * /usr/bin/g++ -c -fPIC -I/usr/include/octave-2.1.35
> -I/usr/include/octave-2.1.35/
> octave -I/usr/include -mieee-fp -fno-implicit-templates -O2 -I.
> plplot_octave.cc
>  -o plplot_octave.o
>  In file included from /usr/include/octave-2.1.35/octave/oct.h:31,
>                   from plplot_octave.cc:11:
> 		  /usr/include/octave-2.1.35/octave/config.h:148: warning:
> HAVE_ISINF' redefined
> ../../config.h:36: warning: this is the location of the previous definition
> /usr/include/octave-2.1.35/octave/config.h:151: warning: HAVE_ISNAN' redefined
> ../../config.h:39: warning: this is the location of the previous definition
> /usr/include/octave-2.1.35/octave/config.h:288: warning: HAVE_FINITE'
> redefined
> ../../config.h:24: warning: this is the location of the previous definition
> /usr/include/octave-2.1.35/octave/config.h:474: warning: HAVE_USLEEP'
> redefined
> ../../config.h:101: warning: this is the location of the previous definition
> 
> Rafael or Joao may want to deal with these warnings before the release.

Joao and I agreed that these are totally harmless warnings.  As Maurice uses
to say, they are just irritating.  In sum, I am not going to fix this before
the release.

> That (and a quick check of RC2 for the above issues) is about all the
> testing I have time for before the release so again, somebody else is going
> to have to step forward if you want a quality release.

It seems that most of the problems that you found are quite minor or even
negligeable for now (besides that LIBRARY_VERSION bizareness).  I think that
we are pretty close to a "quality release".

-- 
Rafael