Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Docker4master #242

Closed
wants to merge 28 commits into from
Closed

Docker4master #242

wants to merge 28 commits into from

Conversation

LynxAbraxas
Copy link
Contributor

This PR is only meant as a reference for #13 (which used LynxAbraxas:master instead of LynxAbraxas:docker4master) and was closed instantly because #23 was merged in favor of the approach taken here.
If needed, the commits of this PR are basically the history of a160504.

LynxAbraxas and others added 28 commits October 8, 2018 19:59
currently, the pwd has to be set to where ctp2 exec is located (otherwise files are not found)
ctp2 saves user setting there, so it has to be writeable for the executing user.
paths adjusted for civctp2 repos
This commit adds Dockerfile and docker_build.sh script for building ctp2
inside docker container.
making sure docker context does not contain files from direct build which can otherwise lead to build errors due to mismatche in build tool versions (e.g. libtool)
@LynxAbraxas
Copy link
Contributor Author

Closed instantly because #23 was merged in favor of the approach taken here.

@LynxAbraxas LynxAbraxas closed this Jan 5, 2020
@LynxAbraxas LynxAbraxas mentioned this pull request Jan 5, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants