search

OpenFactorioServerManager / factorio-server-manager

A tool to help manage Factorio multiplayer servers including mods and save games.
MIT License
561 stars 135 forks source link

fix download URL in docker build due to project move #199

Closed MichaelPereira closed 3 years ago

MichaelPereira commented 3 years ago

quick fix to allow the docker build to download the release package since the project has been moved under a new owner

MichaelPereira commented 3 years ago

I just realized that there is a lot of rework happening on the develop branch with the same problem over there. To avoid conflicts, I can PR this same change over there as well (seems to be a fast-forward operation locally), since it would likely be quite disruptive to rebase nearly a year's worth of commits.

Sorry I didn't realize it sooner @knoxfighter but now I'm eager to try out all these improvements to the docker side of the project :)

mroote commented 3 years ago

Thanks for the PR @MichaelPereira

mroote commented 3 years ago

I tried switching the target branch to develop but we have had a few changes in that dockerfile since. Could you make the same change in the develop branch?

knoxfighter commented 3 years ago

I have fixed that merging error. We also provide the current develop branch as docher images on the dockerhub, so you don't need to build the image yourself: ofsm/ofsm:develop

knoxfighter commented 3 years ago

I forgot to merge this, it is now fixed with PR #200 Sorry for my fail