Re: 2 (possible) problems in documentation

[Ludovic Courtès]

Alex Vong  skribis:

> From e562ebc9a9e3859ecc0c1c2e8be6f39a44b5be7d Mon Sep 17 00:00:00 2001
> From: Alex Vong 
> Date: Mon, 2 Nov 2015 23:34:16 +0800
> Subject: [PATCH] doc: To clarify `./pre-inst-env guix pull` won't upgrade
>  local source tree.
>
> * doc/contributing.texi (Running Guix Before It Is Installed): To clarify
> `./pre-inst-env guix pull` won't upgrade local source tree
> and add a footnote to explain why (thanks to Alex Kost).

I pushed a slightly modified version of this patch.  Thanks!

Ludo’.

Re: 2 (possible) problems in documentation

[Alex Vong]

Hi,

I have written a patch based on the old patch. Does it look good to
you? By the way, does anyone know how to build the guix website from
source? I want to add some labels so that librejs don't complain the
javascripts being non-free on this page
. I have cloned the
guix-artwork repo but I don't know how to set-up or build the websites
(I have zero experience on building websites.).

Cheers,
Alex

On 01/11/2015, Mathieu Lirzin  wrote:
> Hi again,
>
> Mathieu Lirzin  writes:
>
>> What do you think of the following patch?
>>
>>>From ee2f4467d84ad9516b14c7bd14f821e4bec443cc Mon Sep 17 00:00:00 2001
>> From: Mathieu Lirzin 
>> Date: Sun, 1 Nov 2015 16:27:07 +0100
>> Subject: [PATCH] doc: Add an exception in "Running Guix Before It Is
>>  Installed".
>>
>> * doc/contributing.texi (Running Guix Before It Is Installed): Add an
>>   exception footnote for `guix pull'.
>>
>> Suggested-by: Alex Vong 
>
> Please ignore this patch, my mind was confused. :)
>
> --
> Mathieu Lrzin
>
>
From e562ebc9a9e3859ecc0c1c2e8be6f39a44b5be7d Mon Sep 17 00:00:00 2001
From: Alex Vong 
Date: Mon, 2 Nov 2015 23:34:16 +0800
Subject: [PATCH] doc: To clarify `./pre-inst-env guix pull` won't upgrade
 local source tree.

* doc/contributing.texi (Running Guix Before It Is Installed): To clarify
`./pre-inst-env guix pull` won't upgrade local source tree
and add a footnote to explain why (thanks to Alex Kost).
---
 doc/contributing.texi | 6 ++
 1 file changed, 6 insertions(+)

diff --git a/doc/contributing.texi b/doc/contributing.texi
index f855daf..6423d85 100644
--- a/doc/contributing.texi
+++ b/doc/contributing.texi
@@ -82,6 +82,12 @@ $ sudo ./pre-inst-env guix-daemon --build-users-group=guixbuild
 $ ./pre-inst-env guix build hello
 @end example
 
+However, note that @command{./pre-inst-env guix pull} will not upgrade your
+local source tree@footnote{@code{guix pull} fetches the latest guix source,
+compiles it, put it to the store and link "~/.config/guix/latest" to it.}.
+You should run @command{git pull} instead
+if you want to upgrade your local source tree.
+
 @noindent
 Similarly, for a Guile session using the Guix modules:
 
-- 
2.1.4

Re: 2 (possible) problems in documentation

[Mathieu Lirzin]

Hi again,

Mathieu Lirzin  writes:

> What do you think of the following patch?
>
>>From ee2f4467d84ad9516b14c7bd14f821e4bec443cc Mon Sep 17 00:00:00 2001
> From: Mathieu Lirzin 
> Date: Sun, 1 Nov 2015 16:27:07 +0100
> Subject: [PATCH] doc: Add an exception in "Running Guix Before It Is
>  Installed".
>
> * doc/contributing.texi (Running Guix Before It Is Installed): Add an
>   exception footnote for `guix pull'.
>
> Suggested-by: Alex Vong 

Please ignore this patch, my mind was confused. :)

--
Mathieu Lrzin

Re: 2 (possible) problems in documentation

[Alex Kost]

Alex Vong (2015-11-01 06:22 +0300) wrote:

