Newsgroups: php.internals Path: news.php.net Xref: news.php.net php.internals:119536 Return-Path: Delivered-To: mailing list internals@lists.php.net Received: (qmail 22502 invoked from network); 13 Feb 2023 08:35:24 -0000 Received: from unknown (HELO php-smtp4.php.net) (45.112.84.5) by pb1.pair.com with SMTP; 13 Feb 2023 08:35:24 -0000 Received: from php-smtp4.php.net (localhost [127.0.0.1]) by php-smtp4.php.net (Postfix) with ESMTP id E8A7F18050B for ; Mon, 13 Feb 2023 00:35:22 -0800 (PST) X-Spam-Checker-Version: SpamAssassin 3.4.2 (2018-09-13) on php-smtp4.php.net X-Spam-Level: X-Spam-Status: No, score=-1.9 required=5.0 tests=BAYES_00,SPF_HELO_NONE, SPF_PASS,T_SCC_BODY_TEXT_LINE autolearn=no autolearn_force=no version=3.4.2 X-Spam-ASN: AS24940 138.201.0.0/16 X-Spam-Virus: No X-Envelope-From: Received: from swift.blarg.de (swift.blarg.de [138.201.185.127]) by php-smtp4.php.net (Postfix) with ESMTP for ; Mon, 13 Feb 2023 00:35:22 -0800 (PST) Received: from swift.blarg.de (swift.blarg.de [IPv6:2a01:4f8:c17:52a8::2]) (Authenticated sender: max) by swift.blarg.de (Postfix) with ESMTPSA id 76A0B40E38; Mon, 13 Feb 2023 09:35:20 +0100 (CET) Date: Mon, 13 Feb 2023 09:35:19 +0100 To: "G. P. B." Cc: Matthew Weier O'Phinney , internals@lists.php.net Message-ID: References: <1cb213ea-ff7d-c4b2-5345-fadbc5953c94@bastelstu.be> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: Subject: Re: [PHP-DEV] [VOTE] include cleanup From: max+php@blarg.de (Max Kellermann) On 2023/02/13 01:58, "G. P. B." wrote: > We have had completely broken builds for longer days due to some other > random changes, and we didn't revert them but fixed them as a follow-up. > We still, for over 6 months now, have a "broken" ASAN build due to phpdbg > messing up the analyser and crashing the test runner on 8.2 and master, > something that multiple core devs, me included, need to work around by > monkey patching the run-test.php file. I had a feeling there are double standards at play. The way my work was dealt with is unprecedented! In the git history, I could not find any other set of PRs that was reverted completely just for a minor one-line issue. Stuff breaks all the time, and every breakage is, of course, a mistake that should have been handled with more care, and something to learn from. Sometimes, a revert is the right solution, but in my case, the (demand of a) revert was unreasonable and hasty. > As a final note, if the complaint had been made by anyone else other than > Dmitry, I doubt these changes would have been reverted, and can we please > stop pretending otherwise. I forget one include, break an exotic build in master branch, Dmitry demands complete revert of 4 PRs / 61 commits (60 of which are unrelated to the breakage). Dmitry breaks the whole build for everybody (including the CI) and introduces a crash bug, merged through all three branches (https://github.com/php/php-src/commit/a21195650e53), no revert. (Don't misunderstand, everybody makes mistakes, I just point out the obvious double standard.) Then everybody is raving about my include comments and forward declarations, yet the PHP code base has many of these. Turns out those who complained the loudest have authored some of these in the past. Max