[Bug 868266] Review Request: python-pyglet - A cross-platform windowing and multimedia library for Python

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

 



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



--- Comment #44 from Orion Poplawski <orion@xxxxxxxxxxxxx> ---
I'd be happy to comaintain.  FAS - orion

Why the use of %py_byte_compile?  I've never seen that in a package before. 
Macro doesn't seem to exist on EL7.

So, with:

diff --git a/python-pyglet.spec b/python-pyglet.spec
index b08a2c4..ede83af 100644
--- a/python-pyglet.spec
+++ b/python-pyglet.spec
@@ -1,4 +1,6 @@
+%if 0%{?fedora}
 %global with_python3 1
+%endif

 %global srcname pyglet
 %global srcversion 1.2.1
@@ -102,12 +104,12 @@ popd

 %install
 %{__python} setup.py install --skip-build --root %{buildroot}
-%py_byte_compile %{__python} %{buildroot}%{python_sitelib}/%{srcname}
+%{__python} %{buildroot}%{python_sitelib}/%{srcname}

 %if 0%{?with_python3}
 pushd %{py3dir}
 %{__python3} setup.py install --skip-build --root %{buildroot}
-%py_byte_compile %{__python3} %{buildroot}%{python3_sitelib}/%{srcname}
+%{__python3} %{buildroot}%{python3_sitelib}/%{srcname}
 popd
 %endif


I end up with:

+ /usr/bin/python
/builddir/build/BUILDROOT/python-pyglet-1.2.1-1.el7.x86_64/usr/lib/python2.7/site-packages/pyglet
/usr/bin/python: can't find '__main__' module in
'/builddir/build/BUILDROOT/python-pyglet-1.2.1-1.el7.x86_64/usr/lib/python2.7/site-packages/pyglet'

Perhaps the %py_byte_compile was supposed to set that up?

-- 
You are receiving this mail because:
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]