Re: [PATCH 1/2] dix: Return BadWindow rather than BadMatch from dixLookupWindow

2012-01-05 Thread Daniel Stone
Hi, On 5 January 2012 01:22, Christopher James Halse Rogers christopher.halse.rog...@canonical.com wrote: dixLookupWindow uses dixLookupDrawable internally, which returns BadMatch when the XID matches a non-Window drawable.  Users of dixLookupWindow don't care about this, just that it's not a

[BUG] numlock/capslock status broken after switch from framebuffer console to xorg

2012-01-05 Thread Knut Petersen
SYSTEM === xorg: current git master os: linux kernel: 3.1.7, 3.2.0 hardware: AOpen i915Gmm-hfs keyboard: connected to ps/2 port HOW TO REPRODUCE === Bug 1: = 1: activate numlock 2:switch to a framebuffer console 3. switch back to Xorg status: numlock led O 4. depress

Re: [Mesa-dev] [PATCH 20/20] tests/glx: Add unit tests for GLX_ARB_create_context GLX protocol

2012-01-05 Thread Dan Nicholson
2012/1/4 Ian Romanick i...@freedesktop.org: On 01/04/2012 11:05 AM, Michel Dänzer wrote: On Mit, 2012-01-04 at 10:56 -0800, Ian Romanick wrote: On 01/04/2012 10:55 AM, Daniel Stone wrote: Hi, On 4 January 2012 18:45, Ian Romanicki...@freedesktop.org   wrote: Okay, I looked back at your

Re: [PATCH] Change autogen.sh scripts to respect NOCONFIGURE

2012-01-05 Thread Colin Walters
Hi Gaetan, On Wed, 2012-01-04 at 19:48 -0500, Gaetan Nadon wrote: No objections in principle, the doc is convincing. We are talking about 240 modules here. A sizable piece of work. Thanks; any suggestions on how to proceed? I don't have commit privileges to the xorg repositories (and no

Re: [PATCH 00/31] _X_UNEXPORT

2012-01-05 Thread Daniel Stone
Hi, On Fri, Dec 02, 2011 at 01:05:43PM -0800, Aaron Plattner wrote: Of the symbols removed here, we use the following: [...] Thanks for this list -- it's great, exactly what I was hoping for. I've re-exported all these symbols in my current tree, so hopefully it should be fine. Using nobody

