From b6246a92e1b8bbbd4a4b2c064ce4321f8e81e15e Mon Sep 17 00:00:00 2001 From: "W. Trevor King" Date: Thu, 12 Aug 2010 12:12:10 -0400 Subject: [PATCH] Add --save-config to bin/hooke and don't silently save config file. With the old implementation, setting --debug once would effectively set it for all future calls :p. There should be an explicit 'save config' command if the user wishes to do so. --- hooke/hooke.py | 13 +++++++++---- 1 file changed, 9 insertions(+), 4 deletions(-) diff --git a/hooke/hooke.py b/hooke/hooke.py index cf6600e..d9def05 100644 --- a/hooke/hooke.py +++ b/hooke/hooke.py @@ -117,8 +117,9 @@ class Hooke (object): def load_ui(self): self.ui = ui.load_ui(self.config) - def close(self): - self.config.write() # Does not preserve original comments + def close(self, save_config=False): + if save_config == True: + self.config.write() # Does not preserve original comments def run_command(self, command, arguments): """Run `command` with `arguments` using @@ -198,6 +199,10 @@ def main(): '--command-no-exit', dest='command_exit', action='store_false', default=True, help="Don't exit after running a script or commands.") + p.add_option( + '--save-config', dest='save_config', + action='store_true', default=False, + help="Automatically save a changed configuration on exit.") p.add_option( '--debug', dest='debug', action='store_true', default=False, help="Enable debug logging.") @@ -226,10 +231,10 @@ def main(): hooke = runner.run_lines(hooke, options.commands) finally: if options.command_exit == True: - hooke.close() + hooke.close(save_config=options.save_config) sys.exit(0) try: hooke = runner.run(hooke) finally: - hooke.close() + hooke.close(save_config=options.save_config) -- 2.26.2