Perhaps I wasn't clear: the *present* behavior of pgcrypto is to
"compile successfully but ... be unusable" if an invalid random source
is defined. This is prone to error. That patch changes this behavior to
refuse to compile if an invalid random source has been defined.
On Mon, 2004-11-22 at 10:46 -0500, Tom Lane wrote:
> > +$(error "Unrecognized random source: $(random)")
> Doesn't look like a successful compile to me.
Right, that's the whole point.
> What's worse, this would also cause maintenance operations (like
> "make clean") to fail.
I don't think this is a problem (an invalid random source is effectively
a misconfiguration and could only be the result of user error). But if
you like I can change the patch so that we error out when attempting to
> (I see little point in this anyway given that the makefile defaults to
> "random = silly" further up.)
This patch was specifically motivated by a user who was confused when
they edited the Makefile, compiled pgcrypto, and couldn't use it -- as
it turned out the random source was defined with trailing whitespace. I
think the Makefile syntax is sufficiently esoteric that some error
checking would be helpful.
In response to
pgsql-patches by date
|Next:||From: Tom Lane||Date: 2004-11-23 00:10:13|
|Subject: Re: pgcrypto: Makefile check |
|Previous:||From: Tom Lane||Date: 2004-11-22 21:30:32|
|Subject: Re: another plperl bug |