Bug#660260: Fwd: GDL seems to use non public symbols from plplot (Debian Wheezy freeze on Saturday)

2012-06-29 Thread Axel Beckert
Hi,

short status update:

Axel Beckert wrote:
  Both plstrl and plP_mmpcx are used in GDL to handle the WIDTH
  keyword to the XYOUTS procedure. That's surely not a vital feature
  of GDL so perhaps you can simply comment it out from the GDL source
  code for Debian packaging while waiting for a proper solution. Such
  patch would basically undo this commit:
  http://gnudatalanguage.cvs.sf.net/viewvc/gnudatalanguage/gdl/src/plotting.cpp?hideattic=0r1=1.127r2=1.128
 
 Ok, with this advice I got at least one incremental build working. Yay!

I've now got a package which builds.

Thanks Sylwester for his support and hints!

 Now I'm trying to get a clean package out of it. :-)

Still some fine-tuning (Lintian warnings, etc.) to do, but I'll likely
make an upload today.

Regards, Axel
-- 
 ,''`.  |  Axel Beckert a...@debian.org, http://people.debian.org/~abe/
: :' :  |  Debian Developer, ftp.ch.debian.org Admin
`. `'   |  1024D: F067 EA27 26B9 C3FC 1486  202E C09E 1D89 9593 0EDE
  `-|  4096R: 2517 B724 C5F6 CA99 5329  6E61 2FF9 CD59 6126 16B5



-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#660260: Fwd: GDL seems to use non public symbols from plplot (Debian Wheezy freeze on Saturday)

2012-06-28 Thread Axel Beckert
Hi Sylwester,

adding the Cc to the bug report and the others again. I just avoided
all the Cc in my last mail because of the attachments (build log,
etc.).

On Thu, Jun 28, 2012 at 10:12:01AM +0200, Sylwester Arabas wrote:
 That's now in the CVS. Please test if it fixes the issue.
 
 Hrm, not sure if I got all files which belong to the patch. According
 to 
 http://gnudatalanguage.cvs.sourceforge.net/viewvc/gnudatalanguage/gdl/src/?sortby=date
 I'd say Makefile.am, Makefile.in and plotting_surface.cpp have been
 modified and plot3d_nr.cpp has been removed.
 
 That's correct.

Yeah, verified that this morning after my CVS checkout finally finished.

 ... Unfortunately it still doesn't build. It though fails elsewhere.
 Here's the relevant part inline with the whole log attached, too...
 
 Please try with the current CVS version:
 http://gnudatalanguage.cvs.sf.net/viewvc/gnudatalanguage/gdl/src/plotting_surface.cpp?r1=1.7r2=1.8sortby=date

Will do, thanks for your effort.

 I was testing it yesterday on Debian with plplot 5.9.5-4.

