https://bugzilla.redhat.com/show_bug.cgi?id=1445506 --- Comment #6 from Athos Ribeiro <athoscribeiro@xxxxxxxxx> --- 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@xxxxxxxxxxxxxxxxxxxxxxx To unsubscribe send an email to package-review-leave@xxxxxxxxxxxxxxxxxxxxxxx
