Excerpts from Jeff King's message of Mon Mar 26 14:24:27 -0400 2012: Hi Jeff, > OK, that's gross. I would argue that your SANE_TOOL_PATH is not > great, then, because the "sh" is not sane. But that being said, this > is exactly the sort of thing I was talking about in my first > message, which is that SANE_TOOL_PATH is hard to get right. If we > can make things Just Work by using SHELL_PATH, I don't see that as a > bad thing. Ok, thanks for the great feedback. I just sent an updated version of the patch with the squash done and what I hope is a better commit message. > It sounds like using bash is probably the least bad thing to do, and > doing anything else would not be worth the complexity. You could use > "bash -posix" (or an sh-symlink to bash), but in practice I doubt it > is really hurting anybody. I'd need to jump through some hoops to do an sh-symlink that was within our packaging standards, although it's not impossible. Using "bash -posix" is likely a better path for me to travel. I'll still need to plan out the change with lots of notice for users in case there have been bashism's used in local scripts, but as you say it's not likely hurting anyone in practice. Thanks -Ben -- Ben Walton Systems Programmer - CHASS University of Toronto C:416.407.5610 | W:416.978.4302 -- 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