rpms/kexec-tools
7
0
Fork 0
Code Issues Pull Requests Releases Activity

Take closing the reboot dialog as no

Browse Source 
Resolves: bz805782

Copying something from old bug 688150:
Steps to Reproduce:
1.Go through FirstBoot process while get to Kdump
2.Change Kdump memory
3.Press Finish
4.A window pops-up (Changing Kdump setting requires ...)
5.Do not press "No" or "Yes" but pres the "X"  in the right up corner.

Actual results:
The effect of this action is the same as pressing "YES"

Expected results:
The effect of this action is the same as pressing "NO

So this patch fix to only reboot when rc == gtk.RESPONSE_YES.

Port from below patch from rhel:
commit ed0c89d8e42b9205671cb6c81f9f73c275bee72f
Author: amwang <amwang@redhat.com>
Date:   Thu Mar 17 10:07:52 2011 +0000

    Resolves: bug 688150.

Signed-off-by: Dave Young <dyoung@redhat.com>
Acked-by: Vivek Goyal <vgoyal@redhat.com>
This commit is contained in:
Dave Young 2012-08-09 10:22:50 +08:00
parent dfb10612a2
commit 2b11d78e2e
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
firstboot_kdump.py
View File

@ -375,7 +375,7 @@ class moduleClass(Module):
rc = dlg.run()
dlg.destroy()
if rc == gtk.RESPONSE_NO:
if rc != gtk.RESPONSE_YES:
self.reboot = False
return RESULT_SUCCESS
else: