Re: [firstboot] Update to work with new python-meh with report support (#562659)

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



> diff --git a/progs/firstboot b/progs/firstboot
> index 0f8b804..6f14794 100755
> --- a/progs/firstboot
> +++ b/progs/firstboot
> @@ -18,8 +18,11 @@
>  # subject to the GNU General Public License and may only be used or replicated
>  # with the express permission of Red Hat, Inc. 
>  #
> -import meh
> +
>  import os, string, sys
> +from meh.dump import *
> +from meh.ui.gui import GraphicalIntf
> +from meh.handler import *
>  
>  # Since X may be starting up immediately after firstboot crashes, we might
>  # not have any way to see the traceback.  This lets it be logged somewhere
> @@ -175,7 +178,10 @@ if __name__ == "__main__":
>      # This must come as early as possible so we can present the UI for the
>      # widest class of problems, BUT it also has to come late enough for us
>      # to have already imported gtk.
> -    meh.makeRHHandler("firstboot", "@VERSION@", config)
> +    handler_conf = Config(programName="firstboot",
> +                          programVersion="@VERSION@")
> +    handler = ExceptionHandler(handler_conf, GraphicalIntf(), ExceptionDump)
> +    handler.install(config)
>  
>      config.moduleList = loadModules(config.moduleDir, config.mode)
>      if not config.moduleList:

Looks fine to me, as long as you've tested it.  This should go to RHEL6
and master, but not to F13.  The python-meh in F13 does not use report
as it was determined to be too destabilizing of a change.

- Chris

_______________________________________________
Anaconda-devel-list mailing list
Anaconda-devel-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/anaconda-devel-list

[Index of Archives]     [Kickstart]     [Fedora Users]     [Fedora Legacy List]     [Fedora Maintainers]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [Yosemite Photos]     [KDE Users]     [Fedora Tools]
  Powered by Linux