Uploaded image for project: 'Jenkins'
  1. Jenkins
  2. JENKINS-6658

Add option to ignore submodules

    XMLWordPrintable

Details

    • Improvement
    • Status: Closed (View Workflow)
    • Major
    • Resolution: Fixed
    • git-plugin
    • 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.

      Attachments

        Issue Links

          Activity

            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 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 added a comment -

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

            marcnarc marcnarc added a comment - Found the newer repository on github. Here's a patch against 1.0.1.
            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 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.)
            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 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.

            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 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>
            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
            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

            People

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

              Dates

                Created:
                Updated:
                Resolved: