libera/#maemo-leste/ Tuesday, 2024-01-02

uvos__sicelo: yeah i gues thats true, this isent an oversight tho, i want some sort of notification when you dont accept a call as a reminder to call the other party back later, using the missed call machinery was simply eaiyest09:39
uvos__maybe we can create a notification but not log the call as missed or something09:42
uvos__needs sphone internal module api changes09:42
freemangordonuvos__: I think the proper way to do it is to log it ass rejected call11:17
freemangordon*as11:17
freemangordonand to mark it as such in the UI11:17
freemangordonuvos__: also, did you see the issue I found with the sound driver?11:18
freemangordonWizzup: I wonder isn't it better to get rid of glibofono in connui-cell11:19
Wizzupfreemangordon: ok11:23
Wizzupfreemangordon: because of the sync issues, or?11:23
freemangordonbecause they expose almost no method11:24
freemangordonbesides modem methiods, there is nothing else11:25
freemangordonto me it is easier do generate our gdbus wrappers than to make MRs to SF11:25
freemangordon*to11:26
freemangordonesp when our api is expected to be synchronous11:27
uvos__freemangordon: no what issue in the sound driver11:28
uvos__i noticed that since we updated pa11:28
uvos__the volumes stopped restoreing11:28
freemangordonso you think it is PA issue?11:28
uvos__i dont know what issue you are talking about11:29
uvos__i noticed that the volumes no longer restore11:29
freemangordonsec11:29
uvos__but maybe thats just because we changed pa to run as user11:29
uvos__and some file on my system is now owned by root11:29
uvos__and it cant update it11:29
uvos__or something11:29
freemangordonif you reject the incomming call, no notification sound is played and basically sounds do not work for few minutes or until PA is restarted11:30
Wizzupfreemangordon: fine by me11:30
uvos__ok11:30
uvos__well the driver dose some really hacky stuff when a call is started11:30
freemangordonuvos__: to me it seems there is some race or other issue in the driver11:31
uvos__iirc the driver just clobbers some registers11:31
uvos__that subsiquently the alsa framework dosent know about11:31
uvos__should restore them after the call however11:31
freemangordonso I commented out enabling modem dai on incomming call (but before it is accepted) and the issue is gone11:31
freemangordonbut thta'11:31
freemangordon...11:31
freemangordonbut that's just a hack11:31
uvos__yeah the modem dai is not used in a call11:32
uvos__its just for recording audio11:32
uvos__the whole thing is a huge hack11:32
uvos__(voice call on mapphones)11:32
uvos__nothing is correctly implemented here, as i say the driver just overwrites a buntch of registers that the alsa framework expects to control via its controls11:33
freemangordonsorry, gnome crashed11:33
freemangordonso, what needs to be done here?11:33
freemangordon*there11:34
freemangordonwe use ucm, right?11:34
uvos__yes11:34
freemangordondon;t we have the proper controls exported?11:34
uvos__but this isent about that11:34
uvos__the problem is that the snd-soc framwork dosent seam to support audio going coming from a 3rd party device without passing though any dai the kernel controls, but still needing setup/ routing in the analog amps etc that the audio devices the kernel dose control contain11:36
uvos__or at least we dont know how this is supposed to fit in11:36
freemangordonwith this https://github.com/maemo-leste/droid4-linux/blob/maemo-6.1.y/sound/soc/codecs/motmdm.c#L549 commented out it is ok11:36
freemangordonI see11:36
freemangordonok, why not ask on the ML?11:36
uvos__tmlind did in the past, to not that mutch sucess iirc, but yeah we could try again11:37
freemangordonwe'd better do, as now it is very buggy11:37
freemangordontmlind: could you point me to ^^^11:38
uvos__irrc the discussion last time surfced the idea of having a dummy dai, that pretends to be the source of the audio and is active when the modem controls the audio devices11:39
uvos__but dont qoute me on that11:39
uvos__the volumes def also stoped restoreing (on boot and on hp plug) thats def a pa bug we also have11:56
arno11Wizzup: freemangordon: i've got a (huge) memory leak with conversations/tp-haze/FB plugin. not sure what's going on but pkilling conversations solves the issue (otherwise it uses 100% cpu, the entire RAM and more than 500MB of swap !)12:34
arno11it happens when i connect a FB account12:34
Wizzupok, there's been a pretty big rework so it might be solved by now12:41
arno11ok12:41
arno11otherwise it works very well with almost no cpu usage or ram12:42
Wizzupdo you know if you do anything in particular?12:48
Wizzupfreemangordon: btw if we can get tp haze with channels in -devel or os that would be nice, then I can make sure we also deal with say slack channels well12:49
arno11Wizzup: apparently not12:53
Wizzuparno11: ok, well I want to make a few more changes/checks but then I will cut a release12:53
Wizzupthere's still abook integration to be done12:53
Wizzupbut there are a lot of rtcom logging improvements12:54
arno11ok cool12:54
Wizzupand yes, then we will have to hunt for memory issues12:54
Wizzupfor it to work well you might have ot nuke the existing rtcom db12:55
WizzupI'm making some more logging changes to make it (fully) compatible with fremantle12:58
Wizzupthe last thing I have to figure out is why the 'remote uid' for a message that we send to a channel is <channel id>@conference.../jabberid@jabberserver.domain13:15
Wizzupfor irc it is just the nickname13:15
WizzupI guess irc isn't federated and jabber is, so that makes some sense then13:16
arno11you mean the nicknames shown in conversation windows ?13:39
Wizzuparno11: nah just sql logging13:47
arno11ah ok13:47
Wizzupok, figured it out I think14:46
arno11Wizzup: btw do you have an idea why some services show proper nicknames and not others ?15:19
Wizzupwhat do they show if not the nickname?15:19
WizzupI see '(No name)' in contacts, but I presume you are referring to something else15:20
arno11for example irc (and irc through bitlbee/conversations) and skype show proper nicknames in chat windows15:20
arno11not working for sip or FB15:20
arno11SIP shows full address and FB showa an 'ID' number15:21
arno11*shows15:22
arno11instead of nicknames/names15:22
Wizzuphm, right15:23
Wizzupso this is complicated, but this is what I am working on now, now that I think the logging is fixed15:23
Wizzupthere is a nickname property for some of these, but for the actual logging we want to set the id15:23
Wizzupand what we need to do is to log the nickname in remote_name and the id in remote_uid15:24
arno11ok15:24
Wizzupand then optionally also use abook to resolve the remote ids to contacts15:24
Wizzupat least that is what I am thinking right now15:24
WizzupI need to fix two regressions in conversations and then it can go to -devel I think15:24
arno11ok :)15:24
Wizzupas far as the mem usage goes, let's debug that soon too15:26
WizzupI haven't paid too much attention to this but for the qt part I think it should just be ok15:26
Wizzupwas the memory usage you saw in tp-haze or conversations\15:26
Wizzup(or both)15:26
arno11only with conversations15:26
arno11tp haze increase cpu and ram only for few sec when connecting15:27
Wizzupok15:28
arno11btw how can i disable the --background option ? (just to test if there is a diff)15:30
Wizzuparno11: I don't think you need to test this15:37
Wizzupbut background is only default in dsme15:37
arno11yeah ok15:38
Wizzupok yeah now I remember, for mapping im usernames to their display name I have to map a Telepathy::Account to TpAccount*15:39
arno11ok cool15:39
Wizzuphttps://telepathy.freedesktop.org/spec/Connection_Interface_Aliasing.html15:42
WizzupI think I can add this for everything that we don't have a contact for15:42
WizzupProbably shouldn't test in this channel :p15:50
arno11:)15:50
Wizzupfreemangordon: do you want me to build a new haze version?15:57
Wizzupfreemangordon: the sip lookup segfaults in strlen() https://paste.villavu.com/raw/8fPhvnewILtf2zfoQX3gmsUJKz52NeUN4g4EBymC/16:09
freemangordonwill look at it16:32
freemangordonWizzup: new haze? why not16:33
freemangordonlemme push to github16:34
freemangordonWizzup: hmm, actually...16:36
freemangordonnot sure how to deal with upstream16:36
freemangordonadd commits as patches?16:36
Wizzupfreemangordon: don't really care tbh16:40
WizzupI suspect we'd become upstream eventually16:40
freemangordonWizzup: hmm, your sip address is missing sip://16:40
Wizzuplet me see, I might not be passing the right info then16:40
freemangordonthere is a bug in abook, but still16:40
uvos__honestly the storage format is also just bad16:44
uvos__im not sure keeping absolute fremantle compatability is a good idea16:45
uvos__like the fact that tel is just randomly not a url is pretty dumb16:45
uvos__as is the almost random fromat of some other fields16:45
uvos__like the local/remote ids16:46
freemangordonremote id is EDS uid if available16:48
freemangordonwhich is, in most cases16:48
freemangordonunless I am missing something16:50
Wizzupwhat storage format?16:50
freemangordonWizzup: re haze - it is about debian packaging16:51
freemangordonI am not sure what is better16:51
Wizzupfreemangordon: yeah, have no strong preference, I would just add them as commits and cut some small minor release for us for now16:51
WizzupI don't expect upstream to do anything16:51
Wizzupthere is no maintainer afaict16:51
Wizzupuvos__: actually I'm starting to like the rtcom format, it makes a lot more sense once you grok it16:52
Wizzupfreemangordon: I don't think patches vs commits matters16:53
freemangordonbot are not fine, because I used upstream code, not debian one16:53
freemangordonand they differ16:53
freemangordonbut ok, will just impland debian dir16:54
Wizzupwhat did debian change16:54
uvos__maybe, but the fact that sms, tel and messages and all kinds of ip voice are seperate things with different field meansings seams pointless to me16:54
freemangordonnothing, they are just few commits behind16:54
uvos__and creates headaches16:54
Wizzupconversations should (in my branch) deal with it well atm16:54
Wizzupfor sip, sms, tel, etc16:54
Wizzupthere are some oddities I would agree, like the group_uid for smses16:55
freemangordonWizzup: please make user you have master contact id logged properly16:55
Wizzupfreemangordon: can't parse16:55
freemangordon*make sure16:55
Wizzupdo you mean the osso_abook_id ?16:55
freemangordonyes16:55
Wizzupwell, I can't do that part yet until I have the conversion to TpAccount16:56
freemangordonthis is basically EDS master contact UID16:56
Wizzupbut apart from this, everything else seems to be alright now16:56
Wizzupthere is some code to clean up too I suppose16:56
freemangordonok16:56
freemangordonI am just reminding :)16:56
Wizzupis it necessary for the abook api to get a TpAccount* ?16:56
Wizzupcan't we just give it a string describing where to find the account?16:56
freemangordonno16:56
Wizzupthat would make my life much easier16:57
freemangordonwhich api do you mean?16:57
Wizzuphttps://github.com/maemo-leste/osso-abook/blob/master/lib/osso-abook-aggregator.h#L14516:57
Wizzupclearly it needs to know the context for this im16:57
Wizzupbut does it have to be a TpAccount*16:57
Wizzupas opposed to just say sofiasip/sip/NICKNAME@sip.antisip.com or something like that16:58
Wizzupor sofiasip/sip/NICKNAME_40sip_2eantisip_2ecom016:58
freemangordonwhy not just create TpAccount?16:58
freemangordonit should be easy16:58
WizzupI am not sure it is16:58
WizzupI mean, yes, I can make another accountmanager, this time in glib, next to the one we have already in a book16:59
Wizzups/a book/abook/16:59
Wizzup(and next to the qt one that we have)16:59
freemangordonYou should be able to ask tpqt to give you the object path17:00
freemangordonand that's all you need17:00
Wizzupsure objectPath is easy17:00
Wizzupbut then what?17:00
Wizzuphttps://telepathy.freedesktop.org/doc/telepathy-glib-0.24.x/telepathy-glib-account.html#tp-account-parse-object-path ?17:00
freemangordongimme a minute17:01
Wizzupmeant to link to tp_account_new17:01
Wizzupin any case does abook need any of the tp properties, or is it just going to extract the backend name17:02
Wizzupand local_uid17:02
freemangordonno, it needs it to find the master contact17:02
freemangordonnot only the backend17:02
freemangordonsee   osso_abook_aggregator_find_contacts_full()17:02
Wizzupsorry I still don't really understand what the 'master contact' is17:02
freemangordonlemme try to explain17:03
freemangordonfor each person in your abook, you have one *master* contact and zero or more *roster* contacts17:03
freemangordonmaster contact is there no matter online/offline etc17:04
freemangordonso, in your abook, my master contact contains my tel, email, nickname etc17:04
Wizzupmhm17:05
siceloWizzup: freemangordon ... regarding TP, I *think* Ubuntu Touch are also using it. Not sure if there'd be a chance/benefit to collaborate with them17:05
freemangordonand when you merge contacts (from jabber, for example), you attach my jabber username to my master contact17:05
freemangordonsicelo: just a sec please17:05
freemangordonand that way jabber actions appear when you click on 'ivo' in the abook17:05
freemangordonnow, when you want to look at the history of your comunications with 'ivo' you want p[hone calls, sms*es, ims, everything, no?17:06
freemangordonWizzup: is it cleraere now?17:07
freemangordon*clearer17:07
Wizzupyes, but not why you need a TpAccount for it17:07
Wizzupthis is what is stored in the rtcom db as remote_uid17:08
Wizzupwell, wait17:08
Wizzuplocal_uid in this case17:08
Wizzupwhat I mean to say is what TpAccount* to my knowledge doesn't contain anything special here17:09
Wizzupwhat you do to match TpAccount* to eds can be done without the Tpaccount*, you just need specific info17:09
Wizzupbut it seems like it's easier if I just add a third accountmanager to conversations ;)17:09
freemangordonWizzup: this is just the API that's exported17:09
freemangordonsee osso_abook_aggregator_find_contacts_full17:10
freemangordonyou can use it directly17:10
WizzupI don't know what it does and what a OssoABookContactPredicate is :)17:10
Wizzupah it is defined above17:11
freemangordonhttp://maemo.org/api_refs/5.0/5.0-final/libosso-abook/OssoABookAggregator.html#osso-abook-aggregator-find-contacts-full17:11
freemangordonor you can use osso_abook_aggregator_find_contacts17:11
Wizzuplooks like fremantle used the missioncontrol account, did that contain the same data generally speaking?17:13
freemangordonit is the *same*17:14
Wizzupok17:14
freemangordonMcAccount is TpAccount17:14
freemangordonmore or less17:14
freemangordonosso_abook_aggregator_find_contacts_for_im_contact() can get NULL for TpAccount17:14
Wizzupoh17:14
freemangordonsee http://maemo.org/api_refs/5.0/5.0-final/libosso-abook/OssoABookAggregator.html#osso-abook-aggregator-find-contacts-for-im-contact17:15
freemangordonso you only need the username17:15
Wizzupwe probably need to look into generating these docs for us too17:16
freemangordonyes17:16
freemangordonI need a volunteer to copy-paste them :)17:17
Wizzupdidn't spinal wrote code to auto do this?17:17
Wizzups/wrote/write/17:17
freemangordoncould be17:17
freemangordonWizzup: also, keep in mind   OssoABookContact is EContact17:18
Wizzupsent myself a single message and immediately the remote_name updated in the UI with the name set in my contacts app :)17:18
freemangordonWizzup: a good test is to open abook17:19
Wizzupnow we just need to master id lookup I guess17:19
Wizzupfreemangordon: I already changed the nickname in abook17:19
freemangordonno, select "Recent" from the menu17:19
Wizzupyes, it's fine there too17:19
Wizzupconversations reads from the same db17:19
freemangordonnot really, if you don;t log the uid17:20
freemangordonabook has fallback, but that will take time on big db17:20
Wizzupok, so this id is just a property I imagine?17:20
Wizzupjust like name is osso_abook_contact_get_display_name17:21
Wizzuposso_abook_contact_get_master_uids ?17:21
freemangordonhttps://github.com/maemo-leste/osso-abook/blob/master/lib/osso-abook-contact.h#L28117:22
freemangordonI think this17:22
Wizzupok17:22
freemangordonas I said, E_CONTACT_UID17:22
freemangordonAFAIK17:22
freemangordonmaybe check fremantle db17:22
Wizzupsometimes I do wish that C supported multi-value return so I didn't have to invent a struct :)17:22
Wizzupfreemangordon: what do you want from fremantle db?17:22
freemangordonI want nothing :)17:23
freemangordonjust check if abook id there matches E_CONTACT uid17:23
freemangordonunless we have help on rtcom db explicitly saying that17:24
WizzupCREATE TABLE Remotes (local_uid TEXT NOT NULL,remote_uid TEXT NOT NULL,remote_name TEXT,abook_uid TEXT,UNIQUE(local_uid,remote_uid));17:24
freemangordonabook_uid17:24
Wizzupif you just set remote_name (which I already do) it will auto get added to the remotes table17:24
WizzupI suspect the same is true for abook_uid17:24
Wizzupauto get added = rtcom-eventlogger library does this17:24
freemangordonno, abook uid should already exists17:24
freemangordonIIUC17:25
freemangordonthis is EDS UID17:25
Wizzupok, so17:25
Wizzupevents is a table of all the events17:25
freemangordonmhm17:25
Wizzupremotes is a table that maps a remote_uid to a remote_name and abook_uid17:25
freemangordonah17:25
Wizzupnot every event remote_uid is presented here17:25
Wizzupso I think if I just log abook_uid together with remote_name, we're ok17:25
freemangordonright17:25
WizzupI'll do this in a moment and report back17:26
Wizzupthen it's probably good if you can look at some of my code and comment on it later today (once I clean up and push)17:26
Wizzupthere is one regression that I need to fix, that might actually not be present on our devices17:26
Wizzupthen I'll release 0.6.0 to devel17:27
freemangordonok, lemme see how to release haze17:27
freemangordonWizzup: btw, segfault in abook should be fixed17:28
Wizzupsaw that, ty17:31
Wizzupbtw, this tpaccount*, do I need to dispose of it? doc says it's owned by aggregator17:35
Wizzups/tpaccount/OssoABookContact/17:38
freemangordonno17:41
freemangordonjust the list17:41
Wizzupok17:41
freemangordon"...but the list itself must be freed with g_list_free(). "17:41
WizzupI saw that17:43
Wizzupsqlite> select abook_uid from remotes where remote_uid='merlijn-fremantle@xmpp.wajer.org';17:43
Wizzuppas-id-2dca35db338ead7ac0e3a720079deff6bb8901d617:43
Wizzupthis was NULL until just now17:43
Wizzupdoes that seem sensible do you?17:43
Wizzupto you*17:44
freemangordonno idea17:44
Wizzup:)17:44
Wizzupit was mostly integers in fremantle fwiw17:44
freemangordoncheck with EDS17:44
Wizzupah that is not true, not just ints17:45
WizzupI have no idea how to check with eds17:45
freemangordonsec17:45
Wizzupbut I used:17:45
WizzupRTCOM_EL_EVENT_SET_FIELD(ev, remote_ebook_uid, g_strdup(abook_uid));17:45
Wizzupand17:45
Wizzupabook_uid = osso_abook_contact_get_uid(tmpcontact);17:45
freemangordong_strdup(abook_uid)?!?17:46
Wizzupfor rtcom-eventlogger17:46
freemangordonit will free it?17:46
Wizzupyes17:46
freemangordonok17:46
freemangordonlooks ok17:47
Wizzup*shrug*17:47
freemangordonlemme try to start EDS17:47
Wizzupabook ui still looks the same,but it already did the right thing earlier I think17:47
freemangordonyeah17:47
freemangordonWizzup: check the contents of /home/user/.local/share/evolution/addressbook/system17:55
freemangordonI am using mcview for that17:55
Wizzupok,sec17:55
Wizzuplooks ok18:10
freemangordonWizzup: tp-haze is in -devel repo18:18
Wizzupgreat, ty18:19
Wizzuphm18:21
Wizzup# G_MESSAGES_DEBUG=all HAZE_PERSIST=1 HAZE_DEBUG=all /usr/lib/telepathy/telepathy-haze18:21
Wizzup/usr/lib/telepathy/telepathy-haze: error while loading shared libraries: libpurple.so.0: failed to map segment from shared object18:21
freemangordonwhat?18:22
Wizzupwell, that is not supposed to happen right18:22
freemangordonno18:22
freemangordonlermme upgrade here18:23
freemangordonbbl 15 miunutes18:23
Wizzupworks on on armhf it seems18:26
Wizzupfreemangordon: maybe I start it wrong somehow18:36
Wizzupfreemangordon: yes, nevermind18:36
freemangordonis it ok?18:39
Wizzupyes18:40
WizzupI was starting it as root somehow18:40
Wizzupbtw, how do you set purple specific params through mc-tool18:40
Wizzuplike, some of these: https://github.com/dylex/slack-libpurple#configuration-options18:41
freemangordonwhy not use accounts-ui?18:41
Wizzupwell, I am still working on the accounts-ui18:41
freemangordonah18:41
freemangordonI used empathy18:41
freemangordonyou just have to start it with SW GL18:41
freemangordonon d4 that is18:41
Wizzuphum18:42
Wizzupok18:42
WizzupI'll go and make some dinner and try thislater18:42
freemangordonyou should be able to do it with mc-tool as well18:42
Wizzupmc-tool update: Protocol 'slack' does not have parameter 'enable_avatar_download'18:43
freemangordonno idea then18:44
Wizzupurgh empathy pulls modemmanager19:25
freemangordonso?19:35
freemangordonI don't have it installed here19:35
WizzupI removed it again19:36
Wizzupsome recommended thing in debian I guess19:36
freemangordonare you sure it is pulled and not recommends: ?19:36
freemangordonWizzup: when you use mc-tool, you should provide correct parameter type19:37
freemangordonlike bool:enable_avatar_download=true19:37
Wizzupfreemangordon: probably recommends20:03
Wizzupfreemangordon: I did that @ type20:04
Wizzuphm, it looks like slack-libpurple maps channels to ... people?20:08
freemangordoncheck in pidgin20:27
Wizzupwell: Your "open" channels (on the slack bar) are mapped to the buddy list: joining a channel is equivalent to creating a buddy;20:34
Wizzupthat is insane :D20:34
freemangordonin pidgin as well?20:37
WizzupI need to test20:39
freemangordonyes, check with pidgin20:39
Wizzupit's ok in pidgin20:41
Wizzupat least there is a multi person icon20:42
Wizzupand I can see the participants20:42
freemangordonwell, how do you know it is single person th TP then?20:45
freemangordonkeep in mind I havent' implemented full room support in haze20:45
Wizzupsry phone call20:45
freemangordonjust multi-channel supprot20:45
freemangordonumm... multi-user channel20:46
freemangordonbefore that only person-to-person chats worked20:46
freemangordonso there is lot more to implement20:46
Wizzupok20:47
freemangordonalso, how do you know it it is group chat or not?20:48
freemangordonin TP that is20:48
freemangordonyou check the handle type?20:48
Wizzupsec :)20:48
Wizzupfreemangordon: so in slack there are channels, multi user chats and single direct chats21:15
Wizzupthe things I am talking about are channels in slack for sure, but maybe in pidgin not, if that was your question21:15
Wizzupin any case it could very well be some missing parts in haze as you mentioned21:15
Wizzupit was just interesting that I did get the messages in conversations, just not as part of a channel in tp it looked like21:16
Wizzupbut maybe conversations doesn't handle multi user chats well yet :)21:16
freemangordonWizzup: the question is: how conversations knows if it is multi-user chat or not?21:26
freemangordonbecause I think there is a difference between room and multi-user chat21:26
freemangordonmy understanding is that rooms have separate handle type21:27
freemangordonwhile multi-user chat handle has 'groups' interface21:27
freemangordonbut that's my understanding which is no necessarily correct21:28
freemangordon*not21:28
Wizzupfreemangordon: I think it doesn't yet, because I also do not know how it works yet21:28
Wizzupfdo.org is slow agai21:29
Wizzupfreemangordon: I think the groups interface is also for a channel21:30
WizzupHandleTypeContact21:31
WizzupA contact21:31
WizzupHandleTypeRoom21:31
WizzupA chat room21:31
WizzupHandleTypeList21:31
WizzupA server-generated contact list (see Channel.Interface.Group)21:31
WizzupHandleTypeGroup21:31
WizzupA user-defined contact list (see Channel.Interface.Group)21:31
WizzupI think there's a lot about opaque identifiers and whatnot21:33
Wizzuphttps://web.archive.org/web/20231116152329/https://telepathy.freedesktop.org/spec/Channel_Interface_Room.html21:34
freemangordonso, to me at least multi-user chat Channel.Interface.Group21:35
freemangordonbut again, that's my understanding21:35
freemangordonyet again, maybe I shall provide room handle for multi-user chats21:37
WizzupI don't think it is that interface21:37
Wizzupthat is the general interface for requesting users on rooms21:37
WizzupI would find some more links but it's frustrating to navigate fdo when it's down21:38
freemangordonhmm... https://github.com/maemo-leste-upstream-forks/telepathy-haze/blob/maemo/chimaera-devel/src/mu-channel.c#L14121:38
Wizzupin any case conversations assumes that anything is a room as long as:21:38
Wizzupif (channel->targetHandleType() == Tp::HandleTypeContact)21:38
Wizzupas not*21:38
freemangordonsee the link ^^^21:38
WizzupI think everything is a room21:39
freemangordonok, so seems I am doing proper thing21:39
Wizzupdid you check: https://web.archive.org/web/20231116152329/https://telepathy.freedesktop.org/spec/Channel_Interface_Room.html21:39
freemangordonyes21:39
Wizzupok21:39
Wizzupthe way I see it there's no real difference in interface, just in some room properties21:42
Wizzupbtw, the 'No name' bug is quite annoying21:43
freemangordonhmm, maybe I am not detecting properly what purple plugin implements21:45
Wizzupmsgid "addr_li_unnamed_contact"21:46
Wizzupmsgstr "No name"21:46
freemangordonhmm https://github.com/maemo-leste-upstream-forks/telepathy-haze/blob/maemo/chimaera-devel/src/connection.c#L60021:47
freemangordonWizzup: this is eds tp plugin thing21:47
Wizzupare you sure?21:48
freemangordonyes, names come from adress book, whick is created by eds tp plugin21:48
WizzupI think some protocols maybe do not provide a nickname but only a targetid21:48
Wizzupok, well, this string appears in osso-abook 3 times21:48
Wizzup$ grep -r addr_li_unnamed_contact lib/*.c21:49
Wizzuplib/osso-abook-contact.c:    name = g_dgettext("osso-addressbook", "addr_li_unnamed_contact");21:49
Wizzuplib/osso-abook-filter-model.c:    gchar *unnamed_fold = g_utf8_casefold(_("addr_li_unnamed_contact"), -1);21:49
freemangordonwell, if there is no name and nick, I don;t see what abook is supposed to do21:49
Wizzuplib/osso-abook-tree-view.c:    contact_name = _("addr_li_unnamed_contact");21:49
Wizzuptargetid seems sensible21:49
freemangordonwhat is this?21:49
Wizzuppersistent handle of the user on the protocol21:49
freemangordonin terms of vcard/eds?21:49
freemangordonheh21:49
WizzupI mean, literally my entire xmpp address book is No name21:50
freemangordonabook uses EContact, which is EVCard as well21:50
Wizzupas well as the slack ones21:50
Wizzupand now I got a message from someone and remote_name because 'No name' :)21:50
Wizzupin this case the jabber address (somename@foo.org) would be better than 'No name', no?21:50
freemangordoncould be, but that's not abook21:51
freemangordonagain, it is eds tp plugin21:51
freemangordonabook uses vcards21:51
freemangordonmotre or less21:51
freemangordon*more21:51
freemangordonbbl21:51
WizzupI am looking at the plugin21:51
Wizzupfreemangordon: regarding your link, yes, it seems like that would be good @ target handles21:52
Wizzupfreemangordon: the funniest thing is that if you click on a No name contact, it will show the network handle22:02
Wizzupso clearly it knows and gets the info somehow22:02
freemangordonWizzup: does it have proper names in fremantle?22:11
freemangordonhttps://github.com/maemo-leste/eds-backend-telepathy/blob/master/src/e-book-backend-tp-contact.c#L24822:12
freemangordon:p22:12
Wizzupyes22:14
freemangordonin abook?22:15
freemangordonhmm22:15
freemangordonmaybe abook in fremantle is doing something more22:17
Wizzupmy yes was @ your link22:17
Wizzupnot @ fremantle22:17
WizzupI will have to check in fremantle22:17
freemangordonhave to go now, will check tomorrow if I have missed somethiong in abook22:18
freemangordonplease verify in fremantle in the meanwhile22:18
freemangordonif possibl22:18
freemangordone22:18
WizzupI have never seen it there but will check22:27
freemangordonWizzup: maybe debug through   osso_abook_contact_get_name_components() to see why it fails22:58
freemangordonor register an account for me to test22:58
Wizzupfreemangordon: do you have any xmpp account?23:05
Wizzupfreemangordon: but yes, I will register you one with me (didn't I already? I forgot)23:05
freemangordonlemme check23:05
freemangordonno, I don;t have any xmpp account23:06
freemangordonok, I am online there, but I see no contacts23:09
freemangordonso maybe the sever does not provide contact list23:10
* enyc meows :O23:11
enycno news for a while? hrrm23:11
Wizzupfreemangordon: do you have any contacts?23:13
freemangordonno23:13
Wizzupbrb 2mins23:13
freemangordonlets continue tomorrow23:13
Wizzupok23:15

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