curtin-dev team mailing list archive

[Dan Watkins <daniel.watkins@xxxxxxxxxxxxx>]

Dan Watkins has proposed merging ~oddbloke/curtin/+git/curtin:vmtest-remove-release into curtin:master.

Commit message:
tools: rename remove-vmtest-release to vmtest-remove-release
    
This prefix matches other vmtest scripts we have in tools/, making it
easier to discover.

(Also make it executable.)

Requested reviews:
  curtin developers (curtin-dev)

For more details, see:
https://code.launchpad.net/~oddbloke/curtin/+git/curtin/+merge/387527
-- 
Your team curtin developers is requested to review the proposed merge of ~oddbloke/curtin/+git/curtin:vmtest-remove-release into curtin:master.

diff --git a/tools/run-pep8 b/tools/run-pep8
index c27a96c..227129c 100755
--- a/tools/run-pep8
+++ b/tools/run-pep8
@@ -9,10 +9,10 @@ pycheck_dirs=(
     "tools/block-discover-to-config"
     "tools/curtin-log-print"
     "tools/noproxy"
-    "tools/remove-vmtest-release"
     "tools/schema-validate-storage"
     "tools/ssh-keys-list"
     "tools/vmtest-filter"
+    "tools/vmtest-remove-release"
     "tools/vmtest-sync-images"
     "tools/webserv"
     "tools/write-curtin"
diff --git a/tools/remove-vmtest-release b/tools/vmtest-remove-release
similarity index 97%
rename from tools/remove-vmtest-release
rename to tools/vmtest-remove-release
old mode 100644
new mode 100755
index 8ab9b2e..d2c5f83
--- a/tools/remove-vmtest-release
+++ b/tools/vmtest-remove-release
@@ -36,7 +36,7 @@ def clean_file(fname, distro):
 
 if __name__ == "__main__":
     parser = argparse.ArgumentParser(
-        prog="remove-vmtest-release",
+        prog="vmtest-remove-release",
         description="Tool to remove vmtest classes by distro release")
     parser.add_argument('--distro-release', '-d',
                         action='store', required=True)