Bug 2134

Summary:	Review Request: pyspotify - Python bindings for libspotify
Product:	Package Reviews	Reporter:	Jonathan Dieter <jdieter>
Component:	Review Request	Assignee:	RPM Fusion Package Review <rpmfusion-package-review>
Status:	RESOLVED EXPIRED
Severity:	normal	CC:	avibrazil, d0malaga, hobbes1069, rpmfusion-package-review, sergio, stein.magnus, tchollingsworth
Priority:	P5
Version:	Current
Hardware:	All
OS:	GNU/Linux
namespace:	nonfree
Bug Depends on:	2133
Bug Blocks:	2135

Description Jonathan Dieter 2012-01-05 14:55:23 CET

Spec URL: http://www.lesbg.com/jdieter/pyspotify.spec
SRPM URL: http://www.lesbg.com/jdieter/pyspotify-1.6.1-1.fc16.src.rpm
Description:
Python bindings for the official Spotify API

Why not in Fedora:
Requires libspotify which is proprietary

Comment 1 Jonathan Dieter 2012-01-05 14:56:28 CET

$ rpmlint pyspotify.spec 
0 packages and 1 specfiles checked; 0 errors, 0 warnings.

$ rpmlint ../RPMS/x86_64/pyspotify-1.6.1-1.fc16.x86_64.rpm 
pyspotify.x86_64: W: spelling-error Summary(en_US) libspotify -> spottily
1 packages and 0 specfiles checked; 0 errors, 1 warnings.

$ rpmlint ../SRPMS/pyspotify-1.6.1-1.fc16.src.rpm 
pyspotify.src: W: spelling-error Summary(en_US) libspotify -> spottily
1 packages and 0 specfiles checked; 0 errors, 1 warnings.

Comment 2 T.C. Hollingsworth 2013-12-07 18:05:10 CET

If you're still interested in getting this and mopidy into RPMFusion, please update them and I'll review.

Comment 3 Jonathan Dieter 2013-12-07 20:24:12 CET

I've actually switched to Google Play All-Access, so I'm no longer using Spotify.  I don't mind if someone else wants to take over the review request, otherwise I'll just close it.

Comment 4 Avi Alkalay 2015-03-26 15:30:50 CET

I'd like to take over the submission of this new package

http://avi.alkalay.net/software/pyspotify/pyspotify-2.0.0b4-1.fc21.src.rpm
http://avi.alkalay.net/software/pyspotify/pyspotify.spec

I used Dieter's SPEC, updated to upstream and fixed some build dependencies.

Comment 5 Avi Alkalay 2016-08-04 09:39:01 CEST

New packages on http://avi.alkalay.net/software/pyspotify/

What is needed to get this approved ?

Thank you

Comment 6 Richard 2016-08-04 14:50:29 CEST

Since it looks like T.C. is no longer reviewing this request you'll need to find a reviewer. I may have time in the near future but not right at this moment. It would be best to ask on the RPM Fusion developer mailing list. It's also helpful to offer a review swap.

Comment 7 Sérgio Basto 2016-10-02 00:34:28 CEST

(In reply to Avi Alkalay from comment #5)
> New packages on http://avi.alkalay.net/software/pyspotify/
> 
> What is needed to get this approved ?
> 
> Thank you

You need follow http://rpmfusion.org/Contributors , points 1, 2 and 3 


Spec URL: http://avi.alkalay.net/software/pyspotify/pyspotify.spec
SRPM URL: http://avi.alkalay.net/software/pyspotify/pyspotify-2.0.5-2.fc23.src.rpm

we need theses urls to run : 
fedora-review --other-bz https://bugzilla.rpmfusion.org -b 2134 -m fedora-rawhide-x86_64-rpmfusion_nonfree

Comment 8 Sérgio Basto 2016-10-02 01:00:24 CEST

WARNING: Cannot download url: https://github.com/mopidy/pyspotify/archive/pyspotify-2.0.5.tar.gz

ERROR: 'mock build failed, see /home/sergio/rpmbuild/2134-pyspotify/results/build.log'

Download error on https://pypi.python.org/simple/cffi/: [Errno 104] Connection reset by peer -- Some packages may not be found!
Couldn't find index page for u'cffi' (maybe misspelled?)
Download error on https://pypi.python.org/simple/: [Errno 104] Connection reset by peer -- Some packages may not be found!
No local packages or working download links found for cffi>=1.0.0

Also https://fedoraproject.org/wiki/Packaging:Python ,  Must: Python modules must not download any dependencies during the build process. 

So many work to do.