Skip to content

Fix Docker container scripts#47

Merged
tibo-openai merged 3 commits into
mainfrom
eburke/22/1
Apr 16, 2025
Merged

Fix Docker container scripts#47
tibo-openai merged 3 commits into
mainfrom
eburke/22/1

Conversation

@eburke-openai
Copy link
Copy Markdown
Collaborator

@eburke-openai eburke-openai commented Apr 16, 2025

Why/Context/Summary

Fix the build_container.sh and run_in_container.sh scripts.

  • Use public npm installation of codex
  • Control where the build script executes
  • Use the realpath of the workdir
  • Make sure we properly quote all the args to codex

Fixes #22

Signed-off-by:: Eric Burke <eburke@openai.com>
@tibo-openai tibo-openai marked this pull request as ready for review April 16, 2025 19:02
@tibo-openai tibo-openai merged commit b6846ce into main Apr 16, 2025
@tibo-openai tibo-openai deleted the eburke/22/1 branch April 16, 2025 19:02
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 gives an error — ERROR [4/7] COPY dist/codex.tgz codex.tgz`

2 participants