search

larsks / docker-image-logitech-media-server

MIT License
33 stars 74 forks source link

Mysqueezebox.com offer a REST call to determine the latest available … #8

Closed mehstg closed 7 years ago

mehstg commented 7 years ago

…nightly build. I have incorporated this in to the docker build as the existing link is dead.

thenktor commented 7 years ago

Is there also a call to get the amd64 package?

mehstg commented 7 years ago

Unfortunately not by the looks of it. I did look for that.

The only way I can see is some nasty SED, but hardly a clean approach: curl "http://www.mysqueezebox.com/update/?version=7.9.0&revision=1&geturl=1&os=deb" | sed 's/all/amd64/g'

thenktor commented 7 years ago

Yes it's possible with sed, but I also thought it's not perfect. If using sed, do it at least like this: sed 's/_all\.deb/_amd64\.deb/g'

jmccoy555 commented 7 years ago

Worked perfectly for me. With regular updates, what about an option to pull the latest version on start up?

abufilba commented 7 years ago

Gets my vote; at least gets latest version upon image build; updating images on docker startup is orthogonal to the way docker works. This is the best version of the code and should go onto master. Just need a regular automated build to push to docker.io if the image has been mutated.

I can set something up to delta the url and then push to docker.io but to keep to the same naming convention it needs to come from the repo owner really....

I have just built this docker image locally with the latest version http://downloads.slimdevices.com/nightly/7.9/sc/afcfc6d/logitechmediaserver_7.9.0~1480066117_amd64.deb

and it works fine for me.

pgroce commented 7 years ago

+1 for merging this PR, for all the reasons stated above. The last commit to Logitech/slimserver was two days ago, so some change is needed at any rate.

RichieB2B commented 7 years ago

+1 for merging this PR. Pulling the lastest version on image build is very sensible.