whipper
whipper copied to clipboard
Edit tags before rip the CD without needing to use Musicbrainz database.
Hello, would be nice if we could Edit the CD and tracks informations manually without needing the Musicbrainz database because sometimes we found wrong filenames and CD titles on Musicbrainz.
I would also like this feature. The use cases that I have for it are:
-
I have a few music CD-Rs that don't belong in MusicBrainz but I would like to add the metadata before ripping.
-
For some of my CDs the metadata is in MusicBrainz and is correct but I would like to tailor the metadata for my needs. For example shortening lengthy titles or adding a series name to the title so all albums in a series are together when sorted.
-
I make a minor correction in MusicBrainz but have to wait 7 days for the change to be applied. I would like to make the same change to track list whipper uses so I can immediately rip the Cd instead of waiting a week.
I know it is possible to use other programs to edit metadata afterwards but then the cue sheet titles won't match. If whipper opened the metadata in a text editor for editing it would also be possible to store this text file in the output directory so if I ever need to rerip the CD I could reuse the text file with my modifications.
Just created an account here.
This feature would really be cool to have. The use cases are obviously, I can absolutely confirm srussels thoughts on this.
This patch is a hack I have been using for a few months to work around this issue. It adds an "--edit" option that writes the metadata to a text file, opens it in an editor for the user to update, then parses it back into whipper before continuing with the rip.
This patch is a hack I have been using for a few months to work around this issue. It adds an "--edit" option that writes the metadata to a text file, opens it in an editor for the user to update, then parses it back into whipper before continuing with the rip.
@srussel I can't seem to apply that to the latest version -- would you mind submitting a PR or putting it up on a fork?
@Californian https://github.com/srussel/whipper/commits/edit-metadata
Very cool hack tbh, it would be cool to have it done like this in official whipper.
@Californian https://github.com/srussel/whipper/commits/edit-metadata
Yusss thank you!
@srussel I tested https://github.com/srussel/whipper/commits/edit-metadata with CD which occurs #453 .
I can edit metadata with env EDITOR=pluma whipper cd -d /dev/sr1 rip --edit
.But edited title was not used and fails to rip.
@oech3 I have rebased my patch https://github.com/srussel/whipper/tree/edit-metadata
It works for me. If it still does not work, try putting some print statements in readMetadata()
. That is where the text file is parsed and updates the whipper metadata.
@oech3 I found I was only updating the title, not the disambiguated title, which depending on whether you were using %d
or %D
for the disc title may not have worked. I have updated my patch to support editing both.
@srussel Thankyou.I can rip now. I failed to rip due to output_directory/soundtrack/'Various Artists - VERYLONGTITLE' was not editable.
thanks for the work @srussel ! this helped me out a bunch today. hopefully this gets merged w main branch soon enough.