Re: configure --sysconfdir=/.. broken

[Andreas Jaeger <aj at suse dot de>]

>>>>> Andreas Schwab writes:

AS> Andreas Jaeger <aj@suse.de> writes:
AS> |> >>>>> Andreas Schwab writes:
AS> |> 
AS> |>  > Andreas Jaeger <aj@suse.de> writes:
AS> |>  > |> Or how can we fix this?  We could initialize libc_cv_sysconfdir to
AS> |>  > |> sysconfdir instead of leaving it empty as it's done now - but this way
AS> |>  > |> the user can't override libc_cv_sysconfdir when using --prefix=/usr
AS> |>  > |> and I don't see directly how we can check if the user has used the
AS> |>  > |> prefix.
AS> |> 
AS> |>  > If prefix = NONE then --prefix wasn't used.  This is true until AC_OUTPUT
AS> |>  > is called.
AS> |> 
AS> |> Hi Andreas,
AS> |> 
AS> |> I don't understand what you suggest.  
AS> |> 
AS> |> Imagine the following configure lines:
AS> |> - configure --prefix=/usr -> sysconfdir will be /etc
AS> |> - configure --sysconfdir=/etc -> no prefix, sysconfdir should be /etc
AS> |>   but is indeed currently /usr/local/etc
AS> |> - configure --prefix=/test --sysconfdir=/etc -> sysconfdir gets set to
AS> |>   /test/etc

AS> How about this:

AS> if test $sysconfdir = '${prefix}/etc'; then
AS>   libc_cv_sysconfdir=/etc
AS> else
AS>   libc_cv_sysconfdir=$sysconfdir
AS> fi

This works fine, thanks a lot.  I'm appending a patch which implements
this.  Uli, may I commit this?

Andreas

2000-06-08  Andreas Jaeger  <aj@suse.de>

	* sysdeps/unix/sysv/linux/configure.in: Set libc_cv_sysconfdir
	depending on the configure option --sysconfdir.
        Patch by Andreas Schwab <schwab@suse.de>.

============================================================
Index: sysdeps/unix/sysv/linux/configure.in
--- sysdeps/unix/sysv/linux/configure.in	2000/03/21 05:40:49	1.30
+++ sysdeps/unix/sysv/linux/configure.in	2000/06/08 13:39:34
@@ -74,7 +74,12 @@
   else
     libc_cv_slibdir="/lib"
   fi
-  libc_cv_sysconfdir="/etc"
+  # Allow the user to override the path with --sysconfdir
+  if test $sysconfdir = '${prefix}/etc'; then
+    libc_cv_sysconfdir=/etc
+  else
+    libc_cv_sysconfdir=$sysconfdir
+   fi
   libc_cv_rootsbindir="/sbin"
 fi
 

-- 
 Andreas Jaeger
  SuSE Labs aj@suse.de
   private aj@arthur.inka.de