From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on dcvr.yhbt.net X-Spam-Level: X-Spam-ASN: AS54825 139.178.88.0/22 X-Spam-Status: No, score=-3.7 required=3.0 tests=AWL,BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS, MAILING_LIST_MULTI,RCVD_IN_DNSWL_HI,SPF_HELO_NONE,SPF_PASS shortcircuit=no autolearn=ham autolearn_force=no version=3.4.6 Received: from sv.mirrors.kernel.org (sv.mirrors.kernel.org [139.178.88.99]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits)) (No client certificate requested) by dcvr.yhbt.net (Postfix) with ESMTPS id 830BD1F44D for ; Wed, 24 Apr 2024 14:59:35 +0000 (UTC) Authentication-Results: dcvr.yhbt.net; dkim=pass (1024-bit key; unprotected) header.d=pobox.com header.i=@pobox.com header.a=rsa-sha256 header.s=sasl header.b=juD2uuvF; dkim-atps=neutral Received: from smtp.subspace.kernel.org (wormhole.subspace.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by sv.mirrors.kernel.org (Postfix) with ESMTPS id 6B7AF28DAFF for ; Wed, 24 Apr 2024 14:59:35 +0000 (UTC) Received: from localhost.localdomain (localhost.localdomain [127.0.0.1]) by smtp.subspace.kernel.org (Postfix) with ESMTP id 56F1B15EFDB; Wed, 24 Apr 2024 14:59:25 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=pobox.com header.i=@pobox.com header.b="juD2uuvF" Received: from pb-smtp21.pobox.com (pb-smtp21.pobox.com [173.228.157.53]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 9BE5315ECFE for ; Wed, 24 Apr 2024 14:59:22 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=173.228.157.53 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1713970763; cv=none; b=H9aXDfNOmqYAH519IoqBLiPknJ0d+kQhp+oBCZ7SFKBYWuKkoHdoO12lfQhc6EOK+pOisFHHvDcTanj2s5WXe2nqkZUDgNpEU8+iKD8HgO72650keDfX8HGa4dq0ntV1pfFjUJ3Qy6RZRlyik/ZywX4orXgyljup3YtfGtaYyS0= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1713970763; c=relaxed/simple; bh=yrOL96WQB45BsOIOMe1pCQbL636wLaxuVIMO2PXPzWQ=; h=From:To:Cc:Subject:In-Reply-To:References:Date:Message-ID: MIME-Version:Content-Type; b=maqnV5AXel2rX7ewF82GpMcr9dzdAFt+Daj7TzIG13n88NcX+ih6kc3QmG9thi6SVo3rkRtd8mUELPkArk7J8OihePhQzijx06O67wTxslq0agJoyfol/b6oHesTxklLY+bhpoe91p/fm5ZUHjUXXS5OGQVpnMSahRx3a1wedcI= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=pobox.com; spf=pass smtp.mailfrom=pobox.com; dkim=pass (1024-bit key) header.d=pobox.com header.i=@pobox.com header.b=juD2uuvF; arc=none smtp.client-ip=173.228.157.53 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=pobox.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=pobox.com Received: from pb-smtp21.pobox.com (unknown [127.0.0.1]) by pb-smtp21.pobox.com (Postfix) with ESMTP id 2A8EE33ED5; Wed, 24 Apr 2024 10:59:22 -0400 (EDT) (envelope-from junio@pobox.com) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed; d=pobox.com; h=from:to:cc :subject:in-reply-to:references:date:message-id:mime-version :content-type; s=sasl; bh=yrOL96WQB45BsOIOMe1pCQbL636wLaxuVIMO2P XPzWQ=; b=juD2uuvF4VcgoR2vSITtSimdfHKpaodYty/Ly47e/85mC9dqYBdIMS +T4cVOum9BDFgwlCmUo0Q9cKziUVcqw78PXGj8DcIDjwMrMZ276608J8oPAcPZYA vrEMLgM96WBS+wNQeKoBCosJdQRJMPJ/N9JyVuwz4KKtHWlLGepsM= Received: from pb-smtp21.sea.icgroup.com (unknown [127.0.0.1]) by pb-smtp21.pobox.com (Postfix) with ESMTP id 227DB33ED4; Wed, 24 Apr 2024 10:59:22 -0400 (EDT) (envelope-from junio@pobox.com) Received: from pobox.com (unknown [34.125.120.109]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by pb-smtp21.pobox.com (Postfix) with ESMTPSA id 6355133ED3; Wed, 24 Apr 2024 10:59:18 -0400 (EDT) (envelope-from junio@pobox.com) From: Junio C Hamano To: Patrick Steinhardt Cc: phillip.wood@dunelm.org.uk, James Liu , git@vger.kernel.org Subject: Re: [PATCH 0/2] advice: add "all" option to disable all hints In-Reply-To: (Patrick Steinhardt's message of "Wed, 24 Apr 2024 16:07:49 +0200") References: <20240424035857.84583-1-james@jamesliu.io> <7d1d2892-d486-4e91-b100-e44c1d76893c@gmail.com> Date: Wed, 24 Apr 2024 07:59:16 -0700 Message-ID: User-Agent: Gnus/5.13 (Gnus v5.13) Precedence: bulk X-Mailing-List: git@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain X-Pobox-Relay-ID: 399E236C-024B-11EF-B610-A19503B9AAD1-77302942!pb-smtp21.pobox.com Patrick Steinhardt writes: > Filtering out advices would be doable. But we probably wouldn't want to > do so unconditionally whenever we execute Git commands. Can you elaborate? Would you only sometimes show advice messages that come from "git" you invoke? Based on what criteria would it decide whether to filter or not to? Is it purely per location in your program (i.e., "every time the control flow reaches this call to an equivalent of run_command(), we would filter the "hint:" messages")? In an invocation where you would not filter, what effect does users' setting of advice.* configuration variables have, and what effect does a new and unseen kind of advice messages have?