[PULL] XQuartz build fix

2014-02-14 Thread Jeremy Huddleston Sequoia
This was missed in the commit mentioned below. commit 60014a4a98ff924ae7f6840781f768c1cc93bbab Author: Keith Packard kei...@keithp.com Date: Sun Dec 15 01:05:51 2013 -0800 Replace 'pointer' type with 'void *' This lets us stop using the 'pointer' typedef in Xdefs.h as 'pointer'

Re: Fixing the kernels backlight API

2014-02-14 Thread Hans de Goede
Hi, On 02/13/2014 09:12 PM, Matthew Garrett wrote: On Thu, Feb 13, 2014 at 08:43:10PM +0100, Hans de Goede wrote: I can understand where Dave is coming from, from a kernel pov, so this might really be easier to just solve in userspace. I don't know if you've seen my very rough sketch of how

Re: [Intel-gfx] What to do with xf86-video-intel backlight control when running Xorg as non root

2014-02-14 Thread Hans de Goede
Hi, On 02/13/2014 09:24 PM, Mark Kettenis wrote: Date: Thu, 13 Feb 2014 20:37:47 +0100 From: Hans de Goede hdego...@redhat.com snip Right, that is what I was thinking too, so the question then becomes how hard you will scream at me if I add something like this to xf86-video-intel linux

Re: [PULL] XQuartz build fix

2014-02-14 Thread Keith Packard
Jeremy Huddleston Sequoia jerem...@apple.com writes: Jeremy Huddleston Sequoia (1): XQuartz: pointer - void * Merged. 72237e0..f34dc7f master - master -- keith.pack...@intel.com pgp2wjYBYbGxc.pgp Description: PGP signature ___

Re: [PATCH 3/3] xkbcomp: Add missing flag support and correct modifier handling of ISOLock

2014-02-14 Thread Ran Benita
On Thu, Jan 23, 2014 at 08:40:00PM +0100, Andreas Wettstein wrote: Add missing support for affect flag to selectively affect locking or unlocking for for modifier locking, control locking, and ISOLock. Fix some incorrect masking and modifier handling for ISOLock. Some small comments below you

Re: [PATCH 2/3] libxkbfile: Add a few missing flags the output of the xkb layout

2014-02-14 Thread Ran Benita
On Thu, Jan 23, 2014 at 08:39:55PM +0100, Andreas Wettstein wrote: Output affect flags for control locks, modifier locks, and ISOLock. Output flag genKeyEvent flag for message actions. Do not generate output for any of these if the flags have their default setting, to avoid problems of

[PATCH] Get rid of config/non-seat0.conf.multi-seat

2014-02-14 Thread Laércio de Sousa
This file is deprecated by commits c73c36b53 and 46cf2a6093. --- config/Makefile.am | 2 +- config/non-seat0.conf.multi-seat | 18 -- 2 files changed, 1 insertion(+), 19 deletions(-) delete mode 100644 config/non-seat0.conf.multi-seat diff --git

Re: [PATCH 1/3] xkb: Several fixes to ISOLock and a few related changes.

2014-02-14 Thread Ran Benita
On Thu, Jan 23, 2014 at 08:39:49PM +0100, Andreas Wettstein wrote: Several fixes to ISOLock: - Use the proper byte to access the affect flags. - When changing a Set/Latch action to a Lock action, also change the flags for the action. - Respect NoLock/NoUnlock. - Add the missing

[PATCH 1/2] kdrive: add support to +X+Y syntax in -screen option parsing

2014-02-14 Thread Laércio de Sousa
This patch enhances current -screen option parsing for kdrive-based applications. It can parse strings like WIDTHxHEIGHT+XOFFSET+YOFFSET, storing X and Y offsets in KdScreenInfo instances. It will allow e.g. proper Xephyr window placement for multiseat purposes. --- hw/kdrive/src/kdrive.c | 33

[PATCH 0/2] Enable Xephyr window placement with kdrive -screen option extended syntax

