From: Junio C Hamano <gitster@pobox.com>
Date: Tue, 18 Sep 2012 06:54:40 GMT
Subject: Re: [PATCH v7 1/4] make poll available for other platforms lacking it
Message-ID: <7vehlzq0r3.fsf@alter.siamese.dyndns.org>
In-Reply-To: <000101cd9566$df8b8d00$9ea2a700$@schmitz-digital.de>

"Joachim Schmitz" <jojo@schmitz-digital.de> writes:

>> > @@ -1605,6 +1610,11 @@ ifdef NO_GETTEXT
>> >  	BASIC_CFLAGS += -DNO_GETTEXT
>> >  	USE_GETTEXT_SCHEME ?= fallthrough
>> >  endif
>> > +ifdef NO_POLL
>> > +	NO_SYS_POLL_H = YesPlease
>> > +	COMPAT_CFLAGS += -DNO_POLL -Icompat/poll
>> > +	COMPAT_OBJS += compat/poll/poll.o
>> > +endif
>> 
>> I think my guesses above are correct, so will queue with the trivial
>> and obvious fixup.
>
> Well I was unde the impression that is <sys/poll.h> isn't
> available, there might be a <poll.h> and it that isn't available
> either we use compat/.../poll.[ch]. But rethinking your changed
> does makes perfect sense.

Heh, my comment was not about names between NO_POLL_H vs NO_POLL.
With the way you wrote the code, the symbols defined for Windows and
MINGW must match what controls the hunk around ll.1610, so what
makes perfect sense to you is indeed your code ;-)

> OK, so I guess I'm ready for the 'final touch', my NonStop
> specific changes, will post them shortly...