Pull requests

#2 Merged
Repository
hatstand hatstand
Branch
default
Repository
geekman geekman
Branch
default

Win32/MinGW support.

Author
  1. John Maguire
Reviewers
Description

Compiles with mingw-w64 Changed pipe to local socket on win32 to support select() on the notify pipe.

  • Learn about pull requests

Comments (5)

  1. geekman repo owner

    Hi John, thanks for the patch. This might take a while as I'm quite particular about what goes into the tree.

    I did have a version that compiles on Win32, but it's not in the tree because I haven't really figured out how I should handle the pthread dependency and the pipe function. If I remember correctly, the descriptor returned by _pipe doesn't work with select(). And from the changes you've made to testmdnsd.c, it now looks like I need to worry about the IPv6 code as well.

  2. John Maguire author

    This is really step 1 for windows support, ie. making it at least compile. I haven't actually tested it, but I suspect you're right about _pipe().

    It compiled & linked fine for me against the win32 pthread implementation (http://sourceware.org/pthreads-win32/).

    The IPv6 bit isn't a worry, it's just that net_pton() does not exist in mingw so I changed it to getaddrinfo() instead.