From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from gnu.wildebeest.org (wildebeest.demon.nl [212.238.236.112]) by sourceware.org (Postfix) with ESMTPS id E53763938C33 for ; Mon, 3 May 2021 13:19:33 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.3.2 sourceware.org E53763938C33 Authentication-Results: sourceware.org; dmarc=none (p=none dis=none) header.from=klomp.org Authentication-Results: sourceware.org; spf=pass smtp.mailfrom=mark@klomp.org Received: from tarox.wildebeest.org (tarox.wildebeest.org [172.31.17.39]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by gnu.wildebeest.org (Postfix) with ESMTPSA id 584C5302BBED; Mon, 3 May 2021 15:19:32 +0200 (CEST) Received: by tarox.wildebeest.org (Postfix, from userid 1000) id 39F3E413CB92; Mon, 3 May 2021 15:19:32 +0200 (CEST) Message-ID: <4aa0e98c4393f9d30d50b7363b3edc15eb46cd92.camel@klomp.org> Subject: Re: PATCH: PR27784 debuginfod: notify interactive clients one time about usage From: Mark Wielaard To: "Frank Ch. Eigler" , elfutils-devel@sourceware.org Date: Mon, 03 May 2021 15:19:32 +0200 In-Reply-To: <20210429184412.GC6131@redhat.com> References: <20210429184412.GC6131@redhat.com> Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable X-Mailer: Evolution 3.28.5 (3.28.5-10.el7) Mime-Version: 1.0 X-Spam-Status: No, score=-5.2 required=5.0 tests=BAYES_00, KAM_DMARC_STATUS, SPF_HELO_NONE, SPF_PASS, TXREP autolearn=ham autolearn_force=no version=3.4.2 X-Spam-Checker-Version: SpamAssassin 3.4.2 (2018-09-13) on server2.sourceware.org X-BeenThere: elfutils-devel@sourceware.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Elfutils-devel mailing list List-Unsubscribe: , List-Archive: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 03 May 2021 13:19:35 -0000 Hi Frank, On Thu, 2021-04-29 at 14:44 -0400, Frank Ch. Eigler via Elfutils-devel wrote: > Another request from fedora fesco. (I don't have anything else on > the queue for the next elfutils release.) >=20 > commit be5ebebb1f12134b673c5b8cbede62390d077b0d > Author: Frank Ch. Eigler > Date: Thu Apr 29 14:34:06 2021 -0400 >=20 > PR27784 debuginfod: notify interactive clients one time about usage > =20 > Because debuginfod-client functionality makes a user dependent on the > correct operation of remote debuginfod server, it was suggested that > new users be notified of this. This patch adds a one-time > notification to stderr if it isatty(), commemorated by a new cache > notify_p file. (It cannot easily be tested because our test scripts > run without pty/tty enclosure.) I cannot say I am a fan of the library being the one to do this check, especially not with the sleep (3) in it. It also is not really accurate, it only shows the debuginfod server URLs the first time they are set, not when they are changed. I would hide this behind DEBUGINFOD_VERBOSE=3D1 but that already shows the URLs anyway. If there is such a "notification" then it should be from the actual application using the library IMHO. Or when installing a profile if it provides default DEBUGINFOD_URLS. Cheers, Mark