From 109bbd27f20b863c8a353385952aeeb7f47e9737 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?F=C3=A9lix=20Pi=C3=A9dallu?= <felix@piedallu.me>
Date: Mon, 19 Aug 2024 17:09:53 +0200
Subject: [PATCH] autopatches: use argparse

---
 autopatches/autopatch.py | 50 ++++++++++++++++++----------------------
 1 file changed, 23 insertions(+), 27 deletions(-)

diff --git a/autopatches/autopatch.py b/autopatches/autopatch.py
index 20190e3..a1ea77d 100755
--- a/autopatches/autopatch.py
+++ b/autopatches/autopatch.py
@@ -1,5 +1,6 @@
 #!/usr/bin/env python3
 
+import argparse
 import json
 import os
 import subprocess
@@ -193,36 +194,31 @@ def create_pull_request(repo, patch, base_branch, s):
 
 
 def main():
-    action = sys.argv[1]
-    if action == "--help":
-        print(
-            """
-    Example usage:
+    parser = argparse.ArgumentParser()
+    parser.add_argument("the_patch", type=str, nargs="?", help="The name of the patch to apply")
+    parser.add_argument("--cache", "-b", action="store_true", help="Init local git clone for all apps")
+    parser.add_argument("--apply", "-a", action="store_true", help="Apply patch on all local clones")
+    parser.add_argument("--diff", "-d", action="store_true", help="Inspect diff for all apps")
+    parser.add_argument("--push", "-p", action="store_true", help="Push and create pull requests on all apps with non-empty diff")
+    args = parser.parse_args()
 
-# Init local git clone for all apps
-./autopatch.py --build-cache
+    if not (args.cache or args.apply or args.diff or args.push):
+        parser.error("We required --cache, --apply, --diff or --push.")
 
-# Apply patch in all local clones
-./autopatch.py --apply explicit-php-version-in-deps
-
-# Inspect diff for all apps
-./autopatch.py --diff
-
-# Push and create pull requests on all apps with non-empty diff
-./autopatch.py --push explicit-php-version-in-deps
-"""
-        )
-
-    elif action == "--build-cache":
+    if args.cache:
         build_cache()
-    elif action == "--apply":
-        apply(sys.argv[2])
-    elif action == "--diff":
-        diff()
-    elif action == "--push":
-        push(sys.argv[2])
-    else:
-        print("Unknown action %s" % action)
 
+    if args.apply:
+        if not args.the_patch:
+            parser.error("--apply requires the patch name to be passed")
+        apply(args.the_patch)
+
+    if args.diff:
+        diff()
+
+    if args.push:
+        if not args.the_patch:
+            parser.error("--push requires the patch name to be passed")
+        push(args.the_patch)
 
 main()