Re: [SyncEvolution] new command line options: import/export
On Sat, 2010-06-12 at 07:19 +0100, Zhu, Yongsheng wrote:
Patrick,
1) can't compile syncevolution under my environment(Ubuntu10.4). Please see the below
patch.
Thanks, will merge.
2) can't find the help information for these options.
That's intentional:
commit fc8c504859d736d6f03c33da28a9975363360fe3
Author: Patrick Ohly <patrick.ohly(a)intel.com>
Date: Fri Jun 11 15:44:57 2010 +0200
command line: added --import/export/update/print-items
...
The builtin synopsis and usage was intentionally not updated. The
expection is that before this patch lands in the "master" branch, the
builtin text will come directly from README.rst (which was updated).
3) when deleting an non-existing item, show below info:
[ERROR] addressbook: addressbook: reading contact pas-id-4C0DEA8D00000003: EBookStatus
returned 9
[INFO] addressbook: deleting <pas-id-4C0DEA8D00000003>
Also the process returned value is '0'. Do you think it's reasonable?
That's expected. It is what I referred to in the --delete README.rst
entry with:
Trying to remove an item which does not
exist typically leads to an ERROR message, but is not reflected
in a
non-zero result of the command line invocation itself because
the
situation is not reported as an error by backends (removal of
non-existent items is not an error in SyncML).
It's not perfect, but doesn't warrant breaking the backend API.
--
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.