> On 01/11/2015, Alex Kost  wrote:
>> Alex Vong (2015-10-31 14:27 +0300) wrote:
[...]
>>> 2. In
>>> ,
>>> it said "To that end, all the command-line tools can be used even if
>>> you have not run make install.". However, in the case of "guix pull",
>>> it is not true. Even if "guix pull" does run properly, it will not
>>> update the source tree you have downloaded, so you end up with running
>>> the original compiled version of Guix. Obviously, I was a victim :)
>>
>> Sorry, I don't understand what you mean here.  "guix pull" can't update
>> the source tree you downloaded.  It fetches the latest guix source,
>> compiles it, put it to the store and link "~/.config/guix/latest" to it.
>> Then the Guix code from this directory when you run "guix ..." commands.
>>
>> If you use the described "pre-inst-env" method, the code from the git
>> directory is used instead, and running "guix pull" doesn't effect
>> ".../pre-inst-env guix" commands in any way.  Here instead of "guix
>> pull" you use "git pull" and "make".
>>
> What I mean is "guix pull" only works after Guix is installed, so what
> the documentation says, "To that end, all the command-line tools can
> be used even if you have not run make install", is inaccurate. Should
> we note this in the documentation?

IMO it is accurate, because "./pre-inst-env guix pull" also works; it
just doesn't work the way you thought.  But if you find the
documentation confusing you are welcome to improve it :-)

-- 
Alex

Re: 2 (possible) problems in documentation

[Alex Vong]

On 01/11/2015, Alex Kost  wrote:
> Alex Vong (2015-10-31 14:27 +0300) wrote:
>
>> Hi,
>>
>> I hit several problems when I followed the documentation. Please see
>> if it is a real issue or just my misunderstandings.
>>
>> 1. In ,
>> it saids "Essential font packages include gs-fonts, font-dejavu, and
>> font-gnu-freefont.". But I think "font-gnu-freefont" has been renamed
>> to "font-gnu-freefont-ttf", so this has to be changed.
>
> I think this on-line manual is for the current version (0.8.3).  The
> typo you mention was fixed in the git repo:
> 
>
Thanks! I wasn't aware that the web version not being up to date.

>> 2. In
>> ,
>> it said "To that end, all the command-line tools can be used even if
>> you have not run make install.". However, in the case of "guix pull",
>> it is not true. Even if "guix pull" does run properly, it will not
>> update the source tree you have downloaded, so you end up with running
>> the original compiled version of Guix. Obviously, I was a victim :)
>
> Sorry, I don't understand what you mean here.  "guix pull" can't update
> the source tree you downloaded.  It fetches the latest guix source,
> compiles it, put it to the store and link "~/.config/guix/latest" to it.
> Then the Guix code from this directory when you run "guix ..." commands.
>
> If you use the described "pre-inst-env" method, the code from the git
> directory is used instead, and running "guix pull" doesn't effect
> ".../pre-inst-env guix" commands in any way.  Here instead of "guix
> pull" you use "git pull" and "make".
>
What I mean is "guix pull" only works after Guix is installed, so what
the documentation says, "To that end, all the command-line tools can
be used even if you have not run make install", is inaccurate. Should
we note this in the documentation?

> --
> Alex
>

Cheers,
Alex

2 (possible) problems in documentation

[Alex Vong]

Hi,

I hit several problems when I followed the documentation. Please see
if it is a real issue or just my misunderstandings.

1. In ,
it saids "Essential font packages include gs-fonts, font-dejavu, and
font-gnu-freefont.". But I think "font-gnu-freefont" has been renamed
to "font-gnu-freefont-ttf", so this has to be changed.

2. In 
,
it said "To that end, all the command-line tools can be used even if
you have not run make install.". However, in the case of "guix pull",
it is not true. Even if "guix pull" does run properly, it will not
update the source tree you have downloaded, so you end up with running
the original compiled version of Guix. Obviously, I was a victim :)

Does these sound like problems to you?

Cheers,
Alex

Re: 2 (possible) problems in documentation

[Alex Kost]

Alex Vong (2015-10-31 14:27 +0300) wrote:

> Hi,
>
> I hit several problems when I followed the documentation. Please see
> if it is a real issue or just my misunderstandings.
>
> 1. In ,
> it saids "Essential font packages include gs-fonts, font-dejavu, and
> font-gnu-freefont.". But I think "font-gnu-freefont" has been renamed
> to "font-gnu-freefont-ttf", so this has to be changed.

I think this on-line manual is for the current version (0.8.3).  The
typo you mention was fixed in the git repo:


> 2. In
> ,
> it said "To that end, all the command-line tools can be used even if
> you have not run make install.". However, in the case of "guix pull",
> it is not true. Even if "guix pull" does run properly, it will not
> update the source tree you have downloaded, so you end up with running
> the original compiled version of Guix. Obviously, I was a victim :)

Sorry, I don't understand what you mean here.  "guix pull" can't update
the source tree you downloaded.  It fetches the latest guix source,
compiles it, put it to the store and link "~/.config/guix/latest" to it.
Then the Guix code from this directory when you run "guix ..." commands.

If you use the described "pre-inst-env" method, the code from the git
directory is used instead, and running "guix pull" doesn't effect
".../pre-inst-env guix" commands in any way.  Here instead of "guix
pull" you use "git pull" and "make".

-- 
Alex