Yes, I am referring to the comments that preced the actual diff in the patch.
Reviewboard throws them away, which makes it a kind of severe limitation for
us to use in our workflow (which uses git).


On Tue, Aug 4, 2009 at 4:19 PM, Dan Savilonis<> wrote:
> Hi Joyjit,
> Are you talking about comments that precede the actual diff in the
> patch? This is a problem for git patches, and a bug 1229 is open for
> it. I looked into the issue a bit and I think it can be resolved by
> maintaining the extra info as part of the diff set. Right now, RB
> splits the diff into separate file diffs which are then concatenated
> together for downloading the full diff. I'm interested in this change,
> so I'll look into modifying the code when I have a chance.
> Dan
> On Jul 29, 6:11 pm, joyjit <> wrote:
>> When one submits a patch file (in unified diff format), reviewboard
>> seems to throw away the comments that are embedded in the patch file.
>> This is not good.
>> It would be nice to have reviewboard accept those comments and add it
>> to the "Description" field.
>> And it would also be nice for reviewboard to preserve the patch in its
>> entirety when one chooses "Download Diff" from the web gui..
>> Thanks
>> Joyjit
> >

You received this message because you are subscribed to the Google Groups 
"reviewboard" group.
To post to this group, send email to
To unsubscribe from this group, send email to
For more options, visit this group at

Reply via email to