Re: py-curses broken for python310

[Thomas Klausner <wiz%NetBSD.org@localhost>]

On Wed, Dec 08, 2021 at 03:24:50PM +0000, nia wrote:
> On Tue, Dec 07, 2021 at 01:01:02PM +0000, nia wrote:
> > On Tue, Dec 07, 2021 at 12:04:02PM +0100, Thomas Klausner wrote:
> > > On Mon, Dec 06, 2021 at 08:18:44PM +0100, Adam wrote:
> > > > > Hi!
> > > > > 
> > > > > For lib/55220 I wanted to patch py-curses, i.e. the python39 code.
> > > > > This worked and I committed it.
> > > > > 
> > > > > Then I wanted to do the same for python310 but the build failed.
> > > > > This happens even before my (attached) patch.
> > > > > 
> > > > > creating build/temp.netbsd-9.99.92-amd64-3.10/Modules
> > > > > gcc -Wno-unused-result -Wsign-compare -DNDEBUG -O2 -g -fPIC -D_FORTIFY_SOURCE=2 -fstack-check -I/usr/include -I/usr/pkg/include -O2 -g -fPIC -D_FORTIFY_SOURCE=2 -fstack-check -I/usr/include -I/usr/pkg/include -O2 -g -fPIC -D_FORTIFY_SOURCE=2 -fstack-check -I/usr/include -I/usr/pkg/include -O2 -g -fPIC -D_FORTIFY_SOURCE=2 -fstack-check -I/usr/include -I/usr/pkg/include/python3.10 -g -I/usr/include -I/usr/pkg/include/python3.10 -fPIC -DHAVE_NCURSES_H -I/include -I/usr/pkg/include/python3.10 -c Modules/_cursesmodule.c -o build/temp.netbsd-9.99.92-amd64-3.10/Modules/_cursesmodule.o
> > > > > Modules/_cursesmodule.c:106:10: fatal error: pycore_long.h: No such file or directory
> > > > >  106 | #include "pycore_long.h"          // _PyLong_GetZero()
> > > > >      |          ^~~~~~~~~~~~~~~
> > > > > compilation terminated.
> > > > > error: command '/scratch/devel/py-curses/work/.cwrapper/bin/gcc' failed with exit code 1
> > > > > *** Error code 1
> > > > > 
> > > > > 
> > > > > Adding the path where pycore_long.h is installed to the compiler
> > > > > include search paths just causes a different error.
> > > > > 
> > > > > 1. can someone please fix that? or
> > > > > 2. should we just merge py-curses into python?
> > > > > 
> > > > > 2 adds a curses dependency to python. On NetBSD that's a no-op, on
> > > > > other system it causes ncurses to be built.
> > > > 
> > > > I would go for 2.
> > > 
> > > I agree.
> > > 
> > > I'll merge them after the branch has been cut.
> > >  Thomas
> > 
> > I'd rather we didn't cause conflicts during binary package updates.
> 
> Thinking about this some more, I think it would be best to:
> 
> - keep py-curses for versions <=3.9
> - mark py-curses incompatible with later versions
> - merge py-curses into lang/python310

I think that's what I'll go for.

Just for the record, I'd prefer proper SUPERSEDES support in pkg_add
(i.e. if package foo supersedes bar, 'pkg_add foo' makes all
dependencies of bar point to foo and deletes bar before installing
foo; and later 'pkg_add doodad' knows that foo supplies a bar
dependency). Perhaps pkgin would also need it.

The variable was introduced a really long time ago, but noone ever
added proper support by the infrastructure for it :(
 Thomas