unofficial mirror of bug-guix@gnu.org 
 help / color / mirror / code / Atom feed
* bug#71952: documentation inaccuracy?
@ 2024-07-05 18:22 chris
  2024-07-06 13:33 ` jgart via Bug reports for GNU Guix
  2024-07-09 17:08 ` Simon Tournier
  0 siblings, 2 replies; 5+ messages in thread
From: chris @ 2024-07-05 18:22 UTC (permalink / raw)
  To: 71952; +Cc: chris

This section of the documentation seems innacurate,

https://guix.gnu.org/manual/devel/en/html_node/Invoking-guix-home.html#index-home-generations

> When specifying a negative value such as -1, you must precede it with -- to prevent it from being parsed as an option. For example:
>
> guix home switch-generation -- -1

Running the same command here gives the following result

> $ guix home switch-generation -- -1
>
> guix home: error: -1: extraneous command

Instead, `guix home roll-back` and `guix home switch-generation +1` can be used.




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

* bug#71952: documentation inaccuracy?
  2024-07-05 18:22 bug#71952: documentation inaccuracy? chris
@ 2024-07-06 13:33 ` jgart via Bug reports for GNU Guix
  2024-07-09 17:08 ` Simon Tournier
  1 sibling, 0 replies; 5+ messages in thread
From: jgart via Bug reports for GNU Guix @ 2024-07-06 13:33 UTC (permalink / raw)
  To: 71952; +Cc: chris, Simon Tournier


hi chris,

would you like to send a patch fixing this?

hi simon,

wdyt?
-- 
all the best,
jgart




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

* bug#71952: documentation inaccuracy?
  2024-07-05 18:22 bug#71952: documentation inaccuracy? chris
  2024-07-06 13:33 ` jgart via Bug reports for GNU Guix
@ 2024-07-09 17:08 ` Simon Tournier
  2024-07-09 18:58   ` jgart via Bug reports for GNU Guix
  1 sibling, 1 reply; 5+ messages in thread
From: Simon Tournier @ 2024-07-09 17:08 UTC (permalink / raw)
  To: chris, 71952; +Cc: chris, jgart, Andrew Tropin

[-- Attachment #1: Type: text/plain, Size: 378 bytes --]

Hi,

>> $ guix home switch-generation -- -1
>> guix home: error: -1: extraneous command

Indeed, the command-line parser seems incorrect and inconsistent with
the documentation.

The culprit seems the procedure ’parse-args’ from ’guix-home’ command
inside the module (guix scripts home).

Hum, I guess something as the attached patch should fix the issue.


[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: patch --]
[-- Type: text/x-diff, Size: 422 bytes --]

diff --git a/guix/scripts/home.scm b/guix/scripts/home.scm
index b4c82d275f..5c39e83983 100644
--- a/guix/scripts/home.scm
+++ b/guix/scripts/home.scm
@@ -682,6 +682,8 @@ (define-command (guix-home . args)
       (match rest
         (() opts)
         (("--") opts)
+        (("--" n)
+         (alist-cons 'argument n opts))
         (("--" command ...)
          (match (assoc-ref opts 'action)
            ('container

[-- Attachment #3: Type: text/plain, Size: 22 bytes --]


WDYT?

Cheers,
simon

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

* bug#71952: documentation inaccuracy?
  2024-07-09 17:08 ` Simon Tournier
@ 2024-07-09 18:58   ` jgart via Bug reports for GNU Guix
  2024-07-31 13:43     ` Andrew Tropin via Bug reports for GNU Guix
  0 siblings, 1 reply; 5+ messages in thread
From: jgart via Bug reports for GNU Guix @ 2024-07-09 18:58 UTC (permalink / raw)
  To: Simon Tournier, chris, 71952; +Cc: chris, Andrew Tropin

LGTM, Andrew WDYT?

--
jgart




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

* bug#71952: documentation inaccuracy?
  2024-07-09 18:58   ` jgart via Bug reports for GNU Guix
@ 2024-07-31 13:43     ` Andrew Tropin via Bug reports for GNU Guix
  0 siblings, 0 replies; 5+ messages in thread
From: Andrew Tropin via Bug reports for GNU Guix @ 2024-07-31 13:43 UTC (permalink / raw)
  To: jgart, Simon Tournier, chris, 71952; +Cc: chris

[-- Attachment #1: Type: text/plain, Size: 116 bytes --]


> LGTM, Andrew WDYT?

Didn't try the patch, but overall idea seems right.

-- 
Best regards,
Andrew Tropin

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 832 bytes --]

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

end of thread, other threads:[~2024-07-31 13:59 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-07-05 18:22 bug#71952: documentation inaccuracy? chris
2024-07-06 13:33 ` jgart via Bug reports for GNU Guix
2024-07-09 17:08 ` Simon Tournier
2024-07-09 18:58   ` jgart via Bug reports for GNU Guix
2024-07-31 13:43     ` Andrew Tropin via Bug reports for GNU Guix

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).