Bug 1304176 - Remove --venv-path as an alias to --virtualenv-path; r?ted draft
authorGregory Szorc <gps@mozilla.com>
Tue, 20 Sep 2016 14:46:52 -0700
changeset 417183 93562be5a75bc8d0994bcc068dfa2f3584c4c636
parent 417182 fcc14f64db50a6cd36db871b0d6bc0045b8bcd62
child 417184 006b205c5e8f8c5b2398115feb362ba77a096d88
push id30356
push userbmo:gps@mozilla.com
push dateFri, 23 Sep 2016 19:11:51 +0000
reviewersted
bugs1304176, 651974
milestone52.0a1
Bug 1304176 - Remove --venv-path as an alias to --virtualenv-path; r?ted Not sure why we support this. The code goes all the way back to the import of mozharness 0.4 into the old mozharness Mercurial repo in bug 651974. Having fewer variations makes it easier to search for usage. So nuke the variation. MozReview-Commit-ID: IgwLMdvXGB0
testing/mozharness/mozharness/base/python.py
--- a/testing/mozharness/mozharness/base/python.py
+++ b/testing/mozharness/mozharness/base/python.py
@@ -46,17 +46,17 @@ def get_tlsv1_post():
                                            block=block,
                                            ssl_version=ssl.PROTOCOL_TLSv1)
     s = requests.Session()
     s.mount('https://', TLSV1Adapter())
     return s.post
 
 # Virtualenv {{{1
 virtualenv_config_options = [
-    [["--venv-path", "--virtualenv-path"], {
+    [["--virtualenv-path"], {
         "action": "store",
         "dest": "virtualenv_path",
         "default": "venv",
         "help": "Specify the path to the virtualenv top level directory"
     }],
     [["--virtualenv"], {
         "action": "store",
         "dest": "virtualenv",