GNU bug report logs - #51512
[PATCH]: gnu: Add bash-bcu.

Previous Next

Package: guix-patches;

Reported by: david larsson <david.larsson <at> selfhosted.xyz>

Date: Sat, 30 Oct 2021 15:00:01 UTC

Severity: normal

Tags: patch

Full log


View this message in rfc822 format

From: david larsson <david.larsson <at> selfhosted.xyz>
To: Tobias Geerinckx-Rice <me <at> tobias.gr>
Cc: 51512 <at> debbugs.gnu.org
Subject: [bug#51512] [PATCH v 2]: gnu: Add bash-bcu.
Date: Tue, 23 Nov 2021 13:44:07 +0100
> However, you make an excellent point, and this packages's intended
> usage means that it will clobber the caller's environment, not just
> its own.

Yep, it will. Previously I had the dependencies as propagated inputs but 
I think the current approach with putting them as inputs and creating a 
wrapper script is better.

> If I were to use this package (and I might—it's neat) I would almost
> certainly end up using it in a setting where changing PATH at all
> would cause unexpected and frustrating ‘bugs’.
> 
> Hmm…

Im not sure I follow here.. can you give an example? Maybe some info or 
warning can be added to the description?

Personally, I would be happy to see the current patch you provided 
committed to master, except I would just update to the very most recent 
commit again.

Thanks for reviewing this!

Best regards,
David




This bug report was last modified 2 years and 22 days ago.

Previous Next


GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.