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.

Sign up for GitHub

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

Jump to bottom

better git path handling #865

Merged
merged 5 commits into from
Apr 8, 2016
Merged

better git path handling #865

merged 5 commits into from
Apr 8, 2016

Commits on Apr 8, 2016

  1. init.bat: check for git in path 

    The idea is:
* if the users points as to a specific git, use that
* test if a git is in path and if yes, use that
* last, use our vendored git

We don't make any attempt to guess a different location, if a user wants their
own git install, they have to choose "add git to path".

Also check that we have a recent enough version of git (e.g. test for
<GIT>\cmd\git.exe)
    @jankatins
    jankatins committed Apr 8, 2016
    Configuration menu
    Copy the full SHA
    182629d View commit details
    Browse the repository at this point in the history

  2. init.bat: integrate the unix commands at the end of PATH 

    Some unix commands shadow windows commands and therefor the unix commands should
be at the end of the PATH
    @jankatins
    jankatins committed Apr 8, 2016
    Configuration menu
    Copy the full SHA
    2ccc06e View commit details
    Browse the repository at this point in the history

  3. init.bat: remove GIT\bin from path 

    The only reason was bash but bash.exe is also in <GIT>\usr\bin.
    @jankatins
    jankatins committed Apr 8, 2016
    Configuration menu
    Copy the full SHA
    fbccc6b View commit details
    Browse the repository at this point in the history

  4. PS: also add cmd and bin version of git to the path 

    This ensures that the "normal" path entry (cmd) is first and bash is still reachable via the bin path.
    @jankatins
    jankatins committed Apr 8, 2016
    Configuration menu
    Copy the full SHA
    1e7f151 View commit details
    Browse the repository at this point in the history

  5. Remove @ in new code, echo off is enough

    @jankatins
    jankatins committed Apr 8, 2016
    Configuration menu
    Copy the full SHA
    6c6268e View commit details
    Browse the repository at this point in the history