Bug 1361789 - pyftpdlib: python2-pyftpdlib requires both Python 2 and Python 3
Summary: pyftpdlib: python2-pyftpdlib requires both Python 2 and Python 3
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: pyftpdlib
Version: rawhide
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Othman Madjoudj
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Keywords:
Depends On:
Blocks: PYTHON3 PY3PATCH-AVAILABLE
TreeView+ depends on / blocked
 
Reported: 2016-07-30 12:43 UTC by Petr Viktorin
Modified: 2016-07-30 16:11 UTC (History)
2 users (show)

(edit)
Clone Of:
(edit)
Last Closed: 2016-07-30 16:11:52 UTC


Attachments (Terms of Use)
Patch that moves the script (1.36 KB, patch)
2016-07-30 12:43 UTC, Petr Viktorin
no flags Details | Diff

Description Petr Viktorin 2016-07-30 12:43:25 UTC
Created attachment 1185807 [details]
Patch that moves the script

Hello,
Thank you for your effort of porting pyftpdlib to Python 3 in Fedora! It's much appreciated.

However, there's one more issue: the /usr/bin/ftpbench script runs under Python 3 (as it should, according to guidelines), but it's packaged under the python2 subpackage.

Here's a patch to solve the issue.

It's OK to do this in Rawhide only, however, it would be greatly
appreciated if you could push it to Fedora 25 as well.



There is a section on shebangs in the Python RPM Porting Guide [0]
which covers this issue.

If anything is unclear, or if you need any kind of assistance, you can
ask on IRC (#fedora-python on Freenode), or reply here. We'll be happy
to help investigating or fixing this issue!

[0] http://python-rpm-porting.readthedocs.io/en/latest/application-modules.html#are-shebangs-dragging-you-down-to-python-2

Comment 1 Petr Viktorin 2016-07-30 12:44:24 UTC
A Koji scratch build is here: http://koji.fedoraproject.org/koji/taskinfo?taskID=15073823

Comment 2 Othman Madjoudj 2016-07-30 16:11:52 UTC
I missed that one, fixed in rawhide/f25, thanks for the patch.


Note You need to log in before you can comment on or make changes to this bug.