[PATCH] cygwin: make sys/socket.h completely visible from netinet/in.h

Corinna Vinschen corinna-cygwin@cygwin.com
Fri Jan 19 10:19:00 GMT 2018


On Jan 19 01:28, Yaakov Selkowitz wrote:
> On 2018-01-16 03:30, Corinna Vinschen wrote:
> > You don't explain the incentive behind removing sys/time.h. 
> 
> POSIX does not mention the inclusion of <sys/time.h> in <sys/socket.h>
> or <netinet/in.h>, nor is there anything in the latter two that would
> require the former.  I can make this clearer in the commit message.
> 
> > Sure this doesn't break anything?
> Without a mass scratch rebuild I can never be 100% sure. :-)  But so
> far, no issues, and the first part of this definitely helps.

ACK.  Please apply these as two separate patches so we can easily revert
the include *if* it results in problems.


Thanks,
Corinna

-- 
Corinna Vinschen                  Please, send mails regarding Cygwin to
Cygwin Maintainer                 cygwin AT cygwin DOT com
Red Hat
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <http://cygwin.com/pipermail/cygwin-patches/attachments/20180119/dc893863/attachment.sig>


More information about the Cygwin-patches mailing list