autoupdater to keep git up-to-date

[bufferoverflow]

• I was not able to find an open or closed issue matching what I'm seeing

Setup

• Windows Installer 32-bit and 64-bit

Details

• An autoupdater to keep git-for-windows up-to-date would be fantastic.

[dscho]

An autoupdater to keep git-for-windows up-to-date would be fantastic.

Wouldn't it? It would be as fantastic as a contribution adding this feature! 😄

[bufferoverflow]

This might be an option to integrate here: https://github.com/ravibpatel/AutoUpdater.NET

[dscho]

When I said "contribution", I did not mean "throw a random link at the Git for Windows maintainer who is busy releasing an emergency release".

I use this function.

git-update() {
    local yn="$1"
    case $yn in
        -\? | --?\?) echo "Usage: git-update [options]";
        echo "Options:";
        echo "  -y, --yes       Automatic yes to download and install prompt";
        return;;
    esac
    local bit=$(ls --color=never -d /mingw* | sed 's/\/mingw//')
    local releases=$(curl --silent https://api.github.com/repos/git-for-windows/git/releases/latest)
    local latest=$(echo "$releases" | grep '"tag_name": "v' | sed -E 's/.*"tag_name": "v([^"]*).*/\1/')
    local version=$(git --version | sed "s/git version //")
    echo "Git for Windows $version (${bit}bit)"
    if [ "$latest" != "$version" ]
    then
        echo "Update $latest is available"
        local download=$(echo "$releases" | grep '"browser_download_url": "' | grep "$bit\-bit\.exe" | sed -E 's/.*": "([^"]*).*/\1/')
        local filename=$(echo "$download" | sed -E 's/.*\/([^\/]*)$/\1/')
        case $yn in
            -[Yy]* | --[Yy]*) yn="y"; echo "Downloading $filename";;
            *) read -p "Download $filename and install [N/y]? " yn;;
        esac
        case $yn in
            [Yy]* ) curl -# -L -o $filename $download; start $filename; ps | grep bash | awk '{print "kill -9 " $2 ";" }' | sh;;
        esac
        return
    fi
    echo "Up to date"
}

UPDATE 2:

• Made all variables local as suggested by @landstander668
• Bit version now determined from existence of /mingw(32|64) instead of git-scm.com
• Changed process kills to @larsxschneider's updated parent of bash idea

UPDATE 1:

• Now uses the github repos releases URL to find the most up to date release thanks to @dscho
• Still using the git-scm.com URL to find out what bit version to download (open to suggestions on how to find this out without needing to hit a second URL)
• As was suggested by @dscho I've turned off the verbose download progress, and instead switch to a # progress bar because slow connections (like my home ADSL 2) take a lot longer than few seconds to download.
• As suggested by @dscho and @larsxschneider I added the auto run of the installer after download. Using a combination of start $filename; ps | grep 'mintty' | awk '{print $1}' | xargs kill -9 I'm able to detach the installer from the parent process and kill everthing that would prevent the installer from updating successfully. I unsure what the discussion about ConHost is about. For me killing 'mintty' processes seems to do it.

Usasge

$ git-update -?
Usage: git-update [options]
Options:
  -y, --yes       Automatic yes to download and install prompt

Output when up to date with latest version

$ git-update
Git for Windows 2.14.0.windows.2 (64bit)
Up to date

Output when not on the latest version

$ git-update
Git for Windows 2.14.0.windows.1 (64bit)
Update 2.14.0.windows.2 is available
Download Git-2.14.0.2-64-bit.exe and install [N/y]? y
######################################################################## 100.0%

[bufferoverflow]

@dscho I'm very sorry! Just provided that link because I think this might be an option to implement this. I'm willing to contribute be my self or via some of my co-workers. However, I have to get familiar with the code base and installer creation first.

[dscho]

@Between2Spaces that's a nice script! A couple of suggestions:

• use https://api.github.com/repos/git-for-windows/git/releases/latest instead of git-scm.com (the latter is only updated via a cron job, using the former)
• maybe download silently (with the -s option)?
• maybe launch the installer after it downloaded (must be with & so that the calling git.exe process can exit)?

[larsxschneider]

Indeed, nice script @Between2Spaces . You could launch the installer and kill the Git bash like this:

