https://bugs.meego.com/show_bug.cgi?id=14955

pohly <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |ASSIGNED

--- Comment #1 from pohly <[email protected]> 2011-03-28 05:09:11 PDT ---
commit 23464d616a1a6584ea433e64f62b130cfd33205d
Author: Patrick Ohly <[email protected]>
Date:   Mon Mar 28 13:52:28 2011 +0200

    syncevo-dbus-server: shut down after on-disk changes are observed (BMC
#14955)

    syncevo-dbus-server must restart after its installation was updated or
removed.
    Otherwise further sync attempts can fail. This was seen in practice when
    SyncEvolution 1.0 was updated to 1.1 (Debian bug #617320): the in-memory
daemon
    used an old libsynthesis, but the on-disk XML files required more recent
    libsynthesis features.

    In general, *any* update of something loaded into memory should
    trigger a shutdown or restart. A shutdown alone is okay when no
    automatic sync scheduling is needed (auto sync off for all
    configs). Clients will restart the daemon on demand. A restart is
    needed otherwise because without it, automatic syncs would stop to
    run.

    This patch implements the shutdown part. Restart still needs to be
    implemented. A 10 second delay is chosen between "file modified" and
    "shut down". This is meant to ensure that a future restart does not
    occur too soon (before all file changes are done). It's still a bit
    racy, but a better solution would depend integration into
    distro-specific hooks ("package update complete"), which is hard or
    impossible (installation via "make install" or "tar xf").

    This new feature is tested by test-dbus.py, including several corner
    cases:
    - testShutdown: files modified in regular intervals for a while
    - testSession: a running session prevents the shutdown
    - testSession2: same as testSession, with different timing

-- 
Configure bugmail: https://bugs.meego.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are watching someone on the CC list of the bug.
_______________________________________________
Syncevolution-issues mailing list
[email protected]
http://lists.syncevolution.org/listinfo/syncevolution-issues

Reply via email to