diff options
| author | Maxim Cournoyer <maxim.cournoyer@gmail.com> | 2024-05-17 15:06:02 -0400 |
|---|---|---|
| committer | Maxim Cournoyer <maxim.cournoyer@gmail.com> | 2024-05-25 23:28:30 -0400 |
| commit | 77771036f0970083c146b709d792d128f4235125 (patch) | |
| tree | 8fbef75735605f6c5e287a856fb9879a1008e782 | |
| parent | 94c8cec99969fe9f65777637fde1f05e1c576a3f (diff) | |
configure.ac: Set default value for the 'prefix' variable.
The Guix standard configuration uses a localstatedir of /var and a sysconfdir
of /etc. To ease things for everyone, make the default values match that
standard expected configuration. See
<https://lists.gnu.org/archive/html/guix-devel/2024-05/msg00003.html> for a
related discussion.
* configure.ac: Default $prefix to '' unless already set.
* doc/contributing.texi (Building from Git): Streamline doc.
* doc/guix-cookbook.texi (Guix environment via direnv): Likewise.
Reviewed-by: Ludovic Courtès <ludo@gnu.org>
Change-Id: I23cd12b58a842d246fbc9fdc740311c573eb0212
| -rw-r--r-- | configure.ac | 8 | ||||
| -rw-r--r-- | doc/contributing.texi | 15 | ||||
| -rw-r--r-- | doc/guix-cookbook.texi | 2 |
3 files changed, 7 insertions, 18 deletions
diff --git a/configure.ac b/configure.ac index 8c3a06da376..c7b61d56a74 100644 --- a/configure.ac +++ b/configure.ac | |||
| @@ -73,9 +73,11 @@ AC_ARG_ENABLE([daemon], | |||
| 73 | [guix_build_daemon="$enableval"], | 73 | [guix_build_daemon="$enableval"], |
| 74 | [guix_build_daemon="yes"]) | 74 | [guix_build_daemon="yes"]) |
| 75 | 75 | ||
| 76 | # Prepare a version of $localstatedir & co. that does not contain references | 76 | dnl Prepare a version of $localstatedir & co. that does not contain references |
| 77 | # to shell variables. | 77 | dnl to shell variables. Also set some sane default directory variables for |
| 78 | guix_prefix="`eval echo $prefix | sed -e"s|NONE|/usr/local|g"`" | 78 | dnl use with the Guix. This also causes localstatedir to be /var and |
| 79 | dnl sysconfdir to be /etc. | ||
| 80 | test "$prefix" = NONE && prefix= | ||
| 79 | guix_localstatedir="`eval echo $localstatedir | sed -e "s|NONE|$guix_prefix|g"`" | 81 | guix_localstatedir="`eval echo $localstatedir | sed -e "s|NONE|$guix_prefix|g"`" |
| 80 | guix_sysconfdir="`eval echo $sysconfdir | sed -e "s|NONE|$guix_prefix|g"`" | 82 | guix_sysconfdir="`eval echo $sysconfdir | sed -e "s|NONE|$guix_prefix|g"`" |
| 81 | guix_sbindir="`eval echo $sbindir | sed -e "s|NONE|$guix_prefix|g"`" | 83 | guix_sbindir="`eval echo $sbindir | sed -e "s|NONE|$guix_prefix|g"`" |
diff --git a/doc/contributing.texi b/doc/contributing.texi index a0d6b5304ab..b8adc169d80 100644 --- a/doc/contributing.texi +++ b/doc/contributing.texi | |||
| @@ -235,7 +235,7 @@ more information. | |||
| 235 | Then, run: | 235 | Then, run: |
| 236 | 236 | ||
| 237 | @example | 237 | @example |
| 238 | ./configure --localstatedir=/var --sysconfdir=/etc | 238 | ./configure |
| 239 | @end example | 239 | @end example |
| 240 | 240 | ||
| 241 | @noindent | 241 | @noindent |
| @@ -246,19 +246,6 @@ normal @code{sysconfdir} value. Note that you will probably not run | |||
| 246 | important to pass the right @code{localstatedir} and @code{sysconfdir} | 246 | important to pass the right @code{localstatedir} and @code{sysconfdir} |
| 247 | values, which get recorded in the @code{(guix config)} Guile module. | 247 | values, which get recorded in the @code{(guix config)} Guile module. |
| 248 | 248 | ||
| 249 | When configuring Guix on a system that already has a Guix installation, | ||
| 250 | be sure to specify the same state directory as the existing installation | ||
| 251 | using the @option{--localstatedir} option of the @command{configure} | ||
| 252 | script (@pxref{Directory Variables, @code{localstatedir},, standards, | ||
| 253 | GNU Coding Standards}). Usually, this @var{localstatedir} option is set | ||
| 254 | to the value @file{/var}. The @command{configure} script protects | ||
| 255 | against unintended misconfiguration of @var{localstatedir} so you do not | ||
| 256 | inadvertently corrupt your store (@pxref{The Store}). The configuration | ||
| 257 | directory should also be configured by setting the @option{--sysconfdir} | ||
| 258 | option to the @file{/etc} value, which is the location used by Guix to | ||
| 259 | store for example the access control list of authorized machines and the | ||
| 260 | definition of offload machines. | ||
| 261 | |||
| 262 | Finally, you can build Guix and, if you feel so inclined, run the tests | 249 | Finally, you can build Guix and, if you feel so inclined, run the tests |
| 263 | (@pxref{Running the Test Suite}): | 250 | (@pxref{Running the Test Suite}): |
| 264 | 251 | ||
diff --git a/doc/guix-cookbook.texi b/doc/guix-cookbook.texi index 6b6f5460c3e..93874489c23 100644 --- a/doc/guix-cookbook.texi +++ b/doc/guix-cookbook.texi | |||
| @@ -5093,7 +5093,7 @@ use_guix() | |||
| 5093 | # Predefine configure flags. | 5093 | # Predefine configure flags. |
| 5094 | configure() | 5094 | configure() |
| 5095 | @{ | 5095 | @{ |
| 5096 | ./configure --localstatedir=/var --prefix= | 5096 | ./configure |
| 5097 | @} | 5097 | @} |
| 5098 | export_function configure | 5098 | export_function configure |
| 5099 | 5099 | ||
