On Wed, 2 May 2018 17:53:58 -0700 Stefan Beller <sbel...@google.com> wrote:
> + argv_array_pushf(&cp.args, "path=%s; %s", > + path, info->argv[0]); Do we need to quote the path here? (For example, what if path has a quotation mark?) Also, would it be useful to have a test testing a submodule with a "weird" character (e.g. greater-than or single quote) in the name and the path?
