Bug 1200520
Summary: | Review Request: python-debtcollector - A collection of Python deprecation patterns and strategies | ||
---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | Chandan Kumar <chkumar246> |
Component: | Package Review | Assignee: | Parag AN(पराग) <panemade> |
Status: | CLOSED ERRATA | QA Contact: | Fedora Extras Quality Assurance <extras-qa> |
Severity: | medium | Docs Contact: | |
Priority: | unspecified | ||
Version: | rawhide | CC: | chkumar, kushal124, package-review, panemade, robinlee.sysu, zbyszek |
Target Milestone: | --- | Flags: | panemade:
fedora-review+
gwync: fedora-cvs+ |
Target Release: | --- | ||
Hardware: | All | ||
OS: | Linux | ||
Whiteboard: | |||
Fixed In Version: | python-debtcollector-0.3.0-3.fc22 | Doc Type: | Bug Fix |
Doc Text: | Story Points: | --- | |
Clone Of: | Environment: | ||
Last Closed: | 2015-06-05 23:39:43 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: | |||
Bug Depends On: | 1200955 | ||
Bug Blocks: |
Description
Chandan Kumar
2015-03-10 17:59:56 UTC
%description has a repetition a repetition near the end. After reading of the description I still don't know *what* the package does. Is it a collections of classes to help me? Is it a collection of decorators? Does it run some checks on my code? Is it a collection of (anti-)examples? Change %{__python} to %{__python2}. Why delete tests? It would be best to let them be and also run them in %check phase. Use %license for LICENSE. A quick fix in the spec : In the source0 , user %{version} And the fixes as suggested by Zbigniew What's the difference with 1200422? You should keep only one request issue per package. (In reply to Robin Lee from comment #3) > What's the difference with 1200422? You should keep only one request issue > per package. I assume it was a mistake. *** Bug 1200422 has been marked as a duplicate of this bug. *** Hi, Thanks for the review: Here is the updated SPEC: https://chandankumar.fedorapeople.org/python-debtcollector.spec SRPM: https://chandankumar.fedorapeople.org/python-debtcollector-0.3.0-2.fc21.src.rpm and Koji Build: http://koji.fedoraproject.org/koji/taskinfo?taskID=9318824 I have removed the tests from %check section in the spec file because for running tests for debtcollector package, it requires a python package `discover` as a dependency (which is not yet available on fedora koji). Once the python-discover is build, i will add the tests under %check section in spec file. Thanks, Chandan Kumar Review: + Package built successfully in rawhide mock (x86_64) + rpmlint on all generated rpms give result python-debtcollector-doc.noarch: W: wrong-file-end-of-line-encoding /usr/share/doc/python-debtcollector-doc/html/_static/jquery.js 3 packages and 0 specfiles checked; 0 errors, 1 warnings. => I actually don't see any benefit fixing this but if you want you can do in %install as dos2unix doc/build/html/_static/jquery.js but this will require you to add in spec file BuildRequires: dos2unix + Source verified with upstream as (sha256sum) source rpm tarball: 64cf1ab9bacbdda8c83a569349f2a91211a890973e0119514a9b7bf34518373e upstream tarball: 64cf1ab9bacbdda8c83a569349f2a91211a890973e0119514a9b7bf34518373e + License is "ASL 2.0" which is valid and its license text is installed in LICENSE file. Suggestions: 1) I see build.log showing some tracebacks while building documentation. This mean you are missing some packages. For this package you should add in spec BuildRequires: python-six BuildRequires: python-oslo-utils submit new package. Hi Parag, Here is the updated SPEC: https://chandankumar.fedorapeople.org/python-debtcollector.spec SRPM: https://chandankumar.fedorapeople.org/python-debtcollector-0.3.0-3.fc21.src.rpm Thanks, Chandan Kumar Looks good now :) APPROVED. New Package SCM Request ======================= Package Name: python-debtcollector Short Description: A collection of Python deprecation patterns and strategies Upstream URL: https://pypi.python.org/pypi/debtcollector Owners: chandankumar Branches: f22 InitialCC: apevec Git done (by process-git-requests). python-debtcollector-0.3.0-3.fc22 has been submitted as an update for Fedora 22. https://admin.fedoraproject.org/updates/python-debtcollector-0.3.0-3.fc22 python-debtcollector-0.3.0-3.fc22 has been pushed to the Fedora 22 testing repository. python-debtcollector-0.3.0-3.fc22 has been pushed to the Fedora 22 stable repository. |