[Bug 1036396] Review Request: python-social-auth - Social auth made simple
https://bugzilla.redhat.com/show_bug.cgi?id=1036396 Bug 1036396 depends on bug 1036393, which changed state. Bug 1036393 Summary: Review Request: python3-openid - Python 3 port of the python-openid library https://bugzilla.redhat.com/show_bug.cgi?id=1036393 What|Removed |Added Status|ASSIGNED|CLOSED Resolution|--- |CURRENTRELEASE -- 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@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1036396] Review Request: python-social-auth - Social auth made simple
https://bugzilla.redhat.com/show_bug.cgi?id=1036396 Christopher Meng cicku...@gmail.com changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution|--- |RAWHIDE Last Closed||2014-02-23 22:53:00 --- Comment #20 from Christopher Meng cicku...@gmail.com --- No f20 build, no epel6 build, no epel7 build. RAWHIDE only, anyway. -- 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@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1036396] Review Request: python-social-auth - Social auth made simple
https://bugzilla.redhat.com/show_bug.cgi?id=1036396 --- Comment #18 from Jakub Dorňák jdor...@redhat.com --- Well, it now builds for epel7, but there are some broken dependences: On x86_64: python-social-auth-0.1.19-1.el7.noarch requires python-requests-oauthlib python-social-auth-0.1.19-1.el7.noarch requires python-oauthlib On ppc64: python-social-auth-0.1.19-1.el7.noarch requires python-requests-oauthlib python-social-auth-0.1.19-1.el7.noarch requires python-requests python-social-auth-0.1.19-1.el7.noarch requires python-oauthlib -- 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@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1036396] Review Request: python-social-auth - Social auth made simple
https://bugzilla.redhat.com/show_bug.cgi?id=1036396 --- Comment #19 from Jakub Dorňák jdor...@redhat.com --- There are epel7 branches of python-oauthlib and python-requests-oauthlib. I have just requested epel7 branch of python-requests. -- 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@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1036396] Review Request: python-social-auth - Social auth made simple
https://bugzilla.redhat.com/show_bug.cgi?id=1036396
Eduardo Echeverria echevemas...@gmail.com changed:
What|Removed |Added
CC||echevemas...@gmail.com
--- Comment #17 from Eduardo Echeverria echevemas...@gmail.com ---
Jakub I'm using this package, thanks for bring this to Fedora :)
I saw that epel7 build fails, you can fix if put this lines above the spec, due
to py3 doesn't exists in epel7, also python2 macro doesn't exists on epel.
# Workaround for epel versions where don't exists python2 macro
%{!?__python2: %global __python2 %{__python}}
%if 0%{?fedora} 12
%global with_python3 1
%else
%{!?python_sitelib: %global python_sitelib %(%{__python} -c from
distutils.sysconfig import get_python_lib; print (get_python_lib()))}
%endif
Best Regards, sorry for not open a bug for say this, due to isn't a bug, and
the review is recent
--
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@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1036396] Review Request: python-social-auth - Social auth made simple
https://bugzilla.redhat.com/show_bug.cgi?id=1036396 Jon Ciesla limburg...@gmail.com changed: What|Removed |Added Flags|fedora-cvs? |fedora-cvs+ -- 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@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1036396] Review Request: python-social-auth - Social auth made simple
https://bugzilla.redhat.com/show_bug.cgi?id=1036396 --- Comment #16 from Jon Ciesla limburg...@gmail.com --- Git done (by process-git-requests). -- 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@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1036396] Review Request: python-social-auth - Social auth made simple
https://bugzilla.redhat.com/show_bug.cgi?id=1036396 Jakub Dorňák jdor...@redhat.com changed: What|Removed |Added Flags|needinfo?(jdor...@redhat.co |needinfo?(cicku...@gmail.co |m) |m) --- Comment #13 from Jakub Dorňák jdor...@redhat.com --- Hi Christopher, You have asked whether I am going to push it to EPEL. Are there some additional requirements for the package to be pushed to EPEL? How can I request epel branches? Thank You, QB -- 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@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1036396] Review Request: python-social-auth - Social auth made simple
https://bugzilla.redhat.com/show_bug.cgi?id=1036396 Christopher Meng cicku...@gmail.com changed: What|Removed |Added Flags|needinfo?(cicku...@gmail.co | |m) | --- Comment #14 from Christopher Meng cicku...@gmail.com --- For EPEL6 these python versioned macros don't exist, you need to add some hack for epel == 6 with __python macro redefined. But EPEL7 doesn't need this modification. For requesting new branches instructions, please refer to: http://fedoraproject.org/wiki/Package_SCM_admin_requests#Package_Change_Requests_for_existing_packages -- 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@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1036396] Review Request: python-social-auth - Social auth made simple
https://bugzilla.redhat.com/show_bug.cgi?id=1036396 Jakub Dorňák jdor...@redhat.com changed: What|Removed |Added Flags|fedora-cvs+ |fedora-cvs? --- Comment #15 from Jakub Dorňák jdor...@redhat.com --- Package Change Request == Package Name: python-social-auth New Branches: el6 epel7 Owners: jdornak InitialCC: Users requested EPEL 6 and 7 branches. -- 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@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1036396] Review Request: python-social-auth - Social auth made simple
https://bugzilla.redhat.com/show_bug.cgi?id=1036396 Patrick Uiterwijk puiterw...@redhat.com changed: What|Removed |Added CC||puiterw...@redhat.com Flags||needinfo?(jdor...@redhat.co ||m) --- Comment #12 from Patrick Uiterwijk puiterw...@redhat.com --- Could we please get EPEL 6 and 7 branches? I would be more than glad to maintain those, as we need them for Fedora Infrastructure. -- 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@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1036396] Review Request: python-social-auth - Social auth made simple
https://bugzilla.redhat.com/show_bug.cgi?id=1036396 Jakub Dorňák jdor...@redhat.com changed: What|Removed |Added Depends On||1036393 Referenced Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=1036393 [Bug 1036393] Review Request: python3-openid - Python 3 port of the python-openid library -- 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@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1036396] Review Request: python-social-auth - Social auth made simple
https://bugzilla.redhat.com/show_bug.cgi?id=1036396 Jakub Dorňák jdor...@redhat.com changed: What|Removed |Added Flags||fedora-cvs? --- Comment #9 from Jakub Dorňák jdor...@redhat.com --- New Package SCM Request === Package Name: python-social-auth Short Description: Social auth made simple Owners: jdornak Branches: f20 InitialCC: -- 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@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1036396] Review Request: python-social-auth - Social auth made simple
https://bugzilla.redhat.com/show_bug.cgi?id=1036396 Jon Ciesla limburg...@gmail.com changed: What|Removed |Added Flags|fedora-cvs? |fedora-cvs+ -- 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@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1036396] Review Request: python-social-auth - Social auth made simple
https://bugzilla.redhat.com/show_bug.cgi?id=1036396 --- Comment #10 from Jon Ciesla limburg...@gmail.com --- Git done (by process-git-requests). -- 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@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1036396] Review Request: python-social-auth - Social auth made simple
https://bugzilla.redhat.com/show_bug.cgi?id=1036396 --- Comment #11 from Jakub Dorňák jdor...@redhat.com --- Anyway, I have to wait until python3-openid review process is finished first. -- 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@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1036396] Review Request: python-social-auth - Social auth made simple
https://bugzilla.redhat.com/show_bug.cgi?id=1036396 Christopher Meng cicku...@gmail.com changed: What|Removed |Added Flags|fedora-review? |fedora-review+ --- Comment #8 from Christopher Meng cicku...@gmail.com --- (In reply to Jakub Dorňák from comment #7) ad 1.) The source from pypi does not contain docs and examples, while the source from githup does. Is it really better to use the one from pypi? No ;-). But according to the results, this tag has been retagged, you can download source from URL again and then rebuild again with the sources just downloaded. And better file an issue on github of including docs into tarballs on pypi. ad 2.) I am not sure how to fix this. Just BSD, no extra 'License' needed in the field, See: https://fedoraproject.org/wiki/Licensing:Main ad 3.) The dependancies will be solved by other (sub)packages proposed by me. --- I will set fedora-review + for this package, however please fix above #1 and #2 before SCM, I'm watching. ;D PACKAGE APPROVED. -- 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@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1036396] Review Request: python-social-auth - Social auth made simple
https://bugzilla.redhat.com/show_bug.cgi?id=1036396 --- Comment #7 from Jakub Dorňák jdor...@redhat.com --- ad 1.) The source from pypi does not contain docs and examples, while the source from githup does. Is it really better to use the one from pypi? ad 2.) I am not sure how to fix this. ad 3.) The dependancies will be solved by other (sub)packages proposed by me. -- 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@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1036396] Review Request: python-social-auth - Social auth made simple
https://bugzilla.redhat.com/show_bug.cgi?id=1036396 Jakub Dorňák jdor...@redhat.com changed: What|Removed |Added Flags|needinfo?(jdor...@redhat.co | |m) | --- Comment #3 from Jakub Dorňák jdor...@redhat.com --- I do not plan to push it to EPEL until somebody asks for it. What would be the implications? -- 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@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1036396] Review Request: python-social-auth - Social auth made simple
https://bugzilla.redhat.com/show_bug.cgi?id=1036396
--- Comment #4 from Christopher Meng cicku...@gmail.com ---
Assume that you are now only interested in Fedora, please improve:
0. Don't create file list by script, write down them in %files.
1. Version the python2 macro: %{__python} -- %{__python2}; %{python_sitelib}
-- %{python2_sitelib}
--
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@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1036396] Review Request: python-social-auth - Social auth made simple
https://bugzilla.redhat.com/show_bug.cgi?id=1036396 --- Comment #5 from Jakub Dorňák jdor...@redhat.com --- improved -- 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@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1036396] Review Request: python-social-auth - Social auth made simple
https://bugzilla.redhat.com/show_bug.cgi?id=1036396
--- Comment #6 from Christopher Meng cicku...@gmail.com ---
Package Review
==
Legend:
[x] = Pass, [!] = Fail, [-] = Not applicable, [?] = Not evaluated
Issues:
===
- Package installs properly.
Note: Installation errors (see attachment)
See: https://fedoraproject.org/wiki/Packaging:Guidelines
- Sources used to build the package match the upstream source, as provided in
the spec URL.
Note: Check did not completechecksum differs and there are problems running
diff. Please verify manually.
See: http://fedoraproject.org/wiki/Packaging/SourceURL
= MUST items =
Generic:
[x]: Package is licensed with an open-source compatible license and meets
other legal requirements as defined in the legal section of Packaging
Guidelines.
[!]: License field in the package spec file matches the actual license.
Note: Checking patched sources after %prep for licenses. Licenses found:
Unknown or generated. 400 files have unknown license. Detailed output
of licensecheck:
Unknown or generated
python-social-auth-0.1.17/%{py3dir}/docs/conf.py
python-social-auth-0.1.17/%{py3dir}/setup.py
python-social-auth-0.1.17/%{py3dir}/social/__init__.py
python-social-auth-0.1.17/%{py3dir}/social/actions.py
python-social-auth-0.1.17/%{py3dir}/social/apps/django_app/__init__.py
python-social-auth-0.1.17/%{py3dir}/social/apps/django_app/context_processors.py
python-social-auth-0.1.17/%{py3dir}/social/apps/django_app/default/__init__.py
python-social-auth-0.1.17/%{py3dir}/social/apps/django_app/default/admin.py
python-social-auth-0.1.17/%{py3dir}/social/apps/django_app/default/fields.py
python-social-auth-0.1.17/%{py3dir}/social/apps/django_app/default/models.py
python-social-auth-0.1.17/%{py3dir}/social/apps/django_app/default/tests.py
python-social-auth-0.1.17/%{py3dir}/social/apps/django_app/me/__init__.py
python-social-auth-0.1.17/%{py3dir}/social/apps/django_app/me/models.py
python-social-auth-0.1.17/%{py3dir}/social/apps/django_app/me/tests.py
python-social-auth-0.1.17/%{py3dir}/social/apps/django_app/middleware.py
python-social-auth-0.1.17/%{py3dir}/social/apps/django_app/tests.py
python-social-auth-0.1.17/%{py3dir}/social/apps/django_app/urls.py
python-social-auth-0.1.17/%{py3dir}/social/apps/django_app/utils.py
python-social-auth-0.1.17/%{py3dir}/social/apps/django_app/views.py
python-social-auth-0.1.17/%{py3dir}/social/apps/flask_app/fields.py
python-social-auth-0.1.17/%{py3dir}/social/apps/flask_app/models.py
python-social-auth-0.1.17/%{py3dir}/social/apps/flask_app/routes.py
python-social-auth-0.1.17/%{py3dir}/social/apps/flask_app/template_filters.py
python-social-auth-0.1.17/%{py3dir}/social/apps/flask_app/utils.py
python-social-auth-0.1.17/%{py3dir}/social/apps/pyramid_app/__init__.py
python-social-auth-0.1.17/%{py3dir}/social/apps/pyramid_app/fields.py
python-social-auth-0.1.17/%{py3dir}/social/apps/pyramid_app/models.py
python-social-auth-0.1.17/%{py3dir}/social/apps/pyramid_app/utils.py
python-social-auth-0.1.17/%{py3dir}/social/apps/pyramid_app/views.py
python-social-auth-0.1.17/%{py3dir}/social/apps/tornado_app/fields.py
python-social-auth-0.1.17/%{py3dir}/social/apps/tornado_app/handlers.py
python-social-auth-0.1.17/%{py3dir}/social/apps/tornado_app/models.py
python-social-auth-0.1.17/%{py3dir}/social/apps/tornado_app/routes.py
python-social-auth-0.1.17/%{py3dir}/social/apps/tornado_app/utils.py
python-social-auth-0.1.17/%{py3dir}/social/apps/webpy_app/app.py
python-social-auth-0.1.17/%{py3dir}/social/apps/webpy_app/fields.py
python-social-auth-0.1.17/%{py3dir}/social/apps/webpy_app/models.py
python-social-auth-0.1.17/%{py3dir}/social/apps/webpy_app/utils.py
python-social-auth-0.1.17/%{py3dir}/social/backends/amazon.py
python-social-auth-0.1.17/%{py3dir}/social/backends/angel.py
python-social-auth-0.1.17/%{py3dir}/social/backends/aol.py
python-social-auth-0.1.17/%{py3dir}/social/backends/appsfuel.py
python-social-auth-0.1.17/%{py3dir}/social/backends/base.py
python-social-auth-0.1.17/%{py3dir}/social/backends/behance.py
python-social-auth-0.1.17/%{py3dir}/social/backends/belgiumeid.py
python-social-auth-0.1.17/%{py3dir}/social/backends/bitbucket.py
python-social-auth-0.1.17/%{py3dir}/social/backends/box.py
python-social-auth-0.1.17/%{py3dir}/social/backends/dailymotion.py
python-social-auth-0.1.17/%{py3dir}/social/backends/disqus.py
python-social-auth-0.1.17/%{py3dir}/social/backends/douban.py
python-social-auth-0.1.17/%{py3dir}/social/backends/dropbox.py
python-social-auth-0.1.17/%{py3dir}/social/backends/email.py
python-social-auth-0.1.17/%{py3dir}/social/backends/evernote.py
python-social-auth-0.1.17/%{py3dir}/social/backends/exacttarget.py
python-social-auth-0.1.17/%{py3dir}/social/backends/facebook.py
python-social-auth-0.1.17/%{py3dir}/social/backends/fedora.py
python-social-auth-0.1.17/%{py3dir}/social/backends/fitbit.py
python-social-auth-0.1.17/%{py3dir}/social/backends/flickr.py
[Bug 1036396] Review Request: python-social-auth - Social auth made simple
https://bugzilla.redhat.com/show_bug.cgi?id=1036396 Christopher Meng cicku...@gmail.com changed: What|Removed |Added CC||jdor...@redhat.com Flags||needinfo?(jdor...@redhat.co ||m) --- Comment #2 from Christopher Meng cicku...@gmail.com --- Please look at comment 1 and answer my question. -- 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@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1036396] Review Request: python-social-auth - Social auth made simple
https://bugzilla.redhat.com/show_bug.cgi?id=1036396 Christopher Meng cicku...@gmail.com changed: What|Removed |Added Status|NEW |ASSIGNED CC||cicku...@gmail.com Assignee|nob...@fedoraproject.org|cicku...@gmail.com Flags||fedora-review? -- 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@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1036396] Review Request: python-social-auth - Social auth made simple
https://bugzilla.redhat.com/show_bug.cgi?id=1036396 --- Comment #1 from Christopher Meng cicku...@gmail.com --- First, will you push it to EPEL? Issue need to be fixed first: Please write down files list explicitly in %files, don't use .files to ease the work. ;) -- 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@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
