[Bug 1166919] Review Request: python-fastcache - C implementation of python3 lru_cache

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

 



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



--- Comment #21 from Orion Poplawski <orion@xxxxxxxxxxxxx> ---
Would you be okay with this?

--- a/python-fastcache.spec
+++ b/python-fastcache.spec
@@ -6,20 +6,26 @@

 Name:           python-%{pkgname}
 Version:        1.0.2
-Release:        2%{?dist}
+Release:        3%{?dist}
 Summary:        C implementation of python3 lru_cache

 License:        MIT
 URL:            https://github.com/pbrady/%{pkgname}
 Source0:       
https://github.com/pbrady/%{pkgname}/archive/v%{version}.tar.gz

+%if 0%{?fedora}
+# For py.cleanup
 BuildRequires:  pycmd
+%endif
 BuildRequires:  pytest
 BuildRequires:  python2-devel
 BuildRequires:  python-setuptools

 %if 0%{?with_py3}
+%if 0%{?fedora}
+# For py.cleanup
 BuildRequires:  python3-pycmd
+%endif
 BuildRequires:  python3-pytest
 BuildRequires:  python3-devel
 BuildRequires:  python3-setuptools
@@ -77,7 +83,9 @@ popd
 %check
 # Python 2 tests
 pushd %{pkgname}-%{version}
+%if 0%{?fedora}
 py.cleanup-2.7 -p
+%endif
 export PYTHONPATH=build/lib.linux-%{_arch}-2.7/fastcache/tests
 py.test-2.7 -v --ignore=fastcache/tests
 popd
@@ -85,7 +93,9 @@ popd
 %if 0%{?with_py3}
 # Python 3 tests
 pushd python3-%{pkgname}-%{version}
+%if 0%{?fedora}
 py.cleanup-3.4 -p
+%endif
 export PYTHONPATH=build/lib.linux-%{_arch}-3.4/fastcache/tests
 py.test-3.4 -v --ignore=fastcache/tests
 popd
@@ -108,6 +118,9 @@ popd
 %endif

 %changelog
+* Wed May 27 2015 Orion Poplawski <orion@xxxxxxxxxxxxx> - 1.0.2-3
+- Only run py.cleanup on Fedora
+
 * Mon Dec  1 2014 Jerry James <loganjerry@xxxxxxxxx> - 1.0.2-2
 - Be more verbose in the file listing
 - Run the tests verbosely

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