Bug 21104 - DOCS: xorg.conf(5) error
Summary: DOCS: xorg.conf(5) error
Status: RESOLVED INVALID
Alias: None
Product: xorg
Classification: Unclassified
Component: Documentation (show other bugs)
Version: unspecified
Hardware: x86 (IA32) OpenBSD
: medium normal
Assignee: Xorg Project Team
QA Contact: Xorg Project Team
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2009-04-10 00:52 UTC by J.C. Roberts
Modified: 2009-04-10 08:34 UTC (History)
0 users

See Also:
i915 platform:
i915 features:


Attachments
fix (620 bytes, patch)
2009-04-10 00:52 UTC, J.C. Roberts
no flags Details | Splinter Review

Description J.C. Roberts 2009-04-10 00:52:47 UTC
Created attachment 24690 [details] [review]
fix

There is no "-s" flag/switch for xset.
The correct flag/switch is simply "s" without the leading dash.

This error is in both the xserver 1.5 branch and 1.6 branch.

I don't know if this web-form will screw up the patch or not, but I've also attached it as a file.

Index: xserver/hw/xfree86/doc/man/xorg.conf.man.pre
===================================================================
RCS file: /cvs/xenocara/xserver/hw/xfree86/doc/man/xorg.conf.man.pre,v
retrieving revision 1.5
diff -u -r1.5 xorg.conf.man.pre
--- xserver/hw/xfree86/doc/man/xorg.conf.man.pre	12 Jan 2009 20:17:47 -0000	1.5
+++ xserver/hw/xfree86/doc/man/xorg.conf.man.pre	10 Apr 2009 04:58:53 -0000
@@ -550,7 +550,7 @@
 .I time
 is in minutes.
 This is equivalent to the __xservername__ server's
-.B \-s
+.B \s
 flag, and the value can be changed at run\-time with
 .BR xset(__appmansuffix__).
 Default: 10 minutes.
Comment 1 Stefan Dirsch 2009-04-10 08:34:54 UTC
Hi

I can only agree on that ...

Date: Fri, 10 Apr 2009 14:51:30 +0100
From: Julien Cristau <...>
To: "J.C. Roberts" <...>
Cc: xorg-devel@lists.x.org
Subject: Re: PATCH: xorg.conf(4) man page

On Fri, 2009-04-10 at 04:20 -0700, J.C. Roberts wrote:
> It's a trivial one byte patch and was submitted to Bugzilla, but it
> would be nice to know the "right" way to do things for the future.
>
> https://bugs.freedesktop.org/show_bug.cgi?id=21104
>
well, that patch is wrong, afaict... it's talking about the *server*'s
-s flag, not xset's. (plus, even then, you need to remove the \ too, not
just the hyphen)

Cheers,
Julien


Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.