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 <[email protected]> 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. _______________________________________________ SyncEvolution mailing list [email protected] http://lists.syncevolution.org/listinfo/syncevolution
