| *** zbenjamin is now known as Guest75149 | 02:33 | |
| *** zbenjamin_ is now known as zbenjamin | 02:33 | |
| *** arza- is now known as arza | 03:59 | |
| *** haasn` is now known as haasn | 03:59 | |
| *** frinring_ is now known as frinring | 04:02 | |
| *** albertuxone is now known as albertux | 07:23 | |
| dcaliste | Hello chriadam, could you give me 10 more minutes before our meeting please ? | 08:06 |
|---|---|---|
| chriadam | hi dcaliste, yes of course | 08:06 |
| chriadam | no rush | 08:07 |
| dcaliste | Finally, I'm ready. And reading your comment on buteo-syncfw#59. It's definitely valid. I'll has some sync code on startup. This will also provide a bonus point for the transition : users won't have to visit the account setting page to actually sync the two enable properties. | 08:17 |
| dcaliste | I'll add not I'll has... sorry. | 08:17 |
| chriadam | thanks :-) | 08:18 |
| chriadam | sorry that I didn't think of that case previously | 08:18 |
| chriadam | I know that PR is dragging on a bit | 08:18 |
| chriadam | flypig: have you had a chance to take a look at that one, by any chance ^ | 08:18 |
| dcaliste | No problem, it will be much better with your suggestion anyway. | 08:19 |
| flypig | I'll need to check, but I doubt it. | 08:19 |
| chriadam | flypig: it involves some things you've looked at in the past (e.g. signals from accounts-glib etc) so your opinion would be very helpful, if you get time to take a look | 08:20 |
| flypig | Yeah, I've found it now, and I'm afraid I haven't looked at it, although I'm sure you asked. | 08:21 |
| chriadam | :-) | 08:21 |
| dcaliste | flypig, don't worry, it's not really urgent. | 08:21 |
| chriadam | dcaliste: aside from that one, I see pvuorela_ merged the kf5-calendarcore PR (thanks for handling that one) | 08:22 |
| flypig | I'd be happy to take a look, will do. | 08:22 |
| chriadam | flypig: thanks very much | 08:22 |
| dcaliste | chriadam, yes and https://git.sailfishos.org/mer-core/buteo-sync-plugin-caldav/merge_requests/78 is resulting from it. | 08:22 |
| dcaliste | flypig: indeed thank you. | 08:22 |
| chriadam | thanks, I'll check that one | 08:22 |
| chriadam | was there a bug number associated with https://git.sailfishos.org/mer-core/kf5-calendarcore/merge_requests/3/diffs | 08:25 |
| chriadam | hmm, I can't see one, yet the tag triggered and built in OBS | 08:27 |
| chriadam | anyway, I will create a bz bug for the lastModified thing, sec | 08:27 |
| pvuorela_ | think i did an annotated tag. | 08:27 |
| dcaliste | Yeh, I'm afraid not. It's not related to a report on the forum or a bug. | 08:27 |
| dcaliste | If you create one, chriadam, I can add it to the commit message in CalDAV MR. | 08:28 |
| chriadam | created JB#53475 for that one | 08:33 |
| chriadam | thanks very much. we will look into doing similar for google and exchange if needed | 08:33 |
| chriadam | regarding the QMF things: I will do the qmflist_p.h into qmfclient thing tonight | 08:34 |
| chriadam | after that, I think we're getting really close to trying to integrate all those changes | 08:35 |
| dcaliste | Yes, and I'm preparing a MR in CalDAV to migrate the storage of URL and etag from comments to setUrl() and volatile custom property respectively. But doing this noticed that I was doing more and more lastModified(), update, setLastModified() calls. | 08:35 |
| chriadam | I must be mis-remembering, but I thought the setUrl thing was already done | 08:37 |
| chriadam | or maybe that was for notebook and not event | 08:37 |
| dcaliste | Great about QMF. I still need to do a big grep in the code to catch all the possible deserialisation issues between 32 bit counters and 64 ones. I guess on my JollaC it won't be a big issue, but aarch64 ones are coming, aren't they ? | 08:37 |
| chriadam | I assume so | 08:37 |
| dcaliste | In CalDAV, the URL for each incidence is still saved as a comment. | 08:37 |
| chriadam | ah true. thanks for looking into that! | 08:38 |
| dcaliste | For historical reason when custom properties were trashed when an incidence was marked as deleted. | 08:38 |
| chriadam | ah, that's right. | 08:38 |
| dcaliste | Now, they are not trashed anymore, except when the incidence is actually purged from the database. | 08:38 |
| chriadam | flypig: any ETA on when you might get a chance to take a look at the upstream QMF PRs? is it likely to be this week, or next week? | 08:39 |
| dcaliste | But also, upstream introduced a new incidence property to save URL for shared events. It was not available before the transition to KCalendarCore. | 08:39 |
| chriadam | ah, nice | 08:39 |
| chriadam | hmm, interesting thought... | 08:39 |
| chriadam | when you say a "shared event" do you mean an event which exists in multiple calendars on the server? | 08:40 |
| chriadam | is the URL property then storing some canonical URL? or? | 08:40 |
| dcaliste | And I've added it to mKCal some MRs ago, so now it should be ready to be used in CalDAV and clean a bit how we save and store incidence properties relqted to sync qctions. | 08:40 |
| dcaliste | Url for an incidence is single, from the doc, it says that it is used to save a remote location for the incidence. | 08:41 |
| dcaliste | So myself using shared was a bit of an abuse. I was meaning "shared" between device and a single remote server. | 08:41 |
| chriadam | ah cool | 08:41 |
| chriadam | excellent, thanks. | 08:42 |
| dcaliste | Yes, ideally, I would like to arrive to a point where we can call ICalFormat::toString(incidence) without first filtering out CalDAV sync related changes or additions. | 08:44 |
| chriadam | that would indeed be nice | 08:44 |
| dcaliste | After the next MR migrating the url and etag (and some other things), only the UUID changes will remains. But I've no idea at the moment how to change this in a backwrd compatible way. Anyway, let's first clean the url and etag storage. | 08:45 |
| chriadam | the "real" solution would be to allow storing same event in multiple notebooks, I guess | 08:45 |
| chriadam | then we wouldn't need the prefixing stuff | 08:45 |
| chriadam | I know this is something that pvuorela_ has considered in the past | 08:46 |
| dcaliste | Exact but for this, I need to alter the DB and I would like to avoid this at the moment. | 08:46 |
| *** albertux is now known as albertuxone | 08:46 | |
| chriadam | yes, I agree | 08:46 |
| chriadam | on QMF front: I'll hopefully update the qmflist_p.h PR tonight, then try to review any of the earlier PRs which I haven't yet +2'd later this week (or more likely, early next week). hopefully flypig gets a chance to review also. then we can try an integration mid/late next week. | 08:47 |
| chriadam | (if you get a chance to look into the serialisation potential issue further before then, great - but if not, all good, we can address that issue later in any case, I think) | 08:47 |
| flypig | I dropped out for a bit. For info, I plan to look at the QMF PRs today. I'm not sure I'll get through them all though. | 08:47 |
| chriadam | oh, excellent! thanks | 08:47 |
| dcaliste | Good, good. I'll try to look into the desrialisation case before Friday. | 08:48 |
| chriadam | tyvm | 08:48 |
| dcaliste | Thank you flypig. | 08:48 |
| flypig | dcaliste, as an asie thanks for the FingerTerm PR changes. I'll take a look at that too. | 08:48 |
| chriadam | I need to poke blam to +2 my QtPIM PRs also | 08:48 |
| *** albertuxone is now known as albertux | 08:48 | |
| flypig | asie -> aside | 08:49 |
| chriadam | dcaliste: unrelated, but you might be interested to see https://git.sailfishos.org/mer-core/buteo-sync-plugin-carddav/merge_requests/30 related to what I mentioned last week. seems like just a silly bug by me, fortunately. easy to fix :-) | 08:49 |
| dcaliste | flypig, thank for the idea and the intent to look at it again. If this ncurses could be solved at last, it is great. | 08:49 |
| chriadam | the redirect wasn't truly circular after all, as it was a http->https (or https->http, I don't remember which) redirect. | 08:49 |
| chriadam | was there anything else to discuss to tonight? have I forgotten anything I needed to follow up on from last week? | 08:53 |
| dcaliste | chriadam, ok, I see. You wanted to save the redirected URL. I need to look at the QUrl doc, but could a orig != redir be equivalent to orig.path != redir.path || orig.scheme != redir.scheme ? | 08:53 |
| dcaliste | chriadam, no that's all for this week I think. I've a MR to upgrade Calligra to the latest upstream, but I'll discuss this with pvuorela in the comments. | 08:54 |
| chriadam | good point about the equality check, as what is there wouldn't cover the port redirect case either, I suppose | 08:55 |
| chriadam | I have added a comment, will think more about that | 08:55 |
| chriadam | the problem is that we want to avoid possibly unsafe redirects (e.g. to different domain) | 08:56 |
| dcaliste | Ah yes, redirection to another domain would be nasty, indeed. | 08:57 |
| chriadam | so I'm not sure we want to do a normal equality comparison there, in case the host() changes, not just path() or scheme() or port() | 08:57 |
| chriadam | anyway, thanks for raising that - I will look at what can be done to improve | 08:58 |
| chriadam | dcaliste: one final thing: | 08:59 |
| chriadam | flypig is hopefully going to be doing a bi-weekly information thing | 09:00 |
| chriadam | not sure if he's spoken to you about this already? anyway, we were thinking it might be nice if we included that, some brief sentence or two about various contributions during the last two week period etc | 09:00 |
| chriadam | wondering if you'd be ok with that? (you wouldn't have to write anything, we'll do that) | 09:01 |
| chriadam | but basically just summarising what the recent contributions from the community have been, which community members made the contribution, what areas it affected, what the outcome will be, etc. | 09:01 |
| dcaliste | Sure, I've read the two first iterations on the forum with great interest. That's a very nice and welcome initiative in my opinion to difuse information about the OSS parts. It may end up attracting more contributions. | 09:02 |
| chriadam | oh have a couple of them already been posted? I didn't know haha. | 09:03 |
| chriadam | anyway, great :-) | 09:03 |
| dcaliste | You proposed to do it, but I can send him some sentences of previous work from last two weeks regularly. | 09:03 |
| dcaliste | Which day of the week would the best flypig ? | 09:03 |
| chriadam | I mean, that would be even better - not depending on me remembering to do it haha. | 09:04 |
| chriadam | maybe could event just paste something here on Tuesdays, and flypig could copy it | 09:04 |
| flypig | Thanks both. It's supposed to go out on Wednesday/Thursday, and I tend to write the first draft over the weekend. So, I'll take whatever I've got at the weekend in the first instance. | 09:05 |
| flypig | If you could paste it on Tuesday, then I'll add it in. That would be really great. | 09:06 |
| dcaliste | Ok, I can prepare something for Tuesdays (when there is something relevant of course). I'll paste here something about what we've done with attah recently about compiling calligraconverter for SeaPrint to converter documents on the fly to PDF for the printer. | 09:07 |
| dcaliste | s/converter/convert/ | 09:07 |
| flypig | That sounds just the sort of thing. Thank you. | 09:08 |
| chriadam | thanks very much! | 09:08 |
| chriadam | ok, if nothing else, let's close the meeting. thank you again for your time and effort, it is greatly appreciated as always! | 09:09 |
| * chriadam -> away, gnight | 09:09 | |
| dcaliste | flypig, you can modify or repkrase anyting of course : | 09:35 |
| dcaliste | attah who is developping SeaPrint, bringing printing to SailfishOS, is planning to extend document support from PDFs to other available formats like ODT, https://forum.sailfishos.org/t/seaprint-network-printing-for-sailfishos/796/77 | 09:35 |
| dcaliste | One possibility is to reuse the document library that the OS is using : Calligra. It's a KDE project for desktop use but versatile enough to be reused as a backend for the document application in SailfishOS. It is providing a converter but it seems that this tool was suffering issues and was not packaged for SailfishOS. | 09:35 |
| dcaliste | dcaliste corrected the issues upstream https://invent.kde.org/office/calligra/-/merge_requests/19 and inclusion inside SailfishOS is under review https://git.sailfishos.org/mer-core/calligra/merge_requests/56 | 09:35 |
| dcaliste | Still on document front, ApB provided a document making the application crashes, Leinir (who initially ported Calligra to SailfishOS and created the QML components used by the application) accepted the correction upstream to avoid the crash https://invent.kde.org/office/calligra/-/merge_requests/20 Inclusion into SailfishOS is still pending. | 09:35 |
| flypig | Great, thank you dcaliste. I've added it to the next update. In future, if you prefer to write it in the first person, then I can include it as a "quote" attributed to you. Otherwise I can just adjust it to flow in with the rest of the text. So, your choice :) | 10:11 |
| *** albertux is now known as albertuxone | 10:26 | |
| *** albertuxone is now known as albertux | 10:27 | |
| *** albertux is now known as albertuxone | 11:10 | |
| *** albertuxone is now known as albertux | 11:12 | |
| *** albertux is now known as albertuxone | 11:20 | |
| *** albertuxone is now known as albertux | 11:21 | |
| *** vup2 is now known as vup | 13:06 | |
| *** albertux is now known as albertuxone | 13:06 | |
| *** albertuxone is now known as albertux | 13:06 | |
| *** albertux is now known as albertuxone | 13:59 | |
| *** albertuxone is now known as albertux | 14:00 | |
| *** vilpan is now known as Guest14617 | 15:41 | |
| *** Guest14617 is now known as vilpan | 15:42 | |
| *** albertux is now known as albertuxone | 16:26 | |
| *** albertuxone is now known as albertux | 16:27 | |
| *** SpeedEvil is now known as Guest53746 | 20:25 | |
| whodat | sup | 22:02 |
| *** SpeedEvil is now known as Guest21051 | 22:56 | |
Generated by irclog2html.py 2.17.1 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!