Skip to content
This repository was archived by the owner on Nov 9, 2017. It is now read-only.

Use 32-bit cmd.exe to open .sh files even on 64-bit Windows to avoid crashes - #26

Merged
sschuberth merged 1 commit into
msysgit:develfrom
mikofski:devel
Jun 7, 2012
Merged

sschuberth merged 1 commit into
msysgit:develfrom
mikofski:devel

Conversation

@mikofski

@mikofski mikofski commented Jun 7, 2012

Copy link
Copy Markdown
Member

use %WINDIR%\SysWOW64\cmd /C to run "bash.exe --login %1 %*"
use {syswow64} Inno setup variable to determine which cmd version to use (x64 or x86)

tested on Windows XP x64 Edition, Version 2003, SP2

  • registry keys are correct
  • test running a bash script which opens vim - works!
    *

address issue #12 and msysgit/git issue #8

@hvoigt

hvoigt commented Jun 7, 2012

Copy link
Copy Markdown
Member

Testing your code in Windows XP 32bit now. Please cleanup your commit message according to sschuberth's comments and we can apply it.

@hvoigt

hvoigt commented Jun 7, 2012

Copy link
Copy Markdown
Member

Ah and of course the code as well...

@hvoigt

hvoigt commented Jun 7, 2012

Copy link
Copy Markdown
Member

My test on Windows XP 32bit was successful.

@mikofski

mikofski commented Jun 7, 2012

Copy link
Copy Markdown
Member Author

@hvoigt and @sschuberth I hope you can cherry pick the last commit cb1e46944eca118629763740759de57d1e3ed305

@sschuberth

Copy link
Copy Markdown
Contributor

Cherry-picking just your second commit will not suffice as it depends on your first commit. Please squash these two commits into one (using "git rebase -i HEAD~2").

@mikofski

mikofski commented Jun 7, 2012

Copy link
Copy Markdown
Member Author

even tho I already pushed the changes, ok

Sent via Hubroid

@sschuberth

Copy link
Copy Markdown
Contributor

Yes, as your branch in not merged yet it's fine to rebase and force-push it.

use %WINDIR%\SysWOW64\cmd /C to run "sh.exe --login %1 %*"
use {syswow64} variable to determine which cmd to use

addresses issue msysgit#12

Signed-off-by: Mark Mikofski <[email protected]>
@mikofski

mikofski commented Jun 7, 2012

Copy link
Copy Markdown
Member Author

ok, I squashed the commits on my end, but what is going to happen to the pull request when I push?
should I close this pull requests and submit a new one? I apologize for my n00bness - thanks for your help

@sschuberth

Copy link
Copy Markdown
Contributor

AFAIK this pull request will update automatically if you force-push your rebased branch, so there's no need to file a new pull request.

@mikofski

mikofski commented Jun 7, 2012

Copy link
Copy Markdown
Member Author

it worked!

sschuberth added a commit that referenced this pull request Jun 7, 2012
Use 32-bit cmd.exe to open .sh files even on 64-bit Windows to avoid crashes
@sschuberth
sschuberth merged commit f9b4a27 into msysgit:devel Jun 7, 2012
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants