Prospective fix for unnecessary submodule cloning dev_1427450742 40/109440/2
authorSimon Hausmann <simon.hausmann@theqtcompany.com>
Wed, 25 Mar 2015 11:17:47 +0000 (12:17 +0100)
committerJani Heikkinen <jani.heikkinen@theqtcompany.com>
Fri, 27 Mar 2015 10:04:11 +0000 (10:04 +0000)
When cloning qtdeclarative, we end up also cloning qtdeclarative-testsuites,
which is not intentional for users of Qt (and init-repository). The init-repository
script checks for the submodule "repo".update key and skips the sub module if the value
is "ignore". However according to the documentation no checkout is to be done if the value
is "none", which is also what qtdeclarative's .gitmodules specifies for the ecmascript
test suite.

Change-Id: Ieb37607ae4ea7c32d6af886d3e2533fb9dea1f13
Reviewed-by: Jani Heikkinen <jani.heikkinen@theqtcompany.com>
init-repository

index 46456e9..e80c7dc 100755 (executable)
@@ -334,7 +334,7 @@ sub git_clone_all_submodules
             while ($base =~ s,/(?!\.\./)[^/]+/\.\./,/,g) {}
             $subbases{$mod} = $base;
         } elsif ($2 eq "update") {
-            push @subset, '-'.$1 if ($3 eq 'ignore');
+            push @subset, '-'.$1 if ($3 eq 'none');
         } elsif ($2 eq "initrepo") {
             $subinits{$1} = ($3 eq "yes" or $3 eq "true");
         }