From nobody Wed Mar 1 08:15:06 2023 X-Original-To: dev-commits-ports-all@mlmmj.nyi.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2610:1c1:1:606c::19:1]) by mlmmj.nyi.freebsd.org (Postfix) with ESMTP id 4PRRqd5SgMz3vZmf; Wed, 1 Mar 2023 08:15:09 +0000 (UTC) (envelope-from bapt@FreeBSD.org) Received: from smtp.freebsd.org (smtp.freebsd.org [96.47.72.83]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256 client-signature RSA-PSS (4096 bits) client-digest SHA256) (Client CN "smtp.freebsd.org", Issuer "R3" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id 4PRRqd4y9pz43b0; Wed, 1 Mar 2023 08:15:09 +0000 (UTC) (envelope-from bapt@FreeBSD.org) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=freebsd.org; s=dkim; t=1677658509; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: in-reply-to:in-reply-to:references:references; bh=KLJl1xcQkq9nxxXbCnjiPRvtZihju8Yr/ALwjTmUmP4=; b=dERMLOzoB8OVmvpBgFEr0knXsbbILRsSGb+LiwSjPVBavuxKS3oHD8ulb5zd3fbfsI4BQJ drlTFrXagcdIL9b1XcYwBzuuOcLXJc9NSSB4cm3cm2ZXlO5dbOAglKoBqVbteX19cMAd9o AbFZYiWv8M4vZtJkynbrMQZFpq7wPOltTS6Omgmjv0hThi8hW7Xmz0ttDO47letTEeOTkz SmyBPLCj8meCSpe6TImuaS6tdq2fUYtyQoDxilgGSWf70IxWuTt5tfPw786nMGewkbvJkj Uz7V+sX+AHdYUPeSGflz5fco3bhEcZBd32za+iWKDDSG6WZKPMSuu+6mbfAiXA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=freebsd.org; s=dkim; t=1677658509; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: in-reply-to:in-reply-to:references:references; bh=KLJl1xcQkq9nxxXbCnjiPRvtZihju8Yr/ALwjTmUmP4=; b=QhgqIAJ9/OkFpYgzppXfXCfrDWNHgJ2UTFD6h0v39rMBqrqy9YkW8N4kt27VbnhMAe0I7Z Am9UmkAo41rwZv7vELIFHQx8svMqPTdvufdQIHxyKHhLKveHfB1A5A5x93hVme1hDMUrCD 5qUDh6mWMcMZ8q4RDLxkEKI0oMtPbBedZg0ctbTJIZwizcE/bf2RFJJ4gx6Xrr+IJKUB+/ Kw2RJEyOicBTUDbdgUWPAW+I7Qfjg2I7vHUYEwsmiYQNchwgv6k64REZPBCwUbE/6ppafq SX48mMVJKK0FggNkTDyWhV0KWDUR1rtStcunnhF/Oc5OYgZwQdgEucGeN8r9Kw== ARC-Authentication-Results: i=1; mx1.freebsd.org; none ARC-Seal: i=1; s=dkim; d=freebsd.org; t=1677658509; a=rsa-sha256; cv=none; b=yQkT0uTN111ba0D4okhft3wWspdd9IH0RReZOsIhywhDOCC+LslU3s6uwrvPZmwOpsdiB8 z4RPh1MHsllbUEZUt8NFbzCbEi2ahqI/mHpQ1lAIGGACgcrfx4S8po9hj3napqQaWeMzeG RvVG9JmiNIkjcIQ7rw9n0mCvKpoIfp+zchk0D1wS9nTOWAeeOBZkH0J+yBZnzPsYzuXDGB /H/Pvnk9JUAvud8KvUUDCus9Z0TI8OdwKGMe8LnEMFJVeShLIgC6AJGuaEc5uYcyok+Dbc QTf+gzvc8P5wC3xaJe8q5iqWS993PcfU7L53sDI+BEoJxHlVYOzsc9Vuz5nSEA== Received: from aniel.nours.eu (nours.eu [176.31.115.77]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256) (Client did not present a certificate) (Authenticated sender: bapt) by smtp.freebsd.org (Postfix) with ESMTPSA id 4PRRqd3Jptzvc6; Wed, 1 Mar 2023 08:15:09 +0000 (UTC) (envelope-from bapt@FreeBSD.org) Received: by aniel.nours.eu (Postfix, from userid 1001) id A091C1B9D2B; Wed, 1 Mar 2023 09:15:06 +0100 (CET) Date: Wed, 1 Mar 2023 09:15:06 +0100 From: Baptiste Daroussin To: Matthias Andree Cc: Charlie Li , "Tobias C. Berner" , ports-committers@FreeBSD.org, dev-commits-ports-all@FreeBSD.org, dev-commits-ports-main@FreeBSD.org Subject: Re: git: a9d9d3a42723 - main - lang/python: Revert "add bytecode trigger" Message-ID: <20230301081506.yjkbquftisgdb4f5@aniel.nours.eu> References: <202302272105.31RL593a014619@gitrepo.freebsd.org> <20230228075455.b7otamo7ezahtqxi@aniel.nours.eu> List-Id: Commit messages for all branches of the ports repository List-Archive: https://lists.freebsd.org/archives/dev-commits-ports-all List-Help: List-Post: List-Subscribe: List-Unsubscribe: Sender: owner-dev-commits-ports-all@freebsd.org X-BeenThere: dev-commits-ports-all@freebsd.org MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: X-ThisMailContainsUnwantedMimeParts: N On Tue, Feb 28, 2023 at 08:32:22PM +0100, Matthias Andree wrote: > Am 28.02.23 um 08:54 schrieb Baptiste Daroussin: > > On Mon, Feb 27, 2023 at 09:05:09PM +0000, Matthias Andree wrote: > > > The branch main has been updated by mandree: > > > > > > URL: https://cgit.FreeBSD.org/ports/commit/?id=a9d9d3a4272303bf226b9deb55c42303e4fcebdc > > > > > > commit a9d9d3a4272303bf226b9deb55c42303e4fcebdc > > > Author: Matthias Andree > > > AuthorDate: 2023-02-27 21:01:32 +0000 > > > Commit: Matthias Andree > > > CommitDate: 2023-02-27 21:04:52 +0000 > > > > > > lang/python: Revert "add bytecode trigger" > > > This reverts commit c17ddfbf66e2801ec620d49979aca3d7077d7002. > > > This causes breakage on several ports, and the next iteration > > > requires a full exp-run. See: > > > Differential Revision: https://reviews.freebsd.org/D34739 > > > > This is not how this is supposed to be done! while you have a point about the > > issue and the full exp-run, the issue is easily fixable, you should have let > > time to the committer to revert by himself, by asking him to revert instead of > > abruply reverting! > > > > Best regards, > > Bapt > > > Am 28.02.23 um 06:44 schrieb Tobias C. Berner: > > Moin moin > > > > So you reverted the first major change to the python framework in ages > > due to a leaf-port breakin non-fatally? > > This looks a little bit like overstepping the bounds of good taste a bit > to me. > > Baptiste, Tobias, > > First, let's get the timeline straight: > > Feb 15, Charlie commits. > Feb 18, Antoine claims the bug via mailing list. > Feb 18, Mathieu asks why this was committed without prior -exp run, > via reviews.freebsd.org > - silence - > Feb 27, Matthias (yours truly) reverts that breaking change > Feb 27 - 28, people pick at Matthias with emotional and non-technical > reasons which are all besides the factual point. > > There was NO acknowledgment of the bug, no hint that anyone was working, and > we had broken sphinx and breathe ports, so we needed to fix things. > > Then, half of the messages on IRC, mailing list, and the reviews were also > non-technical, attacking me instead of the bug in Charlie's stuff and > revealing who was getting touchy. > > All in all, can we now please stop confusing cause and effect, and can those > who have been touchy now please grow up? I am perfectly aware of the timeline, and the reaction from Antoine and Mathieu. They raised the issue and yes we would have expected an ack (at the very least) from Charlie! However, both seemed to consider this was not worth a revert otherwise they would have asked for the revert. "Hey, can you quickly fix the following issue please, or revert until you can come with a proper fix" This is it should be asked. On Charlie's side, yes he should have a been reactive, yes he should have acknowledged the issue, his interractions should have been way better and he needs to learn about. I didn't react on it because I felt this was being dealt with somehow by others. On the other side I saw your revert without prior benevolent message (like the example above), such action is and as far as a I know has always been considered a toxic behaviour not acceptable in the project. (yes it did happen in the past and iirc always with the same reaction from portmgr, either public or private depending on the cases). It can only happen in rare cases in particular breakage that deeply impact the package building for example. This is why between a technical issue which clearly deservers lot of improvement in the community interraction from Charlie and an aggressive revert, I chose to react on the aggressive revert. Best regards Bapt