GNU bug report logs - #66499
[PATCH] ui: Handle EPIPE errors when displaying Guix version.

Previous Next

Full log

View this message in rfc822 format

From: help-debbugs <at> gnu.org (GNU bug Tracking System)
To: Simon Tournier <zimon.toutoune <at> gmail.com>
Subject: bug#66499: closed (Re: [bug#66499] [PATCH] ui: Handle EPIPE
 errors when displaying Guix version.)
Date: Tue, 17 Oct 2023 12:59:02 +0000

[Message part 1 (text/plain, inline)]

Your bug report

#66499: [PATCH] ui: Handle EPIPE errors when displaying Guix version.

which was filed against the guix-patches package, has been closed.

The explanation is attached below, along with your original report.
If you require more details, please reply to 66499 <at> debbugs.gnu.org.

-- 
66499: https://debbugs.gnu.org/cgi/bugreport.cgi?bug=66499
GNU Bug Tracking System
Contact help-debbugs <at> gnu.org with problems

[Message part 2 (message/rfc822, inline)]

From: Simon Tournier <zimon.toutoune <at> gmail.com>
To: Mathieu Othacehe <othacehe <at> gnu.org>
Cc: Josselin Poiret <dev <at> jpoiret.xyz>, 66499-done <at> debbugs.gnu.org,
 Ludovic Courtès <ludo <at> gnu.org>,
 Tobias Geerinckx-Rice <me <at> tobias.gr>,
 Clément Lassieur <clement <at> lassieur.org>,
 Ricardo Wurmus <rekado <at> elephly.net>, Christopher Baines <guix <at> cbaines.net>
Subject: Re: [bug#66499] [PATCH] ui: Handle EPIPE errors when displaying Guix
 version.
Date: Tue, 17 Oct 2023 14:57:27 +0200

Hi,

On Tue, 17 Oct 2023 at 11:48, Mathieu Othacehe <othacehe <at> gnu.org> wrote:

> It looks OK to me.

Thanks.  Pushed and closing.

Cheers,
simon

[Message part 3 (message/rfc822, inline)]

From: Simon Tournier <zimon.toutoune <at> gmail.com>
To: guix-patches <at> gnu.org
Cc: Simon Tournier <zimon.toutoune <at> gmail.com>
Subject: [PATCH] ui: Handle EPIPE errors when displaying Guix version.
Date: Thu, 12 Oct 2023 15:39:03 +0200

Fixes <https://issues.guix.gnu.org/66254>.
Reported by Clément Lassieur <clement <at> lassieur.org>.

* guix/ui.scm (run-guix): Handle EPIPE errors when displaying version.
---
 guix/ui.scm | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Hi,

It appears to me a trivial patch that could directly could go be merged but
just in case I am not missing the obvious.

Cheers,
simon


diff --git a/guix/ui.scm b/guix/ui.scm
index 6f2d4fe245..507bc67f1d 100644
--- a/guix/ui.scm
+++ b/guix/ui.scm
@@ -2309,7 +2309,7 @@ (define (run-guix . args)
       ((or ("-h") ("--help"))
        (leave-on-EPIPE (show-guix-help)))
       ((or ("-V") ("--version"))
-       (show-version-and-exit "guix"))
+       (leave-on-EPIPE (show-version-and-exit "guix")))
       (((? option? o) args ...)
        (format (current-error-port)
                (G_ "guix: unrecognized option '~a'~%") o)

base-commit: 1328c4cca531318e3ed90c6aecb522a5b22a4bcc
-- 
2.38.1

Previous Next

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