Re: Foreign Visual Studio builds

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/

In response to

Responses

Browse pgsql-hackers by date

  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