On 10/08/2018 09:55 PM, Tom Lane wrote:
Thomas Munro <thomas.mu...@enterprisedb.com> writes:
Rebased again. Patches that touch AC_CHECK_FUNCS are fun like that!
Yeah, I've been burnt by that too recently. It occurs to me we could make
that at least a little less painful if we formatted the macro with one
line per function name:
AC_CHECK_FUNCS([
cbrt
clock_gettime
fdatasync
...
wcstombs_l
])
You'd still get conflicts in configure itself, of course, but that
doesn't require manual work to resolve -- just re-run autoconf.
By and large I think it's better not to submit patches with changes to
configure, but to let the committer run autoconf.
You can avoid getting such changes in your patches by doing something
like this:
git config diff.nodiff.command /bin/true
echo configure diff=nodiff >> .git/info/attributes
If you actually want to turn this off and see any diffs in configure, run
git diff --no-ext-diff
It's also possible to supply a filter expression to 'git diff'.
OTOH, this will probably confuse the heck out of the cfbot patch checker.
cheers
andrew
--
Andrew Dunstan https://www.2ndQuadrant.com
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services
