Skip to content

Commit 05762d0

Browse files
dschoGit for Windows Build Agent
dscho
authored and
Git for Windows Build Agent
committed
Merge pull request #2506 from dscho/issue-2283
Allow running Git directly from `C:\Program Files\Git\mingw64\bin\git.exe`
2 parents 61c24b1 + 51e0607 commit 05762d0

File tree

3 files changed

+131
-4
lines changed

3 files changed

+131
-4
lines changed

‎compat/mingw.c‎

Lines changed: 97 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2858,6 +2858,66 @@ int xwcstoutf(char *utf, const wchar_t *wcs, size_t utflen)
28582858
return -1;
28592859
}
28602860

2861+
#ifdef ENSURE_MSYSTEM_IS_SET
2862+
#if !defined(RUNTIME_PREFIX) || !defined(HAVE_WPGMPTR)
2863+
static size_t append_system_bin_dirs(char *path UNUSED, size_t size UNUSED)
2864+
{
2865+
return 0;
2866+
}
2867+
#else
2868+
static size_t append_system_bin_dirs(char *path, size_t size)
2869+
{
2870+
char prefix[32768];
2871+
const char *slash;
2872+
size_t len = xwcstoutf(prefix, _wpgmptr, sizeof(prefix)), off = 0;
2873+
2874+
if (len == 0 || len >= sizeof(prefix) ||
2875+
!(slash = find_last_dir_sep(prefix)))
2876+
return 0;
2877+
/* strip trailing `git.exe` */
2878+
len = slash - prefix;
2879+
2880+
/* strip trailing `cmd` or `mingw64\bin` or `mingw32\bin` or `bin` or `libexec\git-core` */
2881+
if (strip_suffix_mem(prefix, &len, "\\mingw64\\libexec\\git-core") ||
2882+
strip_suffix_mem(prefix, &len, "\\mingw64\\bin"))
2883+
off += xsnprintf(path + off, size - off,
2884+
"%.*s\\mingw64\\bin;", (int)len, prefix);
2885+
else if (strip_suffix_mem(prefix, &len, "\\clangarm64\\libexec\\git-core") ||
2886+
strip_suffix_mem(prefix, &len, "\\clangarm64\\bin"))
2887+
off += xsnprintf(path + off, size - off,
2888+
"%.*s\\clangarm64\\bin;", (int)len, prefix);
2889+
else if (strip_suffix_mem(prefix, &len, "\\mingw32\\libexec\\git-core") ||
2890+
strip_suffix_mem(prefix, &len, "\\mingw32\\bin"))
2891+
off += xsnprintf(path + off, size - off,
2892+
"%.*s\\mingw32\\bin;", (int)len, prefix);
2893+
else if (strip_suffix_mem(prefix, &len, "\\cmd") ||
2894+
strip_suffix_mem(prefix, &len, "\\bin") ||
2895+
strip_suffix_mem(prefix, &len, "\\libexec\\git-core"))
2896+
off += xsnprintf(path + off, size - off,
2897+
"%.*s\\mingw%d\\bin;", (int)len, prefix,
2898+
(int)(sizeof(void *) * 8));
2899+
else
2900+
return 0;
2901+
2902+
off += xsnprintf(path + off, size - off,
2903+
"%.*s\\usr\\bin;", (int)len, prefix);
2904+
return off;
2905+
}
2906+
#endif
2907+
#endif
2908+
2909+
static int is_system32_path(const char *path)
2910+
{
2911+
WCHAR system32[MAX_PATH], wpath[MAX_PATH];
2912+
2913+
if (xutftowcs_path(wpath, path) < 0 ||
2914+
!GetSystemDirectoryW(system32, ARRAY_SIZE(system32)) ||
2915+
_wcsicmp(system32, wpath))
2916+
return 0;
2917+
2918+
return 1;
2919+
}
2920+
28612921
static void setup_windows_environment(void)
28622922
{
28632923
char *tmp = getenv("TMPDIR");
@@ -2898,7 +2958,8 @@ static void setup_windows_environment(void)
28982958
strbuf_addstr(&buf, tmp);
28992959
if ((tmp = getenv("HOMEPATH"))) {
29002960
strbuf_addstr(&buf, tmp);
2901-
if (is_directory(buf.buf))
2961+
if (!is_system32_path(buf.buf) &&
2962+
is_directory(buf.buf))
29022963
setenv("HOME", buf.buf, 1);
29032964
else
29042965
tmp = NULL; /* use $USERPROFILE */
@@ -2909,6 +2970,41 @@ static void setup_windows_environment(void)
29092970
if (!tmp && (tmp = getenv("USERPROFILE")))
29102971
setenv("HOME", tmp, 1);
29112972
}
2973+
2974+
if (!getenv("PLINK_PROTOCOL"))
2975+
setenv("PLINK_PROTOCOL", "ssh", 0);
2976+
2977+
#ifdef ENSURE_MSYSTEM_IS_SET
2978+
if (!(tmp = getenv("MSYSTEM")) || !tmp[0]) {
2979+
const char *home = getenv("HOME"), *path = getenv("PATH");
2980+
char buf[32768];
2981+
size_t off = 0;
2982+
2983+
#if defined(__aarch64__) || defined(_M_ARM64) || defined(_M_ARM64EC)
2984+
setenv("MSYSTEM", "CLANGARM64", 1);
2985+
#elif defined(__MINGW64__) || defined(_M_AMD64)
2986+
setenv("MSYSTEM", "MINGW64", 1);
2987+
#else
2988+
setenv("MSYSTEM", "MINGW32", 1);
2989+
#endif
2990+
2991+
if (home)
2992+
off += xsnprintf(buf + off, sizeof(buf) - off,
2993+
"%s\\bin;", home);
2994+
off += append_system_bin_dirs(buf + off, sizeof(buf) - off);
2995+
if (path)
2996+
off += xsnprintf(buf + off, sizeof(buf) - off,
2997+
"%s", path);
2998+
else if (off > 0)
2999+
buf[off - 1] = '\0';
3000+
else
3001+
buf[0] = '\0';
3002+
setenv("PATH", buf, 1);
3003+
}
3004+
#endif
3005+
3006+
if (!getenv("LC_ALL") && !getenv("LC_CTYPE") && !getenv("LANG"))
3007+
setenv("LC_CTYPE", "C.UTF-8", 1);
29123008
}
29133009

