Script error on Git Bash (Windows)

[ybiquitous]

I encountered the following script error on Git Bash (Windows 7).

Environment:

• Windows 7 Professional SP1 64bit
• Git for Windows 2.7.1
• pre-git 3.3.0
• Node.js 4.3.0
• npm 3.7.2

Reproduction:
Open Git Bash and run the following commands on terminal.

mkdir foo
cd foo
npm init -y
git init
echo 'node_modules' >> .gitignore
echo 'npm-debug.log' >> .gitignore
npm install --save-dev pre-git
git add .
npm run commit

package.json.txt

npm-debug.log.txt

[ybiquitous]

Perhaps I think this error relates to #63.

[bahmutov]

Probably it is more related to #33

[bahmutov]

tried on Windows v3.4.1, seems to work, reopen if not fixed

[ybiquitous]

@bahmutov
When I tried pre-git@3.4.1, Windows JScript error did not occur.
But a following error occured on npm run commit.

C:\Users\koba\git\foo\node_modules\.bin\pre-commit:2
basedir=$(dirname "$(echo "$0" | sed -e 's,\\,/,g')")
          ^^^^^^^

SyntaxError: missing ) after argument list
    at exports.runInThisContext (vm.js:53:16)
    at Module._compile (module.js:373:25)
    at Object.Module._extensions..js (module.js:416:10)
    at Module.load (module.js:343:32)
    at Function.Module._load (module.js:300:12)
    at Function.Module.runMain (module.js:441:10)
    at startup (node.js:134:18)
    at node.js:962:3

This is the installed node_modules/.bin/pre-commit:

#!/bin/sh
basedir=$(dirname "$(echo "$0" | sed -e 's,\\,/,g')")

case `uname` in
    *CYGWIN*) basedir=`cygpath -w "$basedir"`;;
esac

if [ -x "$basedir/node" ]; then
  "$basedir/node"  "$basedir/../pre-git/bin/pre-commit.js" "$@"
  ret=$?
else 
  node  "$basedir/../pre-git/bin/pre-commit.js" "$@"
  ret=$?
fi
exit $ret

This is the installed .git/hooks/pre-commit:

#!/bin/sh

/usr/bin/env node --harmony ./node_modules/.bin/pre-commit "$@"

[ybiquitous]

@bahmutov

tried on Windows v3.4.1, seems to work, reopen if not fixed

Sorry, I have no right to reopen this issue.

[bahmutov]

Ok, I have reopened the issue @ybiquitous but the solution on Windows is up to you (pull requests are very welcome) - basically I don't have an access to Windows machine for development, and no use for supporting Windows, but if you can do this, it would be great and helpful to many people I think

[ybiquitous]

I will try it. It may take a little time...

[ybiquitous]

First, I tried modifying bin/pre-commit.js as follows:

- #!/usr/bin/env node
+ #!/usr/bin/env node --harmorny

This worked on Windows, but not on Linux.

If using env in shebang, to pass cli options such as --harmony would not work. Reason:

• http://stackoverflow.com/questions/4303128/how-to-use-multiple-arguments-with-a-shebang-i-e
• http://stackoverflow.com/questions/23388255/how-to-use-harmony-flag-in-my-node-app-with-phusion-passenger

So I wrote wrapper shell script such as bin/pre-commit for bin/pre-commit.js.
This wrapper script worked well on both Windows and Linux.

Please give me a review.