Re-added --sdk argument for sdl2 bootstrap, but with a warning

inclement · inclement · commit 726ddebb8708 · 2017-10-05T00:04:39.000+01:00
diff --git a/pythonforandroid/bootstraps/sdl2/build/build.py b/pythonforandroid/bootstraps/sdl2/build/build.py
@@ -470,9 +470,10 @@ def parse_args(args=None):
                     help=('Add a Java .jar to the libs, so you can access its '
                           'classes with pyjnius. You can specify this '
                           'argument more than once to include multiple jars'))
+    ## The --sdk option has been removed, it is ignored in favour of
+    ## --android-api handled by toolchain.py
     ap.add_argument('--sdk', dest='sdk_version', default=-1,
-                    type=int, help=('Android SDK version to use. Default to '
-                                    'the value of minsdk'))
+                    type=int, help=('Deprecated argument, does nothing'))
     ap.add_argument('--minsdk', dest='min_sdk_version',
                     default=default_android_api, type=int,
                     help=('Minimum Android SDK version to use. Default to '
@@ -509,8 +510,12 @@ def parse_args(args=None):
         print('Billing not yet supported in sdl2 bootstrap!')
         exit(1)
 
-    if args.sdk_version == -1:
-        args.sdk_version = args.min_sdk_version
+    # if args.sdk_version == -1:
+    #     args.sdk_version = args.min_sdk_version
+
+    if args.sdk_version != -1:
+        print('WARNING: Received a --sdk argument, but this argument is '
+              'deprecated and does nothing.')
 
     if args.permissions is None:
         args.permissions = []
