From mboxrd@z Thu Jan 1 00:00:00 1970 From: =?UTF-8?q?Nguy=E1=BB=85n=20Th=C3=A1i=20Ng=E1=BB=8Dc=20Duy?= Subject: [PATCH v5 19/28] setup.c: support multi-checkout repo setup Date: Sat, 8 Mar 2014 09:48:11 +0700 Message-ID: <1394246900-31535-20-git-send-email-pclouds@gmail.com> References: <1393675983-3232-1-git-send-email-pclouds@gmail.com> <1394246900-31535-1-git-send-email-pclouds@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: QUOTED-PRINTABLE Cc: Junio C Hamano , =?UTF-8?q?Nguy=E1=BB=85n=20Th=C3=A1i=20Ng=E1=BB=8Dc=20Duy?= To: git@vger.kernel.org X-From: git-owner@vger.kernel.org Sat Mar 08 03:49:56 2014 Return-path: Envelope-to: gcvg-git-2@plane.gmane.org Received: from vger.kernel.org ([209.132.180.67]) by plane.gmane.org with esmtp (Exim 4.69) (envelope-from ) id 1WM7Kl-0005nF-9x for gcvg-git-2@plane.gmane.org; Sat, 08 Mar 2014 03:49:55 +0100 Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753516AbaCHCtu convert rfc822-to-quoted-printable (ORCPT ); Fri, 7 Mar 2014 21:49:50 -0500 Received: from mail-pd0-f172.google.com ([209.85.192.172]:43805 "EHLO mail-pd0-f172.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752305AbaCHCtt (ORCPT ); Fri, 7 Mar 2014 21:49:49 -0500 Received: by mail-pd0-f172.google.com with SMTP id p10so4820906pdj.3 for ; Fri, 07 Mar 2014 18:49:49 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=from:to:cc:subject:date:message-id:in-reply-to:references :mime-version:content-type:content-transfer-encoding; bh=4IcyJHkG0ks9shSfnpppA5gsgzg53eOU2KYdwhvwRbw=; b=jvjHxiHo47ungynjg0qtDJvOfOXdzn53Jsuv5O+b7PvRKTGsyPr03j/4qi2xLv01ZD At7Vp1K1seZZeXAa2tEi2LSzJXidbweXSDV39EnVlaYIRLUpeLukCJh78x0jkWlD5KBd FJV0L319XmzyrkkZmEblg/gjSRvBLItFdZUTicy5cV7sP+IUyIMXAkkWFToxQFlNv4u1 leio+7JsPY2JN14z7wLRFors8NcXE/SvS9vhP4ZCMp4xoqrktfLPj8FfeSMz5qQ+DjFO OJQOs++dZbWzfMGGUQbEqIqHxAZEct1o9IFEHvooaGE59V4s9lYLrzeRhAh/tqm/quMh WgkA== X-Received: by 10.66.146.170 with SMTP id td10mr26017788pab.105.1394246989195; Fri, 07 Mar 2014 18:49:49 -0800 (PST) Received: from lanh ([115.73.205.153]) by mx.google.com with ESMTPSA id ha11sm41862398pbd.17.2014.03.07.18.49.46 for (version=TLSv1 cipher=ECDHE-RSA-RC4-SHA bits=128/128); Fri, 07 Mar 2014 18:49:48 -0800 (PST) Received: by lanh (sSMTP sendmail emulation); Sat, 08 Mar 2014 09:50:22 +0700 X-Mailer: git-send-email 1.9.0.40.gaa8c3ea In-Reply-To: <1394246900-31535-1-git-send-email-pclouds@gmail.com> Sender: git-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: git@vger.kernel.org Archived-At: The repo setup procedure is updated to detect $GIT_DIR/commondir and set $GIT_COMMON_DIR properly. The core.worktree is ignored when $GIT_COMMON_DIR is set. This is because the config file is shared in multi-checkout setup, but checkout directories _are_ different. Making core.worktree effective in all checkouts mean it's back to a single checkout. Signed-off-by: Nguy=E1=BB=85n Th=C3=A1i Ng=E1=BB=8Dc Duy --- Documentation/config.txt | 2 ++ Documentation/git-rev-parse.txt | 3 ++ builtin/rev-parse.c | 4 +++ cache.h | 1 + environment.c | 8 ++--- setup.c | 33 +++++++++++++----- t/t1501-worktree.sh | 76 +++++++++++++++++++++++++++++++++= ++++++++ t/t1510-repo-setup.sh | 1 + trace.c | 1 + 9 files changed, 115 insertions(+), 14 deletions(-) diff --git a/Documentation/config.txt b/Documentation/config.txt index 5f4d793..313d4b3 100644 --- a/Documentation/config.txt +++ b/Documentation/config.txt @@ -381,6 +381,8 @@ false), while all other repositories are assumed to= be bare (bare =20 core.worktree:: Set the path to the root of the working tree. + If GIT_COMMON_DIR environment variable is set, core.worktree + is ignored and not used for determining the root of working tree. This can be overridden by the GIT_WORK_TREE environment variable and the '--work-tree' command line option. The value can be an absolute path or relative to the path to diff --git a/Documentation/git-rev-parse.txt b/Documentation/git-rev-pa= rse.txt index 46020d9..8a9cac8 100644 --- a/Documentation/git-rev-parse.txt +++ b/Documentation/git-rev-parse.txt @@ -215,6 +215,9 @@ If `$GIT_DIR` is not defined and the current direct= ory is not detected to lie in a Git repository or work tree print a message to stderr and exit with nonzero status. =20 +--git-common-dir:: + Show `$GIT_COMMON_DIR` if defined, else `$GIT_DIR`. + --is-inside-git-dir:: When the current working directory is below the repository directory print "true", otherwise "false". diff --git a/builtin/rev-parse.c b/builtin/rev-parse.c index e50bc65..c7057ce 100644 --- a/builtin/rev-parse.c +++ b/builtin/rev-parse.c @@ -744,6 +744,10 @@ int cmd_rev_parse(int argc, const char **argv, con= st char *prefix) printf("%s%s.git\n", cwd, len && cwd[len-1] !=3D '/' ? "/" : ""); continue; } + if (!strcmp(arg, "--git-common-dir")) { + puts(get_git_common_dir()); + continue; + } if (!strcmp(arg, "--resolve-git-dir")) { const char *gitdir =3D resolve_gitdir(argv[i+1]); if (!gitdir) diff --git a/cache.h b/cache.h index 9182427..992e241 100644 --- a/cache.h +++ b/cache.h @@ -407,6 +407,7 @@ extern char *get_object_directory(void); extern char *get_index_file(void); extern char *get_graft_file(void); extern int set_git_dir(const char *path); +extern int get_common_dir(struct strbuf *sb, const char *gitdir); extern const char *get_git_namespace(void); extern const char *strip_namespace(const char *namespaced_ref); extern const char *get_git_work_tree(void); diff --git a/environment.c b/environment.c index c998120..0999fc1 100644 --- a/environment.c +++ b/environment.c @@ -126,6 +126,7 @@ static char *expand_namespace(const char *raw_names= pace) =20 static void setup_git_env(void) { + struct strbuf sb =3D STRBUF_INIT; const char *gitfile; const char *shallow_file; =20 @@ -134,12 +135,9 @@ static void setup_git_env(void) git_dir =3D DEFAULT_GIT_DIR_ENVIRONMENT; gitfile =3D read_gitfile(git_dir); git_dir =3D xstrdup(gitfile ? gitfile : git_dir); - git_common_dir =3D getenv(GIT_COMMON_DIR_ENVIRONMENT); - if (git_common_dir) { + if (get_common_dir(&sb, git_dir)) git_common_dir_env =3D 1; - git_common_dir =3D xstrdup(git_common_dir); - } else - git_common_dir =3D git_dir; + git_common_dir =3D strbuf_detach(&sb, NULL); git_object_dir =3D getenv(DB_ENVIRONMENT); if (!git_object_dir) { git_object_dir =3D xmalloc(strlen(git_common_dir) + 9); diff --git a/setup.c b/setup.c index 42849f3..40ce191 100644 --- a/setup.c +++ b/setup.c @@ -170,14 +170,15 @@ void verify_non_filename(const char *prefix, cons= t char *arg) "'git [...] -- [...]'", arg); } =20 -static void get_common_dir(struct strbuf *sb, const char *gitdir) +int get_common_dir(struct strbuf *sb, const char *gitdir) { struct strbuf data =3D STRBUF_INIT; struct strbuf path =3D STRBUF_INIT; const char *git_common_dir =3D getenv(GIT_COMMON_DIR_ENVIRONMENT); + int ret =3D 0; if (git_common_dir) { strbuf_addstr(sb, git_common_dir); - return; + return 1; } strbuf_addf(&path, "%s/commondir", gitdir); if (file_exists(path.buf)) { @@ -192,10 +193,12 @@ static void get_common_dir(struct strbuf *sb, con= st char *gitdir) strbuf_addf(&path, "%s/", gitdir); strbuf_addbuf(&path, &data); strbuf_addstr(sb, real_path(path.buf)); + ret =3D 1; } else strbuf_addstr(sb, gitdir); strbuf_release(&data); strbuf_release(&path); + return ret; } =20 /* @@ -286,13 +289,26 @@ void setup_work_tree(void) initialized =3D 1; } =20 +static int check_repo_format(const char *var, const char *value, void = *cb) +{ + if (strcmp(var, "core.repositoryformatversion") =3D=3D 0) + repository_format_version =3D git_config_int(var, value); + else if (strcmp(var, "core.sharedrepository") =3D=3D 0) + shared_repository =3D git_config_perm(var, value); + return 0; +} + static int check_repository_format_gently(const char *gitdir, int *non= git_ok) { struct strbuf sb =3D STRBUF_INIT; const char *repo_config; + config_fn_t fn; int ret =3D 0; =20 - get_common_dir(&sb, gitdir); + if (get_common_dir(&sb, gitdir)) + fn =3D check_repo_format; + else + fn =3D check_repository_format_version; strbuf_addstr(&sb, "/config"); repo_config =3D sb.buf; =20 @@ -305,7 +321,7 @@ static int check_repository_format_gently(const cha= r *gitdir, int *nongit_ok) * Use a gentler version of git_config() to check if this repo * is a good one. */ - git_config_early(check_repository_format_version, NULL, repo_config); + git_config_early(fn, NULL, repo_config); if (GIT_REPO_VERSION < repository_format_version) { if (!nongit_ok) die ("Expected git repo version <=3D %d, found %d", @@ -777,11 +793,10 @@ int git_config_perm(const char *var, const char *= value) =20 int check_repository_format_version(const char *var, const char *value= , void *cb) { - if (strcmp(var, "core.repositoryformatversion") =3D=3D 0) - repository_format_version =3D git_config_int(var, value); - else if (strcmp(var, "core.sharedrepository") =3D=3D 0) - shared_repository =3D git_config_perm(var, value); - else if (strcmp(var, "core.bare") =3D=3D 0) { + int ret =3D check_repo_format(var, value, cb); + if (ret) + return ret; + if (strcmp(var, "core.bare") =3D=3D 0) { is_bare_repository_cfg =3D git_config_bool(var, value); if (is_bare_repository_cfg =3D=3D 1) inside_work_tree =3D -1; diff --git a/t/t1501-worktree.sh b/t/t1501-worktree.sh index 8f36aa9..2ac4424 100755 --- a/t/t1501-worktree.sh +++ b/t/t1501-worktree.sh @@ -346,4 +346,80 @@ test_expect_success 'relative $GIT_WORK_TREE and g= it subprocesses' ' test_cmp expected actual ' =20 +test_expect_success 'Multi-worktree setup' ' + mkdir work && + mkdir -p repo.git/repos/foo && + cp repo.git/HEAD repo.git/index repo.git/repos/foo && + sane_unset GIT_DIR GIT_CONFIG GIT_WORK_TREE +' + +test_expect_success 'GIT_DIR set (1)' ' + echo "gitdir: repo.git/repos/foo" >gitfile && + echo ../.. >repo.git/repos/foo/commondir && + ( + cd work && + GIT_DIR=3D../gitfile git rev-parse --git-common-dir >actual && + echo "$TRASH_DIRECTORY/repo.git" >expect && + test_cmp expect actual + ) +' + +test_expect_success 'GIT_DIR set (2)' ' + echo "gitdir: repo.git/repos/foo" >gitfile && + echo "$TRASH_DIRECTORY/repo.git" >repo.git/repos/foo/commondir && + ( + cd work && + GIT_DIR=3D../gitfile git rev-parse --git-common-dir >actual && + echo "$TRASH_DIRECTORY/repo.git" >expect && + test_cmp expect actual + ) +' + +test_expect_success 'Auto discovery' ' + echo "gitdir: repo.git/repos/foo" >.git && + echo ../.. >repo.git/repos/foo/commondir && + ( + cd work && + git rev-parse --git-common-dir >actual && + echo "$TRASH_DIRECTORY/repo.git" >expect && + test_cmp expect actual && + echo haha >data1 && + git add data1 && + git ls-files --full-name :/ | grep data1 >actual && + echo work/data1 >expect && + test_cmp expect actual + ) +' + +test_expect_success '$GIT_DIR/common overrides core.worktree' ' + mkdir elsewhere && + git --git-dir=3Drepo.git config core.worktree "$TRASH_DIRECTORY/elsew= here" && + echo "gitdir: repo.git/repos/foo" >.git && + echo ../.. >repo.git/repos/foo/commondir && + ( + cd work && + git rev-parse --git-common-dir >actual && + echo "$TRASH_DIRECTORY/repo.git" >expect && + test_cmp expect actual && + echo haha >data2 && + git add data2 && + git ls-files --full-name :/ | grep data2 >actual && + echo work/data2 >expect && + test_cmp expect actual + ) +' + +test_expect_success '$GIT_WORK_TREE overrides $GIT_DIR/common' ' + echo "gitdir: repo.git/repos/foo" >.git && + echo ../.. >repo.git/repos/foo/commondir && + ( + cd work && + echo haha >data3 && + git --git-dir=3D../.git --work-tree=3D. add data3 && + git ls-files --full-name -- :/ | grep data3 >actual && + echo data3 >expect && + test_cmp expect actual + ) +' + test_done diff --git a/t/t1510-repo-setup.sh b/t/t1510-repo-setup.sh index cf2ee78..c211ef5 100755 --- a/t/t1510-repo-setup.sh +++ b/t/t1510-repo-setup.sh @@ -106,6 +106,7 @@ setup_env () { expect () { cat >"$1/expected" <<-EOF setup: git_dir: $2 + setup: git_common_dir: $2 setup: worktree: $3 setup: cwd: $4 setup: prefix: $5 diff --git a/trace.c b/trace.c index 08180a9..a594761 100644 --- a/trace.c +++ b/trace.c @@ -173,6 +173,7 @@ void trace_repo_setup(const char *prefix) prefix =3D "(null)"; =20 trace_printf_key(key, "setup: git_dir: %s\n", quote_crnl(get_git_dir(= ))); + trace_printf_key(key, "setup: git_common_dir: %s\n", quote_crnl(get_g= it_common_dir())); trace_printf_key(key, "setup: worktree: %s\n", quote_crnl(git_work_tr= ee)); trace_printf_key(key, "setup: cwd: %s\n", quote_crnl(cwd)); trace_printf_key(key, "setup: prefix: %s\n", quote_crnl(prefix)); --=20 1.9.0.40.gaa8c3ea