<br><font size=2 face="sans-serif">OK, I am trying again.</font>
<br>
<br><font size=2 face="sans-serif">ChangeLog:</font>
<br>
<br><font size=2 face="sans-serif">Definitions for struct ip, tcphdr, and udphdr from BSD are added.</font>
<br>
<br>
<br><font size=2 face="sans-serif">I hope I can do better next time. Sorry for not reading the contrib page very carefully.</font>
<br>
<br><font size=2 face="sans-serif">Regards,</font>
<br>
<br><font size=2 face="sans-serif">Wu Yongwei</font>
<br>
<br>
<br>
<br>
<br><font size=2><tt>On Tue, Apr 09, 2002 at 09:56:37AM +0800, Wu Yongwei wrote:<br>
>ChangeLog: BSD-style header files ip.h, tcp.h, and udp.h are added, which<br>
>include definitions for IP, TCP, and UDP packet header structures.<br>
<br>
>Positions:<br>
>* ip.h.diff is against /usr/include/netinet/ip.h<br>
>* tcp.h.diff is against /usr/include/netinet/tcp.h<br>
>* udp.h should be added to /usr/include/netinet<br>
>* ip.h in /usr/include/cygwin contains only a comment and I suppose it could<br>
>be dropped.<br>
><br>
><br>
>BSD licence:<br>
>1. Redistributions of source code must retain the above copyright<br>
>   notice, this list of conditions and the following disclaimer.<br>
>2. Redistributions in binary form must reproduce the above copyright<br>
>   notice, this list of conditions and the following disclaimer in the<br>
>   documentation and/or other materials provided with the distribution.<br>
>3. All advertising materials mentioning features or use of this software<br>
>   must display the following acknowledgement:<br>
>     This product includes software developed by the University of<br>
>     California, Berkeley and its contributors.<br>
>4. Neither the name of the University nor the names of its contributors<br>
>   may be used to endorse or promote products derived from this software<br>
>   without specific prior written permission.<br>
><br>
>Best regards,<br>
<br>
1) Patches just to cygwin-patches, please.<br>
<br>
2) Your ChangeLog is incorrect.<br>
<br>
3) Please submit 1 (one) patch for everything rather than one patch per file.<br>
   It just makes a reviewer's life harder if you submit multiple patches as<br>
   attachments.  If your mailer allows you to submit patches inline without<br>
   screwing up spacing that is preferred.<br>
<br>
4) The diffs and ChangeLog should make it very obvious what is being changed.<br>
   There is no need for more words at this point.  If this was your first<br>
   submission asking for a change to Cygwin, then, sure, add a description.<br>
   You've already beaten this subject to death, however, so there is no need<br>
   for more justification.<br>
<br>
None of the above is a show stopper except for 2.  As I predicted, your<br>
ChangeLog is incorrect.  Go back to the web page that I keep mentioning<br>
to see why.<br>
<br>
cgf<br>
</tt></font>
<br>