From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from gnu.wildebeest.org (gnu.wildebeest.org [45.83.234.184]) by sourceware.org (Postfix) with ESMTPS id 09FB03861821 for ; Sun, 18 Feb 2024 22:58:11 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.2 sourceware.org 09FB03861821 Authentication-Results: sourceware.org; dmarc=none (p=none dis=none) header.from=klomp.org Authentication-Results: sourceware.org; spf=pass smtp.mailfrom=klomp.org ARC-Filter: OpenARC Filter v1.0.0 sourceware.org 09FB03861821 Authentication-Results: server2.sourceware.org; arc=none smtp.remote-ip=45.83.234.184 ARC-Seal: i=1; a=rsa-sha256; d=sourceware.org; s=key; t=1708297093; cv=none; b=XKrdqTS7yCe4HYarFLXrOGJPDBY485F+s1AoRqlTUj/PO1YzQBwWKGJFMRZaM12x7NJsatJlr8I8FWViimfYDz26GKDsCi7w2gcZ9f3rMa0ktJ/JItTvwazSHfiu/lF9Gl6XF7hutgaJReFhJUrEsWXvzxjxGy/owomk76zk1y8= ARC-Message-Signature: i=1; a=rsa-sha256; d=sourceware.org; s=key; t=1708297093; c=relaxed/simple; bh=UcfOLmiY403mE3rGZnTN1fywPIZ6YR+0drqp1EjfJBU=; h=Date:From:To:Subject:Message-ID:MIME-Version; b=sYiUoq55tOjA2EHv7D4sntUQtyY1evs6kSUc8ZC25Bq0rmypFmykA62qllhKOD358hXBrUDDRMF8H0Wdf88eP9m0QYcOf2hov9mYUsTSciTruw4UzaEnpUvJifLOnc5cmYh1HDve5k15ozBDbTEoIA8o8dE6r3xdURAksaVeIOQ= ARC-Authentication-Results: i=1; server2.sourceware.org Received: by gnu.wildebeest.org (Postfix, from userid 1000) id 26A8A30003D0; Sun, 18 Feb 2024 23:58:10 +0100 (CET) Date: Sun, 18 Feb 2024 23:58:10 +0100 From: Mark Wielaard To: David Malcolm Cc: Joseph Myers , Tobias Burnus , gcc-patches , buildbot@sourceware.org Subject: Re: CI for "Option handling: add documentation URLs" Message-ID: <20240218225810.GH16832@gnu.wildebeest.org> References: <20231210233506.GC14917@gnu.wildebeest.org> <20231214150143.3305661-1-dmalcolm@redhat.com> <20231215005908.GC12145@gnu.wildebeest.org> <2fb7351ea8964914836415cf6cd36c0474cc6a33.camel@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <2fb7351ea8964914836415cf6cd36c0474cc6a33.camel@redhat.com> User-Agent: Mutt/1.5.21 (2010-09-15) X-Spam-Status: No, score=-3.1 required=5.0 tests=BAYES_00,JMQ_SPF_NEUTRAL,KAM_DMARC_STATUS,SPF_HELO_NONE,SPF_PASS,TXREP,T_SCC_BODY_TEXT_LINE 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: Hi David, On Thu, Jan 04, 2024 at 09:57:09AM -0500, David Malcolm wrote: > I've pushed the .opt.urls patch kit to gcc trunk [1], so hopefully the > CI check you wrote can go live now. And then I was on vacation myself and forgot. I am sorry. So, I did try the regenerate-opt-urls locally, and it did generate the attached diff. Which seems to show we really need this automated. Going over the diff. The -Winfinite-recursion in rust does indeed seem new. As do the -mapx-inline-asm-use-gpr32 and mevex512 for i386. And the avr options -mskip-bug, -mflmap and mrodata-in-ram. The change in common.opt.urls for -Wuse-after-free comes from it being moved from c++ to the c-family. The changes in mips.opt.urls seem to come from commit 46df1369 "doc/invoke: Remove duplicate explicit-relocs entry of MIPS". The changes in c.opt.urls seem mostly reordering. The sorting makes more sense after the diff imho. And must have come from commit 4666cbde5 "Sort warning options in c-family/c.opt". Also the documentation for -Warray-parameter was fixed. So I think the regenerate-opt-urls check does work as intended. So lets automate it, because it looks like nobody regenerated the url.opts after updating the documentation. But we should first apply this diff. Could you double check it is sane/correct? Thanks, Mark