unofficial mirror of guix-patches@gnu.org 
 help / color / mirror / Atom feed
* [bug#43361] [PATCH] doc: Mention config.scm in reconfigure description
@ 2020-09-12 19:48 Julien Lepiller
  2020-09-13 12:06 ` Andreas Enge
  2020-09-13 21:10 ` Ludovic Courtès
  0 siblings, 2 replies; 5+ messages in thread
From: Julien Lepiller @ 2020-09-12 19:48 UTC (permalink / raw)
  To: 43361

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

Hi Guix,

this patch is a small change to the manual that should help at least
one confused user from
https://unix.stackexchange.com/questions/550597/what-is-the-file-to-be-specified-in-guix-system-reconfigure

[-- Attachment #2: 0001-doc-Mention-config.scm-in-reconfigure-description.patch --]
[-- Type: text/x-patch, Size: 1318 bytes --]

From dfa9439efbf1de7ffe281dac330a19c1e369e5d8 Mon Sep 17 00:00:00 2001
From: Julien Lepiller <julien@lepiller.eu>
Date: Sat, 12 Sep 2020 21:46:57 +0200
Subject: [PATCH] doc: Mention config.scm in reconfigure description

* doc/guix.texi (Invoking guix system): Mention `/etc/config.scm' in the
description of guix system reconfigure.
---
 doc/guix.texi | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/doc/guix.texi b/doc/guix.texi
index a6e14ea177..c10387441e 100644
--- a/doc/guix.texi
+++ b/doc/guix.texi
@@ -28919,10 +28919,10 @@ As for @command{guix package --search}, the result is written in
 (@pxref{Top, GNU recutils databases,, recutils, GNU recutils manual}).
 
 @item reconfigure
-Build the operating system described in @var{file}, activate it, and
-switch to it@footnote{This action (and the related actions
-@code{switch-generation} and @code{roll-back}) are usable only on
-systems already running Guix System.}.
+Build the operating system described in @var{file} (typically
+@file{/etc/config.scm}), activate it, and switch to it@footnote{This action
+(and the related actions @code{switch-generation} and @code{roll-back}) are
+usable only on systems already running Guix System.}.
 
 @quotation Note
 @c The paragraph below refers to the problem discussed at
-- 
2.28.0


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

* [bug#43361] [PATCH] doc: Mention config.scm in reconfigure description
  2020-09-12 19:48 [bug#43361] [PATCH] doc: Mention config.scm in reconfigure description Julien Lepiller
@ 2020-09-13 12:06 ` Andreas Enge
  2020-09-13 21:10 ` Ludovic Courtès
  1 sibling, 0 replies; 5+ messages in thread
From: Andreas Enge @ 2020-09-13 12:06 UTC (permalink / raw)
  To: Julien Lepiller; +Cc: 43361

Hello,

On Sat, Sep 12, 2020 at 09:48:52PM +0200, Julien Lepiller wrote:
>  @item reconfigure
> -Build the operating system described in @var{file}, activate it, and
> -switch to it@footnote{This action (and the related actions
> -@code{switch-generation} and @code{roll-back}) are usable only on
> -systems already running Guix System.}.
> +Build the operating system described in @var{file} (typically
> +@file{/etc/config.scm}), activate it, and switch to it@footnote{This action
> +(and the related actions @code{switch-generation} and @code{roll-back}) are
> +usable only on systems already running Guix System.}.

to be honest, I find it less clear than before - it now almost sounds as if
providing the file were optional, and that the command might search it
as /etc/config.scm.

Andreas





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

* [bug#43361] [PATCH] doc: Mention config.scm in reconfigure description
  2020-09-12 19:48 [bug#43361] [PATCH] doc: Mention config.scm in reconfigure description Julien Lepiller
  2020-09-13 12:06 ` Andreas Enge
@ 2020-09-13 21:10 ` Ludovic Courtès
  2020-09-13 23:51   ` Julien Lepiller
  1 sibling, 1 reply; 5+ messages in thread
From: Ludovic Courtès @ 2020-09-13 21:10 UTC (permalink / raw)
  To: Julien Lepiller; +Cc: 43361

Hi,

Julien Lepiller <julien@lepiller.eu> skribis:

> this patch is a small change to the manual that should help at least
> one confused user from
> https://unix.stackexchange.com/questions/550597/what-is-the-file-to-be-specified-in-guix-system-reconfigure
>
>>From dfa9439efbf1de7ffe281dac330a19c1e369e5d8 Mon Sep 17 00:00:00 2001
> From: Julien Lepiller <julien@lepiller.eu>
> Date: Sat, 12 Sep 2020 21:46:57 +0200
> Subject: [PATCH] doc: Mention config.scm in reconfigure description
>
> * doc/guix.texi (Invoking guix system): Mention `/etc/config.scm' in the
> description of guix system reconfigure.
> ---
>  doc/guix.texi | 8 ++++----
>  1 file changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/doc/guix.texi b/doc/guix.texi
> index a6e14ea177..c10387441e 100644
> --- a/doc/guix.texi
> +++ b/doc/guix.texi
> @@ -28919,10 +28919,10 @@ As for @command{guix package --search}, the result is written in
>  (@pxref{Top, GNU recutils databases,, recutils, GNU recutils manual}).
>  
>  @item reconfigure
> -Build the operating system described in @var{file}, activate it, and
> -switch to it@footnote{This action (and the related actions
> -@code{switch-generation} and @code{roll-back}) are usable only on
> -systems already running Guix System.}.
> +Build the operating system described in @var{file} (typically
> +@file{/etc/config.scm}), activate it, and switch to it@footnote{This action
> +(and the related actions @code{switch-generation} and @code{roll-back}) are
> +usable only on systems already running Guix System.}.

Perhaps “(typically @file{/etc/config.scm})” should go next to
“@var{file} must be …”?

But if the person doesn’t read more than a few lines of the page, it
won’t help.

I was thinking we could have “guix system reconfigure” without arguments
pick up /run/current-system/configuration.scm.  Perhaps that’d help?

Thanks,
Ludo’.




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

* [bug#43361] [PATCH] doc: Mention config.scm in reconfigure description
  2020-09-13 21:10 ` Ludovic Courtès
@ 2020-09-13 23:51   ` Julien Lepiller
  2020-09-14  7:25     ` Ludovic Courtès
  0 siblings, 1 reply; 5+ messages in thread
From: Julien Lepiller @ 2020-09-13 23:51 UTC (permalink / raw)
  To: Ludovic Courtès; +Cc: 43361



Le 13 septembre 2020 17:10:30 GMT-04:00, "Ludovic Courtès" <ludo@gnu.org> a écrit :
>Hi,
>
>Julien Lepiller <julien@lepiller.eu> skribis:
>
>> this patch is a small change to the manual that should help at least
>> one confused user from
>>
>https://unix.stackexchange.com/questions/550597/what-is-the-file-to-be-specified-in-guix-system-reconfigure
>>
>>>From dfa9439efbf1de7ffe281dac330a19c1e369e5d8 Mon Sep 17 00:00:00
>2001
>> From: Julien Lepiller <julien@lepiller.eu>
>> Date: Sat, 12 Sep 2020 21:46:57 +0200
>> Subject: [PATCH] doc: Mention config.scm in reconfigure description
>>
>> * doc/guix.texi (Invoking guix system): Mention `/etc/config.scm' in
>the
>> description of guix system reconfigure.
>> ---
>>  doc/guix.texi | 8 ++++----
>>  1 file changed, 4 insertions(+), 4 deletions(-)
>>
>> diff --git a/doc/guix.texi b/doc/guix.texi
>> index a6e14ea177..c10387441e 100644
>> --- a/doc/guix.texi
>> +++ b/doc/guix.texi
>> @@ -28919,10 +28919,10 @@ As for @command{guix package --search}, the
>result is written in
>>  (@pxref{Top, GNU recutils databases,, recutils, GNU recutils
>manual}).
>>  
>>  @item reconfigure
>> -Build the operating system described in @var{file}, activate it, and
>> -switch to it@footnote{This action (and the related actions
>> -@code{switch-generation} and @code{roll-back}) are usable only on
>> -systems already running Guix System.}.
>> +Build the operating system described in @var{file} (typically
>> +@file{/etc/config.scm}), activate it, and switch to it@footnote{This
>action
>> +(and the related actions @code{switch-generation} and
>@code{roll-back}) are
>> +usable only on systems already running Guix System.}.
>
>Perhaps “(typically @file{/etc/config.scm})” should go next to
>“@var{file} must be …”?
>
>But if the person doesn’t read more than a few lines of the page, it
>won’t help.
>
>I was thinking we could have “guix system reconfigure” without
>arguments
>pick up /run/current-system/configuration.scm.  Perhaps that’d help?

I think it would be even more confusing, because that file isn't even editable. So you'de get users thinking they must re-install the distro to change that file.

Others would eventually find /etc/config.scm and not pass it as argument, then get their system reconfigured to something different than what they specified.

Maybe it should be "e.g. /etc/config, initially generated by the graphical installer" or something?

>
>Thanks,
>Ludo’.




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

* [bug#43361] [PATCH] doc: Mention config.scm in reconfigure description
  2020-09-13 23:51   ` Julien Lepiller
@ 2020-09-14  7:25     ` Ludovic Courtès
  0 siblings, 0 replies; 5+ messages in thread
From: Ludovic Courtès @ 2020-09-14  7:25 UTC (permalink / raw)
  To: Julien Lepiller; +Cc: 43361

Hi,

Julien Lepiller <julien@lepiller.eu> skribis:

> Le 13 septembre 2020 17:10:30 GMT-04:00, "Ludovic Courtès" <ludo@gnu.org> a écrit :
>>Hi,
>>
>>Julien Lepiller <julien@lepiller.eu> skribis:
>>
>>> this patch is a small change to the manual that should help at least
>>> one confused user from
>>>
>>https://unix.stackexchange.com/questions/550597/what-is-the-file-to-be-specified-in-guix-system-reconfigure
>>>
>>>>From dfa9439efbf1de7ffe281dac330a19c1e369e5d8 Mon Sep 17 00:00:00
>>2001
>>> From: Julien Lepiller <julien@lepiller.eu>
>>> Date: Sat, 12 Sep 2020 21:46:57 +0200
>>> Subject: [PATCH] doc: Mention config.scm in reconfigure description
>>>
>>> * doc/guix.texi (Invoking guix system): Mention `/etc/config.scm' in
>>the
>>> description of guix system reconfigure.
>>> ---
>>>  doc/guix.texi | 8 ++++----
>>>  1 file changed, 4 insertions(+), 4 deletions(-)
>>>
>>> diff --git a/doc/guix.texi b/doc/guix.texi
>>> index a6e14ea177..c10387441e 100644
>>> --- a/doc/guix.texi
>>> +++ b/doc/guix.texi
>>> @@ -28919,10 +28919,10 @@ As for @command{guix package --search}, the
>>result is written in
>>>  (@pxref{Top, GNU recutils databases,, recutils, GNU recutils
>>manual}).
>>>  
>>>  @item reconfigure
>>> -Build the operating system described in @var{file}, activate it, and
>>> -switch to it@footnote{This action (and the related actions
>>> -@code{switch-generation} and @code{roll-back}) are usable only on
>>> -systems already running Guix System.}.
>>> +Build the operating system described in @var{file} (typically
>>> +@file{/etc/config.scm}), activate it, and switch to it@footnote{This
>>action
>>> +(and the related actions @code{switch-generation} and
>>@code{roll-back}) are
>>> +usable only on systems already running Guix System.}.
>>
>>Perhaps “(typically @file{/etc/config.scm})” should go next to
>>“@var{file} must be …”?
>>
>>But if the person doesn’t read more than a few lines of the page, it
>>won’t help.
>>
>>I was thinking we could have “guix system reconfigure” without
>>arguments
>>pick up /run/current-system/configuration.scm.  Perhaps that’d help?
>
> I think it would be even more confusing, because that file isn't even editable. So you'de get users thinking they must re-install the distro to change that file.

I mean you could still pass a file as an argument, but if you don’t,
it’d pick that one.

> Others would eventually find /etc/config.scm and not pass it as argument, then get their system reconfigured to something different than what they specified.
>
> Maybe it should be "e.g. /etc/config, initially generated by the graphical installer" or something?

I think “typically /etc/config.scm” is enough (and more accurate),
dunno.

Ludo’.




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

end of thread, other threads:[~2020-09-14  7:30 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-09-12 19:48 [bug#43361] [PATCH] doc: Mention config.scm in reconfigure description Julien Lepiller
2020-09-13 12:06 ` Andreas Enge
2020-09-13 21:10 ` Ludovic Courtès
2020-09-13 23:51   ` Julien Lepiller
2020-09-14  7:25     ` Ludovic Courtès

unofficial mirror of guix-patches@gnu.org 

This inbox may be cloned and mirrored by anyone:

	git clone --mirror https://yhetil.org/guix-patches/1 guix-patches/git/1.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 guix-patches guix-patches/ https://yhetil.org/guix-patches \
		guix-patches@gnu.org
	public-inbox-index guix-patches

Example config snippet for mirrors.
Newsgroup available over NNTP:
	nntp://news.yhetil.org/yhetil.gnu.guix.patches


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git