Re: [PATCH xserver] meson: Use link_whole() not link_with(), for the Xorg dixmods.

2017-05-08 Thread Peter Hutterer
On Mon, May 08, 2017 at 10:37:58AM -0700, Eric Anholt wrote: > I clearly hadn't run ninja test, since fb no longer had any of the fb > symbols in it. > > Signed-off-by: Eric Anholt woo, it works! :) Reviewed-by: Peter Hutterer Cheers, Peter >

Re: [PATCH xserver 4/7] meson: Make driprotos and libdrm optional

2017-05-08 Thread Jon Turney
On 08/05/2017 18:12, Eric Anholt wrote: Jon Turney writes: +libxserver_dri3 = '' Pretty sure this should be = [] Yeah, me too. But when I try it: Meson encountered an error in file hw/kdrive/ephyr/meson.build, line 40, column 0: Link target [] is not library. Hmmm.. if I add it with

[PATCH xserver] meson: Use link_whole() not link_with(), for the Xorg dixmods.

2017-05-08 Thread Eric Anholt
I clearly hadn't run ninja test, since fb no longer had any of the fb symbols in it. Signed-off-by: Eric Anholt --- hw/xfree86/dixmods/meson.build | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/hw/xfree86/dixmods/meson.build

Re: [PATCH xserver 7/7] meson: An empty array is not a dependency object

2017-05-08 Thread Eric Anholt
Jon Turney writes: > Using the meson idiom of initializing a variaible to an empty array to > represent a null value does not work well for a variable expected to hold a > dependency object, as trying to apply found() to it will fail (currently). > > I think this can

Re: [PATCH xserver 6/7] meson: Refine build_dbus

2017-05-08 Thread Eric Anholt
Jon Turney writes: > As in autotools build, only turn on build_dbus (build dbus-core.c) if it's > needed by build_hal or build_systemd_logind, not just because dbus is > available. > > Building dbus-core.c without NEED_DBUS defined fails, as the contents of >

Re: [PATCH xserver 5/7] meson: Make XV optional

2017-05-08 Thread Eric Anholt
Jon Turney writes: > Signed-off-by: Jon Turney > --- > meson.build | 11 ++- > meson_options.txt | 3 +++ > 2 files changed, 13 insertions(+), 1 deletion(-) > > diff --git a/meson.build b/meson.build > index

Re: [PATCH xserver 4/7] meson: Make driprotos and libdrm optional

