https://bugzilla.redhat.com/show_bug.cgi?id=1531288 Carl George <carl@george.computer> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED --- Comment #1 from Carl George <carl@george.computer> --- https://fedoraproject.org/wiki/Packaging:Guidelines#Spec_File_Naming The spec file must be renamed to python3-gssapi.spec. > # This spec is shared between Fedora, EPEL, and el7. > %global with_python2 0 > %global with_python3 1 Not really, since the Name is different, as well as the file name once you rename it. This spec file will only ever be used for python3. As such, all the python2 parts should be removed, and the with_python3 conditionals aren't necessary. > Version: 1.2.0 > Release: 3%{?dist} You aren't required to match the RHEL version in this package. Feel free to bump this up to the latest 1.3.0 if you like. The release should start at 1 regardless. > BuildRequires: krb5-server >= 1.10 Comparing this to the Rawhide spec file, I think that should be wrapped with a run_tests conditional. > Requires: krb5-libs >= 1.10 Requirements for libkrb5.so.3, libk5crypto.so.3, and libgssapi_krb5.so.2 are automatically added by rpmbuild. RHEL7 has always had krb5 1.11 or higher. You can probably drop this line. > %{?python_provide:%python_provide %{python3}-gssapi} This macro's purpose is to have the subpackage provide the unversioned name (i.e. having python3-gssapi provide python-gssapi in Fedora) once the distribution decides to make that switch. It should not be used in an EPEL package because python34-gssapi will never provide python-gssapi. > %description -n %{python3}-gssapi You repeat the description after this, but you have a _description macro you can use instead. Change it to `%description -n %{python3}-gssapi %_description` and remove the duplicate description text that follows. > %{py3dir} Everything related to this macro can be removed, it isn't necessary. > CFLAGS="%{optflags}" %{__python3} setup.py build Change this to `%py3_build`. > %{__python3} setup.py install --skip-build --root %{buildroot} Change this to `%py3_install`. > # fix permissions on shared objects (mock seems to set them > # to 0775, whereas a normal build gives 0755) I did a mock build without this section, and the permissions were all correctly set to 0755. I think it can be removed. > %{python3_sitearch}/* This is a bit greedy, it would be better to be explicit with `%{python3_sitearch}/gssapi` and `%{python3_sitearch}/gssapi-%{version}-py%{python3_version}.egg-info`. -- 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 To unsubscribe send an email to package-review-leave@xxxxxxxxxxxxxxxxxxxxxxx
