| tqg | hello,does anyone know how to get the "droid-make" command? | 05:55 |
|---|---|---|
| tqg | I am building droidmedia,but can't find the command | 05:57 |
| chriadam | it might be discussed in the HADK docs https://sailfishos.org/content/uploads/2021/06/SailfishOS-HardwareAdaptationDevelopmentKit-4.1.0.0.pdf | 06:03 |
| tqg | chriadam:I have read this,but can't find it. | 06:08 |
| chriadam | well, I know nothing about it - probably better to ask in the porters channel. but I would naively assume that it is part of the android base which you build in the android build rootfs chroot. | 06:11 |
| tqg | chriadam:where is the porters channel.I think I need poters help TKS! :) | 06:18 |
| chriadam | it's listed on the website https://sailfishos.org/develop/hadk/ | 06:20 |
| tqg | ok,thank you. | 06:23 |
| chriadam | np | 06:25 |
| dcaliste | Hello chriadam, how are you ? | 06:57 |
| chriadam | hi dcaliste, I'm well thanks - how are you? | 06:57 |
| dcaliste | I'm fine, thank you. I've just seen your fix for the regression in imap protocol. | 06:59 |
| dcaliste | Sorry for being the author of the regression. | 06:59 |
| chriadam | not at all | 07:00 |
| dcaliste | I'll look at at it more closely this afternoon. | 07:00 |
| chriadam | with so many changes and moving parts, it has to be assumed that something will regress | 07:01 |
| chriadam | thank you for doing the work in the first place to do the backporting, of course | 07:01 |
| chriadam | it's interesting to me that our unit tests didn't catch this case | 07:01 |
| chriadam | which is something I should look into a bit further, tomorrow | 07:01 |
| chriadam | aside from that, I just merged the webcal PR, I will tag it shortly | 07:03 |
| dcaliste | Indeed, I didn't look at the imap protocol tests, is there any actually ? | 07:03 |
| dcaliste | Ok, thank you for the webcal PR. | 07:04 |
| chriadam | I'm not sure ;-) I ran all of the tests manually and didn't see any failures other than the logging ones I mentioned / can ignore | 07:04 |
| chriadam | but I don't remember precisely what the tests covered | 07:04 |
| chriadam | I also asked pvuorela if he could take a look at the buteo-syncfw PR given the DateSet changes, but I guess he might not have had a chance to look at that one, just yet | 07:05 |
| chriadam | flypig is on vacation this week | 07:05 |
| chriadam | well, for two weeks | 07:05 |
| chriadam | so, for this week, I guess we need to review and finally merge/tag the buteo-syncfw + nqpc related PR | 07:06 |
| chriadam | was there anything else outstanding on our side? | 07:06 |
| dcaliste | Ok, thanks. I'm using the logging UI very often at the moment to check for issues. It's giving feedback nicely in my opinion on what is going on well. | 07:06 |
| dcaliste | I'm preparing one PR for CalDAV to log details on failures. | 07:07 |
| dcaliste | But it's not ready yet. | 07:07 |
| chriadam | great! | 07:07 |
| dcaliste | It requires to modify a bit the Request class and friends from CalDAV to also return the message from the servers, not only error codes. | 07:07 |
| dcaliste | About the UI part, moving the EventListDelegate to the component section, I've clean the various cases a bit according to pvuorela remarks. | 07:08 |
| dcaliste | And added comments also to clarify the use cases. | 07:09 |
| dcaliste | It's jolla-calendar#303 | 07:09 |
| chriadam | added one minor comment there | 07:13 |
| chriadam | approved, but would like pvuorela or jpetrell to approve also | 07:13 |
| chriadam | tyvm | 07:13 |
| dcaliste | Yes, I see, you phrasing is clearer. I'll update the comment. | 07:15 |
| dcaliste | Just one out of topic question : do you have any knowledge about notification mechanism in lipstick ? | 07:17 |
| chriadam | hmm, I used to know a bit about it, but I've forgotten | 07:18 |
| chriadam | I think pvuorela and adenexter are the people who know the most | 07:18 |
| chriadam | do you have some specific query about it, or? | 07:18 |
| dcaliste | Ok, no problem, I'll ask them later. It's about fixing the voice mail notification that doesn't do anything when tapping on it. | 07:18 |
| dcaliste | It used to call the voice message server, but not anymore since the notification UI revamp. | 07:19 |
| dcaliste | I guess one config file linking the default action of the notification to the phone UI is missing or something. | 07:19 |
| dcaliste | From my reading through lipstick code, I arrive to the point that tapping on the notification should launch the "default" action. | 07:20 |
| dcaliste | In the Notification DB, there is indeed one entry for the default action, but with no further details. | 07:20 |
| dcaliste | As for other kind of notification default action. | 07:21 |
| dcaliste | So I guess there is a file somewhere describing that default action, but I didn't find it yet. | 07:21 |
| chriadam | so, another thing which might be involved here is sandboxing. the default action in the past could be a uri which would be handled similarly to xdg-open, but it could also (IIRC) be a dbus call | 07:21 |
| chriadam | but now these days there are some complications related to cross-process dbus calls | 07:21 |
| dcaliste | Indeed. | 07:22 |
| chriadam | well, just some extra things needed to ensure that the calling process has been given access to the endpoint | 07:22 |
| chriadam | so it could be that in this voicecall case we have missed something which is required there | 07:22 |
| chriadam | (or it might be something entirely unrelated) | 07:22 |
| chriadam | let me quickly search bugzilla | 07:23 |
| dcaliste | Yes, I'm saying that it is happening since the UI revamp, but sandboxing started around the same time, so it may be related too. | 07:23 |
| dcaliste | There is a forum entry for this bug, let me find it also. | 07:23 |
| dcaliste | https://forum.sailfishos.org/t/4-0-1-voicemail-notifications-dont-call-voicemail-when-clicked/5049 | 07:23 |
| dcaliste | This one is a mix of it also : https://forum.sailfishos.org/t/voicemail-calling-stopped-working-in-koli-4-0-1/4719 | 07:25 |
| dcaliste | The notification issue is independant from the long pressing on 1 in dialer issue. | 07:25 |
| dcaliste | I can long press myself, but the notification does nothing. | 07:25 |
| chriadam | thanks - I've created JB#55258 to track this one internally | 07:31 |
| dcaliste | Great, thank you. | 07:32 |
| dcaliste | I think that's all for today. Thank you for the discussions. I'll approve your QMF fix later today or tomorrow. | 07:34 |
| dcaliste | I need to check if I didn't let the same mistake elsewhere, this capture(0) pattern thing... | 07:35 |
| chriadam | I checked for lastIndexIn() with grep | 07:36 |
| chriadam | but didn't see any other cases -- but I think I only grepped in the imap subfolder | 07:36 |
| chriadam | I will look into the unit test thing tomorrow also (or maybe Thursday) | 07:36 |
| chriadam | thank you again for your time and effort, and for raising that notification issue also | 07:36 |
| chriadam | I hope that you have a great week! | 07:36 |
| dcaliste | Thank you, you too. | 07:37 |
| dcaliste | Good night chriadam. | 07:37 |
| lbt | rinigus: https://repo.sailfishos.org/obs/sailfishos:/chum/ | 09:05 |
| rinigus | lbt: thank you very much! piggz notice https above | 09:07 |
| piggz | lbt: <cough> https://github.com/mer-hybris/community-adaptation/blob/master/adaptation-community.ini.in </cough> | 09:23 |
| mal | lbt: should we change the links in the obs download repo pages to point to sailfishos.org url, for example https://build.sailfishos.org/repositories/nemo:testing:hw:fairphone:fp2-sibon those links still point to merproject | 12:19 |
| mal | piggz: while at it we should thing about the common repo issue on aarch64 | 12:21 |
| lbt | generally the merproject.org urls are deprecated | 13:33 |
| lbt | They should continue to work but I may make mistakes and miss testing them when consolidating services | 13:34 |
| lbt | I'll fix them if they do break though | 13:34 |
| piggz | mal: whats the common repo issue for aarch64? | 13:39 |
| mal | piggz: testing common doesn't have arch in the repos, we have the new naming on obs but it's not in use yet | 13:41 |
| mal | piggz: https://github.com/mer-hybris/community-adaptation/blob/master/adaptation-community-common.ini#L7 | 13:42 |
| piggz | mal: ah yes | 13:42 |
| mal | some smart idea long time ago to not include arch in that :) | 13:44 |
| piggz | mal: why would anyone want more that armv7? | 13:48 |
| mal | piggz: also note that we have had x86 support for a long time :) | 13:49 |
| piggz | rinigus: i like the siggestion the other day that chum was effectively f-droid for sailfish | 14:08 |
| mal | piggz: did you yet make some storeman like app for chum? | 14:50 |
| piggz | mal: no | 14:50 |
| rinigus | mal: we hope to get support for chum apps/libs by openrepos and storeman directly. If it works then there is no need for separate app | 14:55 |
| mal | ok | 14:56 |
| mal | storeman repo handling is quite a mess | 14:56 |
| poetaster | mal, piggz, rinigus: I now see the following coming: supporting jolla store as solidarnosc. supporting chum for broad build suppurt. continue using openrepos for all the cases I can't get into jolla or chum. | 15:07 |
| poetaster | s/suppurt/support/g | 15:08 |
| poetaster | hmm. should not have hacked that. /usr/share/lipstick-jolla-home-qt5/statusarea/StatusArea.qml | 15:22 |
Generated by irclog2html.py 2.17.1 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!