Re: [FFmpeg-devel] When to add 'Reviewed-by: xxx' in commit messages

["softworkz ." <softworkz-at-hotmail.com@ffmpeg.org>]

> -----Original Message-----
> From: ffmpeg-devel <ffmpeg-devel-bounces@ffmpeg.org> On Behalf Of softworkz .
> Sent: Samstag, 3. Mai 2025 11:32
> To: FFmpeg development discussions and patches <ffmpeg-devel@ffmpeg.org>
> Subject: [FFmpeg-devel] When to add 'Reviewed-by: xxx' in commit messages
> 
> Hello everybody,
> 
> I have an organizational question that I cannot quite figure out how to do it
> right:
> 
> Whether and when to add 'Reviewed-by:  xxx' to a commit message?
> 
> Specific example: The “execution graph printing” patchset:
> 
> Andreas had reviewed the patchset initially. I addressed the mentioned
> issues. Is that the point to add 'Reviewed-by:’ already? And what about
> the commits without comments? Should I assume them to be reviewed
> as well? And how about later changes, do they invalidate it?
> 
> From Stefano, I believe that he didn’t review the last two commits (due
> to being outside of the text formatting scope, I suppose). I added him
> as “reviewed by” only to those where he said “should be ok” or similar.
> Andreas didn’t say anything like that, yet I’m sure that he has carefully
> looked over everything.
> 
> It might not be a big thing after all, but I don't want to be unjust to
> anybody and I'm unsure how to handle this, because I can imagine that
> someone might either say "Hey, why is he mentioning me, that's not the
> version that I have reviewed!" but also "Why doesn't he mention me, I've
> reviewed the whole thing in detail?".
> That's the circle by which I got trapped at the moment. 😉
> 
> I'd be glad if somebody could provide me some guidance in this regard.
> 
> Thanks a lot,
> sw
> _______________________________________________

Hello everybody,
 
recently, I've been criticized for this message on IRC as follows:

"softworkz: last month you even had to publicly ask on the mailing list 
about a policy which pretty much everyone understood"


So - what does this mean to express? That it would be an indication of 
incompetence when somebody is asking others for advice on something one
doesn't know?

What _I_ consider as a sign of incompetence is just the opposite: 
Pretending to know everything without knowing, and being afraid of 
asking questions as that might shed a bad light on oneself.

When I have a question on something that I do not know, then I ask that
question, no matter what anybody might think about it.
Because - at the end of the day - I'll be the one who knows the answer
and all those who were just pretending to know it - won't 
(unless public like here).

And anyway, where are we - that asking for advice would be a bad thing?

---

In this regard, I still don't know the answer. It's not about the basic
principle of procedure but goes into some quite specific whereabouts
where I'm not sure how exactly it is supposed to be done.


Thanks a lot for any advice or guidance.
sw

_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".