| 1 | 15:03 -!- dgollub changed the topic of #opensync to: OpenSync project IRC meeting | please no support Question - this is a moderatated |
|---|
| 2 | session |
|---|
| 3 | 15:03 < dgollub> Samm: lets discusss this later |
|---|
| 4 | 15:03 < ianmartin> Samm, I'm looking at writing better plugin development docs. would be happy to help |
|---|
| 5 | 15:04 < dgollub> ok - who's is present? |
|---|
| 6 | 15:04 < dgollub> (let's start) |
|---|
| 7 | 15:04 * ianmartin present |
|---|
| 8 | 15:04 * dgollub present |
|---|
| 9 | 15:04 * ChrisH present |
|---|
| 10 | 15:04 * Samm present |
|---|
| 11 | 15:04 < kaarposoft> present |
|---|
| 12 | 15:05 < dgollub> 1. Action Items from previous Meetings |
|---|
| 13 | 15:05 < dgollub> * AI dgollub: assemble list of terms. to use for developer API |
|---|
| 14 | 15:05 < dgollub> documentation |
|---|
| 15 | 15:05 < dgollub> * AI dgollub: introduce example-plugin using static-capabilities |
|---|
| 16 | 15:05 < dgollub> * AI dgollub: add capabilities chapter as requested by gcobb |
|---|
| 17 | 15:05 < dgollub> * AI dgollub: document each plugin function - which bits of |
|---|
| 18 | 15:05 < dgollub> information can be set/reported at each point - entry |
|---|
| 19 | 15:05 < dgollub> point documentation |
|---|
| 20 | 15:05 * bricks present |
|---|
| 21 | 15:05 < dgollub> ok - i feel pretty bad ... my AI list is growing and growing |
|---|
| 22 | 15:06 < dgollub> ianmartin: i have seen that you committed some stuff for the whitepaper ... did this contain also capabilities chapter |
|---|
| 23 | related stuff? |
|---|
| 24 | 15:06 < ianmartin> No, but working on that now |
|---|
| 25 | 15:06 < dgollub> (didn't managed to read your changes... i discovered the svn hook script was broken to compile the latex document on the |
|---|
| 26 | server) |
|---|
| 27 | 15:06 < ianmartin> so I can take add capabilities chapter as requested by gcob |
|---|
| 28 | 15:06 < dgollub> cool .. can reassing this AI to you? |
|---|
| 29 | 15:07 < dgollub> i guess you already good the idea how the stuff is supposted to work .. at list the opensync-devel gave a very good picture |
|---|
| 30 | 15:07 < dgollub> -good +got |
|---|
| 31 | 15:07 < ianmartin> can write something, ask wher I find gaps and then you can review to make sure it all makes sense |
|---|
| 32 | 15:08 < dgollub> ok cool |
|---|
| 33 | 15:09 < ianmartin> dgollub, I'm missing some styles for the latex stuff - will ask you about this later |
|---|
| 34 | 15:09 < dgollub> ok - no problem .. style is not so important as content is :) |
|---|
| 35 | 15:09 -!- absence [i=torgeihe@horisont.pvv.ntnu.no] has joined #opensync |
|---|
| 36 | 15:09 < dgollub> ok still try to do the other stuff... |
|---|
| 37 | 15:10 < dgollub> maybe i do the example static-capabiliteis stuff before wasting more time on bugfixing |
|---|
| 38 | 15:10 < dgollub> ok .. next topic |
|---|
| 39 | 15:10 < dgollub> 2. Doxygen / API |
|---|
| 40 | 15:10 < dgollub> * Status |
|---|
| 41 | 15:10 < dgollub> * Any trivial work left to give away? |
|---|
| 42 | 15:10 < dgollub> bricks: can you give any status on the api documentation front? |
|---|
| 43 | 15:11 < bricks> most of all c files are cleaned of doxygen annotations |
|---|
| 44 | 15:11 < bricks> currently i am working on opensync module |
|---|
| 45 | 15:11 < bricks> which i'll finish in some minutes |
|---|
| 46 | 15:11 < bricks> all module functions are now internal |
|---|
| 47 | 15:11 < dgollub> ok cool - thanks a lot |
|---|
| 48 | 15:11 < bricks> we need a lot of documentation |
|---|
| 49 | 15:11 < bricks> e.g. engine isn't documented at all |
|---|
| 50 | 15:12 < dgollub> i know :( .. i'll in middle of engien hacking ... i'll do the engine documentation |
|---|
| 51 | 15:12 < bricks> and the engine part is very importent ;) |
|---|
| 52 | 15:12 < dgollub> but is there already full doxygen "coverage" ... speak: does doxygen.log spots all missing/wrong doxygen documetnation? |
|---|
| 53 | 15:13 < bricks> have to take a look at |
|---|
| 54 | 15:13 < dgollub> ok |
|---|
| 55 | 15:13 < bricks> two "modules" are still open to move the doxygen stuff from the c files |
|---|
| 56 | 15:13 < dgollub> is there anything trivial work missing we could announce as easy-task for volunteers? (no programming skills required and |
|---|
| 57 | stuff like that..) |
|---|
| 58 | 15:13 < bricks> version and plugin |
|---|
| 59 | 15:14 < bricks> it is trivial (more or less) but cost et least one our |
|---|
| 60 | 15:14 < bricks> hour |
|---|
| 61 | 15:14 < dgollub> ok .. should we give this task to someone else? so you can concentrate on the more complicated stuff? |
|---|
| 62 | 15:14 < bricks> don't know |
|---|
| 63 | 15:14 < bricks> i can finish this |
|---|
| 64 | 15:14 < dgollub> ChrisH: could you help here again? |
|---|
| 65 | 15:14 < dgollub> ChrisH: iirc you already did some work here? |
|---|
| 66 | 15:15 < ChrisH> I did some before. |
|---|
| 67 | 15:15 < bricks> you can run make DoxygenDoc and see which "modules" are ready |
|---|
| 68 | 15:15 < dgollub> could you take care about version and plugin? |
|---|
| 69 | 15:15 < ChrisH> I can take another module. I cannot promise a timeline. |
|---|
| 70 | 15:15 < dgollub> just do module by module ..once a module is finsihed send a patch to opensycn-devel so we can commit it |
|---|
| 71 | 15:16 < bricks> btw. i updated a lot of doc but i don't know if it is right at all |
|---|
| 72 | 15:16 < dgollub> but at least we can release bricks from that trivial stuff and let him do some more complicated stuff |
|---|
| 73 | 15:17 < dgollub> bricks: people will complain soon enough if they read it :) no worry .. at least we have some docu |
|---|
| 74 | 15:17 < bricks> ;) ok |
|---|
| 75 | 15:17 < dgollub> by time i try to proofread this .. but this might take some more days before this happens |
|---|
| 76 | 15:18 < dgollub> ChrisH: so it's ok wit you to do "version" and "plugin" module? speak: move doxgen stuff from .c files to .h files? |
|---|
| 77 | 15:18 < ChrisH> yes. |
|---|
| 78 | 15:18 < dgollub> AI ChrisH: offload bricks by moving doxygen annotation of "version" and "plugin" modules to header file |
|---|
| 79 | 15:18 < dgollub> ok great |
|---|
| 80 | 15:18 < bricks> please note if any code is not documented |
|---|
| 81 | 15:19 < bricks> or if there are _ functions |
|---|
| 82 | 15:19 < bricks> e.g. _osync_do_anything |
|---|
| 83 | 15:19 < dgollub> ChrisH: like you did already in soem engien function which was #if 0 .. this was fully correct and very appreciated |
|---|
| 84 | 15:19 < bricks> i renamed all _functions and added them into the private header |
|---|
| 85 | 15:20 < dgollub> by reanming you mean _osync_blubb -> osync_blubb - right? |
|---|
| 86 | 15:20 < bricks> right |
|---|
| 87 | 15:21 < bricks> or if the are named like _do_something -> osync_do_something |
|---|
| 88 | 15:22 < ChrisH> Wont that break anything? |
|---|
| 89 | 15:23 < bricks> sometimes yes |
|---|
| 90 | 15:23 < dgollub> ChrisH: right .. it would be if you do for now just the doxygen stuff |
|---|
| 91 | 15:23 < dgollub> ChrisH: i guess that's easier... we do the _function stuff later... i would suggest |
|---|
| 92 | 15:23 < bricks> and write some comments if there are functions with _ |
|---|
| 93 | 15:24 < dgollub> ok - anything else? |
|---|
| 94 | 15:24 < dgollub> next topic? |
|---|
| 95 | 15:25 < dgollub> ok next. |
|---|
| 96 | 15:25 < bricks> yes |
|---|
| 97 | 15:25 < dgollub> 3. Bug/Ticket Status |
|---|
| 98 | 15:25 < dgollub> * Bugs against milestone 0.39 and 0.40 doesn't reflect roadmap |
|---|
| 99 | 15:25 < dgollub> defined guidelines |
|---|
| 100 | 15:25 < dgollub> o lots of bugs with severity normal assigned to milestone 0.39 |
|---|
| 101 | 15:25 < dgollub> * New hot bugs? |
|---|
| 102 | 15:25 < dgollub> * XMLFormat and vformat? |
|---|
| 103 | 15:25 < dgollub> i usual do ticket query like this: |
|---|
| 104 | 15:26 < dgollub> opensync.org/roadmap -> Active tickets for milestone xyz .. and then group by severity |
|---|
| 105 | 15:26 < dgollub> and for 0.39 and 0.40 there are lot "normal" bugs |
|---|
| 106 | 15:26 < dgollub> the roadmap milestone describes that such bugs not get fixed |
|---|
| 107 | 15:27 < dgollub> i already screend some bugs |
|---|
| 108 | 15:27 < dgollub> if you file bugs which have critical severity .. please make sure you also tag them with critical severity |
|---|
| 109 | 15:27 -!- LoneStar [n=xinzhen@114.92.160.17] has quit [Remote closed the connection] |
|---|
| 110 | 15:27 < dgollub> otherwise it might happen that i completely forgot about them |
|---|
| 111 | 15:28 < dgollub> fm is unfortuantely not around... but it would be great if someone could screen the bugs within the next days |
|---|
| 112 | 15:28 < dgollub> ... or even screen your own reports :) |
|---|
| 113 | 15:29 < dgollub> * XMLFormat and vformat? ... there are really loooooots of bugs |
|---|
| 114 | 15:29 < bricks> yes and i have a question about notes. which formats exists for notes? |
|---|
| 115 | 15:29 < dgollub> if you have time feel free to look into those... cstender seems to be out of time - so don't hesistate to touch vformat if |
|---|
| 116 | you like... but it's not high prio. |
|---|
| 117 | 15:30 < dgollub> bricks: vnote11 (IRMC specific) / memo / tomboy-note ... and then the discussion about vJournal |
|---|
| 118 | 15:30 < dgollub> bricks: there was recentlry a thread on opensync-devel about this... |
|---|
| 119 | 15:30 < bricks> that's the part i had in mind |
|---|
| 120 | 15:30 < dgollub> bricks: i'll send you a pointer to the thread alter... (offtopic) |
|---|
| 121 | 15:31 < dgollub> bricks: just remind me after the meeting |
|---|
| 122 | 15:31 < bricks> ok thanks. additionally i would like to say that vnote is also totally broken |
|---|
| 123 | 15:32 -!- absence [i=torgeihe@horisont.pvv.ntnu.no] has quit ["leaving"] |
|---|
| 124 | 15:32 < dgollub> i could imagine that |
|---|
| 125 | 15:33 < bricks> next topic? |
|---|
| 126 | 15:33 < dgollub> * New hot bugs? |
|---|
| 127 | 15:34 < dgollub> bricks: i filled vnote is also totally broken as item in the minutes |
|---|
| 128 | 15:34 < bricks> ok |
|---|
| 129 | 15:34 < bricks> :) |
|---|
| 130 | 15:34 < dgollub> other hot bugs which might slipped? |
|---|
| 131 | 15:34 < dgollub> i'm still fighting with mixed-objtypes ... so maybe i was missing something |
|---|
| 132 | 15:34 < kaarposoft> 1003 ? |
|---|
| 133 | 15:34 < dgollub> whats 1003 about? the syncml warning thing? |
|---|
| 134 | 15:35 < kaarposoft> infinite loop... |
|---|
| 135 | 15:35 < bricks> http://opensync.org/ticket/1003 |
|---|
| 136 | 15:35 < kaarposoft> ... with syncml, yes |
|---|
| 137 | 15:35 < dgollub> ah this one.. is there no timeout? |
|---|
| 138 | 15:35 < dgollub> kaarposoft: could you keep this running for at least 30minutes? ;) |
|---|
| 139 | 15:36 < dgollub> arg.. don't use syncml-client as component.. this used to be another syncml plugin |
|---|
| 140 | 15:36 < dgollub> we need to drop syncml-client |
|---|
| 141 | 15:36 < dgollub> oh wait |
|---|
| 142 | 15:36 < dgollub> michael is assinged.. |
|---|
| 143 | 15:36 < dgollub> ok then it's fine |
|---|
| 144 | 15:36 < kaarposoft> I do not use syncml-client but syncml-obex-client |
|---|
| 145 | 15:37 < dgollub> ok ... other stuff? e.g. engine related? |
|---|
| 146 | 15:37 < dgollub> slow-sync not working or fake conflicts? duplicates? |
|---|
| 147 | 15:38 < dgollub> ok .. next topic? |
|---|
| 148 | 15:38 < dgollub> next .. |
|---|
| 149 | 15:38 < kaarposoft> Well, 1003 is actually intial slow-sync not working for Nokia 6280 |
|---|
| 150 | 15:38 < kaarposoft> I have no clue if it is engine or syncml releated. |
|---|
| 151 | 15:38 < dgollub> kaarposoft: please state in the bug if this cause an timeout after 30 minutes or not |
|---|
| 152 | 15:38 < dgollub> kaarposoft: if this not cause a timeout after 30 minutes -> engine |
|---|
| 153 | 15:38 < dgollub> 4. Splitting XMLFormat-plugin out of OpenSync? |
|---|
| 154 | 15:38 < dgollub> * introduce format plugin merger/demerge function pointers |
|---|
| 155 | 15:38 < dgollub> * drop "internalFormat" from Engine |
|---|
| 156 | 15:38 < dgollub> * for 0.40? |
|---|
| 157 | 15:38 < kaarposoft> Will do |
|---|
| 158 | 15:39 < dgollub> since the quality of xmlformat is not really brillant and very likely we'll see lots of change also after 0.40 ... and |
|---|
| 159 | also to be able to switch to other "internal formats" in future i thought about dropping xmlformat |
|---|
| 160 | 15:39 < dgollub> ... from /trunk |
|---|
| 161 | 15:40 < dgollub> and keep it completely isolated from /trunk |
|---|
| 162 | 15:40 < dgollub> we keep the xmlformat-api to assemble the format |
|---|
| 163 | 15:40 < dgollub> this would also turn out that the merge/demerge calls needs to turn into ObjFormat specific function calls |
|---|
| 164 | 15:41 < kaarposoft> is xmlformat not the "canonical" data format? I mean if everything (eg Vxxx) else fails, then xmlformat should work? |
|---|
| 165 | 15:41 < dgollub> no not only .. currently it's the only format which allows us to merge/demerge changes |
|---|
| 166 | 15:42 < dgollub> and it's the only format which allows "sane" comparsion and mapping... |
|---|
| 167 | 15:42 < dgollub> the vformat compare function today consists of memcmp() which is just a joke |
|---|
| 168 | 15:42 < dgollub> xmlformat_compare() is pretty smart.. and does weak comparsion and returns SAME/SIMILAR/MISMATCH |
|---|
| 169 | 15:42 < dgollub> which is important |
|---|
| 170 | 15:43 < kaarposoft> exactly. so why throw it out of trunk? |
|---|
| 171 | 15:43 < dgollub> if we have bugs in xmlformat_compare() or need to enhance the xmlformat-schemas we have to release opensync |
|---|
| 172 | 15:43 < dgollub> (... i mean to fix the bug .. we need to release opensync) |
|---|
| 173 | 15:44 < dgollub> it's much less maintenacne effort to release xmlformat-plugin seperated |
|---|
| 174 | 15:44 < kaarposoft> well, I have no strong opinion either way - just trying to understand (-; |
|---|
| 175 | 15:44 < dgollub> also the isolation is important later... if someone wants to use the opensycn engine without our xmlformat crap |
|---|
| 176 | 15:44 < dgollub> bricks: what do you think? |
|---|
| 177 | 15:44 < dgollub> ianmartin: thoughts? |
|---|
| 178 | 15:45 < bricks> i don't know if it is possible |
|---|
| 179 | 15:45 < bricks> yes of course we can maintain it outside trunk |
|---|
| 180 | 15:45 < bricks> but the converter does heavily depend on xmlformat |
|---|
| 181 | 15:46 < bricks> and the merger too |
|---|
| 182 | 15:46 < dgollub> right |
|---|
| 183 | 15:46 < bricks> and these are important parts of osync |
|---|
| 184 | 15:46 < dgollub> but if we introduce merger/demrge format-plugin functions .. and keep the internalFormat stuff in the engine for now this |
|---|
| 185 | should still work |
|---|
| 186 | 15:46 < ianmartin> surely the internal format is key to opensync |
|---|
| 187 | 15:47 < bricks> if a user has to install xmlformat to use opensync |
|---|
| 188 | 15:47 < dgollub> actually the conversion path should be able to handle this without that we have to hardcode an internal format |
|---|
| 189 | 15:47 < bricks> then i don't think we should split it form trunk |
|---|
| 190 | 15:47 < dgollub> bricks: thats more a packaging thing.. vformat plugin would recommed/require for isntance the xmlformat plugin |
|---|
| 191 | 15:47 < bricks> dgollub: yes it should but it doesn't |
|---|
| 192 | 15:48 < bricks> and the conversion code is really really not understandable |
|---|
| 193 | 15:48 < ianmartin> but what about the archive? |
|---|
| 194 | 15:48 < dgollub> ianmartin: the archive stores everything |
|---|
| 195 | 15:48 < dgollub> bricks: yeah .. another point to make sure it's independent of xmllformat :) |
|---|
| 196 | 15:49 < ianmartin> indeed, so you need an internal format that supports everything |
|---|
| 197 | 15:49 < dgollub> ianmartin: why? |
|---|
| 198 | 15:49 < dgollub> ianmartin: was this with regard to the archive? |
|---|
| 199 | 15:49 < ianmartin> dgollub, yes |
|---|
| 200 | 15:50 < dgollub> if the merge/demerge can be done with any format-plugin ... why should the archive not handle this? |
|---|
| 201 | 15:50 < dgollub> the problem i see with the internalFormat and the xmlformat is that we restrict our self to xmlformat-only |
|---|
| 202 | 15:51 < dgollub> this should not be the case.. since this would avoid other people/project/companies to use opensync as synchronization |
|---|
| 203 | logic |
|---|
| 204 | 15:51 < dgollub> maybe the want to support a new set of data ... a new objtype/content-type |
|---|
| 205 | 15:52 < dgollub> they have two formats to convert.. and also to deal with capabilities.. with the current implementation it would be not |
|---|
| 206 | possible for them to use the merger and archive |
|---|
| 207 | 15:53 < dgollub> and it shouldn't be so hard to call instaed of osync_xmlformat_merge() a format-plugin format pointer |
|---|
| 208 | 15:53 < dgollub> the only ciritcal thing which i agree on is the current conversion path implemetnation |
|---|
| 209 | 15:54 < dgollub> if we would drop the "internalFormat" which is hardcoded in the engine to force all formats to get converted first to |
|---|
| 210 | xmlformat.. that this might cause problem |
|---|
| 211 | 15:54 < ianmartin> so you would need to know if a format-plugin can merge or no |
|---|
| 212 | 15:54 < dgollub> yes |
|---|
| 213 | 15:54 < bricks> ok i guess we need to rewrite the conversion stuff |
|---|
| 214 | 15:54 < dgollub> but thats quite easy .. if the format plugins doesn't has a merge/demrge function regisrtesd -> not supported |
|---|
| 215 | 15:54 < dgollub> bricks: also an idea |
|---|
| 216 | 15:55 < dgollub> so the longterm goal would be a conversion path which is smart enough to build a conversion path via a format which is |
|---|
| 217 | able to convert first to a format which is able to merge/demerge |
|---|
| 218 | 15:56 < dgollub> and then convert to the target format |
|---|
| 219 | 15:56 < dgollub> but this is only required if we drop the "internalFormat" thing |
|---|
| 220 | 15:56 < bricks> yes and maybe a direct way without internal conversion |
|---|
| 221 | 15:56 < bricks> yes |
|---|
| 222 | 15:56 < dgollub> so first i would suggest to move the merge/demerge stuff into the format-plugin |
|---|
| 223 | 15:56 < dgollub> and then decide if we want to do rewrite conversion path from scratch |
|---|
| 224 | 15:57 < dgollub> what do you think? |
|---|
| 225 | 15:57 < bricks> ok for me |
|---|
| 226 | 15:57 < dgollub> ok cool |
|---|
| 227 | 15:57 < bricks> i don't like the curent code of the converter ;) |
|---|
| 228 | 15:57 < dgollub> bricks: are you interested? :) i know you have expereince in that space |
|---|
| 229 | 15:57 < ianmartin> sounds ok, but also another obstacle to getting something actually released |
|---|
| 230 | 15:57 < bricks> merger/demerge <- no eperience |
|---|
| 231 | 15:58 < bricks> i don't even know when to use it |
|---|
| 232 | 15:58 < dgollub> ianmartin: yeah .. but this would cause a major api break (maybe) |
|---|
| 233 | 15:58 < dgollub> ianmartin: and we coudl also decide to release 0.40 earlier .. indepednet of the qualtiy of xmlformat-plugin |
|---|
| 234 | 15:58 < dgollub> ianmartin: we just say .. opensync is ready ... and you're safe doing backups - but conversion between pim formats is |
|---|
| 235 | completely unsafe |
|---|
| 236 | 15:58 < dgollub> ianmartin: due to xmlformat |
|---|
| 237 | 15:59 < dgollub> ianmartin: and there are lot of people actually just want to sync first abc-sync to file-sync |
|---|
| 238 | 15:59 < bricks> we should provide a opensync version which is capable of api/abi stability in case of internal format chages |
|---|
| 239 | 15:59 < dgollub> bricks: you don't need to touch merger/demerge stuff .. just introduce format-plugin interface |
|---|
| 240 | 16:00 < ianmartin> dgollub, indeed just getting information of a peer and into file-sync is qutie useful! |
|---|
| 241 | 16:00 < dgollub> s... which allow to call the merger of the format instead of the hardcoded one - i'll point you to the code which you have |
|---|
| 242 | to replace in the engine once you introduce the interface |
|---|
| 243 | 16:00 < dgollub> ianmartin: for some crazy reason more people asking for that then acutal sync :) |
|---|
| 244 | 16:01 < dgollub> so the main idea is ... to get opensync 0.40 out with a stable vformat plugin |
|---|
| 245 | 16:01 < dgollub> so people which want to write their sync application without the vformat-crap and sync other staff can pickup |
|---|
| 246 | 16:01 < bricks> right |
|---|
| 247 | 16:02 < dgollub> also that people can write sync stuff which is independent of xmlformat stuff |
|---|
| 248 | 16:02 < dgollub> the conversion path is able to convert via several formats... so i don't want to keep the impression xmlformat is the only |
|---|
| 249 | thing to be used in opensync |
|---|
| 250 | 16:03 < dgollub> if someone shows up with a new format-plugin which is convertingn vCalendar and iCalendar flawless.. perfect! it doesn't |
|---|
| 251 | needs to be done via xmlformat |
|---|
| 252 | 16:03 < bricks> would be great |
|---|
| 253 | 16:03 < bricks> i like xmlformat because it is xml |
|---|
| 254 | 16:03 < bricks> but i don't think it fits all need |
|---|
| 255 | 16:03 < bricks> s |
|---|
| 256 | 16:03 < dgollub> so we're still open to vCalendar <-> iCalendar <-> xmlformat-event <-> libgcal-format-event |
|---|
| 257 | 16:05 < dgollub> ianmartin: so this could be acutally speed up to get things released.. check the 0.39 roadmap .. there are approx. 9 |
|---|
| 258 | tickets only for not xmlformat or vformat related stuff |
|---|
| 259 | 16:05 < dgollub> another cool thing is .. we don't have to freeze the xmlformat schema with 0.40 :) |
|---|
| 260 | 16:06 < dgollub> bricks: so do you take the AI for the merge/demerge functios for format plugins? |
|---|
| 261 | 16:07 < ianmartin> in above example at the moment only xmlformat-event can do (de)merger so xmlformat-events are stored in the archive |
|---|
| 262 | 16:07 < dgollub> bricks: i'll assist you then when it comes to strip the merge out of the engien and do the function calls and implement |
|---|
| 263 | the merging in the xmlformat |
|---|
| 264 | 16:07 < bricks> don't have a clou about the parameters |
|---|
| 265 | 16:07 < dgollub> ianmartin: right |
|---|
| 266 | 16:07 < dgollub> bricks: i'll give you a protoype declartion for this later |
|---|
| 267 | 16:07 < bricks> ok then it's fine |
|---|
| 268 | 16:08 < dgollub> AI bricks, dgollub: introduce merger/demerge format-plugin functions - make merging independent of xmlformat |
|---|
| 269 | 16:08 < ianmartin> then i add a new plugin vCalendar <->libgcal-format-event with merger/demerger, but somehow we must still go via |
|---|
| 270 | xmlformat-event so use the info in the archive |
|---|
| 271 | 16:09 < dgollub> ianmartin: for that we have "internalFormats" in engine ... |
|---|
| 272 | 16:09 < dgollub> ianmartin: for time being... but later the conversion-path-builder should handle that |
|---|
| 273 | 16:10 < ianmartin> ok, so first step is add merger/demerger to plugin interface |
|---|
| 274 | 16:10 < ianmartin> initially this will only apply to xmlformat |
|---|
| 275 | 16:10 < dgollub> http://opensync.org/browser/trunk/opensync/engine/opensync_engine.c#L1242 |
|---|
| 276 | 16:10 < dgollub> ianmartin: correct |
|---|
| 277 | 16:10 < ianmartin> and we the path will still always go via xmlformat for now |
|---|
| 278 | 16:11 < ianmartin> but at later date path can be rewritten to be more intelligent |
|---|
| 279 | 16:11 < dgollub> right |
|---|
| 280 | 16:11 < bricks> right |
|---|
| 281 | 16:11 < dgollub> the link i sent will make sure it will go always via xmlformat |
|---|
| 282 | 16:11 -!- Samm [n=chatzill@129-170-195-217.cust.centrio.cz] has quit [Read error: 60 (Operation timed out)] |
|---|
| 283 | 16:11 < bricks> if it isn't data objtype ;) |
|---|
| 284 | 16:12 < dgollub> bricks: haha .. this changed a bit the last days :) |
|---|
| 285 | 16:12 < dgollub> http://opensync.org/browser/trunk/opensync/engine/opensync_engine.c#L192 |
|---|
| 286 | 16:12 < ianmartin> so one day these internal formats will disappear? |
|---|
| 287 | 16:13 < dgollub> http://opensync.org/browser/trunk/opensync/engine/opensync_engine.c#L221 - detect encapsulated formats .. right now |
|---|
| 288 | limited to "data" |
|---|
| 289 | 16:13 < bricks> yes |
|---|
| 290 | 16:13 < dgollub> ianmartin: yes - when the conversion-path builiding logic is able to make sure merging is done at the right place |
|---|
| 291 | 16:15 < bricks> nice code so file-sync has to use always data |
|---|
| 292 | 16:15 < dgollub> bricks: but later i try to get rid of the "data" stuff as well :) but not for 0.40 |
|---|
| 293 | 16:15 < dgollub> bricks: no |
|---|
| 294 | 16:15 < dgollub> bricks: or.. yes right |
|---|
| 295 | 16:15 < dgollub> bricks: but only temp. .. if i remove this !strcmp("data") thing the testsuite explodes |
|---|
| 296 | 16:16 < dgollub> the mock-format plugin is not clean |
|---|
| 297 | 16:16 -!- juergbi [n=juerg@84-73-62-122.dclient.hispeed.ch] has joined #opensync |
|---|
| 298 | 16:16 < dgollub> it crashes in the detect() call ... i'll fix that once my mixed-objtype stuff is copmleted |
|---|
| 299 | 16:17 < dgollub> but later every format should be checked for encapsulated formats |
|---|
| 300 | 16:17 < dgollub> -format +objtype |
|---|
| 301 | 16:18 < dgollub> so even if file-sync reports "contact" since it has per directory different objtypes ... the detect() functions check if |
|---|
| 302 | the "file" format encapsulates something elese... |
|---|
| 303 | 16:18 < bricks> ok i guess we have a common view of the format conversion |
|---|
| 304 | 16:18 < dgollub> ah wait.. there is anotehr thing |
|---|
| 305 | 16:19 < dgollub> right now we do two times detect.. which is also ugly |
|---|
| 306 | 16:19 < dgollub> but i'll fix that later |
|---|
| 307 | 16:19 < dgollub> first for the objtype detection.. later to find the encapuslated formats |
|---|
| 308 | 16:19 < dgollub> ok .. next topic |
|---|
| 309 | 16:20 < dgollub> 5. Capabilities & Merger testing |
|---|
| 310 | 16:20 < dgollub> * Request for real-life testing of capabilities and merger |
|---|
| 311 | 16:20 < dgollub> * Generating ~/.opensync/groupX/Y/capabilities.xml by hand |
|---|
| 312 | 16:20 < dgollub> * Check for data-loss (on slow-sync) |
|---|
| 313 | 16:20 < dgollub> * Check for duplicate (on slow-sync) |
|---|
| 314 | 16:20 < dgollub> * Check for conflicts (on slow-sync) |
|---|
| 315 | 16:20 < dgollub> maybe we should postpone this one |
|---|
| 316 | 16:20 < bricks> ok |
|---|
| 317 | 16:20 < dgollub> since we agreed on touch the merger stuff |
|---|
| 318 | 16:21 < dgollub> ianmartin: is the merger/capabilties stuff working for you? |
|---|
| 319 | 16:21 < dgollub> ianmartin: the stuff we have today? |
|---|
| 320 | 16:22 -!- amilcar [n=amilcar@dslb-092-076-114-247.pools.arcor-ip.net] has joined #opensync |
|---|
| 321 | 16:22 < dgollub> ok .. we run out of time |
|---|
| 322 | 16:22 < ianmartin> dgollub, not sure, so much goes missing with vformat not getting it's config I stopped checking it |
|---|
| 323 | 16:23 < dgollub> ianmartin: i see.. i reviewed your bug. another point to rewrite the conversion-path buildier :) |
|---|
| 324 | 16:23 < bricks> from scratch! |
|---|
| 325 | 16:24 < dgollub> right :P |
|---|
| 326 | 16:24 < dgollub> ianmartin: but ocne you run discover your member has a capabilites.xml right? |
|---|
| 327 | 16:24 < dgollub> ok next topic" |
|---|
| 328 | 16:24 < dgollub> 6. Plugin Configurations |
|---|
| 329 | 16:24 < dgollub> * Static / Dynamic configuration |
|---|
| 330 | 16:24 < dgollub> * Resource configuration |
|---|
| 331 | 16:24 < dgollub> * Documentation |
|---|
| 332 | 16:24 < dgollub> * Advanced Options |
|---|
| 333 | 16:24 < dgollub> * Option description (i18n) |
|---|
| 334 | 16:24 < dgollub> * Changes for 0.40? |
|---|
| 335 | 16:24 < dgollub> * Who takes care about this? |
|---|
| 336 | 16:24 < dgollub> kaarposoft: around? |
|---|
| 337 | 16:25 < kaarposoft> yes! |
|---|
| 338 | 16:25 < dgollub> you brought about discussion about having different plugin configuration |
|---|
| 339 | 16:25 < kaarposoft> Two points: |
|---|
| 340 | 16:26 < ianmartin> dgollub, yes capabiltieis.xml is generated if you create an OSyncCapabilities during discover |
|---|
| 341 | 16:26 < kaarposoft> 1) Currently we have the possible advanced config options for a plugin in the same file as the actual option settings |
|---|
| 342 | for a group. |
|---|
| 343 | 16:26 < kaarposoft> 2) We need better doc of plugins and their capabilities. |
|---|
| 344 | 16:26 < kaarposoft> two suggestions: |
|---|
| 345 | 16:27 < kaarposoft> 1) split config file in one static for the plugin and one dynamic for group |
|---|
| 346 | 16:27 < kaarposoft> 2) move doc out of config file and into localized language files |
|---|
| 347 | 16:27 < kaarposoft> over... |
|---|
| 348 | 16:28 < dgollub> ok what's the content of the dynamic one? |
|---|
| 349 | 16:28 < kaarposoft> the ACTUAL settings of the options for a specific group |
|---|
| 350 | 16:29 < dgollub> you mean member? |
|---|
| 351 | 16:29 < bricks> an static is e.g. a path to thunderbird |
|---|
| 352 | 16:29 < kaarposoft> sorry, yes, member, not group. |
|---|
| 353 | 16:29 < kaarposoft> Eg. this |
|---|
| 354 | 16:29 < kaarposoft> <AdvancedOption> <DisplayName>SyncML Version</DisplayName> <MaxOccurs>2147483647</MaxOccurs> |
|---|
| 355 | <Max>2147483647</Max> <Name>SANVersion</Name> <Type>string</Type> <ValEnum>1.0</ValEnum> |
|---|
| 356 | <ValEnum>1.1</ValEnum> <ValEnum>1.2</ValEnum> <Value>1.1</Value> </AdvancedOption> |
|---|
| 357 | 16:30 < kaarposoft> would be split into dynamic: |
|---|
| 358 | 16:30 < kaarposoft> <Value>1.1</Value> |
|---|
| 359 | 16:30 < kaarposoft> language: |
|---|
| 360 | 16:30 < kaarposoft> <DisplayName>... |
|---|
| 361 | 16:30 < kaarposoft> and the rest in the static plugin def |
|---|
| 362 | 16:31 < dgollub> ok - where is the static one located? in the member direcotries .. or wouldn't it better to store this a distro direocty |
|---|
| 363 | .. for easier update? |
|---|
| 364 | 16:31 < kaarposoft> I think this is a much cleaner split, and will be easier to maintain |
|---|
| 365 | 16:32 < kaarposoft> yes, the static one and the localization ones should be in distro dir - they are static and never touched by the user. |
|---|
| 366 | 16:32 < dgollub> e.g. /usr/share/libopensyncX/plugin/$plugin/advanced.xml |
|---|
| 367 | 16:32 < kaarposoft> exactly |
|---|
| 368 | 16:32 < dgollub> ok - pretty cool idea |
|---|
| 369 | 16:33 < dgollub> i like the idea - are you interested in taking care about this? |
|---|
| 370 | 16:34 < kaarposoft> and /usr/share/libopensyncX/plugin/$plugin/da-dk.xml , /usr/share/libopensyncX/plugin/$plugin/en-US.xml |
|---|
| 371 | 16:34 < kaarposoft> I have too much on my plate already with the mozilla plugin and frontend )))-: |
|---|
| 372 | 16:34 < kaarposoft> ... and very little knowledge of the internal workings of OpenSync |
|---|
| 373 | 16:35 < dgollub> kaarposoft: best time to start with it :) |
|---|
| 374 | 16:35 < dgollub> the configuration stuff is not that hard... it's only a bit libxml - but the code you need to know already exists |
|---|
| 375 | 16:36 < dgollub> (expect the i18n thing) |
|---|
| 376 | 16:36 < dgollub> you just need to copy&paste and adjus tthe code |
|---|
| 377 | 16:36 < dgollub> thats it |
|---|
| 378 | 16:37 < dgollub> and if you get stuck .. just drop a mail on opensync-devel .. i |
|---|
| 379 | 16:37 < kaarposoft> I am sorry, but I would rather say no now, than do a half-hearted job. I think it is better that I stick to the mozilla |
|---|
| 380 | plugin |
|---|
| 381 | 16:37 < dgollub> i'm sure somewone will help |
|---|
| 382 | 16:37 < kaarposoft> But I can make a ticket with a more detailed description... |
|---|
| 383 | 16:38 < dgollub> kaarposoft: the problem then is that this might not happen... since we already have a lot todo :/ |
|---|
| 384 | 16:38 < dgollub> and once opensync is doen we can help on mozilla-sync :P |
|---|
| 385 | 16:38 < kaarposoft> I know )))-: |
|---|
| 386 | 16:39 < dgollub> so i would be fine with the changes if you could at least try to start with it ... if there is any complicated outstanding |
|---|
| 387 | stuff left i'm sure someone could help fixing it |
|---|
| 388 | 16:39 < dgollub> but we need someone how does detatiled implementation desciprtion/plan/API and then at least start with it |
|---|
| 389 | 16:39 < kaarposoft> one big issue: this will impact all plugins, they will all have to change config files. |
|---|
| 390 | 16:39 < dgollub> thats no big deal today |
|---|
| 391 | 16:39 < dgollub> it's a big deal after 0.40 |
|---|
| 392 | 16:40 -!- Savago [n=adenilso@189-19-47-130.dsl.telesp.net.br] has joined #opensync |
|---|
| 393 | 16:40 < kaarposoft> it's a big deal for me... |
|---|
| 394 | 16:40 < dgollub> we don't care in 0.3x about plugin config difference |
|---|
| 395 | 16:40 < dgollub> why? because you have to touch mozilla-sync? |
|---|
| 396 | 16:40 < dgollub> everyone which has a plugin in 0.3x space would need to touch the plugin |
|---|
| 397 | 16:40 < dgollub> but better to touch it in 0.3x then in 04.x |
|---|
| 398 | 16:41 < kaarposoft> OK, what I can offer is this: I will look into the issue, try to make a plan for the API etc, and maybe - just maybe - |
|---|
| 399 | a patch... |
|---|
| 400 | 16:41 < kaarposoft> why? because you have to touch mozilla-sync? <-- No, because I have to touch all the others! |
|---|
| 401 | 16:42 < bricks> no you don't have to touch the others |
|---|
| 402 | 16:42 < bricks> that has to be done by the other plugin developers |
|---|
| 403 | 16:42 < bricks> like in the past |
|---|
| 404 | 16:42 < dgollub> right |
|---|
| 405 | 16:42 < bricks> where we changed the config in nearly every osync version ;) |
|---|
| 406 | 16:42 < dgollub> kaarposoft: the only thing you should do .. describe what needs to be changed |
|---|
| 407 | 16:43 < kaarposoft> ok, fair enough. Anyway, as I said: I will have a look, but no promises |
|---|
| 408 | 16:43 < dgollub> ok cool |
|---|
| 409 | 16:43 < dgollub> feel free to send even patches in of little changes .. not of the entire implementation. .so we have at least some pieces |
|---|
| 410 | of the implemetnion where to continue |
|---|
| 411 | 16:44 < kaarposoft> should i put the proposed description etc. in opensync-devel, on wiki, or ? |
|---|
| 412 | 16:44 < dgollub> opensync-devel@ easier to reply and add comments |
|---|
| 413 | 16:44 < bricks> ticket and devel would be nice |
|---|
| 414 | 16:44 < kaarposoft> ACK |
|---|
| 415 | 16:44 < dgollub> yeah ticket to track their is something missing before realsing 0.40 |
|---|
| 416 | 16:45 < kaarposoft> 3 * ACK |
|---|
| 417 | 16:45 < dgollub> ok next topic |
|---|
| 418 | 16:45 < dgollub> 7. User vs. Developer documentation |
|---|
| 419 | 16:45 < dgollub> * terminology of target audience |
|---|
| 420 | 16:45 < dgollub> o OpenSync core developers |
|---|
| 421 | 16:45 < dgollub> o Plugin Developer (Format & Connectors) |
|---|
| 422 | 16:45 < dgollub> o Frontend/Server Developer |
|---|
| 423 | 16:45 < dgollub> o "Beta"-Tester (includes Alpha-Tester) |
|---|
| 424 | 16:45 < dgollub> * what is of interested for the Developer? |
|---|
| 425 | 16:45 < dgollub> * what is of interested for the User? |
|---|
| 426 | 16:45 < dgollub> * what to document at all? |
|---|
| 427 | 16:45 < bricks> postpone? |
|---|
| 428 | 16:45 < dgollub> o OpenSync API? (Plugin & Frontend/Server API) |
|---|
| 429 | 16:45 < dgollub> o usage of osynctool/msynctool? |
|---|
| 430 | 16:46 < dgollub> o plugin configuration? |
|---|
| 431 | 16:46 < dgollub> o "Should Suse document Apache? I don't think so." |
|---|
| 432 | 16:46 < dgollub> yeah |
|---|
| 433 | 16:46 < dgollub> Tuju is not around.. i guess he was raelly intrested in this |
|---|
| 434 | 16:46 < kaarposoft> postpone? - yes, running a bit late here ((-; |
|---|
| 435 | 16:46 < dgollub> 8. Next IRC Meeting |
|---|
| 436 | 16:46 < dgollub> * Next IRC Meeting |
|---|
| 437 | 16:46 < dgollub> * Who takes minutes/backup? |
|---|
| 438 | 16:47 < dgollub> ok .. next week would be regular dates |
|---|
| 439 | 16:47 < dgollub> 2nd calendar week |
|---|
| 440 | 16:47 < dgollub> need to check which weekday was even |
|---|
| 441 | 16:47 < kaarposoft> can't promise to be there in IRC, but I will try... |
|---|
| 442 | 16:48 < dgollub> ok it's a thursday |
|---|
| 443 | 16:48 -!- kaarposoft [n=kaarposo@x1-6-00-1c-f0-f7-76-a0.k592.webspeed.dk] has left #opensync ["Bye"] |
|---|
| 444 | 16:49 < dgollub> Next IRC Meeting: 2009-01-08T09:00:00Z |
|---|
| 445 | 16:49 < dgollub> Minutes taker? Backup? |
|---|
| 446 | 16:50 * ChrisH on vacation then.... offline. |
|---|
| 447 | 16:51 < dgollub> kk .. Minutes: TBA |
|---|
| 448 | 16:51 < bricks> I can take care of the meeting next time |
|---|
| 449 | 16:51 < dgollub> ok cool |
|---|
| 450 | 16:51 < dgollub> Minutes: bricks Backup: dgollub |
|---|
| 451 | 16:51 < dgollub> meeting closed |
|---|
| 452 | 16:51 < bricks> ok |
|---|