Hi, I am using RBTools 0.6.2, it is not able to differentiate binary changes. If I have the changes for keystore (.jks) file, it is not able to recognize the changes. Though there are changes, it says : ERROR: There don't seem to be any diffs! Am using command "rbt post changelist#" My friend is using the earlier version of RBTools and he is successfully able to post the changes. So this is a bug specific to this version. Please let me know. Thanks
I'm confused. Is .jks a text file that's being misinterpreted as binary? We've never been able to post changes to binary files.
Yes a jks file is binary. using an older version of rbt that supports post-review the review is still posted even when the only change is a binary file. It seems with the new version the review is not posted because it can't determine any diffs
I don't see how this is a bug. You want a review request with no diffs?
I *really* don't understand what you mean. We've never done diffs of binary files. What behavior are you seeing with the older version (by which I mean what shows up in the "View diff" page?)
When I do a review (using post-review) that contains only a single binary file the review is created and when looking at review board it says that one file has changed. When you go to view the diff for that file it states "This is a binary file. The content cannot be displayed." The main point is that when a review is created that includes a binary file it at least shows up as a file that has changed. Using the newest version when you try and post a review that only contains binary file it errors out with "ERROR: There don't seem to be any diffs!". Does this make sense?
OK, I think I understand. What version control system? Perforce?
This is something that we'll fix eventually, but it's pretty low priority.
- NeedInfo + Confirmed
- Priority-Medium + Priority-Low + Component-RBTools + Project-RBTools
- reviewboard + rbtools
rbt post -I [file]
Where is [file] go???
Where do [file] put in reviewboard server???
- Confirmed + PendingReview
Fixed in RBTools release-1.0.x (c09cb82). This will ship in RBTools 1.0.2. Thanks!
- PendingReview + Fixed