[PATCH xserver] XKB: Redirect actions defunct with Gtk3 (XInput?)

2012-02-19 Thread Andreas Wettstein
When redirect actions are used with Gtk3, Gtk3 complains about events not holding a GdkDevice. This is caused by device IDs not being set for redirect actions. The patch below sets them, but I am not quite sure that the values are correct. Anyway, the warnings from Gtk3 warnings are gone. More

[PATCH libxcb] Fallback to TCP if no protocol is specified and the UNIX connection fails

2012-02-19 Thread Julien Cristau
Signed-off-by: Julien Cristau jcris...@debian.org --- src/xcb_util.c |5 + 1 files changed, 5 insertions(+), 0 deletions(-) diff --git a/src/xcb_util.c b/src/xcb_util.c index fde4f85..6d9efe0 100644 --- a/src/xcb_util.c +++ b/src/xcb_util.c @@ -234,6 +234,11 @@ static int _xcb_open(const

[PATCH libx11] Revert xcb: Add TCP fallback

2012-02-19 Thread Julien Cristau
This reverts commit f09c5299a381e2729e800a0ac43f1c0e371f65f6. The TCP fallback ended up falling back to UNIX socket connection if $DISPLAY was set to e.g. some.host:0 and the initial attempt failed. Debian bug#659558 http://bugs.debian.org/659558 Signed-off-by: Julien Cristau

[PATCH video-modesetting 00/14] Config review for the new modesetting module

2012-02-19 Thread Gaetan Nadon
An assortment of missing/redundant/unneeded/deprecated code patches Gaetan Nadon (14): Fill the COPYING file with license text Add contact information to the README file man: missing AM_V_GEN and hard-coded sed command name Fix typo in .gitignore make: remove redundant AUTOMAKE_OPTIONS

[PATCH video-modesetting 01/14] Fill the COPYING file with license text

2012-02-19 Thread Gaetan Nadon
This reflects the copyright license text in the source code Signed-off-by: Gaetan Nadon mems...@videotron.ca --- COPYING | 48 1 files changed, 40 insertions(+), 8 deletions(-) diff --git a/COPYING b/COPYING index 7f33cbf..9508e25 100644 ---

[PATCH video-modesetting 02/14] Add contact information to the README file

2012-02-19 Thread Gaetan Nadon
Signed-off-by: Gaetan Nadon mems...@videotron.ca --- README | 23 +++ 1 files changed, 23 insertions(+), 0 deletions(-) diff --git a/README b/README index 4578d7f..7ca3286 100644 --- a/README +++ b/README @@ -1,3 +1,26 @@ This is a generic modesetting driver. The idea is

[PATCH video-modesetting 03/14] man: missing AM_V_GEN and hard-coded sed command name

2012-02-19 Thread Gaetan Nadon
Let's use the common xorg makefile for all drivers. This ensures no new problems are introduced. Improvements are welcome and to be applied to all drivers. Signed-off-by: Gaetan Nadon mems...@videotron.ca --- man/Makefile.am |8 1 files changed, 4 insertions(+), 4 deletions(-) diff

[PATCH video-modesetting 04/14] Fix typo in .gitignore

2012-02-19 Thread Gaetan Nadon
Signed-off-by: Gaetan Nadon mems...@videotron.ca --- .gitignore |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/.gitignore b/.gitignore index 4cada5a..dc5a5cd 100644 --- a/.gitignore +++ b/.gitignore @@ -71,9 +71,9 @@ core *.tar.bz2 *.tar.gz # -# Add

[PATCH video-modesetting 05/14] make: remove redundant AUTOMAKE_OPTIONS

2012-02-19 Thread Gaetan Nadon
Already covered by AM_INIT_AUTOMAKE([foreign dist-bzip2]) Signed-off-by: Gaetan Nadon mems...@videotron.ca --- Makefile.am |1 - 1 files changed, 0 insertions(+), 1 deletions(-) diff --git a/Makefile.am b/Makefile.am index 8cbe4a9..9a525f0 100644 --- a/Makefile.am +++ b/Makefile.am @@ -18,7

[PATCH video-modesetting 06/14] Remove redundant EXTRA_DIST for the README file.

2012-02-19 Thread Gaetan Nadon
Autotools know about it, it is always distributed. Signed-off-by: Gaetan Nadon mems...@videotron.ca --- Makefile.am |2 -- 1 files changed, 0 insertions(+), 2 deletions(-) diff --git a/Makefile.am b/Makefile.am index 9a525f0..052c611 100644 --- a/Makefile.am +++ b/Makefile.am @@ -19,5 +19,3

[PATCH video-modesetting 07/14] Add missing targets for ChangeLog and INSTALL

2012-02-19 Thread Gaetan Nadon
Signed-off-by: Gaetan Nadon mems...@videotron.ca --- Makefile.am | 12 1 files changed, 12 insertions(+), 0 deletions(-) diff --git a/Makefile.am b/Makefile.am index 052c611..8e4dfdb 100644 --- a/Makefile.am +++ b/Makefile.am @@ -19,3 +19,15 @@ # CONNECTION WITH THE SOFTWARE OR

[PATCH video-modesetting 08/14] Replace obsolete AM_CONFIG_HEADER with AC_CONFIG_HEADERS

2012-02-19 Thread Gaetan Nadon
Signed-off-by: Gaetan Nadon mems...@videotron.ca --- configure.ac |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/configure.ac b/configure.ac index fcfe330..01596b5 100644 --- a/configure.ac +++ b/configure.ac @@ -27,7 +27,7 @@ AC_INIT([xf86-video-modesetting],

[PATCH video-modesetting 09/14] Remove redundant AC_PROG_CC

2012-02-19 Thread Gaetan Nadon
Already covered by XORG_DEFAULT_OPTIONS Signed-off-by: Gaetan Nadon mems...@videotron.ca --- configure.ac |1 - 1 files changed, 0 insertions(+), 1 deletions(-) diff --git a/configure.ac b/configure.ac index 01596b5..c936683 100644 --- a/configure.ac +++ b/configure.ac @@ -44,7 +44,6 @@

[PATCH video-modesetting 10/14] Remove unneeded AM_PROG_CC_C_CO

2012-02-19 Thread Gaetan Nadon
There are no objects in subdirs or compiled multiple times with different flags. Signed-off-by: Gaetan Nadon mems...@videotron.ca --- configure.ac |1 - 1 files changed, 0 insertions(+), 1 deletions(-) diff --git a/configure.ac b/configure.ac index c936683..a2488f2 100644 --- a/configure.ac

[PATCH video-modesetting 11/14] make: remove empty variable assignment

2012-02-19 Thread Gaetan Nadon
Signed-off-by: Gaetan Nadon mems...@videotron.ca --- src/Makefile.am |3 --- 1 files changed, 0 insertions(+), 3 deletions(-) diff --git a/src/Makefile.am b/src/Makefile.am index efc4bc0..40ebf0a 100644 --- a/src/Makefile.am +++ b/src/Makefile.am @@ -17,7 +17,6 @@ # ADAM JACKSON BE LIABLE

[PATCH video-modesetting 12/14] make: add all warnings according to the platform

2012-02-19 Thread Gaetan Nadon
The current code only adds -Wall and only for gcc. Automake reserves the use of CPPFLAGS for the user to override on the command line. This also breaks the option --enable-strict-compilation The variable CWARNFLAGS contains the complete set of warnings and is platform sensitive. Signed-off-by:

[PATCH video-modesetting 13/14] config: layout and comment the top portion of configure.ac

2012-02-19 Thread Gaetan Nadon
Reorder statements to be consistent with other modules so things are easier to find. Signed-off-by: Gaetan Nadon mems...@videotron.ca --- configure.ac | 20 ++-- 1 files changed, 10 insertions(+), 10 deletions(-) diff --git a/configure.ac b/configure.ac index 610c043..50a258e

[PATCH video-modesetting 14/14] config: replace deprecated AC_HELP_STRING with AS_HELP_STRING

2012-02-19 Thread Gaetan Nadon
Signed-off-by: Gaetan Nadon mems...@videotron.ca --- configure.ac |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/configure.ac b/configure.ac index 50a258e..9758d87 100644 --- a/configure.ac +++ b/configure.ac @@ -48,12 +48,12 @@ XORG_DEFAULT_OPTIONS AH_TOP([#include

Re: [PATCH video-intel] Revert Update autotools configuration

2012-02-19 Thread Matthieu Herrb
On Sat, Feb 18, 2012 at 02:33:34PM -0500, Gaetan Nadon wrote: This reverts commit 9184af921bc2f332fcb6c9b47001414378eab8e2. All X.Org modules must be able to be configured with autoconf 2.60. In addition, version 2.63 has GPL licensing issues which prevents some vendor to release software

Re: [PATCH lib/libX11] Don't use caddr_t casts

2012-02-19 Thread Jon TURNEY
On 18/02/2012 19:11, Jamey Sharp wrote: On Sat, Feb 18, 2012 at 05:21:24PM +, Jon TURNEY wrote: Casting a (const char *) to (caddr_t) to assign to iovec.io_base seems pointless. caddr_t isn't used anywhere else in xcb or libX11 According to the libxcb git history, I replaced (caddr_t)

Re: [PATCH video-modesetting 13/14] config: layout and comment the top portion of configure.ac

2012-02-19 Thread Tormod Volden
On Sun, Feb 19, 2012 at 2:36 PM, Gaetan Nadon mems...@videotron.ca wrote: Reorder statements to be consistent with other modules so things are easier to find. ... +# Initialize X.Org macros 1.8 or later for MAN_SUBSTS set by XORG_MANPAGE_SECTIONS +m4_ifndef([XORG_MACROS_VERSION], +      

Re: [PATCH] Revert render: don't bother with animated cursors on floating slaves (#39989)

2012-02-19 Thread Jon TURNEY
On 17/02/2012 06:41, Jeremy Huddleston wrote: For both: Reviewed-by: Jeremy Huddleston jerem...@apple.com On Feb 15, 2012, at 8:32 PM, Peter Hutterer peter.hutte...@who-t.net wrote: This reverts commit bbb6b8c834e0e1491ca14403b5d0840dd14380d3. With this patch applied, Xorg -retro does

[PATCH v2 video-modesetting 13/14] config: layout and comment the top portion of configure.ac

2012-02-19 Thread Gaetan Nadon
Reorder statements to be consistent with other modules so things are easier to find. Signed-off-by: Gaetan Nadon mems...@videotron.ca --- configure.ac | 20 ++-- 1 files changed, 10 insertions(+), 10 deletions(-) diff --git a/configure.ac b/configure.ac index 610c043..9bbbfb1

Re: [PATCH libxcb] Fallback to TCP if no protocol is specified and the UNIX connection fails

2012-02-19 Thread Jeremy Huddleston
Can we delay the libX11 change until this is in a tagged libxcb (and have libX11 require that version to avoid regression)? All the changes currently on xcb/libxcb master look 1.8.1 worthy, so perhaps we can push this one into the mix and get a 1.8.1 this week. --Jeremy On Feb 19, 2012, at

Re: [PATCH xserver 1/1] os: Pass the FatalError message to OsVendorFatalError

2012-02-19 Thread Jeremy Huddleston
Ping. I know this looks pointless, but I need it for a follwup patch which uses this info in XQuartz's OsVendorFatalError. On Feb 17, 2012, at 1:47 PM, Jeremy Huddleston jerem...@apple.com wrote: Signed-off-by: Jeremy Huddleston jerem...@apple.com --- hw/dmx/dmxinit.c |2 +-

Re: [PATCH libxcb] Fallback to TCP if no protocol is specified and the UNIX connection fails

2012-02-19 Thread Jamey Sharp
I've pushed this patch; thanks! I'll let you and Jeremy sort out the libX11 side. Jamey On Sun, Feb 19, 2012 at 01:43:54PM +0100, Julien Cristau wrote: Signed-off-by: Julien Cristau jcris...@debian.org --- src/xcb_util.c |5 + 1 files changed, 5 insertions(+), 0 deletions(-)

Re: [PULL] Minor XQuartz, Revert an XWarpPointer regression

2012-02-19 Thread Peter Hutterer
On Sun, Feb 19, 2012 at 02:50:42PM -0800, Jeremy Huddleston wrote: This includes two minor XQuartz issues I'd like to address before 1.12 is tagged, but the main issue here is the revert. I emailed the list about this about a month ago, and Peter's been too busy to look into the regression,

[PATCH synaptics] Revert Don't store fake events in the motion history

2012-02-19 Thread Peter Hutterer
This commit introduced a regression. On some touchpads, the pointer keeps moving in the last direction when the finger movement stops but the finger is left on the touchpad. Cause appears to be get_delta() which calculates the deltas based on the motion history but has no control flow for the

[PATCH] dix: IsFloating() on master devices is always false

2012-02-19 Thread Peter Hutterer
There are a few subtle bugs during startup where IsFloating() returns true if the device is a master device that is not yet paired with its keyboard device. Force IsFloating() to always return FALSE for master devices, that was the intent after all and any code that relies on the other behaviour

[PATCH xinput 2/5] Always call XCloseDisplay()

2012-02-19 Thread Peter Hutterer
Signed-off-by: Peter Hutterer peter.hutte...@who-t.net --- src/xinput.c |9 ++--- 1 files changed, 6 insertions(+), 3 deletions(-) diff --git a/src/xinput.c b/src/xinput.c index a336f53..a1bb221 100644 --- a/src/xinput.c +++ b/src/xinput.c @@ -368,17 +368,17 @@ main(int argc, char *

[PATCH xinput 3/5] Don't leak output_info

2012-02-19 Thread Peter Hutterer
This inverts the logic to have a more obvious flow for freeing the output_info. ==26716== 1,161 bytes in 8 blocks are definitely lost in loss record 5 of 7 ==26716==at 0x4A074CD: malloc (vg_replace_malloc.c:236) ==26716==by 0x395D804ABA: XRRGetOutputInfo (in

[PATCH xinput 4/5] Add find_output_xrandr to check for output presence

2012-02-19 Thread Peter Hutterer
Signed-off-by: Peter Hutterer peter.hutte...@who-t.net --- src/transform.c | 31 +-- 1 files changed, 25 insertions(+), 6 deletions(-) diff --git a/src/transform.c b/src/transform.c index 3ea7ff8..15fda59 100644 --- a/src/transform.c +++ b/src/transform.c @@ -135,13

[PATCH xinput 5/5] Replace NVIDIA-specific output checking

2012-02-19 Thread Peter Hutterer
If the binary driver drives at least one screen, the old method did not allow for randr-style screen binding (on the other screen). Change the logic to: - check if the output an randr output - if not, check if the default randr output is present and the output name was HEAD-n - if so, use

Re: [PATCH] dix: IsFloating() on master devices is always false

2012-02-19 Thread Jeremy Huddleston
This one makes me squirm a bit. It *looks* right, but whenever I put one tidbit of input handling into my brain, another bit falls out, and it's usually that fallen-out bit that was the key to understanding why something won't work. AIUI, this change will affect use cases like these:

Re: [PATCH 1/9] dri2: Fix can_exchange() to allow page-flipping on new servers.

2012-02-19 Thread Mario Kleiner
On 02/16/2012 11:04 AM, Michel Dänzer wrote: On Don, 2012-02-16 at 00:45 +0100, Mario Kleiner wrote: can_exchange() fails on at least Xorg 1.12+. This fixes it in the same way it was fixed in the ati intel ddx. Signed-off-by: Mario Kleinermario.klei...@tuebingen.mpg.de ---

Re: [PATCH 9/9] dri2: Fix corner case crash for swaplimit 1

2012-02-19 Thread Mario Kleiner
On 02/16/2012 10:46 AM, Michel Dänzer wrote: On Don, 2012-02-16 at 00:45 +0100, Mario Kleiner wrote: If a swaplimit 1 is set on a server which supports the swaplimit api (XOrg 1.12.0+), the following can happen: 1. Client calls glXSwapBuffersMscOML() with a swap target 1 vblank in the

Re: [PATCH xserver] XKB: Redirect actions defunct with Gtk3 (XInput?)

2012-02-19 Thread Peter Hutterer
On Sun, Feb 19, 2012 at 11:18:19AM +0100, Andreas Wettstein wrote: When redirect actions are used with Gtk3, Gtk3 complains about events not holding a GdkDevice. This is caused by device IDs not being set for redirect actions. The patch below sets them, but I am not quite sure that the values