John Szakmeister <john@xxxxxxxxxxxxxxx> writes: > Sorry about the repeat Junio, I meant to hit "Reply to All". > > On Mon, Mar 25, 2013 at 5:44 PM, Junio C Hamano <gitster@xxxxxxxxx> wrote: > [snip] >> Your patch makes me wonder if >> >> test -n "$TERM_PROGRAM" >> >> without any SECURITYSESSIONID or explicit program name checks should >> suffice, though. > > So, after downloading a couple of other terminals and trying things > out, I think you're suggestion does suffice. Should I send an updated > patch? If that approach is better than what you originally sent, then yes. But I do not use OS X, so you may need to pay attention to possible complaints and comments from other Mac users on this list for a while---there may be people who run the program in question without that environment variable. -- 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