Re: [PATCH v3] Merge parts of <sys/time.h> from FreeBSD

[Corinna Vinschen <vinschen at redhat dot com>]

Hi Steve,

On Mar 19 17:53, Corinna Vinschen wrote:
> On Mar 19 08:32, Steve Ellcey wrote:
> > On Thu, 2015-03-19 at 09:18 +0100, Corinna Vinschen wrote:
> > 
> > > Neither on Cygwin, but in both cases it's probably a difference in the
> > > headers pulled in indirectly.
> > > 
> > > > >Does it help to s/uint64_t/__uint64_t/ in sys/time.h?
> > > > 
> > > > Since we define uint64_t in terms of __uint64_t in <stdint.h> this should be
> > > > fine.
> > > 
> > > I think so, too, but I'd like to hear if it actually fixes Steve's
> > > build problem.
> > > 
> > > 
> > > Thanks,
> > > Corinna
> > 
> > Yes, this fixed the problem.  I also changed uses of uint32_t to
> > __uint32_t.  Here is the complete patch that I tested and that built for
> > me with my mips-mti-elf toolchain.
> 
> Patch is fine, please push.

Thanks, but you forgot to push ChangeLog.  Can you please update it, too?


Thanks,
Corinna

-- 
Corinna Vinschen
Cygwin Maintainer
Red Hat

Attachment: pgpVfqeSD11Io.pgp
Description: PGP signature