Skip to content

Commit e64114c

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 c21fc1c + 495a92f commit e64114c

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
@@ -2645,6 +2645,13 @@ compat/nedmalloc/nedmalloc.sp compat/nedmalloc/nedmalloc.o: EXTRA_CPPFLAGS = \
26452645
compat/nedmalloc/nedmalloc.sp: SP_EXTRA_FLAGS += -Wno-non-pointer-null
26462646
endif
26472647

2648+
headless-git.o: compat/win32/headless.c GIT-CFLAGS
2649+
$(QUIET_CC)$(CC) $(ALL_CFLAGS) $(COMPAT_CFLAGS) \
2650+
-fno-stack-protector -o $@ -c -Wall -Wwrite-strings $<
2651+
2652+
headless-git$X: headless-git.o git.res GIT-LDFLAGS
2653+
$(QUIET_LINK)$(CC) $(ALL_CFLAGS) $(ALL_LDFLAGS) -mwindows -o $@ $< git.res
2654+
26482655
git-%$X: %.o GIT-LDFLAGS $(GITLIBS)
26492656
$(QUIET_LINK)$(CC) $(ALL_CFLAGS) -o $@ $(ALL_LDFLAGS) $(filter %.o,$^) $(LIBS)
26502657

@@ -3313,6 +3320,7 @@ cocciclean:
33133320
clean: profile-clean coverage-clean cocciclean
33143321
$(RM) *.res
33153322
$(RM) $(OBJECTS)
3323+
$(RM) headless-git.o
33163324
$(RM) $(LIB_FILE) $(XDIFF_LIB) $(REFTABLE_LIB) $(REFTABLE_TEST_LIB)
33173325
$(RM) $(ALL_PROGRAMS) $(SCRIPT_LIB) $(BUILT_INS) git$X
33183326
$(RM) $(TEST_PROGRAMS)
@@ -3341,13 +3349,17 @@ endif
33413349
$(RM) GIT-SCRIPT-DEFINES GIT-PERL-DEFINES GIT-PERL-HEADER GIT-PYTHON-VARS
33423350
ifdef MSVC
33433351
$(RM) $(patsubst %.o,%.o.pdb,$(OBJECTS))
3352+
$(RM) headless-git.o.pdb
33443353
$(RM) $(patsubst %.exe,%.pdb,$(OTHER_PROGRAMS))
3354+
$(RM) $(patsubst %.exe,%.ilk,$(OTHER_PROGRAMS))
33453355
$(RM) $(patsubst %.exe,%.iobj,$(OTHER_PROGRAMS))
33463356
$(RM) $(patsubst %.exe,%.ipdb,$(OTHER_PROGRAMS))
33473357
$(RM) $(patsubst %.exe,%.pdb,$(PROGRAMS))
3358+
$(RM) $(patsubst %.exe,%.ilk,$(PROGRAMS))
33483359
$(RM) $(patsubst %.exe,%.iobj,$(PROGRAMS))
33493360
$(RM) $(patsubst %.exe,%.ipdb,$(PROGRAMS))
33503361
$(RM) $(patsubst %.exe,%.pdb,$(TEST_PROGRAMS))
3362+
$(RM) $(patsubst %.exe,%.ilk,$(TEST_PROGRAMS))
33513363
$(RM) $(patsubst %.exe,%.iobj,$(TEST_PROGRAMS))
33523364
$(RM) $(patsubst %.exe,%.ipdb,$(TEST_PROGRAMS))
33533365
$(RM) compat/vcbuild/MSVC-DEFS-GEN

‎builtin/gc.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1927,7 +1927,7 @@ static int schtasks_schedule_task(const char *exec_path, enum schedule_priority
19271927
"</Settings>\n"
19281928
"<Actions Context=\"Author\">\n"
19291929
"<Exec>\n"
1930-
"<Command>\"%s\\git.exe\"</Command>\n"
1930+
"<Command>\"%s\\headless-git.exe\"</Command>\n"
19311931
"<Arguments>--exec-path=\"%s\" for-each-repo --config=maintenance.repo maintenance run --schedule=%s</Arguments>\n"
19321932
"</Exec>\n"
19331933
"</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
@@ -404,7 +404,7 @@ ifeq ($(uname_S),Windows)
404404
# link.exe next to, and required by, cl.exe, we have to prepend this
405405
# onto the existing $PATH.
406406
#
407-
SANE_TOOL_PATH ?= $(msvc_bin_dir_msys)
407+
SANE_TOOL_PATH ?= $(msvc_bin_dir_msys):$(sdk_ver_bin_dir_msys)
408408
HAVE_ALLOCA_H = YesPlease
409409
NO_PREAD = YesPlease
410410
NEEDS_CRYPTO_WITH_SSL = YesPlease
@@ -459,12 +459,14 @@ endif
459459
compat/win32/trace2_win32_process_info.o \
460460
compat/win32/dirent.o
461461
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\"
462-
BASIC_LDFLAGS = -IGNORE:4217 -IGNORE:4049 -NOLOGO -ENTRY:wmainCRTStartup -SUBSYSTEM:CONSOLE
462+
BASIC_LDFLAGS = -IGNORE:4217 -IGNORE:4049 -NOLOGO
463463
# invalidcontinue.obj allows Git's source code to close the same file
464464
# handle twice, or to access the osfhandle of an already-closed stdout
465465
# See https://msdn.microsoft.com/en-us/library/ms235330.aspx
466466
EXTLIBS = user32.lib advapi32.lib shell32.lib wininet.lib ws2_32.lib invalidcontinue.obj kernel32.lib ntdll.lib
467+
GITLIBS += git.res
467468
PTHREAD_LIBS =
469+
RC = compat/vcbuild/scripts/rc.pl
468470
lib =
469471
BASIC_CFLAGS += $(vcpkg_inc) $(sdk_includes) $(msvc_includes)
470472
ifndef DEBUG
@@ -496,6 +498,8 @@ else
496498
endif
497499
X = .exe
498500

501+
EXTRA_PROGRAMS += headless-git$X
502+
499503
compat/msvc.o: compat/msvc.c compat/mingw.c GIT-CFLAGS
500504
endif
501505
ifeq ($(uname_S),Interix)
@@ -640,6 +644,7 @@ ifneq (,$(findstring MINGW,$(uname_S)))
640644
RC = windres -O coff
641645
NATIVE_CRLF = YesPlease
642646
X = .exe
647+
EXTRA_PROGRAMS += headless-git$X
643648
ifneq (,$(wildcard ../THIS_IS_MSYSGIT))
644649
htmldir = doc/git/html/
645650
prefix =

‎contrib/buildsystems/CMakeLists.txt

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -702,6 +702,9 @@ if(WIN32)
702702
else()
703703
message(FATAL_ERROR "Unhandled compiler: ${CMAKE_C_COMPILER_ID}")
704704
endif()
705+
706+
add_executable(headless-git ${CMAKE_SOURCE_DIR}/compat/win32/headless.c)
707+
target_link_options(headless-git PUBLIC /NOLOGO /ENTRY:wWinMainCRTStartup /SUBSYSTEM:WINDOWS)
705708
elseif(UNIX)
706709
target_link_libraries(common-main pthread rt)
707710
endif()

0 commit comments

Comments
 (0)