forgot to cc the list<br><br><div class="gmail_quote">---------- Forwarded message ----------<br>From: <b class="gmail_sendername">Francois Pichet</b> <span dir="ltr"><<a href="mailto:pichet2000@gmail.com">pichet2000@gmail.com</a>></span><br>
Date: Thu, Jan 6, 2011 at 4:13 AM<br>Subject: Re: [llvm-commits] [llvm] r122808 - /llvm/trunk/utils/lit/lit/TestRunner.py<br>To: "David A. Greene" <<a href="mailto:greened@obbligato.org">greened@obbligato.org</a>><br>
<br><br><br><br><div class="gmail_quote"><div><div></div><div class="h5">On Tue, Jan 4, 2011 at 10:20 AM, David A. Greene <span dir="ltr"><<a href="mailto:greened@obbligato.org" target="_blank">greened@obbligato.org</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div><div></div><div>Francois Pichet <<a href="mailto:pichet2000@gmail.com" target="_blank">pichet2000@gmail.com</a>> writes:<br>
<br>
> Author: fpichet<br>
> Date: Tue Jan  4 04:23:42 2011<br>
> New Revision: 122808<br>
><br>
> URL: <a href="http://llvm.org/viewvc/llvm-project?rev=122808&view=rev" target="_blank">http://llvm.org/viewvc/llvm-project?rev=122808&view=rev</a><br>
> Log:<br>
> Disable r122754 on Windows: was causing all lit tests to fail.<br>
><br>
> Modified:<br>
>     llvm/trunk/utils/lit/lit/TestRunner.py<br>
><br>
> Modified: llvm/trunk/utils/lit/lit/TestRunner.py<br>
> URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/utils/lit/lit/TestRunner.py?rev=122808&r1=122807&r2=122808&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/utils/lit/lit/TestRunner.py?rev=122808&r1=122807&r2=122808&view=diff</a><br>


> ==============================================================================<br>
> --- llvm/trunk/utils/lit/lit/TestRunner.py (original)<br>
> +++ llvm/trunk/utils/lit/lit/TestRunner.py Tue Jan  4 04:23:42 2011<br>
> @@ -451,8 +451,12 @@<br>
>      # expression pattern a with substitution b in line ln.<br>
>      def processLine(ln):<br>
>          # Apply substitutions<br>
> +             # FIXME: Investigate why re.sub doesn't work on Windows<br>
>          for a,b in substitutions:<br>
> -            ln = re.sub(a, b, ln)<br>
> +            if kIsWindows:<br>
> +                ln = ln.replace(a,b)<br>
> +            else:<br>
> +                ln = re.sub(a, b, ln)<br>
><br>
>          # Strip the trailing newline and any extra whitespace.<br>
>          return ln.strip()<br>
<br>
</div></div>This can't possibly work.  "a" is a regular expression.  A replace isn't<br>
going to match it.<br>
<br></blockquote><div><br></div></div></div><div>The problem on Windows is that  re.sub will replace \t with a tab character. I still don't understand why you need to do a re.sub here instead of a replace. Doing a replace work for me. Give me an example of  "a" as a regular expression?</div>

<div><br></div><div><br></div></div><br>
</div><br>