[PATCH] configure: introduce --{enable,disable}-libdrm

Dan Nicholson dbn.lists at gmail.com
Mon Jan 25 11:03:04 PST 2010


On Mon, Jan 25, 2010 at 10:15 AM, Tiago Vignatti
<tiago.vignatti at nokia.com> wrote:
> On Fri, Jan 22, 2010 at 11:18:20PM +0100, ext Gaetan Nadon wrote:
>> On Fri, 2010-01-22 at 19:04 +0200, Tiago Vignatti wrote:
>>
>>  if test "x$DRI" = xyes || test "x$DRI2" = xyes; then
>> -       PKG_CHECK_MODULES([LIBDRM], $LIBDRM)
>> -       AC_SUBST(LIBDRM_CFLAGS)
>> -       AC_SUBST(LIBDRM_LIBS)
>> +       if test "x$DRM" = xyes; then
>> +               AC_DEFINE(WITH_LIBDRM, 1, [Building with libdrm support])
>> +               PKG_CHECK_MODULES([LIBDRM], $LIBDRM)
>> +               AC_SUBST(LIBDRM_CFLAGS)
>> +               AC_SUBST(LIBDRM_LIBS)
>>
>>
>>                     nit:        AC_SUBST not required, already done by PKG_CHECK_MODULES
>>
>
> Are you sure about it? If so, we need to patch all configure.ac doing such
> correction.

That's correct. The first argument to PKG_CHECK_MODULES will have
_CFLAGS and _LIBS appended and AC_SUBST'd with the appropriate values.
So, the AC_SUBST is redundant. It doesn't hurt, though.

--
Dan


More information about the xorg-devel mailing list