29143010
static void get_current_user_sid(PSID *sid, HANDLE *linked_token)

‎config.mak.uname‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -528,7 +528,7 @@ endif
528528
compat/win32/pthread.o compat/win32/syslog.o \
529529
compat/win32/trace2_win32_process_info.o \
530530
compat/win32/dirent.o
531-
COMPAT_CFLAGS = -D__USE_MINGW_ACCESS -DDETECT_MSYS_TTY -DNOGDI -DHAVE_STRING_H -Icompat -Icompat/regex -Icompat/win32 -DSTRIP_EXTENSION=\".exe\"
531+
COMPAT_CFLAGS = -D__USE_MINGW_ACCESS -DDETECT_MSYS_TTY -DENSURE_MSYSTEM_IS_SET -DNOGDI -DHAVE_STRING_H -Icompat -Icompat/regex -Icompat/win32 -DSTRIP_EXTENSION=\".exe\"
532532
BASIC_LDFLAGS = -IGNORE:4217 -IGNORE:4049 -NOLOGO -ENTRY:wmainCRTStartup -SUBSYSTEM:CONSOLE
533533
# invalidcontinue.obj allows Git's source code to close the same file
534534
# handle twice, or to access the osfhandle of an already-closed stdout
@@ -761,7 +761,7 @@ ifeq ($(uname_S),MINGW)
761761
endif
762762
CC = gcc
763763
COMPAT_CFLAGS += -D__USE_MINGW_ANSI_STDIO=0 -DDETECT_MSYS_TTY \
764-
-fstack-protector-strong
764+
-DENSURE_MSYSTEM_IS_SET -fstack-protector-strong
765765
EXTLIBS += -lntdll
766766
EXTRA_PROGRAMS += headless-git$X
767767
INSTALL = /bin/install

‎t/t0060-path-utils.sh‎

Lines changed: 32 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -610,7 +610,8 @@ test_expect_success !VALGRIND,RUNTIME_PREFIX,CAN_EXEC_IN_PWD 'RUNTIME_PREFIX wor
610610
echo "echo HERE" | write_script pretend/libexec/git-core/git-here &&
611611
GIT_EXEC_PATH= ./pretend/bin/git here >actual &&
612612
echo HERE >expect &&
613-
test_cmp expect actual'
613+
test_cmp expect actual
614+
'
614615

615616
test_expect_success !VALGRIND,RUNTIME_PREFIX,CAN_EXEC_IN_PWD '%(prefix)/ works' '
616617
git config yes.path "%(prefix)/yes" &&
@@ -619,4 +620,34 @@ test_expect_success !VALGRIND,RUNTIME_PREFIX,CAN_EXEC_IN_PWD '%(prefix)/ works'
619620
test_cmp expect actual
620621
'
621622

623+
test_expect_success MINGW,RUNTIME_PREFIX 'MSYSTEM/PATH is adjusted if necessary' '
624+
if test -z "$MINGW_PREFIX"
625+
then
626+
MINGW_PREFIX="/$(echo "${MSYSTEM:-MINGW64}" | tr A-Z a-z)"
627+
fi &&
628+
mkdir -p "$HOME"/bin pretend"$MINGW_PREFIX"/bin \
629+
pretend"$MINGW_PREFIX"/libexec/git-core pretend/usr/bin &&
630+
cp "$GIT_EXEC_PATH"/git.exe pretend"$MINGW_PREFIX"/bin/ &&
631+
cp "$GIT_EXEC_PATH"/git.exe pretend"$MINGW_PREFIX"/libexec/git-core/ &&
632+
# copy the .dll files, if any (happens when building via CMake)
633+
if test -n "$(ls "$GIT_EXEC_PATH"/*.dll 2>/dev/null)"
634+
then
635+
cp "$GIT_EXEC_PATH"/*.dll pretend"$MINGW_PREFIX"/bin/ &&
636+
cp "$GIT_EXEC_PATH"/*.dll pretend"$MINGW_PREFIX"/libexec/git-core/
637+
fi &&
638+
echo "env | grep MSYSTEM=" | write_script "$HOME"/bin/git-test-home &&
639+
echo "echo ${MINGW_PREFIX#/}" | write_script pretend"$MINGW_PREFIX"/bin/git-test-bin &&
640+
echo "echo usr" | write_script pretend/usr/bin/git-test-bin2 &&
641+
642+
(
643+
MSYSTEM= &&
644+
GIT_EXEC_PATH= &&
645+
pretend"$MINGW_PREFIX"/libexec/git-core/git.exe test-home >actual &&
646+
pretend"$MINGW_PREFIX"/libexec/git-core/git.exe test-bin >>actual &&
647+
pretend"$MINGW_PREFIX"/bin/git.exe test-bin2 >>actual
648+
) &&
649+
test_write_lines MSYSTEM=$MSYSTEM "${MINGW_PREFIX#/}" usr >expect &&
650+
test_cmp expect actual
651+
'
652+
622653
test_done

0 commit comments

Comments
 (0)