Bug#682652: cython: Cython might cause problematic code when trying to build using gcc-4.7

July 24th, 2012 - 06:30 am ET by Andreas Tille | Report spam
Package: cython
Severity: normal

Hi,

there is some evidence that cython from testing/unstable as well as from
experimental (!) generates code which does not compile when using
gcc-4.7. I have no idea whether packages in Wheezy might be affected -
I guess autobuild-test would have unrevealed this, but it seems it is
not a specific issue of the software which I asked for help at
debian-mentors list. To get the full story it is advisable to read
this mail:

Help needed in gcc-4.7 - cython issue:
http://lists.debian.org/debian-ment...00408.html

In the response[0] Sebastian Ramacher <sebastian@ramacher.at> said:

As it seems, the fix has been merged after the 0.16 release. The patch to solve
this in cython can be found at [1].

Until this is fixed one can try to add spaces between PY_FORMAT_SIZE_T and the
quotation marks in the file generated by cython.

[1] https://github.com/cython/cython/pull/109


So it would be nice if you might consider an upload - at least to
experimental - featuring the patch which would enable building this
project using gcc-4.7.

In case you have different plans because no other released package is
affected by this problem please let me know some estimated time until
you consider fixing the problem. If it might take longer than say about
1 month I would consider releasing the packagin question for the moment
without Cython support or explicitely forcing gcc-4.6.

Kind regards and thanks for maintaining Cython

Andreas.


[0] http://lists.debian.org/debian-ment...00411.html


Debian Release: 6.0.5
Architecture: i386 (i686)

Kernel: Linux 2.6.36-xenU-4814-i386 (SMP w/1 CPU core)
Locale: LANG=de_DE.UTF-8, LC_CTYPE=de_DE.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash


To UNSUBSCRIBE, email to debian-bugs-dist-REQUEST@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmaster@lists.debian.org
email Follow the discussionReplies 6 repliesReplies Make a reply

Similar topics

Replies

#1 Yaroslav Halchenko
July 24th, 2012 - 09:10 am ET | Report spam
I will check if the recent 0.17.beta1 is not too bad ( ;-) ) and
might upload it to experimental later on today/tomorrow

Cheers

On Tue, 24 Jul 2012, Andreas Tille wrote:

In case you have different plans because no other released package is
affected by this problem please let me know some estimated time until
you consider fixing the problem. If it might take longer than say about
1 month I would consider releasing the packagin question for the moment
without Cython support or explicitely forcing gcc-4.6.

Kind regards and thanks for maintaining Cython

Andreas.


[0] http://lists.debian.org/debian-ment...00411.html


Yaroslav O. Halchenko
Postdoctoral Fellow, Department of Psychological and Brain Sciences
Dartmouth College, 419 Moore Hall, Hinman Box 6207, Hanover, NH 03755
Phone: +1 (603) 646-9834 Fax: +1 (603) 646-1419
WWW: http://www.linkedin.com/in/yarik


To UNSUBSCRIBE, email to
with a subject of "unsubscribe". Trouble? Contact
Replies Reply to this message
#2 Yaroslav Halchenko
July 27th, 2012 - 03:20 pm ET | Report spam
so -- here you go -- accepted to experimental ;) now 1.7~rc1 is
in experimental... would still fail on some ports, but now should be
sufficient I guess for your needs ;)

On Tue, 24 Jul 2012, Andreas Tille wrote:

Package: cython
Severity: normal

Hi,

there is some evidence that cython from testing/unstable as well as from
experimental (!) generates code which does not compile when using
gcc-4.7. I have no idea whether packages in Wheezy might be affected -
I guess autobuild-test would have unrevealed this, but it seems it is
not a specific issue of the software which I asked for help at
debian-mentors list. To get the full story it is advisable to read
this mail:

Help needed in gcc-4.7 - cython issue:
http://lists.debian.org/debian-ment...00408.html

In the response[0] Sebastian Ramacher said:

As it seems, the fix has been merged after the 0.16 release. The patch to solve
this in cython can be found at [1].

Until this is fixed one can try to add spaces between PY_FORMAT_SIZE_T and the
quotation marks in the file generated by cython.

[1] https://github.com/cython/cython/pull/109


So it would be nice if you might consider an upload - at least to
experimental - featuring the patch which would enable building this
project using gcc-4.7.

In case you have different plans because no other released package is
affected by this problem please let me know some estimated time until
you consider fixing the problem. If it might take longer than say about
1 month I would consider releasing the packagin question for the moment
without Cython support or explicitely forcing gcc-4.6.

Kind regards and thanks for maintaining Cython

Andreas.


[0] http://lists.debian.org/debian-ment...00411.html


Yaroslav O. Halchenko
Postdoctoral Fellow, Department of Psychological and Brain Sciences
Dartmouth College, 419 Moore Hall, Hinman Box 6207, Hanover, NH 03755
Phone: +1 (603) 646-9834 Fax: +1 (603) 646-1419
WWW: http://www.linkedin.com/in/yarik


