[initial-setup][PATCH] Inform user that the system may be rebooted (#1022040)

Vratislav Podzimek vpodzime at redhat.com
Fri Nov 1 06:31:48 UTC 2013


On Thu, 2013-10-31 at 17:35 -0700, Brian C. Lane wrote:
> On Thu, Oct 31, 2013 at 07:47:31AM +0100, Vratislav Podzimek wrote:
> > If the License agreement is not accepted and user quits the Initial Setup, the
> > system should be rebooted. We should inform user about that in the quit dialog.
> > 
> > Signed-off-by: Vratislav Podzimek <vpodzime at redhat.com>
> > ---
> >  initial_setup/gui/gui.py | 3 ++-
> >  initial_setup/tui/tui.py | 3 ++-
> >  2 files changed, 4 insertions(+), 2 deletions(-)
> > 
> > diff --git a/initial_setup/gui/gui.py b/initial_setup/gui/gui.py
> > index 3b525a5..a627827 100644
> > --- a/initial_setup/gui/gui.py
> > +++ b/initial_setup/gui/gui.py
> > @@ -16,7 +16,8 @@ N_ = lambda x: x
> >  
> >  class InitialSetupQuitDialog(QuitDialog):
> >      MESSAGE = N_("Are you sure you want to quit the configuration process?\n"
> > -                 "You might end up with unusable system if you do.")
> > +                 "You might end up with unusable system if you do. Unless the"
> 
> ... end up with an unusable system ...
> 
> also, a space is needed at the end of the line.
> 
> > +                 "Lincense agreement is accepted, the system will be rebooted.")
> 
> License
> 
> 
> >  
> >  @inject(Gdk, product_title = product_title, is_final = is_final)
> >  class InitialSetupGraphicalUserInterface(GraphicalUserInterface):
> > diff --git a/initial_setup/tui/tui.py b/initial_setup/tui/tui.py
> > index de00095..68a96dd 100644
> > --- a/initial_setup/tui/tui.py
> > +++ b/initial_setup/tui/tui.py
> > @@ -14,7 +14,8 @@ _ = lambda x: gettext.ldgettext("initial-setup", x)
> >  N_ = lambda x: x
> >  
> >  QUIT_MESSAGE = N_("Are you sure you want to quit the configuration process?\n"
> > -                  "You might end up with unusable system if you do.")
> > +                  "You might end up with unusable system if you do. Unless the"
> > +                  "Lincense agreement is accepted, the system will be rebooted.")
> 
> Same comments here.
Good catches, thanks! Fixing locally.

-- 
Vratislav Podzimek

Anaconda Rider | Red Hat, Inc. | Brno - Czech Republic



More information about the anaconda-patches mailing list