Merge pull request #572 from cmonr/issue_557

Restored `mbed target --supported` behavior

Author: theotherjimmy

circle.yml

@@ -22,6 +22,57 @@ test:
         - cd .tests/bld-test/mbed && mbed update 85 --clean
         - cd .tests/bld-test && mbed update --clean
         - cd .tests/bld-test && mbed compile -m LPC1768 -j 0
+        - cd .tests && mbed new supported-tests
+        - |-
+            cd .tests/supported-tests
+            echo "Testing mbed-os-5.6.0"
+            mbed update --clean "mbed-os-5.6.0"
+            mbed compile -S
+            mbed toolchain -S
+            mbed target -S
+            mbed export -S
+        - |-
+            cd .tests/supported-tests
+            echo "Testing mbed-os-5.5.0"
+            mbed update --clean "mbed-os-5.5.0"
+            mbed compile -S
+            mbed toolchain -S
+            mbed target -S
+            mbed export -S
+        - |-
+            cd .tests/supported-tests
+            echo "Testing mbed-os-5.4.0"
+            mbed update --clean "mbed-os-5.4.0"
+            mbed compile -S
+            mbed toolchain -S
+            mbed target -S
+            mbed export -S 
+        - |-
+            cd .tests/supported-tests
+            echo "Testing mbed-os-5.3.0"
+            mbed update --clean "mbed-os-5.3.0"
+            mbed compile -S
+            mbed toolchain -S
+            mbed target -S
+            mbed export -S
+        - |-
+            cd .tests/supported-tests
+            echo "Testing mbed-os-5.2.0"
+            mbed update --clean "mbed-os-5.2.0"
+            mbed compile -S
+            mbed toolchain -S
+            mbed target -S
+            mbed export -S
+        - |-
+            cd .tests/supported-tests
+            echo "Testing mbed-os-5.1.0"
+            mbed update --clean "mbed-os-5.1.0"
+            mbed compile -S
+            mbed toolchain -S
+            mbed target -S
+            mbed export -S
+
+
 
 dependencies:
     pre:

mbed/mbed.py

@@ -2256,7 +2256,7 @@ def status_(ignore=False):
     dict(name=['-c', '--clean'], action='store_true', help='Clean the build directory before compiling'),
     dict(name=['-f', '--flash'], action='store_true', help='Flash the built firmware onto a connected target.'),
     dict(name=['-N', '--artifact-name'], help='Name of the built program or library'),
-    dict(name=['-S', '--supported'], dest='supported', const="matrix", choices=["matrix", "toolchains", "targets"], nargs="?", help='Shows supported matrix of targets and toolchains'),
+    dict(name=['-S', '--supported'], dest='supported', const=True, choices=["matrix", "toolchains", "targets"], nargs="?", help='Shows supported matrix of targets and toolchains'),
     dict(name='--app-config', dest="app_config", help="Path of an app configuration file (Default is to look for 'mbed_app.json')"),
     help='Compile code using the mbed build tools',
     description=("Compile this program using the mbed build tools."))
@@ -2280,7 +2280,7 @@ def compile_(toolchain=None, target=None, profile=False, compile_library=False,
 
     if supported:
         popen([python_cmd, '-u', os.path.join(tools_dir, 'make.py')]
-              + (['-S', supported] if supported else []) + (['-v'] if very_verbose else [])
+              + (['-S', supported] if (supported is not True) else ['-S']) + (['-v'] if very_verbose else [])
               + (['--app-config', app_config] if app_config else [])
               + args,
               env=env)
@@ -2470,7 +2470,7 @@ def test_(toolchain=None, target=None, compile_list=False, run_list=False, compi
     dict(name=['-m', '--target'], help='Export for target MCU. Example: K64F, NUCLEO_F401RE, NRF51822...'),
     dict(name='--source', action='append', help='Source directory. Default: . (current dir)'),
     dict(name=['-c', '--clean'], action='store_true', help='Clean the build directory before compiling'),
-    dict(name=['-S', '--supported'], dest='supported', const="matrix", choices=["matrix", "ides"], nargs="?", help='Shows supported matrix of targets and toolchains'),
+    dict(name=['-S', '--supported'], dest='supported', const=True, choices=['matrix', 'ides'], nargs='?', help='Shows supported matrix of targets and toolchains'),
     dict(name='--app-config', dest="app_config", help="Path of an app configuration file (Default is to look for 'mbed_app.json')"),
     help='Generate an IDE project',
     description=(
@@ -2492,7 +2492,7 @@ def export(ide=None, target=None, source=False, clean=False, supported=False, ap
 
     if supported:
         popen([python_cmd, '-u', os.path.join(tools_dir, 'project.py')]
-              + (['-S', supported] if supported else []) + (['-v'] if very_verbose else []),
+              + (['-S', supported] if (supported is not True) else ['-S']) + (['-v'] if very_verbose else []),
               env=env)
         return