Receive use_docker flag from the cli

python/rpdk/python/codegen.py

@@ -49,23 +49,25 @@ def __init__(self):
  self.namespace = None
  self.package_name = None
  self.package_root = None
- self._use_docker = True
+ self._use_docker = None
  self._protocol_version = "2.0.0"
 
  def _init_from_project(self, project):
  self.namespace = tuple(s.lower() for s in project.type_info)
  self.package_name = "_".join(self.namespace)
- self._use_docker = project.settings.get("use_docker", True)
+ self._use_docker = project.settings.get("use_docker")
  self.package_root = project.root / "src"
 
  def _init_settings(self, project):
  LOG.debug("Writing settings")
- self._use_docker = input_with_validation(
+
+ self._use_docker = self._use_docker or input_with_validation(
  "Use docker for platform-independent packaging (Y/n)?\n",
  validate_no,
  "This is highly recommended unless you are experienced \n"
  "with cross-platform Python packaging.",
  )
+
  project.settings["use_docker"] = self._use_docker
  project.settings["protocolVersion"] = self._protocol_version
 

python/rpdk/python/parser.py

@@ -0,0 +1,29 @@
+def setup_subparser(subparsers, parents, python_version):
+ parser = subparsers.add_parser(
+ python_version,
+ description="""This sub command generates IDE and build files for Python {}
+ """.format(
+ "3.6" if python_version == "python36" else "3.7"
+ ),
+ parents=parents,
+ )
+ parser.set_defaults(language=python_version)
+
+ parser.add_argument(
+ "-d",
+ "--use-docker",
+ action="store_true",
+ help="""Use docker for python platform-independent packaging.
+ This is highly recommended unless you are experienced
+ with cross-platform Python packaging.""",
+ )
+
+ return parser
+
+
+def setup_subparser_python36(subparsers, parents):
+ return setup_subparser(subparsers, parents, "python36")
+
+
+def setup_subparser_python37(subparsers, parents):
+ return setup_subparser(subparsers, parents, "python37")

setup.py

@@ -43,7 +43,11 @@ def find_version(*file_paths):
  "rpdk.v1.languages": [
  "python37 = rpdk.python.codegen:Python37LanguagePlugin",
  "python36 = rpdk.python.codegen:Python36LanguagePlugin",
- ]
+ ],
+ "rpdk.v1.parsers": [
+ "python37 = rpdk.python.parser:setup_subparser_python37",
+ "python36 = rpdk.python.parser:setup_subparser_python36",
+ ],
  },
  license="Apache License 2.0",
  classifiers=[

tests/plugin/parser_test.py

@@ -0,0 +1,27 @@
+import argparse
+
+from rpdk.python.parser import setup_subparser_python36, setup_subparser_python37
+
+
+def test_setup_subparser_python36():
+ parser = argparse.ArgumentParser()
+ subparsers = parser.add_subparsers(dest="subparser_name")
+
+ sub_parser = setup_subparser_python36(subparsers, [])
+
+ args = sub_parser.parse_args(["-d"])
+
+ assert args.language == "python36"
+ assert args.use_docker is True
+
+
+def test_setup_subparser_python37():
+ parser = argparse.ArgumentParser()
+ subparsers = parser.add_subparsers(dest="subparser_name")
+
+ sub_parser = setup_subparser_python37(subparsers, [])
+
+ args = sub_parser.parse_args([])
+
+ assert args.language == "python37"
+ assert args.use_docker is False