Mailinglist Archive: radeonhd (300 mails)

[Adam Williamson <awilliamson@xxxxxxxxxxxx>]

On Tue, 2008-05-27 at 15:31 -0700, Adam Williamson wrote:
> On Tue, 2008-05-27 at 18:01 +0200, Matthias Hopf wrote:
> > I just pushed DRI support to master for radeonhd. Yippie!
>
> Having trouble building this on current Mandriva Cooker (which is not
> bleeding edge X.org, in case that's important). Well, I figure it's this
> that's breaking, but hey, it could be the scaler or RS780 stuff.
> Anyway :) Here's the error:
>
> In file included from /usr/include/xorg/opaque.h:48,
>                  from /usr/include/xorg/windowstr.h:60,
>                  from /usr/include/xorg/exa.h:36,
>                  from rhd_dri.c:60:
> /usr/include/setjmp.h:49: error: conflicting types for 'xf86jmp_buf'
> /usr/include/xorg/xf86_libc.h:99: error: previous declaration of 
> 'xf86jmp_buf' was here
> /usr/include/setjmp.h:53: error: expected ')' before '==' token
> /usr/include/setjmp.h:82: error: conflicting types for 'xf86longjmp'
> /usr/include/xorg/xf86_ansic.h:292: error: previous declaration of 
> 'xf86longjmp' was here
>
> any advice on the best fix? Thanks.

Okay, got this fixed thanks to our X guy, Paulo Andrade. He reminded me
that it's the exact same issue we ran into (and was fixed upstream) a
while back. As he puts it:

"We just are not linking by default with the builtin X Server
libc_wrapper, and it was already removed from the X Server long ago...
only still exists on X Server 1.4.  I think SuSE may have some
dependency on libc_wrapper. I left the problem "on purpose" in Mandriva
to ensure new compilations will not link with libc_wrapper, and directly
call libc functions."

The patch to fix this in MDV was basically to stop it ever including
xf86_ansic.h :

***

@@ -131,12 +131,6 @@ AM_CONDITIONAL(USE_DRI, test "x$USE_DRI"
 CFLAGS="$SAVED_CFLAGS"
 CPPFLAGS="$SAVED_CPPFLAGS"
  
 -AC_CHECK_HEADER([${sdkdir}/xf86_ansic.h],
 -  [AC_DEFINE([HAVE_XF86_ANSIC_H], [1],
 -      [Define to 1 if you have <${sdkdir}/xf86_ansic.h>.])
 -  have_xf86_ansic_h=yes])
 -AM_CONDITIONAL([HAVE_XF86_ANSIC_H], [test "x$have_xf86_ansic_h" = "xyes"])
 -
  SAVED_CFLAGS="$CFLAGS"
  CFLAGS="$CFLAGS $XORG_CFLAGS"

***

Thanks, all.
-- 
adamw

-- 
To unsubscribe, e-mail: radeonhd+unsubscribe@xxxxxxxxxxxx
For additional commands, e-mail: radeonhd+help@xxxxxxxxxxxx