Hi, Maxim Cournoyer <maxim.courno...@gmail.com> skribis:
> Ludovic Courtès <l...@gnu.org> writes: > >> Hi Maxim, >> >> Maxim Cournoyer <maxim.courno...@gmail.com> skribis: >> >>> --- a/doc/guix.texi >>> +++ b/doc/guix.texi >>> @@ -2387,8 +2387,8 @@ Success, you've now booted into Guix System! From >>> then on, you can update the >>> system whenever you want by running, say: >>> >>> @example >>> -guix pull >>> -sudo guix system reconfigure /etc/config.scm >>> +sudo -i guix pull >>> +sudo -i guix system reconfigure /etc/config.scm >>> @end example >>> >>> @noindent >>> @@ -2396,14 +2396,6 @@ This builds a new system generation with the latest >>> packages and services >>> (@pxref{Invoking guix system}). We recommend doing that regularly so that >>> your system includes the latest security updates (@pxref{Security >>> Updates}). >>> >>> -@c See >>> <https://lists.gnu.org/archive/html/guix-devel/2019-01/msg00268.html>. >>> -@quotation Note >>> -@cindex sudo vs. @command{guix pull} >>> -Note that @command{sudo guix} runs your user's @command{guix} command and >>> -@emph{not} root's, because @command{sudo} leaves @code{PATH} unchanged. To >>> -explicitly run root's @command{guix}, type @command{sudo -i guix @dots{}}. >>> -@end quotation >> >> I think these bits were correct. > > They are correct when applied to a Guix System; It’s a paragraph that explains how to upgrade Guix System, so I think foreign distros should be discussed elsewhere. (I appreciate your attempt to add proper documentation about that for foreign distros!) > what my commits aim to do is make them generally true regardless of > In the commit message I wrote: > > Although this section is specific to Guix System, it is likely to be > read by users of foreign GNU/Linux distributions as well. Remove the > note, given that sudo doesn't preserve the user's PATH on foreign > distributions such as Debian. But users of foreign distros would hopefully realize that ‘guix system reconfigure’ (in the same snippet) is not for them. :-) What about adding a section that we could call “Upgrading Guix” under “Application Setup”, which is where setup on foreign distros is discussed? The section could say: 1. Run ‘guix pull’ to upgrade Guix. 2. When you need to upgrade the build daemon, for instance, which runs as root, run: sudo -i guix pull How does that sound? Thanks, Ludo’.