Bug 48311

Summary: cvt manpage mistakes
Product: xorg Reporter: Bob Bib <bobbib>
Component: DocumentationAssignee: Alan Coopersmith <alan.coopersmith>
Status: RESOLVED FIXED QA Contact: Xorg Project Team <xorg-team>
Severity: normal    
Priority: medium    
Version: unspecified   
Hardware: All   
OS: All   
URL: https://bugs.launchpad.net/bugs/963375
Whiteboard: 2012BRB_Reviewed
i915 platform: i915 features:

Description Bob Bib 2012-04-04 16:24:36 UTC
"kHz" should be replaced with "Hz" in the following string from cvt manpage:

"refresh Provide a vertical refresh rate in kHz. The CVT standard prefers either 50.0, 60.0, 75.0 or 85.0kHz. The default is 60.0kHz."
Comment 1 Alan Coopersmith 2012-04-07 09:01:59 UTC
Thanks for the report - patch submitted for review:
http://patchwork.freedesktop.org/patch/9769/
Comment 2 Bob Bib 2012-04-10 15:33:39 UTC
Seems it has got an accidental duplicate:
http://patchwork.freedesktop.org/patch/9825/
Comment 3 Alan Coopersmith 2012-04-10 15:35:57 UTC
patchwork didn't associate that the first was sent for review, and after
getting that review, the second was submitted to the release manager to
be applied to the git master repo.
Comment 4 Alan Coopersmith 2012-06-12 07:31:02 UTC
Fix is in git master now:
http://cgit.freedesktop.org/xorg/xserver/commit/?id=c3180a74a4fc6304bcfe676e3285d707874a9fea

and has been cherry-picked into the 1.12 branch for the 1.12.3 release as well:
http://cgit.freedesktop.org/xorg/xserver/commit/?h=server-1.12-branch&id=3484ef2de50970fefe0c2580f52357ee37d3e6f3

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.