>>>>> "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.

Reply via email to