GNU bug report logs - #47401
[PATCH] doc: Mention value /etc to sysconfdir option.

Previous Next

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 47401 in the body.
You can then email your comments to 47401 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):

From: zimoun <zimon.toutoune <at> gmail.com>
To: guix-patches <at> gnu.org
Cc: zimoun <zimon.toutoune <at> gmail.com>
Subject: [PATCH] doc: Mention value /etc to sysconfdir option.
Date: Fri, 26 Mar 2021 07:01:38 +0100

* doc/contributing.texi (Building from Git): Mention value /etc to sysconfdir
option.
---

Hi,

From a discussion [1] about a reproducibility issue,

        As discussed on IRC, ‘guix pull’ preserves the settings of the host
        Guix.  So if your initial ‘guix’ has %sysconfdir set to
        /usr/local/etc, ‘guix pull’ will preserve that.

        It turns out that, as is always the case with GNU Standards compliant
        configure script, the default value for --prefix is /usr/local, and
        the default for --sysconfdir is $prefix/etc.

        Mostly likely, what happened is that at some point you built Guix from
        source using the default prefix and sysconfdir, and then you ran ‘guix
        pull’ from that Guix.

I discovered that the option ’sysconfdir’ at ./configure time is set by
default to /usr/local/etc conforming the GNU Coding standards.  And Guix
expect instead /etc.  As 'localstadir', it is unconventional.

The manual does not mention this subtility.  This patch adds a tiny sentence
to avoid the trap for the attentive reader.

1: <https://yhetil.org/guix/877dnqe3to.fsf <at> gnu.org>


Thanks,
simon

 doc/contributing.texi | 8 ++++++--
 doc/guix.texi         | 2 +-
 2 files changed, 7 insertions(+), 3 deletions(-)

diff --git a/doc/contributing.texi b/doc/contributing.texi
index 58c4dafb9b..53840d81cc 100644
--- a/doc/contributing.texi
+++ b/doc/contributing.texi
@@ -228,8 +228,12 @@ necessary to support this, including @env{PATH} and @env{GUILE_LOAD_PATH}.
 Note that @command{./pre-inst-env guix pull} does @emph{not} upgrade the
 local source tree; it simply updates the @file{~/.config/guix/current}
 symlink (@pxref{Invoking guix pull}).  Run @command{git pull} instead if
-you want to upgrade your local source tree.
-
+you want to upgrade your local source tree.  Moreover, note also that
+@command{guix pull} preserves the settings of the host Guix.  For
+instance, regular Guix passes @code{--sysconfdir=/etc/}, whereas the GNU
+Coding Standards sets by default @var{prefix} to @file{/usr/local} and
+@var{sysconfdir} to @file{$prefix/etc} (@pxref{Directory Variables,,,
+standards, GNU Coding Standards}).
 
 @node The Perfect Setup
 @section The Perfect Setup
diff --git a/doc/guix.texi b/doc/guix.texi
index c23d044ff5..3f2cf4d6e1 100644
--- a/doc/guix.texi
+++ b/doc/guix.texi
@@ -69,8 +69,8 @@ Copyright @copyright{} 2019 Jakob L. Kreuze@*
 Copyright @copyright{} 2019 Kyle Andrews@*
 Copyright @copyright{} 2019 Alex Griffin@*
 Copyright @copyright{} 2019, 2020 Guillaume Le Vaillant@*
+Copyright @copyright{} 2019, 2020, 2021 Simon Tournier@*
 Copyright @copyright{} 2020 Leo Prikler@*
-Copyright @copyright{} 2019, 2020 Simon Tournier@*
 Copyright @copyright{} 2020 Wiktor Żelazny@*
 Copyright @copyright{} 2020 Damien Cassou@*
 Copyright @copyright{} 2020 Jakub Kądziołka@*

base-commit: 6e7ba45357078b31a369b23f8a9f38302dfcbb10
-- 
2.29.2

Message #8 received at 47401 <at> debbugs.gnu.org (full text, mbox):

