Skip to content

Fix Docker container scripts - #47

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

tibo-openai merged 3 commits into
mainfrom
eburke/22/1

Conversation

@eburke-openai

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

Copy link
Copy Markdown
Contributor
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
soddygo added a commit to nuwax-ai/codex that referenced this pull request Oct 21, 2026
…oth macOS hands

The openai#47 section recorded the restricting list as containing the user SID; it does not
on this leg, so the narrative and the receipt are rewritten against the two-list
evidence that now runs. Section 6.42 registers both macOS legs going terminal: the
same app_server_daemon leg died at the 60s bound on both hands, with zero assertion
failures, and both heads are ancestors of the unpushed widening commit, so neither
hand can judge it.
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