Jim Meyering <jim@meyering.net> wrote:
"Daniel P. Berrange" <berrange@redhat.com> wrote:
On Tue, Jan 06, 2009 at 04:38:12PM +0100, Jim Meyering wrote:
"Daniel P. Berrange" <berrange@redhat.com> wrote:
There are a number of problems breaking the windows / mingw build currently.
- Use of 'close' without importing unistd.h - Use of non-existant localtime_r - ERROR macro from logging.h clashes with a symbol imported from windows.h
So this patch does
- Adds the missing unistd.h include - Uses localtime() if localtime_r() is missing (as checked from configure)
That all looks fine. However, since there's an LGPLv2+ gnulib module that provides localtime_r (called time_r) you may want to skip that part. Or just go ahead with what you have and I'll undo things when I've confirmed pulling in the new module works as expected.
I committed everything except the localtime_r() change...
Thanks. Here's the change to pull in the time_r module and to update gnulib to the latest.
I tested by applying the patch to a clean directory and ensuring that autobuild.sh's mingw build succeeds. It also passed "make distcheck" on a rawhide x86_64 system.
Daniel Berrange ACK'd this privately, so I've just committed it.