Skip to content

Commit b698e0b

Browse files
derrickstoleedscho
authored andcommitted
Merge pull request #329: Add git update-microsoft-git
This adds a new builtin, `git update-microsoft-git`, that executes the platform-specific upgrade steps to get the latest version of `microsoft-git`. On Windows, this means running `git update-git-for-windows` which was updated to use the `microsoft/git` releases page, when appropriate. See #321 for details. On macOS, this means running a sequence of `brew` commands. These are adapted from the `UpgradeVerb` in `microsoft/scalar`, with an important simplification: we don't need to differentiate between the `scalar` and `scalar-azrepos` cask.
2 parents 71b77b6 + 02d926c commit b698e0b

File tree

8 files changed

+99
-0
lines changed

8 files changed

+99
-0
lines changed

.gitignore

+1
Original file line numberDiff line numberDiff line change
@@ -174,6 +174,7 @@
174174
/git-unpack-file
175175
/git-unpack-objects
176176
/git-update-index
177+
/git-update-microsoft-git
177178
/git-update-ref
178179
/git-update-server-info
179180
/git-upload-archive
+24
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,24 @@
1+
git-update-microsoft-git(1)
2+
===========================
3+
4+
NAME
5+
----
6+
git-update-microsoft-git - Update the installed version of Git
7+
8+
9+
SYNOPSIS
10+
--------
11+
[verse]
12+
'git update-microsoft-git'
13+
14+
DESCRIPTION
15+
-----------
16+
This version of Git is based on the Microsoft fork of Git, which
17+
has custom capabilities focused on supporting monorepos. This
18+
command checks for the latest release of that fork and installs
19+
it on your machine.
20+
21+
22+
GIT
23+
---
24+
Part of the linkgit:git[1] suite

Documentation/lint-manpages.sh

+1
Original file line numberDiff line numberDiff line change
@@ -28,6 +28,7 @@ check_missing_docs () (
2828
git-remote-*) continue;;
2929
git-stage) continue;;
3030
git-gvfs-helper) continue;;
31+
git-update-microsoft-git) continue;;
3132
git-legacy-*) continue;;
3233
git-?*--?* ) continue ;;
3334
esac

Makefile

+1
Original file line numberDiff line numberDiff line change
@@ -1326,6 +1326,7 @@ BUILTIN_OBJS += builtin/tag.o
13261326
BUILTIN_OBJS += builtin/unpack-file.o
13271327
BUILTIN_OBJS += builtin/unpack-objects.o
13281328
BUILTIN_OBJS += builtin/update-index.o
1329+
BUILTIN_OBJS += builtin/update-microsoft-git.o
13291330
BUILTIN_OBJS += builtin/update-ref.o
13301331
BUILTIN_OBJS += builtin/update-server-info.o
13311332
BUILTIN_OBJS += builtin/upload-archive.o

builtin.h

+1
Original file line numberDiff line numberDiff line change
@@ -239,6 +239,7 @@ int cmd_tag(int argc, const char **argv, const char *prefix, struct repository *
239239
int cmd_unpack_file(int argc, const char **argv, const char *prefix, struct repository *repo);
240240
int cmd_unpack_objects(int argc, const char **argv, const char *prefix, struct repository *repo);
241241
int cmd_update_index(int argc, const char **argv, const char *prefix, struct repository *repo);
242+
int cmd_update_microsoft_git(int argc, const char **argv, const char *prefix, struct repository *repo);
242243
int cmd_update_ref(int argc, const char **argv, const char *prefix, struct repository *repo);
243244
int cmd_update_server_info(int argc, const char **argv, const char *prefix, struct repository *repo);
244245
int cmd_upload_archive(int argc, const char **argv, const char *prefix, struct repository *repo);

builtin/update-microsoft-git.c

+69
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,69 @@
1+
#include "builtin.h"
2+
#include "repository.h"
3+
#include "parse-options.h"
4+
#include "run-command.h"
5+
#include "strvec.h"
6+
7+
#if defined(GIT_WINDOWS_NATIVE)
8+
/*
9+
* On Windows, run 'git update-git-for-windows' which
10+
* is installed by the installer, based on the script
11+
* in git-for-windows/build-extra.
12+
*/
13+
static int platform_specific_upgrade(void)
14+
{
15+
struct child_process cp = CHILD_PROCESS_INIT;
16+
17+
strvec_push(&cp.args, "git-update-git-for-windows");
18+
return run_command(&cp);
19+
}
20+
#elif defined(__APPLE__)
21+
/*
22+
* On macOS, we expect the user to have the microsoft-git
23+
* cask installed via Homebrew. We check using these
24+
* commands:
25+
*
26+
* 1. 'brew update' to get latest versions.
27+
* 2. 'brew upgrade --cask microsoft-git' to get the
28+
* latest version.
29+
*/
30+
static int platform_specific_upgrade(void)
31+
{
32+
int res;
33+
struct child_process update = CHILD_PROCESS_INIT;
34+
struct child_process upgrade = CHILD_PROCESS_INIT;
35+
36+
printf("Updating Homebrew with 'brew update'\n");
37+
38+
strvec_pushl(&update.args, "brew", "update", NULL);
39+
res = run_command(&update);
40+
41+
if (res) {
42+
error(_("'brew update' failed; is brew installed?"));
43+
return 1;
44+
}
45+
46+
printf("Upgrading microsoft-git with 'brew upgrade --cask microsoft-git'\n");
47+
strvec_pushl(&upgrade.args, "brew", "upgrade", "--cask", "microsoft-git", NULL);
48+
res = run_command(&upgrade);
49+
50+
return res;
51+
}
52+
#else
53+
static int platform_specific_upgrade(void)
54+
{
55+
error(_("update-microsoft-git is not supported on this platform"));
56+
return 1;
57+
}
58+
#endif
59+
60+
static const char builtin_update_microsoft_git_usage[] =
61+
N_("git update-microsoft-git");
62+
63+
int cmd_update_microsoft_git(int argc, const char **argv, const char *prefix UNUSED, struct repository *repo UNUSED)
64+
{
65+
if (argc == 2 && !strcmp(argv[1], "-h"))
66+
usage(builtin_update_microsoft_git_usage);
67+
68+
return platform_specific_upgrade();
69+
}

git.c

+1
Original file line numberDiff line numberDiff line change
@@ -708,6 +708,7 @@ static struct cmd_struct commands[] = {
708708
{ "unpack-file", cmd_unpack_file, RUN_SETUP | NO_PARSEOPT },
709709
{ "unpack-objects", cmd_unpack_objects, RUN_SETUP | NO_PARSEOPT },
710710
{ "update-index", cmd_update_index, RUN_SETUP },
711+
{ "update-microsoft-git", cmd_update_microsoft_git },
711712
{ "update-ref", cmd_update_ref, RUN_SETUP },
712713
{ "update-server-info", cmd_update_server_info, RUN_SETUP },
713714
{ "upload-archive", cmd_upload_archive, NO_PARSEOPT },

meson.build

+1
Original file line numberDiff line numberDiff line change
@@ -603,6 +603,7 @@ builtin_sources = [
603603
'builtin/unpack-file.c',
604604
'builtin/unpack-objects.c',
605605
'builtin/update-index.c',
606+
'builtin/update-microsoft-git.c',
606607
'builtin/update-ref.c',
607608
'builtin/update-server-info.c',
608609
'builtin/upload-archive.c',

0 commit comments

Comments
 (0)