View Revisions: Issue #11
Summary | 0000011: [PATCH] guicast/bccmdl.py is not Python 3 compatible | ||
---|---|---|---|
Revision | 2018-12-06 20:26 by Sam | ||
Description | I sent this to [email protected] earlier today, before I'd seen the announcement about the new site. Since it still hasn't posted to the mailing list yet, and since it really more concerns Cinelerra-GG, I thought I'd duplicate it here as well. ========================= The rpmfusion third-party package repository for Fedora Linux is working on packaging the latest Cinelerra-GG for inclusion: https://bugzilla.rpmfusion.org/show_bug.cgi?id=5093 The package reviewer quite correctly flagged the dependency on Python 2 in `guicast/bccmdl.py` as a concern, since Fedora is in the process of deprecating Python 2 entirely and it's highly desirable that all remaining uses of Python throughout the distro have support for Python 3. In the case of `guicast/bccmdl.py`, the only Python 3 incompatibilities were the `print` statements which use deprecated Python 2 semantics. The attached patch updates those all to Python 3 `print()` function form, retaining compatibility with Python 2 via an added instruction at the start of the file: from __future__ import print_function It also updates the Makefile in the `guicast` directory, to invoke the script via `python3 < bccmdl.py` |
||
Revision | 2018-12-04 05:52 by ferdnyc | ||
Description | I sent this to [email protected] earlier today, before I'd seen the announcement about the new site. Since it still hasn't posted to the mailing list yet, and since it really more concerns Cinelerra-GG, I thought I'd duplicate it here as well. ========================= The rpmfusion third-party package repository for Fedora Linux is working on packaging the latest Cinelerra-GG for inclusion: https://bugzilla.rpmfusion.org/show_bug.cgi?id=5093 The package reviewer quite correctly flagged the dependency on Python 2 in `guicast/bccmdl.py` as a concern, since Fedora is in the process of deprecating Python 2 entirely and it's highly desirable that all remaining uses of Python throughout the distro have support for Python 3. In the case of `guicast/bccmdl.py`, the only Python 3 incompatibilities were the `print` statements which use deprecated Python 2 semantics. The attached patch updates those all to Python 3 `print()` function form, retaining compatibility with Python 2 via an added instruction at the start of the file: from __future__ import print_function It also updates the Makefile in the `guicast` directory, to invoke the script via `python3 < bccmdl.py` |