Back-patch some minor bug fixes in GUC code.
authorTom Lane <tgl@sss.pgh.pa.us>
Sun, 28 Jun 2015 22:38:06 +0000 (18:38 -0400)
committerTom Lane <tgl@sss.pgh.pa.us>
Sun, 28 Jun 2015 22:38:06 +0000 (18:38 -0400)
commit5a56c254588a91de393b8ce99f8deeefc6a44d67
tree2aabeffd3113abc227dc92f968c95da6ad6a94bb
parentf9b38ab6536e0c175cf94bea0c8f2a3d9fed175a
Back-patch some minor bug fixes in GUC code.

In 9.4, fix a 9.4.1 regression that allowed multiple entries for a
PGC_POSTMASTER variable to cause bogus complaints in the postmaster log.
(The issue here was that commit bf007a27acd7b2fb unintentionally reverted
3e3f65973a3c94a6, which suppressed any duplicate entries within
ParseConfigFp.  Back-patch the reimplementation just made in HEAD, which
makes use of an "ignore" field to prevent application of superseded items.)

Add missed failure check in AlterSystemSetConfigFile().  We don't really
expect ParseConfigFp() to fail, but that's not an excuse for not checking.

In both 9.3 and 9.4, remove mistaken assignment to ConfigFileLineno that
caused line counting after an include_dir directive to be completely wrong.
src/backend/utils/misc/guc-file.l