Partially revert commit 536d47bd9d5fce8d91929bee3128fa1d08dbcc57.
authorTom Lane <tgl@sss.pgh.pa.us>
Sat, 22 Apr 2017 06:06:16 +0000 (02:06 -0400)
committerTom Lane <tgl@sss.pgh.pa.us>
Sat, 22 Apr 2017 06:06:16 +0000 (02:06 -0400)
commit5041cdf2b79216b10ca90498d9a9b8fd4abf1745
treef930524b5292bd03872dba8ce03bf40374a68df3
parentf58b664393dcfd02c2f57b3ff20fc0aee6dfebf1
Partially revert commit 536d47bd9d5fce8d91929bee3128fa1d08dbcc57.

Per buildfarm, the "#ifdef F_SETFD" removed in that commit actually
is needed on Windows, because fcntl() isn't available at all on that
platform, unless using Cygwin.  We could perhaps spell it more like
"#ifdef HAVE_FCNTL", or "#ifndef WIN32", but it's not clear that
those choices are better.

It does seem that we don't need the bogus manual definition of
FD_CLOEXEC, though, so keep that change.

Discussion: https://postgr.es/m/26254.1492805635@sss.pgh.pa.us
src/interfaces/libpq/fe-connect.c