pgsql: Make the Windows tcp keepalive support depend on the existance of

Started by Magnus Haganderalmost 16 years ago3 messagescomitters
Jump to latest
#1Magnus Hagander
magnus@hagander.net

Log Message:
-----------
Make the Windows tcp keepalive support depend on the existance of the
SIO_KEEPALIVE_VALS define instead of just WIN32, since MingW doesn't
support this API (yet?).

Modified Files:
--------------
pgsql/src/backend/libpq:
pqcomm.c (r1.211 -> r1.212)
(http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/src/backend/libpq/pqcomm.c?r1=1.211&r2=1.212)
pgsql/src/interfaces/libpq:
fe-connect.c (r1.397 -> r1.398)
(http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/src/interfaces/libpq/fe-connect.c?r1=1.397&r2=1.398)

#2Andrew Dunstan
andrew@dunslane.net
In reply to: Magnus Hagander (#1)
Re: pgsql: Make the Windows tcp keepalive support depend on the existance of

Magnus Hagander wrote:

Log Message:
-----------
Make the Windows tcp keepalive support depend on the existance of the
SIO_KEEPALIVE_VALS define instead of just WIN32, since MingW doesn't
support this API (yet?).

Maybe I misunderstood. I thought you were going to include the required
defines if they were missing, rather than just disable the code in that
case.

cheers

andrew

#3Magnus Hagander
magnus@hagander.net
In reply to: Andrew Dunstan (#2)
Re: pgsql: Make the Windows tcp keepalive support depend on the existance of

On Thu, Jul 8, 2010 at 18:40, Andrew Dunstan <andrew@dunslane.net> wrote:

Magnus Hagander wrote:

Log Message:
-----------
Make the Windows tcp keepalive support depend on the existance of the
SIO_KEEPALIVE_VALS define instead of just WIN32, since MingW doesn't
support this API (yet?).

Maybe I misunderstood. I thought you were going to include the required
defines if they were missing, rather than just disable the code in that
case.

One of us definitely misunderstood, because this is how I read Tom's
suggestion. It'll be unsupported on mingw until they add the required
define and structure - and when they do, we'll automatically support
it.

--
Magnus Hagander
Me: http://www.hagander.net/
Work: http://www.redpill-linpro.com/