Thanks for the turbo fix, but you workarounded the "real" bug.

With my patch I get here in build.log:
...
checking for LIBDRM... yes
...
checking for LIBDRM_RADEON... no
...

With setting LIBDRM_RADEON_REQUIRED=2.4.19 I expected that the build
should immediately stop while libdrm package here has version 2.4.18.
BUT, that is not the case!

Intel_drm has in configure.ac:
...
case $DRI_DIRS in
*i915*|*i965*)
    PKG_CHECK_MODULES([INTEL], [libdrm_intel >= 2.4.19])
    ;;
esac
...

radeon_libdrm on the contrary:
...
case $DRI_DIRS in
*radeon*|*r200*|*r300*|*r600*)
    PKG_CHECK_MODULES([LIBDRM_RADEON],
                      [libdrm_radeon libdrm >= $LIBDRM_RADEON_REQUIRED],
                      HAVE_LIBDRM_RADEON=yes,
                      HAVE_LIBDRM_RADEON=no)

    if test "$HAVE_LIBDRM_RADEON" = yes; then
        RADEON_CFLAGS="-DHAVE_LIBDRM_RADEON=1 $LIBDRM_RADEON_CFLAGS"
        RADEON_LDFLAGS=$LIBDRM_RADEON_LIBS
    fi
    ;;
esac
...

IMO checking for LIBDRM_RADEON_REQUIRED has no real effect, but I am
not an autotools expert.
I am not sure if the LIBDRM_RADEON_REQUIRED part could/should be
handled like in libdrm_intel (...be removed and simplified).

Feedback welcome!

--
Sedat

On Tue, Mar 23, 2010 at 1:41 PM, Marek Olšák <mar...@gmail.com> wrote:
> Fixed in master without requiring new libdrm.
>
> -Marek
>
> On Tue, Mar 23, 2010 at 1:01 PM, Sedat Dilek <sedat.di...@googlemail.com>
> wrote:
>>
>> Fixes here latest issues with mesa master GIT [1].
>>
>> --
>> Sedat
>>
>> [1] http://marc.info/?l=mesa3d-dev&m=126934502904478&w=2
>>
>>
>> ------------------------------------------------------------------------------
>> Download Intel&#174; Parallel Studio Eval
>> Try the new software tools for yourself. Speed compiling, find bugs
>> proactively, and fine-tune applications for parallel performance.
>> See why Intel Parallel Studio got high marks during beta.
>> http://p.sf.net/sfu/intel-sw-dev
>> _______________________________________________
>> Mesa3d-dev mailing list
>> Mesa3d-dev@lists.sourceforge.net
>> https://lists.sourceforge.net/lists/listinfo/mesa3d-dev
>>
>
>

Attachment: config.log
Description: Binary data

------------------------------------------------------------------------------
Download Intel&#174; Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev
_______________________________________________
Mesa3d-dev mailing list
Mesa3d-dev@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/mesa3d-dev

Reply via email to