| *** zbenjamin is now known as Guest93573 | 01:32 | |
| *** zbenjamin_ is now known as zbenjamin | 01:32 | |
| *** frinring_ is now known as frinring | 02:02 | |
| *** nyov is now known as Guest47360 | 03:11 | |
| *** JvD_ is now known as jvd | 03:58 | |
| *** PaulePan1er is now known as PaulePanter | 06:11 | |
| dcaliste | Hello chriadam, how are you ? | 06:49 |
|---|---|---|
| chriadam | hi dcaliste | 06:50 |
| chriadam | I'm well thanks | 06:50 |
| chriadam | how are you? | 06:50 |
| dcaliste | I'm fine thanks. | 06:51 |
| chriadam | that's good. I hope your week went well | 06:51 |
| dcaliste | I think you received a mail also from vaskas on TJC with a failing log. | 06:51 |
| dcaliste | It was a slow sync that was failing because of the issue I introduced with malformed recurring events. | 06:52 |
| dcaliste | You patched this already. But there was another bug, with his failing calendar duplicating at each sync. | 06:53 |
| chriadam | I did see that earlier today, yes, thank you for already following up with that one | 06:53 |
| dcaliste | I've submitted a MR for this : https://git.sailfishos.org/mer-core/buteo-sync-plugin-caldav/merge_requests/66 | 06:53 |
| chriadam | thank you, I will check that | 06:54 |
| dcaliste | It was a stupid mistake (as always), not setting the remote path at creation time, and deferring it to the last moment, to save one line :/ | 06:54 |
| dcaliste | But code in between may fail and quick return. | 06:54 |
| chriadam | thank you for investigating, definitely good to get that fixed :-) | 06:55 |
| dcaliste | I've send a compiled rpm to the guy of the master branch with the addition of my read-only branch, the one with the new discovery code. Up to now, he seems happy with it. | 06:57 |
| chriadam | unfortunately I wasn't able to spend much time following up on your other PRs (or even my own) this last week (holidays + being drawn across to a different task, meant that I had limited time) | 06:57 |
| chriadam | I apologise for not progressing those | 06:58 |
| dcaliste | chridama, no problem, we can just quickly review what is still left today. | 06:58 |
| chriadam | sounds good! | 06:59 |
| chriadam | Bea's buteo-syncfw change was merged, so I will merge your result change (MR#45) this week | 06:59 |
| chriadam | the caldav change - should be fine to merge, thank you for addressing the "no calendar list" thing | 07:00 |
| chriadam | I will merge that this week also | 07:00 |
| dcaliste | So, talking about the discovery MR in CalDAV, I've added a commit on top to change the detection of the account service, not based on calendars entry any more but on the profile_id directly, which sounds more intentional. | 07:00 |
| dcaliste | Ah, ok, you've seen it ;) | 07:00 |
| chriadam | kcalcore MR#15 iirc had some open discussion item | 07:00 |
| dcaliste | Yes, I've tried to reproduce the problem faced by flypig, without success at the moment. But I was trying to reproduce it on a slightly simplified setup, which may be the cause for not seeing it. | 07:02 |
| flypig | dcaliste, ah, sorry, I thought you could reproduce it. | 07:02 |
| chriadam | I asked flypig if he could provide a testcase for that one, but it might be tricky to do in isolation :-/ not sure | 07:02 |
| dcaliste | About this !15, it is passing the new test you introduced in mkcal for it, but flypig has this strange behaviour which should be understood first in my opinion. | 07:02 |
| chriadam | yeah, definitely be good if it could be isolated | 07:03 |
| dcaliste | flypig, no not up to now, sorry. And I've the same excuse as chriadam ;op, Easter holidays and various other things to do... | 07:03 |
| chriadam | flypig: was NQPC MR#55 different? was there some other issue there? | 07:04 |
| dcaliste | But this issue is really itching : how can the qml bindings returns invalid timedate depending on startup, even without sync going on. | 07:04 |
| flypig | dcaliste, okay, sorry I hadn't realised. I'll put some effort into trying to figure out what's going on then + creating an EAS test case. | 07:04 |
| chriadam | excellent, thanks | 07:05 |
| chriadam | also, @flypig if you could check nqpc MR#55 that would be appreciated. notebook visibility related | 07:05 |
| flypig | dcaliste, yes, it's on my list, I didn't get around to it yet, sorry. | 07:06 |
| dcaliste | flypig, if yoy try to reproduce, it may be enlightening to dump the component database entry that is displaying strangely (test5 ?) before and after each startup in the no network conditions. | 07:06 |
| dcaliste | Just to exclude any modification of it (I cannot figure out from where, but to be sure). | 07:07 |
| flypig | Okay, noted, I'll try to do that. | 07:07 |
| dcaliste | Then, the idea is to try to think what can make the actually returned value change while store value is not. | 07:07 |
| dcaliste | So you may want to dump the originoftime value from the db also. | 07:08 |
| dcaliste | And maybe the timezone part also, to be sure. | 07:08 |
| dcaliste | Thank you very much, I know it's a long and not fun job to setup all of this ... :( | 07:08 |
| flypig | I'll do my best. It is tricky I'm afraid, just because it's timeconsuming, but I just need to apply a bit of effort :) | 07:09 |
| flypig | chriadam, where you write "was NQPC MR#55 different?" could you elaborate please? | 07:10 |
| chriadam | soryr, I meant, unrelated to that | 07:10 |
| chriadam | (unrelated to the eas one) | 07:11 |
| chriadam | dcaliste: I guess that covers the current PRs unless I'm forgetting somethign? | 07:12 |
| flypig | Ah, sorry, I'm still confused. Was NQPC#55 unrelated to which? | 07:12 |
| dcaliste | chriadam, for nemo-qml-plugin-calendar!55, there is the issue of checking that Google is adding the profile_id setting properly for the calendar service. | 07:12 |
| chriadam | ah, yes | 07:13 |
| dcaliste | Is this to be done in the public repo ? I may give a loom then. | 07:13 |
| chriadam | flypig: nqpc#55 is related to the notebook visibility changes you worked on something related, previously (alarms IIRC) | 07:13 |
| dcaliste | s/loom/look/ | 07:13 |
| chriadam | dcaliste: yes, I believe just in buteo-sync-plugins-social google-calendar | 07:14 |
| dcaliste | Ok, I've no experience with this particular code, but I can give a look. | 07:15 |
| dcaliste | Is there any other calendar service to be known ? Like EAS or something ? | 07:15 |
| chriadam | EAS is the only one which isn't open source afaik | 07:15 |
| dcaliste | Ok, so one should just check that the calendar service has a foo/profile_id setting that is set to the value of the Buteo profile. | 07:16 |
| chriadam | there are a few in buteo-sync-plugins-social (facebook, vk, as well as google) but facebook at least can be ignored (they removed calendar support for third parties), and vk... well, I would be surprised if vk still worked, since no-one tests/uses that afaik | 07:16 |
| dcaliste | For caldav it's something like caldav-sync-XX. | 07:16 |
| chriadam | right | 07:16 |
| chriadam | although I thought we were talking about a field in the mkcal database | 07:17 |
| chriadam | rather than an account setting. did I misunderstand? | 07:17 |
| dcaliste | It should be in both place. | 07:17 |
| dcaliste | The mKCal notebook has a syncProfile() method, returning the name of the sync profile used by this notebook (if properly stored). | 07:18 |
| chriadam | ah. | 07:18 |
| dcaliste | And the account has a setting for the calendar service that is called caldav/profile_id for caldav, storing the same value. | 07:19 |
| dcaliste | So having the notebook, we can link it to the account (via the account() method) and the service (via the profile_id), and vice-versa. | 07:19 |
| chriadam | well, I guess the account setting will be set by jolla-settings-accounts or something, but I assume it does so correctly. IMO the one which might not be set correctly, would be in mkcal's Notebook data, i.e. set by the google-calendars-plugin | 07:19 |
| chriadam | or maybe I'm just confused. I can't promise to look into it this week, but hopefully next week, unless you do so prior | 07:20 |
| dcaliste | Yes,in my device, it's the one stored in the mkcal db that is missing. | 07:20 |
| dcaliste | (if I remember well). | 07:20 |
| chriadam | but I will try to merge the caldav PRs and the buteo-syncfw PR this week | 07:20 |
| dcaliste | I'll look at both places for Google and check that it's coherent, like in the caldav implementation. | 07:21 |
| chriadam | thanks | 07:21 |
| dcaliste | There us one last minor MR : https://git.sailfishos.org/mer-core/messagingframework/merge_requests/39 | 07:21 |
| dcaliste | About silencing the last bits of messages about all accounts not being email ones. | 07:22 |
| dcaliste | s/us/is/ | 07:22 |
| chriadam | why did the patch 0004 change so much? | 07:23 |
| chriadam | oh, did you edit that patch rather than add a new patch on top? | 07:24 |
| dcaliste | I regenerated all the patch with git format-patches, and git changed the hashes length, so there are changes in the patch for every sections. | 07:25 |
| dcaliste | The patch I edited was the one introducing the account support. | 07:25 |
| dcaliste | I think gitlab is confused because there are more changes than the patch0004... | 07:26 |
| chriadam | gitlab is definitely confused | 07:26 |
| chriadam | yup, I'll take a look at that one too. might need a bug or tjc to pass CI though | 07:26 |
| dcaliste | Ah, no my bad... | 07:26 |
| dcaliste | I actually changed just this one ;) | 07:27 |
| chriadam | I wonder if it might be better to just add a new patch on top | 07:27 |
| dcaliste | But most of the changes are for hashes length. | 07:27 |
| chriadam | i.e. 0022-Suppress-warnings.patch etc | 07:27 |
| chriadam | I dunno whether this helps, but my way of working with patches is like this: | 07:29 |
| chriadam | ah, let me pastebin | 07:29 |
| dcaliste | Well, I did this because I'm thinking because we may would like to submit this upsteam, if not under this form, and I'm afraid we may miss patches . | 07:29 |
| chriadam | https://pastebin.com/V11w20Af | 07:30 |
| dcaliste | But I can do as you prefer of course. | 07:30 |
| chriadam | but, true, upstreaming might affect this | 07:30 |
| chriadam | pvuorela: do you have time to check that one, also? related to qmf | 07:31 |
| dcaliste | I did modify the existing patch, because I wanted to actually modify the code introduced by it, and not any of the upstream one. | 07:31 |
| dcaliste | I commented in the MR introduction the various changes that I introduced in the patch. | 07:32 |
| chriadam | true. I guess it doesn't matter too much. And there aren't too many changes, seems pretty simple. | 07:33 |
| chriadam | thanks for that - we will have to figure out a bug number to associate it with, though, or it won't pass CI | 07:33 |
| dcaliste | If for both of you chriadam and pvuorela, it's better to have it a another patch on top I can regenerate it. And merge it with patch 0004 after review. | 07:33 |
| bionade24 | I can't connect to qmllivebench, it always tells me the emulator is offline. Can someone please help me? | 07:33 |
| dcaliste | If you have an ongoing task id for calendar cleanup, it will be just fine. | 07:34 |
| chriadam | checking, sorry taking a while, bz is slow | 07:37 |
| chriadam | maybe just use JB#24579 | 07:39 |
| bionade24 | No one who can help me with qmllivebench? | 07:39 |
| dcaliste | Ok, I'll add it to the commit message, thanks. | 07:39 |
| chriadam | dcaliste: thank you | 07:39 |
| chriadam | bionade24: one sec | 07:39 |
| dcaliste | bionade24, sorry, I'm not using it. | 07:39 |
| chriadam | martyone: is qmllivebench something you might be able to help with? | 07:40 |
| chriadam | dcaliste: thanks again. I had nothing else to discuss today - did you have any other topics? | 07:40 |
| flypig | bionade24, does it work with a real device? | 07:40 |
| martyone | yup | 07:41 |
| dcaliste | chriadam, no it's fine. Thank you for the meeting. I've added the JB id to the commit in QMF. | 07:41 |
| chriadam | dcaliste: thanks! I hope you have a great week | 07:42 |
| martyone | bionade24: when you click the red "offline" button, does it do anything? | 07:42 |
| * chriadam -> away, gnight all | 07:42 | |
| bionade24 | flypig: Haven't tried | 07:42 |
| bionade24 | martyone: No | 07:43 |
| bionade24 | flypig: Should I try? | 07:43 |
| martyone | I assume you are using latest SDK (3.0.7), which platform is it on? | 07:44 |
| bionade24 | martyone: Yes, of course. In can ssh into, too. | 07:46 |
| flypig | bionade24, I always preferred using it on a real device, but that might have been app-specific. | 07:47 |
| bionade24 | flypig: The problem is that I only have one device and I don't want to break it | 07:48 |
| flypig | bionade24, sure, that's an important benefit of using the emulator :) | 07:48 |
| martyone | bionade24: what is the platform you run it on? | 07:49 |
| bionade24 | martyone: Arch Linux. Does the emulator need some authenthication I could debug? | 07:49 |
| martyone | bionade24: try with debug info enabled, running the bench from console with something like `QT_LOGGING_RULES="QmlLive.Bench.ConnectionState.debug=true" ~/SailfishOS/bin/qmllivebench` | 07:50 |
| martyone | (the connection is unauthenticated) | 07:51 |
| bionade24 | martyone: | 07:52 |
| bionade24 | Debug: Host connection error: "Sailfish OS Emulator 3.2.1.20" QAbstractSocket::ConnectionRefusedError ((null):0, (null)) | 07:52 |
| bionade24 | Debug: Host disconnected: "Sailfish OS Emulator 3.2.1.20" ((null):0, (null)) | 07:52 |
| bionade24 | 10234/tcp open unknown | 07:53 |
| martyone | bionade24: there must be an app running inside the emulator with Qt QmlLive enabled - is that true? | 07:54 |
| bionade24 | martyone: Probably not, I couldn't read something about this in the wiki page. How do I do that? | 07:55 |
| martyone | ok, check the help in Qt Creator - there is a section "Using QmlLive with Sailfish OS devices", you can find it with from the locator by typing "? qmllive" thre | 07:57 |
| bionade24 | martyone: Thx. | 07:58 |
| martyone | np | 07:58 |
| bionade24 | martyone: And after that i enabled qmllive I need to deploy once? | 08:05 |
| martyone | yup | 08:07 |
| bionade24 | Next Problem: What can I do when the Build engine doesn't find a header lying in /usr/include. I always used pkgconfig but it doesn't seem to work this time. | 08:10 |
| Nico[m] | Is the header actually in the /usr/include of the target? Maybe you need to add its package to the build requirements | 09:01 |
| bionade24 | Nico[m]: The header is in /usr/include/jansson.h of the mersdk. Is this the wrong path? | 09:08 |
| Nico[m] | It depends, you need to look inside the build target not just the VM | 09:19 |
| bionade24 | Nico[m]: Where is the build target? | 09:20 |
| Nico[m] | I actually have no idea :D | 09:20 |
| bionade24 | Nico[m]: Or explain me a bit more what the build target and the vm differences | 09:20 |
| bionade24 | Nico[m]: You mean those sb2 commands? | 09:21 |
| r0kk3rz | sb2 target | 09:21 |
| r0kk3rz | which is itself a vm of sorts | 09:21 |
| bionade24 | How can I get headers into sb2? | 09:21 |
| Nico[m] | Install them via a package dependency | 09:22 |
| bionade24 | Nico[m]: You mean with pkgconfig? | 09:22 |
| bionade24 | I'm working with qtcreator | 09:23 |
| Nico[m] | You should have an rpm yaml like this: https://nheko.im/nheko-reborn/spoon/-/blob/master/rpm/harbour-spoon.yaml#L21 | 09:25 |
| Nico[m] | Just add your dependencies there | 09:25 |
| Nico[m] | Either via the pkgconfig name under the ConfigBR or the rpm name under PkgBR | 09:26 |
| bionade24 | Nico[m]: Thx, I'm a total noob. But how does sb2 install the packages. I need to add my obs repo | 09:31 |
| Nico[m] | It installs them when building, but if you need an additional repo, that is a bit more involved | 09:32 |
| bionade24 | Nico[m]: I already installed the packages in the vm | 09:33 |
| Nico[m] | Yeah, but you need to install them in the build tools, via sb2 commands | 09:34 |
| r0kk3rz | add the repo instead | 09:34 |
| r0kk3rz | also, dont use yamls, just delete it and write the spec file | 09:34 |
| bionade24 | r0kk3rz: So I have to invoke zypper addrepo? | 09:34 |
| r0kk3rz | ssu ar | 09:35 |
| r0kk3rz | inside the target | 09:35 |
| Nico[m] | sb2 -t SailfishOS-3.0.2.8-i486 -m sdk-install -R ssu ar | 09:37 |
| Nico[m] | or similar | 09:37 |
| bionade24 | Repository http://repo.merproject.org/obs/home:/oroesler/sailfish_latest_i486/home:oroesler.repo does not contain valid URL, skipping and disabling. | 09:40 |
| bionade24 | I got it !! This was fu**ing complicated | 09:50 |
| bionade24 | Nico[m]: r0kk3rz: Thank you for helping me that much | 10:09 |
| *** vup2 is now known as vup | 12:23 | |
| *** Malinux- is now known as Malinux | 15:21 | |
Generated by irclog2html.py 2.17.1 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!