Jacob Keller <[email protected]> writes:

> +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, ' ');
> +
> +             sq_quotef(out, "%s=%s", var, value);

Can a configuration variable that comes from the original command
line be a boolean true that is spelled without "=true", i.e. can
value be NULL here?

> +     }
> +
> +     strbuf_release(&quoted);
> +
> +     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);
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to