| *** zbenjamin_ is now known as zbenjamin | 01:33 | |
| dcaliste | Hello chriadam, how are you ? Thanks for the last reviews and merges. | 06:58 |
|---|---|---|
| chriadam | hi dcaliste, I'm well thanks | 06:59 |
| chriadam | how was your week? | 06:59 |
| dcaliste | Fine, I destroyed my shared calendar yesterday, have to restart from backup ! I synced with a small window which removed my events on device and then I resync with a bigger window which result in deleting the events on servers because deleted events on previous synced were seen as local deletion… | 07:02 |
| chriadam | whoa | 07:02 |
| chriadam | hmm. I see the issue. we need to mark those events as "deleted due to sync" rather than "deleted due to user action" | 07:03 |
| chriadam | so that subsequent syncs don't handle those deletions in the wrong way and push them up to the server | 07:04 |
| dcaliste | I'll try to see what I can do not to see the deletion at a sync because deleted from server, not to see them as local eletion on next sync. | 07:04 |
| dcaliste | Exactly. | 07:04 |
| dcaliste | A trick would be to set the deletion date before the sync date. | 07:04 |
| dcaliste | Then, they will not appear in the deleted list on next sync. | 07:05 |
| dcaliste | Otherwise, put a flag in a comment. | 07:05 |
| dcaliste | Initially, I synced from a small window because I was doing test to solve once and for all the bug MER#1835 | 07:06 |
| chriadam | perhaps not comments | 07:11 |
| chriadam | but could use the flags field | 07:11 |
| chriadam | there might be a valid flag we could set there (or define a new one) | 07:12 |
| dcaliste | flag field is scratched when deleted. | 07:12 |
| dcaliste | :( | 07:12 |
| chriadam | dang | 07:13 |
| chriadam | I wonder why.. | 07:13 |
| chriadam | hrm. | 07:13 |
| dcaliste | It's to save space I guess in the database, but anyway deleted events are kept ad vitam eternam, so it's useless I think. | 07:14 |
| dcaliste | I would like to change mkcal in that sense a bit in medium term I think. | 07:14 |
| dcaliste | Having a way to mark event as deleted, without removing entries in custom properties and recursive… | 07:15 |
| dcaliste | And another way to purge the databse when the deleted event has been deleted upstream also. | 07:15 |
| dcaliste | Like that, we can remove this comment hack and we can reduce the database size when event are deleted. | 07:16 |
| chriadam | right, that sounds like a better direction than what exists currently | 07:16 |
| dcaliste | Yeh, but medium term, because it's a change in API and a good amount of changes in sqlitestorage from mkcal. | 07:17 |
| chriadam | indeed | 07:17 |
| dcaliste | I'm still in the brainstorm part of what API should I need and from which objects. | 07:17 |
| dcaliste | I think, from API point of view a purgeIncidence() from extendedstorage.cpp (implemented properly in sqlitestorage) would suffice. | 07:18 |
| dcaliste | But it's still early stage thinking. | 07:18 |
| chriadam | sure. maybe pvuorela has some thoughts there also | 07:19 |
| pvuorela | oh hi. i'm maybe missing a bit context here :) | 07:21 |
| dcaliste | Yeh, I can come with a proposal on paper later in mkcal. I'll add pvuorela in the box. | 07:21 |
| dcaliste | pvuorela: hello ! It's about deleted event in mkcal. | 07:21 |
| dcaliste | They are kept in database, except their properties like custom flags, or recursive… | 07:22 |
| dcaliste | I would like to change this and have two possibilities: | 07:23 |
| dcaliste | - one, mark an event as (locally) deleted, without touching its properties. These events are then listable by the existing deletedIncidences(). | 07:23 |
| dcaliste | - two, purge events from database completely (including properties), usefull when events have been deleted upstream and we want to get rid off them. | 07:27 |
| dcaliste | Currently _all_ events (without their properties, but with the full set of the components table) are kept forever. | 07:28 |
| dcaliste | pvuorela: that's the medium term reflexion I have on mkcal evolution… | 07:29 |
| pvuorela | hm, could make sense. it seems a bit weird that you cannot (?) remove events so that there's no kind of trace left. | 07:31 |
| pvuorela | kcalcore upstream wouldn't have anything related? | 07:32 |
| pvuorela | (away for a short while->) | 07:34 |
| dcaliste | No, I think it's related to the storage, so it's mkcal only, but I'm not 100% convinced, because kcalcore is providing API for storage (but no impl)… | 07:34 |
| dcaliste | Once again, I need to investigate this further indeed. | 07:34 |
| chriadam | that's my understanding too, but I could be wrong | 07:34 |
| chriadam | yep. anyway, can discuss further later on I guess :-) | 07:35 |
| dcaliste | chriadam: indeed, I need to continue to think about it before arriving with a consistent proposal. | 07:36 |
| dcaliste | pvuorela: if you look at sqlitestorage.h where you have the SQL commands for the database, you can see that there is no DELETE FROM components entry. | 07:36 |
| dcaliste | The delete components macro is indeed doing an update on the components table. | 07:37 |
| dcaliste | chriadam: besides, I've pushed an additional commit on top of caldav !45 | 07:37 |
| dcaliste | This commit is removing code related to test for possible modification due to etag update. | 07:38 |
| dcaliste | Now that last modified date can be changed with mkcal being updated to 0.5.6. | 07:38 |
| chriadam | ah right - jpetrell agreed that bree could test that PR#45 so I will ask her to spend some time on that this week | 07:39 |
| chriadam | if she has availability for that one | 07:39 |
| dcaliste | Still need to be tested further but simple tests of locally adding an event and sync it (thus add etag) does not result of this event being seen as modified on next sync. | 07:39 |
| dcaliste | Thanks for testing this internally also. | 07:39 |
| chriadam | hopefully we get that one in before next meeting (if you think it's ready, and our testing doesn't show any issues)? | 07:40 |
| dcaliste | I need to test now if I modify a recursive event, or add an exception if everything is working expectedly. | 07:40 |
| chriadam | ok | 07:41 |
| pvuorela | (back) | 07:41 |
| dcaliste | chriadam: MR!45 will conflict with the direction one, but I can rebase easily after one of them will be accepted. | 07:42 |
| chriadam | cool, thanks very much | 07:42 |
| dcaliste | pvuorela: see my named comment previously, mainly I'm starting this brainstorm about deleted events in mkcal and I'll come back to you when I have a paper proposal for API changes and we can continue to discuss there. Today was mainly to share these very early thoughts about this topic. | 07:43 |
| dcaliste | pvuorela: by the way, if you have time at one moment to look at https://git.sailfishos.org/mer-core/python3/merge_requests/11 (it's reenabling GPLv3 readline in a separatepackage for Python), that would be great, thanks in advance. | 07:46 |
| pvuorela | dcaliste: alright. for kcalcore was pondering about calendar.h having deleteEvent() and deletedEvent(). | 07:46 |
| dcaliste | pvuorela: yeh, but calendar is just a representation of the data, then the storage can map this as they want and it seems to me that sqlitestorage decided to just mark the event as deleted (so the extendedstorage::deletedIncidences()) can work. | 07:47 |
| pvuorela | dcaliste: sure. on python i'm not following the pip part. only commenting out some lines for packaged files, but no configuration changes? | 07:48 |
| dcaliste | But when marked as deleted, the custom properties are deleted and thus the event listed by deletedIncidences() have only partial information… | 07:48 |
| dcaliste | pvuorela: yeh, strange, but on SDK it didn't compile out of the box (imho). Pip has been disabled in a previous commit but files are still packaged ! | 07:49 |
| dcaliste | chriadam: didn't have time this week to go further for the email folder sync MRs… | 07:50 |
| pvuorela | dcaliste: hm, builds in obs. | 07:50 |
| dcaliste | pvuorela: yeh, that's why I call it strange, I cannot figure out where these files come from in OBS. | 07:51 |
| pvuorela | yea, strange but think that needs to be figured out before merging. for pip i think we should be having a subpackage for that. | 07:53 |
| dcaliste | pvuorela: I can remove the pip stuff which is strange for the moment I agree. Particularly, now that I look again at it, indeed pip has been removed earlier (adding --without-ensurepip) but it has been reenabled after because this without flag is not here anymore. | 07:56 |
| dcaliste | So I propose to remove the pip commit from the MR and I'll retry to see why it's not compiling out of the box in SDK and propose later another PR if something is missing (maybe in the build require). | 07:57 |
| pvuorela | sounds good | 07:58 |
| dcaliste | chriadam: I think that's all for mkcal/buteo for this week, thanks for the meeting. | 07:59 |
| dcaliste | pvuorela: great thanks. | 07:59 |
| chriadam | dcaliste: thanks very much for your effort as always | 07:59 |
| chriadam | if there's any particular MRs which need further attention, please don't hesitate to ping me | 08:00 |
| chriadam | I will ask Bree to test the two caldav PRs this week if possible | 08:00 |
| chriadam | other than that, have a great week! | 08:00 |
| dcaliste | Thanks chriadam, have a nice week too. | 08:01 |
| dcaliste | pvuorela: MR!11 in python3 updated (removed pip commit and MR message). I'm investigating the SDK building issue. | 08:02 |
| pvuorela | dcaliste: nice, thanks! | 08:03 |
| dcaliste | pvuorela: the ensurepip issue is that pip is installed in SDK already and the ensurepip module is doing nothing in that case. See MR: https://git.sailfishos.org/mer-core/python3/merge_requests/12 | 09:42 |
| dcaliste | See for instance this (closed) issue https://github.com/pypa/pip/issues/6355 | 09:43 |
| pvuorela | ok | 09:43 |
| dcaliste | Some users would like to be sure that ensurepip is looking only in the ROOTDIR when --root is given (like for us when building package), but some say that it's inconsistent with Python module discovery… | 09:44 |
| dcaliste | I'm applying one of the user proposal in related https://github.com/pypa/pip/issues/3063 to use the --ignore-installed option of pip. | 09:44 |
| dcaliste | But this option is not available in ensurepip, so there is a patch in the MR to add this option to ensurepip and apss it in the Makefile. | 09:45 |
| *** vals_ is now known as tango_ | 10:30 | |
| adisbladis | Anyone who happens to have a copy of Sailfish 3.0.3.10 for the Gemini? | 13:23 |
| r0kk3rz | TheKit: ^^ | 13:52 |
| TheKit | adisbladis, you mean flashable image? you can flash image from Jolla and OTA update it to 3.0.3.10 | 13:56 |
| adisbladis | TheKit: Flashable image. I can only download 3.1.0.11 | 13:56 |
| adisbladis | I'm trying to work around https://together.jolla.com/question/212150/sailfishx-gemini-version-31011-no-sound/ | 13:57 |
| adisbladis | TheKit: I guess some older version would also be fine as long as I can OTA from that | 14:12 |
| *** frinring_ is now known as frinring | 17:58 | |
Generated by irclog2html.py 2.17.1 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!