Re: mathgl 1.11 upload

2010-11-15 Thread Sylvestre Ledru
No worries.

I tried to build it into a chroot and it fails:
/usr/include/hdf/hdfi.h:1011: error: conflicting declaration 'typedef
long int intf'
/usr/include/zconf.h:340: error: 'intf' has a previous declaration as
'typedef int intf'
mgl_data_io.cpp: In member function 'void mglData::ReadHDF4(const char*,
const char*)':
mgl_data_io.cpp:914: error: invalid conversion from 'int*' to 'int32*'
mgl_data_io.cpp:914: error:   initializing argument 2 of 'intn
SDfileinfo(int32, int32*, int32*)'

Sylvestre
PS: Keep the kind of reply (top-down / down-top).


Le lundi 15 novembre 2010 à 03:19 +0200, Dimitrios Eftaxiopoulos a
écrit :
 My mistake. Actually I tried to push a correction on this a few minutes ago 
 but you had already done it for me.
 
 Regards
 Dimitris
 
 Στις Δευτέρα 15 Νοέμβριος 2010 01:55:22 γράψατε:
  Hello,
  
  Le lundi 15 novembre 2010 à 02:29 +0200, Dimitrios Eftaxiopoulos a
  
  écrit :
   Hello again,
   I think that mathgl 1.11 is ready for upload, probably to experimental.
  
  Why did you tag it as unstable in the changelog ?
  
  Sylvestre
 
 



-- 
To UNSUBSCRIBE, email to debian-science-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Archive: http://lists.debian.org/1289819855.6889.695.ca...@zlarin



Re: mathgl 1.11 upload

2010-11-15 Thread Dimitrios Eftaxiopoulos
Hello,

I have built the package with

$debuild

I have built it with pbuilder  with 

#pbuilder --create (did this once only)
#pbuilder --update
#pbuilder --build mathgl_1.11-1.dsc

and I have built it with

#pdebuild 

from the source directory.

I don't see any failure.

Regards
Dimitris

Στις Δευτέρα 15 Νοέμβριος 2010 13:17:35 γράψατε:
 No worries.
 
 I tried to build it into a chroot and it fails:
 /usr/include/hdf/hdfi.h:1011: error: conflicting declaration 'typedef
 long int intf'
 /usr/include/zconf.h:340: error: 'intf' has a previous declaration as
 'typedef int intf'
 mgl_data_io.cpp: In member function 'void mglData::ReadHDF4(const char*,
 const char*)':
 mgl_data_io.cpp:914: error: invalid conversion from 'int*' to 'int32*'
 mgl_data_io.cpp:914: error:   initializing argument 2 of 'intn
 SDfileinfo(int32, int32*, int32*)'
 
 Sylvestre
 PS: Keep the kind of reply (top-down / down-top).
 
 
 Le lundi 15 novembre 2010 à 03:19 +0200, Dimitrios Eftaxiopoulos a
 
 écrit :
  My mistake. Actually I tried to push a correction on this a few minutes
  ago but you had already done it for me.
  
  Regards
  Dimitris
  
  Στις Δευτέρα 15 Νοέμβριος 2010 01:55:22 γράψατε:
   Hello,
   
   Le lundi 15 novembre 2010 à 02:29 +0200, Dimitrios Eftaxiopoulos a
   
   écrit :
Hello again,
I think that mathgl 1.11 is ready for upload, probably to
experimental.
   
   Why did you tag it as unstable in the changelog ?
   
   Sylvestre


--
To UNSUBSCRIBE, email to debian-science-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Archive: http://lists.debian.org/201011151602.17149.eftax...@otenet.gr



Re: mathgl 1.11 upload

2010-11-15 Thread Aaron M. Ucko
Sylvestre Ledru sylves...@debian.org writes:

 I tried to build it into a chroot and it fails:
 /usr/include/hdf/hdfi.h:1011: error: conflicting declaration 'typedef
 long int intf'
 /usr/include/zconf.h:340: error: 'intf' has a previous declaration as
 'typedef int intf'

This looks like an architecture-specific issue; hdfi.h typedefs intf to
be long on i386 and int on amd64, whereas zconf.h uses int regardless.
The types are the same size on i386, but still formally distinct, hence
the error.  To work around it, you may have to use the preprocessor to
divert one library's definition:

[...]
#include zlib.h
#ifdef HAVE_HDF5
#include hdf5.h
#endif
#ifdef HAVE_HDF4
#define intf hdf4_intf
#include hdf/mfhdf.h
#undef intf
#endif
[...]

-- 
Aaron M. Ucko, KB1CJC (amu at alum.mit.edu, ucko at debian.org)
http://www.mit.edu/~amu/ | http://stuff.mit.edu/cgi/finger/?...@monk.mit.edu


-- 
To UNSUBSCRIBE, email to debian-science-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Archive: http://lists.debian.org/udl7hgemwci@dr-wily.mit.edu



mathgl 1.11 upload

