Mailing List Archive

On Fri, Jan 24, 2020 at 9:07 AM Christian Marillat <
marillat@deb-multimedia.org> wrote:

> Hi,
>
> As you know python 2 has been removed from some distribution (Debian)
> and now MythTV 30 is non installable.
>
> Do have plan to add python 3 support to fixes/30 branch as the code is
> already in master ?
>

For Fedora RPM Fusion I maintain a few patches which seem to work, the last
of which I pulled from a Trac ticket:

https://pkgs.rpmfusion.org/cgit/free/mythtv.git/tree/mythtv-py3_configure.patch
https://pkgs.rpmfusion.org/cgit/free/mythtv.git/tree/mythtv-py3_urllib.patch
https://pkgs.rpmfusion.org/cgit/free/mythtv.git/tree/mythtv-python3.patch

Thanks,
Richard

On 24 janv. 2020 09:21, Richard Shaw <hobbes1069@gmail.com> wrote:

> On Fri, Jan 24, 2020 at 9:07 AM Christian Marillat <
> marillat@deb-multimedia.org> wrote:
>
>> Hi,
>>
>> As you know python 2 has been removed from some distribution (Debian)
>> and now MythTV 30 is non installable.
>>
>> Do have plan to add python 3 support to fixes/30 branch as the code is
>> already in master ?
>>
>
> For Fedora RPM Fusion I maintain a few patches which seem to work, the last
> of which I pulled from a Trac ticket:
>
> https://pkgs.rpmfusion.org/cgit/free/mythtv.git/tree/mythtv-py3_configure.patch
> https://pkgs.rpmfusion.org/cgit/free/mythtv.git/tree/mythtv-py3_urllib.patch
> https://pkgs.rpmfusion.org/cgit/free/mythtv.git/tree/mythtv-python3.patch

From this track https://github.com/MythTV/mythtv/pull/182

and the python code from this repo ?

https://github.com/rcrdnalor/mythtv

Christian
_______________________________________________
mythtv-dev mailing list
mythtv-dev@mythtv.org
http://lists.mythtv.org/mailman/listinfo/mythtv-dev
http://wiki.mythtv.org/Mailing_List_etiquette
MythTV Forums: https://forum.mythtv.org

On Fri, Jan 24, 2020 at 9:27 AM Christian Marillat <
marillat@deb-multimedia.org> wrote:

> On 24 janv. 2020 09:21, Richard Shaw <hobbes1069@gmail.com> wrote:
>
> > On Fri, Jan 24, 2020 at 9:07 AM Christian Marillat <
> > marillat@deb-multimedia.org> wrote:
> >
> >> Hi,
> >>
> >> As you know python 2 has been removed from some distribution (Debian)
> >> and now MythTV 30 is non installable.
> >>
> >> Do have plan to add python 3 support to fixes/30 branch as the code is
> >> already in master ?
> >>
> >
> > For Fedora RPM Fusion I maintain a few patches which seem to work, the
> last
> > of which I pulled from a Trac ticket:
> >
> >
> https://pkgs.rpmfusion.org/cgit/free/mythtv.git/tree/mythtv-py3_configure.patch
> >
> https://pkgs.rpmfusion.org/cgit/free/mythtv.git/tree/mythtv-py3_urllib.patch
> >
> https://pkgs.rpmfusion.org/cgit/free/mythtv.git/tree/mythtv-python3.patch
>
> From this track https://github.com/MythTV/mythtv/pull/182


Looks correct.

Thanks,
Richard

