search

sureshvv / reviewboard

Automatically exported from code.google.com/p/reviewboard
0 stars 0 forks source link

Script error "Unable to parse diff header: +++ " #63

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
This is the error output:
Unable to parse diff header: +++ 
c:\code\xxxx\yyyy.cs Wed May 23 19:30:19 2007

The culprit line of code is
            m = re.search(r'(\d\d\d\d-\d\d-\d\d \d\d:\d\d:\d\d)', dl[1])

Unfortunately i don't know why its failing on my machine only.

Original issue reported on code.google.com by gaur...@gmail.com on 24 May 2007 at 3:10

GoogleCodeExporter commented 9 years ago 
Can you e-mail me the output of: post-review --output-diff changenumber?

Original comment by chip...@gmail.com on 29 May 2007 at 6:47

GoogleCodeExporter commented 9 years ago

Original comment by chip...@gmail.com on 29 May 2007 at 6:50

GoogleCodeExporter commented 9 years ago 
Have you had this happen since?

Original comment by chip...@gmail.com on 5 Jun 2007 at 8:43

GoogleCodeExporter commented 9 years ago

Original comment by trowb...@gmail.com on 9 Jun 2007 at 2:21

GoogleCodeExporter commented 9 years ago 
diff.exe was getting picked up from my vim directory (.  I've fixed this by
reordering my path variable.  Not really your bug.  Have you specified the right
version of diff.exe to use?

C:\code\main>"c:\Program Files\Vim\vim63"\diff -v
diff - GNU diffutils version 2.7

C:\code\main>diff -v
diff (GNU diffutils) 2.8.7
Written by Paul Eggert, Mike Haertel, David Hayes,
Richard Stallman, and Len Tower.

Original comment by gaur...@gmail.com on 16 Jun 2007 at 1:56

GoogleCodeExporter commented 9 years ago 
We don't specify the version but anything modern *should* work.

This seems to be a pretty uncommon case, so I'm going to close this out for now.

Original comment by chip...@gmail.com on 3 Jul 2007 at 5:17

GoogleCodeExporter commented 9 years ago 
I just added a minimum version notice to the UserBasics wiki page.

Original comment by trowb...@gmail.com on 3 Jul 2007 at 5:20