#212 New logging traceback.
Closed: Fixed Opened by jsteffan.

[jon@damaestro revisor]$ sudo ./revisor.py --cli --model=f7-i386 --kickstart=/etc/revisor/conf.d/sample-ks.cfg --install-tree
Running Revisor in CLI mode as specified by command line options
Running...
The directories Revisor uses in /var/tmp/ already exist. This could possibly hold data from a previous run. Please remove or move them to a safe location. Then confirm to continue. If you do not move or remove the files, Revisor will simply delete them
Do you want to continue? [Y/n] n
Traceback (most recent call last):
File "./revisor.py", line 201, in
revisorBase.run()
File "/home/jon/Fedora/Revisor/revisor/revisor/cli.py", line 37, in run
if not self.cfg.check_working_directory():
File "/home/jon/Fedora/Revisor/revisor/revisor/cfg.py", line 256, in check_working_directory
self.log.warning(("The directories Revisor uses in %s already exist. This could possibly hold data from a previous run. Please remove or move them to a safe location. Then confirm to continue. If you do not move or remove the files, Revisor will simply delete them" % self.working_directory))
File "/home/jon/Fedora/Revisor/revisor/revisor/logger.py", line 83, in warning
self.warning_prompt(msg)
File "/home/jon/Fedora/Revisor/revisor/revisor/logger.py", line 128, in warning_prompt
self.log.error(
("Abort! Abort! Abort!"))
AttributeError: Logger instance has no attribute 'log'


So, now when you say 'n'o ... the sys.exit(1) doesn't really exit.

So, now when you say 'n'o ... the sys.exit(1) doesn't really exit.

A log call now prints 3 times:

WARNING:revisor.file:No groups present!
No groups present!
WARNING:revisor.console:No groups present!

A log call now prints 3 times:

WARNING:revisor.file:No groups present!
No groups present!
WARNING:revisor.console:No groups present!

Lol, well we can't be verbose enough about warnings, right? I have self.error on that line. I've committed some other improvements. If it doesn't work for you, please reopen ticket

Lol, well we can't be verbose enough about warnings, right? I have self.error on that line. I've committed some other improvements. If it doesn't work for you, please reopen ticket

Metadata Update from @jsteffan:
- Issue set to the milestone: 2.0.4 Release

Metadata