From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from ffbox0-bg.ffmpeg.org (ffbox0-bg.ffmpeg.org [79.124.17.100]) by master.gitmailbox.com (Postfix) with ESMTPS id 8639A4CFAE for ; Tue, 12 Aug 2025 17:21:04 +0000 (UTC) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.ffmpeg.org (Postfix) with ESMTP id 383BE68CBB5; Tue, 12 Aug 2025 20:21:00 +0300 (EEST) Received: from nef.ens.fr (nef2.ens.fr [129.199.96.40]) by ffbox0-bg.ffmpeg.org (Postfix) with ESMTPS id 98E21687D20 for ; Tue, 12 Aug 2025 20:20:53 +0300 (EEST) X-ENS-nef-client: 129.199.129.80 ( name = phare.normalesup.org ) Received: from phare.normalesup.org (phare.normalesup.org [129.199.129.80]) by nef.ens.fr (8.14.4/1.01.28121999) with ESMTP id 57CHKq7V006339 for ; Tue, 12 Aug 2025 19:20:53 +0200 Received: by phare.normalesup.org (Postfix, from userid 1001) id E38272EFE3; Tue, 12 Aug 2025 19:20:52 +0200 (CEST) Date: Tue, 12 Aug 2025 19:20:52 +0200 From: Nicolas George To: FFmpeg development discussions and patches Message-ID: References: <20250812090205.A5F4D68B0ED@ffbox0-bg.ffmpeg.org> <20250812113408.GB24045@haasn.xyz> <1bfcff88-d626-475b-a506-9d60feb5177e@gyani.pro> <20250812115108.GB29347@haasn.xyz> <20250812120446.GB32174@haasn.xyz> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20250812120446.GB32174@haasn.xyz> X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.4.3 (nef.ens.fr [129.199.96.32]); Tue, 12 Aug 2025 19:20:53 +0200 (CEST) Subject: Re: [FFmpeg-devel] Pushing without approval avfilter/vf_vignette: use AVFilterContext for logging X-BeenThere: ffmpeg-devel@ffmpeg.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: FFmpeg development discussions and patches List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Reply-To: FFmpeg development discussions and patches Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Errors-To: ffmpeg-devel-bounces@ffmpeg.org Sender: "ffmpeg-devel" Archived-At: List-Archive: List-Post: Niklas Haas (HE12025-08-12): > As far as my understanding goes, only people with write access to the > repository status have the ability to merge or approve PRs. This is obviously wrong: write access to the repository is all-or-nothing, expertise in parts of the code is not. This is why we have maintainers. With the recent clarification of who approved the series, we now know that you had reliable but non-expert non-maitainer approval, which should have let you wait a little less but not push immediately. > If you do not > feel somebody can be trusted with write access, that is a separate issue > irrelevant to this discussion. Well, Michael has been giving these away like hotcakes recently, I can think of two cases in recent months, but it is entirely unrelated to this discussion. > That aside, the person who reviewed the PR also has about 100x more activity > than you So what? > Where in "MAINTAINERS" is this written? An unfortunate oversight that will soon be rectified. But I believe it should be common knowledge for somebody who has been in the project for as long as you have, or for somebody who exchanged mails with me recently on the very subject of libavfilter. > Having a clearly defined process is a human solution to a human problem. A bot is not a human solution. -- Nicolas George _______________________________________________ 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".