From: | Andrew Dunstan <andrew(at)dunslane(dot)net> |
---|---|
To: | Magnus Hagander <magnus(at)hagander(dot)net> |
Cc: | Robert Lor <Robert(dot)Lor(at)Sun(dot)COM>, Peter Eisentraut <peter_e(at)gmx(dot)net>, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, pgsql-hackers(at)postgresql(dot)org |
Subject: | Re: [COMMITTERS] pgsql: Enable probes to work with Mac OS X Leopard and other OSes that |
Date: | 2008-03-20 00:05:25 |
Message-ID: | 47E1AA45.4050506@dunslane.net |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-committers pgsql-hackers |
Magnus Hagander wrote:
> Andrew Dunstan wrote:
>>
>>
>> Robert Lor wrote:
>>> Peter Eisentraut wrote:
>>>> Well, yes. I meant to say, a build system that can supply the
>>>> functionality of Gen_fmgrtab can surely implement this new thing.
>>>> I see there is Perl being used, so it should be simple.
>>>>
>>> I was thinking of using a Perl script to generate the dummy header
>>> file but decided against it to avoid disrupting the build on other
>>> platforms. If sed doesn't work on Windows for some reason, we can
>>> use a Perl script instead.
>>>
>>>
>>
>> What we should do in fact is to use psed on Windows to execute the
>> script. This is a sed emulator that is part of the standard perl
>> install, and will save us the bother of having to maintain two
>> generation scripts, or switching from sed to perl. We just need to
>> find where to plug that into the Windows build process.
>
> Hadn't heard about that one. Sounds like a good way to do it, if you
> can find the correct place for it :-)
I'm thinking of the patch below. I'll try it out tomorrow.
cheers
andrew
Index: Solution.pm
===================================================================
RCS file: /cvsroot/pgsql/src/tools/msvc/Solution.pm,v
retrieving revision 1.36
diff -c -r1.36 Solution.pm
*** Solution.pm 28 Feb 2008 12:17:59 -0000 1.36
--- Solution.pm 20 Mar 2008 00:02:05 -0000
***************
*** 205,210 ****
--- 205,216 ----
close(T);
}
+ if
(IsNewer('src\include\utils\probes.h','src\backend\utils\probes.d'))
+ {
+ print "Generating probes.h...\n";
+ system('psed -f src\backend\utils\Gen_dummy_probes.sed
src\backend\utils\probes.d > src\include\utils\probes.h');
+ }
+
if
(IsNewer('src\interfaces\libpq\libpq.rc','src\interfaces\libpq\libpq.rc.in'))
{
print "Generating libpq.rc...\n";
From | Date | Subject | |
---|---|---|---|
Next Message | Heikki Linnakangas | 2008-03-20 10:30:05 | pgsql: Add the missing cyrillic "Yo" characters ('e' and 'E' with two |
Previous Message | Magnus Hagander | 2008-03-19 22:38:03 | Re: [COMMITTERS] pgsql: Enable probes to work with Mac OS X Leopard and other OSes that |
From | Date | Subject | |
---|---|---|---|
Next Message | Tom Lane | 2008-03-20 00:10:47 | Re: Text <-> C string |
Previous Message | Tom Lane | 2008-03-19 23:55:40 | Re: tsearch2 in postgresql 8.3.1 - invalid byte sequence for encoding "UTF8": 0xc3 |