[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.
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
Metadata Update from @jsteffan: - Issue set to the milestone: 2.0.4 Release