[Bug 970896] Review Request: sugar-paths - A sliding tile game to create closed paths

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

 



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



--- Comment #18 from Parag AN(पराग) <panemade@xxxxxxxxx> ---
Danishka,
1) With the introduction of python3 in Fedora and updated guidelines to have a
way to have difference for python2 and python3 code, we now need to make sure
while including code for which python version it runs. so, you can do the same
for this package in %prep as
find . -name '*.py' | xargs sed -i '1s|^#!/usr/bin/env python|#!%{__python2}|'
find . -name '*.py' | xargs sed -i '1s|^#!/usr/bin/python|#!%{__python2}|'


2) I have seen sugar packages adding translation for languages which not even
supported on Fedora and therefore no locale translation directories exists
already. Its better not to package such translations as any way they will not
be working/usable.

rm po/{ayc,aym,fa_AF,hus,mvo,quz,tzm,tzo}.po


so your %prep should look like
%prep
%setup -q -n Paths-%{version}
rm po/{aym,fa_AF,hus,mvo,quz,tzm,tzo}.po
find . -name '*.py' | xargs sed -i '1s|^#!/usr/bin/env python|#!%{__python2}|'
find . -name '*.py' | xargs sed -i '1s|^#!/usr/bin/python|#!%{__python2}|'

-- 
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]