Re: [Opensync-users] Port of synce-plugin

[Markus M. <me...@me...>]

Pawel,

I'm not the maintainer of the SynCE plugin, but I looked at the patch
and think it looks good (though I can't test if it works or not, because
I have no CE device). So I just committed it (if anyone doesn't like
this, feel free to revert the commit, which is easy with SVN).

Anyone else with a CE device wants to test this?


Markus

Pawel Salek schrieb:
> Hi,
>
> I have got the synce plugin working with current libopensync. I had to
> make some changes to avoid occassional crashes when the configuration
> of the plugin was different than assumed in the code. Once I started
> that, I make the file synchronisation work as well. Abridged list of
> changes follows:
>
> * configure.in: current AM_PATH_RRA forgets to add -lmimedir to
>   the library list. Work around it for now.
> * src/synce_plugin.c: use small_caps names from synce_file.c
> * src/synce_file.c: port to the current infrastructure.
>   - prefix with config_file the remote file name correctly in all places.
>   - remove hacks that passed file name in fileFormat::data and
>     crashed when memory block size did not match.
>   - fix some memory leaks.
>   - do not crash in synce_file_commit() when config_file is not set.
> * src/synce_file.h: change prototypes to small caps.
>
> The patch is attached - I hope it will find its way soon to the
> repository!
>
> Best,
>
> Pawel