Re: MinGW build with IPv6

--===============1081895110443922020==
Content-Type: multipart/alternative; boundary=001a113edfda77a50f0529f53709

--001a113edfda77a50f0529f53709
Content-Type: text/plain; charset=UTF-8

I prefer not to use the winsock pollfd strict since polipo does not use the
corresponding and broken WSApoll. I pulled in Windows Vista to get some
IPv6 functionality. Maybe it was for getaddrinfo. I will look at it
tonight, clean up some of the unlink code in diskcache and post a full diff.

Gabe
On Jan 22, 2016 9:23 AM, "Gisle Vanem" <[email protected]> wrote:

> Gabriel Klyber wrote:
>
> > These modifications require Windows Vista or newer. I am not sure how
> > early you need to define WINVER, but in my environment, mingw.h was
> > too late.
>
> I'm not sure if your patch would pull in some Vista+ exports
> from ws2_32.dll? Making it impossible to run Polipo on Win-XP.
>
> But to just use 'struct pollfd', one can do:
>
>   struct win32_pollfd {
>      SOCKET fd;        /* file descriptor */
>      short events;     /* requested events */
>      short revents;    /* returned events */
>  };
>  #define pollfd               win32_pollfd
>  #define poll(x, y, z)        win32_poll(x, y, z)
>
>
> I'm not sure it's a god idea to *require* a specific WINVER
> to build Polipo (default here in my SDK 8.1 is 0x0501).
> There is AFAICS nothing in Polipo that needs anything more advanced
> than is already in Win-XP (0x0501).
>
>
> --gv
>
> --
> --gv
>
>
> ------------------------------------------------------------------------------
> Site24x7 APM Insight: Get Deep Visibility into Application Performance
> APM + Mobile APM + RUM: Monitor 3 App instances at just $35/Month
> Monitor end-to-end web transactions and take corrective actions now
> Troubleshoot faster and improve end-user experience. Signup Now!
> http://pubads.g.doubleclick.net/gampad/clk?id=267308311&iu=/4140
> _______________________________________________
> Polipo-users mailing list
> [email protected]
> https://lists.sourceforge.net/lists/listinfo/polipo-users
>

--001a113edfda77a50f0529f53709
Content-Type: text/html; charset=UTF-8
Content-Transfer-Encoding: quoted-printable

<p dir=3D"ltr">I prefer not to use the winsock pollfd strict since polipo d=
oes not use the corresponding and broken WSApoll. I pulled in Windows Vista=
 to get some IPv6 functionality. Maybe it was for getaddrinfo. I will look =
at it tonight, clean up some of the unlink code in diskcache and post a ful=
l diff.</p>
<p dir=3D"ltr">Gabe<br>
</p>
<div class=3D"gmail_quote">On Jan 22, 2016 9:23 AM, &quot;Gisle Vanem&quot;=
 &lt;<a href=3D"mailto:[email protected]">[email protected]</a>&gt; wrote:<br t=
ype=3D"attribution"><blockquote class=3D"gmail_quote" style=3D"margin:0 0 0=
 .8ex;border-left:1px #ccc solid;padding-left:1ex">Gabriel Klyber wrote:<br=
>
<br>
&gt; These modifications require Windows Vista or newer. I am not sure how<=
br>
&gt; early you need to define WINVER, but in my environment, mingw.h was<br=
>
&gt; too late.<br>
<br>
I&#39;m not sure if your patch would pull in some Vista+ exports<br>
from ws2_32.dll? Making it impossible to run Polipo on Win-XP.<br>
<br>
But to just use &#39;struct pollfd&#39;, one can do:<br>
<br>
=C2=A0 struct win32_pollfd {<br>
=C2=A0 =C2=A0 =C2=A0SOCKET fd;=C2=A0 =C2=A0 =C2=A0 =C2=A0 /* file descripto=
r */<br>
=C2=A0 =C2=A0 =C2=A0short events;=C2=A0 =C2=A0 =C2=A0/* requested events */=
<br>
=C2=A0 =C2=A0 =C2=A0short revents;=C2=A0 =C2=A0 /* returned events */<br>
=C2=A0};<br>
=C2=A0#define pollfd=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0=
win32_pollfd<br>
=C2=A0#define poll(x, y, z)=C2=A0 =C2=A0 =C2=A0 =C2=A0 win32_poll(x, y, z)<=
br>
<br>
<br>
I&#39;m not sure it&#39;s a god idea to *require* a specific WINVER<br>
to build Polipo (default here in my SDK 8.1 is 0x0501).<br>
There is AFAICS nothing in Polipo that needs anything more advanced<br>
than is already in Win-XP (0x0501).<br>
<br>
<br>
--gv<br>
<br>
--<br>
--gv<br>
<br>
---------------------------------------------------------------------------=
---<br>
Site24x7 APM Insight: Get Deep Visibility into Application Performance<br>
APM + Mobile APM + RUM: Monitor 3 App instances at just $35/Month<br>
Monitor end-to-end web transactions and take corrective actions now<br>
Troubleshoot faster and improve end-user experience. Signup Now!<br>
<a href=3D"http://pubads.g.doubleclick.net/gampad/clk?id=3D267308311&amp;iu=
=3D/4140" rel=3D"noreferrer" target=3D"_blank">http://pubads.g.doubleclick.=
net/gampad/clk?id=3D267308311&amp;iu=3D/4140</a><br>
_______________________________________________<br>
Polipo-users mailing list<br>
<a href=3D"mailto:[email protected]">[email protected]=
urceforge.net</a><br>
<a href=3D"https://lists.sourceforge.net/lists/listinfo/polipo-users" rel=
=3D"noreferrer" target=3D"_blank">https://lists.sourceforge.net/lists/listi=
nfo/polipo-users</a><br>
</blockquote></div>

--001a113edfda77a50f0529f53709--


--===============1081895110443922020==
Content-Type: text/plain; charset="us-ascii"
MIME-Version: 1.0
Content-Transfer-Encoding: 7bit
Content-Disposition: inline

------------------------------------------------------------------------------
Site24x7 APM Insight: Get Deep Visibility into Application Performance
APM + Mobile APM + RUM: Monitor 3 App instances at just $35/Month
Monitor end-to-end web transactions and take corrective actions now
Troubleshoot faster and improve end-user experience. Signup Now!
http://pubads.g.doubleclick.net/gampad/clk?id=267308311&iu=/4140
--===============1081895110443922020==
Content-Type: text/plain; charset="us-ascii"
MIME-Version: 1.0
Content-Transfer-Encoding: 7bit
Content-Disposition: inline

_______________________________________________
Polipo-users mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/polipo-users

--===============1081895110443922020==--