libera/#maemo-leste/ Sunday, 2023-01-29

uvosthere for sure is some issue with that (sms), it never happens on android on d4 but it dose on leste (same operator)13:54
uvosalso the remote party dosent ever get a delivery report when this happens13:54
uvosso the modem is not acknowledging the sms to the network or something like that13:55
WizzupI also got it on android on other hw, and on fremantle15:09
Wizzupbut sure it could be some modem bug15:14
bencohit happened to me as well on other platforms, but not that often15:16
Wizzupdo you get it often on leste? I never had it yet15:17
Wizzupas in, that'd be helpful in reproducing15:18
siceloi never got it anywhere else, but did get it on  Leste D4 just once though. i think i got over 30 of the same sms. it's never happened again since17:24
WizzupI have had all sorts of weird problems before freemangordon 's modem fixes17:34
Wizzupall are  gone now17:34
Wizzupuvos: do you have some ideas about how we can have the tp code on maemo, and not have the sms ui at all - since that will be part of conversations in maemo?18:09
WizzupWe could track some branch that's different from master if there's some code you wouldn't like to have merged18:10
uvosWizzup: just dont load the ui18:31
uvosits a module18:31
Wizzupuvos: sure, but to deploy to to leste, do I make some branch and build it in maemo/chimaera?18:49
Wizzuplike how do you envision that? any preference?18:49
uvosWizzup: ? why build anything21:36
uvosWizzup: sphone has sphone.ini and the dir in ~ where it loads config21:36
uvosWizzup: just add another .d somewhere and add config to leste-config or whereever you like21:36
uvosjust add another dir here https://github.com/maemo-leste/sphone/blob/eacc386bf976e999a0c2a0d0a536cebe5b7f788d/src/utils/sphone-conf.c#L33921:40
uvosabove ~/.sphone (so that .sphone remains highest overriede priority)21:40
Wizzupuvos: because we need the code merged21:42
uvosthe code is in no state to be merged anywhere (or deployed)21:43
uvosatm21:43
Wizzupuvos: 'in no state' means:21:56
Wizzuptabs/spaces21:56
Wizzupall the qDebug() needs to be removed/replaced with sphones log functions21:56
Wizzupyeah?21:56
uvosmem leaks on exit, incorrect cmake changes21:59
uvosalso its a temporary solution at best, and should be marked as sutch21:59
uvosalso i dont see the point of merging it at all if non modem calls dont work22:00
uvosalso it fails to set call times22:00
WizzupI didn't see this feedback on gh22:03
uvosthe call times thin i just noticed now22:04
uvoseverything else is there22:04
Wizzupjust to be nitpicky, no, the mem leak on exit and cmake changes aren't show as comments, unless I missed in-code comments somehow22:07
Wizzupbut I'll look at it22:07
WizzupI'm convinced we'll want to use sailfish' voicecall-manager in any case, it's a great fit22:08
uvosif thats the expectation your going to want to merge this with22:08
uvosthen yes you will have to fork22:08
Wizzupok22:08
uvoshttps://github.com/maemo-leste/sphone/pull/422:09
uvoscomments show for me22:09
uvosmem leak comes from your own comment22:10
uvos /* TODO: delete all providers */ <- either its leaking memory or the comment is lieing, needs to fixed either way22:10
WizzupI saw the messages, but thought maybe there was more22:12
Wizzupwhat is the incorrect cmake change?22:12
uvoswhat is sais22:12
uvosone line is unnessecary22:12
uvosand one is unacceptable22:12
uvosyou need to make the build of the module contingent on it finding its depends22:13
uvosnot make the depends required22:13
Wizzupok, I didn't see that in the comments on github, maybe I'm really missing something22:13
* Wizzup checks js settings22:13
WizzupI don't see them, you can give inline comments on gh, like here: https://github.com/maemo-leste/sphone/pull/4/files22:14
uvosthere ARE inline comments there22:14
uvosthey also show on the pr22:14
uvosyour just apearently not seeing them22:14
rafael2khow to keep the screen on all the time?22:14
uvosrafael2k: set the timeout to 022:15
uvosor use the applet22:15
rafael2kbut in the UI I can not find22:15
uvosdisplay settings -> backlight timeout -> 022:15
rafael2kmy daugher is always turning on the screen when watching, and there is no 0 option22:16
rafael2k*watching videos22:16
uvoshmm there are for me22:16
uvos*is22:16
rafael2k10s up to 2min here22:17
uvosyou can also install simple-brightness applet22:17
uvosand use that22:17
rafael2kyay22:17
uvosatho this polls mce22:17
uvosso its sortof stupid22:17
uvosbut i gues pm dosent matter mutch when the screen is on anyhow22:17
rafael2kit is installed22:18
rafael2kah yes22:19
rafael2kfound!22:19
rafael2k: )22:19
rafael2kbut on option in the stock UI for not to turn off would be nice22:19
uvosidk what happened to it22:19
uvosit should be there22:20
Wizzupuvos: also the sip audio not being routed is not a problem in the module, it's a gst problem, so the sphone tp code won't change for it22:20
Wizzupat least iiuc22:20
uvosWizzup: still i dont see the point of this module if all it can do is modem calls, the ofono module has this coevered22:20
Wizzupbecause telepathy sms won't work unless it handles calls.22:20
Wizzupor rather, enabling tp-ring makes it take over ofono22:21
uvosbecause it hangs up incomeing calls?22:21
uvossilly behavior in ring that should be fixed anyhow, ofono can, and dose, have multiple clients22:21
Wizzupyes22:24
Wizzupgreat, maybe in 2026 we'll have a working phone :D22:24
Wizzupwe -don't- want to use ofono directly anyway for this22:24
uvoswell in my opinion it works allready, sans the changes you are adding to the enviroment22:30
uvosalso the bug in ring can hardly be very difficult to fix22:30
Wizzupwe're not going to fix it since it's completely irrelevant fo rus22:31
Wizzupand we'd also need to have sphone not deal with sms or log anything related to it22:31
Wizzupand this can already do multi party with minimal work, which is great22:32
Wizzup(I still need to test that of course)22:32
uvosdealing with sms is trvial issue22:32
uvossince the sphone allready seperates this22:32
Wizzupseems to me like the ofono module does both unconditionally when lodaed22:33
uvosshouldent matter what the ofono module dose22:34
uvosall ofono clients get all messages22:34
uvosso the sms just dies in sphone22:34
Wizzupyes, so it will log to rcom twice?22:34
Wizzuprtcom*22:35
uvosyes. atm, also the libnotify module has this issue (but the gtk notification module dosent)22:35
uvosbut this is trival22:35
uvosit just has to check the config value that exists allreadt22:35

Generated by irclog2html.py 2.17.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!