Bug 1061732 - Review Request: python-flask-restful - RESTful library for Flask
Summary: Review Request: python-flask-restful - RESTful library for Flask
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Miroslav Suchý
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On: 1059659
Blocks:
TreeView+ depends on / blocked
 
Reported: 2014-02-05 14:06 UTC by Jan Sedlák
Modified: 2015-11-30 23:25 UTC (History)
3 users (show)

Fixed In Version: 0.3.4-1.fc23
Clone Of:
Environment:
Last Closed: 2015-11-30 23:25:13 UTC
Type: ---
Embargoed:
msuchy: fedora-review+
kevin: fedora-cvs+


Attachments (Terms of Use)

Description Jan Sedlák 2014-02-05 14:06:04 UTC
Spec URL: http://jsedlak.fedorapeople.org/python-flask-restful.spec
SRPM URL: http://jsedlak.fedorapeople.org/python-flask-restful-0.2.11-1.fc20.src.rpm
Description: Flask-RESTful is Python library for writing RESTful APIs in Flask applications.

It has Python 3 support.
Fedora Account System Username: jsedlak

Comment 1 Jan Sedlák 2014-02-05 14:06:56 UTC
Here is output from fedora-review package: http://jsedlak.fedorapeople.org/review-python-flask-restful/

Comment 2 Miroslav Suchý 2015-01-15 08:17:45 UTC
It is good habit (but not mandatory) to mark which BR are needed for tests. E.g:

# those are needed for tests:
BuildRequires:  python3-nose
BuildRequires:  python3-mock
...


AUTHORS.md, README.md examples and docs should be put in in doc dir. This should be enough in %files sections:
%doc README.md AUTHORS.md examples docs

you are missing license in package. It should be either
  %doc LICENSE
or rather
  %license LICENSE
It should be in both packages.

Comment 3 Miroslav Suchý 2015-01-15 10:03:13 UTC
Also the file utils/ordereddict.py is licensed under MIT (according its header) so license should be BSD and MIT. But in upstream this file is no longer present, so you may just rebase.

Comment 4 Jan Sedlák 2015-01-21 09:12:35 UTC
Ok, I will correct it and repackage to newest version.

Comment 5 Igor Gnatenko 2015-05-26 16:55:34 UTC
(In reply to Jan Sedlák from comment #4)
> Ok, I will correct it and repackage to newest version.

ping?

Comment 6 Jan Sedlák 2015-07-08 08:06:38 UTC
I have updated flask-restful specfile. It uses newest version - 0.3.3.

Since there is a problem with logging into fedorapeople.org, I have uploaded specfile and srpm onto my Dropbox:

https://dl.dropboxusercontent.com/u/90503369/python-flask-restful/python-flask-restful.spec
https://dl.dropboxusercontent.com/u/90503369/python-flask-restful/python-flask-restful-0.3.3-1.fc22.src.rpm

There is also problem with fedora-review (namely licensecheck, bug 1240914) so I cannot run fedora-review on my package just yet.

I am using tag tarball (rather than release tarball), because since 0.3, they don't include documentation, tests and LICENSE in release tarball (see https://github.com/flask-restful/flask-restful/issues/475). I had to also delete one testcase with patch - this testcase works only with newer Werkzeug which isn't in repos.

Comment 7 Jan Sedlák 2015-07-10 08:25:43 UTC
fedora-review is now working (you have to use https://admin.fedoraproject.org/updates/devscripts-2.15.5-6.fc22), so here is its output:

https://www.dropbox.com/sh/qfno8oinv7sqq51/AADgw13JX8r3cAMOBLqbqKVra?dl=0

Comment 8 Miroslav Suchý 2015-08-13 09:28:58 UTC
You are packaging .gitignore, which is wrong:
python-flask-restful.noarch: E: version-control-internal-file /usr/share/doc/python-flask-restful/docs/_themes/.gitignore
python3-flask-restful.noarch: E: version-control-internal-file /usr/share/doc/python3-flask-restful/docs/_themes/.gitignore

Otherwise it is good to me.

Comment 10 Miroslav Suchý 2015-08-17 12:46:32 UTC
Removing FE-NEEDSPONSOR as you were already sponsored by Ralph.

APPROVED

Comment 11 Jan Sedlák 2015-08-17 14:02:09 UTC
New Package SCM Request
=======================
Package Name: python-flask-restful
Short Description: Simple framework for creating REST APIs for Flask
Upstream URL: https://www.github.com/twilio/flask-restful/
Owners: jsedlak
Branches: f21 f22 f23 el6 epel7
InitialCC:

Comment 12 Kevin Fenzi 2015-08-20 03:27:01 UTC
Git done (by process-git-requests).

Comment 13 Fedora Update System 2015-08-31 08:31:59 UTC
python-flask-restful-0.3.4-1.fc23 has been submitted as an update to Fedora 23. https://bodhi.fedoraproject.org/updates/FEDORA-2015-13802

Comment 14 Fedora Update System 2015-08-31 08:35:28 UTC
python-flask-restful-0.3.4-1.fc22 has been submitted as an update to Fedora 22. https://bodhi.fedoraproject.org/updates/FEDORA-2015-13803

Comment 15 Fedora Update System 2015-09-01 03:36:30 UTC
python-flask-restful-0.3.4-1.fc23 has been pushed to the Fedora 23 stable repository. If problems still persist, please make note of it in this bug report.

Comment 16 Fedora Update System 2015-11-13 12:01:45 UTC
python-flask-restful-0.3.4-3.fc22 has been submitted as an update to Fedora 22. https://bodhi.fedoraproject.org/updates/FEDORA-2015-7311b57ad7

Comment 17 Fedora Update System 2015-11-14 03:21:57 UTC
python-flask-restful-0.3.4-3.fc22 has been pushed to the Fedora 22 testing repository. If problems still persist, please make note of it in this bug report.
If you want to test the update, you can install it with
$ su -c 'dnf --enablerepo=updates-testing update python-flask-restful'
You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2015-7311b57ad7

Comment 18 Fedora Update System 2015-11-30 23:25:10 UTC
python-flask-restful-0.3.4-3.fc22 has been pushed to the Fedora 22 stable repository. If problems still persist, please make note of it in this bug report.


Note You need to log in before you can comment on or make changes to this bug.