Vasco Almeida <vascomalmeida@xxxxxxx> writes: > + gettextln "You are not currently on a branch." > + if test $op_type = "rebase" if test "$op_type" = rebase that is, $op_type could be an empty string that 'test' may not even see hence it should be double-quoted to ensure 'test' sees an empty string in that case, while "rebase" cannot be an empty string by definition so there is no need to quote it. Thanks. -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html