mingw: rename WIN32 cpp macro to GIT_WINDOWS_NATIVE
Throughout git, it is assumed that the WIN32 preprocessor symbol is defined on native Windows setups (mingw and msvc) and not on Cygwin. On Cygwin, most of the time git can pretend this is just another Unix machine, and Windows-specific magic is generally counterproductive. Unfortunately Cygwin *does* define the WIN32 symbol in some headers. Best to rely on a new git-specific symbol GIT_WINDOWS_NATIVE instead, defined as follows: #if defined(WIN32) && !defined(__CYGWIN__) # define GIT_WINDOWS_NATIVE #endif After this change, it should be possible to drop the CYGWIN_V15_WIN32API setting without any negative effect. [rj: %s/WINDOWS_NATIVE/GIT_WINDOWS_NATIVE/g ] Signed-off-by: Jonathan Nieder <jrnieder@gmail.com> Signed-off-by: Ramsay Jones <ramsay@ramsay1.demon.co.uk> Signed-off-by: Junio C Hamano <gitster@pobox.com>
This commit is contained in:

committed by
Junio C Hamano

parent
b75cdfaa88
commit
380395d094
@ -72,7 +72,7 @@ static inline void close_pair(int fd[2])
|
||||
close(fd[1]);
|
||||
}
|
||||
|
||||
#ifndef WIN32
|
||||
#ifndef GIT_WINDOWS_NATIVE
|
||||
static inline void dup_devnull(int to)
|
||||
{
|
||||
int fd = open("/dev/null", O_RDWR);
|
||||
@ -159,7 +159,7 @@ static const char **prepare_shell_cmd(const char **argv)
|
||||
die("BUG: shell command is empty");
|
||||
|
||||
if (strcspn(argv[0], "|&;<>()$`\\\"' \t\n*?[#~=%") != strlen(argv[0])) {
|
||||
#ifndef WIN32
|
||||
#ifndef GIT_WINDOWS_NATIVE
|
||||
nargv[nargc++] = SHELL_PATH;
|
||||
#else
|
||||
nargv[nargc++] = "sh";
|
||||
@ -182,7 +182,7 @@ static const char **prepare_shell_cmd(const char **argv)
|
||||
return nargv;
|
||||
}
|
||||
|
||||
#ifndef WIN32
|
||||
#ifndef GIT_WINDOWS_NATIVE
|
||||
static int execv_shell_cmd(const char **argv)
|
||||
{
|
||||
const char **nargv = prepare_shell_cmd(argv);
|
||||
@ -193,7 +193,7 @@ static int execv_shell_cmd(const char **argv)
|
||||
}
|
||||
#endif
|
||||
|
||||
#ifndef WIN32
|
||||
#ifndef GIT_WINDOWS_NATIVE
|
||||
static int child_err = 2;
|
||||
static int child_notifier = -1;
|
||||
|
||||
@ -334,7 +334,7 @@ fail_pipe:
|
||||
trace_argv_printf(cmd->argv, "trace: run_command:");
|
||||
fflush(NULL);
|
||||
|
||||
#ifndef WIN32
|
||||
#ifndef GIT_WINDOWS_NATIVE
|
||||
{
|
||||
int notify_pipe[2];
|
||||
if (pipe(notify_pipe))
|
||||
|
Reference in New Issue
Block a user