Re: [PATCHES] backslashes in pgindent

2005-07-15 Thread Luke Lonergan
Bruce, On 7/15/05 9:59 PM, "Bruce Momjian" wrote: > Actually, mine returns ')' too for the last command. I didn't copy > that into the email. How about the top tests? Notice I get an error on > the first one without the backslash. Are you OK escaping '(' but not > ')'? That might be a solu

Re: [PATCHES] backslashes in pgindent

2005-07-15 Thread Bruce Momjian
Luke Lonergan wrote: > Bruce, > > I found that parentheses in gawk regular expressions require backslashes > > so they are not treated as regex groupings: > > > > $ echo '('|awk '$0 ~ /(/ {print $0}' > > awk: cmd. line:1: fatal: Unmatched ( or \(: /(/ > > $ echo '('|awk '$0 ~ /\(/ {print $0}' > >

Re: [PATCHES] backslashes in pgindent

2005-07-15 Thread Luke Lonergan
Bruce, > I found that parentheses in gawk regular expressions require backslashes > so they are not treated as regex groupings: > > $ echo '('|awk '$0 ~ /(/ {print $0}' > awk: cmd. line:1: fatal: Unmatched ( or \(: /(/ > $ echo '('|awk '$0 ~ /\(/ {print $0}' > ( > Now, it seems closing parenthe

[PATCHES] [patch 2/3] Fortuna fixes

2005-07-15 Thread Marko Kreen
After studying Fortuna more, I found out that I missed some of the details. - reseeding should happen only if pool #0 has aquired additional entropy. - a 'rekeying' operation should happend after each request and also after 1M of extracted data. That means taking next two blocks and using

[PATCHES] [patch 3/3] new documentation

2005-07-15 Thread Marko Kreen
- reformat in asciidoc syntax - clarify various parts - add high-level description of PGP structure - add a comparison of crypt vs. regular hashes Index: pgsql/contrib/pgcrypto/README.pgcrypto === *** pgsql.orig/contrib/pgcrypto/READ

[PATCHES] [patch 1/3] small cleanups

2005-07-15 Thread Marko Kreen
- Fix couple comments. - internal.c didnt clean hash contexts when freeing - zero the system randomness buffer too - Google tells that we can use /dev/urandom on Cygwin, HPUX and AIX. Add them to random.c - remove a debug reference from pgp.h Index: pgsql/contrib/pgcrypto/fortuna.c ===

[PATCHES] [patch 0/3] last large update to pgcrypto

2005-07-15 Thread Marko Kreen
Those 3 patches hopefully end my pgcrypto patch-bombing. (Well, at least I am offline next couple weeks...) 1. Small fixes 2. Fortuna fixes 3. New README There are 3 important points about new readme: * It is formatted for use by asciidoc. To be able to generate html from it. As I consider t

Re: [PATCHES] Change Ownership Permission Checks

2005-07-15 Thread Tom Lane
Stephen Frost <[EMAIL PROTECTED]> writes: > When writing this patch it occurred to me that we nuke our > member-of-role cache for one-off lookups on occation. I don't > particularly like that, especially when we *know* it's a one-off lookup, Yeah. What I had been thinking about is that maybe it

Re: [PATCHES] Change Ownership Permission Checks

2005-07-15 Thread Stephen Frost
* Tom Lane ([EMAIL PROTECTED]) wrote: > Stephen Frost <[EMAIL PROTECTED]> writes: > > Attached please find a patch to change how the permissions checking > > for alter-owner is done. With roles there can be more than one > > 'owner' of an object and therefore it becomes sensible to allow > >

Re: [PATCHES] thousands comma numeric formatting in psql

2005-07-15 Thread Bruce Momjian
Eugen Nedelcu wrote: > The new code is broken. Please test it with resonably large tables. > Do not test it with querys like: select -132323435.34343; > > If I use a query like: > > select * from my_table limit 100; > > I can't see anything on my screen until I hit CTRL^C > > If I use a pager (

[PATCHES] AIX FAQ Updates

2005-07-15 Thread Chris Browne
I believe this change will apply equally to 7.4, 8.0, and CVS HEAD. Index: FAQ_AIX === RCS file: /projects/cvsroot/pgsql/doc/FAQ_AIX,v retrieving revision 1.11 diff -c -u -r1.11 FAQ_AIX --- FAQ_AIX 12 Nov 2002 20:02:32 -

[PATCHES] fixing REL7_3_STABLE build issues

2005-07-15 Thread Andrew Dunstan
The attached (new) src/test/regress/expected/geometry_9.out, intended only for the 7.3 stable branch, allows a clean regression pass on my FC4 box. I called it that to avoid conflicts with other geometry_n files on later branches. The attached patch for contrib/seg/segparse.y allows a clean

Re: [PATCHES] PL/PGSQL: Dynamic Record Introspection

2005-07-15 Thread Neil Conway
Titus von Boxberg wrote: Having read the FAQ, it's not clear to me, where the patches go (and when) that I would send into this list. These are my open questions: Are the patches applied to HEAD? It depends on the patch. New features are almost exclusively only applied to HEAD, whereas patche

Re: [PATCHES] PL/PGSQL: Dynamic Record Introspection

2005-07-15 Thread Titus von Boxberg
> -Ursprüngliche Nachricht- > Von: Tom Lane [mailto:[EMAIL PROTECTED] > Neil Conway <[EMAIL PROTECTED]> writes: > > Not from me. You missed the 8.1 freeze by about two weeks, so I'm not > > sure if this is a candidate for 8.1 though. > > Not a chance. > > regards, tom

Re: [PATCHES] PL/PGSQL: Dynamic Record Introspection

2005-07-15 Thread Tom Lane
Neil Conway <[EMAIL PROTECTED]> writes: > Not from me. You missed the 8.1 freeze by about two weeks, so I'm not > sure if this is a candidate for 8.1 though. Not a chance. regards, tom lane ---(end of broadcast)--- TIP 3: H

Re: [PATCHES] PL/PGSQL: Dynamic Record Introspection

2005-07-15 Thread Neil Conway
Titus von Boxberg wrote: What do you mean by "right syntax". There are hundreds of examples in programming languages where a "small" syntactic difference leads to totally different semantics. Well, I'm just not happy that "foo.bar" means something completely different than "foo%bar" -- foo%ba