unofficial mirror of guix-patches@gnu.org 
 help / color / mirror / code / Atom feed
* [bug#59235] [PATCH] gnu: %options: Add short flag -R for --list-transitive option.
@ 2022-11-13  3:28 jgart via Guix-patches via
  2022-11-13  7:56 ` ( via Guix-patches via
                   ` (3 more replies)
  0 siblings, 4 replies; 8+ messages in thread
From: jgart via Guix-patches via @ 2022-11-13  3:28 UTC (permalink / raw)
  To: 59235; +Cc: jgart

* guix/scripts/refresh.scm (%options): Add short flag -R.
---
 guix/scripts/refresh.scm | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/guix/scripts/refresh.scm b/guix/scripts/refresh.scm
index 14329751f8..aff32c13de 100644
--- a/guix/scripts/refresh.scm
+++ b/guix/scripts/refresh.scm
@@ -9,6 +9,7 @@
 ;;; Copyright © 2019 Ricardo Wurmus <rekado@elephly.net>
 ;;; Copyright © 2020 Simon Tournier <zimon.toutoune@gmail.com>
 ;;; Copyright © 2021 Sarah Morgensen <iskarian@mgsn.dev>
+;;; Copyright © 2022 jgart <jgart@dismail.de>
 ;;;
 ;;; This file is part of GNU Guix.
 ;;;
@@ -99,7 +100,7 @@ (define %options
         (option '(#\r "recursive") #f #f
                 (lambda (opt name arg result)
                   (alist-cons 'recursive? #t result)))
-        (option '("list-transitive") #f #f
+        (option '(#\R "list-transitive") #f #f
                 (lambda (opt name arg result)
                   (alist-cons 'list-transitive? #t result)))
 
@@ -157,7 +158,7 @@ (define (show-help)
   (display (G_ "
   -r, --recursive        check the PACKAGE and its inputs for upgrades"))
   (display (G_ "
-      --list-transitive  list all the packages that PACKAGE depends on"))
+  -R, --list-transitive  list all the packages that PACKAGE depends on"))
   (newline)
   (display (G_ "
       --keyring=FILE     use FILE as the keyring of upstream OpenPGP keys"))
-- 
2.38.1





^ permalink raw reply related	[flat|nested] 8+ messages in thread

* [bug#59235] [PATCH] gnu: %options: Add short flag -R for --list-transitive option.
  2022-11-13  3:28 [bug#59235] [PATCH] gnu: %options: Add short flag -R for --list-transitive option jgart via Guix-patches via
@ 2022-11-13  7:56 ` ( via Guix-patches via
  2022-11-13 15:05 ` [bug#59235] [PATCH v2] guix: scripts: refresh: " jgart via Guix-patches via
                   ` (2 subsequent siblings)
  3 siblings, 0 replies; 8+ messages in thread
From: ( via Guix-patches via @ 2022-11-13  7:56 UTC (permalink / raw)
  To: jgart, 59235

On Sun Nov 13, 2022 at 3:28 AM GMT, jgart via Guix-patches via wrote:
> gnu: %options: ...

This should be:

  scripts: refresh: ...

> * guix/scripts/refresh.scm (%options): Add short flag -R.

Other than that, LGTM! :)

    -- (




^ permalink raw reply	[flat|nested] 8+ messages in thread

* [bug#59235] [PATCH v2] guix: scripts: refresh: Add short flag -R for --list-transitive option.
  2022-11-13  3:28 [bug#59235] [PATCH] gnu: %options: Add short flag -R for --list-transitive option jgart via Guix-patches via
  2022-11-13  7:56 ` ( via Guix-patches via
@ 2022-11-13 15:05 ` jgart via Guix-patches via
  2022-11-13 15:38 ` [bug#59235] [PATCH v3 1/2] " jgart via Guix-patches via
  2022-11-13 16:27 ` [bug#59235] [PATCH v4 1/2] " jgart via Guix-patches via
  3 siblings, 0 replies; 8+ messages in thread
From: jgart via Guix-patches via @ 2022-11-13 15:05 UTC (permalink / raw)
  To: 59235; +Cc: (, jgart

* guix/scripts/refresh.scm (%options): Add short flag -R.

Here's v2 with your requests. Thanks for the review, paren.

all best,

jgart
---
 guix/scripts/refresh.scm | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/guix/scripts/refresh.scm b/guix/scripts/refresh.scm
index 14329751f8..aff32c13de 100644
--- a/guix/scripts/refresh.scm
+++ b/guix/scripts/refresh.scm
@@ -9,6 +9,7 @@
 ;;; Copyright © 2019 Ricardo Wurmus <rekado@elephly.net>
 ;;; Copyright © 2020 Simon Tournier <zimon.toutoune@gmail.com>
 ;;; Copyright © 2021 Sarah Morgensen <iskarian@mgsn.dev>
+;;; Copyright © 2022 jgart <jgart@dismail.de>
 ;;;
 ;;; This file is part of GNU Guix.
 ;;;
@@ -99,7 +100,7 @@ (define %options
         (option '(#\r "recursive") #f #f
                 (lambda (opt name arg result)
                   (alist-cons 'recursive? #t result)))
-        (option '("list-transitive") #f #f
+        (option '(#\R "list-transitive") #f #f
                 (lambda (opt name arg result)
                   (alist-cons 'list-transitive? #t result)))
 
@@ -157,7 +158,7 @@ (define (show-help)
   (display (G_ "
   -r, --recursive        check the PACKAGE and its inputs for upgrades"))
   (display (G_ "
-      --list-transitive  list all the packages that PACKAGE depends on"))
+  -R, --list-transitive  list all the packages that PACKAGE depends on"))
   (newline)
   (display (G_ "
       --keyring=FILE     use FILE as the keyring of upstream OpenPGP keys"))
-- 
2.38.1





^ permalink raw reply related	[flat|nested] 8+ messages in thread

* [bug#59235] [PATCH v3 1/2] guix: scripts: refresh: Add short flag -R for --list-transitive option.
  2022-11-13  3:28 [bug#59235] [PATCH] gnu: %options: Add short flag -R for --list-transitive option jgart via Guix-patches via
  2022-11-13  7:56 ` ( via Guix-patches via
  2022-11-13 15:05 ` [bug#59235] [PATCH v2] guix: scripts: refresh: " jgart via Guix-patches via
@ 2022-11-13 15:38 ` jgart via Guix-patches via
  2022-11-13 15:38   ` [bug#59235] [PATCH v3 2/2] doc: Mention --list-transitive short flag in the manual jgart via Guix-patches via
  2022-11-13 16:01   ` [bug#59235] [PATCH v3 1/2] guix: scripts: refresh: Add short flag -R for --list-transitive option ( via Guix-patches via
  2022-11-13 16:27 ` [bug#59235] [PATCH v4 1/2] " jgart via Guix-patches via
  3 siblings, 2 replies; 8+ messages in thread
From: jgart via Guix-patches via @ 2022-11-13 15:38 UTC (permalink / raw)
  To: 59235; +Cc: jgart

* guix/scripts/refresh.scm (%options): Add short flag -R.

Here's v3 adding a doc entry.

all best,

jgart
---
 guix/scripts/refresh.scm | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/guix/scripts/refresh.scm b/guix/scripts/refresh.scm
index 14329751f8..aff32c13de 100644
--- a/guix/scripts/refresh.scm
+++ b/guix/scripts/refresh.scm
@@ -9,6 +9,7 @@
 ;;; Copyright © 2019 Ricardo Wurmus <rekado@elephly.net>
 ;;; Copyright © 2020 Simon Tournier <zimon.toutoune@gmail.com>
 ;;; Copyright © 2021 Sarah Morgensen <iskarian@mgsn.dev>
+;;; Copyright © 2022 jgart <jgart@dismail.de>
 ;;;
 ;;; This file is part of GNU Guix.
 ;;;
@@ -99,7 +100,7 @@ (define %options
         (option '(#\r "recursive") #f #f
                 (lambda (opt name arg result)
                   (alist-cons 'recursive? #t result)))
-        (option '("list-transitive") #f #f
+        (option '(#\R "list-transitive") #f #f
                 (lambda (opt name arg result)
                   (alist-cons 'list-transitive? #t result)))
 
@@ -157,7 +158,7 @@ (define (show-help)
   (display (G_ "
   -r, --recursive        check the PACKAGE and its inputs for upgrades"))
   (display (G_ "
-      --list-transitive  list all the packages that PACKAGE depends on"))
+  -R, --list-transitive  list all the packages that PACKAGE depends on"))
   (newline)
   (display (G_ "
       --keyring=FILE     use FILE as the keyring of upstream OpenPGP keys"))
-- 
2.38.1





^ permalink raw reply related	[flat|nested] 8+ messages in thread

* [bug#59235] [PATCH v3 2/2] doc: Mention --list-transitive short flag in the manual.
  2022-11-13 15:38 ` [bug#59235] [PATCH v3 1/2] " jgart via Guix-patches via
@ 2022-11-13 15:38   ` jgart via Guix-patches via
  2022-11-13 16:01   ` [bug#59235] [PATCH v3 1/2] guix: scripts: refresh: Add short flag -R for --list-transitive option ( via Guix-patches via
  1 sibling, 0 replies; 8+ messages in thread
From: jgart via Guix-patches via @ 2022-11-13 15:38 UTC (permalink / raw)
  To: 59235; +Cc: jgart

* doc/guix.texi (Invoking guix refresh): Mention -R short flag.
---
 doc/guix.texi | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/doc/guix.texi b/doc/guix.texi
index 3f76184495..b7f0775ae4 100644
--- a/doc/guix.texi
+++ b/doc/guix.texi
@@ -109,6 +109,7 @@ Copyright @copyright{} 2022 Reily Siegel@*
 Copyright @copyright{} 2022 Simon Streit@*
 Copyright @copyright{} 2022 (@*
 Copyright @copyright{} 2022 John Kehayias@*
+Copyright @copyright{} 2022 jgart@*
 
 Permission is granted to copy, distribute and/or modify this document
 under the terms of the GNU Free Documentation License, Version 1.3 or
@@ -14135,6 +14136,7 @@ for compatibility with an upgraded @code{flex} package.
 @table @code
 
 @item --list-transitive
+@itemx -R
 List all the packages which one or more packages depend upon.
 
 @example
-- 
2.38.1





^ permalink raw reply related	[flat|nested] 8+ messages in thread

* [bug#59235] [PATCH v3 1/2] guix: scripts: refresh: Add short flag -R for --list-transitive option.
  2022-11-13 15:38 ` [bug#59235] [PATCH v3 1/2] " jgart via Guix-patches via
  2022-11-13 15:38   ` [bug#59235] [PATCH v3 2/2] doc: Mention --list-transitive short flag in the manual jgart via Guix-patches via
@ 2022-11-13 16:01   ` ( via Guix-patches via
  1 sibling, 0 replies; 8+ messages in thread
From: ( via Guix-patches via @ 2022-11-13 16:01 UTC (permalink / raw)
  To: jgart, 59235

On Sun Nov 13, 2022 at 3:38 PM GMT, jgart via Guix-patches via wrote:
> guix: scripts: refresh: ...

Nitpick again: it's just ``scripts: refresh: ...'' ;)

    -- (




^ permalink raw reply	[flat|nested] 8+ messages in thread

* [bug#59235] [PATCH v4 1/2] scripts: refresh: Add short flag -R for --list-transitive option.
  2022-11-13  3:28 [bug#59235] [PATCH] gnu: %options: Add short flag -R for --list-transitive option jgart via Guix-patches via
                   ` (2 preceding siblings ...)
  2022-11-13 15:38 ` [bug#59235] [PATCH v3 1/2] " jgart via Guix-patches via
@ 2022-11-13 16:27 ` jgart via Guix-patches via
  2022-11-13 16:27   ` [bug#59235] [PATCH v4 2/2] doc: Mention --list-transitive short flag in the manual jgart via Guix-patches via
  3 siblings, 1 reply; 8+ messages in thread
From: jgart via Guix-patches via @ 2022-11-13 16:27 UTC (permalink / raw)
  To: 59235; +Cc: (, jgart

* guix/scripts/refresh.scm (%options): Add short flag -R.

no probs, v4 thnx!
---
 guix/scripts/refresh.scm | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/guix/scripts/refresh.scm b/guix/scripts/refresh.scm
index 14329751f8..aff32c13de 100644
--- a/guix/scripts/refresh.scm
+++ b/guix/scripts/refresh.scm
@@ -9,6 +9,7 @@
 ;;; Copyright © 2019 Ricardo Wurmus <rekado@elephly.net>
 ;;; Copyright © 2020 Simon Tournier <zimon.toutoune@gmail.com>
 ;;; Copyright © 2021 Sarah Morgensen <iskarian@mgsn.dev>
+;;; Copyright © 2022 jgart <jgart@dismail.de>
 ;;;
 ;;; This file is part of GNU Guix.
 ;;;
@@ -99,7 +100,7 @@ (define %options
         (option '(#\r "recursive") #f #f
                 (lambda (opt name arg result)
                   (alist-cons 'recursive? #t result)))
-        (option '("list-transitive") #f #f
+        (option '(#\R "list-transitive") #f #f
                 (lambda (opt name arg result)
                   (alist-cons 'list-transitive? #t result)))
 
@@ -157,7 +158,7 @@ (define (show-help)
   (display (G_ "
   -r, --recursive        check the PACKAGE and its inputs for upgrades"))
   (display (G_ "
-      --list-transitive  list all the packages that PACKAGE depends on"))
+  -R, --list-transitive  list all the packages that PACKAGE depends on"))
   (newline)
   (display (G_ "
       --keyring=FILE     use FILE as the keyring of upstream OpenPGP keys"))
-- 
2.38.1





^ permalink raw reply related	[flat|nested] 8+ messages in thread

* [bug#59235] [PATCH v4 2/2] doc: Mention --list-transitive short flag in the manual.
  2022-11-13 16:27 ` [bug#59235] [PATCH v4 1/2] " jgart via Guix-patches via
@ 2022-11-13 16:27   ` jgart via Guix-patches via
  0 siblings, 0 replies; 8+ messages in thread
From: jgart via Guix-patches via @ 2022-11-13 16:27 UTC (permalink / raw)
  To: 59235; +Cc: jgart

* doc/guix.texi (Invoking guix refresh): Mention -R short flag.
---
 doc/guix.texi | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/doc/guix.texi b/doc/guix.texi
index 3f76184495..b7f0775ae4 100644
--- a/doc/guix.texi
+++ b/doc/guix.texi
@@ -109,6 +109,7 @@ Copyright @copyright{} 2022 Reily Siegel@*
 Copyright @copyright{} 2022 Simon Streit@*
 Copyright @copyright{} 2022 (@*
 Copyright @copyright{} 2022 John Kehayias@*
+Copyright @copyright{} 2022 jgart@*
 
 Permission is granted to copy, distribute and/or modify this document
 under the terms of the GNU Free Documentation License, Version 1.3 or
@@ -14135,6 +14136,7 @@ for compatibility with an upgraded @code{flex} package.
 @table @code
 
 @item --list-transitive
+@itemx -R
 List all the packages which one or more packages depend upon.
 
 @example
-- 
2.38.1





^ permalink raw reply related	[flat|nested] 8+ messages in thread

end of thread, other threads:[~2022-11-13 16:29 UTC | newest]

Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-11-13  3:28 [bug#59235] [PATCH] gnu: %options: Add short flag -R for --list-transitive option jgart via Guix-patches via
2022-11-13  7:56 ` ( via Guix-patches via
2022-11-13 15:05 ` [bug#59235] [PATCH v2] guix: scripts: refresh: " jgart via Guix-patches via
2022-11-13 15:38 ` [bug#59235] [PATCH v3 1/2] " jgart via Guix-patches via
2022-11-13 15:38   ` [bug#59235] [PATCH v3 2/2] doc: Mention --list-transitive short flag in the manual jgart via Guix-patches via
2022-11-13 16:01   ` [bug#59235] [PATCH v3 1/2] guix: scripts: refresh: Add short flag -R for --list-transitive option ( via Guix-patches via
2022-11-13 16:27 ` [bug#59235] [PATCH v4 1/2] " jgart via Guix-patches via
2022-11-13 16:27   ` [bug#59235] [PATCH v4 2/2] doc: Mention --list-transitive short flag in the manual jgart via Guix-patches via

Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/guix.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).