Re: [PATCH:xrdb 2/3] Replace complex malloc calculations with asprintf()

2011-01-05 Thread Julien Cristau
- 2 files changed, 47 insertions(+), 28 deletions(-) This is going to conflict with the -P patch all over the place :) Reviewed-by: Julien Cristau jcris...@debian.org Cheers, Julien ___ xorg-devel@lists.x.org

Re: [PATCH:xrdb 3/3] Merge usage() printf() strings/calls into a single string/call

2011-01-05 Thread Julien Cristau
On Tue, Jan 4, 2011 at 20:31:42 -0800, Alan Coopersmith wrote: Signed-off-by: Alan Coopersmith alan.coopersm...@oracle.com --- xrdb.c | 73 +--- 1 files changed, 24 insertions(+), 49 deletions(-) Reviewed-by: Julien Cristau

[PATCH 1/5] glx: make sure screen is non-negative in validGlxScreen

2011-01-03 Thread Julien Cristau
Reviewed-by: Adam Jackson a...@redhat.com Signed-off-by: Julien Cristau jcris...@debian.org --- glx/glxcmds.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/glx/glxcmds.c b/glx/glxcmds.c index de9c3f0..419cc46 100644 --- a/glx/glxcmds.c +++ b/glx/glxcmds.c @@ -56,7

[PATCH 2/5] glx: validate request lengths

2011-01-03 Thread Julien Cristau
Reviewed-by: Adam Jackson a...@redhat.com Signed-off-by: Julien Cristau jcris...@debian.org --- glx/glxcmds.c | 142 + glx/xfont.c |2 + 2 files changed, 135 insertions(+), 9 deletions(-) diff --git a/glx/glxcmds.c b/glx/glxcmds.c

[PATCH 3/5] glx: check request length before swapping

2011-01-03 Thread Julien Cristau
Signed-off-by: Julien Cristau jcris...@debian.org --- glx/glxcmdsswap.c | 135 +++-- 1 files changed, 130 insertions(+), 5 deletions(-) diff --git a/glx/glxcmdsswap.c b/glx/glxcmdsswap.c index ce4d69a..9276e2f 100644 --- a/glx/glxcmdsswap.c +++ b

[PATCH 4/5] glx: swap the request arrays entirely, not just half of them

2011-01-03 Thread Julien Cristau
Various glx request include a list of pairs of attributes. We were only swapping the first half. Signed-off-by: Julien Cristau jcris...@debian.org --- glx/glxcmdsswap.c | 10 +- 1 files changed, 5 insertions(+), 5 deletions(-) diff --git a/glx/glxcmdsswap.c b/glx/glxcmdsswap.c index

[PATCH 5/5] glx: validate numAttribs field before using it

2011-01-03 Thread Julien Cristau
Signed-off-by: Julien Cristau jcris...@debian.org --- glx/glxcmds.c | 25 + glx/glxcmdsswap.c | 20 2 files changed, 45 insertions(+), 0 deletions(-) diff --git a/glx/glxcmds.c b/glx/glxcmds.c index 566dbbe..3ef567d 100644 --- a/glx/glxcmds.c

Re: [PATCH modular 2/4] build.sh: use accurate values for HOST_CPU from config.guess

2011-01-02 Thread Julien Cristau
On Sun, Jan 2, 2011 at 08:58:03 -0500, Gaetan Nadon wrote: Using x86*64* rather than x86_64 is misleading to the user as it implies there are a large number of possibilities. What user are you talking about? Cheers, Julien ___

Re: [PATCH modular 2/4] build.sh: use accurate values for HOST_CPU from config.guess

2011-01-02 Thread Julien Cristau
On Sun, Jan 2, 2011 at 14:34:49 -0500, Gaetan Nadon wrote: On Sun, 2011-01-02 at 15:48 +0100, Mark Kettenis wrote: This bit is almost certainly not correct. The canonical HOST_CPU value for 64-bit SPARC (aka SPARC V9) is sparc64. Yes, I missed this one as I saw a lot of sparc

Re: xserver: Branch 'master' - 11 commits

2010-12-30 Thread Julien Cristau
On Mon, Dec 6, 2010 at 19:59:18 -0800, Keith Packard wrote: commit 27593eea7efcbed8de0c6e8233cbd1a1b8a50459 Author: James Jones jajo...@nvidia.com Date: Mon Dec 6 13:36:14 2010 -0800 Add and use SERVER_SYNC_*_VERSION Most extensions have a version defined in the

Re: [PATCH:xorg-docs] X.man: Remove DECnet addresses for font servers as well

2010-12-27 Thread Julien Cristau
(-) Sorry for this oversight. Acked-by: Julien Cristau jcris...@debian.org Cheers, Julien ___ xorg-devel@lists.x.org: X.Org development Archives: http://lists.x.org/archives/xorg-devel Info: http://lists.x.org/mailman/listinfo/xorg-devel

Re: [PATCH bitmap] Fix memory leak.

2010-12-23 Thread Julien Cristau
On Thu, Dec 23, 2010 at 12:01:03 -0500, Trevor Woerner wrote: From: Trevor Woerner twoer...@gmail.com Signed-off-by: Trevor Woerner twoer...@gmail.com --- bmtoa.c |1 + 1 files changed, 1 insertions(+), 0 deletions(-) Reviewed-by: Julien Cristau jcris...@debian.org Cheers, Julien

Re: [PATCH:xf86-input-keyboard] sun_kbd: Replace deprecated xfree/xcalloc with free calloc

