| freemangordon | Wizzup: https://github.com/maemo-leste-extras/rtcom-accounts-plugin-matrix/pull/1 | 09:25 |
|---|---|---|
| freemangordon | Wizzup: play with https://github.com/maemo-leste-extras/rtcom-accounts-plugin-matrix/blob/username/src/matrix-plugin.c#L33 to see which one you would like more | 09:26 |
| Wizzup | freemangordon: oh if you were testing in the vm, you need to (temporarily) disable apparmor for tp-tank to work (like haze with telegram) | 09:28 |
| Wizzup | this might be why sign in didn't work | 09:28 |
| Wizzup | 'sudo aa-teardown' | 09:28 |
| Wizzup | and yes, we need to fix tp apparmor profiles | 09:28 |
| freemangordon | lemme try | 09:28 |
| freemangordon | no change | 09:32 |
| Wizzup | ok | 09:32 |
| freemangordon | Wizzup: btw, force-pushed some changes, in case you pulled in the meanwhile | 09:32 |
| Wizzup | didn't pull yet | 09:41 |
| freemangordon | Wizzup: please, when adding accounts_fi_device_name to translations, consider adding accounts_fi_facebook_sync_interval, accounts_fi_facebook_mark_read, accounts_fi_facebook_mark_read_available, accounts_fi_nickname, accounts_fi_server and accounts_fi_real_name as well | 10:26 |
| freemangordon | or give me hints on how to do it | 10:27 |
| freemangordon | ugh, scratch that | 10:28 |
| freemangordon | seems I already did it | 10:28 |
| freemangordon | shall I add accounts_fi_device_name to translations? | 10:44 |
| Wizzup | sounds good | 10:46 |
| freemangordon | ok, but I have no idea how to release it | 10:46 |
| freemangordon | translations are on weblate and that's it | 10:46 |
| freemangordon | maybe you can translate a bit, ^^^ strings have only en and bg, perhaps you can add nl and dunno what | 10:47 |
| Wizzup | weblate is good yes | 10:52 |
| Wizzup | please do it on weblate | 10:52 |
| Wizzup | I go through it some of the time to make new releases | 10:52 |
| freemangordon | ok, I added it, how to make a release? | 10:56 |
| Wizzup | Let me look | 11:06 |
| Wizzup | freemangordon: which did you work on | 11:07 |
| Wizzup | https://github.com/maemo-leste-translations/osso-applet-accounts-l10n ? | 11:08 |
| Wizzup | there are also open pull requests there | 11:09 |
| Wizzup | https://github.com/maemo-leste-translations/hildon-control-panel-l10n/commits/master/ | 11:10 |
| Wizzup | here you can see what I do | 11:10 |
| Wizzup | 1. merge any pull requests | 11:10 |
| Wizzup | 2. make release | 11:10 |
| Wizzup | that's it | 11:10 |
| Wizzup | basically | 11:10 |
| Wizzup | https://github.com/pulls?q=is%3Aopen+is%3Apr+user%3Amaemo-leste-translations+archived%3Afalse+ these are all the pulls open for translations | 11:11 |
| Wizzup | I should probably go through them today | 11:11 |
| Wizzup | ah just https://github.com/pulls?user=maemo-leste-translations | 11:11 |
| freemangordon | Wizzup: oh, I see | 11:19 |
| Wizzup | I don't have a way to auto make a debian/changelog entry so it's tiresome :) \ | 11:25 |
| freemangordon | I'll make a new release for accounts translations | 11:27 |
| freemangordon | but have to run after that, please go through other pull requests | 11:27 |
| Wizzup | it will take a few hours but yeah | 11:29 |
| freemangordon | yeah, but better start doing it | 11:29 |
| freemangordon | no need to finish at once | 11:29 |
| freemangordon | I will help once I am back | 11:30 |
| freemangordon | ugh, does not build | 11:32 |
| Wizzup | all this work for a few words changed :D | 11:36 |
| freemangordon | yeah :) | 11:40 |
| Wizzup | down from 46 to 30 | 11:43 |
| Wizzup | going to take a break | 11:43 |
| Wizzup | calendar-l10n failed | 11:43 |
| Wizzup | freemangordon: sent you an email | 11:44 |
| Wizzup | some weblate warning | 11:44 |
| freemangordon | yeah, just hit that when trying to build | 11:45 |
| freemangordon | fixed it on github, lets see if weblate will know about that fix | 11:45 |
| Wizzup | there's some new armenian translations though | 11:46 |
| Wizzup | if we switched translations to date as version it would be easier to automate | 11:47 |
| freemangordon | yeah, why not | 11:47 |
| Wizzup | ok | 11:49 |
| freemangordon | did you test matrix accounts plugin? | 11:54 |
| Wizzup | no, I am doing translations first | 12:01 |
| Wizzup | I made some script now | 12:01 |
| freemangordon | cool | 12:01 |
| freemangordon | Wizzup: "gbp:error: 2024-08 is not a valid treeish" | 12:02 |
| freemangordon | you have to tag as well | 12:03 |
| freemangordon | afk, ttyl | 12:03 |
| Wizzup | I do tag | 12:05 |
| Wizzup | I will check. | 12:05 |
| Wizzup | I have to go | 12:05 |
| Wizzup | back later | 12:05 |
| Wizzup | ok, I think the problem is just in the version | 12:43 |
| Wizzup | will fix | 12:43 |
| Wizzup | freemangordon: most are done now, a few stragglers but I'm building it all now | 13:00 |
| arno11 | freemangordon: almost 24h with shader changes, no issue so far | 13:49 |
| arno11 | changing HD, xorg, mce and systemui priorities seem to cause no issue as well | 13:54 |
| arno11 | (and it makes 250 freq ok with no more lags with ondemand governor) | 13:56 |
| arno11 | schedutil gov should be better but it is not activated in kernel config | 13:58 |
| arno11 | i wonder why | 13:59 |
| Wizzup | arno11: we can enable it in kernel if you want, what would it do? | 14:31 |
| Wizzup | wouldn't tha tjust let userspace decide? | 14:31 |
| arno11 | maybe just enabling it could be useful for testing purpose (supposed to works better than ondemand, but not on all devices apparently) | 14:34 |
| arno11 | if we keep ondemand as default, no trouble and we can test schedutil | 14:34 |
| arno11 | *from userspace | 14:35 |
| arno11 | actually ondemand only switch from 250 when doing nothing to 600 when doing anything, that's it lol. so not really optimised | 14:45 |
| arno11 | schedutil is supposed to predict what to do with freqs and switching them in a clever way but not sure it works fine with arm | 14:47 |
| arno11 | so maybe creating and tweaking our own gov is better, i don't know | 14:48 |
| arno11 | anyway, all actual default govs seem obsolete for phones | 14:54 |
| Wizzup | I don't know if we need to go super crazy with this, but we can enable some others if you want to toy with it | 14:55 |
| arno11 | oh, ofc lot of other things are far more important, sorry. but indeed, i'd like to toy a bit with govs | 14:58 |
| freemangordon | I don;t think we need more schedulers | 14:58 |
| freemangordon | cgroups/nice is what we have to implement | 14:58 |
| freemangordon | because all we want is responsive UI, this is not some server we shall optimize | 14:58 |
| arno11 | sure for responsive UI, but the way ondemand gov (and others) works actually can't help for cpu life, temp and current draw imo | 15:01 |
| Wizzup | yes, ondemand is not ideal for phones, also pm draw wise | 15:02 |
| arno11 | yes and conservative gov is...well..to conservative using too much the lower freq. Performance gov only use max freq and powersave only use the min freq. | 15:05 |
| arno11 | all of this is not really 'well balanced' | 15:06 |
| Wizzup | freemangordon: all translations went through apart from calendar-l10n and maybe yours | 15:07 |
| freemangordon | 'mine' went few hours ago | 15:07 |
| Wizzup | ok | 15:20 |
| Wizzup | so just calendar-l10n then | 15:20 |
| Wizzup | freemangordon: we probably also need to (later) upgrade the debian/control entries when new languages are added | 16:58 |
| freemangordon | yeah | 17:57 |
| Wizzup | freemangordon: ok, going to look at plugin now | 19:56 |
| Wizzup | freemangordon: username branch, yeah? | 20:00 |
| Wizzup | freemangordon: https://github.com/libcgroup/libcgroup/blob/main/README is this is for v2 we might have something | 20:01 |
| Wizzup | https://github.com/libcgroup/libcgroup/blob/main/README_daemon | 20:01 |
| Wizzup | https://forum.artixlinux.org/index.php/topic,1156.0.html | 20:03 |
| Wizzup | freemangordon: what is this sign in stuff and does it require some special tp interface? | 20:08 |
| Wizzup | like this is not necessary for irc right? | 20:21 |
| Wizzup | hmm looks like it is | 20:21 |
| Wizzup | freemangordon: libquotient (and thus tp-tank) requires the user field to also contain the server | 20:30 |
| Wizzup | oh, it seems to use QUrl.isValid | 20:31 |
| Wizzup | freemangordon: ok, with some minor tweak the 'sign in' worked | 20:43 |
| Wizzup | not a good change but just to test: | 20:43 |
| Wizzup | - rtcom_account_item_store_param_string(item, "user", arr[0]); | 20:43 |
| Wizzup | + rtcom_account_item_store_param_string(item, "user", g_value_get_string(account)); | 20:43 |
| Wizzup | if we just make this change now then I think we're good | 20:47 |
| Wizzup | freemangordon: now is the last change to tell me to change the 'make it work' commit msg | 21:15 |
| Wizzup | otherwise I will build, and also push your accounts change to stable when you're ok with it | 21:15 |
| freemangordon | Wizzup: can't parse | 21:19 |
| freemangordon | most of it :) | 21:19 |
| freemangordon | but, how is 'user' differenet to 'account' then? | 21:19 |
| freemangordon | and if 'user' contains domain, what is 'server' then | 21:20 |
| freemangordon | ? | 21:20 |
| freemangordon | Wizzup: also, did you test both ways of entering account? | 21:20 |
| freemangordon | please undefine https://github.com/maemo-leste-extras/rtcom-accounts-plugin-matrix/blob/master/src/matrix-plugin.c#L33 and check it | 21:22 |
| freemangordon | I think with this undefined it looks better | 21:23 |
| freemangordon | but I left it to you to decide | 21:23 |
| Wizzup | freemangordon: I tested both ways | 21:28 |
| Wizzup | I liked with it defined, but that's mostly because it just said "username" | 21:29 |
| Wizzup | 21:19 < freemangordon> but, how is 'user' differenet to 'account' then? | 21:29 |
| Wizzup | it's not currently in tp-tank/libquotient | 21:29 |
| freemangordon | heh | 21:29 |
| Wizzup | but libquotient does something dumb here | 21:29 |
| freemangordon | or we use it in a dumb way? | 21:29 |
| freemangordon | but anyway, not so important | 21:29 |
| Wizzup | no | 21:29 |
| Wizzup | noI checked | 21:29 |
| Wizzup | we just pass it the server url and it says it's not a valid server url | 21:30 |
| Wizzup | anyway, you also currently need to enter an '@' in front of the username in the split screen | 21:30 |
| Wizzup | which is not ideal IMO | 21:30 |
| freemangordon | sure, why not? | 21:30 |
| Wizzup | otherwise it doesn't work | 21:30 |
| Wizzup | the @ is not really part of the username | 21:30 |
| freemangordon | wasn't that the whole point of you having issues entering username? | 21:31 |
| freemangordon | I am lost, officially : | 21:31 |
| freemangordon | Matrix user IDs (MXID) are unique user IDs. They are in the format @username:homeserver. tld. | 21:31 |
| Wizzup | no, the problem was rtcom-accounts-ui rejecting an account formed as '@uesrname:server.tld' | 21:31 |
| Wizzup | so either: | 21:31 |
| Wizzup | userfield: wizzup | 21:31 |
| Wizzup | server: server.tld | 21:31 |
| Wizzup | OR | 21:31 |
| Wizzup | user: @wizzup:server.tld | 21:32 |
| freemangordon | oh, I see | 21:32 |
| Wizzup | but userfield: @wizzup and server: server.tld is weird | 21:32 |
| Wizzup | can we push the rtcom lib fix to stable? | 21:33 |
| freemangordon | so, just prepend '@' to 'account' or 'user' or whatever in https://github.com/maemo-leste-extras/rtcom-accounts-plugin-matrix/blob/master/src/matrix-plugin.c#L89 | 21:33 |
| Wizzup | yeah | 21:33 |
| Wizzup | so I pushed to master on the plugin | 21:34 |
| freemangordon | Wizzup: if rtcom lib == rtcom accounts ui, then yes | 21:34 |
| Wizzup | please let me know if you want to force push to change your commit msg | 21:34 |
| Wizzup | freemangordon: yes that is what I meant @ stable | 21:34 |
| freemangordon | I am fine | 21:34 |
| Wizzup | ok | 21:34 |
| freemangordon | like, I just put some commit message for the POC | 21:34 |
| freemangordon | do you want me to push it to stable? | 21:35 |
| freemangordon | ( rtcom accounts ui) | 21:35 |
| Wizzup | I am doing it now but you can do it too | 21:35 |
| freemangordon | I will | 21:35 |
| Wizzup | ok | 21:35 |
| freemangordon | building | 21:37 |
| Wizzup | cool | 21:37 |
| Wizzup | freemangordon: also please see my above cgroups msgs | 21:37 |
| Wizzup | I think I found what we can use | 21:38 |
| freemangordon | I saw it | 21:38 |
| freemangordon | but not sure I can grok now | 21:38 |
| freemangordon | like, I think what we have in chimaera does not support v2 | 21:39 |
| freemangordon | but, might be wrong | 21:39 |
| Wizzup | yes it does | 21:41 |
| Wizzup | well, let me double check | 21:41 |
| Wizzup | just set rc_cgroup_mode="unified" | 21:42 |
| freemangordon | heh: | 21:51 |
| freemangordon | Makefile.am: error: required file './AUTHORS' not found | 21:51 |
| Wizzup | lol | 21:55 |
| Wizzup | freemangordon: don't set the above key in rc.conf, this currently results in unbootable vm | 21:56 |
| Wizzup | I will debug later, but our openrc def. does v2 as well | 21:56 |
| freemangordon | for sure I will not | 21:59 |
| freemangordon | (rc.conf) | 21:59 |
| Wizzup | in any case, the right tool exists | 21:59 |
| Wizzup | and it does the right job :) | 21:59 |
| freemangordon | great | 21:59 |
| Wizzup | v2 is mounted at /sys/fs/cgroups/unified | 21:59 |
| Wizzup | checking for libglade-2.0... no | 22:02 |
| Wizzup | configure: error: Package requirements (libglade-2.0) were not met: | 22:02 |
| Wizzup | No package 'libglade-2.0' found | 22:02 |
| Wizzup | heh | 22:02 |
| Wizzup | you removed that I bet | 22:02 |
| freemangordon | yes, as it is not needed | 22:03 |
| freemangordon | but forgot to remove it from configure.ac | 22:03 |
| freemangordon | sorry | 22:03 |
| freemangordon | do you want me to fix it? | 22:06 |
| Wizzup | no it is ok | 22:07 |
| Wizzup | I guess I still need to fix the maemo ham icon | 22:18 |
| freemangordon | also the matrix icon (in accounts ui) looks ugly | 22:19 |
| Wizzup | it does, but this is the icon that came with tp-tank | 22:20 |
| Wizzup | we can replace it of course | 22:20 |
| freemangordon | the problem is that is black/white | 22:20 |
| Wizzup | ok it works on my d4 | 22:21 |
| freemangordon | cool | 22:21 |
| Wizzup | I forgot to write code to append '@' sign though | 22:21 |
| Wizzup | maybe we just undef and go with that? | 22:21 |
| freemangordon | I think so | 22:21 |
| freemangordon | there you also has placeholder | 22:22 |
| freemangordon | *have | 22:22 |
| Wizzup | placeholder for? | 22:25 |
| freemangordon | matrix id | 22:25 |
| freemangordon | BTW, consider https://printerval.com/uk/the-matrix-code-circle-sticker-3-p17716 for icon | 22:26 |
| freemangordon | or something like | 22:26 |
| Wizzup | if you can find it at 48x48, let me know | 22:27 |
| Wizzup | I don't think they intended the logo to be like this btw | 22:27 |
| Wizzup | https://en.m.wikipedia.org/wiki/File:Matrix_logo.svg | 22:27 |
| freemangordon | yes, I saw that, but IMO it is useless | 22:28 |
| freemangordon | as icon that is | 22:28 |
| freemangordon | didn;t we have some artist/designer here? | 22:30 |
| freemangordon | going to have some sleep, night! | 22:34 |
| Wizzup | freemangordon: well we should use their logo I think | 23:05 |
| Wizzup | gn | 23:06 |
| uvos | square logo for matrix is usualy [m] | 23:06 |
| Wizzup | yeah it's pretty ugly | 23:15 |
Generated by irclog2html.py 2.17.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!
