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

Option to remove external album art after embedding #733

Closed
RevenantOne opened this issue Apr 29, 2014 · 9 comments
Closed

Option to remove external album art after embedding #733

RevenantOne opened this issue Apr 29, 2014 · 9 comments
Labels
feature features we would like to implement

Comments

@RevenantOne
Copy link

I find the fetchart and embedart plugins really useful but once the album art has been embded I don't really have a use for the external cover.jpg file any more.

I'm unable to find any command to delete them or remove them from the library (please correct me if I'm wrong), so if I manually delete them beets will complain that they there are files missing for certain queries.

@sampsyo sampsyo changed the title Add option to remove external album art after embedding. Option to remove external album art after embedding Apr 29, 2014
@sampsyo
Copy link
Member

sampsyo commented Apr 29, 2014

Thanks!

I'm not sure where something like this should live. In one of the plugins? In the core?

Also, FWIW, this relates to #111, where we're developing a full arsenal of tools for dealing with attachments (of which album art is just a special case).

@RevenantOne
Copy link
Author

I would think it would make sense for it to be part of the embedart plugin as this issue occurs after embedding the art.

I read through the comments for #111 and saw the link to the wiki page which mentions the coverart plugin, but no mention of removed attached/image files after embedding.

@sampsyo
Copy link
Member

sampsyo commented Apr 30, 2014

Sure, an automatic hook to remove download art files would go in fetchart. But it seems like people might want to remove album art even if they're not embedding it (explicitly).

@pinchal
Copy link

pinchal commented Jun 5, 2014

I have the same problem, I don't know why theses covers are missing.
As a result I cannot apply any move: "Error: No such file or directory while moving ...".

@jimpriest
Copy link

Just getting started with Beets - love it so far!!

I'd love an option to purge / remove album art. Both files, and embedded info.

@sampsyo
Copy link
Member

sampsyo commented Nov 15, 2014

@jimpriest Cool! Explicit commands to remove art would probably be a different feature. If you're interested, can you open a separate ticket (with some thoughts about how this would work)?

@fohrums
Copy link

fohrums commented Dec 15, 2014

@samdoshi
what ever image embedart works with then have that reference to the image be the path for the rm command to be applied to prior to import.

or it could also work by the import command where it regards image file (.jpg,.png, etc).

But, to limit it through one action I would think having the image be deleted by the embedart command would be effecient enough since it's the one responsible.

@Schweinepriester
Copy link
Contributor

was about to open another issue for this, but here is just a push :)

i think this should be an option for embedart, something like: removeArtFile: yes|no

Sure, an automatic hook to remove download art files would go in fetchart. But it seems like people might want to remove album art even if they're not embedding it (explicitly).

is there a use case where one would fetch albumart but then remove it?

if not, embedart it is :)

@Schweinepriester
Copy link
Contributor

I just wrote a tiny (buggy? bad?) plugin, which fits for me so far: https://github.com/Schweinepriester/beets-removeartfile

Still hoping for a more sophisticated and better supported solution, though.

@sampsyo sampsyo closed this as completed in 9a76a1e Sep 7, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature features we would like to implement
Projects
None yet
Development

No branches or pull requests

6 participants