Bug 78190

Summary: [PATCH] [TRIVIAL] misparses command line badly.
Product: [Retired] Red Hat Linux Reporter: Crutcher Dunnavant <crutcher+bugzilla>
Component: redhat-config-xfree86Assignee: Alexander Larsson <alexl>
Status: CLOSED RAWHIDE QA Contact:
Severity: medium Docs Contact:
Priority: medium    
Version: 8.0   
Target Milestone: ---   
Target Release: ---   
Hardware: i386   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2002-11-19 21:59:47 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---

Description Crutcher Dunnavant 2002-11-19 21:59:39 UTC
Description of Problem:
There is a brown-paper-bag bug in /usr/share/redhat-config-xfree86/xconf.py,
which completely fubars its parsing of --set-card and --set-resolution.

A comma is missing.

Version-Release number of selected component (if applicable):
redhat-config-xfree86-0.6.7-1

--- xconf.py.oops       2002-11-19 16:01:24.000000000 -0600
+++ xconf.py    2002-11-19 16:01:35.000000000 -0600
@@ -1534,7 +1534,7 @@
                                ["help", "reconfig", "noui", "verbose",
                                 "forceui", "output=",
                                 "set-hsync=", "set-vsync=",
-                                "set-videoram=", "set-driver=", "set-card="
+                                "set-videoram=", "set-driver=", "set-card=",
                                 "set-resolution=", "set-depth="])
 except (getopt.GetoptError), e:
     print e

Comment 1 Alexander Larsson 2002-11-20 10:56:11 UTC
Yes. That was a bit embarrasing. But it is fixed in CVS, and i believe the fix
has now propagated to rawhide.