Spec URL: http://kinginuyasha.enanocms.org/downloads/python-pyrpmmd.spec SRPM URL: http://kinginuyasha.enanocms.org/downloads/python-pyrpmmd-0.1.0-1.fc27.src.rpm Description: pyrpmmd is an independent Python module for reading rpm-md repository metadata. The code is derived from the repomd parsing code from Yum. Successful scratch build: https://koji.fedoraproject.org/koji/taskinfo?taskID=19550430 Fedora Account System Username: ngompa
Taken.
Package Review ============== [!]: Package consistently uses macros (instead of hard-coded directory names). Since you're defining the source project name as a global, you should consistently use it, too (missing on lines 41, 51). [!]: Final provides and requires are sane: Provides -------- python3-pyrpmmd: python3-pyrpmmd python3.6dist(pyrpmmd) python3dist(pyrpmmd) python2-pyrpmmd: python-pyrpmmd python2-pyrpmmd python2.7dist(pyrpmmd) python2dist(pyrpmmd) As far as I can tell from the Packaging:Python guidelines, "python-pyrpmmd" should be provided by the subpackage using the current system python implementation (python3 on rawhide, so it should be provided by python3-pyrpmmd). Since the appropriate macro is used, %python_provide seems to be broken there. rpmlint issues -------------- Additionally, there are some pylint errors you might want to inform "upstream" about: python2-pyrpmmd.noarch: E: incorrect-fsf-address /usr/lib/python2.7/site-packages/rpmmd/Errors.py python2-pyrpmmd.noarch: E: incorrect-fsf-address /usr/lib/python2.7/site-packages/rpmmd/misc.py python2-pyrpmmd.noarch: E: incorrect-fsf-address /usr/lib/python2.7/site-packages/rpmmd/repoMDObject.py python3-pyrpmmd.noarch: E: incorrect-fsf-address /usr/lib/python3.6/site-packages/rpmmd/Errors.py python3-pyrpmmd.noarch: E: incorrect-fsf-address /usr/lib/python3.6/site-packages/rpmmd/misc.py python3-pyrpmmd.noarch: E: incorrect-fsf-address /usr/lib/python3.6/site-packages/rpmmd/repoMDObject.py
(In reply to Fabio Valentini from comment #2) > Package Review > ============== > > [!]: Package consistently uses macros (instead of hard-coded directory > names). > > Since you're defining the source project name as a global, you should > consistently use it, too (missing on lines 41, 51). > D'oh. Fixed. > > [!]: Final provides and requires are sane: > > Provides > -------- > python3-pyrpmmd: > python3-pyrpmmd > python3.6dist(pyrpmmd) > python3dist(pyrpmmd) > > python2-pyrpmmd: > python-pyrpmmd > python2-pyrpmmd > python2.7dist(pyrpmmd) > python2dist(pyrpmmd) > > As far as I can tell from the Packaging:Python guidelines, "python-pyrpmmd" > should be provided by the subpackage using the current system python > implementation (python3 on rawhide, so it should be provided by > python3-pyrpmmd). Since the appropriate macro is used, %python_provide seems > to be broken there. > Unfortunately, this is by design. The macro has not been switched to have python-* on Python 3 packages. That decision will come up in the coming years... > > rpmlint issues > -------------- > > Additionally, there are some pylint errors you might want to inform > "upstream" about: > > python2-pyrpmmd.noarch: E: incorrect-fsf-address > /usr/lib/python2.7/site-packages/rpmmd/Errors.py > python2-pyrpmmd.noarch: E: incorrect-fsf-address > /usr/lib/python2.7/site-packages/rpmmd/misc.py > python2-pyrpmmd.noarch: E: incorrect-fsf-address > /usr/lib/python2.7/site-packages/rpmmd/repoMDObject.py > > python3-pyrpmmd.noarch: E: incorrect-fsf-address > /usr/lib/python3.6/site-packages/rpmmd/Errors.py > python3-pyrpmmd.noarch: E: incorrect-fsf-address > /usr/lib/python3.6/site-packages/rpmmd/misc.py > python3-pyrpmmd.noarch: E: incorrect-fsf-address > /usr/lib/python3.6/site-packages/rpmmd/repoMDObject.py Will do. :)
Revised version uses the same URL as before.
(In reply to Neal Gompa from comment #3) > (In reply to Fabio Valentini from comment #2) > > Package Review > > ============== > > > > [!]: Package consistently uses macros (instead of hard-coded directory > > names). > > > > Since you're defining the source project name as a global, you should > > consistently use it, too (missing on lines 41, 51). > > > > D'oh. Fixed. > > > > > [!]: Final provides and requires are sane: > > > > Provides > > -------- > > python3-pyrpmmd: > > python3-pyrpmmd > > python3.6dist(pyrpmmd) > > python3dist(pyrpmmd) > > > > python2-pyrpmmd: > > python-pyrpmmd > > python2-pyrpmmd > > python2.7dist(pyrpmmd) > > python2dist(pyrpmmd) > > > > As far as I can tell from the Packaging:Python guidelines, "python-pyrpmmd" > > should be provided by the subpackage using the current system python > > implementation (python3 on rawhide, so it should be provided by > > python3-pyrpmmd). Since the appropriate macro is used, %python_provide seems > > to be broken there. > > > > Unfortunately, this is by design. The macro has not been switched to have > python-* on Python 3 packages. That decision will come up in the coming > years... Well, then the wording of the Packaging:Python wiki page is strange if that hasn't been switched over, despite python3 being the default now. ... > > > > rpmlint issues > > -------------- > > > > Additionally, there are some pylint errors you might want to inform > > "upstream" about: > > > > python2-pyrpmmd.noarch: E: incorrect-fsf-address > > /usr/lib/python2.7/site-packages/rpmmd/Errors.py > > python2-pyrpmmd.noarch: E: incorrect-fsf-address > > /usr/lib/python2.7/site-packages/rpmmd/misc.py > > python2-pyrpmmd.noarch: E: incorrect-fsf-address > > /usr/lib/python2.7/site-packages/rpmmd/repoMDObject.py > > > > python3-pyrpmmd.noarch: E: incorrect-fsf-address > > /usr/lib/python3.6/site-packages/rpmmd/Errors.py > > python3-pyrpmmd.noarch: E: incorrect-fsf-address > > /usr/lib/python3.6/site-packages/rpmmd/misc.py > > python3-pyrpmmd.noarch: E: incorrect-fsf-address > > /usr/lib/python3.6/site-packages/rpmmd/repoMDObject.py > > Will do. :) Since you fixed the things I pointed out and those were the only issues I could find, I've approved the package.
Package request has been approved: https://admin.fedoraproject.org/pkgdb/package/rpms/python-pyrpmmd
python-pyrpmmd-0.1.1-1.el6 has been submitted as an update to Fedora EPEL 6. https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2017-6f3295ae47
python-pyrpmmd-0.1.1-1.fc25 has been pushed to the Fedora 25 testing repository. If problems still persist, please make note of it in this bug report. See https://fedoraproject.org/wiki/QA:Updates_Testing for instructions on how to install test updates. You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2017-2ac21e21a2
python-pyrpmmd-0.1.1-1.fc26 has been pushed to the Fedora 26 testing repository. If problems still persist, please make note of it in this bug report. See https://fedoraproject.org/wiki/QA:Updates_Testing for instructions on how to install test updates. You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2017-619b2eca37
python-pyrpmmd-0.1.1-1.el6 has been pushed to the Fedora EPEL 6 testing repository. If problems still persist, please make note of it in this bug report. See https://fedoraproject.org/wiki/QA:Updates_Testing for instructions on how to install test updates. You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2017-6f3295ae47
python-pyrpmmd-0.1.1-1.el7 has been pushed to the Fedora EPEL 7 testing repository. If problems still persist, please make note of it in this bug report. See https://fedoraproject.org/wiki/QA:Updates_Testing for instructions on how to install test updates. You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2017-e14c9dca44
python-pyrpmmd-0.1.1-1.fc26 has been pushed to the Fedora 26 stable repository. If problems still persist, please make note of it in this bug report.
python-pyrpmmd-0.1.1-1.fc25 has been pushed to the Fedora 25 stable repository. If problems still persist, please make note of it in this bug report.
python-pyrpmmd-0.1.1-1.el6 has been pushed to the Fedora EPEL 6 stable repository. If problems still persist, please make note of it in this bug report.
python-pyrpmmd-0.1.1-1.el7 has been pushed to the Fedora EPEL 7 stable repository. If problems still persist, please make note of it in this bug report.