git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: [PATCH 1/5] msvc: Fix compilation errors caused by poll.h emulation

From
SSSven Strickroth <sven@cs-ware.de>
Date
May 9, 2013, 21:12 UTC
Message-ID
<518C113C.3080106@cs-ware.de>
In-Reply-To
<518C0890.9050508@ramsay1.demon.co.uk>
Am 09.05.2013 22:35 schrieb Ramsay Jones:
Show 15 quoted lines
> Note that '-Icompat/poll' is passed on the command-line (it is split at -Icomp
> above), which comes from:
> 
>     ramsay (tmp) ms $ git grep -n 'compat/poll'
>     Makefile:647:LIB_H += compat/poll/poll.h
>     Makefile:1235:  COMPAT_CFLAGS += -DNO_POLL -Icompat/poll
>     Makefile:1236:  COMPAT_OBJS += compat/poll/poll.o
>     ramsay (tmp) ms $
> 
> I have a patch which allows me to generate the preprocessed output file
> and, after running it on git.c (=> git.i), I can see that msvc reads the
> compat/poll/poll.h file. (from line 128 of git-compat-util.h).
> 
> Are you using a MSVC "project" file? If so, is it equivalent to the Makefile?
> Has the "#include" behaviour changed between MSVC 2008 -> MSVC 2012?

Thank you for your explanation. The problem for me was, that _WIN32_WINNT was set to 0x0600 and thus winsck2.h was also declaring pollfd.

Building libgit works now.
-- 
Best regards,
 Sven Strickroth
 PGP key id F5A9D4C4 @ any key-server
Previous: Ramsay Jones
Message 4 of 4 in “msvc: Fix compilation errors caused by poll.h emulation”
  1. 1/5 msvc: Fix compilation errors caused by poll.h emulationRamsay Jones, Jan 31, 2013
  2. Sven StrickrothMay 9, 2013
  3. Ramsay JonesMay 9, 2013
  4. Sven StrickrothMay 9, 2013

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.