[PATCH video-ati] Revert "radeon: enable glamor by default (v4)"
memsize at videotron.ca
Wed Oct 2 17:27:08 PDT 2013
On 13-10-02 05:17 PM, Alex Deucher wrote:
> On Wed, Oct 2, 2013 at 5:13 PM, Gaetan Nadon <memsize at videotron.ca> wrote:
>> This reverts commit 282587cd0709850e7bacb1d8307065d95dc2c97d.
>> Requiring the glamor package by default breaks the build as this package
>> contains development files and is generally not installed by default in distros.
>> It also goes against user expectations that the X source tree can be built
>> out of the box without having to set configure options.
>> Moreover, the package is only available on a few very recent distros, for
>> example Ubuntu saucy 13.10. The package name is libglamor-dev.
>> My assumption is that the dev package is not required at run time but only
>> required for ati driver developers who are writing code against glamor.
>> Signed-off-by: Gaetan Nadon <memsize at videotron.ca>
> NAK. glamor is required for acceleration on newer asics.
Ok, patch withdrawn.
>> There is no .pc file for runtime time package, so I don't know how you
>> can tell if it is installed or not. Perhaps other distros don't make a
>> distinction and lump all files together.
>> Here is the link to the three Ubuntu packages:
>> I don't have a good enough understanding to propose a patch for a real solution
>> if all my assumptions are correct.
>> configure.ac | 6 +++---
>> 1 file changed, 3 insertions(+), 3 deletions(-)
>> diff --git a/configure.ac b/configure.ac
>> index 0fe0991..8b83a7c 100644
>> --- a/configure.ac
>> +++ b/configure.ac
>> @@ -100,10 +100,10 @@ CPPFLAGS="$CPPFLAGS $XORG_CFLAGS"
>> AC_MSG_CHECKING([whether to include GLAMOR support])
>> - AS_HELP_STRING([--disable-glamor],
>> - [Disable glamor, a new GL-based acceleration [default=enabled]]),
>> + AS_HELP_STRING([--enable-glamor],
>> + [Enable glamor, a new GL-based acceleration [default=no]]),
>> - [GLAMOR=yes])
>> + [GLAMOR=no])
>> AM_CONDITIONAL(GLAMOR, test x$GLAMOR != xno)
>> if test "x$GLAMOR" != "xno"; then
>> xorg-devel at lists.x.org: X.Org development
>> Archives: http://lists.x.org/archives/xorg-devel
>> Info: http://lists.x.org/mailman/listinfo/xorg-devel
More information about the xorg-devel