GNU bug report logs - #68479
[PATCH] weather: Add ‘-e’.

Previous Next

Full log

View this message in rfc822 format

From: help-debbugs <at> gnu.org (GNU bug Tracking System)
To: Ludovic Courtès <ludo <at> gnu.org>
Subject: bug#68479: closed (Re: [bug#68479] [PATCH] weather: Add
 ‘-e’.)
Date: Sun, 28 Jan 2024 21:26:02 +0000

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

Your bug report

#68479: [PATCH] weather: Add ‘-e’.

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 68479 <at> debbugs.gnu.org.

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

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

From: Ludovic Courtès <ludo <at> gnu.org>
To: Simon Tournier <zimon.toutoune <at> gmail.com>
Cc: 68479-done <at> debbugs.gnu.org
Subject: Re: [bug#68479] [PATCH] weather: Add ‘-e’.
Date: Sun, 28 Jan 2024 22:25:37 +0100

Simon Tournier <zimon.toutoune <at> gmail.com> skribis:

> On lun., 15 janv. 2024 at 18:21, Ludovic Courtès <ludo <at> gnu.org> wrote:
>> * guix/scripts/weather.scm (show-help, %options): Add ‘-e’.
>> (guix-weather): Handle it.
>> * doc/guix.texi (Invoking guix weather): Document it.
>>
>> Change-Id: I6dc97ec2b8226b57be33247b05a34c23b573a64f
>> ---
>>  doc/guix.texi            | 16 +++++++++++++++-
>>  guix/scripts/weather.scm |  9 ++++++++-
>>  2 files changed, 23 insertions(+), 2 deletions(-)
>>
>> Hi!
>>
>> This patch adds to ‘guix weather’ the ‘-e’ option that ‘guix build’
>> and most other commands already support.
>
> LGTM.

Pushed as 6cca8f069431f4475d8eaf9336cb952ce5694e55, thanks!

Ludo'.

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

From: Ludovic Courtès <ludo <at> gnu.org>
To: guix-patches <at> gnu.org
Cc: Ludovic Courtès <ludo <at> gnu.org>
Subject: [PATCH] weather: Add ‘-e’.
Date: Mon, 15 Jan 2024 18:21:32 +0100

* guix/scripts/weather.scm (show-help, %options): Add ‘-e’.
(guix-weather): Handle it.
* doc/guix.texi (Invoking guix weather): Document it.

Change-Id: I6dc97ec2b8226b57be33247b05a34c23b573a64f
---
 doc/guix.texi            | 16 +++++++++++++++-
 guix/scripts/weather.scm |  9 ++++++++-
 2 files changed, 23 insertions(+), 2 deletions(-)

Hi!

This patch adds to ‘guix weather’ the ‘-e’ option that ‘guix build’
and most other commands already support.

Thoughts?

Ludo’.

diff --git a/doc/guix.texi b/doc/guix.texi
index a66005ee9d..fe16dd30e6 100644
--- a/doc/guix.texi
+++ b/doc/guix.texi
@@ -22,7 +22,7 @@
 @set SUBSTITUTE-URLS https://@value{SUBSTITUTE-SERVER-1} https://@value{SUBSTITUTE-SERVER-2}
 
 @copying
-Copyright @copyright{} 2012-2023 Ludovic Courtès@*
+Copyright @copyright{} 2012-2024 Ludovic Courtès@*
 Copyright @copyright{} 2013, 2014, 2016 Andreas Enge@*
 Copyright @copyright{} 2013 Nikita Karetnikov@*
 Copyright @copyright{} 2014, 2015, 2016 Alex Kost@*
@@ -16721,6 +16721,20 @@ Invoking guix weather
 This option can be repeated several times, in which case the manifests
 are concatenated.
 
+@item --expression=@var{expr}
+@itemx -e @var{expr}
+Consider the package @var{expr} evaluates to.
+
+A typical use case for this option is specifying a package that is
+hidden and thus cannot be referred to in the usual way, as in this
+example:
+
+@example
+guix weather -e '(@@@@ (gnu packages rust) rust-bootstrap)'
+@end example
+
+This option can be repeated.
+
 @item --coverage[=@var{count}]
 @itemx -c [@var{count}]
 Report on substitute coverage for packages: list packages with at least
diff --git a/guix/scripts/weather.scm b/guix/scripts/weather.scm
index 2f8985593d..08a1b22a74 100644
--- a/guix/scripts/weather.scm
+++ b/guix/scripts/weather.scm
@@ -1,5 +1,5 @@
 ;;; GNU Guix --- Functional package management for GNU
-;;; Copyright © 2017-2023 Ludovic Courtès <ludo <at> gnu.org>
+;;; Copyright © 2017-2024 Ludovic Courtès <ludo <at> gnu.org>
 ;;; Copyright © 2017 Ricardo Wurmus <rekado <at> elephly.net>
 ;;; Copyright © 2018 Kyle Meyer <kyle <at> kyleam.com>
 ;;; Copyright © 2020 Simon Tournier <zimon.toutoune <at> gmail.com>
@@ -388,6 +388,8 @@ (define (show-help)
   -m, --manifest=MANIFEST
                          look up substitutes for packages specified in MANIFEST"))
   (display (G_ "
+  -e, --expression=EXPR  build the object EXPR evaluates to"))
+  (display (G_ "
   -c, --coverage[=COUNT]
                          show substitute coverage for packages with at least
                          COUNT dependents"))
@@ -426,6 +428,9 @@ (define %options
          (option '(#\m "manifest") #t #f
                  (lambda (opt name arg result)
                    (alist-cons 'manifest arg result)))
+         (option '(#\e "expression") #t #f
+                 (lambda (opt name arg result)
+                   (alist-cons 'expression arg result)))
          (option '(#\c "coverage") #f #t
                  (lambda (opt name arg result)
                    (alist-cons 'coverage
@@ -611,6 +616,8 @@ (define-command (guix-weather . args)
           (base  (filter-map (match-lambda
                                (('argument . spec)
                                 (specification->package spec))
+                               (('expression . str)
+                                (read/eval-package-expression str))
                                (_
                                 #f))
                              opts)))

base-commit: 162d6a2fdd6af13272967c77347a54934ecb45e6
-- 
2.41.0

Previous Next

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