Scott Marlowe <scott.marlowe@xxxxxxxxx> writes: > In the init script from the PGDG rpms there's this block of code: > PGDATA=/var/lib/pgsql/data > if [ -f "$PGDATA/PG_VERSION" ] && [ -d "$PGDATA/base/template1" ] > then > echo "Using old-style directory structure" > else > PGDATA=/var/lib/pgsql/data > fi > Is it just me, or is the else extra noise? Just wondering if there's > a reason a config variable is in two places at once. In the original coding, the first assignment was PGDATA=/var/lib/pgsql and thus the if-test did indeed do something useful with setting PGDATA differently in the two cases. However, there is no reason whatsoever for this initscript to be prepared to work with postmaster versions that would be old enough for the if-test to succeed. I took the whole if-block out of the just-updated Fedora RPMs, and would recommend the same for PGDG. regards, tom lane -- Sent via pgsql-general mailing list (pgsql-general@xxxxxxxxxxxxxx) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-general
