Skip to content

Fix Docker Build permission issue#95

Merged
sethtroisi merged 1 commit intogoogle-gemini:mainfrom
sethtroisi:fix_node_perm
Apr 21, 2025
Merged

Fix Docker Build permission issue#95
sethtroisi merged 1 commit intogoogle-gemini:mainfrom
sethtroisi:fix_node_perm

Conversation

@sethtroisi
Copy link
Copy Markdown
Contributor

During docker buildnpm install running as node was exiting with 243 (EACCES) from trying to install the tgz files because npm pack created the files with 400 permissions on my system.

Docker running as node was exiting with 243 (EACCES) from trying to
install the tgz files because npm pack created the files with 400
permissions on my system.
Copy link
Copy Markdown
Collaborator

@NTaylorMullen NTaylorMullen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good catch!

@sethtroisi sethtroisi merged commit 7663ccf into google-gemini:main Apr 21, 2025
2 checks passed
@sethtroisi sethtroisi deleted the fix_node_perm branch April 21, 2025 21:35
keithballinger pushed a commit that referenced this pull request Jun 13, 2025
During docker build `npm install` running as node was exiting with 243 (EACCES) from trying to install the tgz files because `npm pack` created the files with 400 permissions on my system.
yaleh pushed a commit to yaleh/gemini-cli that referenced this pull request Jun 29, 2025
During docker build `npm install` running as node was exiting with 243 (EACCES) from trying to install the tgz files because `npm pack` created the files with 400 permissions on my system.
viridivn pushed a commit to viridivn/gemini-cli that referenced this pull request Jun 29, 2025
During docker build `npm install` running as node was exiting with 243 (EACCES) from trying to install the tgz files because `npm pack` created the files with 400 permissions on my system.
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.

2 participants