From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from hamza.pair.com (hamza.pair.com [209.68.5.143]) by sourceware.org (Postfix) with ESMTPS id 35D4C3858D37 for ; Thu, 20 Apr 2023 20:14:36 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.2 sourceware.org 35D4C3858D37 Authentication-Results: sourceware.org; dmarc=none (p=none dis=none) header.from=pfeifer.com Authentication-Results: sourceware.org; spf=pass smtp.mailfrom=pfeifer.com Received: from hamza.pair.com (localhost [127.0.0.1]) by hamza.pair.com (Postfix) with ESMTP id BC23D33E53; Thu, 20 Apr 2023 16:14:35 -0400 (EDT) Received: from [10.30.2.33] (unknown [195.77.44.100]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits) server-digest SHA256) (No client certificate requested) by hamza.pair.com (Postfix) with ESMTPSA id 3643733E8A; Thu, 20 Apr 2023 16:14:35 -0400 (EDT) Date: Thu, 20 Apr 2023 22:14:33 +0200 (CEST) From: Gerald Pfeifer To: =?ISO-8859-2?Q?Arsen_Arsenovi=E6?= cc: gcc-patches@gcc.gnu.org Subject: Re: [PATCH] update_web_docs_git: Add updated Texinfo to PATH In-Reply-To: <87edomcim4.fsf@aarsen.me> Message-ID: <7af14864-04f1-0b56-2236-507b0eefcbf8@pfeifer.com> References: <20230406103533.1087349-1-arsen@aarsen.me> <86a5zf2x7x.fsf@aarsen.me> <545acffe-c604-5583-0046-2b3710819b4f@pfeifer.com> <87edomcim4.fsf@aarsen.me> MIME-Version: 1.0 Content-Type: multipart/mixed; BOUNDARY="8323328-450487868-1682021365=:4410" X-Scanned-By: mailmunge 3.11 on 209.68.5.143 X-Spam-Status: No, score=-3.7 required=5.0 tests=BAYES_00,KAM_DMARC_STATUS,SPF_HELO_NONE,SPF_PASS,TXREP,T_SCC_BODY_TEXT_LINE autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on server2.sourceware.org List-Id: This message is in MIME format. The first part should be readable text, while the remaining parts are likely unreadable without MIME-aware tools. --8323328-450487868-1682021365=:4410 Content-Type: text/plain; CHARSET=ISO-8859-2 Content-Transfer-Encoding: 8BIT Hi Arsen, On Fri, 14 Apr 2023, Arsen Arsenoviæ wrote: >> Did you intentionally not implement the following part of my suggestion >> >> if [ x${MAKEINFO}x = xx ]; then >> : > > that is, allowing to override from the command-line (or crontab)? > (answering both the questions) > > This := operator is a handy "default assign" operator. It's a bit of an > oddity of the POSIX shell, but it works well. The line: > > : "${foo:=bar}" > > is a convenient way of spelling "if foo is unset or null, set it to > bar". the initial ':' there serves to discard the result of this > evaluation (so that only its side effect of updating foo if necessary is > kept) I understand, just am wondering whether and why the : is required? I don't think we are using this construct anywhere else? (I was aware of the ${foo:=bar} syntax, just caught up by you pushing that part of the logic to the lowest level whereas I had it at the top level. That's purely on me.) Please go ahead and push this (or a variant without the : commands) and I'll then pick it up from there. Gerald --8323328-450487868-1682021365=:4410--