<div dir="ltr"><div class="gmail_default" style="font-family:'trebuchet ms',sans-serif">Filed <a href="http://public.kitware.com/Bug/view.php?id=14524" style="font-family:arial">http://public.kitware.com/Bug/view.php?id=14524</a> to monitor this issue. I already have a patch for this. Thanks Igor!</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 30, 2013 at 10:50 AM, Pedro Navarro <span dir="ltr"><<a href="mailto:pnavarro@netflix.com" target="_blank">pnavarro@netflix.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div dir="ltr"><div class="gmail_default" style="font-family:trebuchet ms,sans-serif">Yes, I would prefer the first variant also. I assumed the variable would already come quoted (the second variant) because that's what I thought $CDASH_GIT_COMMAND was using, but looking at the git code I see that the comand is quoted when it's run. I'll add something similar.</div>
<span class="HOEnZb"><font color="#888888">
<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></font></span></div><div class="HOEnZb"><div class="h5"><div class="gmail_extra">
<br><br><div class="gmail_quote">
On Wed, Oct 30, 2013 at 3:17 AM, Igor Murzov <span dir="ltr"><<a href="mailto:e-mail@date.by" target="_blank">e-mail@date.by</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<div>> > >   $p4command = $CDASH_P4_COMMAND." ".$root." -L en";<br>
> ><br>
> > >  $raw_output = `$p4command changes $branch@"$fromtime","$totime"<br>
> > > 2>&1`;<br>
> ><br>
> > Can it happen that $root or $CDASH_P4_COMMAND contain spaces or some<br>
> > other special characters? I think some quoting might be needed here.<br>
> ><br>
> ><br>
> ><br>
> I guess it could happen, yes, but wouldn't these values come already<br>
> quoted from the configuration settings?<br>
<br>
</div>I'm not sure what do you mean by this. Do you want users to quote every<br>
special character?<br>
For instance, if $CDASH_P4_COMMAND is set like:<br>
<br>
  $CDASH_P4_COMMAND = '/prefix/with space/p4';<br>
<br>
then your code will fail. But it would work if $CDASH_P4_COMMAND is set<br>
like this:<br>
<br>
  $CDASH_P4_COMMAND = '/prefix/with\ space/p4';<br>
<br>
If I used perforce, I would prefer the first variant :)<br>
<span><font color="#888888"><br>
<br>
-- Igor<br>
</font></span></blockquote></div><br></div>
</div></div></blockquote></div><br></div>