Re: [DejaGnu] Fwd: PATCH to dejagnu/lib/remote.exp:check_for_board_status

[Jason Merrill]

>>>>> "Rob" == Rob Savoye <address@hidden> writes:

> On Sun, Apr 14, 2002 at 11:06:55PM +0100, Jason Merrill wrote:
>> I don't have a savannah account, though I suppose I ought to get one set
>> up; the GCC CVS repository is on gcc.gnu.org.  Would you mind checking the
>> patch in for me?

>   I can do that as soon as I test it against another patch to the same line...
> That line was recently modified to be:
> regsub -all "(^|\[\r\n\]|\r\n)\\*\\*\\* EXIT code 
> \[^\r\n\]*(\[\r\n\]\[\r\n\]?|$)" $output "" output;

> And I need to see if your patch needs to be merged into the same line, or 
> which
> patch is the correct one. I think your patch does the same basic thing.

Looks like; mine is just a shorter way of writing the same thing.  Though
the pattern at the beginning of the line you cite will handle "\r", and the
one at the end will also handle "\n\r"; mine only strips "\n" and "\r\n".
Whatever you think best.

Jason