docker-spotweb icon indicating copy to clipboard operation
docker-spotweb copied to clipboard

Select branch for spotweb

Open mesa57 opened this issue 3 years ago • 9 comments

A request for an enhancement : It would be nice if there comes some parameter to determine the branch to checkout. Default should be master, but other branches should be possible (like develop). This would also greatly help our team in testing issue's and solutions.

mesa57 avatar Aug 07 '21 10:08 mesa57

Sorry, I was of the grid for a while. I could do this, no problem. But then you'd have to build a new image yourself for a different version then master / stable.

jgeusebroek avatar Aug 15 '21 20:08 jgeusebroek

Master should be the default of course, so that would imply no change for current users. I understand that when a image is run, a new pull of spotweb is done ?

mesa57 avatar Aug 16 '21 09:08 mesa57

Ok, after reading some documentation I understand this is only possible at docker image build time. Maybe you could introduce a seperate tag for this purpose ? Last week 3 issues where reported which where all resolved in the develop branch. Users advised each other to switch to https://hub.docker.com/r/erikdevries/spotweb as that image uses the develop branch. I thought that this is not desirable.

Of course we will merge develop with master soon, but it is always lagging behind.

mesa57 avatar Aug 16 '21 10:08 mesa57

I could probably have 2 tags; master and develop.

jgeusebroek avatar Aug 16 '21 19:08 jgeusebroek

Do you have any idea how https://hub.docker.com/r/erikdevries/spotweb implements that ?

mesa57 avatar Aug 17 '21 09:08 mesa57

As far as I can see it is building the master branch?

https://github.com/edv/docker-spotweb/blob/master/x86/Dockerfile

jgeusebroek avatar Aug 17 '21 18:08 jgeusebroek

Hmmz, strange that users succesfully changed to that docker to solve an issue with IMDB which was solved in the spotweb/develop branch. Anyway : today we merged develop with master. I don't know if you have to update anything (tags?) for that. Thank you anyway, I will close the issue.

mesa57 avatar Aug 18 '21 09:08 mesa57

What's strange. Maybe they checked out a different build from within a running container? I've forced a new build, it should be done in a few minutes.

jgeusebroek avatar Aug 18 '21 20:08 jgeusebroek

For you're reference : the erikdevries image does not specify a branch. Therefore git will pull the default branch which we set to develop Still it would be nice if you're dockerfile gives an opportunity to select develop or master.

mesa57 avatar Nov 01 '21 16:11 mesa57