Re: [Plplot-devel] Second release candidate (rc2) for PLplot 5.3.0 available

[Rafael L. <rla...@us...>]

* Alan W. Irwin <ir...@be...> [2004-01-27 13:18]:

> Can you confirm that hang for plplot-test.sh? 
>
> It is possible some interactive stuff has been put into x01c.m for RC2 which
> makes it impossible to run it with -dev psc (which is what test_octave.sh
> attempts to do).  If this is the problem, can you change x01c.m to work once
> again with file devices?

I confirm it here for RC2.  Also, when the old RC1 x01c.m is used, the hang
disappears.

The patch to fix this problem is quite trivial:

--- x01c.m-orig 2004-01-27 23:46:39.000000000 +0000
+++ x01c.m      2004-01-27 23:45:31.000000000 +0000
@@ -103,7 +103,9 @@
       if (status != 0)
        printf("wx=%.3f wy=%.3f dx=%.3f dy=%.3f c=0x%02x str=%s mb=%d mod=%0x swin=%d\n", ...
               wX, wY, dX, dY, keysym, string, button, mod, swin);
-      endif
+      else
+        break; 
+      endif      
 
       fflush(stdout);
     endwhile
     
Joao, if you agree with the patch above, I will apply it for the final
tarball.

> I suspect that now also occur for the latest octave as well (which I cannot
> test, of course, but you, Brian, and Rafael have access to it).

I tried plplot-test.sh with Octave 2.1.53 on Debian stable and it fails for
x03c.m:

error: dimension mismatch in argument x
error: called from `plline'
error: evaluating for command near line 44, column 3
error: called from `x03c' in file `/var/tmp/examples/octave/x03c.m'
error: evaluating for command near line 4, column 1

This is a 2.1.53-specific problem.  We should not bother fixing it
pre-release, but we should put a note saying that the Octave PLplot bindings
will not fully work for version > 2.1.50.

> I hope you are also able to sort out the linking problem you found for
> octave-2.0.17.  In my opinion, that is a separate but release-critical
> mkoctfile issue that should be resolved before release or else octave-2.0.x
> should be excluded for the release.

I vote for the second option.  We should not hold the release just because
there are version-specific problems with OCtave.

-- 
Rafael