[PATCH v3 xserver] config: declare xserver private dependencies in xorg-server.pc
dbn.lists at gmail.com
Tue Jun 15 08:59:18 PDT 2010
On Tue, Jun 15, 2010 at 8:52 AM, Gaetan Nadon <memsize at videotron.ca> wrote:
> Any module (drivers) depending on xserver also depends on some of the
> server private dependencies. Any driver including xf86.h depends on
> xext, kbproto, inputproto and randr.
> These dependencies are in separate packages, so anything can happen,
> removal, wrong version, etc... and the driver fails during compilation.
> Having the private dependencies declared will ensure all packages the
> server depends on are present and at the correct version.
> Currently each module attempts to check for server dependencies with
> various degrees of accuracy. With this patch, the driver will only need
> to check for its own explicit dependencies.
> Now that xproto is included in Requires.private it is removed from Requires.
> All the cflags from both Requires and Requires.private are returned
> to caller to pkg-config.
> Signed-off-by: Gaetan Nadon <memsize at videotron.ca>
> configure.ac | 3 +++
> xorg-server.pc.in | 3 ++-
> 2 files changed, 5 insertions(+), 1 deletions(-)
> diff --git a/configure.ac b/configure.ac
> index 4ada8f5..cfe8041 100644
> --- a/configure.ac
> +++ b/configure.ac
> @@ -796,6 +796,9 @@ dnl Core modules for most extensions, et al.
> REQUIRED_MODULES="[randrproto >= 126.96.36.199] [renderproto >= 0.11] [fixesproto >= 4.1] [damageproto >= 1.1] [xcmiscproto >= 1.2.0] [xextproto >= 188.8.131.52] [xproto >= 7.0.17] [xtrans >= 1.2.2] [bigreqsproto >= 1.1.0] fontsproto [inputproto >= 184.108.40.2062] [kbproto >= 1.0.3]"
> REQUIRED_LIBS="xfont xau"
> +# Make REQUIRED_MODULES available for inclusion in xorg-server.pc
Why did you remove the SDK_REQUIRED_MODULES setting? We only want to
add dependencies to xorg-server.pc that are exposed in the sdk.
More information about the xorg-devel