<p><b>@b4n</b> commented on this pull request.</p>

<hr>

<p>In <a href="https://github.com/geany/geany/pull/2222#discussion_r308570897">src/socket.c</a>:</p>
<pre style='color:#555'>> @@ -439,20 +441,35 @@ static gint socket_fd_open_unix(const gchar *path)
                return -1;
        }
 
-       /* fix for #1888561:
-        * in case the configuration directory is located on a network file system or any other
-        * file system which doesn't support sockets, we just link the socket there and create the
-        * real socket in the system's tmp directory assuming it supports sockets */
-       real_path = g_strdup_printf("%s%cgeany_socket.%08x",
-               g_get_tmp_dir(), G_DIR_SEPARATOR, g_random_int());
+       /* Try to place the socket in XDG_RUNTIME_DIR, according to XDG Base
+        * Directory Specification, see
+        * https://specifications.freedesktop.org/basedir-spec/latest */
+       real_dir = g_build_filename(g_get_user_runtime_dir(), "geany", NULL);
+       if (g_mkdir_with_parents(real_dir, 0755) == 0)
+       {
+               basename = g_path_get_basename(path);
+               real_path = g_build_filename(real_dir, basename, NULL);
+       }
</pre>
<p>This should already be fixed in the current version of the changes, it uses the same basename as it used to, including the random integer (which is admittedly not necessarily the smartest idea as there's a faint chance of clashes, but well, it seems to have served us good enough 'til now).</p>

<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br />You are receiving this because you are subscribed to this thread.<br />Reply to this email directly, <a href="https://github.com/geany/geany/pull/2222?email_source=notifications&email_token=AAIOWJYSR2G2P62DMALNZ2TQB7U2NA5CNFSM4IE3PHAKYY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOB76HIRI#discussion_r308570897">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AAIOWJ4HXOQBYA4YDO7Q3ILQB7U2NANCNFSM4IE3PHAA">mute the thread</a>.<img src="https://github.com/notifications/beacon/AAIOWJZ7YAGCFMX2R4RR2ALQB7U2NA5CNFSM4IE3PHAKYY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOB76HIRI.gif" height="1" width="1" alt="" /></p>
<script type="application/ld+json">[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/geany/geany/pull/2222?email_source=notifications\u0026email_token=AAIOWJYSR2G2P62DMALNZ2TQB7U2NA5CNFSM4IE3PHAKYY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOB76HIRI#discussion_r308570897",
"url": "https://github.com/geany/geany/pull/2222?email_source=notifications\u0026email_token=AAIOWJYSR2G2P62DMALNZ2TQB7U2NA5CNFSM4IE3PHAKYY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOB76HIRI#discussion_r308570897",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>