On 24/01/2020 15:31, Richard Shaw wrote:
> On Fri, Jan 24, 2020 at 9:27 AM Christian Marillat
> <marillat@deb-multimedia.org <mailto:marillat@deb-multimedia.org>> wrote:
>
> On 24 janv. 2020 09:21, Richard Shaw <hobbes1069@gmail.com
> <mailto:hobbes1069@gmail.com>> wrote:
>
> > On Fri, Jan 24, 2020 at 9:07 AM Christian Marillat <
> > marillat@deb-multimedia.org <mailto:marillat@deb-multimedia.org>>
> wrote:
> >
> >> Hi,
> >>
> >> As you know python 2 has been removed from some distribution
> (Debian)
> >> and now MythTV 30 is non installable.
> >>
> >> Do have plan to add python 3 support to fixes/30 branch as the
> code is
> >> already in master ?
> >>
> >
> > For Fedora RPM Fusion I maintain a few patches which seem to
> work, the last
> > of which I pulled from a Trac ticket:
> >
> >
> https://pkgs.rpmfusion.org/cgit/free/mythtv.git/tree/mythtv-py3_configure.patch
> >
> https://pkgs.rpmfusion.org/cgit/free/mythtv.git/tree/mythtv-py3_urllib.patch
> >
> https://pkgs.rpmfusion.org/cgit/free/mythtv.git/tree/mythtv-python3.patch
>
> >From this track https://github.com/MythTV/mythtv/pull/182
>
>
> Looks correct.
>
> Thanks,
> Richard
>

And you might want this too? Upcoming for MythArchive.

https://code.mythtv.org/trac/ticket/13562#comment:20
_______________________________________________
mythtv-dev mailing list
mythtv-dev@mythtv.org
http://lists.mythtv.org/mailman/listinfo/mythtv-dev
http://wiki.mythtv.org/Mailing_List_etiquette
MythTV Forums: https://forum.mythtv.org

On 24/01/2020 15:06, Christian Marillat wrote:
> Hi,
>
> As you know python 2 has been removed from some distribution (Debian)
> and now MythTV 30 is non installable.
>
> Do have plan to add python 3 support to fixes/30 branch as the code is
> already in master ?
>

We are also hoping to cut fixes/31 soon....


Regards
Stuart
_______________________________________________
mythtv-dev mailing list
mythtv-dev@mythtv.org
http://lists.mythtv.org/mailman/listinfo/mythtv-dev
http://wiki.mythtv.org/Mailing_List_etiquette
MythTV Forums: https://forum.mythtv.org

On Fri, Jan 24, 2020 at 9:51 AM John Pilkington <johnpilk222@gmail.com>
wrote:

>
> And you might want this too? Upcoming for MythArchive.
>
> https://code.mythtv.org/trac/ticket/13562#comment:20


That resulted in a build error...

mythburn.cpp: In member function 'void MythBurn::runScript()':
mythburn.cpp:948:19: error: 'PYTHON_EXE' was not declared in this scope
948 | commandline = PYTHON_EXE;
| ^~~~~~~~~~


Thanks,
Richard

On 1/24/20 10:46 AM, Richard Shaw wrote:
>
> That resulted in a build error...
>
> mythburn.cpp: In member function 'void MythBurn::runScript()':
> mythburn.cpp:948:19: error: 'PYTHON_EXE' was not declared in this scope
>   948 |     commandline = PYTHON_EXE;
>       |                   ^~~~~~~~~~

Support for Python v3 is intended for MythTV v31 only. The error above is
because at ./configure time, the Python executable is set based on the
--python switch (or its default of python3.) This is used in a number of
mythProgramNames that call Python scripts as above.

There are a lot more commits above Pull Request 182. Quick search:

