<html> <head> <base href="https://bugs.freedesktop.org/" /> </head> <body> <div> <a class="bz_bug_link bz_status_NEW " title="NEW --- - Unix spool is located at /var/spool/sockets/X11/... or /usr/spool/sockets/X11/... under HPUX" href="https://bugs.freedesktop.org/show_bug.cgi?id=69118#c2">Comment # 2</a> on <a class="bz_bug_link bz_status_NEW " title="NEW --- - Unix spool is located at /var/spool/sockets/X11/... or /usr/spool/sockets/X11/... under HPUX" href="https://bugs.freedesktop.org/show_bug.cgi?id=69118">bug 69118</a> from <a class="email" href="mailto:josh@freedesktop.org" title="Josh Triplett <josh@freedesktop.org>"> Josh Triplett</a> <pre>Comment on <a href="attachment.cgi?id=85462" name="attach_85462" title="Also looks for unix socket in /{usr,var}/spool/sockets/X11/...">attachment 85462</a> <a href="attachment.cgi?id=85462&action=edit" title="Also looks for unix socket in /{usr,var}/spool/sockets/X11/...">[details]</a> Also looks for unix socket in /{usr,var}/spool/sockets/X11/... Interesting approach. A few comments: Please drop "sep", and just make the separator part of the base string. You can then just use an array of strings rather than an array of structures. Please wrap the "spool" paths you're adding in an appropriate ifdef for HP-UX; they aren't appropriate for other platforms. You might Please turn this into two separate patches: one patch that has no functional effect but just converts to the "bases" approach, and one that adds the new bases you need.</pre> </div> <hr> You are receiving this mail because: <ul> <li>You are the QA Contact for the bug.</li> <li>You are the assignee for the bug.</li> </ul> </body> </html>