Bug 752938
Summary: | Review Request: python-straight-plugin - Python plugin loader | ||
---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | Pierre-YvesChibon <pingou> |
Component: | Package Review | Assignee: | Toshio Ernie Kuratomi <a.badger> |
Status: | CLOSED ERRATA | QA Contact: | Fedora Extras Quality Assurance <extras-qa> |
Severity: | medium | Docs Contact: | |
Priority: | medium | ||
Version: | rawhide | CC: | a.badger, notting, package-review, terje.rosten |
Target Milestone: | --- | Flags: | a.badger:
fedora-review+
pingou: fedora-cvs+ |
Target Release: | --- | ||
Hardware: | All | ||
OS: | Linux | ||
Whiteboard: | |||
Fixed In Version: | python-straight-plugin-1.1.1-0.7.20111110.git57ef11c.el6 | Doc Type: | Bug Fix |
Doc Text: | Story Points: | --- | |
Clone Of: | Environment: | ||
Last Closed: | 2011-11-23 23:28:40 UTC | Type: | --- |
Regression: | --- | Mount Type: | --- |
Documentation: | --- | CRM: | |
Verified Versions: | Category: | --- | |
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
Cloudforms Team: | --- | Target Upstream Version: | |
Embargoed: |
Description
Pierre-YvesChibon
2011-11-10 19:43:37 UTC
Talked on IRC and spec file has been updated. This review is based on the tarball from the srpm and the new spec file rpmlint: python-straight.plugin.noarch: W: spelling-error %description -l en_US plugins -> plug ins, plug-ins, plugging [snip other spelling warnings] These are all false positives -- the words are common in computing. python-straight.plugin.src: W: invalid-url Source0: ironfroggy-straight.plugin-57ef11c.tar.gz There is a comment above the Source0 explaining how to retrieve the tarball from github so this is fine. Good: * License is BSD in source and spec * LICENSE file included * Spec file is legible * Sources match with upstream by following the github url to get the snapshot. * Package builds in koji for f16 * No localized files so no i18n concerns * Not a shared library package so no concerns there * No bundled libraries * Package owns the directories it creates and nothing more * Files listed once in the %files section * File permissions are correct * Macros used consistently * Code, not content * No large documentation files * Documentation is not used at runtime * Not a GUI app * No supbackages * All filenames are valid utf-8 Needswork: * Naming guidelines want "." replaced with "-" so this should be named python-straight-plugin. (and spec file renamed to match). Cosmetic: * Could BuildRequire: python2-devel instead of python-devel * The description says python3 compatible but you aren't building a python3 subpackage. I'd either build the subpackage or remove that line from the description. Fix the naming and I'll approve this. Thanks for your comments (here and on irc) and the review. I changed the name and added a python3 sub-package. I used 2to3 to generate the sub-package although it seems unnecessary but for the future, better safe than sorry. Spec URL: http://pingou.fedorapeople.org/RPMs/python-straight-plugin.spec SRPM URL: http://pingou.fedorapeople.org/RPMs/python-straight-plugin-1.1.1-0.2.20111110git57ef11c.fc16.src.rpm There is the new version with the correct use of %{__python3} on the build and test Spec URL: http://pingou.fedorapeople.org/RPMs/python-straight-plugin.spec SRPM URL: http://pingou.fedorapeople.org/RPMs/python-straight-plugin-1.1.1-0.3.20111110git57ef11c.fc16.src.rpm All problems have been fixed. APPROVED. New Package SCM Request ======================= Package Name: python-straight-plugin Short Description: Python plugin loader Owners: pingou sochotnicky Branches: f15 f16 el6 Fix Stanislav's FAS username New Package SCM Request ======================= Package Name: python-straight-plugin Short Description: Python plugin loader Owners: pingou sochotni Branches: f15 f16 el6 Git done (by process-git-requests). python-straight-plugin-1.1.1-0.7.20111110.git57ef11c.fc15 has been submitted as an update for Fedora 15. https://admin.fedoraproject.org/updates/python-straight-plugin-1.1.1-0.7.20111110.git57ef11c.fc15 python-straight-plugin-1.1.1-0.7.20111110.git57ef11c.fc16 has been submitted as an update for Fedora 16. https://admin.fedoraproject.org/updates/python-straight-plugin-1.1.1-0.7.20111110.git57ef11c.fc16 python-straight-plugin-1.1.1-0.7.20111110.git57ef11c.el6 has been submitted as an update for Fedora EPEL 6. https://admin.fedoraproject.org/updates/python-straight-plugin-1.1.1-0.7.20111110.git57ef11c.el6 python-straight-plugin-1.1.1-0.7.20111110.git57ef11c.el6 has been pushed to the Fedora EPEL 6 testing repository. python-straight-plugin-1.1.1-0.7.20111110.git57ef11c.fc15 has been pushed to the Fedora 15 stable repository. python-straight-plugin-1.1.1-0.7.20111110.git57ef11c.fc16 has been pushed to the Fedora 16 stable repository. python-straight-plugin-1.1.1-0.7.20111110.git57ef11c.el6 has been pushed to the Fedora EPEL 6 stable repository. Package Change Request ====================== Package Name: python-straight-plugin New Branches: epel7 Owners: pingou Thanks @Till! |