Re: [Cocci] Python 3.10 again: _Py_fopen deprecated
On Sat, Jan 09, 2021 at 12:10:18AM +0100, Thierry Martinez wrote: > Hi, Richard. > > Thank you very much for your two successive reports about > incompatibilities between pyml and Python 3.10. They should have been > fixed now. > > Richard: > >> What I don't understand from the pyxml code is why we use these > >> internal Python functions at all, instead of calling regular C > >> functions like fopen etc. In fact it seems like for Python 2 we did > >> call fopen ... > > Indeed, for Python 2, there is a fallback to fopen since Py_fopen is not > available. When Py_fopen or Py_wfopen are available, we prefer to use > them because they handle the Windows special case. Oh I see, that makes sense. Rich. > Best regards. > -- > Thierry. > > Julia Lawall : > > On Tue, 5 Jan 2021, Richard W.M. Jones wrote: > > > >> Firstly a gentle reminder that there's a patch waiting to be applied: > >> https://systeme.lip6.fr/pipermail/cocci/2020-November/thread.html#8398 > >> > >> Different from that patch, but still related to Python 3.10, we've got > >> another bug report here: > >> https://bugzilla.redhat.com/show_bug.cgi?id=1912931 > >> > >> This time _Py_fopen has been deprecated, replaced by _Py_wfopen or > >> _Py_fopen_obj. It's unclear which is better. The two functions are > >> documented here: > >> https://github.com/python/cpython/blob/master/Python/fileutils.c#L1418 > >> > >> What I don't understand from the pyxml code is why we use these > >> internal Python functions at all, instead of calling regular C > >> functions like fopen etc. In fact it seems like for Python 2 we did > >> call fopen ... > > > > Everything should be up to date now on github. Thanks for your help. > > Thierry will contact you directly about the choice of fopen. > > > > julia > > > >> > >> Rich. > >> > >> -- > >> Richard Jones, Virtualization Group, Red Hat > >> http://people.redhat.com/~rjones > >> Read my programming and virtualization blog: http://rwmj.wordpress.com > >> Fedora Windows cross-compiler. Compile Windows programs, test, and > >> build Windows installers. Over 100 libraries supported. > >> http://fedoraproject.org/wiki/MinGW > >> > >> ___ > >> Cocci mailing list > >> Cocci@systeme.lip6.fr > >> https://systeme.lip6.fr/mailman/listinfo/cocci > >> > > ___ > > Cocci mailing list > > Cocci@systeme.lip6.fr > > https://systeme.lip6.fr/mailman/listinfo/cocci -- Richard Jones, Virtualization Group, Red Hat http://people.redhat.com/~rjones Read my programming and virtualization blog: http://rwmj.wordpress.com virt-p2v converts physical machines to virtual machines. Boot with a live CD or over the network (PXE) and turn machines into KVM guests. http://libguestfs.org/virt-v2v ___ Cocci mailing list Cocci@systeme.lip6.fr https://systeme.lip6.fr/mailman/listinfo/cocci
Re: [Cocci] Python 3.10 again: _Py_fopen deprecated
Hi, Richard. Thank you very much for your two successive reports about incompatibilities between pyml and Python 3.10. They should have been fixed now. Richard: >> What I don't understand from the pyxml code is why we use these >> internal Python functions at all, instead of calling regular C >> functions like fopen etc. In fact it seems like for Python 2 we did >> call fopen ... Indeed, for Python 2, there is a fallback to fopen since Py_fopen is not available. When Py_fopen or Py_wfopen are available, we prefer to use them because they handle the Windows special case. Best regards. -- Thierry. Julia Lawall : > On Tue, 5 Jan 2021, Richard W.M. Jones wrote: > >> Firstly a gentle reminder that there's a patch waiting to be applied: >> https://systeme.lip6.fr/pipermail/cocci/2020-November/thread.html#8398 >> >> Different from that patch, but still related to Python 3.10, we've got >> another bug report here: >> https://bugzilla.redhat.com/show_bug.cgi?id=1912931 >> >> This time _Py_fopen has been deprecated, replaced by _Py_wfopen or >> _Py_fopen_obj. It's unclear which is better. The two functions are >> documented here: >> https://github.com/python/cpython/blob/master/Python/fileutils.c#L1418 >> >> What I don't understand from the pyxml code is why we use these >> internal Python functions at all, instead of calling regular C >> functions like fopen etc. In fact it seems like for Python 2 we did >> call fopen ... > > Everything should be up to date now on github. Thanks for your help. > Thierry will contact you directly about the choice of fopen. > > julia > >> >> Rich. >> >> -- >> Richard Jones, Virtualization Group, Red Hat http://people.redhat.com/~rjones >> Read my programming and virtualization blog: http://rwmj.wordpress.com >> Fedora Windows cross-compiler. Compile Windows programs, test, and >> build Windows installers. Over 100 libraries supported. >> http://fedoraproject.org/wiki/MinGW >> >> ___ >> Cocci mailing list >> Cocci@systeme.lip6.fr >> https://systeme.lip6.fr/mailman/listinfo/cocci >> > ___ > Cocci mailing list > Cocci@systeme.lip6.fr > https://systeme.lip6.fr/mailman/listinfo/cocci ___ Cocci mailing list Cocci@systeme.lip6.fr https://systeme.lip6.fr/mailman/listinfo/cocci
Re: [Cocci] Python 3.10 again: _Py_fopen deprecated
On Tue, 5 Jan 2021, Richard W.M. Jones wrote: > Firstly a gentle reminder that there's a patch waiting to be applied: > https://systeme.lip6.fr/pipermail/cocci/2020-November/thread.html#8398 > > Different from that patch, but still related to Python 3.10, we've got > another bug report here: > https://bugzilla.redhat.com/show_bug.cgi?id=1912931 > > This time _Py_fopen has been deprecated, replaced by _Py_wfopen or > _Py_fopen_obj. It's unclear which is better. The two functions are > documented here: > https://github.com/python/cpython/blob/master/Python/fileutils.c#L1418 > > What I don't understand from the pyxml code is why we use these > internal Python functions at all, instead of calling regular C > functions like fopen etc. In fact it seems like for Python 2 we did > call fopen ... Everything should be up to date now on github. Thanks for your help. Thierry will contact you directly about the choice of fopen. julia > > Rich. > > -- > Richard Jones, Virtualization Group, Red Hat http://people.redhat.com/~rjones > Read my programming and virtualization blog: http://rwmj.wordpress.com > Fedora Windows cross-compiler. Compile Windows programs, test, and > build Windows installers. Over 100 libraries supported. > http://fedoraproject.org/wiki/MinGW > > ___ > Cocci mailing list > Cocci@systeme.lip6.fr > https://systeme.lip6.fr/mailman/listinfo/cocci > ___ Cocci mailing list Cocci@systeme.lip6.fr https://systeme.lip6.fr/mailman/listinfo/cocci
[Cocci] Python 3.10 again: _Py_fopen deprecated
Firstly a gentle reminder that there's a patch waiting to be applied: https://systeme.lip6.fr/pipermail/cocci/2020-November/thread.html#8398 Different from that patch, but still related to Python 3.10, we've got another bug report here: https://bugzilla.redhat.com/show_bug.cgi?id=1912931 This time _Py_fopen has been deprecated, replaced by _Py_wfopen or _Py_fopen_obj. It's unclear which is better. The two functions are documented here: https://github.com/python/cpython/blob/master/Python/fileutils.c#L1418 What I don't understand from the pyxml code is why we use these internal Python functions at all, instead of calling regular C functions like fopen etc. In fact it seems like for Python 2 we did call fopen ... Rich. -- Richard Jones, Virtualization Group, Red Hat http://people.redhat.com/~rjones Read my programming and virtualization blog: http://rwmj.wordpress.com Fedora Windows cross-compiler. Compile Windows programs, test, and build Windows installers. Over 100 libraries supported. http://fedoraproject.org/wiki/MinGW ___ Cocci mailing list Cocci@systeme.lip6.fr https://systeme.lip6.fr/mailman/listinfo/cocci