Re: [PATCH v3 12/14] mingw: import poll-emulation from gnulib

[Erik Faye-Lund <kusmabite@gmail.com>]

On Sun, Oct 10, 2010 at 4:28 PM, Erik Faye-Lund <kusmabite@gmail.com> wrote:
> On Sun, Oct 10, 2010 at 4:15 PM, Ævar Arnfjörð Bjarmason
> <avarab@gmail.com> wrote:
>> On Sun, Oct 10, 2010 at 13:20, Erik Faye-Lund <kusmabite@gmail.com> wrote:
>>> lib/poll.c and lib/poll.in.h imported from 0a05120 in
>>> git://git.savannah.gnu.org/gnulib.git
>>
>> Having fought with importing things from gnulib myself using their
>> tools it would be useful to note in the commit message *how* you
>> imported this. Did you use the gnulib command with some archane
>> options so it wouldn't touch the build system while it was at it, or
>> did you just copy the relevant files manually?
>>
>
> Sorry if that was unclear - I just copied the files (verbatim).
> Patching to make it compile for us comes in the next patch.
>
> I didn't even know that there was a gnulib tool to extract code, but a
> quick google-search shows that there is. I'll look into using the tool
> instead for the next round.
>

I've had a quick look at it, and it really doesn't seem like
gnulib-tool is suited for us here. It seems to be intended on pure
autoconf-projects, and starts including all kinds of things that we
don't need. We only care about poll-emulation on Windows, and we don't
need autoconf to tell us if it should be used or not.

So I'm not in favor of using gnulib-tool, and going with the current
method of verbatim copy with a separate fix-up commit. But perhaps I
should clarify the commit message so other people can easily upgrade
the emulation later...