From: Christopher Baines <mail <at> cbaines.net>
To: zimoun <zimon.toutoune <at> gmail.com>
Cc: 47401 <at> debbugs.gnu.org
Subject: Re: [bug#47401] [PATCH] doc: Mention value /etc to sysconfdir option.
Date: Wed, 31 Mar 2021 08:10:40 +0100

[Message part 1 (text/plain, inline)]

zimoun <zimon.toutoune <at> gmail.com> writes:

> * doc/contributing.texi (Building from Git): Mention value /etc to sysconfdir
> option.
> ---
>
> Hi,
>
> From a discussion [1] about a reproducibility issue,
>
>         As discussed on IRC, ‘guix pull’ preserves the settings of the host
>         Guix.  So if your initial ‘guix’ has %sysconfdir set to
>         /usr/local/etc, ‘guix pull’ will preserve that.
>
>         It turns out that, as is always the case with GNU Standards compliant
>         configure script, the default value for --prefix is /usr/local, and
>         the default for --sysconfdir is $prefix/etc.
>
>         Mostly likely, what happened is that at some point you built Guix from
>         source using the default prefix and sysconfdir, and then you ran ‘guix
>         pull’ from that Guix.
>
> I discovered that the option ’sysconfdir’ at ./configure time is set by
> default to /usr/local/etc conforming the GNU Coding standards.  And Guix
> expect instead /etc.  As 'localstadir', it is unconventional.
>
> The manual does not mention this subtility.  This patch adds a tiny sentence
> to avoid the trap for the attentive reader.
>
> 1: <https://yhetil.org/guix/877dnqe3to.fsf <at> gnu.org>
>
>
> Thanks,
> simon
>
>  doc/contributing.texi | 8 ++++++--
>  doc/guix.texi         | 2 +-
>  2 files changed, 7 insertions(+), 3 deletions(-)
>
> diff --git a/doc/contributing.texi b/doc/contributing.texi
> index 58c4dafb9b..53840d81cc 100644
> --- a/doc/contributing.texi
> +++ b/doc/contributing.texi
> @@ -228,8 +228,12 @@ necessary to support this, including @env{PATH} and @env{GUILE_LOAD_PATH}.
>  Note that @command{./pre-inst-env guix pull} does @emph{not} upgrade the
>  local source tree; it simply updates the @file{~/.config/guix/current}
>  symlink (@pxref{Invoking guix pull}).  Run @command{git pull} instead if
> -you want to upgrade your local source tree.
> -
> +you want to upgrade your local source tree.  Moreover, note also that
> +@command{guix pull} preserves the settings of the host Guix.  For
> +instance, regular Guix passes @code{--sysconfdir=/etc/}, whereas the GNU
> +Coding Standards sets by default @var{prefix} to @file{/usr/local} and
> +@var{sysconfdir} to @file{$prefix/etc} (@pxref{Directory Variables,,,
> +standards, GNU Coding Standards}).

I think this is maybe useful to note somewhere, but not to add to the
existing paragraph here.

The paragraph starting on line 228 looks to me to be aimed at new users
who might be at risk of confusing guix pull and git pull, whereas the
bit you're adding is about something completely different, explaining a
subtlety of running guix pull (something an earlier part of the
paragraph is saying not to run).

Maybe there is a more appropriate place in the main manual?

[signature.asc (application/pgp-signature, inline)]

Message #13 received at 47401-done <at> debbugs.gnu.org (full text, mbox):

From: Maxim Cournoyer <maxim.cournoyer <at> gmail.com>
To: "jgart" <jgart <at> dismail.de>
Cc: Mekeor Melire <mekeor <at> posteo.de>, help-guix <at> gnu.org,
 47401-done <at> debbugs.gnu.org
Subject: Re: guix build: error: gcry_md_hash_buffer: Function not implemented
Date: Mon, 15 May 2023 09:40:28 -0400

Hi!

"jgart" <jgart <at> dismail.de> writes:

> Hi,
>
> Can this be closed now given the recent mention of --sysconfdir?
>
> https://issues.guix.gnu.org/47401#1

I think so.  Done!

-- 
Thanks,
Maxim

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.