Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 434720 - x11-drivers/xf86-video-intel-2.20.7: add USE xvmc
Summary: x11-drivers/xf86-video-intel-2.20.7: add USE xvmc
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: [OLD] Library (show other bugs)
Hardware: All Linux
: Normal enhancement (vote)
Assignee: Gentoo X packagers
URL:
Whiteboard:
Keywords: PATCH
Depends on:
Blocks:
 
Reported: 2012-09-11 12:26 UTC by Nikoli
Modified: 2012-09-12 06:47 UTC (History)
1 user (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
patch, works fine for me (xf86-video-intel.diff,1.63 KB, text/plain)
2012-09-11 12:26 UTC, Nikoli
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Nikoli 2012-09-11 12:26:10 UTC
Created attachment 323502 [details]
patch, works fine for me

Please make x11-libs/libXvMC optional: xvmc does not support h264 and vp8 and is useless for me. I consider xvmc obsolete and do not want to have anything related to it in my systems. Attached patch works fine for me: KDE session restarts, watching video with mplayer2 works.

P.S. Seems there is missing sys-fs/udev RDEPEND and possibly udev USE.
Also what about adding debug, dga, kms-only, xaa USE flags? Not sure about "userptr" and "async-swap" options.

$ grep 'AS_HELP_STRING(\[--' configure.ac  -A1
            AS_HELP_STRING([--with-builderstring=BUILDERSTRING],
                           [Additional builder string (default: use git describe)]),
--
            AS_HELP_STRING([--with-xorg-module-dir=DIR],
                           [Default xorg module directory [[default=$libdir/xorg/modules]]]),
--
AC_ARG_ENABLE(dri, AS_HELP_STRING([--disable-dri],
                                  [Disable DRI support [[default=auto]]]))
--
AC_ARG_ENABLE(xvmc, AS_HELP_STRING([--disable-xvmc],
                                  [Disable XvMC support [[default=yes]]]),
--
AC_ARG_ENABLE(kms-only, AS_HELP_STRING([--enable-kms-only],
                                  [Assume KMS support [[default=no]]]),
--
              AS_HELP_STRING([--enable-sna],
                             [Enable SandyBridge's New Acceleration (SNA) [default=auto]]),
--
              AS_HELP_STRING([--enable-uxa],
                             [Enable Unified Acceleration Architecture (UXA) [default=yes]]),
--
              AS_HELP_STRING([--enable-glamor],
                             [Enable glamor, a new GL-based acceleration [default=no]]),
--
              AS_HELP_STRING([--enable-xaa],
                             [Enable legacy X Acceleration Architecture (XAA) [default=auto]]),
--
              AS_HELP_STRING([--enable-dga],
                             [Enable legacy Direct Graphics Access (DGA) [default=auto]]),
--
            AS_HELP_STRING([--with-default-accel],
                           [Select the default acceleration method [default=uxa if enabled, otherwise sna]]),
--
              AS_HELP_STRING([--enable-userptr],
                             [Enable use of userptr (experimental) [default=no]]),
--
              AS_HELP_STRING([--enable-async-swap],
                             [Enable use of asynchronous swaps (experimental) [default=no]]),
--
              AS_HELP_STRING([--enable-debug],
                             [Enables internal debugging [default=no]]),
$ git describe 
2.20.7-4-gbc73acb
Comment 1 Matt Turner gentoo-dev 2012-09-11 16:28:00 UTC
Seems reasonable. I don't want to expose all those other options though.
Comment 2 Rémi Cardona (RETIRED) gentoo-dev 2012-09-11 17:44:43 UTC
On it.
Comment 3 Rémi Cardona (RETIRED) gentoo-dev 2012-09-11 21:18:52 UTC
Fixed in portage and x11 overlay.

FTR, XvMC support is really small, and though I know it's only useful for MPEG-2, we forced it in for everyone since its deps were really limited. The USE flag is indeed a nice improvement.

I want to thank you for a very nice and clean patch, it makes our job a whole lot easier. :)

Cheers
Comment 4 Nikoli 2012-09-12 06:47:56 UTC
Please look at bug #434794