I don't think we've heard about this before. Mind filing a bug? -David
On Thu, Mar 10, 2016 at 9:15 AM Steve <[email protected]> wrote: > I've noticed that the --diff-only option to rbt post is no longer working > the way it used to with Perforce. It may be trying to be too clever. If I > create a review with this perforce change description: > > Description: > Bogus review request to test --diff-only option > > > Using > > $ rbt post CL > > I get a review request where both the title and description read: "Bogus > review request to test --diff-only option". That's good. Then I modify > the file and the change list description so the new change list description > reads: > > > Description: > Bogus review request to test --diff-only option > > I'm adding this comment for Rev 2. Using --diff-only, this text > should not appear in ReviewBoard > > > and do > > $ rbt post --diff-only CL > > When I look at the new review draft, the title remains the same (good), > but the description field now says: > > > "I'm adding this comment for Rev 2. Using --diff-only, this text should > not appear in ReviewBoard" > > which is not good. > > Even worse is this case: > > I edit the description field in Review Board to add aditional information > directly. It now reads: > > ''' > Bogus review request to test --diff-only option > > Someone reported a problem with 'rbt post --diff-only' I'm using this > bogus review request to test and see if this is broken. I expect this > comment to remain after an update using --diff-only. > ''' > > Now I run > > $ rbt post --diff-only CL > > And my description in Review Board has been overwritten with the > description in my perforce change list. > > I searched for '--diff-only' in the bug tracker and didn't find anything, > so I'm posting here to see if this is a known issue. > > Thanks > > --Steve > > -- > 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 [email protected]. > For more options, visit https://groups.google.com/d/optout. > -- -David -- 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 [email protected]. For more options, visit https://groups.google.com/d/optout.
