[FFmpeg-devel] [PATCH 7/9] MAINTAINERS: Update the entries for the release maintainer for FFmpeg

[Anton Khirnov]

Quoting Michael Niedermayer (2024-06-18 01:48:25)
> On Mon, Jun 17, 2024 at 09:07:23AM +0200, Anton Khirnov wrote:
> > Quoting Michael Niedermayer (2024-06-17 01:08:29)
> > > Ive been told that someone at the BCN video tech meetup claimed to be the
> > > "release maintainer for FFmpeg".
> > > 
> > > If you have any doubt who maintains releases, just do something like the following and look at the output:
> > > VER=5.1
> > > echo commiters ; git shortlog  --group=committer -s  n$VER..release/$VER -n ;\
> > > echo authors   ; git shortlog                    -s  n$VER..release/$VER -n
> > 
> > Passive aggressive gossip does not belong in a commit message.
> 
> we generally explain in a commit message the "why" and "what" and so on.
> There was no intention of any aggression.

The commit message is indirectly accusing an unindentified person of
trying to usurp your position as a release maintainer. The accusation is
aggressive, and the fact that it is indirect makes it passive
aggressive.

Not providing any specifics beyond "someone said that someone said"
makes it gossip.

Each of those factors in isolation IMO makes this text inappropriate for
a commit message.

> But how would you word it instead ?

I would not word it at all, as there is no evidence that this needs
clarifying. Gossip is not evidence.

> The example git commands simply show the authors and commiters on the release branch
> since the first release on it. I think its reasonable to provide these as reference

I don't. Nobody is disputing your role as a release maintainer.

-- 
Anton Khirnov