Newsgroups: php.internals Path: news.php.net Xref: news.php.net php.internals:119539 Return-Path: Delivered-To: mailing list internals@lists.php.net Received: (qmail 30880 invoked from network); 13 Feb 2023 09:51:37 -0000 Received: from unknown (HELO php-smtp4.php.net) (45.112.84.5) by pb1.pair.com with SMTP; 13 Feb 2023 09:51:37 -0000 Received: from php-smtp4.php.net (localhost [127.0.0.1]) by php-smtp4.php.net (Postfix) with ESMTP id 9371A1804F2 for ; Mon, 13 Feb 2023 01:51:36 -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 01:51:35 -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 7606940EC0; Mon, 13 Feb 2023 10:51:34 +0100 (CET) Date: Mon, 13 Feb 2023 10:51:33 +0100 To: Dmitry Stogov Cc: 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 10:28, Dmitry Stogov wrote: > It's OK when commits are reverted. > You are working in a common repository, and if your commits become stoppers > for others they have to be reverted. > Some of my commits were reverted as well. That doesn't explain why you demanded to revert everything instead of applying my trivial single-line fix. > Having too many dependent commits and inability to revert a single one > became an additional trouble and drew more attention to things you are > doing... > I didn't care about a single header change, but I do care about 100 > dependent commits. I don't get it, what's your point here? The fact that these dependencies exist is a result of the unclean state of the PHP code base, something my work aims to improve. The number of dependencies after my PRs was lower than before, and that's good. The "inability to revert a single one" is not a problem that needed to be solved, because a revert was never necessary - there was a trivial fix (that did work, contrary to your assertion). > After all, this includes cleanup is really questionable, and the current > vote result shows that is not my sole opinion. > Personally, I think this work might be very welcome during PHP-7.0 > development together with other re-factoring(s). > Massive permutation changes in a minor release are not acceptable for me. > Maybe it makes sense to target them to PHP-9.0 The vote is not about WHEN this cleanup can be done - it's about WHETHER at all. Voting "yes" does not imply that this must be done for 8.3. The RFC suggests that it could be 8.3 or 9.0, but that decision is not part of the vote. Voting "no" means you never want this cleanup to happen, ever, not for 8.3 and not for 9.0. So if you're not really opposed to such a kind of cleanup in general, don't vote "no". Max