Re: RBTools 0.7.4: rbt post --diff-only no longer works
Hi David, Ticket opened: https://hellosplat.com/s/beanbag/tickets/4368/ --Steve On Monday, March 14, 2016 at 2:34:51 PM UTC-7, David Trowbridge wrote: > > 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 > > 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 reviewboard...@googlegroups.com . >> 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 reviewboard+unsubscr...@googlegroups.com. For more options, visit https://groups.google.com/d/optout.
Re: RBTools 0.7.4: rbt post --diff-only no longer works
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 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 reviewboard+unsubscr...@googlegroups.com. > 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 reviewboard+unsubscr...@googlegroups.com. For more options, visit https://groups.google.com/d/optout.
RBTools 0.7.4: rbt post --diff-only no longer works
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 reviewboard+unsubscr...@googlegroups.com. For more options, visit https://groups.google.com/d/optout.