On Thu, Apr 28, 2016 at 7:12 AM, Mike Hommey <[email protected]> wrote: > Some remote systems can employ restricted shells that aren't very smart > with quotes, so avoid quoting when it's not strictly necessary. > > The list of "safe" characters comes from Mercurial's shell quoting > function used for its ssh client side. There likely are more that could > be added to the list. >
Would it make sense to move the new code into its own function and document it with this paragraph of the commit message, i.e. hinting at Mercurial safe characters or others? > Signed-off-by: Mike Hommey <[email protected]> > --- > connect.c | 11 +++++++++++ > 1 file changed, 11 insertions(+) > > diff --git a/connect.c b/connect.c > index 96c8c1d..919bf9e 100644 > --- a/connect.c > +++ b/connect.c > @@ -668,6 +668,17 @@ static void prepare_connect_command(struct strbuf *cmd, > const char *prog, > strbuf_addstr(cmd, prog); > strbuf_addch(cmd, ' '); > } > + if (quote) { > + const char *p; > + for (p = path; *p; p++) { > + if (!isalnum(*p) && *p != '@' && *p != '%' && > + *p != '_' && *p != '+' && *p != '=' && *p != ':' > && > + *p != ',' && *p != '.' && *p != '/' && *p != '-') > + break; > + } > + if (!*p) > + quote = 0; > + } > if (quote) > sq_quote_buf(cmd, path); > else > -- > 2.8.1.5.g18c8a48 > > -- > 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 -- 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
