From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from lucy.dinwoodie.org (b.8.0.0.8.9.b.0.2.f.0.9.2.a.d.b.d.a.0.2.5.1.e.d.0.b.8.0.1.0.0.2.ip6.arpa [IPv6:2001:8b0:de15:20ad:bda2:90f2:b98:8b]) by sourceware.org (Postfix) with ESMTPS id 03D843858D1E for ; Sat, 5 Mar 2022 13:25:25 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.1 sourceware.org 03D843858D1E Authentication-Results: sourceware.org; dmarc=none (p=none dis=none) header.from=dinwoodie.org Authentication-Results: sourceware.org; spf=pass smtp.mailfrom=dinwoodie.org Received: from adam by lucy.dinwoodie.org with local (Exim 4.94.2) (envelope-from ) id 1nQUPS-00BxLx-RC for cygwin@cygwin.com; Sat, 05 Mar 2022 13:25:22 +0000 Date: Sat, 5 Mar 2022 13:25:22 +0000 From: Adam Dinwoodie To: cygwin@cygwin.com Subject: Re: minor error in skel/.bashrc Message-ID: <20220305132522.uq4yfrvmnrn4szvf@lucy.dinwoodie.org> Reply-To: cygwin@cygwin.com References: MIME-Version: 1.0 Content-Type: multipart/mixed; boundary="dcpfph6wzvqlzcv4" Content-Disposition: inline In-Reply-To: X-Spam-Status: No, score=-7.1 required=5.0 tests=BAYES_00, GIT_PATCH_0, KAM_DMARC_STATUS, PDS_RDNS_DYNAMIC_FP, RDNS_DYNAMIC, SPF_HELO_PASS, SPF_PASS, TXREP, T_SCC_BODY_TEXT_LINE autolearn=ham autolearn_force=no version=3.4.4 X-Spam-Checker-Version: SpamAssassin 3.4.4 (2020-01-24) on server2.sourceware.org X-BeenThere: cygwin@cygwin.com X-Mailman-Version: 2.1.29 Precedence: list List-Id: General Cygwin discussions and problem reports List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sat, 05 Mar 2022 13:25:26 -0000 --dcpfph6wzvqlzcv4 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline On Sat, Mar 05, 2022 at 09:26:06AM +0300, lvm wrote: > The identical /etc/skel/.bashrc and /etc/defaults/etc/skel/.bashrc contain > the same line > > # export HISTCONTROL=$HISTCONTROL${HISTCONTROL+,}ignoredups > > But HISTCONTROL values should be colon separated, not comma separated. This > line is commented out, but if someone like me tries to use it without much > thinking... Confirmed. Achim, I've attached a patch that should fix this, based on the tip of https://sourceware.org/git/cygwin-apps/base-files.git HTH Adam --dcpfph6wzvqlzcv4 Content-Type: text/x-diff; charset=us-ascii Content-Disposition: attachment; filename="0001-Correct-HISTCONTROL-in-.bashrc.patch" >From 90bd8c6b46a467b26972f5ecac5c2f05ed321f49 Mon Sep 17 00:00:00 2001 From: Adam Dinwoodie Date: Sat, 5 Mar 2022 13:14:42 +0000 Subject: [PATCH] Correct HISTCONTROL in .bashrc Per bash.1, HISTCONTROL values should be colon-separated, not comma-separated, so correct the sample line for adding "ignoredups" to HISTCONTROL to use the correct separator. At the same time, use the separator if and only if there is a non-null value already set in HISTCONTROL. The previous version of this sample code would insert the separator if HISTCONTROL were defined as the empty string, where it'd be exactly as unnecessary as if it weren't defined at all. --- etc/defaults/etc/skel/.bashrc | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/etc/defaults/etc/skel/.bashrc b/etc/defaults/etc/skel/.bashrc index 3a4f59a..b6a48cf 100644 --- a/etc/defaults/etc/skel/.bashrc +++ b/etc/defaults/etc/skel/.bashrc @@ -54,7 +54,7 @@ # History Options # # Don't put duplicate lines in the history. -# export HISTCONTROL=$HISTCONTROL${HISTCONTROL+,}ignoredups +# export HISTCONTROL=$HISTCONTROL${HISTCONTROL:+:}ignoredups # # Ignore some controlling instructions # HISTIGNORE is a colon-delimited list of patterns which should be excluded. -- 2.35.1 --dcpfph6wzvqlzcv4--