Bug 159140 - gdm-early-login fails if firstboot rpm isn't installed
gdm-early-login fails if firstboot rpm isn't installed
Status: CLOSED WONTFIX
Product: Fedora
Classification: Fedora
Component: gdm (Show other bugs)
rawhide
All Linux
medium Severity medium
: ---
: ---
Assigned To: Ray Strode [halfline]
Mike McLean
:
Depends On:
Blocks: FC4Update
  Show dependency treegraph
 
Reported: 2005-05-30 11:54 EDT by Valdis Kletnieks
Modified: 2007-11-30 17:11 EST (History)
1 user (show)

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Environment:
Last Closed: 2007-03-06 14:27:26 EST
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---


Attachments (Terms of Use)
works-for-me patch against gdm startup scripts (973 bytes, patch)
2005-05-30 11:54 EDT, Valdis Kletnieks
no flags Details | Diff
Updated patch... (1.42 KB, patch)
2005-06-01 12:43 EDT, Valdis Kletnieks
no flags Details | Diff
Argh. Try again... (1.42 KB, patch)
2005-06-01 13:01 EDT, Valdis Kletnieks
no flags Details | Diff

  None (edit)
Description Valdis Kletnieks 2005-05-30 11:54:38 EDT
Description of problem:
A bad test for /etc/sysconfig/firstboot causes gdm-early-login to exit
incorrectly.  If the file doesn't exist, it falls out of the end of the
do_firstboot function with a default 'return 0' instead of doing a return 1.

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

How reproducible:
Remove the firstboot rpm and reboot with gdm-early-login configured

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


Expected results:


Additional info:
Patch against /etc/init.d scripts attached.
Comment 1 Valdis Kletnieks 2005-05-30 11:54:38 EDT
Created attachment 114974 [details]
works-for-me patch against gdm startup scripts
Comment 2 Valdis Kletnieks 2005-06-01 12:43:28 EDT
Created attachment 115040 [details]
Updated patch...

Damn. I missed the zzz-bootup-complete script, which needs the same fix as the
other two.
Comment 3 Valdis Kletnieks 2005-06-01 13:01:47 EDT
Created attachment 115042 [details]
Argh. Try again...

It's definitely a Murphy day. Missed a '!' in there - not sure why I didn't see
a problme when I tested it.
Comment 4 Ray Strode [halfline] 2007-03-06 14:27:26 EST
we aren't supporting early boot anymore, so i'm going to close this out. Thanks
for the patch, though.

Note You need to log in before you can comment on or make changes to this bug.