ampache / ampache-docker

Ampache docker package
http://ampache.org
89 stars 67 forks source link

Beets catalog #75

Open kuzi-moto opened 2 years ago

kuzi-moto commented 2 years ago

Should we include Beets so that the beets catalog type works? Requires Beets to be installed to work properly.

lachlan-00 commented 2 years ago

if it requires it then we should include it.

lachlan-00 commented 2 years ago

do we just need to add the beets package (https://packages.debian.org/bullseye/beets) or is there anything else the catalog requires?

kuzi-moto commented 2 years ago

I think it only requires that the beet command is available, so I think just installing the package should do it. I could try testing it out if we wanted to be safe.

lachlan-00 commented 2 years ago

it's not a big package so i've included it with logrotate config as well

the new images have been built to include beets so have a go

tgdnt commented 4 months ago

@lachlan-00 thanks for your work on this. I saw your commit, but I don't see beets added there, only logrotate and when I tested, I could not use a beets catalog. I edited the Dockerfile and tested it, and that is the only thing missing to get beets catalogs working with the docker deployment. Do you want me to create a PR to close this ticket with?

lachlan-00 commented 4 months ago

might help to actually ADD the package...

lachlan-00 commented 4 months ago

i'm building new images for 6.3.0 with beets added to the dockerfile