2014-02-14 Thread Laércio de Sousa
A very requested feature for Xephyr-based multiseat setups is the capability to put Xephyr windows at a given place. There was a historical controversy about providing a -geometry option to Xephyr 6~7 years ago (see bug #12221), but now this feature is much easier to implement and maintain than in

[PATCH 2/2] ephyr: enable screen window placement following kdrive -screen option new syntax

2014-02-14 Thread Laércio de Sousa
With this patch, one can launch Xephyr with option -screen WxH+X+Y to place its window origin at (X,Y). This patch relies on a previous one that extends kdrive -screen option syntax to parse +X+Y substring as expected. In my tests, any try to set Xephyr window placement using extended -screen

[PATCH] ephyr: enable screen window placement following kdrive -screen option new syntax

2014-02-14 Thread Laércio de Sousa
With this patch, one can launch Xephyr with option -screen WxH+X+Y to place its window origin at (X,Y). This patch relies on a previous one that extends kdrive -screen option syntax to parse +X+Y substring as expected. In my tests, any try to set Xephyr window placement using extended -screen

Re: [PATCH] dix: Clear any existing selections before initializing privates

2014-02-14 Thread Andrew Eikum
On Mon, Feb 03, 2014 at 04:16:54PM -0500, Adam Jackson wrote: On Mon, 2013-12-30 at 09:15 -0600, Andrew Eikum wrote: To fix this, we should delete any existing selections before calling dixResetPrivates(). This will properly release the selection's privates and avoid the crash.

Re: [PATCH 3/3] xkbcomp: Add missing flag support and correct modifier handling of ISOLock

2014-02-14 Thread wettstein509
Some small comments below you might want to address, but: Thank you for looking at it. I will look at your comments and send a revised patch. Andreas ___ xorg-devel@lists.x.org: X.Org development Archives: http://lists.x.org/archives/xorg-devel Info:

Re: [PATCH 2/3] libxkbfile: Add a few missing flags the output of the xkb layout

2014-02-14 Thread wettstein509
Everything looks good to me, but I think you should split this to 3 patches (see comments). Also you've introduced some tabs. Thank you. This makes sense, I will split as you suggested and watch better for the tabs. Andreas ___

Re: [PATCH 1/3] xkb: Several fixes to ISOLock and a few related changes.

2014-02-14 Thread wettstein509
I've yet to look at the xserver changes, but may I ask to split all those fixes up? Ok, this makes even more sense than for libxkbfile... Andreas ___ xorg-devel@lists.x.org: X.Org development Archives: http://lists.x.org/archives/xorg-devel Info:

Compiling xserver with --disable-xorg fails in test subdir

2014-02-14 Thread Gaetan Nadon
make[1]: Entering directory `/home/nadon/xorg/src/xserver/test' ../dix/dix.O: In function `dix_main': /home/nadon/xorg/src/xserver/dix/main.c:200: undefined reference to `InitOutput' /home/nadon/xorg/src/xserver/dix/main.c:264: undefined reference to `InitInput'

Re: [PATCH 09/15] glamor: Unifdef the picture-format-to-format-and-type functions.

2014-02-14 Thread Eric Anholt
Keith Packard kei...@keithp.com writes: Eric Anholt e...@anholt.net writes: There's no way these should be in a header file, but I'll leave that cleanup until later. I don't understand this comment; did you flip the order of 08/15 and 09/15 at some point and not rewrite it? Yep.

[PULL] reviewed glamor changes

2014-02-14 Thread Eric Anholt
Here's the pull request for the reviewed parts of the glamor stuff I'd sent out last week. I dropped the xephyr parts since keithp had some comments about fixing the init sequence (and there's definitely some bogus stuff in the init sequence I had). Hopefully next week I can get out a series

Re: [PATCH xfs 3/3] Pass a const char * port to TRANS(ReopenCOTSServer).

2014-02-14 Thread Alan Coopersmith
On 01/29/14 01:14 PM, Keith Packard wrote: This patch requires a related patch to libxtrans which accepts a const char * value for this parameter. The string passed is constant, so we should make sure that gets respected through the whole API stack. Signed-off-by: Keith Packard

Re: [PATCH xfs 2/3] Define VErrorF and then try to get libxtrans to use our error log

2014-02-14 Thread Alan Coopersmith
On 01/29/14 01:14 PM, Keith Packard wrote: void -NoticeF(const char *f, ...) +VErrorF(const char *f, va_list args) { -/* XXX should Notices just be ignored if not using syslog? */ -va_list args; -va_start(args, f); #ifdef USE_SYSLOG if (UseSyslog) {