[PATCH input-evdev 3/3] config: evdev depends on RANDR through xf86.h

[Julien Cristau]

On Wed, Jun  9, 2010 at 17:42:00 -0400, Gaetan Nadon wrote:

> On Wed, 2010-06-09 at 23:24 +0200, Julien Cristau wrote:
> 
> > On Wed, Jun  9, 2010 at 17:17:44 -0400, Gaetan Nadon wrote:
> > 
> > > The xf86.h file contains external declarations specific to RANDR
> > > If this extension is defined (installed) evdev will not compile
> > > if randrproto is not installed.
> > > Because this is an optional extension, XORG_DRIVER_CHECK_EXT
> > > must be used.
> > > 
> > If the only thing pulling in randr headers is xf86.h then the dependency
> > on randrproto should come from xorg-server.pc, not from evdev.
> > 
> 
> The protocols are independent packages which can be removed/uninstalled
> at any time. I simply go to proto/randrproto and "make uninstall". Then
> evdev fails to compile. Same issue with xext and kbproto. The xf86.h
> file is "for video driver" according to it's own comment.
> 
> I would love for this dependency not to exist. I originally thought it
> did not.
> 
I'm not sure what you're getting at, but that has nothing to do with
what I said afaict.  I'm saying 'Requires.private: randrproto' in
xorg-server.pc would be the correct way to express the fact that xf86.h
needs randrproto.h.

Cheers,
Julien