Details

    • Type: Improvement
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Component/s: git-plugin
    • Labels:
      None

      Description

      We have many submodules in our repo, and different build targets require different submodules. Our build process takes care of obtaining the correct submodules depending on what's being built.

      The git plugin's default practice of cloning all the submodules in the repo is inefficient for us (many of our submodules are full Linux kernels, for example).

      It would be great if there were an option to ignore submodules in the repo.

        Issue Links

          Activity

          marcnarc marcnarc created issue -
          Hide
          marcnarc marcnarc added a comment -

          Here's a patch that implements this as an "advanced" option.

          Unfortunately, this patch is against 0.8.3 – I'm using the repo on github as my baseline, and AFAICT that hasn't been updated in ages.

          Show
          marcnarc marcnarc added a comment - Here's a patch that implements this as an "advanced" option. Unfortunately, this patch is against 0.8.3 – I'm using the repo on github as my baseline, and AFAICT that hasn't been updated in ages.
          marcnarc marcnarc made changes -
          Field Original Value New Value
          Attachment 0001-Add-option-to-ignore-submodules.patch [ 19645 ]
          Hide
          marcnarc marcnarc added a comment -

          Found the newer repository on github. Here's a patch against 1.0.1.

          Show
          marcnarc marcnarc added a comment - Found the newer repository on github. Here's a patch against 1.0.1.
          marcnarc marcnarc made changes -
          Attachment 1.0.1-ignore_submodules.patch [ 19646 ]
          Hide
          marcnarc marcnarc added a comment -

          Updated the patch for 1.1.

          (The change made for JENKINS-6258 only applies to submodules that contain other submodules. This patch allows a job to ignore all submodules.)

          Show
          marcnarc marcnarc added a comment - Updated the patch for 1.1. (The change made for JENKINS-6258 only applies to submodules that contain other submodules. This patch allows a job to ignore all submodules.)
          marcnarc marcnarc made changes -
          Attachment 1.1-ignore-submodules.patch [ 19911 ]
          zaytsev Yury Zaytsev made changes -
          Status Open [ 1 ] In Progress [ 3 ]
          Hide
          zaytsev Yury Zaytsev added a comment -

          Hi!

          I have created a pull request: https://github.com/jenkinsci/git-plugin/pull/48 . I found this issue only later. I think they should be linked together. Also there is an earlier pull request: https://github.com/jenkinsci/git-plugin/pull/43 ...

          Z.

          Show
          zaytsev Yury Zaytsev added a comment - Hi! I have created a pull request: https://github.com/jenkinsci/git-plugin/pull/48 . I found this issue only later. I think they should be linked together. Also there is an earlier pull request: https://github.com/jenkinsci/git-plugin/pull/43 ... Z.
          zaytsev Yury Zaytsev made changes -
          Link This issue is blocking JENKINS-8904 [ JENKINS-8904 ]
          Hide
          scm_issue_link SCM/JIRA link daemon added a comment -

          Code changed in jenkins
          User: Yury V. Zaytsev
          Path:
          src/main/webapp/help-disableSubmodules.html
          http://jenkins-ci.org/commit/git-plugin/645ec1328f2342cf2edda93957a0bbbbb908812b
          Log:
          [FIXED JENKINS-6658] Reworded the help message as per @jieryn suggestion

          Signed-off-by: Yury V. Zaytsev <yury@shurup.com>

          Show
          scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Yury V. Zaytsev Path: src/main/webapp/help-disableSubmodules.html http://jenkins-ci.org/commit/git-plugin/645ec1328f2342cf2edda93957a0bbbbb908812b Log: [FIXED JENKINS-6658] Reworded the help message as per @jieryn suggestion Signed-off-by: Yury V. Zaytsev <yury@shurup.com>
          scm_issue_link SCM/JIRA link daemon made changes -
          Status In Progress [ 3 ] Resolved [ 5 ]
          Resolution Fixed [ 1 ]
          Hide
          dogfood dogfood added a comment -

          Integrated in plugins_git-plugin #260
          [FIXED JENKINS-6658] Reworded the help message as per @jieryn suggestion

          yury : 645ec1328f2342cf2edda93957a0bbbbb908812b
          Files :

          • src/main/webapp/help-disableSubmodules.html
          Show
          dogfood dogfood added a comment - Integrated in plugins_git-plugin #260 [FIXED JENKINS-6658] Reworded the help message as per @jieryn suggestion yury : 645ec1328f2342cf2edda93957a0bbbbb908812b Files : src/main/webapp/help-disableSubmodules.html
          markewaite Mark Waite made changes -
          Status Resolved [ 5 ] Closed [ 6 ]

            People

            • Assignee:
              magnayn magnayn
              Reporter:
              marcnarc marcnarc
            • Votes:
              4 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: