This is the mail archive of the gdb-patches@sourceware.org mailing list for the GDB project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: Gerrit request


On 2019-10-15 1:12 p.m., Sergio Durigan Junior wrote:
> On Tuesday, October 15 2019, Tom Tromey wrote:
> 
>>>>>>> "Simon" == Simon Marchi <simon.marchi@polymtl.ca> writes:
>>
>> Tom> This would be a lot better if the email included more of the patch
>> Tom> context.
>>
>> Simon> It doesn't seem to be possible out of the box.  The template can only work
>> Simon> with the data passed by Gerrit.  For a line comment, we only get the single
>> Simon> line it refers to:
>>
>> Simon>   https://gerrit.googlesource.com/gerrit/+/refs/heads/master/java/com/google/gerrit/server/mail/send/CommentSender.java#328
>>
>> That's unfortunate.  For me this makes the patch reviews pretty
>> unreadable on the list.
>>
>> Maybe a gerrit feature request is in order.
> 
> I had found this limitation yesterday before Simon's reply, and I was
> looking into opening a bug report/feature request against gerrit, but
> they use Chromium's bugzilla, which requires a Google account to post
> things, so that's a no-no for me.
> 
> Maybe someone else can do that.

Well, I built Gerrit locally and was able to make a change to allow displaying a fixed
number of lines before the line the comment is attached to.  I'm just not sure I am
ready to go through the patch submission process, as it probably involves adding some
configuration option, documentation, testing, etc.  Just like when submitting a patch to
GDB :).

But I can at least open a feature request describing what I have prototyped.  I'll try to
do that later today.

Simon


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]