start "" "path/to/installer"
ps | grep mintty | awk '{print "kill -9 " $1 ";" }' | sh

[dscho]

ps | grep mintty | awk '{print "kill -9 " $1 ";" }' | sh

Nice command-line, but please keep in mind that Git Bash can be configured to be run via ConHost instead of MinTTY.

Thanks for your kind words. I've taken on board @dscho 's suggestions and updated the script (see previous post for the updated script).

The auto run of the installer is problematic. Thanks for the alternative process kill suggestion @larsxschneider, I haven't tried that yet but don't want to go down that path if "mintty" is not the right process name to be grepping for.

I've left my attempts in the updated script above, which means it's currently buggy. While it will find the latest version (and identify the correct 32/64 bit exe for your OS) and download it, the auto run of the installer currently fails. Leaving it up to the user to Cancel, find the dowloaded exe and re-run manually after ensuring all Git for Windows related processes are closed.

UPDATE: I seem to have solved the auto run using @larsxschneider's suggestion . Thought the issue about 'mintty' not always being the correct process to kill is something I'm unsure what to do about.

[dscho]

@Between2Spaces how about putting this under version control, e.g. into https://github.com/git-for-windows/build-extra/tree/master/git-extra?

[larsxschneider]

@dscho Would this be more general?

ps | grep bash | awk '{print "kill -9 " $2 ";" }' | sh

Here I grep for bash and use the parent process of bash (= $2) which should be MinTTY or something else. A minor annoyance in that solution is that grep seems to invoke a second bash process. But that doesn't matter as we kill everything anyways.

[landstander668]

@Between2Spaces I'd suggest making the internal variables local, to avoid inadvertently clobbering items which were set outside of the function.

--- git-update.orig     2017-08-08 12:48:05.307610800 -0500
+++ git-update  2017-08-08 12:48:58.443392900 -0500
@@ -1,21 +1,21 @@
 git-update() {
-    yn="$1"
+    local yn="$1"
     case $yn in
         -\? | --?\?) echo "Usage: git-update [options]";
         echo "Options:";
         echo "  -y, --yes       Automatic yes to download and install prompt";
         return;;
     esac
-    releases=$(curl --silent https://api.github.com/repos/git-for-windows/git/releases/latest)
-    latest=$(echo "$releases" | grep '"tag_name": "v' | sed -E 's/.*"tag_name": "v([^"]*).*/\1/')
-    version=$(git --version | sed "s/git version //")
+    local releases=$(curl --silent https://api.github.com/repos/git-for-windows/git/releases/latest)
+    local latest=$(echo "$releases" | grep '"tag_name": "v' | sed -E 's/.*"tag_name": "v([^"]*).*/\1/')
+    local version=$(git --version | sed "s/git version //")
     echo "Git for Windows $version"
     if [ "$latest" != "$version" ]
     then
         echo "Update $latest is available"
-        bit=$(curl -s https://git-scm.com/download/win | grep 'click here to download manually' | sed -E 's/.*-([0-9]+)-bit\.exe".*/\1/')
-        download=$(echo "$releases" | grep '"browser_download_url": "' | grep "$bit\-bit\.exe" | sed -E 's/.*": "([^"]*).*/\1/')
-        filename=$(echo "$download" | sed -E 's/.*\/([^\/]*)$/\1/')
+        local bit=$(curl -s https://git-scm.com/download/win | grep 'click here to download manually' | sed -E 's/.*-([0-9]+)-bit\.exe".*/\1/')
+        local download=$(echo "$releases" | grep '"browser_download_url": "' | grep "$bit\-bit\.exe" | sed -E 's/.*": "([^"]*).*/\1/')
+        local filename=$(echo "$download" | sed -E 's/.*\/([^\/]*)$/\1/')
         case $yn in
             -[Yy]* | --[Yy]*) yn="y";;
             *) read -p "Download $filename and install [N/y]? " yn;;

@dscho Probably a good idea to put it somewhere. Is build-extra the right spot? I don't have or intend to use build-extra's.

@landstander668 Awesome. Thanks for the improvements. I've updated the function.

@larsxschneider Good idea. I tried your parent of bash idea and it seems to work. I've incorporated it into the updated function.