Bug 1297491

SRPM name should be python-multilib, python3 support might be added later. Please convert to create python2-multilib binary subpackage [
https://fedoraproject.org/wiki/Packaging:Python#Example_common_spec_file].

Source0 refers to "master", which can change over time. It must refer to a specific commit or tag that is immutable over time.

Tests should be run in %check.

Entries in the changelog should be separated by blank lines.

SPEC URL: https://ausil.us/packages/python-multilib.spec
SRPM URL: https://ausil.us/packages/python-multilib-1.1-2.fc24.src.rpm

I have sent in a bunch of fixes as pull requests. this sets up to run tests (needs test data) and support python3 as the code does.

%{python_provide} macro must be used [https://fedoraproject.org/wiki/Packaging:Python#The_.25python_provide_macro].

It seems that upstream hasn't made any releases. A git commit should be used for the package version [https://fedoraproject.org/wiki/Packaging:NamingGuidelines#Pre-Release_packages].
Full URL should be used for Source0 [https://fedoraproject.org/wiki/Packaging:SourceURL#Git_Hosting_Services].

Group tag can be dropped [https://fedoraproject.org/wiki/Packaging:Guidelines#Tags_and_Sections].

Looks OK otherwise.

I took in Dennis' changes and Zbigniew's suggestions.

SPEC URL: https://people.redhat.com/jgregusk/not-piracy/srpms/python-multilib.spec
SRPM URL: https://people.redhat.com/jgregusk/not-piracy/srpms/python-multilib-1.1-2.fc23.src.rpm

> I took in Dennis' changes and Zbigniew's suggestions.

Not really, I don't see python_provide anywhere, also no version change.

I haven't used this macro before. Something like this?

http://www.fpaste.org/357076/

No, not like this :(

Please see my original comment, it gives a specific link to documentation.

(In reply to Zbigniew Jędrzejewski-Szmek from comment #3)
> %{python_provide} macro must be used
> [https://fedoraproject.org/wiki/Packaging:Python#The_.
> 25python_provide_macro].

OK, sorry, now that I look at the linked section in the guidelines, it doesn't actually say how to use the macro. Please see https://fedoraproject.org/wiki/Packaging:Python#Example_common_spec_file for the proper way to use the macro.

Thanks for your help. Does this look better?

https://people.redhat.com/jgregusk/not-piracy/srpms/python-multilib.spec
https://people.redhat.com/jgregusk/not-piracy/srpms/python-multilib-1.1-3.fc23.src.rpm

Requires on python2 and python3 are not needed. (A dependency on python(abi) = X.Y is added automatically.)

Why are the tests in %check disabled? If possible, enable them, and if not, please add a comment (in the spec file) explaining why.

To make things a bit simpler to edit in the future, you can:

- replace second and third Summary lines with:
Summary: %{summary}

- define a macro for the descriptions:
%global _description \
A Python module that supports several multilib "methods" useful for determining \
if a 32-bit package should be included with its 64-bit analogue in a compose.

%description %_description
%description -n python2-multilib %_description
%description -n python3-multilib %_description

Thanks for the suggestions. How about now?

https://people.redhat.com/jgregusk/not-piracy/srpms/python-multilib.spec
https://people.redhat.com/jgregusk/not-piracy/srpms/python-multilib-1.1-4.fc23.src.rpm

+ latest version
+ package name is OK
+ provides/requires are OK
+ %python_require is used
+ "common python template" is used
+ license file is present, %license is used
+ license is acceptable (GPLv2)

rpmlint:
2 packages and 0 specfiles checked; 0 errors, 0 warnings.

Package is APPROVED.

Package request has been approved: https://admin.fedoraproject.org/pkgdb/package/rpms/python-multilib

This looks already built and is available in Fedora 24.