>>>>> "Branden" == Branden Robinson <g.branden.robin...@gmail.com> writes:
Branden> Your patch looks good, except that I would quote the Branden> expansion of $XDG_RUNTIME_DIR when invoking mkdir. If Branden> $XDG_RUNTIME_DIR contains whitespace, the shell will Branden> tokenize it in a surprising way and create different Branden> directory names than are expected. I'll submit a new patch with the obvious change. One question first. Do I need to support the case where two sessions are using the same XDG_RUNTIME_DIR? (possibly by using sock.$$ instead of sock in the agent arguments)? I'm guessing that's not an issue, but while we're fine tuning I'll ask about that.