use argparse instead of optparse

1 files changed, 8 insertions, 9 deletions

diff --git a/build/scripts/copy_to_dir.py b/build/scripts/copy_to_dir.py
index 9baeb5ffac..b53db8d066 100644
--- a/build/scripts/copy_to_dir.py
+++ b/build/scripts/copy_to_dir.py
@@ -1,16 +1,17 @@
+import argparse
 import errno
 import sys
 import os
 import shutil
-import optparse
 import tarfile
 
 
 def parse_args():
-    parser = optparse.OptionParser()
-    parser.add_option('--build-root')
-    parser.add_option('--dest-dir')
-    parser.add_option('--dest-arch')
+    parser = argparse.ArgumentParser()
+    parser.add_argument('--build-root', required=True)
+    parser.add_argument('--dest-arch', default=None)
+    parser.add_argument('--dest-dir', required=True)
+    parser.add_argument('args', nargs='*')
     return parser.parse_args()
 
 
@@ -41,9 +42,7 @@ def hardlink_or_copy(src, dst):
 
 
 def main():
-    opts, args = parse_args()
-    assert opts.build_root
-    assert opts.dest_dir
+    opts = parse_args()
 
     dest_arch = None
     if opts.dest_arch:
@@ -55,7 +54,7 @@ def main():
             # TODO: move check to graph generation stage
             raise Exception('Unsopported archive type for {}. Use one of: tar, tar.gz, tgz.'.format(os.path.basename(opts.dest_arch)))
 
-    for arg in args:
+    for arg in opts.args:
         dst = arg
         if dst.startswith(opts.build_root):
             dst = dst[len(opts.build_root) + 1:]