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

On Arch Linux, CDDB does not know how to install morituri. #28

Closed
BoBeR182 opened this issue Aug 12, 2016 · 8 comments
Closed

On Arch Linux, CDDB does not know how to install morituri. #28

BoBeR182 opened this issue Aug 12, 2016 · 8 comments
Labels
Support Questions that needs answering with no code changes needed or that only require a one time change

Comments

@BoBeR182
Copy link

Checking device /dev/sr0
CDDB disc id: 84081a0c
MusicBrainz disc id VJ3Z4dFI8V3VWPBbiV9J0g0xqjk-
MusicBrainz lookup URL https://musicbrainz.org/cdtoc/attach?toc=1+12+155706+150+2956+15144+25747+40355+61695+77412+87641+106201+112959+123863+136355&tracks=12&id=VJ3Z4dFI8V3VWPBbiV9J0g0xqjk-
Disc duration: 00:34:34.080, 12 audio tracks
Continuing without metadata
Submit this disc to MusicBrainz at the above URL.

Could not import python module 'CDDB'
This module is part of python-CDDB.
See http://cddb-py.sourceforge.net/ for more information.

On Arch Linux, CDDB does not know how to install morituri.
Please file a bug at:
http://thomas.apestaart.org/morituri/trac/newticket?summary=DEP%3A%20CDDB%2C%20Arch%20Linux
with instructions on how to install the dependency so we can add it.

Please install CDDB and try again.

You can confirm it is installed by starting Python and running:
import CDDB

@JoeLametta JoeLametta added the Rejected Proposal/issue which won't be addressed label Aug 21, 2016
@JoeLametta JoeLametta added this to the 2.0 milestone Aug 21, 2016
@JoeLametta
Copy link
Collaborator

Thanks for reporting this issue. This one isn't new.
It can be fixed but, right now, it isn't the best thing to do because, in the future, whipper is going to abandon python-deps (see #5) which causes this bug.

I'm keeping this issue open as reminder.

@JoeLametta
Copy link
Collaborator

@BoBeR182 As python-deps has already been dropped can you test if now it works correctly?

Thanks

@BoBeR182
Copy link
Author

BoBeR182 commented Sep 19, 2016

Checking device /dev/sr0
CDDB disc id: 84081a0c                        
MusicBrainz disc id VJ3Z4dFI8V3VWPBbiV9J0g0xqjk-
MusicBrainz lookup URL https://musicbrainz.org/cdtoc/attach?toc=1+12+155706+150+2956+15144+25747+40355+61695+77412+87641+106201+112959+123863+136355&tracks=12&id=VJ3Z4dFI8V3VWPBbiV9J0g0xqjk-
Disc duration: 00:34:34.080, 12 audio tracks
Continuing without metadata
Submit this disc to MusicBrainz at the above URL.

Traceback (most recent call last):
  File "/usr/sbin/rip", line 37, in <module>
    sys.exit(main.main(sys.argv[1:]))
  File "/usr/lib/python2.7/site-packages/morituri/rip/main.py", line 45, in main
    h.handleImportError(e)
NameError: global name 'h' is not defined

@JoeLametta
Copy link
Collaborator

@BoBeR182 Updated again: fixed some python-deps leftovers, try it again please.
Now, if I did it right, I expect it to raise an exception because of unmet dependencies.

@BoBeR182
Copy link
Author

BoBeR182 commented Sep 19, 2016

Using configured read offset 6
Checking device /dev/sr0
CDDB disc id: 84081a0c                        
MusicBrainz disc id VJ3Z4dFI8V3VWPBbiV9J0g0xqjk-
MusicBrainz lookup URL https://musicbrainz.org/cdtoc/attach?toc=1+12+155706+150+2956+15144+25747+40355+61695+77412+87641+106201+112959+123863+136355&tracks=12&id=VJ3Z4dFI8V3VWPBbiV9J0g0xqjk-
Disc duration: 00:34:34.080, 12 audio tracks
Continuing without metadata
Submit this disc to MusicBrainz at the above URL.

Traceback (most recent call last):
  File "/usr/sbin/rip", line 38, in <module>
    raise ImportError(e)
ImportError: No module named CDDB

@BoBeR182
Copy link
Author

Fixed by installing py-cddb

Using configured read offset 6
Checking device /dev/sr0
CDDB disc id: 84081a0c                        
MusicBrainz disc id VJ3Z4dFI8V3VWPBbiV9J0g0xqjk-
MusicBrainz lookup URL https://musicbrainz.org/cdtoc/attach?toc=1+12+155706+150+2956+15144+25747+40355+61695+77412+87641+106201+112959+123863+136355&tracks=12&id=VJ3Z4dFI8V3VWPBbiV9J0g0xqjk-
Disc duration: 00:34:34.080, 12 audio tracks
Continuing without metadata
Submit this disc to MusicBrainz at the above URL.

FreeDB identifies disc as The Knockouts / Main Attraction

@JoeLametta
Copy link
Collaborator

Fixed by installing py-cddb

Great!
Yes, that package is required (mentioned in whipper's README as python-cddb but, as you've written, it's named cddb-py in Arch Linux).

I'll check if cddb-py is a strict requirement: in that case I'll ask @neitsab if he can update his AURwhipper-git package to mark it as a dependency (not optional anymore).

@BoBeR182 Do you consider this issue to be solved now? (can it be closed?)

@BoBeR182
Copy link
Author

Yup

@JoeLametta JoeLametta added Support Questions that needs answering with no code changes needed or that only require a one time change and removed Rejected Proposal/issue which won't be addressed labels Nov 12, 2018
@JoeLametta JoeLametta removed this from the 2.0 milestone Nov 12, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Support Questions that needs answering with no code changes needed or that only require a one time change
Projects
None yet
Development

No branches or pull requests

2 participants