Skip to content

Commit 43d2a64

Browse files
dschoGit for Windows Build Agent
dscho
authored and
Git for Windows Build Agent
committed
Merge pull request #2974 from derrickstolee/maintenance-and-headless
Include Windows-specific maintenance and headless-git
2 parents 0bb57c5 + 7998a36 commit 43d2a64

File tree

10 files changed

+240
-13
lines changed

10 files changed

+240
-13
lines changed

‎Makefile

Lines changed: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2566,6 +2566,13 @@ compat/nedmalloc/nedmalloc.sp compat/nedmalloc/nedmalloc.o: EXTRA_CPPFLAGS = \
25662566
compat/nedmalloc/nedmalloc.sp: SP_EXTRA_FLAGS += -Wno-non-pointer-null
25672567
endif
25682568

2569+
headless-git.o: compat/win32/headless.c GIT-CFLAGS
2570+
$(QUIET_CC)$(CC) $(ALL_CFLAGS) $(COMPAT_CFLAGS) \
2571+
-fno-stack-protector -o $@ -c -Wall -Wwrite-strings $<
2572+
2573+
headless-git$X: headless-git.o git.res GIT-LDFLAGS
2574+
$(QUIET_LINK)$(CC) $(ALL_CFLAGS) $(ALL_LDFLAGS) -mwindows -o $@ $< git.res
2575+
25692576
git-%$X: %.o GIT-LDFLAGS $(GITLIBS)
25702577
$(QUIET_LINK)$(CC) $(ALL_CFLAGS) -o $@ $(ALL_LDFLAGS) $(filter %.o,$^) $(LIBS)
25712578

@@ -3215,6 +3222,7 @@ cocciclean:
32153222
clean: profile-clean coverage-clean cocciclean
32163223
$(RM) *.res
32173224
$(RM) $(OBJECTS)
3225+
$(RM) headless-git.o
32183226
$(RM) $(LIB_FILE) $(XDIFF_LIB)
32193227
$(RM) $(ALL_PROGRAMS) $(SCRIPT_LIB) $(BUILT_INS) git$X
32203228
$(RM) $(TEST_PROGRAMS)
@@ -3243,13 +3251,17 @@ endif
32433251
$(RM) GIT-SCRIPT-DEFINES GIT-PERL-DEFINES GIT-PERL-HEADER GIT-PYTHON-VARS
32443252
ifdef MSVC
32453253
$(RM) $(patsubst %.o,%.o.pdb,$(OBJECTS))
3254+
$(RM) headless-git.o.pdb
32463255
$(RM) $(patsubst %.exe,%.pdb,$(OTHER_PROGRAMS))
3256+
$(RM) $(patsubst %.exe,%.ilk,$(OTHER_PROGRAMS))
32473257
$(RM) $(patsubst %.exe,%.iobj,$(OTHER_PROGRAMS))
32483258
$(RM) $(patsubst %.exe,%.ipdb,$(OTHER_PROGRAMS))
32493259
$(RM) $(patsubst %.exe,%.pdb,$(PROGRAMS))
3260+
$(RM) $(patsubst %.exe,%.ilk,$(PROGRAMS))
32503261
$(RM) $(patsubst %.exe,%.iobj,$(PROGRAMS))
32513262
$(RM) $(patsubst %.exe,%.ipdb,$(PROGRAMS))
32523263
$(RM) $(patsubst %.exe,%.pdb,$(TEST_PROGRAMS))
3264+
$(RM) $(patsubst %.exe,%.ilk,$(TEST_PROGRAMS))
32533265
$(RM) $(patsubst %.exe,%.iobj,$(TEST_PROGRAMS))
32543266
$(RM) $(patsubst %.exe,%.ipdb,$(TEST_PROGRAMS))
32553267
$(RM) compat/vcbuild/MSVC-DEFS-GEN

