<div dir="ltr"><div class="gmail_default" style="font-family:'trebuchet ms',sans-serif">The issue with the P4LANGUAGE environment variable, as well as the -L switch is that I don't know the language string I need to set it to. Another thing is that Perforce says that it will change the text of the error messages, so it's not clear if the ouput I'm parsing will be affected or not. I guess we would need to set up a temporary P4 server and test all of this but, for now, we offer enough flexibility to let users set the language if they have a non-English Perforce.</div>
<div class="gmail_default" style="font-family:'trebuchet ms',sans-serif"><br></div><div class="gmail_default" style="font-family:'trebuchet ms',sans-serif">Pedro</div></div><div class="gmail_extra"><br><br>
<div class="gmail_quote">On Wed, Oct 2, 2013 at 1:19 PM, Rolf Eike Beer <span dir="ltr"><<a href="mailto:eike@sf-mail.de" target="_blank">eike@sf-mail.de</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Am Mittwoch, 2. Oktober 2013, 13:10:20 schrieb Pedro Navarro:<br>
<div class="im">> > >    - It requires an English version of Perforce (use the P4_OPTIONS<br>
> > >    variable, documented below to pass the -L switch to change the<br>
> > >    message<br>
> > >    language if you installation has a different one). It shouldn't be<br>
> > >    too<br>
> > ><br>
> > > hard to change the regular expressions used for parsing to not key on a<br>
> > > specific word but on character ranges. I'll look into that if this<br>
> ><br>
> > becomes<br>
> ><br>
> > > an issue.<br>
> ><br>
> > Maybe set P4_OPTIONS automatically to -Lenglish (or whatever) in case it<br>
> > is<br>
> > not set?<br>
><br>
> I tried that originally but had no effect. Also, on the perforce<br>
> documentation it says that "Specifies the language to use for error<br>
> messages from the Perforce service. In order for this flag to work, your<br>
> administrator must have loaded support for non-English messages in the<br>
> database", so I didn't feel comfortable adding a flag globably without<br>
> knowing it might even trigger an error (like invalid language or something<br>
> like that).<br>
><br>
> Perforce offers the P4LANGUAGE environment variable which can be used to<br>
> configure the client for updates, and we also have the P4_OPTIONS variable,<br>
> so I think for now it's a good solution/workaround until we have more<br>
> information.<br>
<br>
</div>Can you set P4LANGUANGE unconditionally then? Like it is done in<br>
FindSubversion.cmake to get proper output.<br>
<br>
And of course this needs testcases, and a host that has Perforce installed and<br>
drives those tests. ;)<br>
<span class="HOEnZb"><font color="#888888"><br>
Eike</font></span><br>--<br>
<br>
Powered by <a href="http://www.kitware.com" target="_blank">www.kitware.com</a><br>
<br>
Visit other Kitware open-source projects at <a href="http://www.kitware.com/opensource/opensource.html" target="_blank">http://www.kitware.com/opensource/opensource.html</a><br>
<br>
Please keep messages on-topic and check the CMake FAQ at: <a href="http://www.cmake.org/Wiki/CMake_FAQ" target="_blank">http://www.cmake.org/Wiki/CMake_FAQ</a><br>
<br>
Follow this link to subscribe/unsubscribe:<br>
<a href="http://public.kitware.com/cgi-bin/mailman/listinfo/cmake-developers" target="_blank">http://public.kitware.com/cgi-bin/mailman/listinfo/cmake-developers</a><br></blockquote></div><br></div>