Skip to content

Commit

Permalink
fixup for #963: we now have multiple cleanup methods for local sockets
Browse files Browse the repository at this point in the history
git-svn-id: https://xpra.org/svn/Xpra/trunk@11722 3bb7dfac-3a0b-4e04-842a-767bc560f471
  • Loading branch information
totaam committed Jan 22, 2016
1 parent 6595711 commit 1848e0a
Showing 1 changed file with 8 additions and 3 deletions.
11 changes: 8 additions & 3 deletions src/xpra/scripts/server.py
Original file line number Diff line number Diff line change
Expand Up @@ -1068,10 +1068,15 @@ def kill_xvfb():
from xpra.server.server_core import ServerCore
if e==ServerCore.EXITING_CODE:
log.info("exiting: not cleaning up Xvfb")
elif cleanup_socket in _cleanups:
elif local_sockets:
# don't delete the new sockets (probably not ours!)
# FIXME: keep track of which server (new or old) owns which socket,
# so we can delete the ones that have not been replaced
for socket, cleanup_socket in local_sockets:
if cleanup_socket in _cleanups:
log("removing cleanup: %s for %s", cleanup_socket, socket)
_cleanups.remove(cleanup_socket)
log.info("upgrading: not cleaning up Xvfb or socket")
# don't delete the new socket (not ours)
_cleanups.remove(cleanup_socket)
log("cleanups=%s", _cleanups)
e = 0
return e

0 comments on commit 1848e0a

Please sign in to comment.