[Bug 1531288] Review Request: python3-gssapi - Python 3 Bindings for GSSAPI (RFC 2743/ 2744 and extensions)

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

 



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




[Index of Archives]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite Conditions]     [KDE Users]

  Powered by Linux