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

"git-sh-setup: file not found" during `git submodule update` #2420

Open
raphinesse opened this Issue Jun 7, 2018 · 3 comments

Comments

Projects
None yet
3 participants
@raphinesse
Copy link

raphinesse commented Jun 7, 2018

Command failed: C:\Program Files\Git\cmd\git.EXE submodule update -q --init --recursive
C:/Program Files/Git/mingw64/libexec/git-core\git-submodule: line 19: .: git-sh-setup: file not found

Interesting enough, this only happens with environment.matrix.nodejs_version: "10". For node 6 and 8 this error does not occur. Here's a failing build for example.

@raphinesse

This comment has been minimized.

Copy link
Author

raphinesse commented Jun 7, 2018

This workaround fixes it for us:

test_script:
  # Workaround for https://github.com/appveyor/ci/issues/2420
  - set "PATH=%PATH%;C:\Program Files\Git\mingw64\libexec\git-core"
  - npm test

huan added a commit to wechaty/wechaty-getting-started that referenced this issue Jul 19, 2018

@admosity

This comment has been minimized.

Copy link

admosity commented Aug 15, 2018

Thanks @raphinesse for the workaround. This was happening on some local setups (in which case we just added that globally to our $PATH).

I have a feeling this may have to do with firing commands in a sub shell environment that may not have the same startup scripts.

@XhmikosR

This comment has been minimized.

Copy link
Contributor

XhmikosR commented Oct 14, 2018

Does this happen with 2.19.1 locally? I have #2683 for the update.

nornagon added a commit to electron/electron that referenced this issue Mar 5, 2019

deepak1556 added a commit to electron/electron that referenced this issue Mar 6, 2019

deepak1556 added a commit to electron/electron that referenced this issue Mar 7, 2019

MarshallOfSound added a commit to electron/electron that referenced this issue Mar 8, 2019

MarshallOfSound added a commit to electron/electron that referenced this issue Mar 8, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.