* [PATCH i-g-t v2] configure.ac: Disable chamelium by default and add enable argument
@ 2017-07-26 8:11 Paul Kocialkowski
2017-07-26 16:02 ` Lyude Paul
0 siblings, 1 reply; 2+ messages in thread
From: Paul Kocialkowski @ 2017-07-26 8:11 UTC (permalink / raw)
To: intel-gfx
Since the chamelium is not a very usual piece of hardware and requires
pulling-in lots of specific dependencies, it makes sense to keep it
disabled by default.
An explicit --enable-chamelium argument is provided to enable it when
necessary. This also leads to more predictable results than
automatically enabling it when its dependencies are met.
Finally, whether chamelium support is enabled is printed by the
information summary at end of the configure run.
Signed-off-by: Paul Kocialkowski <paul.kocialkowski@linux.intel.com>
---
configure.ac | 18 ++++++++++--------
1 file changed, 10 insertions(+), 8 deletions(-)
diff --git a/configure.ac b/configure.ac
index a6ab9e4a..6d3db878 100644
--- a/configure.ac
+++ b/configure.ac
@@ -185,23 +185,24 @@ PKG_CHECK_MODULES(GSL, [gsl], [gsl=yes], [gsl=no])
AM_CONDITIONAL(HAVE_GSL, [test "x$gsl" = xyes])
# for chamelium
-AC_ARG_ENABLE(chamelium, AS_HELP_STRING([--disable-chamelium],
- [Enable building of chamelium libraries and tests (default: yes)]),
- [enable_chamelium=no], [enable_chamelium=yes])
+AC_ARG_ENABLE(chamelium, AS_HELP_STRING([--enable-chamelium],
+ [Enable building of chamelium libraries and tests (default: no)]),
+ [if test x$enableval = xyes; then enable_chamelium=yes; fi],
+ [enable_chamelium=no])
AM_CONDITIONAL(HAVE_CHAMELIUM, [test "x$enable_chamelium" = xyes])
if test "x$enable_chamelium" = xyes; then
PKG_CHECK_MODULES(XMLRPC, xmlrpc xmlrpc_util xmlrpc_client, [],
- [AC_MSG_ERROR([Failed to find xmlrpc, required by chamelium. Use --disable-chamelium to disable chamelium support.])])
+ [AC_MSG_ERROR([Failed to find xmlrpc, required by chamelium.])])
PKG_CHECK_MODULES(PIXMAN, pixman-1, [],
- [AC_MSG_ERROR([Failed to find pixman, required by chamelium. Use --disable-chamelium to disable chamelium support.])])
+ [AC_MSG_ERROR([Failed to find pixman, required by chamelium.])])
if test x"$udev" != xyes; then
- AC_MSG_ERROR([Failed to find udev, required by chamelium. Use --disable-chamelium to disable chamelium support.])
+ AC_MSG_ERROR([Failed to find udev, required by chamelium.])
fi
if test x"$glib" != xyes; then
- AC_MSG_ERROR([Failed to find glib, required by chamelium. Use --disable-chamelium to disable chamelium support.])
+ AC_MSG_ERROR([Failed to find glib, required by chamelium.])
fi
if test x"$gsl" != xyes; then
- AC_MSG_ERROR([Failed to find gsl, required by chamelium. Use --disable-chamelium to disable chamelium support.])
+ AC_MSG_ERROR([Failed to find gsl, required by chamelium.])
fi
AC_DEFINE(HAVE_CHAMELIUM, 1, [Enable Chamelium support])
@@ -404,6 +405,7 @@ echo "Intel GPU tools"
echo ""
echo " • Tests:"
echo " Build tests : ${BUILD_TESTS}"
+echo " Chamelium tests : ${enable_chamelium}"
echo " Compile prime tests: ${NOUVEAU}"
echo " Print stack traces : ${with_libunwind}"
echo " Debug flags : ${DEBUG_CFLAGS}"
--
2.13.2
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH i-g-t v2] configure.ac: Disable chamelium by default and add enable argument
2017-07-26 8:11 [PATCH i-g-t v2] configure.ac: Disable chamelium by default and add enable argument Paul Kocialkowski
@ 2017-07-26 16:02 ` Lyude Paul
0 siblings, 0 replies; 2+ messages in thread
From: Lyude Paul @ 2017-07-26 16:02 UTC (permalink / raw)
To: Paul Kocialkowski, intel-gfx
r-b'd and pushed, thanks!
On Wed, 2017-07-26 at 11:11 +0300, Paul Kocialkowski wrote:
> Since the chamelium is not a very usual piece of hardware and
> requires
> pulling-in lots of specific dependencies, it makes sense to keep it
> disabled by default.
>
> An explicit --enable-chamelium argument is provided to enable it when
> necessary. This also leads to more predictable results than
> automatically enabling it when its dependencies are met.
>
> Finally, whether chamelium support is enabled is printed by the
> information summary at end of the configure run.
>
> Signed-off-by: Paul Kocialkowski <paul.kocialkowski@linux.intel.com>
> ---
> configure.ac | 18 ++++++++++--------
> 1 file changed, 10 insertions(+), 8 deletions(-)
>
> diff --git a/configure.ac b/configure.ac
> index a6ab9e4a..6d3db878 100644
> --- a/configure.ac
> +++ b/configure.ac
> @@ -185,23 +185,24 @@ PKG_CHECK_MODULES(GSL, [gsl], [gsl=yes],
> [gsl=no])
> AM_CONDITIONAL(HAVE_GSL, [test "x$gsl" = xyes])
>
> # for chamelium
> -AC_ARG_ENABLE(chamelium, AS_HELP_STRING([--disable-chamelium],
> - [Enable building of chamelium libraries and tests
> (default: yes)]),
> - [enable_chamelium=no], [enable_chamelium=yes])
> +AC_ARG_ENABLE(chamelium, AS_HELP_STRING([--enable-chamelium],
> + [Enable building of chamelium libraries and tests
> (default: no)]),
> + [if test x$enableval = xyes; then
> enable_chamelium=yes; fi],
> + [enable_chamelium=no])
> AM_CONDITIONAL(HAVE_CHAMELIUM, [test "x$enable_chamelium" = xyes])
> if test "x$enable_chamelium" = xyes; then
> PKG_CHECK_MODULES(XMLRPC, xmlrpc xmlrpc_util xmlrpc_client,
> [],
> - [AC_MSG_ERROR([Failed to find xmlrpc,
> required by chamelium. Use --disable-chamelium to disable chamelium
> support.])])
> + [AC_MSG_ERROR([Failed to find xmlrpc,
> required by chamelium.])])
> PKG_CHECK_MODULES(PIXMAN, pixman-1, [],
> - [AC_MSG_ERROR([Failed to find pixman,
> required by chamelium. Use --disable-chamelium to disable chamelium
> support.])])
> + [AC_MSG_ERROR([Failed to find pixman,
> required by chamelium.])])
> if test x"$udev" != xyes; then
> - AC_MSG_ERROR([Failed to find udev, required by
> chamelium. Use --disable-chamelium to disable chamelium support.])
> + AC_MSG_ERROR([Failed to find udev, required by
> chamelium.])
> fi
> if test x"$glib" != xyes; then
> - AC_MSG_ERROR([Failed to find glib, required by
> chamelium. Use --disable-chamelium to disable chamelium support.])
> + AC_MSG_ERROR([Failed to find glib, required by
> chamelium.])
> fi
> if test x"$gsl" != xyes; then
> - AC_MSG_ERROR([Failed to find gsl, required by
> chamelium. Use --disable-chamelium to disable chamelium support.])
> + AC_MSG_ERROR([Failed to find gsl, required by
> chamelium.])
> fi
>
> AC_DEFINE(HAVE_CHAMELIUM, 1, [Enable Chamelium support])
> @@ -404,6 +405,7 @@ echo "Intel GPU tools"
> echo ""
> echo " • Tests:"
> echo " Build tests : ${BUILD_TESTS}"
> +echo " Chamelium tests : ${enable_chamelium}"
> echo " Compile prime tests: ${NOUVEAU}"
> echo " Print stack traces : ${with_libunwind}"
> echo " Debug flags : ${DEBUG_CFLAGS}"
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2017-07-26 16:02 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2017-07-26 8:11 [PATCH i-g-t v2] configure.ac: Disable chamelium by default and add enable argument Paul Kocialkowski
2017-07-26 16:02 ` Lyude Paul
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).