<div dir="ltr"><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Aug 22, 2013 at 7:09 PM, Sebastian Ramacher <span dir="ltr"><<a href="mailto:sramacher@debian.org" target="_blank">sramacher@debian.org</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">On 2013-08-19 14:23:17, Jean-Paul Saman wrote:<br>
> On Mon, Aug 19, 2013 at 11:38 AM, Alessio Treglia <<a href="mailto:alessio@debian.org">alessio@debian.org</a>>wrote:<br>
><br>
> > Hi there,<br>
> ><br>
> > libdvbpsi failed to build on Debian kfreebsd in the past.<br>
> > Sebastian Ramacher worked on a patch to fix such issue, the description<br>
> > follows:<br>
> > MSG_CMSG_CLOEXEC is a flag for recvmsg and not recv. Thus rewrite<br>
> > {udp,tcp}_open to create sockets with SOCK_CLOEXEC. If SOCK_CLOEXEC is<br>
> > not<br>
> > available, FD_CLOEXEC is set instead.<br>
> ><br>
> > You find the patch attached, thanks for considering.<br>
> > Cheers,<br>
> ><br>
><br>
> Thanks for sending in this patch. I reviewed and modified the patch and<br>
> attached it here for double checking by you.<br>
<br>
</div>With the new patch, #include <stdbool.h> is needed in both tcp.c and<br>
udp.c:<br></blockquote><div><br></div><div>oops. thanks for testing.<br><br></div><div>Kind regards,<br><br>Jean-Paul Saman<br> <br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
| tcp.c:62:1: error: unknown type name 'bool'<br>
| static bool set_fdsocketclosexec(int s)<br>
| ^<br>
| tcp.c: In function 'set_fdsocketclosexec':<br>
| tcp.c:69:20: error: 'true' undeclared (first use in this function)<br>
| return true;<br>
| ^<br>
| tcp.c:69:20: note: each undeclared identifier is reported only once for each function it appears in<br>
| tcp.c:74:12: error: 'false' undeclared (first use in this function)<br>
| return false;<br>
| ^<br>
| tcp.c:75:1: error: control reaches end of non-void function [-Werror=return-type]<br>
| }<br>
| ^<br>
| cc1: all warnings being treated as errors<br>
<br>
(libdvbpsi 1.1.0 with the patch applied.)<br>
<br>
Regards<br>
<span class="HOEnZb"><font color="#888888">--<br>
Sebastian Ramacher<br>
</font></span></blockquote></div><br></div></div>