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.
Thanks! 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 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.