That's the version from Debian stable which AFAIK has no
-fvisibility=hidden which causes the linking issues.

 I've just tested the current version with plplot 5.9.5-4 as well.
 
 After upgrading to 5.9.9-4 I have non-related linking problems:
 
 gdl-image.o: In function `lib::loadct(EnvT*)':
 /usr/local/src/gdl/src/image.cpp:90: undefined reference to
 `plstream::scmap1(int*, int*, int*, int)'
 gdl-plotting_xyouts.o: In function
 `lib::xyouts_call::old_body(EnvT*, GDLGStream*)':
 /usr/local/src/gdl/src/plotting_xyouts.cpp:256: undefined reference
 to `plP_mmpcy'
 /usr/local/src/gdl/src/plotting_xyouts.cpp:341: undefined reference
 to `plP_mmpcx'
 /usr/local/src/gdl/src/plotting_xyouts.cpp:328: undefined reference
 to `plstrl'
 /usr/local/src/gdl/src/plotting_xyouts.cpp:288: undefined reference
 to `plstrl'

Yes, it fails at that point now for me, too:

/bin/bash ../libtool --tag=CXX   --mode=link x86_64-linux-gnu-g++  -g -O2  
-Wl,-z,defs -fopenmp -lMagick++ -lMagickCore   -rdynamic -Wl,-z,muldefs -o gdl 
gdl-allix.o gdl-assocdata.o gdl-basic_fun_cl.o gdl-basic_fun.o 
gdl-basic_fun_jmg.o gdl-basic_op.o gdl-basic_op_new.o gdl-basic_pro.o 
gdl-basic_pro_jmg.o gdl-CFMTLexer.o gdl-color.o gdl-convert2.o gdl-datatypes.o 
gdl-dcommon.o gdl-dcompiler.o gdl-default_io.o gdl-dinterpreter.o gdl-dnode.o 
gdl-dpro.o gdl-dstructdesc.o gdl-dstructgdl.o gdl-dvar.o gdl-envt.o 
gdl-extrat.o gdl-FMTIn.o gdl-FMTLexer.o gdl-fmtnode.o gdl-FMTOut.o 
gdl-FMTParser.o gdl-gdleventhandler.o gdl-gdlexception.o gdl-gdlgstream.o 
gdl-GDLInterpreter.o gdl-GDLLexer.o gdl-GDLParser.o gdl-gdlpsstream.o 
gdl-gdlsvgstream.o gdl-gdlpython.o gdl-GDLTreeParser.o gdl-gdlwinstream.o 
gdl-gdlxstream.o gdl-getfmtast.o gdl-graphics.o gdl-gsl_fun.o gdl-ifmt.o 
gdl-initct.o gdl-initsysvar.o gdl-io.o gdl-libinit_cl.o gdl-libinit.o 
gdl-libinit_jmg.o gdl-math_fun.o gdl-math_fun_jmg.o gdl-math_utl.o 
gdl-ncdf_att_cl.o gdl-ncdf_cl.o gdl-ncdf_dim_cl.o gdl-ncdf_var_cl.o gdl-new.o 
gdl-objects.o gdl-ofmt.o gdl-math_fun_ac.o gdl-libinit_ac.o gdl-math_fun_gm.o 
gdl-libinit_gm.o gdl-math_fun_ng.o gdl-libinit_ng.o gdl-plotting.o gdl-print.o 
gdl-print_tree.o gdl-read.o gdl-str.o gdl-terminfo.o gdl-topython.o 
gdl-typetraits.o gdl-hdf_fun.o gdl-hdf_pro.o gdl-magick_cl.o gdl-gdlwidget.o 
gdl-widget.o gdl-basegdl.o gdl-hdf5_fun.o gdl-libinit_mes.o gdl-file.o 
gdl-image.o gdl-gdljournal.o gdl-convol.o gdl-convol_inc0.o gdl-convol_inc1.o 
gdl-convol_inc2.o gdl-sigfpehandler.o gdl-gdlzstream.o gdl-arrayindex.o 
gdl-fftw.o gdl-mpi.o gdl-grib.o gdl-prognode.o gdl-prognode_lexpr.o 
gdl-prognodeexpr.o gdl-datatypesref.o gdl-lapack.o gdl-gshhs.o 
gdl-newprognode.o gdl-plotting_plot.o gdl-plotting_oplot.o 
gdl-plotting_device.o gdl-plotting_cursor.o gdl-plotting_contour.o 
gdl-plotting_surface.o gdl-plotting_axis.o gdl-plotting_plots.o 
gdl-plotting_xyouts.o gdl-plotting_polyfill.o gdl-plotting_windows.o 
gdl-plotting_convert_coord.o gdl-plotting_map_proj.o gdl-plotting_misc.o 
gdl-plotting_erase.o gdl-semshm.o gdl-gdl.o gdl-lapack.o /usr/lib/libantlr.a 
-L/usr/X11R6/lib64 -L/usr/X11R6/lib -lX11 -lmfhdfalt -lgslcblas -lgsl -lncurses 
-ldl  -ldl -lreadline -lz -lgsl -lgslcblas -lplplotd -lplplotcxxd 
-L/usr/lib/x86_64-linux-gnu -pthread   -L/usr/lib/x86_64-linux-gnu   
-lwx_gtk2u_richtext-2.8 -lwx_gtk2u_aui-2.8 -lwx_gtk2u_xrc-2.8 -lwx_gtk2u_qa-2.8 
-lwx_gtk2u_html-2.8 -lwx_gtk2u_adv-2.8 -lwx_gtk2u_core-2.8 -lwx_baseu_xml-2.8 
-lwx_baseu_net-2.8 -lwx_baseu-2.8  -lMagick++ -lMagickCore   
-L/usr/lib/netcdf-3 -lnetcdf -L/usr/lib/hdf -lmfhdfalt -ldfalt -ljpeg -lz 
-L/usr/lib/hdf -L/usr/lib/hdf5 -lhdf5 -lfftw3 -lfftw3f -lpython2.7 -ludunits2 
-lgrib_api -lps
libtool: link: x86_64-linux-gnu-g++ -g -O2 -Wl,-z -Wl,defs -fopenmp -rdynamic 
-Wl,-z -Wl,muldefs -o gdl gdl-allix.o gdl-assocdata.o gdl-basic_fun_cl.o 
gdl-basic_fun.o gdl-basic_fun_jmg.o gdl-basic_op.o gdl-basic_op_new.o 
gdl-basic_pro.o gdl-basic_pro_jmg.o gdl-CFMTLexer.o gdl-color.o 

Bug#660260: Fwd: GDL seems to use non public symbols from plplot (Debian Wheezy freeze on Saturday)

2012-06-28 Thread Sylwester Arabas

Hi Axel,

On 28/06/12 12:19, Axel Beckert wrote:
 ...

/home/abe/gnudatalanguage/gnudatalanguage/src/plotting_xyouts.cpp:260: 
undefined reference to `plP_mmpcy'
/home/abe/gnudatalanguage/gnudatalanguage/src/plotting_xyouts.cpp:332: 
undefined reference to `plstrl'
/home/abe/gnudatalanguage/gnudatalanguage/src/plotting_xyouts.cpp:345: 
undefined reference to `plP_mmpcx'
/home/abe/gnudatalanguage/gnudatalanguage/src/plotting_xyouts.cpp:292: 
undefined reference to `plstrl'
collect2: error: ld returned 1 exit status
...

are the above critical for this week?


Well, it's way easier to get it into Wheezy if I upload the package
before the Freeze on Saturday evening. I can try to get it in later,
but I can't promise it, I'm sorry.

If more time helps to get the thing fixed for sure, I can ask the
release team in advance for a freeze exception. But again no promises.


I've just asked on plplot mailing list about it:
https://sourceforge.net/projects/plplot/forums/forum/132889/topic/5389011

Both plstrl and plP_mmpcx are used in GDL to handle the WIDTH keyword to 
the XYOUTS procedure. That's surely not a vital feature of GDL so 
perhaps you can simply comment it out from the GDL source code for 
Debian packaging while waiting for a proper solution. Such patch would 
basically undo this commit:

http://gnudatalanguage.cvs.sf.net/viewvc/gnudatalanguage/gdl/src/plotting.cpp?hideattic=0r1=1.127r2=1.128

HTH,
Sylwester

--
http://www.igf.fuw.edu.pl/~slayoo/



--
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#660260: Fwd: GDL seems to use non public symbols from plplot (Debian Wheezy freeze on Saturday)

2012-06-28 Thread Axel Beckert
Hi!

Sylwester Arabas wrote:
 Both plstrl and plP_mmpcx are used in GDL to handle the WIDTH
 keyword to the XYOUTS procedure. That's surely not a vital feature
 of GDL so perhaps you can simply comment it out from the GDL source
 code for Debian packaging while waiting for a proper solution. Such
 patch would basically undo this commit:
 http://gnudatalanguage.cvs.sf.net/viewvc/gnudatalanguage/gdl/src/plotting.cpp?hideattic=0r1=1.127r2=1.128

Ok, with this advice I got at least one incremental build working. Yay!

Now I'm trying to get a clean package out of it. :-)

Will keep you informed.

Regards, Axel
-- 
 ,''`.  |  Axel Beckert a...@debian.org, http://people.debian.org/~abe/
