Hi Jay,

  I believe someone broke this because I was pretty sure mu implementation 
used "os.separator" in that place.
This is something which definitly must to be fixed.


W dniu piątek, 1 marca 2013 21:55:45 UTC+1 użytkownik Jay Lawrence napisał:
> My experimentation with ReviewBoard and Clearcase continues....
> In trying to do a post-checkin review, I used the --revision-range option.
> According to 
> http://www.reviewboard.org/docs/manual/1.7/users/tools/post-review/#posting-committed-code
> The syntax is:
> $ post-review 
> --revision-range="/vobs/proj/file.c@@/main/0:/vobs/proj/file.c@@/main/1"
> However, in the RBTools code base (python 2.6), in the 
> 'diff_between_revisions' method,  clearcase.py uses
>       revision_range = revision_range.split(';')
> to breakup the command line argument. So ';' is actually the separator.
> This is difficult on Linux because the ';' must be escaped.
> Should I file a bug report and/or patch on this issue?
> Jay

Want to help the Review Board project? Donate today at 
Happy user? Let us know at http://www.reviewboard.org/users/
To unsubscribe from this group, send email to 
For more options, visit this group at 
You received this message because you are subscribed to the Google Groups 
"reviewboard" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/groups/opt_out.

Reply via email to