Re: Extensions, patch 22 (cleanup, review, cleanup)

From: "Erik Rijkers" <er(at)xs4all(dot)nl>
To: "Dimitri Fontaine" <dimitri(at)2ndQuadrant(dot)fr>
Cc: "PostgreSQL Hackers" <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: Extensions, patch 22 (cleanup, review, cleanup)
Date: 2010-12-20 21:55:52
Message-ID: 3ec5f19d5783a5f4f65f9ac1f8090864.squirrel@webmail.xs4all.nl
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Mon, December 20, 2010 22:35, Dimitri Fontaine wrote:
> Hi,
>
> From last round of review from Robert and Ã&#65533;lvaro, here's the patch
> version 22. Changes:
>
> - cleanup contrib/ and 'Adjust search_path' comments
> - remove contrib/*/uninstall* scripts
> - add some documentation to the NO USER DATA option
> - remove objects locking in the pg_extension_objects() SRF, per Robert
> - propose split patches
>

I used the 'full' patch.

During configure I spotted this:

[...]
checking for bison... /usr/bin/bison
configure: using bison (GNU Bison) 2.3
gawk: { if ($4 < 1.875-extension) exit 0; else exit 1;}
gawk: ^ syntax error
gawk: { if ($4 < 1.875-extension) exit 0; else exit 1;}
gawk: ^ syntax error
checking for flex... /usr/local/bin/flex
configure: using flex 2.5.3
[...]

Otherwise the patch applies, and compiles, checks, installs and runs OK. (obviously I haven't
tested anything yet)

Linux Centos 5.4

Erik Rijkers

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Erik Rijkers 2010-12-20 22:06:08 Re: Extensions, patch 22 (cleanup, review, cleanup)
Previous Message Dimitri Fontaine 2010-12-20 21:35:44 Extensions, patch 22 (cleanup, review, cleanup)