The code depending on the definitions is already wrapped in the same conditional so go ahead and wrap the include.
Otherwise we'll brake compilation on platforms that are missing the header. Add assert.h in there as well, as it is introduced and used in the same fashon. Cc: Eric Anholt <e...@anholt.net> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=74122 Signed-off-by: Emil Velikov <emil.l.veli...@gmail.com> --- src/loader/loader.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/src/loader/loader.c b/src/loader/loader.c index 5a36423..626046a 100644 --- a/src/loader/loader.c +++ b/src/loader/loader.c @@ -67,8 +67,10 @@ #include <stdarg.h> #include <stdio.h> #include <string.h> +#ifdef HAVE_LIBUDEV #include <assert.h> #include <dlfcn.h> +#endif #include "loader.h" #ifndef __NOT_HAVE_DRM_H -- 1.8.5.2 _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev
