Processed: Re: Bug#1056531: cython 3.x (for Python 3.12)

2023-12-15 Thread Debian Bug Tracking System
Processing commands for cont...@bugs.debian.org:

> severity 1056531 important
Bug #1056531 [src:spyder-kernels] spyder-kernels autopkg tests fail with Python 
3.12
Severity set to 'important' from 'serious'
> retitle 1056531 spyder-kernels: cannot handle Cython files with Python 3.12
Bug #1056531 [src:spyder-kernels] spyder-kernels autopkg tests fail with Python 
3.12
Changed Bug title to 'spyder-kernels: cannot handle Cython files with Python 
3.12' from 'spyder-kernels autopkg tests fail with Python 3.12'.
> thanks
Stopping processing here.

Please contact me if you need assistance.
-- 
1056531: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=1056531
Debian Bug Tracking System
Contact ow...@bugs.debian.org with problems



Bug#1056531: cython 3.x (for Python 3.12)

2023-12-15 Thread Julian Gilbey
severity 1056531 important
retitle 1056531 spyder-kernels: cannot handle Cython files with Python 3.12
thanks

Update on this bug.  cython3-legacy and the current cython3 (which
I've just updated to not fail the autopkgtest tests with Python 3.12)
do not have working pyximport modules.  So I'm skipping the tests in
this package which depend on that for the time being.  When Cython 3.x
is in testing, I will re-enable those tests and depend on the new
version of cython3.  In the meantime, Spyder will have difficulties
with Cython source files if Python 3.12 is the default Python.  As
long as Python 3.11 is the default, it will be fine.

So I'm lowering the severity to important for now, and will close it
once I can depend on Cython 3.x.

Best wishes,

   Julian



Bug#1056531: cython 3.x (for Python 3.12)

2023-12-11 Thread Julian Gilbey
On Mon, Dec 11, 2023 at 08:06:41PM +0100, Matthias Klose wrote:
> > [...]
> > Excellent - I didn't know about that.  Are you OK for me to upload
> > versions of cython and cython-legacy which depend on this to fix the
> > Python 3.12 breakage?
> 
> not for cython, which won't need that soonish for 3.0.x.  and if you have to
> update the b-d to cython3-legacy, why not add the zombie-imp dependency as
> well manually for the few packages that need it?

The problem is not with other packages importing imp (which need to
be fixed in such a case), rather that pyximport (in
cython/cython-legacy) imports imp.  So it's cython that needs to be
patched.

I'm wondering what the provisional timescale for cython 3.0.x is?
Should I just leave my package with an autopkgtest failure until
cython 3.0.x is in unstable or ideally testing?  That's why I was
thinking of also patching cython in the short term until we are ready
for cython 3.0.x to enter unstable.

Best wishes,

   Julian



Bug#1056531: cython 3.x (for Python 3.12)

2023-12-11 Thread Matthias Klose

On 11.12.23 19:55, Julian Gilbey wrote:

On Mon, Dec 11, 2023 at 04:34:17PM +0100, Matthias Klose wrote:

On 11.12.23 16:19, Julian Gilbey wrote:

On Mon, Dec 11, 2023 at 08:09:31AM +0100, Matthias Klose wrote:

[...]
You could package a non-conflicting cython-legacy, however that would
require more changes, also how to build it.


Hi Matthias,

Unfortunately, at least some of cython3-legacy doesn't currently work
with Python 3.12, and is the primary cause of (at least) #1056531.
cython3 provides the pyximport module, and that uses the imp module
which has been removed from Python 3.12.

Two possible ways forward on this particular bug:

- Disable all of the cython tests for this package for the time being,
until cython 3.x migrates to testing - this is simple and effective.

- Patch cython3-legacy to use importlib rather than imp.  This is
probably a good thing to do anyway.  (It may also be good to do this
with cython3 version 0.x currently in testing/unstable until cython
3.x is able to be uploaded to unstable.)  Then have my package's
autopkgtest depend on cython3-legacy (unless cython3 0.x is also
patched).


I won't working on this. Have you tried to depend on the python3-zombie-imp
instead?


Excellent - I didn't know about that.  Are you OK for me to upload
versions of cython and cython-legacy which depend on this to fix the
Python 3.12 breakage?


not for cython, which won't need that soonish for 3.0.x.  and if you 
have to update the b-d to cython3-legacy, why not add the zombie-imp 
dependency as well manually for the few packages that need it?




Bug#1056531: cython 3.x (for Python 3.12)

2023-12-11 Thread Julian Gilbey
On Mon, Dec 11, 2023 at 04:34:17PM +0100, Matthias Klose wrote:
> On 11.12.23 16:19, Julian Gilbey wrote:
> > On Mon, Dec 11, 2023 at 08:09:31AM +0100, Matthias Klose wrote:
> > > [...]
> > > You could package a non-conflicting cython-legacy, however that would
> > > require more changes, also how to build it.
> > 
> > Hi Matthias,
> > 
> > Unfortunately, at least some of cython3-legacy doesn't currently work
> > with Python 3.12, and is the primary cause of (at least) #1056531.
> > cython3 provides the pyximport module, and that uses the imp module
> > which has been removed from Python 3.12.
> > 
> > Two possible ways forward on this particular bug:
> > 
> > - Disable all of the cython tests for this package for the time being,
> >until cython 3.x migrates to testing - this is simple and effective.
> > 
> > - Patch cython3-legacy to use importlib rather than imp.  This is
> >probably a good thing to do anyway.  (It may also be good to do this
> >with cython3 version 0.x currently in testing/unstable until cython
> >3.x is able to be uploaded to unstable.)  Then have my package's
> >autopkgtest depend on cython3-legacy (unless cython3 0.x is also
> >patched).
> 
> I won't working on this. Have you tried to depend on the python3-zombie-imp
> instead?

Excellent - I didn't know about that.  Are you OK for me to upload
versions of cython and cython-legacy which depend on this to fix the
Python 3.12 breakage?

Best wishes,

   Julian



Bug#1056531: cython 3.x (for Python 3.12)

2023-12-11 Thread Matthias Klose

On 11.12.23 16:19, Julian Gilbey wrote:

On Mon, Dec 11, 2023 at 08:09:31AM +0100, Matthias Klose wrote:

[...]
You could package a non-conflicting cython-legacy, however that would
require more changes, also how to build it.


Hi Matthias,

Unfortunately, at least some of cython3-legacy doesn't currently work
with Python 3.12, and is the primary cause of (at least) #1056531.
cython3 provides the pyximport module, and that uses the imp module
which has been removed from Python 3.12.

Two possible ways forward on this particular bug:

- Disable all of the cython tests for this package for the time being,
   until cython 3.x migrates to testing - this is simple and effective.

- Patch cython3-legacy to use importlib rather than imp.  This is
   probably a good thing to do anyway.  (It may also be good to do this
   with cython3 version 0.x currently in testing/unstable until cython
   3.x is able to be uploaded to unstable.)  Then have my package's
   autopkgtest depend on cython3-legacy (unless cython3 0.x is also
   patched).


I won't working on this. Have you tried to depend on the 
python3-zombie-imp instead?




Bug#1056531: cython 3.x (for Python 3.12)

2023-12-11 Thread Julian Gilbey
On Mon, Dec 11, 2023 at 08:09:31AM +0100, Matthias Klose wrote:
> [...]
> You could package a non-conflicting cython-legacy, however that would
> require more changes, also how to build it.

Hi Matthias,

Unfortunately, at least some of cython3-legacy doesn't currently work
with Python 3.12, and is the primary cause of (at least) #1056531.
cython3 provides the pyximport module, and that uses the imp module
which has been removed from Python 3.12.

Two possible ways forward on this particular bug:

- Disable all of the cython tests for this package for the time being,
  until cython 3.x migrates to testing - this is simple and effective.

- Patch cython3-legacy to use importlib rather than imp.  This is
  probably a good thing to do anyway.  (It may also be good to do this
  with cython3 version 0.x currently in testing/unstable until cython
  3.x is able to be uploaded to unstable.)  Then have my package's
  autopkgtest depend on cython3-legacy (unless cython3 0.x is also
  patched).

Thoughts?

   Julian