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 
http://www.reviewboard.org/donate/
Happy user? Let us know at http://www.reviewboard.org/users/
-~----------~----~----~----~------~----~------~--~---
To unsubscribe from this group, send email to 
reviewboard+unsubscr...@googlegroups.com
For more options, visit this group at 
http://groups.google.com/group/reviewboard?hl=en
--- 
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