Re: [BUGS] grant/revoke bug with delete/update

2000-06-13 Thread Jerome Alet

On Fri, 9 Jun 2000, Bruce Momjian wrote:

> Are we addressing this?

Yes, please do.

And please don't forget the following: 

when dropping an user postgresql (actually the superuser must do it
manually) should first revoke all user's permissions on all databases,
because the deleted userid is reused on the next create user so the new
user inherits all permissions from the deleted user => may be very very
bad (an example of what can be done is not necessary I suppose ?)

> > And for the bug report I posted on Feb 23rd on "drop user" which keeps the
> > user's acl in the database, and the deleted user id being reused, I've not
> > done anything, but I consider this a major problem. Please consider it for
> > a next version.

bye,
Jerome ALET - [EMAIL PROTECTED] - http://cortex.unice.fr/~jerome
Faculte de Medecine de Nice - http://noe.unice.fr - Tel: 04 93 37 76 30 
28 Avenue de Valombrose - 06107 NICE Cedex 2 - FRANCE





Re: [BUGS] Small bug with numeric in 7.0 (also in 6.5.3)

2000-06-13 Thread Bruce Momjian

Any comments on this one?


> Raul Chirea <[EMAIL PROTECTED]> writes:
> > So, it seems to be a parser difficulty (bug) to correctly determine
> > the type of a numeric constant with decimals (like 99.9).
> 
> Yes.  You can find more about this in the pgsql-hackers archives.
> We've been aware of the problem for a while but are unsure as yet
> how to solve it without breaking other cases --- ie, cases where
> you *do* want such a constant to be treated as float not numeric.
> 
> For the moment, we have left the behavior as it's always been in
> Postgres (ie, undecorated non-integral constants are taken to be
> 'float8'), so as not to break existing applications.
> 
>   regards, tom lane
> 


-- 
  Bruce Momjian|  http://www.op.net/~candle
  [EMAIL PROTECTED]   |  (610) 853-3000
  +  If your life is a hard drive, |  830 Blythe Avenue
  +  Christ can be your backup.|  Drexel Hill, Pennsylvania 19026



Re: [BUGS] jdbc2 bug in absolute (ResultSet.java)

2000-06-13 Thread Bruce Momjian

> The "absolute" method is called by the relative method to change the
> cursor position.  However, I noticed that absolute does not change
> current_row, it only loads the data.
> 
> Shoudn't absolute contain the following line :
> 
> current_row=index;
> 

The current code shows:

current_row=internalIndex;

Seems it has been fixed by someone.  Yes, I see now that the 7.0.0
version has this missing.

Not sure how to handle this.  The fix will be in 7.1.  Just not sure if
I should back-patch this into 7.0.X because there are other fixes in
those java patches I don't understand.

-- 
  Bruce Momjian|  http://www.op.net/~candle
  [EMAIL PROTECTED]   |  (610) 853-3000
  +  If your life is a hard drive, |  830 Blythe Avenue
  +  Christ can be your backup.|  Drexel Hill, Pennsylvania 19026



Re: [BUGS] libpgtcl

2000-06-13 Thread Bruce Momjian

Applied.


> Hello,
> 
> in the Directory src/bin/pgtclsh there is a good mechanism to integrate
> the
> tcl-spec in the Makefile.
> The patch in the attachemant does this in the src/interfaces/libpgtcl
> too
> 
> Thank you
>  Rudolf
> --
> ---
> Rudolf Weber
> http://www.infnet.verein.de/home/rw 
> E-Mail: [EMAIL PROTECTED]