: :' :  |  Debian Developer, ftp.ch.debian.org Admin
`. `'   |  1024D: F067 EA27 26B9 C3FC 1486  202E C09E 1D89 9593 0EDE
  `-|  4096R: 2517 B724 C5F6 CA99 5329  6E61 2FF9 CD59 6126 16B5



-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#660260: Fwd: GDL seems to use non public symbols from plplot (Debian Wheezy freeze on Saturday)

2012-06-27 Thread Axel Beckert
Hi,

sorry to bother you again, but the freeze for Debian 7.0 Wheezy is
this Satuday (30th of June) and I still haven't heard or seen anything
on this issue:

On Tue, Apr 17, 2012 at 07:54:11PM -0400, gal...@cox.net wrote:
  Axel Beckert a...@deuxchevaux.org wrote:
  On Sat, Feb 18, 2012 at 01:09:13AM +0100, Sylwester Arabas wrote:
   There's a Debian bug report regarding GDL's usage of plplot's plP_* calls:
   http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=660260

I also reported it to GDL's SF bug tracker at
http://sourceforge.net/tracker/?func=detailaid=3529264group_id=97659atid=618683

GDL unfortunately will not be part of the next Debian stable release
if this can't be fixed soon.

   What were the reasons to put this plplot's (src/plot3d.c) file in the GDL 
   tree?
  
  Any news here? This _is_ definitely an issue on Ubuntu and may become
  one on Debian, too.

While GDL made it into Ubuntu 12.04 LTS because some PLPLOT changes
have been reversed there after the first build failures have been
reported, this will likely affect Ubuntu 12.10, too.

 I originally placed this file (in modified from) in the GDL
 repository to support non-rectangular grids for the SURFACE and
 CONTOUR functions.
 
 If PLPLOT can now support this feature natively then this code can
 be replaced.

I see[1] that you are preparing a 0.9.3 release but haven't noticed
any chnagelog entry related to this issue.

Any chance that the issue gets fixed for 0.9.3?

When will the 0.9.3 release happen? If I know in advance, there may be
a chance to get a freeze exception. (But I can't promise that.)

  [1] 
http://gnudatalanguage.cvs.sourceforge.net/viewvc/gnudatalanguage/gdl/NEWS?revision=1.101view=markupsortby=date

If you have a patch for the issue, please tell me (or send it to the
above mentioned and Cc'ed Debian bug report) so that I can apply the
patch to 0.9.2 and upload a fixed version as soon as possible without
waiting for the 0.9.3 release. (I would actually prefer that variant
currently as it's the most promising one for still getting into Debian
Wheezy.)

Regards, Axel
-- 
 ,''`.  |  Axel Beckert a...@debian.org, http://people.debian.org/~abe/
