From: | Magnus Hagander <magnus(at)hagander(dot)net> |
---|---|
To: | Greg Smith <greg(at)2ndquadrant(dot)com>, Hiroshi Saito <z-saito(at)guitar(dot)ocn(dot)ne(dot)jp> |
Cc: | PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: Foreign Visual Studio builds |
Date: | 2010-10-15 15:13:45 |
Message-ID: | AANLkTikdW+tjwgpkPKte3-WeNHPhN_nd1WLjPVi8x7d7@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Thu, Oct 14, 2010 at 19:37, Greg Smith <greg(at)2ndquadrant(dot)com> wrote:
> We got an interesting documentation document left by Christian Freund
> recently, in regards to
> http://www.postgresql.org/docs/9.0/interactive/install-windows-full.html ;
> it says:
>
> Regarding 16.1.3 - "perl mkvcbuild.pl"
>
> In case you use a German version of VC change line 69 in "Solution.pm" to:
>
> if ($line !~ /^Microsoft\s*\(R\) Visual C\+\+-Projekt-Generator -
> \D+(\d+)\.00\.\d+/)
>
>
> I'm going to approve the doc comment so that it might help someone else that
> runs into this in the short-term, but I thought it was was worth sharing as
> something that might be improved in the build code instead. That regex
> seems a bit too specific.
yeah. I wonder if we can't just get away with:
if ($line !~ /^Microsoft\s*\(R\) Visual C\+\+ [^-]+ - \D+(\d+)\.00\.\d+/)
That's still quite specific, but should work as long as they don't
localize the product name itself. I doubt they do that even in
Japanese/Chinese - Hiroshi (or someone else with a Japanese install),
can you confirm what the output from "vcbuild /?" is on a Japanese
machine?
--
Magnus Hagander
Me: http://www.hagander.net/
Work: http://www.redpill-linpro.com/
From | Date | Subject | |
---|---|---|---|
Next Message | Teodor Sigaev | 2010-10-15 15:39:40 | Re: knngist - 0.8 |
Previous Message | Tom Lane | 2010-10-15 14:54:57 | Re: string function - "format" function proposal |