Oops, I didn't notice this thread was happening.  I was working with
PostgreSQL release engineering until recently but have been
transferred to other tasks.  Anyway, just one little comment:

On 30/09 14.58, Mayuresh Nirhali wrote:
> Nils,
> 
> Thanks for putting together the fix,
> 
> 1. SFW consolidation also includes version 8.2 of postgres and the bug 
> 3298 also applies to that version.
> I think it makes sense to fix 8.2 as well with the fix for this bug. Let 
> me know what you think.
> 
> 2. I see following files in the 32 bit pgxs directory, please check if 
> they should be present in 64 bit directories as well.
> d none usr/postgres/8.3/lib/pgxs/src/test 755 root bin
> d none usr/postgres/8.3/lib/pgxs/src/test/regress 755 root bin
> f none usr/postgres/8.3/lib/pgxs/src/test/regress/pg_regress 555 root bin

pg_regress is a test driver which does not link with any other
PostgreSQL code, so there's really no need to have a 64 bit version of
it, except for the sake of completeness.

I may be back later to look at the rest of this discussion.

-- 
Bjorn Munch      Bjorn.Munch at sun.com
Database Group, Sun Microsystems

Reply via email to