From: Stefan Beller <sbeller@google.com>
To: jrnieder@gmail.com
Cc: gitster@pobox.com, git@vger.kernel.org, Jens.Lehmann@web.de,
pclouds@gmail.com, Stefan Beller <sbeller@google.com>
Subject: [PATCH 05/15] submodule-config: check if submodule a submodule is in a group
Date: Tue, 26 Apr 2016 13:50:23 -0700 [thread overview]
Message-ID: <1461703833-10350-6-git-send-email-sbeller@google.com> (raw)
In-Reply-To: <1461703833-10350-1-git-send-email-sbeller@google.com>
In later patches we need to tell if a submodule is in a group,
which is defined by name, path or labels.
Signed-off-by: Stefan Beller <sbeller@google.com>
---
builtin/submodule--helper.c | 43 ++++++++++++++++++++++-
submodule-config.c | 50 +++++++++++++++++++++++++++
submodule-config.h | 3 ++
t/t7413-submodule--helper.sh | 81 ++++++++++++++++++++++++++++++++++++++++++++
4 files changed, 176 insertions(+), 1 deletion(-)
create mode 100755 t/t7413-submodule--helper.sh
diff --git a/builtin/submodule--helper.c b/builtin/submodule--helper.c
index b6d4f27..23d7224 100644
--- a/builtin/submodule--helper.c
+++ b/builtin/submodule--helper.c
@@ -814,6 +814,46 @@ static int update_clone(int argc, const char **argv, const char *prefix)
return 0;
}
+int in_group(int argc, const char **argv, const char *prefix)
+{
+ const struct string_list *list;
+ struct string_list actual_list = STRING_LIST_INIT_DUP;
+ const struct submodule *sub;
+ const char *group = NULL;
+
+ struct option default_group_options[] = {
+ OPT_STRING('g', "group", &group, N_("group"),
+ N_("group specifier for submodules")),
+ OPT_END()
+ };
+
+ const char *const git_submodule_helper_usage[] = {
+ N_("git submodule--helper in-group <path>"),
+ NULL
+ };
+
+ argc = parse_options(argc, argv, prefix, default_group_options,
+ git_submodule_helper_usage, 0);
+
+ /* Overlay the parsed .gitmodules file with .git/config */
+ gitmodules_config();
+ git_config(submodule_config, NULL);
+
+ if (argc != 1)
+ usage(git_submodule_helper_usage[0]);
+
+ sub = submodule_from_path(null_sha1, argv[0]);
+
+ if (!group)
+ list = git_config_get_value_multi("submodule.defaultGroup");
+ else {
+ string_list_split(&actual_list, group, ',', -1);
+ list = &actual_list;
+ }
+
+ return !submodule_in_group(list, sub);
+}
+
struct cmd_struct {
const char *cmd;
int (*fn)(int, const char **, const char *);
@@ -826,7 +866,8 @@ static struct cmd_struct commands[] = {
{"update-clone", update_clone},
{"resolve-relative-url", resolve_relative_url},
{"resolve-relative-url-test", resolve_relative_url_test},
- {"init", module_init}
+ {"init", module_init},
+ {"in-group", in_group}
};
int cmd_submodule__helper(int argc, const char **argv, const char *prefix)
diff --git a/submodule-config.c b/submodule-config.c
index 0cdb47e..ebed0f2 100644
--- a/submodule-config.c
+++ b/submodule-config.c
@@ -522,3 +522,53 @@ void submodule_free(void)
cache_free(&cache);
is_cache_init = 0;
}
+
+int submodule_in_group(const struct string_list *group,
+ const struct submodule *sub)
+{
+ int matched = 0;
+ struct strbuf sb = STRBUF_INIT;
+
+ if (!group)
+ /*
+ * If no group is specified all, all submodules match to
+ * keep traditional behavior
+ */
+ return 1;
+
+ if (sub->labels) {
+ struct string_list_item *item;
+ for_each_string_list_item(item, sub->labels) {
+ strbuf_reset(&sb);
+ strbuf_addf(&sb, "*%s", item->string);
+ if (string_list_has_string(group, sb.buf)) {
+ matched = 1;
+ break;
+ }
+ }
+ }
+ if (sub->path) {
+ /*
+ * NEEDSWORK: This currently works only for
+ * exact paths, but we want to enable
+ * inexact matches such wildcards.
+ */
+ strbuf_reset(&sb);
+ strbuf_addf(&sb, "./%s", sub->path);
+ if (string_list_has_string(group, sb.buf))
+ matched = 1;
+ }
+ if (sub->name) {
+ /*
+ * NEEDSWORK: Same as with path. Do we want to
+ * support wildcards or such?
+ */
+ strbuf_reset(&sb);
+ strbuf_addf(&sb, ":%s", sub->name);
+ if (string_list_has_string(group, sb.buf))
+ matched = 1;
+ }
+ strbuf_release(&sb);
+
+ return matched;
+}
diff --git a/submodule-config.h b/submodule-config.h
index d57da59..4c696cc 100644
--- a/submodule-config.h
+++ b/submodule-config.h
@@ -31,4 +31,7 @@ const struct submodule *submodule_from_path(const unsigned char *commit_sha1,
const char *path);
void submodule_free(void);
+int submodule_in_group(const struct string_list *group,
+ const struct submodule *sub);
+
#endif /* SUBMODULE_CONFIG_H */
diff --git a/t/t7413-submodule--helper.sh b/t/t7413-submodule--helper.sh
new file mode 100755
index 0000000..c6939ab
--- /dev/null
+++ b/t/t7413-submodule--helper.sh
@@ -0,0 +1,81 @@
+#!/bin/sh
+
+# This should be merged with t7412 eventually.
+# (currently in flight as jk/submodule-c-credential)
+
+
+test_description='Basic plumbing support of submodule--helper
+
+This test verifies the submodule--helper plumbing command used to implement
+git-submodule.
+'
+
+. ./test-lib.sh
+
+test_expect_success 'setup superproject with submodules' '
+
+ mkdir sub &&
+ (
+ cd sub &&
+ git init &&
+ test_commit test
+ test_commit test2
+ ) &&
+ mkdir super &&
+ (
+ cd super &&
+ git init &&
+ git submodule add ../sub sub0 &&
+ git submodule add -l bit1 ../sub sub1 &&
+ git submodule add -l bit2 ../sub sub2 &&
+ git submodule add -l bit2 -l bit1 ../sub sub3 &&
+ git commit -m "add labeled submodules"
+ )
+'
+
+test_expect_success 'in-group' '
+ (
+ cd super &&
+ # we do not specify a group nor have set a default group,
+ # any submodule should be in the default group:
+ git submodule--helper in-group sub0 &&
+ git submodule--helper in-group sub1 &&
+ git submodule--helper in-group sub2 &&
+ git submodule--helper in-group sub3 &&
+
+ # test bit1:
+ test_must_fail git submodule--helper in-group --group=*bit1 sub0 &&
+ git submodule--helper in-group --group=*bit1 sub1 &&
+ test_must_fail git submodule--helper in-group --group=*bit1 sub2 &&
+ git submodule--helper in-group --group=*bit1 sub3 &&
+
+ # test by path:
+ git submodule--helper in-group --group=./sub0 sub0 &&
+ test_must_fail git submodule--helper in-group --group=./sub0 sub1 &&
+ test_must_fail git submodule--helper in-group --group=./sub0 sub2 &&
+ test_must_fail git submodule--helper in-group --group=./sub0 sub3 &&
+
+ # test by name:
+ git submodule--helper in-group --group=:sub0 sub0 &&
+ test_must_fail git submodule--helper in-group --group=:sub0 sub1 &&
+ test_must_fail git submodule--helper in-group --group=:sub0 sub2 &&
+ test_must_fail git submodule--helper in-group --group=:sub0 sub3 &&
+
+ # logical OR of path and labels
+ git submodule--helper in-group --group=*bit1,./sub0 sub0 &&
+ git submodule--helper in-group --group=*bit1,./sub0 sub1 &&
+ test_must_fail git submodule--helper in-group --group=*bit1,./sub0 sub2 &&
+ git submodule--helper in-group --group=*bit1,./sub0 sub3 &&
+
+ # test if the config option is picked up
+ git config --add submodule.defaultGroup *bit1 &&
+ git config --add submodule.defaultGroup ./sub0 &&
+
+ git submodule--helper in-group sub0 &&
+ git submodule--helper in-group sub1 &&
+ test_must_fail git submodule--helper in-group sub2 &&
+ git submodule--helper in-group sub3
+ )
+'
+
+test_done
--
2.8.0.41.g8d9aeb3
next prev parent reply other threads:[~2016-04-26 20:51 UTC|newest]
Thread overview: 48+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-04-26 20:50 [PATCH 00/15] submodule groups (once again) Stefan Beller
2016-04-26 20:50 ` [PATCH 01/15] string_list: add string_list_duplicate Stefan Beller
2016-04-26 22:37 ` Junio C Hamano
2016-04-27 18:02 ` Stefan Beller
2016-04-27 21:11 ` Junio C Hamano
2016-04-27 21:17 ` Stefan Beller
2016-04-27 23:17 ` Junio C Hamano
2016-04-27 23:24 ` Stefan Beller
2016-04-26 20:50 ` [PATCH 02/15] submodule doc: write down what we want to achieve in this series Stefan Beller
2016-04-26 22:42 ` Junio C Hamano
2016-04-26 20:50 ` [PATCH 03/15] submodule add: label submodules if asked to Stefan Beller
2016-04-26 22:49 ` Junio C Hamano
2016-04-26 22:50 ` Jacob Keller
2016-04-26 23:19 ` Stefan Beller
2016-04-27 3:24 ` Jacob Keller
2016-04-26 20:50 ` [PATCH 04/15] submodule-config: keep labels around Stefan Beller
2016-04-26 20:50 ` Stefan Beller [this message]
2016-04-26 22:58 ` [PATCH 05/15] submodule-config: check if submodule a submodule is in a group Junio C Hamano
2016-04-26 23:17 ` Junio C Hamano
2016-04-27 23:00 ` Stefan Beller
2016-04-26 20:50 ` [PATCH 06/15] submodule init: redirect stdout to stderr Stefan Beller
2016-04-29 18:27 ` Junio C Hamano
2016-04-29 18:38 ` Stefan Beller
2016-04-26 20:50 ` [PATCH 07/15] submodule deinit: loose requirement for giving '.' Stefan Beller
2016-04-29 18:27 ` Junio C Hamano
2016-04-26 20:50 ` [PATCH 08/15] submodule--helper list: respect submodule groups Stefan Beller
2016-04-29 18:31 ` Junio C Hamano
2016-04-26 20:50 ` [PATCH 09/15] submodule--helper init: " Stefan Beller
2016-04-26 20:50 ` [PATCH 10/15] submodule--helper update_clone: " Stefan Beller
2016-04-26 20:50 ` [PATCH 11/15] diff: ignore submodules excluded by groups Stefan Beller
2016-04-29 18:37 ` Junio C Hamano
2016-04-29 19:17 ` Stefan Beller
2016-05-05 19:57 ` Stefan Beller
2016-05-05 20:19 ` Junio C Hamano
2016-05-05 21:02 ` Stefan Beller
2016-05-05 22:20 ` Junio C Hamano
2016-05-05 22:50 ` Stefan Beller
2016-05-05 23:07 ` Junio C Hamano
2016-05-06 6:08 ` Junio C Hamano
2016-05-06 18:23 ` Stefan Beller
2016-05-06 18:59 ` Junio C Hamano
2016-04-26 20:50 ` [PATCH 12/15] git submodule summary respects groups Stefan Beller
2016-04-29 18:38 ` Junio C Hamano
2016-04-26 20:50 ` [PATCH 13/15] cmd_status: respect submodule groups Stefan Beller
2016-04-26 20:50 ` [PATCH 14/15] cmd_diff: " Stefan Beller
2016-04-26 20:50 ` [PATCH 15/15] clone: allow specification of submodules to be cloned Stefan Beller
2016-04-26 22:19 ` [PATCH 00/15] submodule groups (once again) Junio C Hamano
2016-04-26 22:26 ` Junio C Hamano
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
List information: http://vger.kernel.org/majordomo-info.html
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1461703833-10350-6-git-send-email-sbeller@google.com \
--to=sbeller@google.com \
--cc=Jens.Lehmann@web.de \
--cc=git@vger.kernel.org \
--cc=gitster@pobox.com \
--cc=jrnieder@gmail.com \
--cc=pclouds@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this public inbox
https://80x24.org/mirrors/git.git
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).