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

fix: docker build and run corrections #259

Merged
merged 2 commits into from
Oct 21, 2024

Conversation

chriswilty
Copy link
Member

@chriswilty chriswilty commented Oct 18, 2024

There was a problem with gem 'ffi' brought in via gem github-pages:
ffi/ffi#1103

I've applied the fix in our Gemfile, with a clarifying comment.

I also fixed a path-related problem when using Git bash on Windows, as Docker does not recognise Git bash's linuxified drive paths, such as /c/dev/hello-world. This must be translated to C:/dev/hello-world. Now both build and serve work in Git bash on Windows machines 🥳

resolves #257

@chriswilty chriswilty requested a review from a team October 18, 2024 14:26
@chriswilty chriswilty self-assigned this Oct 18, 2024
@chriswilty
Copy link
Member Author

Obvs I'd love someone to try this out 🧐

If you already have a Gemfile.lock file and/or container_gem_cache folder you should probably delete those, to get a clean build.

@chriswilty chriswilty merged commit 1cbe74e into ScottLogic:gh-pages Oct 21, 2024
1 check passed
@chriswilty chriswilty deleted the 257-docker-fix branch October 21, 2024 14:33
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.

Docker build is broken
3 participants