# [PATCH v4 1/3] t/lib-httpd: load mod_unixd 8 messages from 2016-02-26 to 2016-02-27. Participants: Jacob Keller, Stefan Beller, Jeff King. Thread: https://gitlist.dev/t/41538 ## Jacob Keller, 2016-02-26 19:18 Subject: [PATCH v4 1/3] t/lib-httpd: load mod_unixd Message-ID: <1456514328-10153-1-git-send-email-jacob.e.keller@intel.com> URL: https://gitlist.dev/e/1456514328-10153-1-git-send-email-jacob.e.keller%40intel.com ``` From: Michael J Gruber In contrast to apache 2.2, apache 2.4 does not load mod_unixd in its default configuration (because there are choices). Thus, with the current config, apache 2.4.10 will not be started and the httpd tests will not run on distros with default apache config (RedHat type). Enable mod_unixd to make the httpd tests run. This does not affect distros negatively which have that config already in their default (Debian type). httpd tests will run on these before and after this patch. Signed-off-by: Michael J Gruber Signed-off-by: Junio C Hamano --- I am sending this version strictly from PU since I need it to pass the httpd tests. t/lib-httpd/apache.conf | 3 +++ 1 file changed, 3 insertions(+) diff --git a/t/lib-httpd/apache.conf b/t/lib-httpd/apache.conf index 7d15e6d44c83..f667e7ce2f33 100644 --- a/t/lib-httpd/apache.conf +++ b/t/lib-httpd/apache.conf @@ -64,6 +64,9 @@ LockFile accept.lock LoadModule mpm_prefork_module modules/mod_mpm_prefork.so + + LoadModule unixd_module modules/mod_unixd.so + PassEnv GIT_VALGRIND -- 2.7.1.429.g45cd78e ``` ## Jacob Keller, 2016-02-26 19:18 Subject: [PATCH v4 2/3] sumodule--helper: fix submodule--helper clone usage and check argc count Message-ID: <1456514328-10153-2-git-send-email-jacob.e.keller@intel.com> URL: https://gitlist.dev/e/1456514328-10153-2-git-send-email-jacob.e.keller%40intel.com In-Reply-To: <1456514328-10153-1-git-send-email-jacob.e.keller@intel.com> ``` From: Jacob Keller git submodule--helper clone usage specified that paths come after the -- as a sequence. However, the actual implementation does not, and requires only a single path passed in via --path. In addition, argc was unchecked. (allowing arbitrary extra arguments that were silently ignored). Fix the usage description to match implementation. Add an argc check to enforce no extra arguments. Fix a bug in the argument passing in git-submodule.sh which would pass --reference and --depth as empty strings when they were unused, resulting in extra argc after parsing options. Signed-off-by: Jacob Keller --- builtin/submodule--helper.c | 5 ++++- git-submodule.sh | 4 ++-- 2 files changed, 6 insertions(+), 3 deletions(-) diff --git a/builtin/submodule--helper.c b/builtin/submodule--helper.c index f4c3eff179b5..072d9bbd12a8 100644 --- a/builtin/submodule--helper.c +++ b/builtin/submodule--helper.c @@ -187,13 +187,16 @@ static int module_clone(int argc, const char **argv, const char *prefix) const char *const git_submodule_helper_usage[] = { N_("git submodule--helper clone [--prefix=] [--quiet] " "[--reference ] [--name ] [--url ]" - "[--depth ] [--] [...]"), + "[--depth ] [--path ]"), NULL }; argc = parse_options(argc, argv, prefix, module_clone_options, git_submodule_helper_usage, 0); + if (argc) + usage(*git_submodule_helper_usage); + strbuf_addf(&sb, "%s/modules/%s", get_git_dir(), name); sm_gitdir = strbuf_detach(&sb, NULL); diff --git a/git-submodule.sh b/git-submodule.sh index 9bc5c5f94d1d..2dd29b3df0e6 100755 --- a/git-submodule.sh +++ b/git-submodule.sh @@ -347,7 +347,7 @@ Use -f if you really want to add it." >&2 echo "$(eval_gettext "Reactivating local git directory for submodule '\$sm_name'.")" fi fi - git submodule--helper clone ${GIT_QUIET:+--quiet} --prefix "$wt_prefix" --path "$sm_path" --name "$sm_name" --url "$realrepo" "$reference" "$depth" || exit + git submodule--helper clone ${GIT_QUIET:+--quiet} --prefix "$wt_prefix" --path "$sm_path" --name "$sm_name" --url "$realrepo" ${reference:+"$reference"} ${depth:+"$depth"} || exit ( clear_local_git_env cd "$sm_path" && @@ -709,7 +709,7 @@ Maybe you want to use 'update --init'?")" if ! test -d "$sm_path"/.git && ! test -f "$sm_path"/.git then - git submodule--helper clone ${GIT_QUIET:+--quiet} --prefix "$prefix" --path "$sm_path" --name "$name" --url "$url" "$reference" "$depth" || exit + git submodule--helper clone ${GIT_QUIET:+--quiet} --prefix "$prefix" --path "$sm_path" --name "$name" --url "$url" ${reference:+"$reference"} ${depth:+"$depth"} || exit cloned_modules="$cloned_modules;$name" subsha1= else -- 2.7.1.429.g45cd78e ``` ## Jacob Keller, 2016-02-26 19:18 Subject: [PATCH v4 3/3] git: submodule honor -c credential.* from command line Message-ID: <1456514328-10153-3-git-send-email-jacob.e.keller@intel.com> URL: https://gitlist.dev/e/1456514328-10153-3-git-send-email-jacob.e.keller%40intel.com In-Reply-To: <1456514328-10153-1-git-send-email-jacob.e.keller@intel.com> ``` From: Jacob Keller Due to the way that the git-submodule code works, it clears all local git environment variables before entering submodules. This is normally a good thing since we want to clear settings such as GIT_WORKTREE and other variables which would affect the operation of submodule commands. However, GIT_CONFIG_PARAMETERS is special, and we actually do want to preserve these settings. However, we do not want to preserve all configuration as many things should be left specific to the parent project. Add a git submodule--helper function, sanitize-config, which shall be used to sanitize GIT_CONFIG_PARAMETERS, removing all key/value pairs except a small subset that are known to be safe and necessary. Replace all the calls to clear_local_git_env with a wrapped function that filters GIT_CONFIG_PARAMETERS using the new helper and then restores it to the filtered subset after clearing the rest of the environment. Signed-off-by: Jacob Keller --- Notes: - v2 * Clarify which paramaters are left after the sanitization, and don't seem to indicate it is our goal to extend the list. * add a comment in the submodule_config_ok function indicating the same - v3 * Remove extraneous comments * add strbuf_release calls * rename sanitize_local_git_env * remove use of local * add C equivalent of sanitize_config * add a test for the credential passing - v4 * use argc check instead of empty options check * fix brain-melting quotes in t7412-submodule--helper.sh builtin/submodule--helper.c | 77 +++++++++++++++++++++++++++++++++++++++++++- git-submodule.sh | 36 +++++++++++++-------- t/t5550-http-fetch-dumb.sh | 17 ++++++++++ t/t7412-submodule--helper.sh | 27 ++++++++++++++++ 4 files changed, 143 insertions(+), 14 deletions(-) create mode 100755 t/t7412-submodule--helper.sh diff --git a/builtin/submodule--helper.c b/builtin/submodule--helper.c index 072d9bbd12a8..c6c0a3b06e27 100644 --- a/builtin/submodule--helper.c +++ b/builtin/submodule--helper.c @@ -124,6 +124,64 @@ static int module_name(int argc, const char **argv, const char *prefix) return 0; } + +/* + * Rules to sanitize configuration variables that are Ok to be passed into + * submodule operations from the parent project using "-c". Should only + * include keys which are both (a) safe and (b) necessary for proper + * operation. + */ +static int submodule_config_ok(const char *var) +{ + if (starts_with(var, "credential.")) + return 1; + return 0; +} + +static int sanitize_submodule_config(const char *var, const char *value, void *data) +{ + struct strbuf quoted = STRBUF_INIT; + struct strbuf *out = data; + + if (submodule_config_ok(var)) { + if (out->len) + strbuf_addch(out, ' '); + + /* + * The GIT_CONFIG_PARAMETERS parser is a bit picky and + * requires that each var=value pair be quoted as a single + * unit. + */ + strbuf_addstr("ed, var); + strbuf_addch("ed, '='); + strbuf_addstr("ed, value); + + sq_quote_buf(out, quoted.buf); + } + + strbuf_release("ed); + + return 0; +} + +static void prepare_submodule_repo_env(struct argv_array *out) +{ + const char * const *var; + + for (var = local_repo_env; *var; var++) { + if (!strcmp(*var, CONFIG_DATA_ENVIRONMENT)) { + struct strbuf sanitized_config = STRBUF_INIT; + git_config_from_parameters(sanitize_submodule_config, + &sanitized_config); + argv_array_pushf(out, "%s=%s", *var, sanitized_config.buf); + strbuf_release(&sanitized_config); + } else { + argv_array_push(out, *var); + } + } + +} + static int clone_submodule(const char *path, const char *gitdir, const char *url, const char *depth, const char *reference, int quiet) { @@ -145,7 +203,7 @@ static int clone_submodule(const char *path, const char *gitdir, const char *url argv_array_push(&cp.args, path); cp.git_cmd = 1; - cp.env = local_repo_env; + prepare_submodule_repo_env(&cp.env_array); cp.no_stdin = 1; return run_command(&cp); @@ -258,6 +316,22 @@ static int module_clone(int argc, const char **argv, const char *prefix) return 0; } +static int module_sanitize_config(int argc, const char **argv, const char *prefix) +{ + struct strbuf sanitized_config = STRBUF_INIT; + + if (argc > 1) + usage(_("git submodule--helper sanitize-config")); + + git_config_from_parameters(sanitize_submodule_config, &sanitized_config); + if (sanitized_config.len) + printf("%s\n", sanitized_config.buf); + + strbuf_release(&sanitized_config); + + return 0; +} + struct cmd_struct { const char *cmd; int (*fn)(int, const char **, const char *); @@ -267,6 +341,7 @@ static struct cmd_struct commands[] = { {"list", module_list}, {"name", module_name}, {"clone", module_clone}, + {"sanitize-config", module_sanitize_config}, }; int cmd_submodule__helper(int argc, const char **argv, const char *prefix) diff --git a/git-submodule.sh b/git-submodule.sh index 2dd29b3df0e6..1f132b489b81 100755 --- a/git-submodule.sh +++ b/git-submodule.sh @@ -192,6 +192,16 @@ isnumber() n=$(($1 + 0)) 2>/dev/null && test "$n" = "$1" } +# Sanitize the local git environment for use within a submodule. We +# can't simply use clear_local_git_env since we want to preserve some +# of the settings from GIT_CONFIG_PARAMETERS. +sanitize_submodule_env() +{ + sanitized_config=$(git submodule--helper sanitize-config) + clear_local_git_env + GIT_CONFIG_PARAMETERS=$sanitized_config +} + # # Add a new submodule to the working tree, .gitmodules and the index # @@ -349,7 +359,7 @@ Use -f if you really want to add it." >&2 fi git submodule--helper clone ${GIT_QUIET:+--quiet} --prefix "$wt_prefix" --path "$sm_path" --name "$sm_name" --url "$realrepo" ${reference:+"$reference"} ${depth:+"$depth"} || exit ( - clear_local_git_env + sanitize_submodule_env cd "$sm_path" && # ash fails to wordsplit ${branch:+-b "$branch"...} case "$branch" in @@ -418,7 +428,7 @@ cmd_foreach() name=$(git submodule--helper name "$sm_path") ( prefix="$prefix$sm_path/" - clear_local_git_env + sanitize_submodule_env cd "$sm_path" && sm_path=$(relative_path "$sm_path") && # we make $path available to scripts ... @@ -713,7 +723,7 @@ Maybe you want to use 'update --init'?")" cloned_modules="$cloned_modules;$name" subsha1= else - subsha1=$(clear_local_git_env; cd "$sm_path" && + subsha1=$(sanitize_submodule_env; cd "$sm_path" && git rev-parse --verify HEAD) || die "$(eval_gettext "Unable to find current revision in submodule path '\$displaypath'")" fi @@ -723,11 +733,11 @@ Maybe you want to use 'update --init'?")" if test -z "$nofetch" then # Fetch remote before determining tracking $sha1 - (clear_local_git_env; cd "$sm_path" && git-fetch) || + (sanitize_submodule_env; cd "$sm_path" && git-fetch) || die "$(eval_gettext "Unable to fetch in submodule path '\$sm_path'")" fi - remote_name=$(clear_local_git_env; cd "$sm_path" && get_default_remote) - sha1=$(clear_local_git_env; cd "$sm_path" && + remote_name=$(sanitize_submodule_env; cd "$sm_path" && get_default_remote) + sha1=$(sanitize_submodule_env; cd "$sm_path" && git rev-parse --verify "${remote_name}/${branch}") || die "$(eval_gettext "Unable to find current ${remote_name}/${branch} revision in submodule path '\$sm_path'")" fi @@ -745,7 +755,7 @@ Maybe you want to use 'update --init'?")" then # Run fetch only if $sha1 isn't present or it # is not reachable from a ref. - (clear_local_git_env; cd "$sm_path" && + (sanitize_submodule_env; cd "$sm_path" && ( (rev=$(git rev-list -n 1 $sha1 --not --all 2>/dev/null) && test -z "$rev") || git-fetch)) || die "$(eval_gettext "Unable to fetch in submodule path '\$displaypath'")" @@ -787,7 +797,7 @@ Maybe you want to use 'update --init'?")" die "$(eval_gettext "Invalid update mode '$update_module' for submodule '$name'")" esac - if (clear_local_git_env; cd "$sm_path" && $command "$sha1") + if (sanitize_submodule_env; cd "$sm_path" && $command "$sha1") then say "$say_msg" elif test -n "$must_die_on_failure" @@ -803,7 +813,7 @@ Maybe you want to use 'update --init'?")" then ( prefix="$prefix$sm_path/" - clear_local_git_env + sanitize_submodule_env cd "$sm_path" && eval cmd_update ) @@ -841,7 +851,7 @@ Maybe you want to use 'update --init'?")" set_name_rev () { revname=$( ( - clear_local_git_env + sanitize_submodule_env cd "$1" && { git describe "$2" 2>/dev/null || git describe --tags "$2" 2>/dev/null || @@ -1125,7 +1135,7 @@ cmd_status() else if test -z "$cached" then - sha1=$(clear_local_git_env; cd "$sm_path" && git rev-parse --verify HEAD) + sha1=$(sanitize_submodule_env; cd "$sm_path" && git rev-parse --verify HEAD) fi set_name_rev "$sm_path" "$sha1" say "+$sha1 $displaypath$revname" @@ -1135,7 +1145,7 @@ cmd_status() then ( prefix="$displaypath/" - clear_local_git_env + sanitize_submodule_env cd "$sm_path" && eval cmd_status ) || @@ -1209,7 +1219,7 @@ cmd_sync() if test -e "$sm_path"/.git then ( - clear_local_git_env + sanitize_submodule_env cd "$sm_path" remote=$(get_default_remote) git config remote."$remote".url "$sub_origin_url" diff --git a/t/t5550-http-fetch-dumb.sh b/t/t5550-http-fetch-dumb.sh index 64146352ae20..48e2ab62da7a 100755 --- a/t/t5550-http-fetch-dumb.sh +++ b/t/t5550-http-fetch-dumb.sh @@ -91,6 +91,23 @@ test_expect_success 'configured username does not override URL' ' expect_askpass pass user@host ' +test_expect_success 'cmdline credential config passes into submodules' ' + git init super && + set_askpass user@host pass@host && + ( + cd super && + git submodule add "$HTTPD_URL/auth/dumb/repo.git" sub && + git commit -m "add submodule" + ) && + set_askpass wrong pass@host && + test_must_fail git clone --recursive super super-clone && + rm -rf super-clone && + set_askpass wrong pass@host && + git -c "credential.$HTTP_URL.username=user@host" \ + clone --recursive super super-clone && + expect_askpass pass user@host +' + test_expect_success 'fetch changes via http' ' echo content >>file && git commit -a -m two && diff --git a/t/t7412-submodule--helper.sh b/t/t7412-submodule--helper.sh new file mode 100755 index 000000000000..9d937de8b224 --- /dev/null +++ b/t/t7412-submodule--helper.sh @@ -0,0 +1,27 @@ +#!/bin/sh +# +# Copyright (c) 2016 Jacob Keller +# + +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 'sanitize-config clears configuration' ' + git -c user.name="Some User" submodule--helper sanitize-config >actual && + test_must_be_empty actual +' + +test_expect_success 'sanitize-config keeps credential.helper' " + git -c credential.helper=helper submodule--helper sanitize-config >actual && + cat >expect <<-EOF && + 'credential.helper=helper' + EOF + test_cmp expect actual +" + +test_done -- 2.7.1.429.g45cd78e ``` ## Stefan Beller, 2016-02-26 19:31 Subject: Re: [PATCH v4 2/3] sumodule--helper: fix submodule--helper clone usage and check argc count Message-ID: URL: https://gitlist.dev/e/CAGZ79kbD7_CYo9KT185oTxrjqch-otwfG0mWqAV2QiwrC-ch9Q%40mail.gmail.com In-Reply-To: <1456514328-10153-2-git-send-email-jacob.e.keller@intel.com> ``` On Fri, Feb 26, 2016 at 11:18 AM, Jacob Keller wrote: > From: Jacob Keller > > git submodule--helper clone usage specified that paths come after the -- > as a sequence. However, the actual implementation does not, and requires > only a single path passed in via --path. In addition, argc was > unchecked. (allowing arbitrary extra arguments that were silently > ignored). > > Fix the usage description to match implementation. Add an argc check to > enforce no extra arguments. Fix a bug in the argument passing in > git-submodule.sh which would pass --reference and --depth as empty > strings when they were unused, resulting in extra argc after parsing > options. > > Signed-off-by: Jacob Keller > --- > builtin/submodule--helper.c | 5 ++++- > git-submodule.sh | 4 ++-- > 2 files changed, 6 insertions(+), 3 deletions(-) > > diff --git a/builtin/submodule--helper.c b/builtin/submodule--helper.c > index f4c3eff179b5..072d9bbd12a8 100644 > --- a/builtin/submodule--helper.c > +++ b/builtin/submodule--helper.c > @@ -187,13 +187,16 @@ static int module_clone(int argc, const char **argv, const char *prefix) > const char *const git_submodule_helper_usage[] = { > N_("git submodule--helper clone [--prefix=] [--quiet] " > "[--reference ] [--name ] [--url ]" > - "[--depth ] [--] [...]"), > + "[--depth ] [--path ]"), Right, no extra arguments. > NULL > }; > > argc = parse_options(argc, argv, prefix, module_clone_options, > git_submodule_helper_usage, 0); > > + if (argc) > + usage(*git_submodule_helper_usage); > + This is the fix to check for more arguments. > strbuf_addf(&sb, "%s/modules/%s", get_git_dir(), name); > sm_gitdir = strbuf_detach(&sb, NULL); > > diff --git a/git-submodule.sh b/git-submodule.sh > index 9bc5c5f94d1d..2dd29b3df0e6 100755 > --- a/git-submodule.sh > +++ b/git-submodule.sh > @@ -347,7 +347,7 @@ Use -f if you really want to add it." >&2 > echo "$(eval_gettext "Reactivating local git directory for submodule '\$sm_name'.")" > fi > fi > - git submodule--helper clone ${GIT_QUIET:+--quiet} --prefix "$wt_prefix" --path "$sm_path" --name "$sm_name" --url "$realrepo" "$reference" "$depth" || exit > + git submodule--helper clone ${GIT_QUIET:+--quiet} --prefix "$wt_prefix" --path "$sm_path" --name "$sm_name" --url "$realrepo" ${reference:+"$reference"} ${depth:+"$depth"} || exit By having this additional fix (i.e. no '--depth', '' is passed to the submodule helper, we can improve the submodule helper further in clone_submodule we can drop the double check for `depth` and `reference` (as well as `gitdir`, that double check is unneeded as of now already), by just checking for the pointer to be non NULL and not further checking the dereferenced pointer. That can go in either squashed into this commit or on top of it, either is fine. That said: Reviewed-by: Stefan Beller > ( > clear_local_git_env > cd "$sm_path" && > @@ -709,7 +709,7 @@ Maybe you want to use 'update --init'?")" > > if ! test -d "$sm_path"/.git && ! test -f "$sm_path"/.git > then > - git submodule--helper clone ${GIT_QUIET:+--quiet} --prefix "$prefix" --path "$sm_path" --name "$name" --url "$url" "$reference" "$depth" || exit > + git submodule--helper clone ${GIT_QUIET:+--quiet} --prefix "$prefix" --path "$sm_path" --name "$name" --url "$url" ${reference:+"$reference"} ${depth:+"$depth"} || exit > cloned_modules="$cloned_modules;$name" > subsha1= > else > -- > 2.7.1.429.g45cd78e > ``` ## Jeff King, 2016-02-26 22:05 Subject: Re: [PATCH v4 3/3] git: submodule honor -c credential.* from command line Message-ID: <20160226220553.GA1835@sigill.intra.peff.net> URL: https://gitlist.dev/e/20160226220553.GA1835%40sigill.intra.peff.net In-Reply-To: <1456514328-10153-3-git-send-email-jacob.e.keller@intel.com> ``` On Fri, Feb 26, 2016 at 11:18:48AM -0800, Jacob Keller wrote: > +test_expect_success 'sanitize-config keeps credential.helper' " > + git -c credential.helper=helper submodule--helper sanitize-config >actual && > + cat >expect <<-EOF && > + 'credential.helper=helper' > + EOF > + test_cmp expect actual > +" This can (and should) be "<<-\EOF", right? I happened to be writing a test with the exact same problem (embedded single-quotes) today, and realized we have another solution which is used elsewhere in the test suite: sq="'" test_expect_success '...' ' echo "${sq}credential.helper=helper${sq}" >expect && ... ' that is slightly more verbose, but it does let us keep the main body inside single-quotes, without restoring to confusing backslash escaping. -Peff ``` ## Jacob Keller, 2016-02-26 22:08 Subject: Re: [PATCH v4 2/3] sumodule--helper: fix submodule--helper clone usage and check argc count Message-ID: URL: https://gitlist.dev/e/CA%2BP7%2Bxq8T8BsjfH9hrwULZiyTutt%2B6-qxqBZbZFHk490my_ofQ%40mail.gmail.com In-Reply-To: ``` On Fri, Feb 26, 2016 at 11:31 AM, Stefan Beller wrote: > On Fri, Feb 26, 2016 at 11:18 AM, Jacob Keller wrote: >> From: Jacob Keller >> - git submodule--helper clone ${GIT_QUIET:+--quiet} --prefix "$wt_prefix" --path "$sm_path" --name "$sm_name" --url "$realrepo" "$reference" "$depth" || exit >> + git submodule--helper clone ${GIT_QUIET:+--quiet} --prefix "$wt_prefix" --path "$sm_path" --name "$sm_name" --url "$realrepo" ${reference:+"$reference"} ${depth:+"$depth"} || exit > > By having this additional fix (i.e. no '--depth', '' is > passed to the > submodule helper, we can improve the submodule helper further > in clone_submodule we can drop the double check for `depth` and `reference` > (as well as `gitdir`, that double check is unneeded as of now already), > by just checking for the pointer to be non NULL and not further checking > the dereferenced pointer. > > That can go in either squashed into this commit or on top of it, either is fine. > > That said: > Reviewed-by: Stefan Beller > To be clear, what this *actually* does is prevent passing "" when depth or reference are empty. It never passed "--depth" "" together, or "--reference" "". It *does* pass --prefix "" sometimes, but not always, from what I could tell when using print debug statements while running the submodule tests. I am not sure if it ever passes "--depth" "" in any case, but I don't believe so. What this final change is needed for is that without it, after parse_options, argc is equal 2. Thanks, Jake ``` ## Jacob Keller, 2016-02-26 22:20 Subject: Re: [PATCH v4 3/3] git: submodule honor -c credential.* from command line Message-ID: URL: https://gitlist.dev/e/CA%2BP7%2BxqSZGAJD7ryvx2A2qM1wd0rKMmjT2tcBPann33sUUHGcA%40mail.gmail.com In-Reply-To: <20160226220553.GA1835@sigill.intra.peff.net> ``` On Fri, Feb 26, 2016 at 2:05 PM, Jeff King wrote: > On Fri, Feb 26, 2016 at 11:18:48AM -0800, Jacob Keller wrote: > >> +test_expect_success 'sanitize-config keeps credential.helper' " >> + git -c credential.helper=helper submodule--helper sanitize-config >actual && >> + cat >expect <<-EOF && >> + 'credential.helper=helper' >> + EOF >> + test_cmp expect actual >> +" > > This can (and should) be "<<-\EOF", right? > Yes, I actually meant <<-\EOF but forgot while writing it. > I happened to be writing a test with the exact same problem (embedded > single-quotes) today, and realized we have another solution which is > used elsewhere in the test suite: > > sq="'" > test_expect_success '...' ' > echo "${sq}credential.helper=helper${sq}" >expect && > ... > ' > > that is slightly more verbose, but it does let us keep the main body > inside single-quotes, without restoring to confusing backslash escaping. > I think I prefer the double quotes myself but will use this if people prefer? > -Peff ``` ## Jacob Keller, 2016-02-27 00:01 Subject: Re: [PATCH v4 3/3] git: submodule honor -c credential.* from command line Message-ID: URL: https://gitlist.dev/e/CA%2BP7%2BxroP8PFKe%2BzSdT-j9oq%2BAPT6OKOkbzS_DM8HyVjM-v_og%40mail.gmail.com In-Reply-To: ``` On Fri, Feb 26, 2016 at 2:20 PM, Jacob Keller wrote: >> that is slightly more verbose, but it does let us keep the main body >> inside single-quotes, without restoring to confusing backslash escaping. >> > > I think I prefer the double quotes myself but will use this if people prefer? > >> -Peff On second thought, I'd prefer consistency and I think using single quotes is probably better, so I'll use this solution in v5. Thanks, Jake ```