Re: Require Assistance Solving Problem - erroneous pg_hba.conf

"Greg Kelley" <gkelley(at)britannicaviation(dot)com> writes:
> A RedHat Upgrade Saturday from 8.0 to 9.0 just did a 'minor' version
> upgrade from 7.3.1 to 7.3.2.
> [ and broke his pg_hba.conf ]

Ah-hah, I now know where the behavioral change came from: the Red Hat
RPMs contain this patch:

--- postgresql-7.3.2/src/backend/libpq/hba.c.sopwith 2003-01-09 09:09:01.000000000 -0500
+++ postgresql-7.3.2/src/backend/libpq/hba.c 2003-01-09 09:09:09.000000000 -0500
@@ -72,11 +72,13 @@
* Some standard C libraries, including GNU, have an isblank() function.
* Others, including Solaris, do not. So we have our own.
*/
+#ifndef isblank
static bool
isblank(const char c)
{
return c == ' ' || c == '\t' || c == '\r';
}
+#endif


/*

which on RHL 9 causes the system definition of isblank() to be
substituted for Postgres'. That means \r is no longer treated as
whitespace, as it was in prior versions.

There is a fix in our CVS tree that prevents this problem and restores
the old treatment of \r, but for now you gotta get rid of the Windows-
style newlines.

regards, tom lane