* [PATCH] Fix handling of configure option --with-xorg-conf-dir
@ 2016-09-16 7:10 Michel Dänzer
[not found] ` <20160916071017.27036-1-michel-otUistvHUpPR7s880joybQ@public.gmane.org>
0 siblings, 1 reply; 2+ messages in thread
From: Michel Dänzer @ 2016-09-16 7:10 UTC (permalink / raw)
To: amd-gfx-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW
From: Michel Dänzer <michel.daenzer@amd.com>
There were two problems:
I accidentally changed the variable name in the AC_ARG_WITH stanza from
configdir to xorgconfigdir, so specifying --with-xorg-conf-dir wouldn't
work correctly. Fix this back to configdir.
If neither --with-xorg-conf-dir nor --prefix is specified on the command
line, the $prefix variable doesn't contain "/usr/local" (the default
prefix) yet at this point but "NONE". So make install would attempt to
install 10-amdgpu.conf in ${DESTDIR}NONE/share/X11/xorg.conf.d/ . Fix
this by leaving ${prefix} verbatim in the default value, to be resolved
by make.
Also print the configdir value along with the values of other similar
configuration variables.
Reported-by: Timo Aaltonen <tjaalton@debian.org>
Signed-off-by: Michel Dänzer <michel.daenzer@amd.com>
---
I'll need to push this within a few hours for the 1.1.2 release. If you
see any potential issue with any of this, please speak up.
configure.ac | 7 ++++---
1 file changed, 4 insertions(+), 3 deletions(-)
diff --git a/configure.ac b/configure.ac
index 9166c3f..68e1feb 100644
--- a/configure.ac
+++ b/configure.ac
@@ -89,9 +89,9 @@ PKG_CHECK_EXISTS([xorg-server >= 1.16],
[sysconfigdir=""])
AC_ARG_WITH(xorg-conf-dir,
AS_HELP_STRING([--with-xorg-conf-dir=DIR],
- [Default xorg.conf.d directory [[default=$prefix/share/X11/xorg.conf.d/]]]),
- [xorgconfdir="$withval"],
- [xorgconfdir="$prefix/share/X11/xorg.conf.d"])
+ [Default xorg.conf.d directory [[default=${prefix}/share/X11/xorg.conf.d]]]),
+ [configdir="$withval"],
+ [configdir='${prefix}/share/X11/xorg.conf.d'])
AC_SUBST(configdir)
AM_CONDITIONAL(HAS_XORG_CONF_DIR, [test "x$sysconfigdir" != "x"])
@@ -279,6 +279,7 @@ echo " prefix: $prefix"
echo " exec_prefix: $exec_prefix"
echo " libdir: $libdir"
echo " includedir: $includedir"
+echo " configdir: $configdir"
echo ""
echo " CFLAGS: $CFLAGS"
--
2.9.3
_______________________________________________
amd-gfx mailing list
amd-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/amd-gfx
^ permalink raw reply related [flat|nested] 2+ messages in thread
* RE: [PATCH] Fix handling of configure option --with-xorg-conf-dir
[not found] ` <20160916071017.27036-1-michel-otUistvHUpPR7s880joybQ@public.gmane.org>
@ 2016-09-16 13:26 ` Deucher, Alexander
0 siblings, 0 replies; 2+ messages in thread
From: Deucher, Alexander @ 2016-09-16 13:26 UTC (permalink / raw)
To: 'Michel Dänzer', amd-gfx-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW
> -----Original Message-----
> From: amd-gfx [mailto:amd-gfx-bounces@lists.freedesktop.org] On Behalf
> Of Michel Dänzer
> Sent: Friday, September 16, 2016 3:10 AM
> To: amd-gfx@lists.freedesktop.org
> Subject: [PATCH] Fix handling of configure option --with-xorg-conf-dir
>
> From: Michel Dänzer <michel.daenzer@amd.com>
>
> There were two problems:
>
> I accidentally changed the variable name in the AC_ARG_WITH stanza from
> configdir to xorgconfigdir, so specifying --with-xorg-conf-dir wouldn't
> work correctly. Fix this back to configdir.
>
> If neither --with-xorg-conf-dir nor --prefix is specified on the command
> line, the $prefix variable doesn't contain "/usr/local" (the default
> prefix) yet at this point but "NONE". So make install would attempt to
> install 10-amdgpu.conf in ${DESTDIR}NONE/share/X11/xorg.conf.d/ . Fix
> this by leaving ${prefix} verbatim in the default value, to be resolved
> by make.
>
> Also print the configdir value along with the values of other similar
> configuration variables.
>
> Reported-by: Timo Aaltonen <tjaalton@debian.org>
> Signed-off-by: Michel Dänzer <michel.daenzer@amd.com>
Reviewed-by: Alex Deucher <alexander.deucher@amd.com>
> ---
>
> I'll need to push this within a few hours for the 1.1.2 release. If you
> see any potential issue with any of this, please speak up.
>
> configure.ac | 7 ++++---
> 1 file changed, 4 insertions(+), 3 deletions(-)
>
> diff --git a/configure.ac b/configure.ac
> index 9166c3f..68e1feb 100644
> --- a/configure.ac
> +++ b/configure.ac
> @@ -89,9 +89,9 @@ PKG_CHECK_EXISTS([xorg-server >= 1.16],
> [sysconfigdir=""])
> AC_ARG_WITH(xorg-conf-dir,
> AS_HELP_STRING([--with-xorg-conf-dir=DIR],
> - [Default xorg.conf.d directory
> [[default=$prefix/share/X11/xorg.conf.d/]]]),
> - [xorgconfdir="$withval"],
> - [xorgconfdir="$prefix/share/X11/xorg.conf.d"])
> + [Default xorg.conf.d directory
> [[default=${prefix}/share/X11/xorg.conf.d]]]),
> + [configdir="$withval"],
> + [configdir='${prefix}/share/X11/xorg.conf.d'])
> AC_SUBST(configdir)
> AM_CONDITIONAL(HAS_XORG_CONF_DIR, [test "x$sysconfigdir" != "x"])
>
> @@ -279,6 +279,7 @@ echo " prefix: $prefix"
> echo " exec_prefix: $exec_prefix"
> echo " libdir: $libdir"
> echo " includedir: $includedir"
> +echo " configdir: $configdir"
>
> echo ""
> echo " CFLAGS: $CFLAGS"
> --
> 2.9.3
>
> _______________________________________________
> amd-gfx mailing list
> amd-gfx@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/amd-gfx
_______________________________________________
amd-gfx mailing list
amd-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/amd-gfx
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2016-09-16 13:26 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-09-16 7:10 [PATCH] Fix handling of configure option --with-xorg-conf-dir Michel Dänzer
[not found] ` <20160916071017.27036-1-michel-otUistvHUpPR7s880joybQ@public.gmane.org>
2016-09-16 13:26 ` Deucher, Alexander
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.