: :' :  |  Debian Developer, ftp.ch.debian.org Admin
`. `'   |  1024D: F067 EA27 26B9 C3FC 1486  202E C09E 1D89 9593 0EDE
  `-|  4096R: 2517 B724 C5F6 CA99 5329  6E61 2FF9 CD59 6126 16B5



-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#660260: Fwd: GDL seems to use non public symbols from plplot (Debian Wheezy freeze on Saturday)

2012-06-27 Thread Alain Coulais
 Hi,

 sorry to bother you again, but the freeze for Debian 7.0 Wheezy is
 this Satuday (30th of June) and I still haven't heard or seen anything
 on this issue:


thanks

this is really urgent !

Alain

 On Tue, Apr 17, 2012 at 07:54:11PM -0400, gal...@cox.net wrote:
  Axel Beckert a...@deuxchevaux.org wrote:
  On Sat, Feb 18, 2012 at 01:09:13AM +0100, Sylwester Arabas wrote:
   There's a Debian bug report regarding GDL's usage of plplot's plP_*
 calls:
   http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=660260

 I also reported it to GDL's SF bug tracker at
 http://sourceforge.net/tracker/?func=detailaid=3529264group_id=97659atid=618683

 GDL unfortunately will not be part of the next Debian stable release
 if this can't be fixed soon.

   What were the reasons to put this plplot's (src/plot3d.c) file in
 the GDL tree?
 
  Any news here? This _is_ definitely an issue on Ubuntu and may become
  one on Debian, too.

 While GDL made it into Ubuntu 12.04 LTS because some PLPLOT changes
 have been reversed there after the first build failures have been
 reported, this will likely affect Ubuntu 12.10, too.

 I originally placed this file (in modified from) in the GDL
 repository to support non-rectangular grids for the SURFACE and
 CONTOUR functions.

 If PLPLOT can now support this feature natively then this code can
 be replaced.

 I see[1] that you are preparing a 0.9.3 release but haven't noticed
 any chnagelog entry related to this issue.

 Any chance that the issue gets fixed for 0.9.3?

 When will the 0.9.3 release happen? If I know in advance, there may be
 a chance to get a freeze exception. (But I can't promise that.)

   [1]
 http://gnudatalanguage.cvs.sourceforge.net/viewvc/gnudatalanguage/gdl/NEWS?revision=1.101view=markupsortby=date

 If you have a patch for the issue, please tell me (or send it to the
 above mentioned and Cc'ed Debian bug report) so that I can apply the
 patch to 0.9.2 and upload a fixed version as soon as possible without
 waiting for the 0.9.3 release. (I would actually prefer that variant
 currently as it's the most promising one for still getting into Debian
 Wheezy.)

   Regards, Axel
 --
  ,''`.  |  Axel Beckert a...@debian.org, http://people.debian.org/~abe/
 : :' :  |  Debian Developer, ftp.ch.debian.org Admin
 `. `'   |  1024D: F067 EA27 26B9 C3FC 1486  202E C09E 1D89 9593 0EDE
   `-|  4096R: 2517 B724 C5F6 CA99 5329  6E61 2FF9 CD59 6126 16B5






-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#660260: Fwd: GDL seems to use non public symbols from plplot (Debian Wheezy freeze on Saturday)

2012-06-27 Thread Sylwester Arabas

Hi Axel,
CC: Joel, Alain, Marc, Debian bug-tracker

On 27/06/12 18:07, Axel Beckert wrote:

... 
http://sourceforge.net/tracker/?func=detailaid=3529264group_id=97659atid=618683

GDL unfortunately will not be part of the next Debian stable release
if this can't be fixed soon.


Here's a fix suggested by Joel:

On 05/06/12 14:09, gal...@cox.net wrote:
 I would suggest replacing:

 lib::mesh_nr(xVec1, yVec1, z1, (long int) xEl, 1,1);

 in plotting.cpp by

 plmesh(xVec1, yVec1, z1, (long int) xEl, 1,1);

 I think the plplot library now has the functionality that was
 missing before.

 If we remove the call to mesh_nr then we no longer have to include
 plot3d_nr.cpp which was causing the problem.

I'll try to test it and commit to the repository today or tomorrow.

Sorry for the delay and big thanks for the reminders!
Best,
Sylwester

--
http://www.igf.fuw.edu.pl/~slayoo/



--
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#660260: Fwd: GDL seems to use non public symbols from plplot (Debian Wheezy freeze on Saturday)

2012-06-27 Thread Sylwester Arabas

Hi All,

On 27/06/12 18:59, Sylwester Arabas wrote:

Here's a fix suggested by Joel:

On 05/06/12 14:09, gal...@cox.net wrote:
  I would suggest replacing:
 
  lib::mesh_nr(xVec1, yVec1, z1, (long int) xEl, 1,1);
 
  in plotting.cpp by
 
  plmesh(xVec1, yVec1, z1, (long int) xEl, 1,1);
 
  I think the plplot library now has the functionality that was
  missing before.
 
  If we remove the call to mesh_nr then we no longer have to include
  plot3d_nr.cpp which was causing the problem.

I'll try to test it and commit to the repository today or tomorrow.


That's now in the CVS. Please test if it fixes the issue.
Best,
Sylwester

--
http://www.igf.fuw.edu.pl/~slayoo/



--
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org