Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add output folder, skip already ripped #4

Open
wants to merge 16 commits into
base: master
Choose a base branch
from

Conversation

AndersTornkvist
Copy link

Already ripped tracks will be skipped when ripping a playlist. The user can provide an optional fourth argument to change the output dir.

…mands properly. Added optional output directory. Added --ignoreerrors (skip to next track in playlist on error).
playback = False # set if you want to listen to the tracks that are currently ripped (start with "padsp ./jbripper.py ..." if using pulse audio)
# Music library imports
import fnmatch
import eyed3

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

for me it has to be "import eyeD3" to be working on Ubuntu 13.10 and Python 2.7.4. I'm not good in Python but would it be possible to check for both or depending on python version?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Strange! It would be solved by changing this line to:

try:
    import eyed3
except ImportError:
    import eyeD3 as eyed3

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I updated to eyeD3 version 0.7.4 and it works now with "eyed3". Before I was using something version 0.6.x. Thanks!

wolkenschieber and others added 8 commits December 25, 2014 11:18
- Added link types album and artist
- Cover downloading only if exists
@see api.h of pyspotify lilbrary
Added link types album, artist and starred. Updated README.md. Set preferred bitrate to 320kps.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants