On Fri, 2014-05-23 at 13:02 +0200, Patrick Ohly wrote: > Let's revive this mail thread once more. The goal is to agree on a > change of the README.rst, in particular in the terminology section, and > then make the necessary code changes to match that change. > > I don't want to make code changes unless other developers or users agree > that the changes are an improvement, so please provide feedback.
There hasn't been much feedback on this particular proposal. Looks like I need to make a judgment call myself. I think we identified some useful improvements, so I went ahead with the proposal. The updated "doc" branch addresses the TODOs identified in the proposal and should be ready for merging into the master branch. My plan is to run a full test run over the weekend and give some time for feedback, say until the middle of next week. Then I could release a 1.4.99.4 snapshot either before or after a two week vacation - perhaps better after it, in case the update breaks anything. Attached is the resulting README.html file. Note in particular that this finally addresses the current situation with Google syncing (SyncML turned off, CardDAV/CalDAV better done with OAuth) by introducing a single Google template that works for CardDAV and CalDAV in a single context. Changes since the last update: commit 57f44fe607292d624135d11f96eb1c3cefa093b6 Author: Patrick Ohly <[email protected]> Date: Wed Jul 23 20:21:14 2014 +0200 config: allow storing credentials for email address When configuring a WebDAV server with username = email address and no URL (which is possible if the server supports service discovery via the domain in the email address), then storing the credentials in the GNOME keyring used to fail with "cannot store password in GNOME keyring, not enough attributes". That is because GNOME keyring seemed to get confused when a network login has no server name and some extra safeguards were added to SyncEvolution to avoid this. To store the credentials in the case above, the email address now gets split into user and domain part and together get used to look up the password. commit 5ab328af07a746c5bf881b05e5d4a4bd546fc36d Author: Patrick Ohly <[email protected]> Date: Thu Jul 24 10:51:13 2014 +0200 D-Bus testing: fix race condition in TestLocalSync.testNoParent The first progress signal gets emitted after sleeping for 10 seconds at the start of the sync and then killing syncevo-dbus-server races with completing the sync. What we want is to kill during the 10 second wait, so we better wait for the debug output directly before it and then kill directly. commit 8c6f770e386f63b6b2d83edca374c20e75e59ec9 Author: Patrick Ohly <[email protected]> Date: Wed Jul 23 21:12:25 2014 +0200 local sync: allow config name in syncURL=local:// Previously, only syncURL=local://@<context name> was allowed and used the "target-config@context name" config as target side in the local sync. Now "local://config-name@context-name" or simply "local://config-name" are also allowed. "target-config" is still the fallback if only a context is give. It also has one more special meaning: "--configure target-config@google-calendar" will pick the "Google_Calendar" template automatically because it knows that the intention is to configure the target side of a local sync. It does not know that when using some other name for the config, in which case the template (if needed) must be specified explicitly. The process name in output from the target side now also includes the configuration name if it is not the default "target-config". commit f14dafbd33cd8d6695c7aa791baf8ea4e1083aa9 Author: Patrick Ohly <[email protected]> Date: Fri Jul 25 11:09:12 2014 +0200 WebDAV: support multiple URLs in syncURL The syncURL property may contain multiple different space or tab separated URLs. Previously, the WebDAV backend only used the first one when scanning for databases. Now it tries all of them. This will be useful for configuring all Google endpoints in one template. commit df1205c141c7bc70fe6d456ec3ff92bca5f5aac5 Author: Patrick Ohly <[email protected]> Date: Fri Jul 25 14:02:29 2014 +0200 Google: remove SyncML template, combine CalDAV/CardDAV Google has turned off their SyncML server, so the corresponding "Google Contacts" template became useless and needs to be removed. It gets replaced by a "Google" template which combines the three different URLs currently used by Google for CalDAV/CardDAV. This new template can be used to configure a "target-config@google" with default calendar and address book database already enabled. The actual URL of these databases will be determined during the first sync using them. The template relies on the WebDAV backend's new capability to search multiple different entries in the syncURL property for databases. To avoid listing each calendar twice (once for the legacy URL, once with the new one) when using basic username/password authentication, the backend needs a special case for Google and detect that the legacy URL does not need to be checked. commit 6bbde198c96e5717bdb19d9d1b6d0a7a5ae9fd05 Author: Patrick Ohly <[email protected]> Date: Wed Jul 23 20:27:54 2014 +0200 doc: remove TODO about username = email address Fix pending for inclusion in master branch. commit b24008b28001082333c7c3a7a2a7b45e829677b5 Author: Patrick Ohly <[email protected]> Date: Wed Jul 23 20:31:44 2014 +0200 doc: fix compilation, minor language fixes reStructured text syntax was slightly broken, preventing compilation. Also, syncevolution does not know about --datastore-properties yet. Changed language slightly during fresh pass. commit 5e6efbbd6b9386778e3ff1b40ecea0982d8b7b0d Author: Patrick Ohly <[email protected]> Date: Thu Jul 24 11:13:14 2014 +0200 source -> datastore rename "--datastore-property" is now the preferred parameter name. "--source-property" is still supported as alias for backward compatibility. As many user-visible instances of "source" as possible got replaced in text strings by the newer term "datastore". Debug messages were left unchanged unless some regex happened to match it. The source code will continue to use the old variable and class names based on "source". commit 45888df64c550638c7eb991f735ea91e2aae1fb6 Author: Patrick Ohly <[email protected]> Date: Fri Jul 25 14:43:17 2014 +0200 doc: remove TODO about preventSlowSync This is now tracked in FDO #76471. commit 8c2ab09f7864ac89596c53e542c1b3a3e2b7bf63 Author: Patrick Ohly <[email protected]> Date: Fri Jul 25 15:42:52 2014 +0200 doc: syncURL property help text formatting When embedding help text in README.rst, a blank line is needed after the verbatim code block. commit d22335d8d6db3a3bb5792564d2feff43ea8614cb Author: Patrick Ohly <[email protected]> Date: Fri Jul 25 15:44:51 2014 +0200 doc: document new Google template and OAuth via GOA -- Best Regards, Patrick Ohly The content of this message is my personal opinion only and although I am an employee of Intel, the statements I make here in no way represent Intel's position on the issue, nor am I authorized to speak on behalf of Intel on this matter.
README.html.gz
Description: application/gzip
_______________________________________________ SyncEvolution mailing list [email protected] https://lists.syncevolution.org/mailman/listinfo/syncevolution
