[PATCH:xpr] Remove unneeded uses of & -lm

Can't see why they were ever needed. Signed-off-by: Alan Coopersmith --- Makefile.am |2 +- x2pmp.c |1 - 2 files changed, 1 insertion(+), 2 deletions(-) diff --git a/Makefile.am b/Makefile.am index c96f863..8b75b07 100644 --- a/Makefile.am +++ b/Makefile.am @@ -22,7 +22,7 @@ bin_P

typo on dev doc

http://xorg.freedesktop.org/wiki/Development/Documentation/PointerAcceleration/ The AccelerationProfile [integer] subsection, "-1 none" for common courtesy isn't it supposed to be '0 none' ? :) (xinput --set-prop command doesn't warrant -1, but does so starting from 0) Not a high priority, an

Re: [PATCH 1/2] glx: Fix incorrect use of dri_interface.h version defines in extensions.

On 11/14/2013 05:40 PM, Eric Anholt wrote: > Those defines are so you can compile-time check "do I have a > dri_interface.h that defines this new field of the struct?" You don't > want the server to claim it implements the new struct just because you > installed a new copy of Mesa. Series is Rev

Re: [PATCH] xfree86: Prefer fbdev to vesa

On Sat, Nov 16, 2013 at 6:15 AM, Adam Jackson wrote: > On UEFI machines you'd prefer fbdev to grab efifb instead of vesa trying > to initialize and failing in a way we can't unwind from. On BIOS > machines this is harmless: either there is an fbdev driver and it'll > probably be more capable, or

[PATCH:mkfontscale 17/17] Print which option was in error along with usage message

Signed-off-by: Alan Coopersmith --- mkfontscale.c | 25 ++--- 1 file changed, 18 insertions(+), 7 deletions(-) diff --git a/mkfontscale.c b/mkfontscale.c index d27bb8d..a494f93 100644 --- a/mkfontscale.c +++ b/mkfontscale.c @@ -113,11 +113,12 @@ static ListPtr encodingsToDo

[PATCH:iceauth 12/17] Add -V flag to print version

Signed-off-by: Alan Coopersmith --- iceauth.c |4 man/iceauth.man |2 ++ 2 files changed, 6 insertions(+) diff --git a/iceauth.c b/iceauth.c index e729ed5..c1688c0 100644 --- a/iceauth.c +++ b/iceauth.c @@ -63,6 +63,7 @@ usage (void) "-q turn o

Re: [PATCH:libX11] Don't need to link libX11-xcb against libX11

On 11/15/13 06:18 PM, Thomas Klausner wrote: On Fri, Nov 15, 2013 at 06:03:25PM -0800, Alan Coopersmith wrote: libX11-xcb only accesses data structures defined in X11 headers, it doesn't call any functions or reference any global variables in libX11 itself. (Seems to have been left from previou

[PATCH:mkfontscale 15/17] Add -v flag to print program version

Signed-off-by: Alan Coopersmith --- man/mkfontscale.man |5 + mkfontscale.c |5 - 2 files changed, 9 insertions(+), 1 deletion(-) diff --git a/man/mkfontscale.man b/man/mkfontscale.man index 4aa88ab..cb18458 100644 --- a/man/mkfontscale.man +++ b/man/mkfontscale.man @@ -38,

[PATCH:fstobdf 06/17] Spell out -server in usage message to match man page

Code matches -s*, so either form works, but better to have them agree. Signed-off-by: Alan Coopersmith --- fstobdf.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/fstobdf.c b/fstobdf.c index 8d029a5..db168c9 100644 --- a/fstobdf.c +++ b/fstobdf.c @@ -52,7 +52,7 @@ in thi

[PATCH:fslsfonts 04/17] Add -version option to print version

Signed-off-by: Alan Coopersmith --- fslsfonts.c |9 + man/fslsfonts.man |3 +++ 2 files changed, 12 insertions(+) diff --git a/fslsfonts.c b/fslsfonts.c index 6ff6a2d..9de741e 100644 --- a/fslsfonts.c +++ b/fslsfonts.c @@ -43,6 +43,10 @@ in this Software without prior writt

[PATCH:iceauth 13/17] Print which option was in error along with usage message

