Yes, split_lines=True is what I needed to add to get it to work. I'm
less sure about the other changes. They seem like they might be a
problem with your version of python on Windows. I'll add comments to
the review.


On Jul 10, 11:22 am, Bartlomiej Celary <>
> Hi Dan,
> > We did have to modify one line of code in post-review to get it to run
> Could you check if those changes are not similar to mine
> ( know I had difficulties
> submitting a diff for a clearcase repo and those changes fixed it.
> I think that we should also make a hard-coded part configurable.
> Thanks,
> Bartek
You received this message because you are subscribed to the Google Groups 
"reviewboard" group.
To post to this group, send email to
To unsubscribe from this group, send email to
For more options, visit this group at

Reply via email to