Re: [PATCH] dix: fix wrong condition checking for attached slave (#44003)

2012-01-05 Thread Chase Douglas
On 01/04/2012 09:58 PM, Peter Hutterer wrote: We need to update the master if the device is not a master _and_ it is not floating. Triggered since 51437995a5041a8c53c33b508b1607c78a5fa463, introduced in dc57f89959e549403f8488eb9f23425bd7118b22. X.Org Bug 44003

Re: [PATCH v2 1/2] dix: send focus events to the immediate parent (#44079)

2012-01-05 Thread Chase Douglas
On 01/04/2012 09:49 PM, Peter Hutterer wrote: For a transition from windows A to B, A-parent did not receive an event. DeviceFocusOutEvents sends to windows ]from, to[, so start with the actual window, not it's parent. X.Org Bug 44079 http://bugs.freedesktop.org/show_bug.cgi?id=44079

Re: [PATCH 2/2] dix: on PointerRootWin send a FocusIn to the sprite window too

2012-01-05 Thread Chase Douglas
On 01/04/2012 09:49 PM, Peter Hutterer wrote: XTS XSetDeviceFocus-7 Signed-off-by: Peter Hutterer peter.hutte...@who-t.net Reviewed-by: Chase Douglas chase.doug...@canonical.com ___ xorg-devel@lists.x.org: X.Org development Archives:

Re: [PATCH XTS 1/2] xts5: Fix XSetDeviceFocus-4

2012-01-05 Thread Chase Douglas
On 01/04/2012 09:02 PM, Peter Hutterer wrote: XDeviceKeyEvent and XKeyEvent are _not_ binary compatible. Expecting a random byte location inside the device event to be of the value it'd have in a core event is an amusing pasttime, but highly inappropriate for a test. Signed-off-by: Peter

Re: [PATCH XTS 2/2] xts5: fix XSetDeviceFocus-5

2012-01-05 Thread Chase Douglas
On 01/04/2012 09:02 PM, Peter Hutterer wrote: t003() selects for key events on the root window. t005() takes over and then complains when the focus correctly follows the pointer and an event is delivered to the root window. If you don't expect events on the root window, unselect them first.

Re: [PATCH] Change autogen.sh scripts to respect NOCONFIGURE

2012-01-05 Thread Gaetan Nadon
On 12-01-05 09:17 AM, Colin Walters wrote: Hi Gaetan, On Wed, 2012-01-04 at 19:48 -0500, Gaetan Nadon wrote: No objections in principle, the doc is convincing. We are talking about 240 modules here. A sizable piece of work. Thanks; any suggestions on how to proceed? I don't have commit

Re: [PATCH] Change autogen.sh scripts to respect NOCONFIGURE

2012-01-05 Thread Cyril Brulebois
Gaetan Nadon mems...@videotron.ca (05/01/2012): #!/bin/sh exec xorg-autogen.sh $@ Note that I will also have to update configure.ac in each module to mandate the latest version of xorg-macros (as usual): XORG_MACROS_VERSION(1.17) So you'll have a check in configure.ac to make sure

Re: [PATCH] Change autogen.sh scripts to respect NOCONFIGURE

2012-01-05 Thread Colin Walters
On Thu, 2012-01-05 at 17:50 +0100, Cyril Brulebois wrote: Gaetan Nadon mems...@videotron.ca (05/01/2012): #!/bin/sh exec xorg-autogen.sh $@ Note that I will also have to update configure.ac in each module to mandate the latest version of xorg-macros (as usual):

Re: [PATCH v2 1/2] dix: send focus events to the immediate parent (#44079)

2012-01-05 Thread Keith Packard
On Thu, 5 Jan 2012 15:49:19 +1000, Peter Hutterer peter.hutte...@who-t.net wrote: For a transition from windows A to B, A-parent did not receive an event. DeviceFocusOutEvents sends to windows ]from, to[, so start with the actual window, not it's parent. This matches the implementation for

Re: [PATCH:xts 1/6] configure.ac: Quote argument to m4_pattern_forbid.

2012-01-05 Thread Aaron Plattner
On 12/22/2011 04:23 PM, Cyril Brulebois wrote: Without this, configure spits out: ./configure: line 12801: ac_fn_c_check_member: command not found ./configure: line 12815: ac_fn_c_check_type: command not found I'm confused. It looks like the argument is already quoted and this change

Re: [PATCH] Change autogen.sh scripts to respect NOCONFIGURE

2012-01-05 Thread Gaetan Nadon
On 12-01-05 12:01 PM, Colin Walters wrote: On Thu, 2012-01-05 at 17:50 +0100, Cyril Brulebois wrote: Gaetan Nadon mems...@videotron.ca (05/01/2012): #!/bin/sh exec xorg-autogen.sh $@ Note that I will also have to update configure.ac in each module to mandate the latest version of xorg-macros

Re: [PATCH:xts 1/6] configure.ac: Quote argument to m4_pattern_forbid.

2012-01-05 Thread Cyril Brulebois
Aaron Plattner aplatt...@nvidia.com (05/01/2012): On 12/22/2011 04:23 PM, Cyril Brulebois wrote: Without this, configure spits out: ./configure: line 12801: ac_fn_c_check_member: command not found ./configure: line 12815: ac_fn_c_check_type: command not found I'm confused. It looks

Re: Some patches for xts

2012-01-05 Thread Aaron Plattner
On 12/22/2011 04:23 PM, Cyril Brulebois wrote: Hi, this series should be somehow easy to review. That one might be a bit controversial, but I'm proposing it anyway, just in case: [PATCH:xts 6/6] Remove old files from the TET3 import. As discussed, I dropped #1 and merged the rest: To

Re: [PATCH] Change autogen.sh scripts to respect NOCONFIGURE

2012-01-05 Thread Alan Coopersmith
On 01/05/12 09:01, Colin Walters wrote: Regardless we could certainly change the patch to all modules to be: #!/bin/sh common_autogen=`which xorg-util-autogen.sh 2/dev/null` if test -n $common_autogen; then exec $common_autogen $@ else # copypasted version here fi That would allow

Re: [PATCH] Change autogen.sh scripts to respect NOCONFIGURE

2012-01-05 Thread Alan Coopersmith
On 01/05/12 08:41, Gaetan Nadon wrote: At this time the xserver is in a code freeze for the 1.12 version, so it may be delayed a few weeks. Since the autogen in each module shouldn't be packaged, just used for git builds (though we haven't fixed that in every module yet), it seems most

Re: [PATCH] Change autogen.sh scripts to respect NOCONFIGURE

2012-01-05 Thread Gaetan Nadon
On 12-01-05 02:05 PM, Alan Coopersmith wrote: On 01/05/12 08:41, Gaetan Nadon wrote: At this time the xserver is in a code freeze for the 1.12 version, so it may be delayed a few weeks. Since the autogen in each module shouldn't be packaged, just used for git builds (though we haven't fixed

Re: [PATCH] Change autogen.sh scripts to respect NOCONFIGURE

2012-01-05 Thread Julien Cristau
On Thu, Jan 5, 2012 at 09:17:20 -0500, Colin Walters wrote: Hi Gaetan, On Wed, 2012-01-04 at 19:48 -0500, Gaetan Nadon wrote: No objections in principle, the doc is convincing. We are talking about 240 modules here. A sizable piece of work. Thanks; any suggestions on how to

[PATCH XTS v2 2/2] xts5: fix XSetDeviceFocus-5

2012-01-05 Thread Peter Hutterer
t003() selects for key events on the root window. t005() takes over and then complains when the focus correctly follows the pointer and an event is delivered to the root window. If you don't expect events on the root window, unselect them first. We could also XCloseDisplay() at the end of each

[PATCH XTS v2 2/2] xts5: remove (type *)NULL casts

2012-01-05 Thread Peter Hutterer
Coccinelle patch: @@ type T; @@ - (T *)NULL + NULL with a bit of manual massaging to skip botched hunks Signed-off-by: Peter Hutterer peter.hutte...@who-t.net --- Changes to v1: - said botched hunks massaged out of the patch xts5/XI/AllowDeviceEvents.m |2 +-

Re: [PATCH XTS v2 2/2] xts5: fix XSetDeviceFocus-5

2012-01-05 Thread Peter Hutterer
On Thu, Jan 05, 2012 at 02:41:31PM -0800, Chase Douglas wrote: On 01/05/2012 02:28 PM, Peter Hutterer wrote: t003() selects for key events on the root window. t005() takes over and then complains when the focus correctly follows the pointer and an event is delivered to the root window. If

[PULL] focus fixes and a crasher fix

2012-01-05 Thread Peter Hutterer
The following changes since commit 98cde254acb9b98337ddecf64c138d38c14ec2bf: Bump version to 1.11.99.901 (1.12 RC1) (2011-12-27 14:02:39 -0800) are available in the git repository at: git://people.freedesktop.org/~whot/xserver for-keith Peter Hutterer (4): xfree86: split warning about

[PATCH] Xi: change XIChangeDeviceProperty from const pointer to const void *

2012-01-05 Thread Peter Hutterer
According to Daniel Kurtz, a typedev void *pointer is a atomic type. So a const pointer is equivalent to 'void const *' instead of the intended 'const void *'. This technically changes the ABI, but we don't bump it for this. Signed-off-by: Peter Hutterer peter.hutte...@who-t.net ---

Re: [PATCH] Xi: change XIChangeDeviceProperty from const pointer to const void *

2012-01-05 Thread Chase Douglas
On 01/05/2012 04:15 PM, Peter Hutterer wrote: According to Daniel Kurtz, a typedev void *pointer is a atomic type. So a const pointer is equivalent to 'void const *' instead of the intended 'const void *'. I believe const pointer is equivalent to 'void * const'. The rule I remember is that the

Re: Some patches for xts

2012-01-05 Thread Cyril Brulebois
Aaron Plattner aplatt...@nvidia.com (05/01/2012): To git+ssh://git.freedesktop.org/git/xorg/test/xts 141d7e2..7032632 master - master Cheers; I guess I should put xorg-test@ in Cc for further patch reviews? Are there any plans to update what got copied from tet 3.3? Interesting. I

Re: [PATCH intel-gpu-tools 00/10] Upgrade module configuration and packaging (reposted to .cc)

2012-01-05 Thread Gaetan Nadon
On 12-01-04 09:38 AM, Daniel Vetter wrote: On Wed, Jan 4, 2012 at 15:33, Gaetan Nadon mems...@videotron.ca wrote: On 12-01-04 04:52 AM, Daniel Vetter wrote: While I have the attention of someone versed in buildsystem-fu: intel-gpu-tools also contains a set of tests for the i915 kernel module

Re: [PATCH v2 xf85-input-evdev] Set the default resolution to 0

2012-01-05 Thread Peter Hutterer
On Tue, Jan 03, 2012 at 03:31:43PM -0800, Chase Douglas wrote: If we don't know the resolution, set it to 0. This is invalid, and tells the X client that we don't know the resolution, rather than reporting an incorrect value. This value was originally from commit

Re: support for APL character overstrikes in en_US.UTF-8/Compose

2012-01-05 Thread Alan Coopersmith
On 01/03/12 07:28, Geoff Streeter wrote: Is en_US.UTF-8/Compose.pre the only file I should be changing? Should I be creating an apl locale with its own Compose file that is then included in en_US.UTF-8/Compose? I wouldn't create an apl locale per se, and at some point we really need to move

Re: [PATCH xserver] dix: Set XITouchEmulatingPointer on events from the touch sequence emulating pointer events

2012-01-05 Thread Peter Hutterer
On Tue, Jan 03, 2012 at 02:43:40AM +0100, carl...@gnome.org wrote: From: Carlos Garnacho carl...@gnome.org The internal flag is kept around, merely translated to XITouchEmulatingPointer when creating the XI2 events that will be delivered to the client. Signed-off-by: Carlos Garnacho

Re: PING: [PATCH xserver] xkb: Message actions suppress other key presses #28575

2012-01-05 Thread Peter Hutterer
On Tue, Jan 03, 2012 at 11:32:41AM +, Daniel Stone wrote: Hi, On 2 January 2012 20:44, Jeremy Huddleston jerem...@apple.com wrote: Poking Daniel, Chase, and Peter directly since they understand this magic best. On Nov 30, 2011, at 2:20 PM, Andreas Wettstein wrote: When a key

Re: [PATCH] Don't end touchpoint if owning client hasn't accepted/rejected

2012-01-05 Thread Peter Hutterer
On Wed, Jan 04, 2012 at 03:50:40PM -0800, Chase Douglas wrote: A touchpoint is ended when no further processing will take place for it. This includes the situation where there is only one grabbing client, and the client receives a touch end before it has accepted/rejected the touchpoint.

linux intel and xorg status on i915GM hardware

2012-01-05 Thread Knut Petersen
Some comments related to linux and xorg git master on an AOpen i915GMm-hfs: 1: Xorg.0.log is full of (besides timestamp) identical EDID entries: == [ 2502.358] (II) intel(0): EDID vendor ENC, prod id 5769 [ 2502.358] (II) intel(0): Using EDID