Special case -u, since it's documented as printing the usage message (sort of like an ancient --help), so shouldn't be called unrecognized. Signed-off-by: Alan Coopersmith --- iceauth.c | 16 1 file changed, 12 insertions(+), 4 deletions(-) diff --git a/iceauth.c b/iceauth.c

[PATCH:mkfontscale 16/17] Make usage() always exit(1)

Avoid duplicating exit(1) calls after every single call to usage() Signed-off-by: Alan Coopersmith --- configure.ac |2 +- mkfontscale.c | 12 +++- 2 files changed, 4 insertions(+), 10 deletions(-) diff --git a/configure.ac b/configure.ac index 971ae5c..cc13259 100644 --- a/confi

[PATCH:bdftopcf 02/17] Print which option was in error along with usage message

Signed-off-by: Alan Coopersmith --- bdftopcf.c |2 ++ 1 file changed, 2 insertions(+) diff --git a/bdftopcf.c b/bdftopcf.c index 1b24aa1..2208fee 100644 --- a/bdftopcf.c +++ b/bdftopcf.c @@ -145,6 +145,8 @@ main(int argc, char *argv[]) else { if (input_name) { usage:

[PATCH:iceauth 10/17] Actually print command list in usage message

The xauth routine this was based on took three arguments: a file pointer, a command name, and a prefix. If a command name is given, prints help on that command, else if it's null, prints help on all commands. The iceauth version has only two arguments: file pointer & command name. It was being ca

[PATCH 00/17] CLI option handling updates for various apps

One of the tasks I've been doing at work is looking at what it will take to bring our software into compliance with the latest accessibility standards, specifically http://www.w3.org/TR/wcag2ict/ which seems to be one of the standards which many countries will be adopting in their acessibility law

[PATCH:bitmap 03/17] Print which option was in error along with usage message

Signed-off-by: Alan Coopersmith --- atobm.c | 26 +++--- bmtoa.c |8 +++- configure.ac |2 +- 3 files changed, 27 insertions(+), 9 deletions(-) diff --git a/atobm.c b/atobm.c index 040fecb..ef6da7e 100644 --- a/atobm.c +++ b/atobm.c @@ -42,9 +42,11 @@

[PATCH:fslsfonts 05/17] Print which option was in error along with usage message

Signed-off-by: Alan Coopersmith --- configure.ac |2 +- fslsfonts.c | 35 --- 2 files changed, 25 insertions(+), 12 deletions(-) diff --git a/configure.ac b/configure.ac index e828085..58a4607 100644 --- a/configure.ac +++ b/configure.ac @@ -36,7 +36,7 @@ X

[PATCH:mkfontscale 14/17] Add missing newline to usage output

Signed-off-by: Alan Coopersmith --- mkfontscale.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/mkfontscale.c b/mkfontscale.c index a67f283..1194145 100644 --- a/mkfontscale.c +++ b/mkfontscale.c @@ -118,7 +118,7 @@ usage(void) { fprintf(stderr, "mkfont

[PATCH:fstobdf 07/17] Print which option was in error along with usage message

Signed-off-by: Alan Coopersmith --- configure.ac |2 +- fstobdf.c| 19 ++- 2 files changed, 11 insertions(+), 10 deletions(-) diff --git a/configure.ac b/configure.ac index 1dbf55a..0ef434b 100644 --- a/configure.ac +++ b/configure.ac @@ -36,7 +36,7 @@ XORG_DEFAULT_OPTI

[PATCH:iceauth 11/17] Fix "noexistent" typo in error message

Signed-off-by: Alan Coopersmith --- process.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/process.c b/process.c index 97bcfa9..09cb7ef 100644 --- a/process.c +++ b/process.c @@ -1060,7 +1060,7 @@ static int do_help ( if (n == 0) { prefix (inputfilename, lin

[PATCH:appres 01/17] Print which option was in error along with usage message

Signed-off-by: Alan Coopersmith --- appres.c |5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/appres.c b/appres.c index 2f43d2d..fad870f 100644 --- a/appres.c +++ b/appres.c @@ -156,8 +156,11 @@ main (int argc, char *argv[]) printf("%s\n", PACKAGE_STRING);

[PATCH:fstobdf 09/17] Print error and usage on invalid arg, instead of ignoring it

Signed-off-by: Alan Coopersmith --- fstobdf.c |5 + 1 file changed, 5 insertions(+) diff --git a/fstobdf.c b/fstobdf.c index 5fe4816..255d82e 100644 --- a/fstobdf.c +++ b/fstobdf.c @@ -109,6 +109,11 @@ main(int argc, char *argv[]) printf("%s\n", PACKAGE_STRING); e

[PATCH:fstobdf 08/17] Add -version option to print version

Signed-off-by: Alan Coopersmith --- fstobdf.c | 14 -- man/fstobdf.man |6 ++ 2 files changed, 18 insertions(+), 2 deletions(-) diff --git a/fstobdf.c b/fstobdf.c index bcdffbc..5fe4816 100644 --- a/fstobdf.c +++ b/fstobdf.c @@ -43,6 +43,10 @@ in this Software without

Re: [PATCH:libX11] Don't need to link libX11-xcb against libX11

On Fri, Nov 15, 2013 at 06:03:25PM -0800, Alan Coopersmith wrote: > libX11-xcb only accesses data structures defined in X11 headers, > it doesn't call any functions or reference any global variables > in libX11 itself. (Seems to have been left from previous XCL > implementation.) You should bump

[PATCH:libX11] Don't need to link libX11-xcb against libX11

libX11-xcb only accesses data structures defined in X11 headers, it doesn't call any functions or reference any global variables in libX11 itself. (Seems to have been left from previous XCL implementation.) Signed-off-by: Alan Coopersmith --- src/Makefile.am |1 - 1 file changed, 1 deletion

[PATCH 3/3] configure.ac: Require libpciaccess for int10

A --disable-pciaccess build will fail with an int10 module other than stub. Signed-off-by: Connor Behan --- configure.ac | 3 +++ 1 file changed, 3 insertions(+) diff --git a/configure.ac b/configure.ac index 5e621e0..a843770 100644 --- a/configure.ac +++ b/configure.ac @@ -1822,6 +1822,9 @@ if

[PATCH 2/3] configure.ac: Require libpciaccess for platform bus support

There is currently no reason to build with --enable-config-udev-kms and --disable-pciaccess but anyone who tries this should know that the build will fail. Signed-off-by: Connor Behan --- configure.ac | 4 1 file changed, 4 insertions(+) diff --git a/configure.ac b/configure.ac index e8597

[PATCH 1/3] configure.ac: Add whitespace near PCI configuration

Signed-off-by: Connor Behan --- configure.ac | 35 +-- 1 file changed, 17 insertions(+), 18 deletions(-) diff --git a/configure.ac b/configure.ac index 1e6f813..e8597a7 100644 --- a/configure.ac +++ b/configure.ac @@ -1801,24 +1801,23 @@ if test "x$XORG" = xyes; t

Re: [PATCH] xfree86: Prefer fbdev to vesa

Adam Jackson writes: > On UEFI machines you'd prefer fbdev to grab efifb instead of vesa trying > to initialize and failing in a way we can't unwind from. On BIOS > machines this is harmless: either there is an fbdev driver and it'll > probably be more capable, or there's not and vesa will kick

Re: [PATCH] xfree86: Prefer fbdev to vesa

On Fri, Nov 15, 2013 at 3:15 PM, Adam Jackson wrote: > On UEFI machines you'd prefer fbdev to grab efifb instead of vesa trying > to initialize and failing in a way we can't unwind from. On BIOS > machines this is harmless: either there is an fbdev driver and it'll > probably be more capable, or

[PATCH] xfree86: Prefer fbdev to vesa

On UEFI machines you'd prefer fbdev to grab efifb instead of vesa trying to initialize and failing in a way we can't unwind from. On BIOS machines this is harmless: either there is an fbdev driver and it'll probably be more capable, or there's not and vesa will kick in anyway. Signed-off-by: Adam

Re: modular: Changes to 'master'

On 13-11-15 02:32 PM, Jon TURNEY wrote: > On 15/11/2013 19:27, Rob Clark wrote: >> Well, to be fair, a number of the DDX drivers don't build on ARM. I >> remove them locally in my in my xorg.modules. >> >> I don't claim to be a jhbuild expert, but it is probably easier to >> have everything in xor

Re: modular: Changes to 'master'

On 15/11/2013 19:27, Rob Clark wrote: > Well, to be fair, a number of the DDX drivers don't build on ARM. I > remove them locally in my in my xorg.modules. > > I don't claim to be a jhbuild expert, but it is probably easier to > have everything in xorg.modules, and let people remove the depends f

Re: modular: Changes to 'master'

Well, to be fair, a number of the DDX drivers don't build on ARM. I remove them locally in my in my xorg.modules. I don't claim to be a jhbuild expert, but it is probably easier to have everything in xorg.modules, and let people remove the depends for what they don't want to build. It seems easi

Re: [PATCH] hw/xfree86: Make strings in DriverRec and ScrnInfoRec const

Keith Packard writes: > This avoids compiler warnings when initializing with string constants. > > Signed-off-by: Keith Packard I've apparently misplaced my patch that did the same thing. Reviewed-by: Eric Anholt pgpf7MVcX5gyJ.pgp Description: PGP signature _

Self introduction Hans de Goede

Hi All, My name is Hans de Goede, and I'm active as a FOSS contributor / developer since 1997. Recently I've mainly been working on hwmon kernel drivers, usb webcam kernel drivers and userspace support libs, usb emulation and redirection in qemu, libusb and usbdevfs. I've been working for Re

Re: Fwd: [oss-security] "I miss LSD", slides, paper and tools relating to finding UNIX system level vulnerabilities (as given at 44CON)

>>> My understanding is that the new code, by passing shared memory >>> through fd is a lot better since [...] >> In those respects, yes. But it's worse in that it requires write >> access to a filesystem - a filesystem which supports mmap - with >> space enough to hold the shared memory segments,

[PATCH v3] OS/ACPI: Make socket to acpid is non-blocking

If a connect to the acpid socket is delayed connect will hang stalling the Xserver. To avoid this this create a non-blocking socket. If connect returns EINPROGRESS use select() and getsockopt() to check for completion of the connect request. Use a different timeout to select() wether we connect fo

Re: Fwd: [oss-security] "I miss LSD", slides, paper and tools relating to finding UNIX system level vulnerabilities (as given at 44CON)

* Alan Coopersmith (alan.coopersm...@oracle.com) wrote: > As we expand our use of shared memory via the new extensions, we should try > to make sure we're not making more problems along the lines of those mentioned > in the presentation linked below & associated whitepaper published at: > http://la

Re: [PATCH v2] OS/ACPI: Make socket to acpid is non-blocking

On Fri, Nov 15, 2013 at 11:25:49AM +0100, Egbert Eich wrote: > On Thu, Nov 14, 2013 at 01:20:44PM +0900, Keith Packard wrote: > > Egbert Eich writes: > > > > > static CARD32 > > > lnxACPICheckTimer(OsTimerPtr timer, CARD32 now, pointer arg) > > > { > > > +struct timeval timeval = { 0, 0 };

[PATCH libXrender] Fix request length calculation for XRenderCompositeText32

Hi, Request length calculation inside XRenderCompositeText32 is broken for the case where the number of glyphs fits exactky inside the last xGlyphElt. In XRenderCompositeText8 and XRenderCompositeText16 this case is handled properly, somehow the "-1" got missing in XRenderCompositeText32. src/Gl

Re: Fwd: [oss-security] "I miss LSD", slides, paper and tools relating to finding UNIX system level vulnerabilities (as given at 44CON)

> Date: Fri, 15 Nov 2013 02:16:19 -0500 (EST) > From: Mouse > > > My understanding is that the new code, by passing shared memory > > through fd is a lot better since [...] > > In those respects, yes. But it's worse in that it requires write > access to a filesystem - a filesystem which support

Re: [PATCH v2] OS/ACPI: Make socket to acpid is non-blocking

On Thu, Nov 14, 2013 at 01:20:44PM +0900, Keith Packard wrote: > Egbert Eich writes: > > > static CARD32 > > lnxACPICheckTimer(OsTimerPtr timer, CARD32 now, pointer arg) > > { > > +struct timeval timeval = { 0, 0 }; > > DebugF("ACPI: trying to reopen\n"); > > -if (doLnxACPIOpen())