> --- ./src/interfaces/libpgtcl/mkMakefile.tcldefs.sh.in.orig   Tue Jun  6 01:03:10 
>2000
> +++ ./src/interfaces/libpgtcl/mkMakefile.tcldefs.sh.inTue Jun  6 01:02:19 
>2000
> @@ -0,0 +1,16 @@
> +
> +if [ ! -f @TCL_CONFIG_SH@ ]; then
> +echo "@TCL_CONFIG_SH@ not found"
> +echo "I need this file! Please make a symbolic link to this file"
> +echo "and start make again."
> +exit 1
> +fi
> +
> +cat @TCL_CONFIG_SH@ |
> +egrep '^TCL_' |
> +while read inp
> +do
> + eval eval echo $inp
> +done >Makefile.tcldefs
> +
> +exit 0
> --- ./src/interfaces/libpgtcl/mkMakefile.tkdefs.sh.in.origTue Jun  6 01:03:21 
>2000
> +++ ./src/interfaces/libpgtcl/mkMakefile.tkdefs.sh.in Tue Jun  6 01:02:19 2000
> @@ -0,0 +1,18 @@
> +
> +if [ ! -f @TK_CONFIG_SH@ ]; then
> +echo "@TK_CONFIG_SH@ not found"
> +echo "I need this file! Please make a symbolic link to this file"
> +echo "and start make again."
> +exit 1
> +fi
> +
> +. @TK_CONFIG_SH@
> +
> +cat @TK_CONFIG_SH@ |
> +egrep '^TK_' |
> +while read inp
> +do
> + eval eval echo $inp
> +done >Makefile.tkdefs
> +
> +exit 0
> --- ./src/interfaces/libpgtcl/Makefile.in.origWed Mar  8 02:58:36 2000
> +++ ./src/interfaces/libpgtcl/Makefile.in Tue Jun  6 01:22:46 2000
> @@ -17,6 +17,14 @@
>  SRCDIR= @top_srcdir@
>  include $(SRCDIR)/Makefile.global
>  
> +#
> +# Include definitions from the tclConfig.sh file
> +#
> +include Makefile.tcldefs
> +ifeq ($(USE_TK), true)
> +include Makefile.tkdefs
> +endif
> +
>  CFLAGS+= -I$(SRCDIR)/backend \
>   -I$(SRCDIR)/include \
>   -I$(LIBPQDIR)
> @@ -48,6 +56,11 @@
>  beforeinstall-headers:
>   -@if [ ! -d $(HEADERDIR) ]; then mkdir $(HEADERDIR); fi
>  
> +Makefile.tcldefs: mkMakefile.tcldefs.sh
> + /bin/sh mkMakefile.tcldefs.sh
> +
> +Makefile.tkdefs: mkMakefile.tkdefs.sh
> + /bin/sh mkMakefile.tkdefs.sh
>  
>  .PHONY: clean
>  
> --- ./src/configure.in.orig   Thu May 25 00:43:59 2000
> +++ ./src/configure.inTue Jun  6 01:16:10 2000
> @@ -1371,6 +1371,8 @@
>   interfaces/ecpg/preproc/Makefile
>   interfaces/libpq++/Makefile
>   interfaces/libpgeasy/Makefile
> + interfaces/libpgtcl/mkMakefile.tcldefs.sh
> + interfaces/libpgtcl/mkMakefile.tkdefs.sh
>   interfaces/libpgtcl/Makefile
>   interfaces/odbc/GNUmakefile
>   interfaces/odbc/Makefile.global


-- 
  Bruce Momjian|  http://www.op.net/~candle
  [EMAIL PROTECTED]   |  (610) 853-3000
  +  If your life is a hard drive, |  830 Blythe Avenue
  +  Christ can be your backup.|  Drexel Hill, Pennsylvania 19026



Re: [BUGS] Problem with ecpg

2000-06-13 Thread Tom Lane

Bruce Momjian <[EMAIL PROTECTED]> writes:
> Can someone comment on this patch?

>> This forbids ":" to be the second (or above) char in an operator.
>> Is there any situation where it should be?

It would surely be intolerable for ecpg to have different ideas about
what is an operator name than the postgres backend does.  So I'm not
happy about patching ecpg this way without changing the backend.

However, we have already deprecated the ':' and ';' operators with
the intention of removing those operators in 7.1, and forbidding
those two characters entirely in operator names.  So the problem
should go away in 7.1.

I haven't yet done the physical removal of ':' and ';' but it's on
the hit list...

regards, tom lane