[sync] Palm Tungsten C, Synthesis, Results, Patches, and Progress

Chuck Hagenbuch chuck at horde.org
Wed Jun 15 13:49:35 PDT 2005


Quoting Nathan Sharp <horde at nps.parcellsharp.net>:

> 3) I found a couple of bugs in the Encoder.php and Decoder.php.  First,
> they open up the third pipe for reading, which is technically
> incorrect.  It seems to have the side-effect of working as if you opened
> up /dev/null, which perhaps is a good thing since you aren't reading the
> stderr anyhow, although I was having some strange things happening until
> I made it write to a log file.  In my patch below, I open up a hardcoded
> log file for writing.  More importantly is that writing to the stdin
> entirely, then reading from the stdout is prone to deadlock, and in fact
> when my client tried to send the first 67 items (~35k of xml), it hung.
> My patch below shows me writing the input to a temp file and telling the
> sub-process to read from it, instead of from us.  This makes
> communication work correctly for me.

I've committed variants of these changes, thanks. Didn't change the 
default paths, though.

> 6) In order to cull down the number of items, I attempted to set
> synthesis to only sync the last 7 days and next 14.  I ran into the same
> problem that the poster yesterday mentioned, but was able to get around
> it with this cheap and dirty fix; I simply cull off anything after the ?
> in the database name.  This is probably not technically correct, but
> works for me:

Committed but using preg, thanks.

> 7) My last problem is that when the server tries to send an event to the
> Palm, the palm sends a "format not supported error" (I forgot to write
> down the http code).  My guess is that it doesn't support vcalendar
> version 2 format (the client always sends v1 events).  Looking at the
> code it seems like the right thing to do is to write a Device in the
> Devices directory for Synthesis that does the conversion.  I haven't had
> time to mess with this yet.

There is already a ticket on bugs.horde.org for this.

-chuck

-- 
"But she goes not abroad in search of monsters to destroy." - John 
Quincy Adams


More information about the sync mailing list