On Wed, Apr 11, 2012 at 02:54:59AM +0530, Atri wrote:
>
> Hi!
> Sorry for the long delay in between my last test and the present one. I
> got the svn version to build on openSUSE:Factory (with Lua 5.2) finally.
> I used for my build the svn trunk directory (at revision 12185) without
> applying any patch except this following one, which is necessary to
> avoid warnings with rpmlint during the build [1].
>
> ====================================================
> diff -ur plplot-5.9.8.orig//drivers/wxwidgets.cpp
> plplot-5.9.8//drivers/wxwidgets.cpp
> --- plplot-5.9.8.orig//drivers/wxwidgets.cpp 2011-08-01
> 23:29:39.000000000 +0530
> +++ plplot-5.9.8//drivers/wxwidgets.cpp 2011-08-02 01:06:57.323000001
> +0530
> @@ -231,7 +231,7 @@
> if ( ucs4[i] != (PLUNICODE) plplotEsc ) // a character to
> display
> {
> ucs4_to_utf8( ucs4[i], utf8 );
> - strncat( utf8_string, utf8, max_string_length );
> + strncat( utf8_string, utf8,
> sizeof(utf8_string)-strlen(utf8_string)-1 );
> i++;
> continue;
> }
> @@ -239,7 +239,7 @@
> if ( ucs4[i] == (PLUNICODE) plplotEsc ) // a escape
> character to display
> {
> ucs4_to_utf8( ucs4[i], utf8 );
> - strncat( utf8_string, utf8, max_string_length );
> + strncat( utf8_string, utf8,
> sizeof(utf8_string)-strlen(utf8_string)-1);
> i++;
> continue;
> }
> ====================================================
>
> After the building is done, I tested the package by running the
> plplot_test.sh script followed by test_diff.sh test for lua and python.
> The result is perfect, because the test reports no difference between
> the psc plots obtained with c/cpp and those obtained with lua 5.2. This
> is what the result says (I understand that the x00 example which was
> missing from lua/python previously is now also available for these two
> bindings):
> ============================================
> python
> Missing examples :
> Differing postscript output :
> Missing stdout :
> Differing stdout :
> lua
> Missing examples :
> Differing postscript output :
> Missing stdout :
> Differing stdout :
> ============================================
>
> I can confirm the same result by manual inspection of the psc output
> files, especially example 21 which was giving us problems earlier but
> now looks exactly the same whether generated by a c/cpp code or by lua.
> I believe we can now treat plplot at the current svn revision to be
> perfectly compatible with lua 5.2 as well. Thanks a bunch for all of
> your help with the tests, building etc. and for fixing everything too.
>
> There is an unrelated problem about the GNAT library not being detected
> by the build system, but I will report that in a separate mail. Please
> let me know if you think any more tests could help with the lua 5.2
> issue.
Dear Atri,
Many thanks for your report. It is good to know that you have
succeeded. Also, thanks for the patch to fix the strncat arguments.
I have committed this to the svn repository now. This is a common mistake
that has slipped through the net. I'll take a quick look to see if there
are any other occurences in the source.
Regards
Andrew
------------------------------------------------------------------------------
Better than sec? Nothing is better than sec when it comes to
monitoring Big Data applications. Try Boundary one-second
resolution app monitoring today. Free.
http://p.sf.net/sfu/Boundary-dev2dev
_______________________________________________
Plplot-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/plplot-devel
