Merge pull request #237 from Mbed-TLS/dev/bensze01/submodules

Clone all submodules without groovy handlers

Author: bensze01

vars/checkout_repo.groovy

@@ -86,7 +86,7 @@ String get_submodule_commit(String working_dir = '.', String submodule) {
     ).trim()
 }
 
-void checkout_framework_repo(BranchInfo info) {
+private void checkout_framework_repo(BranchInfo info) {
     if (env.TARGET_REPO == 'framework' && env.CHECKOUT_METHOD == 'scm') {
         checkout_report_errors(scm)
     } else {
@@ -104,7 +104,7 @@ void checkout_framework_repo(BranchInfo info) {
     }
 }
 
-void checkout_tf_psa_crypto_repo(BranchInfo info) {
+private void checkout_tf_psa_crypto_repo(BranchInfo info) {
     if (env.TARGET_REPO == 'tf-psa-crypto' && env.CHECKOUT_METHOD == 'scm') {
         checkout_report_errors(scm)
         if (!info.framework_override) {
@@ -140,7 +140,7 @@ void checkout_tf_psa_crypto_repo(BranchInfo info) {
     }
 }
 
-Map<String, String> checkout_tls_repo(BranchInfo info) {
+private Map<String, String> checkout_tls_repo(BranchInfo info) {
     if (info.repo != 'tls') {
         throw new IllegalArgumentException("checkout_tls_repo() called with BranchInfo for repo '$info.repo'")
     }
@@ -188,6 +188,19 @@ void checkout_repo(BranchInfo info) {
                             error("Invalid repo: ${info.repo}")
                     }
 
+                    echo 'Clone all remaining submodules without dedicated handlers'
+                    sh '''
+# Initialize submodule variables (required by `git submodule status`)
+git submodule init
+git submodule foreach 'git submodule init'
+
+# Get list of missing submodules (lines starting with '-')
+missing_submodules=$(git submodule status --recursive | awk '/^-/{print $2}')
+for module in $missing_submodules; do
+    git -C "$module" submodule update --recursive --init --depth=1 .
+done
+'''
+
                     stash name: stashName, includes: '**/*', useDefaultExcludes: false
                     info.stash = stashName
                     needUnstash = false

vars/gen_jobs.groovy

@@ -657,7 +657,7 @@ def gen_coverity_push_jobs(BranchInfo info) {
             try {
                 dir("src") {
                     deleteDir()
-                    checkout_repo.checkout_tls_repo(info)
+                    checkout_repo.checkout_repo(info)
                     sshagent([env.GIT_CREDENTIALS_ID]) {
                         analysis.record_inner_timestamps('container-host', job_name) {
                             // Git complains about non-fast-forward operations when trying to push a shallow commit