Call for an assistance pushing patch review forward

[David Blaikie via cfe-commits]

Beyond that it may be useful to look at who's been contributing to the
code you're trying to change and consider cc'ing them on the review.
(if they can't help, maybe they can point you to someone who can)

On Fri, May 27, 2022 at 2:23 PM stryku_t via cfe-commits
<cfe-commits at lists.llvm.org> wrote:
>
> Hi Paul,
>
> Thank you for the tips.
> Pinging is indeed mentioned in the docs, but I didn't want to piss everyone off, so after two pings I thought that I'm doing something wrong. From now on I'll ping once a week in such situations.
>
> Got review comments now, so we're back on track. Gonna implement changes over the weekend.
>
> Thanks,
> Mateusz Janek
>
> czw., 26 maj 2022 o 22:51 Robinson, Paul <paul.robinson at sony.com> napisał(a):
>>
>> Hi Mateusz,
>>
>>
>>
>> I wouldn’t worry too much about the failed build.  I took a peek and it looks like the failures are mostly in places very unrelated to your patch.  If your own testing shows no problems, it’s very likely you’re fine.
>>
>>
>>
>> Regarding lack of response, this is unfortunately more common than it should be.  Our recommended practice—and I’m surprised we don’t say anything on the website—is to add a “ping” comment to your review, maybe once a week.  This can “bump” it up in someone’s to-be-reviewed list; if nothing else, there’s another email to the list that will hopefully catch someone’s attention.
>>
>>
>>
>> Good luck,
>>
>> --paulr
>>
>>
>>
>> From: cfe-commits <cfe-commits-bounces at lists.llvm.org> On Behalf Of stryku_t via cfe-commits
>> Sent: Thursday, May 26, 2022 4:20 PM
>> To: cfe-commits at lists.llvm.org
>> Subject: Call for an assistance pushing patch review forward
>>
>>
>>
>> Hi,
>>
>> Some time ago I submitted a Clang patch for review: https://reviews.llvm.org/D123532
>>
>> It's been some time and I can't make progress with it.
>> I'm aware that there is failed build. But, as I mentioned in one of the comments, I can't reproduce it locally.
>> Tried to reach out to the people mentioned in the review, but I struggle getting answers.
>> At this point I'm not sure if I'm doing something wrong or people are just busy. Should tag different people? Different project? Am I expected to figure out failed builds on my own before someone will review the changes?
>>
>> Could someone from the community please assist me how for move forward with this patch? Also, I'd be grateful if there's someone who can hint me how to reproduce failed builds locally or using some CI server.
>>
>> Thanks in advance!
>>
>> Best regards,
>> Mateusz Janek
>
> _______________________________________________
> cfe-commits mailing list
> cfe-commits at lists.llvm.org
> https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits