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



--- Comment #6 from Athos Ribeiro <athoscribe...@gmail.com> ---
Created attachment 1279834
  --> https://bugzilla.redhat.com/attachment.cgi?id=1279834&action=edit
Fix python2 subpackage Requires

> - Actually, you do need to include the requires under the python2 subpackage
> (the ones you do not need to move there are the BRs):
> 
>   $ rpm -qp --requires
> python2-fedmsg-rabbitmq-serializer-0.0.5-1.fc27.noarch.rpm
>   python(abi) = 2.7
>   rpmlib(CompressedFileNames) <= 3.0.4-1
>   rpmlib(FileDigests) <= 4.6.0-1
>   rpmlib(PartialHardlinkSets) <= 4.0.4-1
>   rpmlib(PayloadFilesHavePrefix) <= 4.0-1
>   rpmlib(PayloadIsXz) <= 5.2-1

Your Requires: are not in the python2 subpackage. If you try installing and
using this package, you get:

>>> from fedmsg_rabbitmq_serializer import consumer
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File
"/usr/lib/python2.7/site-packages/fedmsg_rabbitmq_serializer/consumer.py", line
7, in <module>
    import fedmsg.consumers
ImportError: No module named fedmsg.consumers

Answering the question: yes, the "Requires:" lines should be moved into the
python2 subpackage.

This patch should fix it. You can either apply it or just move the Requires
into the python2 subpackage so we can approve this :)

-- 
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@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

Reply via email to