2010-11-14 Thread Dimitrios Eftaxiopoulos
Hello again,
I think that mathgl 1.11 is ready for upload, probably to experimental. Thanks 
Ruben for the hint (I had read it before but I was somehow lost in it).

Regards
Dimitris

Στις Κυριακή 14 Νοέμβριος 2010 17:18:07 γράψατε:
 El dom, 14-11-2010 a las 17:39 +0200, Dimitrios Eftaxiopoulos escribió:
  E: libmgl5: binary-or-shlib-defines-rpath ./usr/lib/libmgl.so.5.0.0
  /tmp/buildd/libhdf4-4.2r4/HDF4/hdf/src/.libs
  E: libmgl5: binary-or-shlib-defines-rpath ./usr/lib/libmgl.so.5.0.0
  /tmp/buildd/libhdf4-4.2r4/HDF4/mfhdf/libsrc/.libs
  ..
 
 Hi Dimitrios,
 Please take a look at http://wiki.debian.org/RpathIssue :)
 Thanks for working in mathgl.
 Ruben


--
To UNSUBSCRIBE, email to debian-science-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Archive: http://lists.debian.org/201011150229.43982.eftax...@otenet.gr



Re: mathgl 1.11 upload

2010-11-14 Thread Sylvestre Ledru
Hello,

Le lundi 15 novembre 2010 à 02:29 +0200, Dimitrios Eftaxiopoulos a
écrit :
 Hello again,
 I think that mathgl 1.11 is ready for upload, probably to experimental.
Why did you tag it as unstable in the changelog ?

Sylvestre



-- 
To UNSUBSCRIBE, email to debian-science-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Archive: http://lists.debian.org/1289778922.20382.10840.ca...@zlarin



Re: mathgl 1.11 upload

2010-11-14 Thread Dimitrios Eftaxiopoulos
My mistake. Actually I tried to push a correction on this a few minutes ago 
but you had already done it for me.

Regards
Dimitris

Στις Δευτέρα 15 Νοέμβριος 2010 01:55:22 γράψατε:
 Hello,
 
 Le lundi 15 novembre 2010 à 02:29 +0200, Dimitrios Eftaxiopoulos a
 
 écrit :
  Hello again,
  I think that mathgl 1.11 is ready for upload, probably to experimental.
 
 Why did you tag it as unstable in the changelog ?
 
 Sylvestre


--
To UNSUBSCRIBE, email to debian-science-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Archive: http://lists.debian.org/201011150319.04802.eftax...@otenet.gr



Re: mathgl 1.11 upload

2010-11-14 Thread Julian Taylor
On Mon, 2010-11-15 at 02:29 +0200, Dimitrios Eftaxiopoulos wrote:
 Hello again,
 I think that mathgl 1.11 is ready for upload, probably to experimental. 
 Thanks 
 Ruben for the hint (I had read it before but I was somehow lost in it).
 

Hi,
first I am not DD or DM, I am just learning packaging, so some of the
following may be incorrect.
Sorry if some of this may have already been mentioned/resolved, I am
only following this list since a short while.


some issues with the copyright:
Mikhail Barg (include/mgl/mgl_pas.pas)
and Алексей Балакин (texinfo/mgl_ru.texi) do not appear in the
debian/copyright file
(note this was just a brief check, may miss further persons)

licensecheck reports: GPL (v2 or later) (with incorrect FSF address)
this must probably be resolved with upstream.


I also have some more cosmetic comments on the package:

- you have changed it to source format 3.0 (quilt). this means you can
drop the dependency on quilt in the control file and you can remove the
patch targets and quilt references from the rules files.


- line 76 in rules file:
gzip -9 debian/mathgl/usr/share/man/man1/*.1
doesn't dh_installman man handle compression for you?

- the gigantic patch in debian/patches is probably hard to handle:
cat patches/debian-changes-1.11-1 | diffstat
 Makefile.in   |6 
 aclocal.m4| 7971
--
 configure |   36 
 examples/Makefile.in  |6 
 include/Makefile.in   |6 
 lang/Makefile.in  |6 
 lang/numpy.i  |2 
 mathgl-1.11/texinfo/mgl_ru.info-1 | 6077 
 mathgl-1.11/texinfo/mgl_ru.info-2 |  889 
 mgl/Makefile.in   |6 
 texinfo/Makefile.in   |6 
 texinfo/mathgl_en.texi|8 
 texinfo/mathgl_ru.texi|8 
 texinfo/mgl_en.texi   |8 
 texinfo/mgl_ru.texi   |8 
 texinfo/png/Makefile.in   |6 
 utils/Makefile.in |6 
 widgets/Makefile.in   |6 
 18 files changed, 7053 insertions(+), 8008 deletions(-)

you can probably just remove the aclocal.m4 in the rules file (is this
even necessary with dh_autoreconf?)
and maybe split the rest of the patch in smaller per-topic chunks.

thanks for the work on the package.

Best Regards,
Julian Taylor


signature.asc
Description: This is a digitally signed message part