‎builtin/gc.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1817,7 +1817,7 @@ static int schtasks_schedule_task(const char *exec_path, enum schedule_priority
18171817
"</Settings>\n"
18181818
"<Actions Context=\"Author\">\n"
18191819
"<Exec>\n"
1820-
"<Command>\"%s\\git.exe\"</Command>\n"
1820+
"<Command>\"%s\\headless-git.exe\"</Command>\n"
18211821
"<Arguments>--exec-path=\"%s\" for-each-repo --config=maintenance.repo maintenance run --schedule=%s</Arguments>\n"
18221822
"</Exec>\n"
18231823
"</Actions>\n"

‎compat/vcbuild/find_vs_env.bat

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -99,6 +99,7 @@ REM ================================================================
9999

100100
SET sdk_dir=%WindowsSdkDir%
101101
SET sdk_ver=%WindowsSDKVersion%
102+
SET sdk_ver_bin_dir=%WindowsSdkVerBinPath%%tgt%
102103
SET si=%sdk_dir%Include\%sdk_ver%
103104
SET sdk_includes=-I"%si%ucrt" -I"%si%um" -I"%si%shared"
104105
SET sl=%sdk_dir%lib\%sdk_ver%
@@ -130,6 +131,7 @@ REM ================================================================
130131

131132
SET sdk_dir=%WindowsSdkDir%
132133
SET sdk_ver=%WindowsSDKVersion%
134+
SET sdk_ver_bin_dir=%WindowsSdkVerBinPath%bin\amd64
133135
SET si=%sdk_dir%Include\%sdk_ver%
134136
SET sdk_includes=-I"%si%ucrt" -I"%si%um" -I"%si%shared" -I"%si%winrt"
135137
SET sl=%sdk_dir%lib\%sdk_ver%
@@ -160,6 +162,11 @@ REM ================================================================
160162
echo msvc_includes=%msvc_includes%
161163
echo msvc_libs=%msvc_libs%
162164

165+
echo sdk_ver_bin_dir=%sdk_ver_bin_dir%
166+
SET X1=%sdk_ver_bin_dir:C:=/C%
167+
SET X2=%X1:\=/%
168+
echo sdk_ver_bin_dir_msys=%X2%
169+
163170
echo sdk_includes=%sdk_includes%
164171
echo sdk_libs=%sdk_libs%
165172

‎compat/vcbuild/scripts/clink.pl

Lines changed: 17 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -15,6 +15,7 @@
1515
my @lflags = ();
1616
my $is_linking = 0;
1717
my $is_debug = 0;
18+
my $is_gui = 0;
1819
while (@ARGV) {
1920
my $arg = shift @ARGV;
2021
if ("$arg" eq "-DDEBUG") {
@@ -67,7 +68,11 @@
6768
}
6869
push(@args, $lib);
6970
} elsif ("$arg" eq "-lexpat") {
71+
if ($is_debug) {
72+
push(@args, "libexpatd.lib");
73+
} else {
7074
push(@args, "libexpat.lib");
75+
}
7176
} elsif ("$arg" =~ /^-L/ && "$arg" ne "-LTCG") {
7277
$arg =~ s/^-L/-LIBPATH:/;
7378
push(@lflags, $arg);
@@ -119,11 +124,23 @@
119124
push(@cflags, "-wd4996");
120125
} elsif ("$arg" =~ /^-W[a-z]/) {
121126
# let's ignore those
127+
} elsif ("$arg" eq "-fno-stack-protector") {
128+
# eat this
129+
} elsif ("$arg" eq "-mwindows") {
130+
$is_gui = 1;
122131
} else {
123132
push(@args, $arg);
124133
}
125134
}
126135
if ($is_linking) {
136+
if ($is_gui) {
137+
push(@args, "-ENTRY:wWinMainCRTStartup");
138+
push(@args, "-SUBSYSTEM:WINDOWS");
139+
} else {
140+
push(@args, "-ENTRY:wmainCRTStartup");
141+
push(@args, "-SUBSYSTEM:CONSOLE");
142+
}
143+
127144
push(@args, @lflags);
128145
unshift(@args, "link.exe");
129146
} else {

‎compat/vcbuild/scripts/rc.pl

Lines changed: 46 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,46 @@
1+
#!/usr/bin/perl -w
2+
######################################################################
3+
# Compile Resources on Windows
4+
#
5+
# This is a wrapper to facilitate the compilation of Git with MSVC
6+
# using GNU Make as the build system. So, instead of manipulating the
7+
# Makefile into something nasty, just to support non-space arguments
8+
# etc, we use this wrapper to fix the command line options
9+
#
10+
######################################################################
11+
use strict;
12+
my @args = ();
13+
my @input = ();
14+
15+
while (@ARGV) {
16+
my $arg = shift @ARGV;
17+
if ("$arg" =~ /^-[dD]/) {
18+
# GIT_VERSION gets passed with too many
19+
# layers of dquote escaping.
20+
$arg =~ s/\\"/"/g;
21+
22+
push(@args, $arg);
23+
24+
} elsif ("$arg" eq "-i") {
25+
my $arg = shift @ARGV;
26+
# TODO complain if NULL or is dashed ??
27+
push(@input, $arg);
28+
29+
} elsif ("$arg" eq "-o") {
30+
my $arg = shift @ARGV;
31+
# TODO complain if NULL or is dashed ??
32+
push(@args, "-fo$arg");
33+
34+
} else {
35+
push(@args, $arg);
36+
}
37+
}
38+
39+
push(@args, "-nologo");
40+
push(@args, "-v");
41+
push(@args, @input);
42+
43+
unshift(@args, "rc.exe");
44+
printf("**** @args\n");
45+
46+
exit (system(@args) != 0);

‎compat/win32/headless.c

Lines changed: 114 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,114 @@
1+
/*
2+
* headless Git - run Git without opening a console window on Windows
3+
*/
4+
5+
#define STRICT
6+
#define WIN32_LEAN_AND_MEAN
7+
#define UNICODE
8+
#define _UNICODE
9+
#include <windows.h>
10+
#include <stdio.h>
11+
#include <stdlib.h>
12+
#include <wchar.h>
13+
14+
/*
15+
* If `dir` contains the path to a Git exec directory, extend `PATH` to
16+
* include the corresponding `bin/` directory (which is where all those
17+
* `.dll` files needed by `git.exe` are, on Windows).
18+
*/
19+
static int extend_path(wchar_t *dir, size_t dir_len)
20+
{
21+
const wchar_t *suffix = L"\\libexec\\git-core";
22+
size_t suffix_len = wcslen(suffix);
23+
wchar_t *env;
24+
DWORD len;
25+
26+
if (dir_len < suffix_len)
27+
return 0;
28+
29+
dir_len -= suffix_len;
30+
if (memcmp(dir + dir_len, suffix, suffix_len * sizeof(wchar_t)))
31+
return 0;
32+
33+
len = GetEnvironmentVariableW(L"PATH", NULL, 0);
34+
if (!len)
35+
return 0;
36+
37+
env = _alloca((dir_len + 5 + len) * sizeof(wchar_t));
38+
wcsncpy(env, dir, dir_len);
39+
wcscpy(env + dir_len, L"\\bin;");
40+
if (!GetEnvironmentVariableW(L"PATH", env + dir_len + 5, len))
41+
return 0;
42+
43+
SetEnvironmentVariableW(L"PATH", env);
44+
return 1;
45+
}
46+
47+
int WINAPI wWinMain(HINSTANCE instance, HINSTANCE previous_instance,
48+
wchar_t *command_line, int show)
49+
{
50+
wchar_t git_command_line[32768];
51+
size_t size = sizeof(git_command_line) / sizeof(wchar_t);
52+
const wchar_t *needs_quotes = L"";
53+
int slash = 0, i;
54+
55+
STARTUPINFO startup_info = {
56+
.dwFlags = STARTF_USESHOWWINDOW,
57+
.wShowWindow = SW_HIDE,
58+
};
59+
PROCESS_INFORMATION process_info = { 0 };
60+
DWORD creation_flags = CREATE_UNICODE_ENVIRONMENT |
61+
CREATE_NEW_CONSOLE | CREATE_NO_WINDOW;
62+
DWORD exit_code;
63+
64+
/* First, determine the full path of argv[0] */
65+
for (i = 0; _wpgmptr[i]; i++)
66+
if (_wpgmptr[i] == L' ')
67+
needs_quotes = L"\"";
68+
else if (_wpgmptr[i] == L'\\')
69+
slash = i;
70+
71+
if (slash + 11 >= sizeof(git_command_line) / sizeof(wchar_t))
72+
return 127; /* Too long path */
73+
74+
/* If it is in Git's exec path, add the bin/ directory to the PATH */
75+
extend_path(_wpgmptr, slash);
76+
77+
/* Then, add the full path of `git.exe` as argv[0] */
78+
i = swprintf_s(git_command_line, size, L"%ls%.*ls\\git.exe%ls",
79+
needs_quotes, slash, _wpgmptr, needs_quotes);
80+
if (i < 0)
81+
return 127; /* Too long path */
82+
83+
if (*command_line) {
84+
/* Now, append the command-line arguments */
85+
i = swprintf_s(git_command_line + i, size - i,
86+
L" %ls", command_line);
87+
if (i < 0)
88+
return 127;
89+
}
90+
91+
startup_info.cb = sizeof(STARTUPINFO);
92+
93+
startup_info.hStdInput = GetStdHandle(STD_INPUT_HANDLE);
94+
startup_info.hStdOutput = GetStdHandle(STD_OUTPUT_HANDLE);
95+
startup_info.hStdError = GetStdHandle(STD_ERROR_HANDLE);
96+
97+
if (!CreateProcess(NULL, /* infer argv[0] from the command line */
98+
git_command_line, /* modified command line */
99+
NULL, /* inherit process handles? */
100+
NULL, /* inherit thread handles? */
101+
FALSE, /* handles inheritable? */
102+
creation_flags,
103+
NULL, /* use this process' environment */
104+
NULL, /* use this process' working directory */
105+
&startup_info, &process_info))
106+
return 129; /* could not start */
107+
WaitForSingleObject(process_info.hProcess, INFINITE);
108+
if (!GetExitCodeProcess(process_info.hProcess, &exit_code)) {
109+
CloseHandle(process_info.hProcess);
110+
return 130; /* Could not determine exit code? */
111+
}
112+
113+
return (int)exit_code;
114+
}

‎config.mak.uname

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -399,7 +399,7 @@ ifeq ($(uname_S),Windows)
399399
# link.exe next to, and required by, cl.exe, we have to prepend this
400400
# onto the existing $PATH.
401401
#
402-
SANE_TOOL_PATH ?= $(msvc_bin_dir_msys)
402+
SANE_TOOL_PATH ?= $(msvc_bin_dir_msys):$(sdk_ver_bin_dir_msys)
403403
HAVE_ALLOCA_H = YesPlease
404404
NO_PREAD = YesPlease
405405
NEEDS_CRYPTO_WITH_SSL = YesPlease
@@ -453,12 +453,14 @@ endif
453453
compat/win32/trace2_win32_process_info.o \
454454
compat/win32/dirent.o
455455
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\"
456-
BASIC_LDFLAGS = -IGNORE:4217 -IGNORE:4049 -NOLOGO -ENTRY:wmainCRTStartup -SUBSYSTEM:CONSOLE
456+
BASIC_LDFLAGS = -IGNORE:4217 -IGNORE:4049 -NOLOGO
457457
# invalidcontinue.obj allows Git's source code to close the same file
458458
# handle twice, or to access the osfhandle of an already-closed stdout
459459
# See https://msdn.microsoft.com/en-us/library/ms235330.aspx
460460
EXTLIBS = user32.lib advapi32.lib shell32.lib wininet.lib ws2_32.lib invalidcontinue.obj kernel32.lib ntdll.lib
461+
GITLIBS += git.res
461462
PTHREAD_LIBS =
463+
RC = compat/vcbuild/scripts/rc.pl
462464
lib =
463465
BASIC_CFLAGS += $(vcpkg_inc) $(sdk_includes) $(msvc_includes)
464466
ifndef DEBUG
@@ -490,6 +492,8 @@ else
490492
endif
491493
X = .exe
492494

495+
EXTRA_PROGRAMS += headless-git$X
496+
493497
compat/msvc.o: compat/msvc.c compat/mingw.c GIT-CFLAGS
494498
endif
495499
ifeq ($(uname_S),Interix)
@@ -632,6 +636,7 @@ ifneq (,$(findstring MINGW,$(uname_S)))
632636
RC = windres -O coff
633637
NATIVE_CRLF = YesPlease
634638
X = .exe
639+
EXTRA_PROGRAMS += headless-git$X
635640
ifneq (,$(wildcard ../THIS_IS_MSYSGIT))
636641
htmldir = doc/git/html/
637642
prefix =

‎contrib/buildsystems/CMakeLists.txt

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -681,6 +681,9 @@ if(WIN32)
681681
else()
682682
message(FATAL_ERROR "Unhandled compiler: ${CMAKE_C_COMPILER_ID}")
683683
endif()
684+
685+
add_executable(headless-git ${CMAKE_SOURCE_DIR}/compat/win32/headless.c)
686+
target_link_options(headless-git PUBLIC /NOLOGO /ENTRY:wWinMainCRTStartup /SUBSYSTEM:WINDOWS)
684687
elseif(UNIX)
685688
target_link_libraries(common-main pthread rt)
686689
endif()

0 commit comments

Comments
 (0)