To UNSUBSCRIBE, email to
with a subject of "unsubscribe". Trouble? Contact
Replies Reply to this message
#3 Andreas Tille
July 27th, 2012 - 03:40 pm ET | Report spam
Hi Yaroslav,

thanks for the upload and the information. This will enable us
uploading python-biom-format at least to experimental. It would be
really great if you could also drop me a note once version >= 0.17 would
be moved to unstable to let this package also migrate to unstable.

Have a nice weekend

Andreas.

On Fri, Jul 27, 2012 at 03:08:31PM -0400, Yaroslav Halchenko wrote:
so -- here you go -- accepted to experimental ;) now 1.7~rc1 is
in experimental... would still fail on some ports, but now should be
sufficient I guess for your needs ;)

On Tue, 24 Jul 2012, Andreas Tille wrote:

> Package: cython
> Severity: normal

> Hi,

> there is some evidence that cython from testing/unstable as well as from
> experimental (!) generates code which does not compile when using
> gcc-4.7. I have no idea whether packages in Wheezy might be affected -
> I guess autobuild-test would have unrevealed this, but it seems it is
> not a specific issue of the software which I asked for help at
> debian-mentors list. To get the full story it is advisable to read
> this mail:

> Help needed in gcc-4.7 - cython issue:
> http://lists.debian.org/debian-ment...00408.html

> In the response[0] Sebastian Ramacher said:

> As it seems, the fix has been merged after the 0.16 release. The patch to solve
> this in cython can be found at [1].

> Until this is fixed one can try to add spaces between PY_FORMAT_SIZE_T and the
> quotation marks in the file generated by cython.

> [1] https://github.com/cython/cython/pull/109


> So it would be nice if you might consider an upload - at least to
> experimental - featuring the patch which would enable building this
> project using gcc-4.7.

> In case you have different plans because no other released package is
> affected by this problem please let me know some estimated time until
> you consider fixing the problem. If it might take longer than say about
> 1 month I would consider releasing the packagin question for the moment
> without Cython support or explicitely forcing gcc-4.6.

> Kind regards and thanks for maintaining Cython

> Andreas.


> [0] http://lists.debian.org/debian-ment...00411.html
Yaroslav O. Halchenko
Postdoctoral Fellow, Department of Psychological and Brain Sciences
Dartmouth College, 419 Moore Hall, Hinman Box 6207, Hanover, NH 03755
Phone: +1 (603) 646-9834 Fax: +1 (603) 646-1419
WWW: http://www.linkedin.com/in/yarik




http://fam-tille.de


To UNSUBSCRIBE, email to
with a subject of "unsubscribe". Trouble? Contact
Replies Reply to this message
#4 Yaroslav Halchenko
July 27th, 2012 - 03:50 pm ET | Report spam
not after the release I guess... if I were you I would have just
pregenerated .c files and shipped those for the systems which lack up to
date cython (ftpmasters might be of different opinion... but there are
pkgs in archive which just rely on pregenerated .c altogether)

see e.g. pandas where I do the same for easy backporting

On Fri, 27 Jul 2012, Andreas Tille wrote:

Hi Yaroslav,

thanks for the upload and the information. This will enable us
uploading python-biom-format at least to experimental. It would be
really great if you could also drop me a note once version >= 0.17 would
be moved to unstable to let this package also migrate to unstable.

Have a nice weekend

Andreas.


Yaroslav O. Halchenko
Postdoctoral Fellow, Department of Psychological and Brain Sciences
Dartmouth College, 419 Moore Hall, Hinman Box 6207, Hanover, NH 03755
Phone: +1 (603) 646-9834 Fax: +1 (603) 646-1419
WWW: http://www.linkedin.com/in/yarik


To UNSUBSCRIBE, email to
with a subject of "unsubscribe". Trouble? Contact
Replies Reply to this message
#5 Andreas Tille
July 27th, 2012 - 04:10 pm ET | Report spam
On Fri, Jul 27, 2012 at 03:43:57PM -0400, Yaroslav Halchenko wrote:
not after the release I guess...



Not *after* means *before* the release. ;-))
OK, understood (and expected) the motivation.

if I were you I would have just
pregenerated .c files and shipped those for the systems which lack up to
date cython (ftpmasters might be of different opinion... but there are
pkgs in archive which just rely on pregenerated .c altogether)

see e.g. pandas where I do the same for easy backporting



I admit I considered the same but I hesitated because I have no idea
whether the generated C files might be arch independant. If the
resulting C++ files would be different per architecture this could
case problems - if you would confirm that this is not the case I
would go this path.

Kind regards

Andreas.

http://fam-tille.de


To UNSUBSCRIBE, email to
with a subject of "unsubscribe". Trouble? Contact
Replies Reply to this message
Help Create a new topicNext page Replies Make a reply
Search Make your own search