Bug 808919

Summary: firstboot uses an invalid option for xrdb
Product: [Fedora] Fedora Reporter: Bruno Wolff III <bruno>
Component: firstbootAssignee: Martin Gracik <mgracik>
Status: CLOSED ERRATA QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: 17CC: bcl, dmach, hidoufr, mgracik
Target Milestone: ---   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: firstboot-17.2-1.fc17 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2012-05-04 22:55:56 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:

Description Bruno Wolff III 2012-04-01 18:05:01 UTC
Description of problem:
In at least f17 and f18 xrdb uses one dash for options. In firstboot-16.1/firstboot/frontend.py the --merge option is passed to xrdb instead of what probably should be the -merge option.

Version-Release number of selected component (if applicable):


How reproducible:


Steps to Reproduce:
1.
2.
3.
  
Actual results:


Expected results:


Additional info:

Comment 1 Hamidou Dia 2012-04-11 01:58:00 UTC
Hi Bruno, Martin,

Seeing the same in F17RC4 released yesterday.
Though, I haven't seen apparent any side effects resulting from this. 

Apr 11 03:01:28 testf17rc4 firstboot[536]: usage:  xrdb [-options ...]
[filename]
Apr 11 03:01:28 testf17rc4 firstboot[536]: where options include:
Apr 11 03:01:28 testf17rc4 firstboot[536]: -display host:dpy   display to use
Apr 11 03:01:28 testf17rc4 firstboot[536]: -all                do all
resources [default]
Apr 11 03:01:28 testf17rc4 firstboot[536]: -global             do
screen-independent resources
Apr 11 03:01:28 testf17rc4 firstboot[536]: -screen             do
screen-specific resources for one screen
Apr 11 03:01:28 testf17rc4 firstboot[536]: -screens            do
screen-specific resources for all screens
Apr 11 03:01:28 testf17rc4 firstboot[536]: -n                  show
but don't do changes
Apr 11 03:01:28 testf17rc4 firstboot[536]: -cpp filename
preprocessor to use [/usr/bin/mcpp]
Apr 11 03:01:28 testf17rc4 firstboot[536]: -nocpp              do not
use a preprocessor
Apr 11 03:01:28 testf17rc4 firstboot[536]: -query              query resources
Apr 11 03:01:28 testf17rc4 firstboot[536]: -load               load
resources from file [default]
Apr 11 03:01:28 testf17rc4 firstboot[536]: -override           add in
resources from file
Apr 11 03:01:28 testf17rc4 firstboot[536]: -merge              merge
resources from file & sort
Apr 11 03:01:28 testf17rc4 firstboot[536]: -edit filename      edit
resources into file
Apr 11 03:01:28 testf17rc4 firstboot[536]: -backup string      backup
suffix for -edit [.bak]
Apr 11 03:01:28 testf17rc4 firstboot[536]: -symbols            show
preprocessor symbols
Apr 11 03:01:28 testf17rc4 firstboot[536]: -remove             remove resources
Apr 11 03:01:28 testf17rc4 firstboot[536]: -retain             avoid
server reset (avoid using this)
Apr 11 03:01:28 testf17rc4 firstboot[536]: -quiet              don't
warn about duplicates
Apr 11 03:01:28 testf17rc4 firstboot[536]: -Dname[=value], -Uname,
-Idirectory    passed to preprocessor
Apr 11 03:01:28 testf17rc4 firstboot[536]: A - or no input filename
represents stdin.

Regards


--
Fedora Bugzappers volunteer triage team
https://fedoraproject.org/wiki/BugZappers

Comment 2 Fedora Update System 2012-04-11 14:23:53 UTC
firstboot-17.1-1.fc17 has been submitted as an update for Fedora 17.
https://admin.fedoraproject.org/updates/firstboot-17.1-1.fc17

Comment 3 Fedora Update System 2012-04-12 01:08:10 UTC
Package firstboot-17.1-1.fc17:
* should fix your issue,
* was pushed to the Fedora 17 testing repository,
* should be available at your local mirror within two days.
Update it with:
# su -c 'yum update --enablerepo=updates-testing firstboot-17.1-1.fc17'
as soon as you are able to.
Please go to the following url:
https://admin.fedoraproject.org/updates/FEDORA-2012-5692/firstboot-17.1-1.fc17
then log in and leave karma (feedback).

Comment 4 Fedora Update System 2012-04-20 07:31:35 UTC
firstboot-17.2-1.fc17 has been submitted as an update for Fedora 17.
https://admin.fedoraproject.org/updates/firstboot-17.2-1.fc17

Comment 5 Fedora Update System 2012-05-04 22:55:56 UTC
firstboot-17.2-1.fc17 has been pushed to the Fedora 17 stable repository.  If problems still persist, please make note of it in this bug report.