0763e32999 Python Bindings: support --systemd-journal logging
186e24a018 Python Bindings: use configurable "python" choices
bafc80d9ff Python Bindings: Fix 3 exceptions and remove unused query
f1d8870c1d Python Bindings: adds python3 compatibility to Music scripts
871accb425 Add the 'recordedfile' table to python bindings and make use of it
a2ae8119b4 Silence warning on exit of python if a mysql cursor is in use
d6af02f8e9 Add Compatibility to Python3 for metaclasses
fcc4990946 MythTV: Add compatibility to python3 for '__repr__' methods
4d9c4ea5ef TMDB3: Add compatibility to python3 for '__repr__' methods
623778ee2c Update Movie Grabber tmdb3.py to meet python3 compatibility, again
7234e888f5 Python Bindings: Remove deprecation warnings (python3)
9be0f39afa Fix compatibility to python3 in 'VideoGrabber.grabInetref'.
4eb515926f Fix missing conversion to python3 in mythwikiscripts.
8ec6a1d1f4 Make mythpython script compatible to python3.
bca53f0e88 Add compatibility to python3 to tmdb3 module.
834cbb0473 Make mythwikiscripts compatible to python3.
e8ab22178c Fix Python Bindings to be compatible to python3 as well.
9127e3436b Fix mysql connection of the Python Bindings for python3.
378cfe017a Fix check in Python's Job.fromProgram classmethod.
d2d3cb8f76 Python Bindings Recorded.formatPath() bails if airdate is not set.
1393503938 Various fixes of MythTV Python Bindings in mythproto.py.
35ab79122c Allow storage group paths without trailing slashes in Python Binding findfile method.
4572f98040 Fix MythSytemEvent class of MythTV's python bindings
771796578b Adapt mythpython's MythXML class to new Services/API.
6333c3bde9 Fix logging in Mythtv's python bindings msearch.py:
8baf4db3f0 Correct typo in MythTV's python binding datetime.fromRfc().
0b5b6b9e46 Update MythTV's python binding 'utility/dt.py' according patch from Ticket #13299...
75c2982a1e MythTV python: Class System.system: correct typo.
706f14596d Python Bindings: Set shebangs to the configured python version
8dce45294c HTML/Python Tidy: Clean up various file modes/shebangs
479082eaa6 Python Bindings: Clean-up or delete HardwareProfiler shebangs
bdfb7690ba Python Bindings: switch mythplugins configure to python3
a9dd37c788 Python Bindings: Update version in setup.py
955eb754f5 Python Bindings: Switch default configure selection to python3
5bc77ef100 configure: Remove unnecessary Python packages

Changes to the various shebangs are an important part of the conversion too
and programs will fail without them.

--
Bill
_______________________________________________
mythtv-dev mailing list
mythtv-dev@mythtv.org
http://lists.mythtv.org/mailman/listinfo/mythtv-dev
http://wiki.mythtv.org/Mailing_List_etiquette
MythTV Forums: https://forum.mythtv.org

Fri, Jan 24, 2020 at 10:55 AM Bill Meek <keemllib@gmail.com> wrote:

> On 1/24/20 10:46 AM, Richard Shaw wrote:
> >
> > That resulted in a build error...
> >
> > mythburn.cpp: In member function 'void MythBurn::runScript()':
> > mythburn.cpp:948:19: error: 'PYTHON_EXE' was not declared in this scope
> > 948 | commandline = PYTHON_EXE;
> > | ^~~~~~~~~~
>
> Support for Python v3 is intended for MythTV v31 only. The error above is
> because at ./configure time, the Python executable is set based on the
> --python switch (or its default of python3.) This is used in a number of
> mythProgramNames that call Python scripts as above.
>
> There are a lot more commits above Pull Request 182. Quick search:
>

<SNIP>

I think I'll wait for v31 then :)

Thanks,
Richard

On 24 janv. 2020 11:07, Richard Shaw <hobbes1069@gmail.com> wrote:

> Fri, Jan 24, 2020 at 10:55 AM Bill Meek <keemllib@gmail.com> wrote:
>
>> On 1/24/20 10:46 AM, Richard Shaw wrote:
>> >
>> > That resulted in a build error...
>> >
>> > mythburn.cpp: In member function 'void MythBurn::runScript()':
>> > mythburn.cpp:948:19: error: 'PYTHON_EXE' was not declared in this scope
>> > 948 | commandline = PYTHON_EXE;
>> > | ^~~~~~~~~~
>>
>> Support for Python v3 is intended for MythTV v31 only. The error above is
>> because at ./configure time, the Python executable is set based on the
>> --python switch (or its default of python3.) This is used in a number of
>> mythProgramNames that call Python scripts as above.
>>
>> There are a lot more commits above Pull Request 182. Quick search:
>>
>
> <SNIP>
>
> I think I'll wait for v31 then :)

I'll do the same :)

Christian
_______________________________________________
mythtv-dev mailing list
mythtv-dev@mythtv.org
http://lists.mythtv.org/mailman/listinfo/mythtv-dev
http://wiki.mythtv.org/Mailing_List_etiquette
MythTV Forums: https://forum.mythtv.org