valknar/llmx
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix: update scripts/build_container.sh to use pnpm instead of npm (#631)

Browse Source 
I suspect this is why some contributors kept accidentally including a
new `codex-cli/package-lock.json` in their PRs.

Note the `Dockerfile` still uses `npm` instead of `pnpm`, but that
appears to be fine. (Probably nicer to globally install as few things as
possible in the image.)
This commit is contained in:
Michael Bolin
2025-04-24 16:38:57 -07:00
committed by GitHub
parent b34ed2ab83
commit bb2d411043
1 changed files with 3 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
codex-cli/scripts/build_container.sh
View File

@@ -8,9 +8,9 @@ pushd "$SCRIPT_DIR/.." >> /dev/null || {
echo "Error: Failed to change directory to $SCRIPT_DIR/.." echo "Error: Failed to change directory to $SCRIPT_DIR/.."
exit 1 exit 1
} }
npm install pnpm install
npm run build pnpm run build
rm -rf ./dist/openai-codex-*.tgz rm -rf ./dist/openai-codex-*.tgz
npm pack --pack-destination ./dist pnpm pack --pack-destination ./dist
mv ./dist/openai-codex-*.tgz ./dist/codex.tgz mv ./dist/openai-codex-*.tgz ./dist/codex.tgz
docker build -t codex -f "./Dockerfile" . docker build -t codex -f "./Dockerfile" .