Re: [PATCH 1/2] MinGW: fix compile error due to missing ELOOP

2014-07-16 Thread Junio C Hamano
Karsten Blees writes: > MinGW and MSVC before 2010 don't define ELOOP, use EMLINK (aka "Too many > links") instead. > > Signed-off-by: Karsten Blees > --- Thanks; will apply directly to 'master'. > compat/mingw.h | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/compat/mingw.h b/com

Re: [PATCH 1/2] MinGW: fix compile error due to missing ELOOP

2014-07-15 Thread Karsten Blees
Am 16.07.2014 01:42, schrieb Jonathan Nieder: > Karsten Blees wrote: > >> MinGW and MSVC before 2010 don't define ELOOP, use EMLINK (aka "Too many >> links") instead. > [...] >> +#ifndef ELOOP >> +#define ELOOP EMLINK >> +#endif > > This could use > > #define ELOOP WSAELOOP > > as an alte

Re: [PATCH 1/2] MinGW: fix compile error due to missing ELOOP

2014-07-15 Thread Jonathan Nieder
Karsten Blees wrote: > MinGW and MSVC before 2010 don't define ELOOP, use EMLINK (aka "Too many > links") instead. [...] > +#ifndef ELOOP > +#define ELOOP EMLINK > +#endif This could use #define ELOOP WSAELOOP as an alternative. But it shouldn't matter since git doesn't look for EMLINK

[PATCH 1/2] MinGW: fix compile error due to missing ELOOP

2014-07-15 Thread Karsten Blees
MinGW and MSVC before 2010 don't define ELOOP, use EMLINK (aka "Too many links") instead. Signed-off-by: Karsten Blees --- compat/mingw.h | 3 +++ 1 file changed, 3 insertions(+) diff --git a/compat/mingw.h b/compat/mingw.h index 405c08f..510530c 100644 --- a/compat/mingw.h +++ b/compat/mingw.h