From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from gproxy1-pub.mail.unifiedlayer.com (gproxy1-pub.mail.unifiedlayer.com [69.89.25.95]) by sourceware.org (Postfix) with ESMTPS id AA0953858D35 for ; Thu, 12 Jan 2023 03:01:07 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.2 sourceware.org AA0953858D35 Authentication-Results: sourceware.org; dmarc=none (p=none dis=none) header.from=tromey.com Authentication-Results: sourceware.org; spf=pass smtp.mailfrom=tromey.com Received: from cmgw12.mail.unifiedlayer.com (unknown [10.0.90.127]) by progateway3.mail.pro1.eigbox.com (Postfix) with ESMTP id B88751004811F for ; Thu, 12 Jan 2023 03:01:06 +0000 (UTC) Received: from box5379.bluehost.com ([162.241.216.53]) by cmsmtp with ESMTP id FnpypqUlpB18UFnpypA3YX; Thu, 12 Jan 2023 03:01:06 +0000 X-Authority-Reason: nr=8 X-Authority-Analysis: v=2.4 cv=Ns4Uz+RJ c=1 sm=1 tr=0 ts=63bf77f2 a=ApxJNpeYhEAb1aAlGBBbmA==:117 a=ApxJNpeYhEAb1aAlGBBbmA==:17 a=dLZJa+xiwSxG16/P+YVxDGlgEgI=:19 a=RvmDmJFTN0MA:10:nop_rcvd_month_year a=Qbun_eYptAEA:10:endurance_base64_authed_username_1 a=CCpqsmhAAAAA:8 a=_1sFaUNRbXHV5H4zFP0A:9 a=4a9ptq71g-gA:10:demote_hacked_domain_1 a=ul9cdbp4aOFLsgKbc677:22 DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=tromey.com; s=default; h=Content-Transfer-Encoding:MIME-Version:Message-Id:Date:Subject: To:From:Sender:Reply-To:Cc:Content-Type:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: In-Reply-To:References:List-Id:List-Help:List-Unsubscribe:List-Subscribe: List-Post:List-Owner:List-Archive; bh=UT/B1ForvTjpPnRP83dvAXxxq9OXzR6nRgyR/PizUVU=; b=MxUIj4L8vBIDIJc1SYKDKfLPNU OjXK2WGuGSH9z6Oem1Na645CUmgtMRJOxr+PWtNOpzsyUS0/QirXkkBCeJR7PNMMHgouH68LyfNG+ Dm37ciihTpEkwgnKetug5FQgN; Received: from 97-122-76-186.hlrn.qwest.net ([97.122.76.186]:53100 helo=localhost.localdomain) by box5379.bluehost.com with esmtpsa (TLS1.2) tls TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384 (Exim 4.95) (envelope-from ) id 1pFnpy-000lEr-Bx for gdb-patches@sourceware.org; Wed, 11 Jan 2023 20:01:06 -0700 From: Tom Tromey To: gdb-patches@sourceware.org Subject: [PATCH v2 00/79] Rewrite "require" test procedure and use it more often Date: Wed, 11 Jan 2023 19:59:33 -0700 Message-Id: <20230112030052.3306113-1-tom@tromey.com> X-Mailer: git-send-email 2.39.0 MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-AntiAbuse: This header was added to track abuse, please include it with any abuse report X-AntiAbuse: Primary Hostname - box5379.bluehost.com X-AntiAbuse: Original Domain - sourceware.org X-AntiAbuse: Originator/Caller UID/GID - [47 12] / [47 12] X-AntiAbuse: Sender Address Domain - tromey.com X-BWhitelist: no X-Source-IP: 97.122.76.186 X-Source-L: No X-Exim-ID: 1pFnpy-000lEr-Bx X-Source: X-Source-Args: X-Source-Dir: X-Source-Sender: 97-122-76-186.hlrn.qwest.net (localhost.localdomain) [97.122.76.186]:53100 X-Source-Auth: tom+tromey.com X-Email-Count: 1 X-Source-Cap: ZWx5bnJvYmk7ZWx5bnJvYmk7Ym94NTM3OS5ibHVlaG9zdC5jb20= X-Local-Domain: yes X-Spam-Status: No, score=-3022.1 required=5.0 tests=BAYES_00,DKIM_SIGNED,DKIM_VALID,JMQ_SPF_NEUTRAL,RCVD_IN_DNSWL_NONE,RCVD_IN_MSPIKE_H2,SPF_HELO_NONE,SPF_PASS,TXREP autolearn=no autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on server2.sourceware.org List-Id: Here's v2 of the series to change "require". v1 was here: https://sourceware.org/pipermail/gdb-patches/2022-December/194843.html ... with some reviews in January. This version addresses the "double negative" issue by renaming all "skip_" procs that are used in "require" to be "allow_" procs instead. While doing this, I tackled a couple other minor things. I changed the checks for Python and Guile to check the output of "gdb --configuration". This lets these checks be done without a running gdb, which IMO is a nicer way for them to work. Then I added --config output so that this same change could be done for the TUI. I found one minor bug while working on this -- a hidden dependency, see patch 49. Regression tested on x86-64 Fedora 36. This is the sort of patch that has to be regression tested after each rebase, because other patches may introduce new uses of the removed procs. Also it perhaps should be tested on other platforms. While working on this, I got annoyed by load_lib being needed everywhere. I have another patch to introduce the use of Tcl autoloading, but this proves to be tricky due to the code that saves and restores global variables. It can be done -- and I think it's worth doing -- but this series is already too long. Tom