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

documentation should say "do not put spaces in your home directory" #134

Open
strega-nil opened this issue Apr 1, 2020 · 5 comments
Open
Assignees
Labels

Comments

@strega-nil
Copy link

Hook 'vpython.bat src/electron/script/apply_all_patches.py src/electron/patches/config.json' took 15669.14 secs
subprocess2(154) __init__:python3 src/electron/script/update-external-binaries.py;  cwd=C:\src\electron
________ running 'python3 src/electron/script/update-external-binaries.py' in 'C:\src\electron'
Downloading https://electron-build-tools.s3-us-west-2.amazonaws.com/build-dependencies/b6a20fd1c2026f3792e7286bc768a7ebc261847b76449b49f55455e1f841fecd/sccache-win32-x64.zip to C:\Users\NICOLE~1\AppData\Local\Temp\electron-xzab9lsb\sccache-win32-x64.zip
Download sccache-win32-x64.zip     4151947  [103798675.0%]
Hook 'python3 src/electron/script/update-external-binaries.py' took 11.25 secs
subprocess2(154) __init__:vpython.bat -c import os, subprocess; os.chdir(os.path.join("src", "electron")); subprocess.check_call(["python", "script/lib/npx.py", "[email protected]", "install", "--frozen-lockfile"]);;  cwd=C:\src\electron
________ running 'vpython.bat -c import os, subprocess; os.chdir(os.path.join("src", "electron")); subprocess.check_call(["python", "script/lib/npx.py", "[email protected]", "install", "--frozen-lockfile"]);' in 'C:\src\electron'
Error: EPERM: operation not permitted, mkdir 'C:\Users\Nicole'
@MarshallOfSound
Copy link
Member

Error: EPERM: operation not permitted, mkdir 'C:\Users\Nicole'

This is weird given it's supposedly init'ing in 'C:\src\electron

Is there any more of a trace where that error comes from / what script?

@strega-nil
Copy link
Author

Kind of; unfortunately, I can't redirect to a file, or I get a different error, but here's the amount I could copy from the backlog: https://gist.github.com/strega-nil/3fbd34423486a1073d6c5d87044940ca

@MarshallOfSound
Copy link
Member

@strega-nil If you run npx yarn --version on your system, what happens?

@strega-nil
Copy link
Author

I had the latest version from npm. Unfortunately, I just reinstalled windows, and my home directory no longer contains a space; I'll have to add a new user that does have a space in the home directory for future debugging.

@ckerr
Copy link
Member

ckerr commented Sep 15, 2022

This topic came up today at the Electron Summit when discussing build-tools, and the consensus was that the edge cases that come from this extend into gn and may be outside of the scope of this repo to fix.

So we're probably not going to fix this, but we should add a warning somewhere e.g. in the README

@ckerr ckerr changed the title e init fails on windows when there are spaces in your home directory documentation should say "do not put spaces in your home directory" Sep 15, 2022
@ckerr ckerr self-assigned this Sep 15, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants