Submodule

.moban.cd/changelog.yml

@@ -1,6 +1,12 @@
 name: moban
 organisation: moremoban
 releases:
+- changes:
+  - action: Added
+    details:
+    - "alternative and expanded syntax for requires, so as to accomendate github submodule recursive"
+  date: unreleased
+  version: 0.3.3
 - changes:
   - action: Added
     details:

.moban.cd/moban.yml

@@ -3,8 +3,8 @@ organisation: moremoban
 author: C. W.
 contact: wangc_2011@hotmail.com
 license: MIT
-version: 0.3.2
-current_version: 0.3.2
+version: 0.3.3
+current_version: 0.3.3
 release: 0.3.2
 branch: master
 command_line_interface: "moban"

CHANGELOG.rst

@@ -1,6 +1,15 @@
 Change log
 ================================================================================
 
+0.3.3 - unreleased
+--------------------------------------------------------------------------------
+
+Added
+^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
+
+#. alternative and expanded syntax for requires, so as to accomendate github
+   submodule recursive
+
 0.3.2 - 05-11-2018
 --------------------------------------------------------------------------------
 

docs/conf.py

@@ -30,7 +30,7 @@
 # The short X.Y version
 version = u'0.3.2'
 # The full version, including alpha/beta/rc tags
-release = u'0.3.2'
+release = u'0.3.3'
 
 
 # -- General configuration ---------------------------------------------------

docs/level-10-moban-dependency-as-git-repo/README.rst

@@ -23,4 +23,17 @@ Here are the sample file::
       - test.txt: demo.txt.jj2
 
 where `requires` lead to a list of pypi packages. And when you refer to it,
-please use "pypi-mobans:"
+as in level-9 section, please use "pypi-mobans:"
+
+
+Alternative syntax when submodule exists
+--------------------------------------------------------------------------------
+
+The alternative syntax is::
+
+    requires:
+       - type: git
+         url: https://github.com/your-git-url
+         submodule: true
+    ...
+

docs/level-9-moban-dependency-as-pypi-package/README.rst

@@ -21,5 +21,29 @@ Here are the sample file::
       - mytravis.yml: travis.yml.jj2
       - test.txt: demo.txt.jj2
 
-where `requires` lead to a list of pypi packages. And when you refer to it,
-please use "pypi-mobans:"
+where `requires` lead to a list of pypi packages. The short syntax is::
+
+    requires:
+      - python-package-name
+
+When you refer to it in configuration section, here is the syntax::
+
+    configuration:
+      - template_dir:
+        - "python-package-name:relative-folder-inside-the-package"
+
+Note: when you do not have relative directory, please keep semi-colon::
+
+    configuration:
+      template_dir:
+        - "python-package-name:"
+
+Alternative syntax
+--------------------------------------------------------------------------------
+
+The alternative syntax is::
+
+    requires:
+       - type: pypi
+         name: pypi-mobans
+    ...

moban/_version.py

@@ -1,2 +1,2 @@
-__version__ = "0.3.2"
+__version__ = "0.3.3"
 __author__ = "C. W."

moban/constants.py

@@ -72,6 +72,14 @@
 ERROR = 2
 NO_CHANGES = 0
 
+# Require
+GIT_REQUIRE = "GIT"
+GIT_HAS_SUBMODULE = "submodule"
+GIT_URL = "url"
+PYPI_REQUIRE = "PYPI"
+PYPI_PACKAGE_NAME = "name"
+REQUIRE_TYPE = "type"
+
 
 # Extension
 JINJA_FILTER_EXTENSION = "jinja_filter"

moban/mobanfile.py

@@ -156,15 +156,29 @@ def extract_target(options):
 def handle_requires(requires):
     pypi_pkgs = []
     git_repos = []
+    git_repos_with_sub = []
     for require in requires:
-        if is_repo(require):
-            git_repos.append(require)
+        if isinstance(require, dict):
+            require_type = require.get(constants.REQUIRE_TYPE, "")
+            if require_type.upper() == constants.GIT_REQUIRE:
+                submodule_flag = require.get(constants.GIT_HAS_SUBMODULE)
+                if submodule_flag is True:
+                    git_repos_with_sub.append(require.get(constants.GIT_URL))
+                else:
+                    git_repos.append(require.get(constants.GIT_URL))
+            elif require_type.upper() == constants.PYPI_REQUIRE:
+                pypi_pkgs.append(require.get(constants.PYPI_PACKAGE_NAME))
         else:
-            pypi_pkgs.append(require)
+            if is_repo(require):
+                git_repos.append(require)
+            else:
+                pypi_pkgs.append(require)
     if pypi_pkgs:
         pip_install(pypi_pkgs)
     if git_repos:
         git_clone(git_repos)
+    if git_repos_with_sub:
+        git_clone(git_repos_with_sub, submodule=True)
 
 
 def is_repo(require):

moban/utils.py

@@ -149,7 +149,7 @@ def pip_install(packages):
     )
 
 
-def git_clone(repos):
+def git_clone(repos, submodule=False):
     import subprocess
 
     moban_home = get_moban_home()
@@ -163,10 +163,16 @@ def git_clone(repos):
             reporter.report_git_pull(repo_name)
             os.chdir(local_repo_folder)
             subprocess.check_call(["git", "pull"])
+            if submodule:
+                subprocess.check_call(["git", "submodule", "update"])
         else:
             reporter.report_git_clone(repo_name)
             os.chdir(moban_home)
             subprocess.check_call(["git", "clone", repo, repo_name])
+            if submodule:
+                os.chdir(os.path.join(moban_home, repo_name))
+                subprocess.check_call(["git", "submodule", "init"])
+                subprocess.check_call(["git", "submodule", "update"])
         os.chdir(current_working_dir)
 
 

setup.py

@@ -13,7 +13,7 @@
 
 NAME = 'moban'
 AUTHOR = 'C. W.'
-VERSION = '0.3.2'
+VERSION = '0.3.3'
 EMAIL = 'wangc_2011@hotmail.com'
 LICENSE = 'MIT'
 ENTRY_POINTS = {

tests/test_moban_file.py

@@ -43,6 +43,15 @@ def test_handle_requires_pypkg(fake_pip_install):
     fake_pip_install.assert_called_with(modules)
 
 
+@patch("moban.mobanfile.pip_install")
+def test_handle_requires_pypkg_with_alternative_syntax(fake_pip_install):
+    modules = [{"type": "pypi", "name": "pypi-mobans"}]
+    from moban.mobanfile import handle_requires
+
+    handle_requires(modules)
+    fake_pip_install.assert_called_with(["pypi-mobans"])
+
+
 @patch("moban.mobanfile.git_clone")
 def test_handle_requires_repos(fake_git_clone):
     repos = ["https://github.com/my/repo", "https://gitlab.com/my/repo"]
@@ -52,6 +61,27 @@ def test_handle_requires_repos(fake_git_clone):
     fake_git_clone.assert_called_with(repos)
 
 
+@patch("moban.mobanfile.git_clone")
+def test_handle_requires_repos_with_alternative_syntax(fake_git_clone):
+    repos = [{"type": "git", "url": "https://github.com/my/repo"}]
+    from moban.mobanfile import handle_requires
+
+    handle_requires(repos)
+    fake_git_clone.assert_called_with(["https://github.com/my/repo"])
+
+
+@patch("moban.mobanfile.git_clone")
+def test_handle_requires_repos_with_submodule(fake_git_clone):
+    repos = [
+        {"type": "git", "url": "https://github.com/my/repo", "submodule": True}
+    ]
+    from moban.mobanfile import handle_requires
+
+    expected = ["https://github.com/my/repo"]
+    handle_requires(repos)
+    fake_git_clone.assert_called_with(expected, submodule=True)
+
+
 def test_is_repo():
     repos = [
         "https://github.com/my/repo",

tests/test_utils.py

@@ -126,6 +126,31 @@ def test_git_clone(fake_check_all):
     )
 
 
+@patch("os.chdir")
+@patch("subprocess.check_call")
+def test_git_clone_with_submodules(fake_check_all, _):
+    from moban.utils import git_clone
+
+    git_clone(
+        ["https://github.com/my/repo", "https://gitlab.com/my/repo"],
+        submodule=True,
+    )
+    fake_check_all.assert_called_with(["git", "submodule", "update"])
+
+
+@patch("os.path.exists", return_value=True)
+@patch("os.chdir")
+@patch("subprocess.check_call")
+def test_git_clone_with_existing_repo(fake_check_all, _, __):
+    from moban.utils import git_clone
+
+    git_clone(
+        ["https://github.com/my/repo", "https://gitlab.com/my/repo"],
+        submodule=True,
+    )
+    fake_check_all.assert_called_with(["git", "submodule", "update"])
+
+
 def test_get_repo_name():
     repos = ["https://github.com/abc/repo", "https://github.com/abc/repo/"]
     actual = [get_repo_name(repo) for repo in repos]