2017-05-08 Thread Eric Anholt
Jon Turney writes: > Add options for DRI{1,2,3} > shmfence is required for DRI3 > libdrm is required for any DRI{1,2,3} > Consolidate calls to dependency('libdrm') > Set WITH_LIBDRM when building with libdrm > > (configure.ac had LIBDRM as independently disableable

Re: [PATCH app/xdpyinfo] Use XRANDR 1.2 extension for reporting dimensions and resolution per output

2017-05-08 Thread Eric Anholt
Pali Rohár writes: > [ Unknown signature status ] > On Saturday 06 May 2017 13:28:17 Julien Cristau wrote: >> On Fri, May 5, 2017 at 10:08:14 +0200, Pali Rohár wrote: >> > PING. >> > >> > I would like to know if there is some problem with this and >> > something needs to

Re: [PATCH xserver resend] xf86: dri2: Use va_gl as vdpau_driver for Intel i965 GPUs

2017-05-08 Thread Eric Anholt
Hans de Goede writes: > The modesetting driver (which now often is used with Intel GPUs), > relies on dri2_probe_driver_name() to get the dri and vdpau driver > names, before this commit it would always assign the same name to > the 2 names. But the vdpau driver for i965

Re: Proposal for RandR version 1.6, Leases and EDID-based output grabs

2017-05-08 Thread Keith Packard
Pekka Paalanen writes: > I forget if I mentioned this to you personally yet: > https://gist.github.com/ppaalanen/e0d2744ff71188e9a294726a37ca83c2 Thanks, that's a helpful bit of thinking. It looks like most of that is still relevant, the only piece we'd swap out is how the

Re: Proposal for RandR version 1.6, Leases and EDID-based output grabs

2017-05-08 Thread Pekka Paalanen
On Fri, 05 May 2017 07:25:14 -0700 Keith Packard wrote: > Pekka Paalanen writes: > > > I disagree on the details, more below. > > > Such a RandR request is something I would not like to have to replicate > > on Wayland. The display server contains the

[PATCH xserver 4/7] meson: Make driprotos and libdrm optional

2017-05-08 Thread Jon Turney
Add options for DRI{1,2,3} shmfence is required for DRI3 libdrm is required for any DRI{1,2,3} Consolidate calls to dependency('libdrm') Set WITH_LIBDRM when building with libdrm (configure.ac had LIBDRM as independently disableable even if DRI{1,2,3} was enabled, but does that make sense?)

[PATCH xserver 7/7] meson: An empty array is not a dependency object

2017-05-08 Thread Jon Turney
Using the meson idiom of initializing a variaible to an empty array to represent a null value does not work well for a variable expected to hold a dependency object, as trying to apply found() to it will fail (currently). I think this can be demonstrated by configuing with Dglamor=no, then

[PATCH xserver 6/7] meson: Refine build_dbus

2017-05-08 Thread Jon Turney
As in autotools build, only turn on build_dbus (build dbus-core.c) if it's needed by build_hal or build_systemd_logind, not just because dbus is available. Building dbus-core.c without NEED_DBUS defined fails, as the contents of dbus-core.h are turned off. Signed-off-by: Jon Turney

[PATCH xserver 5/7] meson: Make XV optional

2017-05-08 Thread Jon Turney
Signed-off-by: Jon Turney --- meson.build | 11 ++- meson_options.txt | 3 +++ 2 files changed, 13 insertions(+), 1 deletion(-) diff --git a/meson.build b/meson.build index c7c9c79c3..5f30b996c 100644 --- a/meson.build +++ b/meson.build @@ -251,6

[PATCH xserver 1/7] meson: Remove stray whitespace

2017-05-08 Thread Jon Turney
Signed-off-by: Jon Turney --- include/meson.build | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/include/meson.build b/include/meson.build index 761f158d7..b7414bd67 100644 --- a/include/meson.build +++ b/include/meson.build @@ -163,7 +163,7 @@

[PATCH xserver 2/7] meson: Nettle is required if it's the only SHA1 choice

2017-05-08 Thread Jon Turney
Signed-off-by: Jon Turney --- meson.build | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/meson.build b/meson.build index 0a57ad24b..401b4064d 100644 --- a/meson.build +++ b/meson.build @@ -47,7 +47,7 @@ pixman_dep = dependency('pixman-1')

[PATCH xserver 3/7] meson: Don't require xf86dgaproto

2017-05-08 Thread Jon Turney
This dependency is never used, and build_dga is hardwired to false later on, anyhow. Signed-off-by: Jon Turney --- meson.build | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/meson.build b/meson.build index 401b4064d..df0d3aefa 100644 ---

[PATCH xserver 0/7] More meson fixes

2017-05-08 Thread Jon Turney
Jon Turney (7): meson: Remove stray whitespace meson: Nettle is required if it's the only SHA1 choice meson: Don't require xf86dgaproto meson: Make driprotos and libdrm optional meson: Make XV optional meson: Refine build_dbus meson: An empty array is not a dependency object

Re: [PATCH] Improved autoconfig drivers matching

2017-05-08 Thread Michel Dänzer
On 04/05/17 05:21 AM, Aaron Plattner wrote: > On 05/03/2017 01:11 PM, Adam Jackson wrote: >> On Mon, 2017-05-01 at 18:03 +0900, Michel Dänzer wrote: >>> On 28/04/17 04:11 AM, Adam Jackson wrote: Fixed up (and rebased and made meson-aware) and merged: To

Re: [PATCH xserver 3/3] kdrive: Remove dead slots from KdCardFuncs

2017-05-08 Thread Michel Dänzer
On 04/05/17 05:12 AM, Adam Jackson wrote: > On Mon, 2017-03-27 at 13:14 -0400, Adam Jackson wrote: > >> I am a filthy dirty liar. It was set to a stub that returned TRUE. >> >> TRUE appears to be what the rest of the DPMS stubs do, so how about I >> change KdSaveScreen to match? > > Did this and

Re: Proposal for RandR version 1.6, Leases and EDID-based output grabs

2017-05-08 Thread Pekka Paalanen
On Sat, 6 May 2017 13:34:44 +0200 Mario Kleiner wrote: > Just please make sure that one (user configurable/opt-in if necessary) > policy from the beginning is to allow leasing out any output to > applications, not just HMDs. My type of scientific/medical