hashmap: use *_entry APIs for iteration
[git] / submodule-config.c
1 #include "cache.h"
2 #include "dir.h"
3 #include "repository.h"
4 #include "config.h"
5 #include "submodule-config.h"
6 #include "submodule.h"
7 #include "strbuf.h"
8 #include "object-store.h"
9 #include "parse-options.h"
10
11 /*
12  * submodule cache lookup structure
13  * There is one shared set of 'struct submodule' entries which can be
14  * looked up by their sha1 blob id of the .gitmodules file and either
15  * using path or name as key.
16  * for_path stores submodule entries with path as key
17  * for_name stores submodule entries with name as key
18  */
19 struct submodule_cache {
20         struct hashmap for_path;
21         struct hashmap for_name;
22         unsigned initialized:1;
23         unsigned gitmodules_read:1;
24 };
25
26 /*
27  * thin wrapper struct needed to insert 'struct submodule' entries to
28  * the hashmap
29  */
30 struct submodule_entry {
31         struct hashmap_entry ent;
32         struct submodule *config;
33 };
34
35 enum lookup_type {
36         lookup_name,
37         lookup_path
38 };
39
40 static int config_path_cmp(const void *unused_cmp_data,
41                            const struct hashmap_entry *eptr,
42                            const struct hashmap_entry *entry_or_key,
43                            const void *unused_keydata)
44 {
45         const struct submodule_entry *a, *b;
46
47         a = container_of(eptr, const struct submodule_entry, ent);
48         b = container_of(entry_or_key, const struct submodule_entry, ent);
49
50         return strcmp(a->config->path, b->config->path) ||
51                !oideq(&a->config->gitmodules_oid, &b->config->gitmodules_oid);
52 }
53
54 static int config_name_cmp(const void *unused_cmp_data,
55                            const struct hashmap_entry *eptr,
56                            const struct hashmap_entry *entry_or_key,
57                            const void *unused_keydata)
58 {
59         const struct submodule_entry *a, *b;
60
61         a = container_of(eptr, const struct submodule_entry, ent);
62         b = container_of(entry_or_key, const struct submodule_entry, ent);
63
64         return strcmp(a->config->name, b->config->name) ||
65                !oideq(&a->config->gitmodules_oid, &b->config->gitmodules_oid);
66 }
67
68 static struct submodule_cache *submodule_cache_alloc(void)
69 {
70         return xcalloc(1, sizeof(struct submodule_cache));
71 }
72
73 static void submodule_cache_init(struct submodule_cache *cache)
74 {
75         hashmap_init(&cache->for_path, config_path_cmp, NULL, 0);
76         hashmap_init(&cache->for_name, config_name_cmp, NULL, 0);
77         cache->initialized = 1;
78 }
79
80 static void free_one_config(struct submodule_entry *entry)
81 {
82         free((void *) entry->config->path);
83         free((void *) entry->config->name);
84         free((void *) entry->config->branch);
85         free((void *) entry->config->update_strategy.command);
86         free(entry->config);
87 }
88
89 static void submodule_cache_clear(struct submodule_cache *cache)
90 {
91         struct hashmap_iter iter;
92         struct submodule_entry *entry;
93
94         if (!cache->initialized)
95                 return;
96
97         /*
98          * We iterate over the name hash here to be symmetric with the
99          * allocation of struct submodule entries. Each is allocated by
100          * their .gitmodules blob sha1 and submodule name.
101          */
102         hashmap_for_each_entry(&cache->for_name, &iter, entry,
103                                 struct submodule_entry, ent /* member name */)
104                 free_one_config(entry);
105
106         hashmap_free(&cache->for_path, 1);
107         hashmap_free(&cache->for_name, 1);
108         cache->initialized = 0;
109         cache->gitmodules_read = 0;
110 }
111
112 void submodule_cache_free(struct submodule_cache *cache)
113 {
114         submodule_cache_clear(cache);
115         free(cache);
116 }
117
118 static unsigned int hash_oid_string(const struct object_id *oid,
119                                     const char *string)
120 {
121         return memhash(oid->hash, the_hash_algo->rawsz) + strhash(string);
122 }
123
124 static void cache_put_path(struct submodule_cache *cache,
125                            struct submodule *submodule)
126 {
127         unsigned int hash = hash_oid_string(&submodule->gitmodules_oid,
128                                             submodule->path);
129         struct submodule_entry *e = xmalloc(sizeof(*e));
130         hashmap_entry_init(&e->ent, hash);
131         e->config = submodule;
132         hashmap_put(&cache->for_path, &e->ent);
133 }
134
135 static void cache_remove_path(struct submodule_cache *cache,
136                               struct submodule *submodule)
137 {
138         unsigned int hash = hash_oid_string(&submodule->gitmodules_oid,
139                                             submodule->path);
140         struct submodule_entry e;
141         struct submodule_entry *removed;
142         hashmap_entry_init(&e.ent, hash);
143         e.config = submodule;
144         removed = hashmap_remove(&cache->for_path, &e.ent, NULL);
145         free(removed);
146 }
147
148 static void cache_add(struct submodule_cache *cache,
149                       struct submodule *submodule)
150 {
151         unsigned int hash = hash_oid_string(&submodule->gitmodules_oid,
152                                             submodule->name);
153         struct submodule_entry *e = xmalloc(sizeof(*e));
154         hashmap_entry_init(&e->ent, hash);
155         e->config = submodule;
156         hashmap_add(&cache->for_name, &e->ent);
157 }
158
159 static const struct submodule *cache_lookup_path(struct submodule_cache *cache,
160                 const struct object_id *gitmodules_oid, const char *path)
161 {
162         struct submodule_entry *entry;
163         unsigned int hash = hash_oid_string(gitmodules_oid, path);
164         struct submodule_entry key;
165         struct submodule key_config;
166
167         oidcpy(&key_config.gitmodules_oid, gitmodules_oid);
168         key_config.path = path;
169
170         hashmap_entry_init(&key.ent, hash);
171         key.config = &key_config;
172
173         entry = hashmap_get_entry(&cache->for_path, &key, NULL,
174                                 struct submodule_entry, ent);
175         if (entry)
176                 return entry->config;
177         return NULL;
178 }
179
180 static struct submodule *cache_lookup_name(struct submodule_cache *cache,
181                 const struct object_id *gitmodules_oid, const char *name)
182 {
183         struct submodule_entry *entry;
184         unsigned int hash = hash_oid_string(gitmodules_oid, name);
185         struct submodule_entry key;
186         struct submodule key_config;
187
188         oidcpy(&key_config.gitmodules_oid, gitmodules_oid);
189         key_config.name = name;
190
191         hashmap_entry_init(&key.ent, hash);
192         key.config = &key_config;
193
194         entry = hashmap_get_entry(&cache->for_name, &key, NULL,
195                                 struct submodule_entry, ent);
196         if (entry)
197                 return entry->config;
198         return NULL;
199 }
200
201 int check_submodule_name(const char *name)
202 {
203         /* Disallow empty names */
204         if (!*name)
205                 return -1;
206
207         /*
208          * Look for '..' as a path component. Check both '/' and '\\' as
209          * separators rather than is_dir_sep(), because we want the name rules
210          * to be consistent across platforms.
211          */
212         goto in_component; /* always start inside component */
213         while (*name) {
214                 char c = *name++;
215                 if (c == '/' || c == '\\') {
216 in_component:
217                         if (name[0] == '.' && name[1] == '.' &&
218                             (!name[2] || name[2] == '/' || name[2] == '\\'))
219                                 return -1;
220                 }
221         }
222
223         return 0;
224 }
225
226 static int name_and_item_from_var(const char *var, struct strbuf *name,
227                                   struct strbuf *item)
228 {
229         const char *subsection, *key;
230         int subsection_len, parse;
231         parse = parse_config_key(var, "submodule", &subsection,
232                         &subsection_len, &key);
233         if (parse < 0 || !subsection)
234                 return 0;
235
236         strbuf_add(name, subsection, subsection_len);
237         if (check_submodule_name(name->buf) < 0) {
238                 warning(_("ignoring suspicious submodule name: %s"), name->buf);
239                 strbuf_release(name);
240                 return 0;
241         }
242
243         strbuf_addstr(item, key);
244
245         return 1;
246 }
247
248 static struct submodule *lookup_or_create_by_name(struct submodule_cache *cache,
249                 const struct object_id *gitmodules_oid, const char *name)
250 {
251         struct submodule *submodule;
252         struct strbuf name_buf = STRBUF_INIT;
253
254         submodule = cache_lookup_name(cache, gitmodules_oid, name);
255         if (submodule)
256                 return submodule;
257
258         submodule = xmalloc(sizeof(*submodule));
259
260         strbuf_addstr(&name_buf, name);
261         submodule->name = strbuf_detach(&name_buf, NULL);
262
263         submodule->path = NULL;
264         submodule->url = NULL;
265         submodule->update_strategy.type = SM_UPDATE_UNSPECIFIED;
266         submodule->update_strategy.command = NULL;
267         submodule->fetch_recurse = RECURSE_SUBMODULES_NONE;
268         submodule->ignore = NULL;
269         submodule->branch = NULL;
270         submodule->recommend_shallow = -1;
271
272         oidcpy(&submodule->gitmodules_oid, gitmodules_oid);
273
274         cache_add(cache, submodule);
275
276         return submodule;
277 }
278
279 static int parse_fetch_recurse(const char *opt, const char *arg,
280                                int die_on_error)
281 {
282         switch (git_parse_maybe_bool(arg)) {
283         case 1:
284                 return RECURSE_SUBMODULES_ON;
285         case 0:
286                 return RECURSE_SUBMODULES_OFF;
287         default:
288                 if (!strcmp(arg, "on-demand"))
289                         return RECURSE_SUBMODULES_ON_DEMAND;
290                 /*
291                  * Please update $__git_fetch_recurse_submodules in
292                  * git-completion.bash when you add new options.
293                  */
294                 if (die_on_error)
295                         die("bad %s argument: %s", opt, arg);
296                 else
297                         return RECURSE_SUBMODULES_ERROR;
298         }
299 }
300
301 int parse_submodule_fetchjobs(const char *var, const char *value)
302 {
303         int fetchjobs = git_config_int(var, value);
304         if (fetchjobs < 0)
305                 die(_("negative values not allowed for submodule.fetchjobs"));
306         return fetchjobs;
307 }
308
309 int parse_fetch_recurse_submodules_arg(const char *opt, const char *arg)
310 {
311         return parse_fetch_recurse(opt, arg, 1);
312 }
313
314 int option_fetch_parse_recurse_submodules(const struct option *opt,
315                                           const char *arg, int unset)
316 {
317         int *v;
318
319         if (!opt->value)
320                 return -1;
321
322         v = opt->value;
323
324         if (unset) {
325                 *v = RECURSE_SUBMODULES_OFF;
326         } else {
327                 if (arg)
328                         *v = parse_fetch_recurse_submodules_arg(opt->long_name, arg);
329                 else
330                         *v = RECURSE_SUBMODULES_ON;
331         }
332         return 0;
333 }
334
335 static int parse_update_recurse(const char *opt, const char *arg,
336                                 int die_on_error)
337 {
338         switch (git_parse_maybe_bool(arg)) {
339         case 1:
340                 return RECURSE_SUBMODULES_ON;
341         case 0:
342                 return RECURSE_SUBMODULES_OFF;
343         default:
344                 if (die_on_error)
345                         die("bad %s argument: %s", opt, arg);
346                 return RECURSE_SUBMODULES_ERROR;
347         }
348 }
349
350 int parse_update_recurse_submodules_arg(const char *opt, const char *arg)
351 {
352         return parse_update_recurse(opt, arg, 1);
353 }
354
355 static int parse_push_recurse(const char *opt, const char *arg,
356                                int die_on_error)
357 {
358         switch (git_parse_maybe_bool(arg)) {
359         case 1:
360                 /* There's no simple "on" value when pushing */
361                 if (die_on_error)
362                         die("bad %s argument: %s", opt, arg);
363                 else
364                         return RECURSE_SUBMODULES_ERROR;
365         case 0:
366                 return RECURSE_SUBMODULES_OFF;
367         default:
368                 if (!strcmp(arg, "on-demand"))
369                         return RECURSE_SUBMODULES_ON_DEMAND;
370                 else if (!strcmp(arg, "check"))
371                         return RECURSE_SUBMODULES_CHECK;
372                 else if (!strcmp(arg, "only"))
373                         return RECURSE_SUBMODULES_ONLY;
374                 /*
375                  * Please update $__git_push_recurse_submodules in
376                  * git-completion.bash when you add new modes.
377                  */
378                 else if (die_on_error)
379                         die("bad %s argument: %s", opt, arg);
380                 else
381                         return RECURSE_SUBMODULES_ERROR;
382         }
383 }
384
385 int parse_push_recurse_submodules_arg(const char *opt, const char *arg)
386 {
387         return parse_push_recurse(opt, arg, 1);
388 }
389
390 static void warn_multiple_config(const struct object_id *treeish_name,
391                                  const char *name, const char *option)
392 {
393         const char *commit_string = "WORKTREE";
394         if (treeish_name)
395                 commit_string = oid_to_hex(treeish_name);
396         warning("%s:.gitmodules, multiple configurations found for "
397                         "'submodule.%s.%s'. Skipping second one!",
398                         commit_string, name, option);
399 }
400
401 static void warn_command_line_option(const char *var, const char *value)
402 {
403         warning(_("ignoring '%s' which may be interpreted as"
404                   " a command-line option: %s"), var, value);
405 }
406
407 struct parse_config_parameter {
408         struct submodule_cache *cache;
409         const struct object_id *treeish_name;
410         const struct object_id *gitmodules_oid;
411         int overwrite;
412 };
413
414 static int parse_config(const char *var, const char *value, void *data)
415 {
416         struct parse_config_parameter *me = data;
417         struct submodule *submodule;
418         struct strbuf name = STRBUF_INIT, item = STRBUF_INIT;
419         int ret = 0;
420
421         /* this also ensures that we only parse submodule entries */
422         if (!name_and_item_from_var(var, &name, &item))
423                 return 0;
424
425         submodule = lookup_or_create_by_name(me->cache,
426                                              me->gitmodules_oid,
427                                              name.buf);
428
429         if (!strcmp(item.buf, "path")) {
430                 if (!value)
431                         ret = config_error_nonbool(var);
432                 else if (looks_like_command_line_option(value))
433                         warn_command_line_option(var, value);
434                 else if (!me->overwrite && submodule->path)
435                         warn_multiple_config(me->treeish_name, submodule->name,
436                                         "path");
437                 else {
438                         if (submodule->path)
439                                 cache_remove_path(me->cache, submodule);
440                         free((void *) submodule->path);
441                         submodule->path = xstrdup(value);
442                         cache_put_path(me->cache, submodule);
443                 }
444         } else if (!strcmp(item.buf, "fetchrecursesubmodules")) {
445                 /* when parsing worktree configurations we can die early */
446                 int die_on_error = is_null_oid(me->gitmodules_oid);
447                 if (!me->overwrite &&
448                     submodule->fetch_recurse != RECURSE_SUBMODULES_NONE)
449                         warn_multiple_config(me->treeish_name, submodule->name,
450                                         "fetchrecursesubmodules");
451                 else
452                         submodule->fetch_recurse = parse_fetch_recurse(
453                                                                 var, value,
454                                                                 die_on_error);
455         } else if (!strcmp(item.buf, "ignore")) {
456                 if (!value)
457                         ret = config_error_nonbool(var);
458                 else if (!me->overwrite && submodule->ignore)
459                         warn_multiple_config(me->treeish_name, submodule->name,
460                                         "ignore");
461                 else if (strcmp(value, "untracked") &&
462                          strcmp(value, "dirty") &&
463                          strcmp(value, "all") &&
464                          strcmp(value, "none"))
465                         warning("Invalid parameter '%s' for config option "
466                                         "'submodule.%s.ignore'", value, name.buf);
467                 else {
468                         free((void *) submodule->ignore);
469                         submodule->ignore = xstrdup(value);
470                 }
471         } else if (!strcmp(item.buf, "url")) {
472                 if (!value) {
473                         ret = config_error_nonbool(var);
474                 } else if (looks_like_command_line_option(value)) {
475                         warn_command_line_option(var, value);
476                 } else if (!me->overwrite && submodule->url) {
477                         warn_multiple_config(me->treeish_name, submodule->name,
478                                         "url");
479                 } else {
480                         free((void *) submodule->url);
481                         submodule->url = xstrdup(value);
482                 }
483         } else if (!strcmp(item.buf, "update")) {
484                 if (!value)
485                         ret = config_error_nonbool(var);
486                 else if (!me->overwrite &&
487                          submodule->update_strategy.type != SM_UPDATE_UNSPECIFIED)
488                         warn_multiple_config(me->treeish_name, submodule->name,
489                                              "update");
490                 else if (parse_submodule_update_strategy(value,
491                          &submodule->update_strategy) < 0)
492                                 die(_("invalid value for %s"), var);
493         } else if (!strcmp(item.buf, "shallow")) {
494                 if (!me->overwrite && submodule->recommend_shallow != -1)
495                         warn_multiple_config(me->treeish_name, submodule->name,
496                                              "shallow");
497                 else
498                         submodule->recommend_shallow =
499                                 git_config_bool(var, value);
500         } else if (!strcmp(item.buf, "branch")) {
501                 if (!me->overwrite && submodule->branch)
502                         warn_multiple_config(me->treeish_name, submodule->name,
503                                              "branch");
504                 else {
505                         free((void *)submodule->branch);
506                         submodule->branch = xstrdup(value);
507                 }
508         }
509
510         strbuf_release(&name);
511         strbuf_release(&item);
512
513         return ret;
514 }
515
516 static int gitmodule_oid_from_commit(const struct object_id *treeish_name,
517                                      struct object_id *gitmodules_oid,
518                                      struct strbuf *rev)
519 {
520         int ret = 0;
521
522         if (is_null_oid(treeish_name)) {
523                 oidclr(gitmodules_oid);
524                 return 1;
525         }
526
527         strbuf_addf(rev, "%s:.gitmodules", oid_to_hex(treeish_name));
528         if (get_oid(rev->buf, gitmodules_oid) >= 0)
529                 ret = 1;
530
531         return ret;
532 }
533
534 /* This does a lookup of a submodule configuration by name or by path
535  * (key) with on-demand reading of the appropriate .gitmodules from
536  * revisions.
537  */
538 static const struct submodule *config_from(struct submodule_cache *cache,
539                 const struct object_id *treeish_name, const char *key,
540                 enum lookup_type lookup_type)
541 {
542         struct strbuf rev = STRBUF_INIT;
543         unsigned long config_size;
544         char *config = NULL;
545         struct object_id oid;
546         enum object_type type;
547         const struct submodule *submodule = NULL;
548         struct parse_config_parameter parameter;
549
550         /*
551          * If any parameter except the cache is a NULL pointer just
552          * return the first submodule. Can be used to check whether
553          * there are any submodules parsed.
554          */
555         if (!treeish_name || !key) {
556                 struct hashmap_iter iter;
557                 struct submodule_entry *entry;
558
559                 entry = hashmap_iter_first_entry(&cache->for_name, &iter,
560                                                 struct submodule_entry,
561                                                 ent /* member name */);
562                 if (!entry)
563                         return NULL;
564                 return entry->config;
565         }
566
567         if (!gitmodule_oid_from_commit(treeish_name, &oid, &rev))
568                 goto out;
569
570         switch (lookup_type) {
571         case lookup_name:
572                 submodule = cache_lookup_name(cache, &oid, key);
573                 break;
574         case lookup_path:
575                 submodule = cache_lookup_path(cache, &oid, key);
576                 break;
577         }
578         if (submodule)
579                 goto out;
580
581         config = read_object_file(&oid, &type, &config_size);
582         if (!config || type != OBJ_BLOB)
583                 goto out;
584
585         /* fill the submodule config into the cache */
586         parameter.cache = cache;
587         parameter.treeish_name = treeish_name;
588         parameter.gitmodules_oid = &oid;
589         parameter.overwrite = 0;
590         git_config_from_mem(parse_config, CONFIG_ORIGIN_SUBMODULE_BLOB, rev.buf,
591                         config, config_size, &parameter, NULL);
592         strbuf_release(&rev);
593         free(config);
594
595         switch (lookup_type) {
596         case lookup_name:
597                 return cache_lookup_name(cache, &oid, key);
598         case lookup_path:
599                 return cache_lookup_path(cache, &oid, key);
600         default:
601                 return NULL;
602         }
603
604 out:
605         strbuf_release(&rev);
606         free(config);
607         return submodule;
608 }
609
610 static void submodule_cache_check_init(struct repository *repo)
611 {
612         if (repo->submodule_cache && repo->submodule_cache->initialized)
613                 return;
614
615         if (!repo->submodule_cache)
616                 repo->submodule_cache = submodule_cache_alloc();
617
618         submodule_cache_init(repo->submodule_cache);
619 }
620
621 /*
622  * Note: This function is private for a reason, the '.gitmodules' file should
623  * not be used as as a mechanism to retrieve arbitrary configuration stored in
624  * the repository.
625  *
626  * Runs the provided config function on the '.gitmodules' file found in the
627  * working directory.
628  */
629 static void config_from_gitmodules(config_fn_t fn, struct repository *repo, void *data)
630 {
631         if (repo->worktree) {
632                 struct git_config_source config_source = { 0 };
633                 const struct config_options opts = { 0 };
634                 struct object_id oid;
635                 char *file;
636                 char *oidstr = NULL;
637
638                 file = repo_worktree_path(repo, GITMODULES_FILE);
639                 if (file_exists(file)) {
640                         config_source.file = file;
641                 } else if (repo_get_oid(repo, GITMODULES_INDEX, &oid) >= 0 ||
642                            repo_get_oid(repo, GITMODULES_HEAD, &oid) >= 0) {
643                         config_source.blob = oidstr = xstrdup(oid_to_hex(&oid));
644                         if (repo != the_repository)
645                                 add_to_alternates_memory(repo->objects->odb->path);
646                 } else {
647                         goto out;
648                 }
649
650                 config_with_options(fn, data, &config_source, &opts);
651
652 out:
653                 free(oidstr);
654                 free(file);
655         }
656 }
657
658 static int gitmodules_cb(const char *var, const char *value, void *data)
659 {
660         struct repository *repo = data;
661         struct parse_config_parameter parameter;
662
663         parameter.cache = repo->submodule_cache;
664         parameter.treeish_name = NULL;
665         parameter.gitmodules_oid = &null_oid;
666         parameter.overwrite = 1;
667
668         return parse_config(var, value, &parameter);
669 }
670
671 void repo_read_gitmodules(struct repository *repo)
672 {
673         submodule_cache_check_init(repo);
674
675         if (repo_read_index(repo) < 0)
676                 return;
677
678         if (!is_gitmodules_unmerged(repo->index))
679                 config_from_gitmodules(gitmodules_cb, repo, repo);
680
681         repo->submodule_cache->gitmodules_read = 1;
682 }
683
684 void gitmodules_config_oid(const struct object_id *commit_oid)
685 {
686         struct strbuf rev = STRBUF_INIT;
687         struct object_id oid;
688
689         submodule_cache_check_init(the_repository);
690
691         if (gitmodule_oid_from_commit(commit_oid, &oid, &rev)) {
692                 git_config_from_blob_oid(gitmodules_cb, rev.buf,
693                                          &oid, the_repository);
694         }
695         strbuf_release(&rev);
696
697         the_repository->submodule_cache->gitmodules_read = 1;
698 }
699
700 static void gitmodules_read_check(struct repository *repo)
701 {
702         submodule_cache_check_init(repo);
703
704         /* read the repo's .gitmodules file if it hasn't been already */
705         if (!repo->submodule_cache->gitmodules_read)
706                 repo_read_gitmodules(repo);
707 }
708
709 const struct submodule *submodule_from_name(struct repository *r,
710                                             const struct object_id *treeish_name,
711                 const char *name)
712 {
713         gitmodules_read_check(r);
714         return config_from(r->submodule_cache, treeish_name, name, lookup_name);
715 }
716
717 const struct submodule *submodule_from_path(struct repository *r,
718                                             const struct object_id *treeish_name,
719                 const char *path)
720 {
721         gitmodules_read_check(r);
722         return config_from(r->submodule_cache, treeish_name, path, lookup_path);
723 }
724
725 void submodule_free(struct repository *r)
726 {
727         if (r->submodule_cache)
728                 submodule_cache_clear(r->submodule_cache);
729 }
730
731 static int config_print_callback(const char *var, const char *value, void *cb_data)
732 {
733         char *wanted_key = cb_data;
734
735         if (!strcmp(wanted_key, var))
736                 printf("%s\n", value);
737
738         return 0;
739 }
740
741 int print_config_from_gitmodules(struct repository *repo, const char *key)
742 {
743         int ret;
744         char *store_key;
745
746         ret = git_config_parse_key(key, &store_key, NULL);
747         if (ret < 0)
748                 return CONFIG_INVALID_KEY;
749
750         config_from_gitmodules(config_print_callback, repo, store_key);
751
752         free(store_key);
753         return 0;
754 }
755
756 int config_set_in_gitmodules_file_gently(const char *key, const char *value)
757 {
758         int ret;
759
760         ret = git_config_set_in_file_gently(GITMODULES_FILE, key, value);
761         if (ret < 0)
762                 /* Maybe the user already did that, don't error out here */
763                 warning(_("Could not update .gitmodules entry %s"), key);
764
765         return ret;
766 }
767
768 struct fetch_config {
769         int *max_children;
770         int *recurse_submodules;
771 };
772
773 static int gitmodules_fetch_config(const char *var, const char *value, void *cb)
774 {
775         struct fetch_config *config = cb;
776         if (!strcmp(var, "submodule.fetchjobs")) {
777                 *(config->max_children) = parse_submodule_fetchjobs(var, value);
778                 return 0;
779         } else if (!strcmp(var, "fetch.recursesubmodules")) {
780                 *(config->recurse_submodules) = parse_fetch_recurse_submodules_arg(var, value);
781                 return 0;
782         }
783
784         return 0;
785 }
786
787 void fetch_config_from_gitmodules(int *max_children, int *recurse_submodules)
788 {
789         struct fetch_config config = {
790                 .max_children = max_children,
791                 .recurse_submodules = recurse_submodules
792         };
793         config_from_gitmodules(gitmodules_fetch_config, the_repository, &config);
794 }
795
796 static int gitmodules_update_clone_config(const char *var, const char *value,
797                                           void *cb)
798 {
799         int *max_jobs = cb;
800         if (!strcmp(var, "submodule.fetchjobs"))
801                 *max_jobs = parse_submodule_fetchjobs(var, value);
802         return 0;
803 }
804
805 void update_clone_config_from_gitmodules(int *max_jobs)
806 {
807         config_from_gitmodules(gitmodules_update_clone_config, the_repository, &max_jobs);
808 }