Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

[WIP] Burgwar Docker image - Xmake compatible version #64

Merged
merged 17 commits into from
Nov 11, 2021
Merged

[WIP] Burgwar Docker image - Xmake compatible version #64

merged 17 commits into from
Nov 11, 2021

Conversation

Elanis
Copy link
Contributor

@Elanis Elanis commented Aug 17, 2021

This is the first working version of Docker image.

How to test it ?

  • git clone the folder
  • Build image with docker build --tag burgwar-server:latest .
  • Run server:
docker run -ti --rm --name=burgwar-server -p 14768:14768/udp \
 -v /relativeOrAbsolute/path/to/assets:/srv/assets/ \
 -v /relativeOrAbsolute/path/to/mods:/srv/mods/ \
 -v /relativeOrAbsolute/path/to/scripts:/srv/scripts/ \
 -v /absolute/path/to/serverconfig.lua:/srv/serverconfig.lua \
 burgwar-server:latest

Warning: If you are using Docker Desktop on Windows, you should execute this command under a WSL console. Executing with Git bash for example won't work.

TO DO before merge

  • use /home/burgwar instead of /home/burgwar/BurgWar
  • use default xmake.lua
  • Only keep artifacts to make a light image
  • Set server path in runtime image to /srv/
  • stop using git clone and use local repo ?

Next steps

TODO:
- stop using git clone and use local repo
- use environment variables instead of serverconfig.lua
- create empty assets folder
- use /home/burgwar instead of /home/burgwar/BurgWar
- use custom map file
- remove nazara legacy apt when we migrated
@CLAassistant
Copy link

CLAassistant commented Aug 17, 2021

CLA assistant check
All committers have signed the CLA.

@SirLynix
Copy link
Member

Thanks!

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants