Attachment 92914 Details for Bug 74122 – loader: include dlfcn.h when building with HAVE_LIBUDEV

[patch] loader: include dlfcn.h when building with HAVE_LIBUDEV

0001-loader-include-dlfcn.h-when-building-with-HAVE_LIBUD.patch (text/plain), 1.04 KB, created by Emil Velikov on 2014-01-28 10:23:16 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Emil Velikov

Created: 2014-01-28 10:23:16 UTC

Size: 1.04 KB

patch

obsolete

>From c4d40d51fbbeb4b0d78e35f245c09f1f438062bb Mon Sep 17 00:00:00 2001
>From: Emil Velikov <emil.l.velikov@gmail.com>
>Date: Tue, 28 Jan 2014 10:15:17 +0000
>Subject: [PATCH] loader: include dlfcn.h when building with HAVE_LIBUDEV
>
>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 <eric@anholt.net>
>Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=74122
>Signed-off-by: Emil Velikov <emil.l.velikov@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 6d179a1..ee4c1e0 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
>

Actions: View | Splinter Review

Attachments on bug 74122: 92914

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.