[Bug 1262644] Review Request: python-Pyped - Replace sed/grep/cut/awk by letting you execute Python one-liners

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



https://bugzilla.redhat.com/show_bug.cgi?id=1262644



--- Comment #5 from René Ribaud <rene.ribaud@xxxxxxxxx> ---
Hello Julien,

>- I guess you intended to use the %sum variable
I used it now.
>- Python3 by default https://fedoraproject.org wiki/Packaging:Python#Avoiding_collisions_between_the_python_2_and_python_3_stacks
Fixed as porposed in the doco. Please tell me if it is fine from your point of
view.
>- Not sure if your patch is useful
Yep otherwise I get an error at build step.
>- To get rid of the E: non-executable-script /usr/lib/python3.4/site-packages pyped.py 644 /usr/bin/env you should remove the shebang from pyped.py. The typical way to do this is to call sed -i '1{\@^#!/usr/bin/env python@d}' %{buildroot}%{python3_sitearch}/pyped.py in %install (the same for python 2). I will wait for this to be fixed before approving the package.
I have just made the required file executable as it should be run by the user.

New packages location :
Spec URL: http://uggla.free.fr/rpmbuild/SPECS/python-Pyped.spec
SRPM URL: http://uggla.free.fr/rpmbuild/SRPMS/python-Pyped-1.4-2.fc22.src.rpm

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
_______________________________________________
package-review mailing list
package-review@xxxxxxxxxxxxxxxxxxxxxxx
https://admin.fedoraproject.org/mailman/listinfo/package-review




[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]