2010-12-23 Thread Julien Cristau
On Thu, Dec 23, 2010 at 16:00:33 -0800, Alan Coopersmith wrote: Reworked char * pointers to avoid introducing new warnings of: sun_kbd.c:512: warning: passing arg 1 of `free' discards qualifiers from pointer target type Signed-off-by: Alan Coopersmith alan.coopersm...@oracle.com ---

Re: [PATCH] x86emu: Fix more mis-decoding of the data prefix

2010-12-20 Thread Julien Cristau
On Mon, Dec 20, 2010 at 16:55:18 -0500, Adam Jackson wrote: On Mon, 2010-12-20 at 19:41 +0100, Julien Cristau wrote: On Fri, Dec 10, 2010 at 14:47:20 -0500, Adam Jackson wrote: @@ -7084,8 +7089,12 @@ static void x86emuOp_call_far_IMM(u8 X86EMU_UNUSED(op1)) TRACE_AND_STEP

Re: [PATCH] Bug 32436 - hw/xfree86/parser/Makefile.am TOP_SRCDIR should be top_srcdir

2010-12-17 Thread Julien Cristau
@@ libxf86config_internal_la_SOURCES = \ $(INTERNAL_SOURCES) libxf86config_a_SOURCES = \ - $(TOP_SRCDIR)/os/xprintf.c \ + $(top_srcdir)/os/xprintf.c \ $(INTERNAL_SOURCES) libxf86config_a_CFLAGS = $(AM_CFLAGS) Reviewed-by: Julien Cristau jcris...@debian.org Cheers, Julien

Re: [PATCH 1/2] GetTimeInMillis: Use CLOCK_MONOTONIC_COARSE where available

2010-12-17 Thread Julien Cristau
-by: Daniel Stone dan...@fooishbar.org --- os/utils.c | 16 +++- 1 files changed, 15 insertions(+), 1 deletions(-) Reviewed-by: Julien Cristau jcris...@debian.org Cheers, Julien ___ xorg-devel@lists.x.org: X.Org development Archives

Re: [PATCH] Remove SCO support

2010-12-16 Thread Julien Cristau
On Thu, Dec 16, 2010 at 16:03:11 -0500, Adam Jackson wrote: This has never been buildable in any modular server release. Signed-off-by: Adam Jackson a...@redhat.com Reviewed-by: Julien Cristau jcris...@debian.org Cheers, Julien ___ xorg-devel

Re: [PATCH:libICE] Convert src/process.c from malloc + sprintf to asprintf

2010-12-09 Thread Julien Cristau
+ 2 files changed, 62 insertions(+), 32 deletions(-) Reviewed-by: Julien Cristau jcris...@debian.org Cheers, Julien ___ xorg-devel@lists.x.org: X.Org development Archives: http://lists.x.org/archives/xorg

Re: [PATCH 1/4] Fix dmx linking.

2010-12-07 Thread Julien Cristau
the problem was that once (core) dmx got rid of the deprecated functions, the linker would discard dix/deprecated.o before seeing glxProxy/libglxproxy.a, which did need it. I think I'd go for the first patch just because it's simpler. So add my Reviewed-by: Julien Cristau jcris...@debian.org

Re: [PATCH xf86-video-sunffb] Remove xf86DPMSInit() duplication

2010-12-07 Thread Julien Cristau
On Tue, Dec 7, 2010 at 20:49:20 +0100, Mark Kettenis wrote: This driver calls xf86DPMSInit() twice, which is a bad idea since this function wraps CloseScreen(). Remove the second call. Fixes a crash on server shutdown with xserver 1.9.2 on OpenBSD/sparc64. Signed-off-by: Mark Kettenis

Re: [PATCH xf86-video-sunffb] Remove xf86DPMSInit() duplication

2010-12-07 Thread Julien Cristau
On Tue, Dec 7, 2010 at 20:59:36 +0100, Julien Cristau wrote: FFBDPMSSet() calls FFBDacDPMSMode(GET_FFB_FROM_SCRN(pScrn), DPMSMode, 0), ignoring the flags. Which one was correct? Nevermind, FFBDacDPMSMode doesn't care about the flags. Reviewed-by: Julien Cristau jcris...@debian.org Cheers

Re: [PATCH libSM] Export only public API symbols

2010-12-06 Thread Julien Cristau
On Mon, Dec 6, 2010 at 12:19:12 -0300, Cristian Rodríguez wrote: Signed-off-by: Cristian Rodríguez cristian.rodrig...@opensuse.org --- src/Makefile.am |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/src/Makefile.am b/src/Makefile.am index 1098679..d194d3f

Re: [PATCH v4 6/7] Replace alloc+strcpy+strcat with asprintf() XNFasprintf() calls

2010-12-05 Thread Julien Cristau
On Sun, Dec 5, 2010 at 11:10:23 +0100, walter harms wrote: diff --git a/hw/xfree86/common/xf86ShowOpts.c b/hw/xfree86/common/xf86ShowOpts.c index ce86090..c0fa80a 100644 --- a/hw/xfree86/common/xf86ShowOpts.c +++ b/hw/xfree86/common/xf86ShowOpts.c @@ -97,11 +97,8 @@ void

Re: [PATCH:libXmu] XmuCvtStringToCursor: sprintf - snprintf check for XtMalloc failure

2010-12-05 Thread Julien Cristau
On Sun, Dec 5, 2010 at 21:43:50 +0100, walter harms wrote: Can you use XFNasprintf() here ? libXmu is not the X server. Cheers, Julien ___ xorg-devel@lists.x.org: X.Org development Archives: http://lists.x.org/archives/xorg-devel Info:

Re: [PATCH 1/3] Fix missing string.h include.

2010-12-04 Thread Julien Cristau
--- a/include/os.h +++ b/include/os.h @@ -51,6 +51,7 @@ SOFTWARE. #include misc.h #include stdarg.h +#include string.h #define SCREEN_SAVER_ON 0 #define SCREEN_SAVER_OFF 1 Reviewed-by: Julien Cristau jcris...@debian.org Cheers, Julien

Re: [PATCH libXaw] fix potential infinte loop in XawBoxQueryGeometry() (bug 11569)

2010-12-04 Thread Julien Cristau
On Sat, Dec 4, 2010 at 13:26:09 +0100, walter harms wrote: there seems some more bugs described in the bugzilla, some a are and some seems fixed already but appear still open. Who can close these bugs ? Anyone. You just need a bugzilla login. If you'd like to triage Xaw bugs in

Re: [PATCH citron] Fail configure. This driver is unmaintained.

2010-12-02 Thread Julien Cristau
On Thu, Dec 2, 2010 at 14:42:44 +0100, Mark Kettenis wrote: Date: Wed, 01 Dec 2010 21:31:04 -0800 From: Alan Coopersmith alan.coopersm...@oracle.com The MAINTAINERS file already says it's deactivated. Where does the MAINTAINERS file live? I should probably list myself there for

Re: [PATCH libXext (rev3) 1/5] Backwards compat for newer XSync + older servers

2010-12-02 Thread Julien Cristau
On Thu, Dec 2, 2010 at 09:26:27 -0800, James Jones wrote: +for (i = 0; i NUM_VERSIONS; i++) { + if (supported_versions[i].major == rep.majorVersion + supported_versions[i].minor == rep.minorVersion) { + return supported_versions[i]; + } +} + Should this

Re: [PATCH 1/6 v2] Add X*asprintf() routines to mirror common asprintf() routines

2010-12-01 Thread Julien Cristau
| 163 +++-- 7 files changed, 216 insertions(+), 29 deletions(-) create mode 100644 include/Xprintf.h Reviewed-by: Julien Cristau jcris...@debian.org Cheers, Julien ___ xorg-devel@lists.x.org: X.Org development

Re: [PATCH 5/6 v2] Replace alloc+strcpy+strcat with Xasprintf calls

2010-12-01 Thread Julien Cristau
+ hw/xfree86/common/xf86ShowOpts.c|7 ++- hw/xfree86/dixmods/extmod/modinit.c |5 + hw/xfree86/loader/loadmod.c |6 ++ hw/xfree86/modes/xf86Crtc.c |8 ++-- 7 files changed, 20 insertions(+), 32 deletions(-) Reviewed-by: Julien Cristau jcris

Re: [PATCH 2/6 v2] Convert existing Xprintf style calls to Xasprintf style

2010-12-01 Thread Julien Cristau
| 23 +++--- xkb/ddxList.c | 43 ++- xkb/ddxLoad.c |8 -- 12 files changed, 80 insertions(+), 53 deletions(-) Reviewed-by: Julien Cristau jcris...@debian.org Cheers, Julien

Re: [PATCH 2/2] render: Fix byteswapping of gradient stops

2010-12-01 Thread Julien Cristau
-by: Julien Cristau jcris...@debian.org Keith, can you take this one? It would be nice to have it on the stable branches. Cheers, Julien ___ xorg-devel@lists.x.org: X.Org development Archives: http://lists.x.org/archives/xorg-devel Info: http://lists.x.org

Re: [PATCH 3/3] Fix compiler warnings in hw/xfree86/os-support/solaris

2010-11-30 Thread Julien Cristau
+++ hw/xfree86/os-support/solaris/sun_vid.c |5 ++--- 3 files changed, 6 insertions(+), 8 deletions(-) Not that I can test this or anything, but looks like a good change, so Reviewed-by: Julien Cristau jcris...@debian.org Cheers, Julien ___ xorg

Re: [PATCH 01/11] xf86ValidateModes: xnfalloc(strlen) + strcpy = xnfstrdup

2010-11-30 Thread Julien Cristau
On Mon, Nov 29, 2010 at 20:57:38 -0800, Alan Coopersmith wrote: Signed-off-by: Alan Coopersmith alan.coopersm...@oracle.com --- hw/xfree86/common/xf86Mode.c |6 ++ 1 files changed, 2 insertions(+), 4 deletions(-) Reviewed-by: Julien Cristau jcris...@debian.org Cheers, Julien

Re: [PATCH 02/11] xf86AutoConfig: make copyScreen memory allocation error handling more sane

2010-11-30 Thread Julien Cristau
allocation, drop your previous allocations before returning to others. Signed-off-by: Alan Coopersmith alan.coopersm...@oracle.com --- hw/xfree86/common/xf86AutoConfig.c | 20 +++- 1 files changed, 15 insertions(+), 5 deletions(-) Reviewed-by: Julien Cristau jcris...@debian.org

Re: [PATCH 03/11] xf86VIDrvMsgVerb: print args, not format string

2010-11-30 Thread Julien Cristau
On Mon, Nov 29, 2010 at 20:57:40 -0800, Alan Coopersmith wrote: Signed-off-by: Alan Coopersmith alan.coopersm...@oracle.com --- hw/xfree86/common/xf86Helper.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) Reviewed-by: Julien Cristau jcris...@debian.org Cheers, Julien

Re: [PATCH 04/11] FindModuleInSubdir: Stop allocating one more byte than needed

2010-11-30 Thread Julien Cristau
+++--- 1 files changed, 3 insertions(+), 3 deletions(-) Reviewed-by: Julien Cristau jcris...@debian.org Cheers, Julien ___ xorg-devel@lists.x.org: X.Org development Archives: http://lists.x.org/archives/xorg-devel Info: http://lists.x.org/mailman/listinfo

Re: [PATCH 05/11] Simplify Error() - don't allocate temporary copy of error string

2010-11-30 Thread Julien Cristau
++- 2 files changed, 7 insertions(+), 14 deletions(-) Reviewed-by: Julien Cristau jcris...@debian.org Cheers, Julien ___ xorg-devel@lists.x.org: X.Org development Archives: http://lists.x.org/archives/xorg-devel Info: http://lists.x.org

Re: [PATCH 06/11] Add X*asprintf() routines to mirror common asprintf() routines

2010-11-30 Thread Julien Cristau
On Mon, Nov 29, 2010 at 20:57:43 -0800, Alan Coopersmith wrote: +/* Old api, now deprecated, may be removed in a future release */ +char * +Xvprintf(const char *format, va_list va) +{ +char *ret; + +Xvasprintf(ret, format, va); + return ret; } @@ -68,7 +191,7 @@ char

Re: [PATCH 08/11] Convert hw/xfree86/parser code to use Xasprintf() calls

2010-11-30 Thread Julien Cristau
+ hw/xfree86/parser/Pointer.c | 16 ++-- 3 files changed, 11 insertions(+), 19 deletions(-) Reviewed-by: Julien Cristau jcris...@debian.org Cheers, Julien ___ xorg-devel@lists.x.org: X.Org development Archives: http://lists.x.org

Re: [PATCH 09/11] Convert malloc+sprintf pairs into X*asprintf() calls

2010-11-30 Thread Julien Cristau
On Mon, Nov 29, 2010 at 20:57:46 -0800, Alan Coopersmith wrote: diff --git a/hw/xfree86/loader/loadmod.c b/hw/xfree86/loader/loadmod.c index 06d082b..751fec6 100644 --- a/hw/xfree86/loader/loadmod.c +++ b/hw/xfree86/loader/loadmod.c @@ -406,22 +406,22 @@ FindModuleInSubdir(const char

Re: [PATCH 10/11] Replace alloc+strcpy+strcat with Xasprintf calls

2010-11-30 Thread Julien Cristau
On Mon, Nov 29, 2010 at 20:57:47 -0800, Alan Coopersmith wrote: diff --git a/dix/devices.c b/dix/devices.c index 708860a..db38c1a 100644 --- a/dix/devices.c +++ b/dix/devices.c @@ -2524,9 +2524,10 @@ AllocDevicePair (ClientPtr client, char* name, if (!pointer) return

Re: [PATCH 11/11] Convert cvt code to use Xasprintf() calls

2010-11-30 Thread Julien Cristau
/xfree86/utils/cvt/cvt.c | 18 ++ 3 files changed, 23 insertions(+), 10 deletions(-) Reviewed-by: Julien Cristau jcris...@debian.org Cheers, Julien ___ xorg-devel@lists.x.org: X.Org development Archives: http://lists.x.org/archives

[PATCH] DGA: fix screen number check

2010-11-30 Thread Julien Cristau
screenInfo.numScreens is not a valid screen number, they go from 0 to numScreens - 1. Signed-off-by: Julien Cristau jcris...@debian.org --- hw/xfree86/dixmods/extmod/xf86dga2.c | 46 +- 1 files changed, 23 insertions(+), 23 deletions(-) diff --git a/hw/xfree86

Re: [PATCH] xfree86: Remove the xf8_16bpp overlay module

2010-11-30 Thread Julien Cristau
case, Reviewed-by: Julien Cristau jcris...@debian.org Cheers, Julien ___ xorg-devel@lists.x.org: X.Org development Archives: http://lists.x.org/archives/xorg-devel Info: http://lists.x.org/mailman/listinfo/xorg-devel

Re: [PATCH 09/11] Convert malloc+sprintf pairs into X*asprintf() calls

2010-11-30 Thread Julien Cristau
On Tue, Nov 30, 2010 at 15:15:22 -0800, Alan Coopersmith wrote: Julien Cristau wrote: Should these use XNFasprintf? Certainly the previous code effectively did that (though by just SEGV on writing to the NULL value returned by malloc, instead of an orderly abort), but since the callers

Re: [PATCH] dix: do not use bit-wise operators on the boolean result of BitIsOn

2010-11-29 Thread Julien Cristau
On Mon, Nov 29, 2010 at 17:09:31 +0100, Ferry Huberts wrote: From: Ferry Huberts ferry.hube...@pelagic.nl Signed-off-by: Ferry Huberts ferry.hube...@pelagic.nl --- dix/getevents.c | 12 ++-- 1 files changed, 6 insertions(+), 6 deletions(-) I'm missing the why for this change.

Bug#551840: xserver-xorg-video-radeon: X server won't start with disabled kms

2010-11-27 Thread Julien Cristau
On Sat, Nov 27, 2010 at 14:09:05 +0500, Eugene V. Samusev wrote: Package: xserver-xorg-video-radeon Version: 1:6.13.1-2+squeeze1 Severity: normal With enabled kms xserver start successfully, but i get incorrectly displayed fonts. So i disabled kms and lived happily for a long time. But

Re: [PATCH v2] include: let BitIsOn() return a boolean value.

2010-11-25 Thread Julien Cristau
/bit_test_macros, include_bit_test_macros); g_test_add_func(/Xi/xiproperty/register-unregister, xi_unregister_handlers); Reviewed-by: Julien Cristau jcris...@debian.org Cheers, Julien ___ xorg-devel@lists.x.org: X.Org development Archives: http

Re: [PATCH:libxt] Fix missing const qualifier.

2010-11-25 Thread Julien Cristau
On Thu, Nov 25, 2010 at 18:54:55 +0100, Cyril Brulebois wrote: In libX11's c8701115, atom names argument to XInternAtoms were marked as const. This fixes the build with CFLAGS=-Wall -Werror: | CC Selection.lo | cc1: warnings being treated as errors | Selection.c: In function

Bug#585777: xserver-xorg-video-radeon: Radeon gamma bug still there

2010-11-23 Thread Julien Cristau
On Tue, Nov 23, 2010 at 22:10:18 +0100, V. Gaibler wrote: On 23.11.2010 08:31, Julien Cristau wrote: Volker, you seem to be using 2.6.32-27, which has that patch. Yes and no Yes, the kernel sources I get with aptitude install linux-source-2.6.32 are version 2.6.32-27 and include

Re: [PATCH] test: compare byte padding macros against the expected bytes.

2010-11-22 Thread Julien Cristau
On Fri, Nov 19, 2010 at 11:26:13 +1000, Peter Hutterer wrote: We calculate the expected bytes for each value, let's use it. Signed-off-by: Peter Hutterer peter.hutte...@who-t.net --- test/input.c |3 +++ 1 files changed, 3 insertions(+), 0 deletions(-) Reviewed-by: Julien Cristau

Re: [PATCH] test: reduce range of byte-padding macro tests.

2010-11-22 Thread Julien Cristau
. } static void xi_unregister_handlers(void) Other than that, Reviewed-by: Julien Cristau jcris...@debian.org Cheers, Julien ___ xorg-devel@lists.x.org: X.Org development Archives: http://lists.x.org/archives/xorg-devel Info: http://lists.x.org

Re: [PATCH libICE docs] Add top-level target for documentation.

2010-11-22 Thread Julien Cristau
On Mon, Nov 22, 2010 at 17:26:38 -0500, Trevor Woerner wrote: From: Trevor Woerner twoer...@gmail.com Builds user and developer documentation/specifications. Signed-off-by: Trevor Woerner twoer...@gmail.com --- Makefile.am |7 ++- 1 files changed, 6 insertions(+), 1

Bug#585777: xserver-xorg-video-radeon: Radeon gamma bug still there

2010-11-22 Thread Julien Cristau
On Tue, Nov 23, 2010 at 01:56:13 -0500, Alex Deucher wrote: On Mon, Nov 22, 2010 at 9:14 PM, V. Gaibler li...@volker-gaibler.de wrote: Package: xserver-xorg-video-radeon Version: 1:6.13.1-2+squeeze1 Severity: normal The problem is still there in squeeze for graphics card Radeon

Re: [PATCH:libXaw 1/2] convert header checks/ifdefs to autoconf standard AC_CHECK_HEADERS

2010-11-21 Thread Julien Cristau
On Sat, Nov 20, 2010 at 20:11:12 -0800, Alan Coopersmith wrote: Signed-off-by: Alan Coopersmith alan.coopersm...@oracle.com --- configure.ac | 10 +- src/XawI18n.h |6 +++--- 2 files changed, 4 insertions(+), 12 deletions(-) Reviewed-by: Julien Cristau jcris...@debian.org

Re: [PATCH:libXaw 2/2] Check for getpagesize() with autoconf instead of #ifdef osname

2010-11-21 Thread Julien Cristau
On Sat, Nov 20, 2010 at 20:11:13 -0800, Alan Coopersmith wrote: Signed-off-by: Alan Coopersmith alan.coopersm...@oracle.com --- configure.ac |2 +- src/OS.c | 11 +-- 2 files changed, 2 insertions(+), 11 deletions(-) Reviewed-by: Julien Cristau jcris...@debian.org

Re: [PATCH:libfontenc] Fail configure if zlib.h -lz not found

2010-11-20 Thread Julien Cristau
On Sat, Nov 20, 2010 at 08:22:09 -0800, Dan Nicholson wrote: On Fri, Nov 19, 2010 at 10:04 PM, Alan Coopersmith alan.coopersm...@oracle.com wrote: Fixes https://bugs.freedesktop.org/show_bug.cgi?id=31595 Signed-off-by: Alan Coopersmith alan.coopersm...@oracle.com ---  configure.ac |  

Re: [PATCH xfs] config: look for fontpath.d only if --with-default-font-path wasn't specified #31290

2010-11-20 Thread Julien Cristau
On Sat, Nov 20, 2010 at 08:16:23 -0800, Dan Nicholson wrote: xserver master has this wrapped in a $cross_compiling test. What branch is this patch against? It's against xfs, not xserver. Cheers, Julien ___ xorg-devel@lists.x.org: X.Org development

Re: [RFC XI 2.1 - xinput 3/3] Add experimental multitouch support from XI 2.1

2010-11-19 Thread Julien Cristau
On Fri, Nov 12, 2010 at 17:35:17 -0500, Chase Douglas wrote: From: Daniel Stone dan...@fooishbar.org This patch adds experimental support for listening to touch streams (TouchBegin, TouchMotion and TouchEnd) with test-xi2, as well as showing TouchClass information with list.

Re: [PATCH] glx: Remove swap barrier and hyperpipe support

2010-11-19 Thread Julien Cristau
implementation of these should be done with additional DRI driver API, like the swap control extension. Signed-off-by: Adam Jackson a...@redhat.com Reviewed-by: Julien Cristau jcris...@debian.org Cheers, Julien ___ xorg-devel@lists.x.org: X.Org

Re: [PATCH] fix xserver segfaulting in glxext.c:DrawableGone() (NULL pointer)

2010-11-17 Thread Julien Cristau
On Wed, Nov 17, 2010 at 07:53:30 +0100, Thierry Vignaud wrote: Hi I've attached a patch[1] against bug #31538 [2] that fix xserver segfaulting in glxext.c:DrawableGone() on NULL pointer. See stack trace there[3]. It's against xserver-1.7. xserver-1.9 doesn't trigger it anymore but I

Re: [PATCH] Build libxf86config with -fPIC.

2010-11-15 Thread Julien Cristau
On Sun, Nov 14, 2010 at 21:31:42 -0800, Dan Nicholson wrote: Why not just make libxf86config into a libtool library? Then libtool will take care of all the PIC details, follow the --with-pic configuration, and be static or shared as requested by the user. Here's an untested diff, but I think

Re: [PATCH] Build libxf86config with -fPIC.

2010-11-15 Thread Julien Cristau
On Mon, Nov 15, 2010 at 13:28:34 -0500, Gaetan Nadon wrote: On Mon, 2010-11-15 at 15:35 +0100, Julien Cristau wrote: If you want to install it as a shared library you also want to track its ABI and give it a SONAME, which is more work. Shipping it as a PIC static lib which you don't

Re: [PATCH] autoconfig: select nouveau by default for NVIDIA GPUs

2010-11-15 Thread Julien Cristau
On Tue, Nov 16, 2010 at 00:09:09 +0100, Thierry Vignaud wrote: - case 0x10de: case 0x12d2: driverList[0] = nv; break; + case 0x10de: case 0x12d2: + switch (dev-device_id) { + /* NV1 */ + case 0x0008: + case 0x0009: + driverList[0]

Re: Broken make check with 1.9.3 rc1

2010-11-14 Thread Julien Cristau
On Sun, Nov 14, 2010 at 11:23:04 +0100, Cyril Brulebois wrote: Hi Jeremy, looks like 1.9.3 rc1 doesn't pass “make check”: | /dix/input/init-valuators: OK | /dix/input/event-core-conversion: [dix] EventToCore: Not implemented yet | [dix] EventToCore: Not implemented yet | [dix]

Bug#603469: xserver-xorg-video-ati: screen randomly goes black with no way to bring it back

2010-11-14 Thread Julien Cristau
reassign 603469 linux-2.6 2.6.32-27 kthxbye On Sun, Nov 14, 2010 at 14:32:40 +0100, lauren wrote: Package: xserver-xorg-video-ati Version: 1:6.13.1-2 Severity: important after some random time period my screen goes black and nothing i do short of rebooting can bring it back ... i have

Re: [PATCH] xfree86: rename allowEmptyInput to forceInputDevices.

2010-11-11 Thread Julien Cristau
+- hw/xfree86/os-support/linux/lnx_init.c |2 +- 4 files changed, 23 insertions(+), 22 deletions(-) Reviewed-by: Julien Cristau jcris...@debian.org Cheers, Julien ___ xorg-devel@lists.x.org: X.Org development Archives: http://lists.x.org

Re: [PATCH] xfree86: rename allowEmptyInput to forceInputDevices.

2010-11-11 Thread Julien Cristau
On Thu, Nov 11, 2010 at 10:08:20 +0100, Matthieu Herrb wrote: I don't see a diff hunk to actually rename the option in xorg.conf and provide backward compatibility with the previous name, nor any associated documentation update. Am I missing something or what are your intents wrt the

Re: [PATCH] Xi: move property reset from extension shutdown to init.

2010-11-11 Thread Julien Cristau
before input devices are initialized before the first property is requested. Signed-off-by: Peter Hutterer peter.hutte...@who-t.net Looks like this one fell through the cracks, noticed while going through the rhel6 patch queue... Reviewed-by: Julien Cristau jcris...@debian.org Cheers, Julien

Re: [PATCH] os: include dix-config.h, not xorg-config.h

2010-11-10 Thread Julien Cristau
On Tue, May 18, 2010 at 14:58:15 +1000, Peter Hutterer wrote: On Mon, May 17, 2010 at 07:39:54PM +0200, Julien Cristau wrote: os/strlc{at,py}.c were trying to include xorg-config.h, which is not available in dix. Signed-off-by: Julien Cristau jcris...@debian.org Acked-by: Peter

Re: fop requiring a GUI

2010-11-10 Thread Julien Cristau
On Wed, Nov 10, 2010 at 06:06:22 -0800, Dan Nicholson wrote: On Tue, Nov 9, 2010 at 7:43 PM, Jeremy Huddleston jerem...@apple.com wrote: This came up a while ago that building documentation failed without a GUI due to some oddities in fop... this can be worked around by adding

Bug#595767: Won't load hardware acceleration after system start

2010-11-09 Thread Julien Cristau
On Tue, Nov 9, 2010 at 23:40:12 +0100, Ivan Vilata i Balaguer wrote: How should I disable it? Something like 'radeonfb.disable' on the kernel command line might prevent it from loading (untested). Cheers, Julien signature.asc Description: Digital signature

Re: [PATCH] try the newport driver on mips

2010-11-08 Thread Julien Cristau
On Mon, Nov 8, 2010 at 08:30:37 +0100, Mark Kettenis wrote: Actually, I don't think this is a good idea. Unless I'm mistaken, the newport driver only supports one particular range of SGI graphics hardware. There are quite a few other SGI graphics boards out there and other non-SGI mips

Re: [PATCH 1/3] Xi: split hierarchy manipulation into static functions.

2010-11-08 Thread Julien Cristau
!= XIAttachToMaster +r-return_mode != XIFloating) +return BadValue; + Probably for a followup patch, but set client-errorValue to r-return_mode here? There's a couple other places around there that might want to set errorValue as well, I think. For the patch itself, Reviewed-by: Julien

Re: [PATCH 2/3] Xi: rename two variables from ptr to dev.

2010-11-08 Thread Julien Cristau
++-- 1 files changed, 14 insertions(+), 14 deletions(-) Reviewed-by: Julien Cristau jcris...@debian.org Cheers, Julien ___ xorg-devel@lists.x.org: X.Org development Archives: http://lists.x.org/archives/xorg-devel Info: http

Re: [PATCH 3/3] Xi: if XTEST device creation fails, fail the master devices.

2010-11-08 Thread Julien Cristau
-by: Julien Cristau jcris...@debian.org Cheers, Julien ___ xorg-devel@lists.x.org: X.Org development Archives: http://lists.x.org/archives/xorg-devel Info: http://lists.x.org/mailman/listinfo/xorg-devel

Re: [PATCH] Allow building without XV.

2010-11-08 Thread Julien Cristau
On Mon, Nov 8, 2010 at 13:41:35 +0100, Cyril Brulebois wrote: Julien Cristau jcris...@debian.org (07/11/2010): This should probably just add #ifdef XV around that block in src/fbdev.c, so it follows the configuration of the server you're building against (xorg-server.h). In which case

Re: [PATCH] Stop mentioning server's -scanpci option.

2010-11-08 Thread Julien Cristau
a 'man:' prefix to the commit message, but Reviewed-by: Julien Cristau jcris...@debian.org Cheers, Julien signature.asc Description: Digital signature ___ xorg-devel@lists.x.org: X.Org development Archives: http://lists.x.org/archives/xorg-devel Info

Re: [PATCH] Avoid unused variables when XSERVER_LIBPCIACCESS is defined.

2010-11-08 Thread Julien Cristau
On Mon, Nov 8, 2010 at 13:36:47 +0100, Cyril Brulebois wrote: Signed-off-by: Cyril Brulebois k...@debian.org --- src/fbdev.c |2 ++ 1 files changed, 2 insertions(+), 0 deletions(-) Reviewed-by: Julien Cristau jcris...@debian.org Cheers, Julien signature.asc Description: Digital

Re: [PATCH] Perform XV initialization only if the server supports it.

2010-11-08 Thread Julien Cristau
{ XF86VideoAdaptorPtr *ptr; @@ -904,6 +905,7 @@ FBDevScreenInit(int scrnIndex, ScreenPtr pScreen, int argc, char **argv) xf86XVScreenInit(pScreen,ptr,n); } } +#endif TRACE_EXIT(FBDevScreenInit); Reviewed-by: Julien Cristau jcris

[PATCH xmessage] man: point to the X manpage for more command line options

2010-11-08 Thread Julien Cristau
It was already listed in 'SEE ALSO', but adding a reference in the 'OPTIONS' section is probably easier to find. Reported-by: Ivan Vilata i Balaguer i...@selidor.net Cc: 505...@bugs.debian.org Signed-off-by: Julien Cristau jcris...@debian.org --- xmessage.man |4 +++- 1 files changed, 3

Re: [PATCH x11perf] Install x11perfcomp auxiliary scripts in $datadir, not $libdir.

2010-11-08 Thread Julien Cristau
. Acked-by: Julien Cristau jcris...@debian.org Cheers, Julien ___ xorg-devel@lists.x.org: X.Org development Archives: http://lists.x.org/archives/xorg-devel Info: http://lists.x.org/mailman/listinfo/xorg-devel

Re: [PATCH fbdev 3/3] Kill unused variables.

2010-11-07 Thread Julien Cristau
On Sun, Nov 7, 2010 at 15:54:51 +0100, Cyril Brulebois wrote: Signed-off-by: Cyril Brulebois k...@debian.org --- src/fbdev.c |1 - 1 files changed, 0 insertions(+), 1 deletions(-) diff --git a/src/fbdev.c b/src/fbdev.c index 4cde790..4c0fc31 100644 --- a/src/fbdev.c +++

Re: [PATCH fbdev 1/3] Convert x{re,c,}alloc and xfree.

2010-11-07 Thread Julien Cristau
On Sun, Nov 7, 2010 at 15:54:49 +0100, Cyril Brulebois wrote: Signed-off-by: Cyril Brulebois k...@debian.org --- src/fbdev.c | 18 +- 1 files changed, 9 insertions(+), 9 deletions(-) Reviewed-by: Julien Cristau jcris...@debian.org Cheers, Julien signature.asc

Re: [PATCH fbdev 2/3] Fix escaping in the manpage.

2010-11-07 Thread Julien Cristau
On Sun, Nov 7, 2010 at 15:54:50 +0100, Cyril Brulebois wrote: Signed-off-by: Cyril Brulebois k...@debian.org --- man/fbdev.man |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/man/fbdev.man b/man/fbdev.man index 9c3d24a..a2ed073 100644 --- a/man/fbdev.man +++

Re: [PATCH] Allow building without XV.

2010-11-07 Thread Julien Cristau
On Sun, Nov 7, 2010 at 15:55:47 +0100, Cyril Brulebois wrote: It is useful when one is trying to build an X system with a limited amount of features, like in the Debian Graphical Installer. This should probably just add #ifdef XV around that block in src/fbdev.c, so it follows the

Re: [PATCH libX11 1/2] config: Fix architecture check for OS/2 to skip nios2 cpu

2010-11-06 Thread Julien Cristau
details for the build system. See: http://www.gnu.org/software/automake/manual/automake.html#Cross_002dCompilation $host_os is already being used to determine the transport and threading options. Signed-off-by: Dan Nicholson dbn.li...@gmail.com For both of those: Reviewed-by: Julien

Re: [PATCH] First round of updates for X11R7.6 release notes

2010-11-04 Thread Julien Cristau
On Thu, Nov 4, 2010 at 00:05:11 -0700, Alan Coopersmith wrote: + varlistentry + termXsdl server/term + listitem + para + The experimental Xsdl server has never been finished or + maintained, and will be removed in future X

Re: [PATCH] xfree86: remove two unused variables from Makefile.am

2010-11-04 Thread Julien Cristau
On Thu, Nov 4, 2010 at 14:10:24 +1000, Peter Hutterer wrote: Signed-off-by: Peter Hutterer peter.hutte...@who-t.net --- hw/xfree86/common/Makefile.am |2 -- 1 files changed, 0 insertions(+), 2 deletions(-) Reviewed-by: Julien Cristau jcris...@debian.org Cheers, Julien

Re: [PATCH v3] dix: adds support for none root window background

2010-11-04 Thread Julien Cristau
On Thu, Nov 4, 2010 at 20:27:17 +0100, Mark Kettenis wrote: Uh, do you really want to make the nv driver show whatever crap is in the framebuffer when X starts up? In an environment where that crap is the splash screen they booted with, I guess they do. Cheers, Julien

Re: [PATCH] XKB-Config.xml: update for current releases

2010-11-02 Thread Julien Cristau
On Mon, Nov 1, 2010 at 19:42:09 -0700, Alan Coopersmith wrote: -Option XkbKeycodes xorg +Option XkbKeycodes xfree86 Option XkbTypesdefault Option XkbSymbols en_US(pc104)+de+swapcaps Option XkbGeometry pc(pc104) These options don't exist anymore in the kbd

Re: [PATCH libXi] man: XGetDeviceProperty(3) has no parameter 'pending'.

2010-11-02 Thread Julien Cristau
On Tue, Nov 2, 2010 at 11:22:35 +1000, Peter Hutterer wrote: Signed-off-by: Peter Hutterer peter.hutte...@who-t.net --- man/XGetDeviceProperty.txt |5 - 1 files changed, 0 insertions(+), 5 deletions(-) Reviewed-by: Julien Cristau jcris...@debian.org Cheers, Julien

Re: [PATCH] XKB-Config.xml: update for current releases

2010-11-02 Thread Julien Cristau
On Tue, Nov 2, 2010 at 10:30:38 -0700, Alan Coopersmith wrote: Julien Cristau wrote: On Mon, Nov 1, 2010 at 19:42:09 -0700, Alan Coopersmith wrote: -Option XkbKeycodes xorg +Option XkbKeycodes xfree86 Option XkbTypesdefault Option XkbSymbols en_US(pc104)+de

Re: [PATCH xsetpointer] Fail configure. RIP xsetpointer.

2010-11-02 Thread Julien Cristau
(xsetpointer -l) is available through xinput list. Signed-off-by: Peter Hutterer peter.hutte...@who-t.net Acked-by: Julien Cristau jcris...@debian.org Cheers, Julien ___ xorg-devel@lists.x.org: X.Org development Archives: http://lists.x.org/archives/xorg

Re: [PATCH v3] XKB-Config.xml: update for current releases

2010-11-02 Thread Julien Cristau
zapping sequence - Delete Direct XKB Configuration section, since that no longer works Signed-off-by: Alan Coopersmith alan.coopersm...@oracle.com Reviewed-by: Julien Cristau jcris...@debian.org Cheers, Julien ___ xorg-devel@lists.x.org: X.Org

Bug#601444: upgrade-reports: 3D games throwing X errors after Lenny-Squeeze upgrade

2010-11-02 Thread Julien Cristau
On Tue, Oct 26, 2010 at 08:55:22 +0100, Lawrence Woodman wrote: Package: upgrade-reports Severity: grave Justification: renders package unusable After upgrading from Lenny to Squeeze I have found that Teeworlds, Nexuiz, Egoboo and Stratagus refuse to start and throw the following error:

Re: RFC: Use libxkbcommon in Xorg

2010-10-31 Thread Julien Cristau
On Fri, Oct 29, 2010 at 18:48:43 +0300, Tiago Vignatti wrote: On Fri, Oct 22, 2010 at 04:26:38AM +0200, ext Kristian Høgsberg wrote: At this point libxkbcommon only depends on xproto (for keycodes and a few other defines) and kbproto. it seems that only here is a lot actually. I mean,

Re: [PATCH] configure.ac: Add -fno-strict-aliasing to CFLAGS

2010-10-31 Thread Julien Cristau
...@apple.com Reviewed-by: Julien Cristau jcris...@debian.org Cheers, Julien ___ xorg-devel@lists.x.org: X.Org development Archives: http://lists.x.org/archives/xorg-devel Info: http://lists.x.org/mailman/listinfo/xorg-devel

Re: [PATCH] Allow uploading a keymap to a single device

2010-10-29 Thread Julien Cristau
On Fri, Oct 29, 2010 at 09:31:35 +0200, Dirk Wallenstein wrote: Signed-off-by: Dirk Wallenstein hals...@t-online.de --- This should do the trick. I tested it with two keyboards. Should this also be done for the INPUT_XKM case? Cheers, Julien ___

<    2   3   4   5   6   7   8   9   10   11   >