Martijn van Oosterhout <kleptog(at)svana(dot)org> writes:
> This implements a proposal made last november:
Ah, good, I'd been meaning to do this.
> changes in any of the following:
> the size of 8 basic C types
> The compiler used (only brand, not version)
That seems way overkill to me. FUNC_MAX_ARGS is good to check, but
most of those other things are noncritical for typical add-on modules.
In particular I strongly object to the check on compiler. Some of us do
use systems where gcc and vendor compilers are supposed to interoperate
... and aren't all those Windows compilers supposed to, too? AFAIK
it's considered the linker's job to prevent loading 32-bit code into
a 64-bit executable or vice versa, so I don't think we need to be
checking for common assumptions about sizeof(long).
> Currently, modules without a magic block are merely logged at LOG
> level. This needs some discussion though.
I'm pretty sure we had agreed that magic blocks should be required;
otherwise this check will accomplish little.
regards, tom lane
In response to
pgsql-hackers by date
|Next:||From: Bruce Momjian||Date: 2006-05-08 02:39:52|
|Subject: Re: [HACKERS] Question on win32 semaphore simulation|
|Previous:||From: Martijn van Oosterhout||Date: 2006-05-07 21:17:05|
|Subject: [PATCH] Magic block for modules|
pgsql-patches by date
|Next:||From: Bruce Momjian||Date: 2006-05-08 02:20:17|
|Subject: Re: pgstat: remove delayed destroy / pipe:|
|Previous:||From: Tom Lane||Date: 2006-05-08 00:11:17|
|Subject: Re: Page at a time index scan |