From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.2 (2018-09-13) on dcvr.yhbt.net X-Spam-Level: X-Spam-ASN: AS3215 2.6.0.0/16 X-Spam-Status: No, score=-3.9 required=3.0 tests=AWL,BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS, MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS,T_SCC_BODY_TEXT_LINE shortcircuit=no autolearn=ham autolearn_force=no version=3.4.2 Received: from out1.vger.email (out1.vger.email [IPv6:2620:137:e000::1:20]) by dcvr.yhbt.net (Postfix) with ESMTP id C952B1F8C4 for ; Fri, 25 Mar 2022 19:27:48 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229569AbiCYT06 (ORCPT ); Fri, 25 Mar 2022 15:26:58 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:58148 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229708AbiCYT0o (ORCPT ); Fri, 25 Mar 2022 15:26:44 -0400 Received: from pb-smtp2.pobox.com (pb-smtp2.pobox.com [64.147.108.71]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 891211E113E for ; Fri, 25 Mar 2022 11:59:41 -0700 (PDT) Received: from pb-smtp2.pobox.com (unknown [127.0.0.1]) by pb-smtp2.pobox.com (Postfix) with ESMTP id E30A910F3DD; Fri, 25 Mar 2022 13:25:28 -0400 (EDT) (envelope-from junio@pobox.com) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed; d=pobox.com; h=from:to:cc :subject:references:date:in-reply-to:message-id:mime-version :content-type; s=sasl; bh=ZGLP8png6nI9nxXVu9DB90aZULd9Y1m16dXQjm x0eOY=; b=kSbQ0eu7/a44/c7JaKfJQ4H4ShWcKNhxHMrahDKoJspl3CHObbCpso MJzzVHiP6IDVn0qGYRQz/1NtJQ9N8u1XM/DSncou4H6GiP8lTFr7gJRrYsOYxJNY pjbja8JJII//2qa/bwN5LlfRyqhecfHOq8onsNfgel/c6CPiWphAo= Received: from pb-smtp2.nyi.icgroup.com (unknown [127.0.0.1]) by pb-smtp2.pobox.com (Postfix) with ESMTP id D954510F3DC; Fri, 25 Mar 2022 13:25:28 -0400 (EDT) (envelope-from junio@pobox.com) Received: from pobox.com (unknown [35.227.145.180]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by pb-smtp2.pobox.com (Postfix) with ESMTPSA id 43C3510F3DB; Fri, 25 Mar 2022 13:25:28 -0400 (EDT) (envelope-from junio@pobox.com) From: Junio C Hamano To: Derrick Stolee Cc: Derrick Stolee via GitGitGadget , git@vger.kernel.org, chakrabortyabhradeep79@gmail.com, Taylor Blau Subject: Re: [PATCH v2 2/2] test-lib-functions: fix test_subcommand_inexact References: Date: Fri, 25 Mar 2022 10:25:27 -0700 In-Reply-To: (Derrick Stolee's message of "Fri, 25 Mar 2022 10:03:33 -0400") Message-ID: User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/27.2 (gnu/linux) MIME-Version: 1.0 Content-Type: text/plain X-Pobox-Relay-ID: 9082D1A2-AC60-11EC-975F-CB998F0A682E-77302942!pb-smtp2.pobox.com Precedence: bulk List-ID: X-Mailing-List: git@vger.kernel.org Derrick Stolee writes: > On 3/24/2022 4:48 PM, Junio C Hamano wrote: >> "Derrick Stolee via GitGitGadget" writes: > >>> All existing tests continue to pass with this change. There was one >>> instance from t7700-repack.sh that was taking advantage of this >>> flexibility, but it was removed in the previous change. >> >> Of course all existing tests continue to pass, as we no longer have >> any user of test_subcommand_inexact after the previous step ;-). > > Yeah, I definitely should have checked to see if there were other > uses of this. I thought there was, but I was mistaken. > >> Among >> >> (1) doing nothing, >> (2) removing, and >> (3) clarifying the implementation, >> >> my preference would be 2 > 1 > 3. If we add > > I agree that (2) is the best option here. > >> (4) clarify the implementation and document what kind of inexactness we >> tolerate with an updated comment" >> >> to the mix, that would come before all 3 others, though. > > Is there value in fixing the implementation and adding this comment > if we are to just delete the helper? I suppose that we could prevent > a future contribution from reintroducing the broken implementation. That is a good thoguth to take into account. > My current feeling is that we should just delete this and refer > to that deletion if anyone considers needing something like it. I am very much in favor of deleting it. Thanks.