Diffs don't contain contents for binary files, so most tooling is unable to do anything with them other than state that a file is binary.
We're evaluating ways to go provide more information, including contents, of binary files in diffs for display, but given the lack of binary support in diffs, we have to come up with our own solution to this. If you're not even seeing an entry for binary files, then something is likely excluding them from the diff building. I'd need to know what type of repository you're working with. For most, we call out to the repository command line tool's diff builder to generate these. Also worth mentioning, RBTools 0.7.5 is 7 years old, and much has changed since then. If in doubt, try upgrading that. Christian On Sat, Jul 30, 2022 at 7:29 AM Rajan M <rajan.m...@gmail.com> wrote: > Hi, > In my review, I have binary files and non-binary files in the Changeset. > > Basically, I don't see the binary files in my diffs. How to make the > binary file should go(check-in) and available in PerForce with Jenkins > build? (please note not the real SUBMIT). > > Highly appreciate your help here. > > I use RB Tools version 0.7.5 > > On Tuesday, May 3, 2016 at 12:12:05 AM UTC+5:30 totoro...@gmail.com wrote: > >> Hi Christian, >> >> Than you for the information, >> and that's the plan I'm doing right now. >> >> I created a BASH script wrapper on top of rbt command line tool >> to handle "rbt post" that contains binary changes, >> and tar all the changed binaries into a single tarball to attach to a >> review. >> >> Some side-notes: >> While developing the wrapper script, >> I found out there's no "rbt attach" update commands to update specific >> attachment. >> I see "Update" feature is available on Web UI. >> >> Another thing is SVN property changes handling (svn propset, proplist). >> "rbt diff / rbt patch" is able to correctly handling the changes in "SVN >> property". >> >> eg. (rbt diff) >> Index: x86_64/dd >> =================================================================== >> --- x86_64/dd (revision 7144) >> +++ x86_64/dd (working copy) >> >> Property changes on: x86_64/dd >> ___________________________________________________________________ >> Added: opensource >> ## -0,0 +1 ## >> +ON >> \ No newline at end of property >> >> And when I do "rbt patch", >> the corresponding SVN property changes applied correctly. >> >> But from the WebUI, >> I can't find any trace to the changed file "dd". >> Unlike binary file change, >> the webUI diff would still show: >> This is a binary file. The content cannot be displayed. >> >> I'd suggest it should display the same as "rbt diff". >> >> >> >> >> >> >> Rick >> >> >> >> >> On Friday, April 22, 2016 at 4:33:33 PM UTC-7, Christian Hammond wrote: >>> >>> Hi Rick, >>> >>> There isn't currently a standard for binary files in diffs, and >>> Subversion has no support whatsoever. You can attach binary files to review >>> requests as file attachments. >>> >>> Marking Ship It doesn't merge changes, so that still has to be done >>> locally. Presumably, you have those binary files in your working directory, >>> and can commit them along with the code changes. >>> >>> We have plans for more intelligently handling binary files, and are >>> aiming to provide that as part of Review Board 3.5 (3.0 is in development >>> right now). >>> >>> Christian >>> >>> -- >>> Christian Hammond >>> President/CEO of Beanbag <https://www.beanbaginc.com/> >>> Makers of Review Board <https://www.reviewboard.org/> >>> >>> On Fri, Apr 22, 2016 at 12:29 PM, Rick Liu <totoro...@gmail.com> wrote: >>> >>>> >>>> Environment: >>>> OS: Ubuntu 14.04 x64 >>>> Reviewboard: 2.5.4 >>>> Python 2.7.6 >>>> RBTools 0.7.5 >>>> >>>> >>>> We use "rbt post" to post a review contains binary file. >>>> The web front-end correctly identified the file as binary file in diff >>>> view. >>>> But: >>>> 1. >>>> I can't find any link to download the new binary file to be submitted. >>>> 2. >>>> When I "ship-it" to submit the review, >>>> it failed to be merged into SVN. >>>> >>>> I googled the binary support for reviewboard. >>>> Some said yes and some said no, >>>> so I'm confused here. >>>> >>>> Thanks, >>>> >>>> Rick >>>> >>>> -- >>>> Supercharge your Review Board with Power Pack: >>>> https://www.reviewboard.org/powerpack/ >>>> Want us to host Review Board for you? Check out RBCommons: >>>> https://rbcommons.com/ >>>> Happy user? Let us know! https://www.reviewboard.org/users/ >>>> --- >>>> 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...@googlegroups.com. >>>> For more options, visit https://groups.google.com/d/optout. >>>> >>> >>> -- > Supercharge your Review Board with Power Pack: > https://www.reviewboard.org/powerpack/ > Want us to host Review Board for you? Check out RBCommons: > https://rbcommons.com/ > Happy user? Let us know! https://www.reviewboard.org/users/ > --- > You received this message because you are subscribed to the Google Groups > "Review Board Community" group. > To unsubscribe from this group and stop receiving emails from it, send an > email to reviewboard+unsubscr...@googlegroups.com. > To view this discussion on the web visit > https://groups.google.com/d/msgid/reviewboard/1741ac4b-4fdd-4b54-a787-b0f62ee1984fn%40googlegroups.com > <https://groups.google.com/d/msgid/reviewboard/1741ac4b-4fdd-4b54-a787-b0f62ee1984fn%40googlegroups.com?utm_medium=email&utm_source=footer> > . > -- Christian Hammond President/CEO of Beanbag <https://www.beanbaginc.com/> Makers of Review Board <https://www.reviewboard.org/> -- Supercharge your Review Board with Power Pack: https://www.reviewboard.org/powerpack/ Want us to host Review Board for you? Check out RBCommons: https://rbcommons.com/ Happy user? Let us know! https://www.reviewboard.org/users/ --- You received this message because you are subscribed to the Google Groups "Review Board Community" group. To unsubscribe from this group and stop receiving emails from it, send an email to reviewboard+unsubscr...@googlegroups.com. To view this discussion on the web visit https://groups.google.com/d/msgid/reviewboard/CAE7Vndmrkk04HkPAZhoaVKYCUffLLRbsjga4V6jTo0gP9RwjNw%40mail.gmail.com.
