diff mbox

[libdrm] intel: Fix build failure in test_decode.c

Message ID 20120630131245.00cebd69.cand@gmx.com (mailing list archive)
State New, archived
Headers show

Commit Message

Lauri Kasanen June 30, 2012, 10:12 a.m. UTC
Hi list

The recently released libdrm 2.4.37 does not compile the Intel part:

test_decode.c: In function 'compare_batch':
test_decode.c:107: error: implicit declaration of function 'open_memstream'

PS: Please CC me.

Signed-off-by: Lauri Kasanen <cand@gmx.com>
---
 intel/test_decode.c |    2 ++
 1 files changed, 2 insertions(+), 0 deletions(-)

Comments

Ben Widawsky July 2, 2012, 9:54 p.m. UTC | #1
On Sat, 30 Jun 2012 13:12:45 +0300
Lauri Kasanen <cand@gmx.com> wrote:

> Hi list
> 
> The recently released libdrm 2.4.37 does not compile the Intel part:
> 
> test_decode.c: In function 'compare_batch':
> test_decode.c:107: error: implicit declaration of function 'open_memstream'
> 
> PS: Please CC me.
> 
> Signed-off-by: Lauri Kasanen <cand@gmx.com>
> ---
>  intel/test_decode.c |    2 ++
>  1 files changed, 2 insertions(+), 0 deletions(-)
> 
> diff --git a/intel/test_decode.c b/intel/test_decode.c
> index c9ab7ad..0fcdf3b 100644
> --- a/intel/test_decode.c
> +++ b/intel/test_decode.c
> @@ -21,6 +21,8 @@
>   * IN THE SOFTWARE.
>   */
>  
> +#define _GNU_SOURCE
> +
>  #include <string.h>
>  #include <stdlib.h>
>  #include <stdio.h>

I can't reproduce this. Can anyone else confirm this is broken, and if
so that the above patch fixes it?
Lauri Kasanen July 3, 2012, 9:21 a.m. UTC | #2
On Mon, 2 Jul 2012 14:54:58 -0700
Ben Widawsky <ben@bwidawsk.net> wrote:

> > +#define _GNU_SOURCE
> > +
> >  #include <string.h>
> >  #include <stdlib.h>
> >  #include <stdio.h>
> 
> I can't reproduce this. Can anyone else confirm this is broken, and if
> so that the above patch fixes it?

See the manpage, it depends on the glibc version. Libdrm is broken on glibc < 2.10.

- Lauri
Ben Widawsky July 3, 2012, 4:57 p.m. UTC | #3
On Tue, 3 Jul 2012 12:21:27 +0300
Lauri Kasanen <cand@gmx.com> wrote:

> On Mon, 2 Jul 2012 14:54:58 -0700
> Ben Widawsky <ben@bwidawsk.net> wrote:
> 
> > > +#define _GNU_SOURCE
> > > +
> > >  #include <string.h>
> > >  #include <stdlib.h>
> > >  #include <stdio.h>
> > 
> > I can't reproduce this. Can anyone else confirm this is broken, and if
> > so that the above patch fixes it?
> 
> See the manpage, it depends on the glibc version. Libdrm is broken on glibc < 2.10.
> 
> - Lauri
> _______________________________________________
> dri-devel mailing list
> dri-devel@lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/dri-devel

Pushed. Thanks for the patch.
diff mbox

Patch

diff --git a/intel/test_decode.c b/intel/test_decode.c
index c9ab7ad..0fcdf3b 100644
--- a/intel/test_decode.c
+++ b/intel/test_decode.c
@@ -21,6 +21,8 @@ 
  * IN THE SOFTWARE.
  */
 
+#define _GNU_SOURCE
+
 #include <string.h>
 #include <stdlib.h>
 #include <stdio.h>