On 07/25/2013 09:48 AM, Tom Lane wrote:
> Andres Freund <andres@2ndquadrant.com> writes:
>>> Before that commit the checks for cmsgcred which includes sys/ucred.h
>>> happened to include params.h... Patch attached, missing the configure
>>> update since I don't have a compatible autoconf on my laptop, to produce
>>> a minimal diff.
>> Could somebody apply the fix (including regenerating /configure)?
> The proposed patch seems a bit overcomplicated --- isn't the real
> problem that I changed the ordering of the header probes in
> be4585b1c27ac5dbdd0d61740d18f7ad9a00e268? I think I just alphabetized
> them in a fit of neatnik-ism, not realizing that there were order
> dependencies on some platforms.
>
>
It looks to me like you didn't reorder anything, you added a test for
sys/ucred.h.
I haven't seen the proposed patch, though.
cheers
andrew
