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
> The syntax is:
> $ post-review
> 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?
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
To unsubscribe from this group and stop receiving emails from it, send an email
For more options, visit https://groups.google.com/groups/opt_out.