From 89cce78cf1d5a9ca7b0fe840d0035a2ba4c16ec2 Mon Sep 17 00:00:00 2001 From: Derrick Stolee Date: Thu, 29 Apr 2021 10:58:39 -0400 Subject: [PATCH 1/3] update-microsoft-git: create barebones builtin Just do the boilerplate stuff of making a new builtin, including documentation and integration with git.c. Signed-off-by: Derrick Stolee --- .gitignore | 1 + Documentation/git-update-microsoft-git.txt | 24 ++++++++++++++++++++++ Makefile | 1 + builtin.h | 1 + builtin/update-microsoft-git.c | 20 ++++++++++++++++++ git.c | 1 + 6 files changed, 48 insertions(+) create mode 100644 Documentation/git-update-microsoft-git.txt create mode 100644 builtin/update-microsoft-git.c diff --git a/.gitignore b/.gitignore index fe226cbbe8643a..d5c886dab572c4 100644 --- a/.gitignore +++ b/.gitignore @@ -173,6 +173,7 @@ /git-unpack-file /git-unpack-objects /git-update-index +/git-update-microsoft-git /git-update-ref /git-update-server-info /git-upload-archive diff --git a/Documentation/git-update-microsoft-git.txt b/Documentation/git-update-microsoft-git.txt new file mode 100644 index 00000000000000..724bfc172f8ab7 --- /dev/null +++ b/Documentation/git-update-microsoft-git.txt @@ -0,0 +1,24 @@ +git-update-microsoft-git(1) +=========================== + +NAME +---- +git-update-microsoft-git - Update the installed version of Git + + +SYNOPSIS +-------- +[verse] +'git update-microsoft-git' + +DESCRIPTION +----------- +This version of Git is based on the Microsoft fork of Git, which +has custom capabilities focused on supporting monorepos. This +command checks for the latest release of that fork and installs +it on your machine. + + +GIT +--- +Part of the linkgit:git[1] suite diff --git a/Makefile b/Makefile index 67ddd99891755e..8aefc46034f774 100644 --- a/Makefile +++ b/Makefile @@ -1176,6 +1176,7 @@ BUILTIN_OBJS += builtin/tag.o BUILTIN_OBJS += builtin/unpack-file.o BUILTIN_OBJS += builtin/unpack-objects.o BUILTIN_OBJS += builtin/update-index.o +BUILTIN_OBJS += builtin/update-microsoft-git.o BUILTIN_OBJS += builtin/update-ref.o BUILTIN_OBJS += builtin/update-server-info.o BUILTIN_OBJS += builtin/upload-archive.o diff --git a/builtin.h b/builtin.h index 2470d1cd3a267a..0573c1f11be12b 100644 --- a/builtin.h +++ b/builtin.h @@ -230,6 +230,7 @@ int cmd_tar_tree(int argc, const char **argv, const char *prefix); int cmd_unpack_file(int argc, const char **argv, const char *prefix); int cmd_unpack_objects(int argc, const char **argv, const char *prefix); int cmd_update_index(int argc, const char **argv, const char *prefix); +int cmd_update_microsoft_git(int argc, const char **argv, const char *prefix); int cmd_update_ref(int argc, const char **argv, const char *prefix); int cmd_update_server_info(int argc, const char **argv, const char *prefix); int cmd_upload_archive(int argc, const char **argv, const char *prefix); diff --git a/builtin/update-microsoft-git.c b/builtin/update-microsoft-git.c new file mode 100644 index 00000000000000..329f6d77c0d1c9 --- /dev/null +++ b/builtin/update-microsoft-git.c @@ -0,0 +1,20 @@ +#include "builtin.h" +#include "repository.h" +#include "parse-options.h" +#include "run-command.h" + +static int platform_specific_upgrade(void) +{ + return 1; +} + +static const char builtin_update_microsoft_git_usage[] = + N_("git update-microsoft-git"); + +int cmd_update_microsoft_git(int argc, const char **argv, const char *prefix) +{ + if (argc == 2 && !strcmp(argv[1], "-h")) + usage(builtin_update_microsoft_git_usage); + + return platform_specific_upgrade(); +} diff --git a/git.c b/git.c index b09389e7e77e91..9375533617dc6e 100644 --- a/git.c +++ b/git.c @@ -680,6 +680,7 @@ static struct cmd_struct commands[] = { { "unpack-file", cmd_unpack_file, RUN_SETUP | NO_PARSEOPT }, { "unpack-objects", cmd_unpack_objects, RUN_SETUP | NO_PARSEOPT }, { "update-index", cmd_update_index, RUN_SETUP }, + { "update-microsoft-git", cmd_update_microsoft_git }, { "update-ref", cmd_update_ref, RUN_SETUP }, { "update-server-info", cmd_update_server_info, RUN_SETUP }, { "upload-archive", cmd_upload_archive, NO_PARSEOPT }, From ca0e0bf80b0b3764acdb68264d3add5a1fdafc2b Mon Sep 17 00:00:00 2001 From: Derrick Stolee Date: Thu, 29 Apr 2021 11:02:07 -0400 Subject: [PATCH 2/3] update-microsoft-git: Windows implementation On Windows, we have the 'git update-git-for-windows' command. It is poorly named within the microsoft/git fork, because the script has been updated to look at the GitHub releases of microsoft/git, not git-for-windows/git. Still, it handles all the complicated details about downloading, verifying, and running the installer. Signed-off-by: Derrick Stolee --- builtin/update-microsoft-git.c | 20 ++++++++++++++++++++ 1 file changed, 20 insertions(+) diff --git a/builtin/update-microsoft-git.c b/builtin/update-microsoft-git.c index 329f6d77c0d1c9..8c4d2bd6a8b14d 100644 --- a/builtin/update-microsoft-git.c +++ b/builtin/update-microsoft-git.c @@ -2,11 +2,31 @@ #include "repository.h" #include "parse-options.h" #include "run-command.h" +#include "strvec.h" +#if defined(GIT_WINDOWS_NATIVE) +/* + * On Windows, run 'git update-git-for-windows' which + * is installed by the installer, based on the script + * in git-for-windows/build-extra. + */ static int platform_specific_upgrade(void) { + int res; + struct strvec args = STRVEC_INIT; + + strvec_push(&args, "git-update-git-for-windows"); + res = run_command_v_opt(args.v, 0); + strvec_clear(&args); + return res; +} +#else +static int platform_specific_upgrade(void) +{ + error(_("update-microsoft-git is not supported on this platform")); return 1; } +#endif static const char builtin_update_microsoft_git_usage[] = N_("git update-microsoft-git"); From 14c9bd1d0853a3e8b0238b8def01e4616f7be24f Mon Sep 17 00:00:00 2001 From: Derrick Stolee Date: Thu, 29 Apr 2021 11:18:46 -0400 Subject: [PATCH 3/3] update-microsoft-git: use brew on macOS The steps to update the microsoft-git cask are: 1. brew update 2. brew upgrade --cask microsoft-git This is adapted from the UpgradeVerb within microsoft/scalar. There is one important simplification: Scalar needed to check 'brew list --cask' to find out if the 'scalar' cask or the 'scalar-azrepos' cask was installed (which determined if the 'microsoft-git' cask was a necessary dependency). We do not need that here, since we are already in the microsoft-git cask. Signed-off-by: Derrick Stolee --- builtin/update-microsoft-git.c | 33 +++++++++++++++++++++++++++++++++ 1 file changed, 33 insertions(+) diff --git a/builtin/update-microsoft-git.c b/builtin/update-microsoft-git.c index 8c4d2bd6a8b14d..f943b808615fd7 100644 --- a/builtin/update-microsoft-git.c +++ b/builtin/update-microsoft-git.c @@ -20,6 +20,39 @@ static int platform_specific_upgrade(void) strvec_clear(&args); return res; } +#elif defined(__APPLE__) +/* + * On macOS, we expect the user to have the microsoft-git + * cask installed via Homebrew. We check using these + * commands: + * + * 1. 'brew update' to get latest versions. + * 2. 'brew upgrade --cask microsoft-git' to get the + * latest version. + */ +static int platform_specific_upgrade(void) +{ + int res; + struct strvec args = STRVEC_INIT; + + printf("Updating Homebrew with 'brew update'\n"); + + strvec_pushl(&args, "brew", "update", NULL); + res = run_command_v_opt(args.v, 0); + strvec_clear(&args); + + if (res) { + error(_("'brew update' failed; is brew installed?")); + return 1; + } + + printf("Upgrading microsoft-git with 'brew upgrade --cask microsoft-git'\n"); + strvec_pushl(&args, "brew", "upgrade", "--cask", "microsoft-git", NULL); + res = run_command_v_opt(args.v, 0); + strvec_clear(&args); + + return res; +} #else static int platform_specific_upgrade(void) {