public inbox for cygwin-developers@cygwin.com
 help / color / mirror / Atom feed
* About <sys/select.h> and <sys/time.h>, without gnulib
@ 2016-05-13 10:46 Michael Haubenwallner
  2016-05-18 19:07 ` Corinna Vinschen
  0 siblings, 1 reply; 2+ messages in thread
From: Michael Haubenwallner @ 2016-05-13 10:46 UTC (permalink / raw)
  To: cygwin-developers

Hi,

independent of gnulib, I don't believe this include order should break:

   $ cat sys-time-h.c
   #include <sys/select.h>
   #include <sys/time.h>

   $ gcc -c sys-time-h.c
   In file included from /usr/include/time.h:12:0,
                    from /usr/include/sys/time.h:268,
                    from sys-time-h.c:2:
   /usr/include/sys/reent.h:276:3: error: expected specifier-qualifier-list before ‘_fpos64_t’
      _fpos64_t _EXFNPTR(_seek64, (struct _reent *, _PTR, _fpos64_t, int));
      ^
   $

This is with 64-bit cygwin-2.5.1 and gcc-5.3.0.

It does help to reverse their order, or include <sys/types.h> before,
but I've failed to find something like that to be required by the specs...

Thoughts?
/haubi/

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: About <sys/select.h> and <sys/time.h>, without gnulib
  2016-05-13 10:46 About <sys/select.h> and <sys/time.h>, without gnulib Michael Haubenwallner
@ 2016-05-18 19:07 ` Corinna Vinschen
  0 siblings, 0 replies; 2+ messages in thread
From: Corinna Vinschen @ 2016-05-18 19:07 UTC (permalink / raw)
  To: cygwin-developers

[-- Attachment #1: Type: text/plain, Size: 1134 bytes --]

On May 13 12:46, Michael Haubenwallner wrote:
> Hi,
> 
> independent of gnulib, I don't believe this include order should break:
> 
>   $ cat sys-time-h.c
>   #include <sys/select.h>
>   #include <sys/time.h>
> 
>   $ gcc -c sys-time-h.c
>   In file included from /usr/include/time.h:12:0,
>                    from /usr/include/sys/time.h:268,
>                    from sys-time-h.c:2:
>   /usr/include/sys/reent.h:276:3: error: expected specifier-qualifier-list before ‘_fpos64_t’
>      _fpos64_t _EXFNPTR(_seek64, (struct _reent *, _PTR, _fpos64_t, int));
>      ^
>   $
> 
> This is with 64-bit cygwin-2.5.1 and gcc-5.3.0.
> 
> It does help to reverse their order, or include <sys/types.h> before,
> but I've failed to find something like that to be required by the specs...
> 
> Thoughts?

Fixed in the repo.  The new newlib header layout missed to include
the configuration info from sys/config.h before including sys/reent.h.


HTH,
Corinna

-- 
Corinna Vinschen                  Please, send mails regarding Cygwin to
Cygwin Maintainer                 cygwin AT cygwin DOT com
Red Hat

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 819 bytes --]

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2016-05-18 19:07 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-05-13 10:46 About <sys/select.h> and <sys/time.h>, without gnulib Michael Haubenwallner
2016-05-18 19:07 ` Corinna Vinschen

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).