From b9acaccc1d59a4efb70903fc3aa3e00a6c62749b Mon Sep 17 00:00:00 2001 From: Michael Scherer Date: Sat, 13 Sep 2014 12:27:47 +0200 Subject: [PATCH] Do not remove local role file if given on the commandline using this (for testing purpose) : $ ansible-galaxy install COPYING - error: the file downloaded was not a tar.gz - COPYING was NOT installed successfully. - you can use --ignore-errors to skip failed roles. this result in COPYING being erased, which is surprising for the user. This also prevent erasing requirements.yml if someone use the wrong flag. --- bin/ansible-galaxy | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/bin/ansible-galaxy b/bin/ansible-galaxy index 146361da93f..81be7b111af 100755 --- a/bin/ansible-galaxy +++ b/bin/ansible-galaxy @@ -787,7 +787,8 @@ def execute_install(args, options, parser): if tmp_file: installed = install_role(role.get("name"), role.get("version"), tmp_file, options) # we're done with the temp file, clean it up - os.unlink(tmp_file) + if tmp_file != role_src: + os.unlink(tmp_file) # install dependencies, if we want them if not no_deps and installed: if not role_data: