revisor

#238 Traceback when incorrect model is selected (does not exist)
Closed: Fixed Opened by jsteffan.

Closed: Fixed
Reopen Issue

revisor.py --live-optical --model f7-i386 --kickstart /etc/revisor/conf.d/sample-ks.cfg
Invalid model f7-i386, configuration file /etc/revisor/conf.d/revisor-f7-i386.conf not readable
Traceback (most recent call last):
File "./revisor.py", line 214, in
cfg.setup_cfg()
File "/home/jon/Fedora/Revisor/revisor/revisor/cfg.py", line 65, in setup_cfg
self.options_set_from_config(self.cli_options)
File "/home/jon/Fedora/Revisor/revisor/revisor/cfg.py", line 732, in options_set_from_config
self.log.error(_("Invalid model %s, configuration file %s not readable" % (self.revisor_model,config.get(self.revisor_model,"main"))))
File "/home/jon/Fedora/Revisor/revisor/revisor/logger.py", line 63, in error
if self.cfg.revisorUseGUI:
AttributeError: ConfigStore instance has no attribute 'revisorUseGUI'

There may be other problems in it, but a missing initialization seems to be one of them.

--- a/revisor/cfg.py
+++ b/revisor/cfg.py
@@ -497,7 +497,11 @@ EndSection

     # Default, ask user what they want to do
     self.answer_yes = False

  • Defaults may be changed with --cli or --gui

  • self.revisorUseCLI = False
  • self.revisorUseGUI = False
  •  # Media Type defaults for Advanced Configuration
 self.media_installation = False
 self.media_live = False

I'm thinking I fixed this, please confirm

Metadata Update from @jsteffan:
- Issue set to the milestone: 2.0.4 Release
Metadata
None
None
None
None
None
Static archive. pagure.io was sunset in 2026; this is a read-only snapshot.
Red Hat Logo

Fedora is sponsored by Red Hat. Learn more about the relationship between Red Hat and Fedora »

© Red Hat, Inc. and others.