**** BEGIN LOGGING AT Sat Aug 13 02:59:59 2016 Aug 13 06:15:51 ka6sox: Thnx working indeed Aug 13 07:11:39 Morning! Aug 13 07:12:01 Herrie: I think I've now merged most of filmor's improvements Aug 13 07:12:06 I'll PR that Aug 13 07:21:04 Herrie: so, with the updated PR, we'll now need /var/preferences/com.palm.purple/transport directory and com.palm.purple.config:1 dbkind. Also we'll need that db to be populated by imaccountvalidator. Aug 13 07:22:11 I've tested the PR on qemu, it seems ok. Aug 13 07:30:21 Tofe: I guess we can add a mkdir or similar to the imaccountvalidator or imlibpurpleservice recipe Aug 13 07:30:29 Similar to what we did for contact linker plugins Aug 13 07:39:08 Tofe: I guess we should compare the imaccountvalidator code with filmors too. Changes seem very minimal from what I can see. It are 5 cpp files and 4 headers Aug 13 07:39:52 Yes Aug 13 07:42:15 I see nothing that we'd really need, but a 2nd pair of eyes doesn't hurt Aug 13 07:52:49 In the IMAccountValidator we have the following already: purple_util_set_user_dir("/var/preferences/com.palm.imaccountvalidator") Aug 13 07:53:16 Well, we'll have to choose, at some point Aug 13 07:53:16 Might be an idea to see if we can adjust your path to something similar ? Aug 13 07:56:47 mmh I'm not so sure we need to have similar paths, here Aug 13 07:57:24 and if I had to choose, I prefer /var/preferences/com.palm.purple/transport, it looks more generic Aug 13 07:58:16 Tofe: Yeah true. But we call it libpurple everywhere I think (db kinds). So would be good to have it consistent. Aug 13 07:58:33 Also the whitespaces are a bit messy on GitHub. Aug 13 07:59:11 codewise would be good if someone can have a look who knows c++ better ;-) Aug 13 07:59:20 nizovn: ^ Aug 13 08:39:47 Herrie: I've just realized that we are missing from pieces related to the configuration Aug 13 08:40:21 There should be two more methods proposed for each capabilityProvider: onCreate and onDelete Aug 13 08:41:06 In our templates you mean? Aug 13 08:41:11 yes Aug 13 08:41:24 OK I don't think filmor had these or I must have overlooked it Aug 13 08:41:30 I've just merged their implementation in imlibpurpleservice Aug 13 08:41:58 It's what create's deletes entries in the purple.config db Aug 13 08:42:32 creates/deletes Aug 13 08:42:56 Though I couldn't find where these are called precisely Aug 13 08:42:56 Ah it's here: https://github.com/filmor/webos-messaging/blob/021600c7fa1052641ab277e97f968f807c36495e/accounts/prototype.json Aug 13 08:43:06 yup Aug 13 08:43:23 filmor had some python script to generate templates dynamically Aug 13 08:44:47 ah, it's the accounts app that calls it Aug 13 08:44:52 Yeah Aug 13 08:44:59 Well those are easy to add in general Aug 13 08:45:06 I guess I need to cleanup the templates anyway Aug 13 08:45:17 Since they're not very consistent it seems Aug 13 08:45:17 so that's probably one of the last missing piece for the config Aug 13 08:45:25 I'll get to that :) Aug 13 08:45:28 ok :) Aug 13 08:46:37 Seems we're only doing onenabled currently Aug 13 08:46:43 exaclty Aug 13 08:48:09 Let me add the oncreate & ondelete quickly for AIM, ICQ, Skype Aug 13 08:48:19 I'll do a proper cleanup tonight for all I guess Aug 13 08:48:33 Going through the prototype.json and see what else I can add Aug 13 08:48:41 And standardize our templates a bit Aug 13 08:49:09 Herrie: don't add "prpl", as we are not using it currently and it would just bring confusion Aug 13 08:49:42 oh, it's already there Aug 13 08:50:01 ah no, just on the account itself Aug 13 08:50:25 weird place to put it Aug 13 08:51:02 I can remove it if you want? Aug 13 08:51:09 But I think we use it in the validator app Aug 13 08:51:26 if it's already there, we can let it be Aug 13 08:51:59 Seems we use it Aug 13 08:52:22 https://github.com/webOS-ports/messaging-accounts/blob/master/application/source/Validator.js#L73 & https://github.com/webOS-ports/messaging-accounts/blob/master/application/source/Validator.js#L39 Aug 13 08:52:36 So let's leave it for now and look at this later Aug 13 08:52:47 yeah Aug 13 08:54:30 You should still update the config kind name ;) Aug 13 08:54:33 Also in added commit Aug 13 08:54:34 Herrie: I've modified only the jabber one, and now I'm logged in into GTalk :p Aug 13 08:54:40 :D Aug 13 08:54:45 I got GTalk too :P Aug 13 08:54:59 herrie82 ;) Aug 13 08:56:59 Tofe: https://github.com/webOS-ports/messaging-accounts/commit/062005ccd40c42c0017f2e0af915f0329de38f63 Aug 13 08:57:43 I pushed it to a separate branch for now Aug 13 08:57:56 I think it'd be worth having it for Jabber Aug 13 08:58:22 Let me add it quickly Aug 13 08:59:02 In general I'm not sure we should have this all in the templates if it's the same for all anyway Aug 13 08:59:11 We might just as well hard code it in our custom validator Aug 13 08:59:27 mmmh libpurple didn't like me adding you... Aug 13 08:59:38 LOL Aug 13 08:59:50 Maybe you need to add full address? herrie82@gmail.com Aug 13 08:59:52 even on desktop, my account now refuses to log in :p Aug 13 08:59:58 Not sure how GTalk works Aug 13 09:00:19 I've put the address already, it's how a jabber address is supposed to look like anyway Aug 13 09:00:32 Hmmz :S Aug 13 09:00:51 Anyway I pushed the Jabber change: https://github.com/webOS-ports/messaging-accounts/commit/eff2f4eb4ce50351365fefaa459cac9c3d3ab4e3 Aug 13 09:01:19 thanks Aug 13 09:05:26 Tofe: We also got some user feedback on the release :) http://forums.webosnation.com/luneos/331330-pivotce-luneos-august-stable-release-cafe-touba.html Aug 13 09:05:43 Will see if I can fix some of them, but might need some help from you or nizovn on the audio bits ;) Aug 13 09:12:00 Wow, daily driver use? These people are real enthousiasts :) Aug 13 09:35:04 Tofe: Yeah LOL ;) Aug 13 09:35:17 Well I can switch when some things work a bit better Aug 13 09:35:23 Not that far off really tbh Aug 13 09:36:10 But then again I'm not that demanding ;) I play music, do calls, SMS, IM, Twitter and that's about it ;) Aug 13 09:36:15 Occasional browsing Aug 13 09:36:21 No, not that far, that's true; it's just that I would be tempted to hack around all the time, and it would certainly end up with a disaster :p Aug 13 09:36:38 Tofe: Yeah LOL Aug 13 09:44:21 Pretty amusing that I'm still under total 5 posts on the forum :p Aug 13 09:51:27 Tofe: LOL ;) Aug 13 10:16:59 Herrie: http://paste.ubuntu.com/23051478/ Aug 13 10:17:27 I'll add the owner field Aug 13 10:30:07 OK Aug 13 10:31:29 ... but now it refuses to log in :( Aug 13 10:34:26 Too many attempts maybe or some silly bug somewhere? Aug 13 10:37:18 ok no it was just me. Aug 13 10:37:36 but the config.libpurple db isn't populated, though Aug 13 10:39:46 ah but I forgot to modify the providers... I have to get some lunch :p Aug 13 10:47:25 morning Aug 13 10:53:43 hi all Aug 13 10:54:34 @Herrie, if you want, I can make the template-templating more standalone Aug 13 10:55:09 you need python installed for bitbake anyhow, so having that as an additional build-step of the account-validator should be fine Aug 13 10:55:25 it's really a pain to modify all of those manually Aug 13 11:50:57 filmor: That would be nice in general :) Aug 13 11:51:35 filmor: Then again we only need to make them once, so it's not that much work... But i guess with the translations etc it might become more :P Aug 13 11:52:45 filmor: We're on python 2 now I think, but bitbake will soon switch to 3, so would be good if it's compatible with both ;) Aug 13 11:53:38 nizovn: morning btw ;) Aug 13 11:54:09 nizovn: When you have some time, can you check https://github.com/webOS-ports/imlibpurpleservice/pull/5 to see if you see anything odd? Aug 13 12:32:34 Herrie: looks ok, except two mentioned memory leaks Aug 13 13:03:51 Herrie: have you tried to reproduce suspend resume issue on touchpad, by chance? Aug 13 13:12:21 nizovn: Not yet but might be able to do tonight. What I need to do again exactly? Aug 13 13:12:29 I'll flash TP with latest nightly and see Aug 13 13:13:26 nizovn: I don't see comments in the PR or I'm not looking at the right place? Aug 13 13:15:09 yes, the leaks are mentioned in PR's code Aug 13 13:16:01 resume issue: make sure usb is disconnected, turn off screen, wait 10 sec, and press power button Aug 13 13:30:36 nizovn: OK and then it won't turn on or.... ? Aug 13 13:30:50 Tofe: I got an email invitation for Hangouts instead of something in GTalk ;) Aug 13 13:30:54 Not sure what's up with that Aug 13 13:31:58 Herrie: yes, it will reboot Aug 13 13:34:07 nizovn: OK need to head out now, but will put latest nightly to download so I can test when I'm back later Aug 13 13:36:39 ok Aug 13 13:58:44 Herrie: ok... mmhh... Aug 13 14:00:10 nizovn, Herrie: yes the remarks about the leaks come from filmor, which I left as-is. I didn't find any obvious solution, so I just merged it. Anyway it should leak far less than legacy's code, I think. Aug 13 14:53:20 We have valgrind in the images so once stuff is working we can have a look to see if it gives something useful Aug 13 15:38:45 Herrie, I just configured a monitoring check for the DNS resolution of webos-ports.org, along with IRC notifications Aug 13 15:39:03 so if my previous changes don't resolve the DNS issue, we should know about it quickly after it happens at least Aug 13 15:39:38 Herrie: oh but for the 2 leaks in the PR, we know they exist, it's just not sure how to fix it Aug 13 18:01:34 Tofe: OK Aug 13 18:01:44 Did you fix the config bits that you discovered before lunch? Aug 13 18:02:13 yes Aug 13 18:02:21 it's in the later commits Aug 13 18:05:06 Tofe: OK so it's ready to merge? Aug 13 18:05:11 nizovn: flashing TP now Aug 13 18:05:11 mmmh how long does it take for webosnation to moderate a single post ? I still don't see my post of noon Aug 13 18:05:20 Herrie: oh, yes, it is Aug 13 18:05:23 Tofe: Not sure, it are US guys Aug 13 18:05:27 Mainly Aug 13 18:06:05 And I guess that with a post count of "1", I'm a low priority :D Aug 13 18:07:39 I see 1 post from you on the ringtone bits ;) Aug 13 18:09:38 on the touba thread? Aug 13 18:09:45 I don't see it myself, it's weird Aug 13 18:13:19 Seems I'm moderator on LuneOS part :P Aug 13 18:13:23 Now it's approved :D Aug 13 18:13:30 I guess Alan sorted that sometime :P Aug 13 18:14:36 hehe :p Aug 13 18:15:45 cryptk: Thnx Aug 13 18:18:12 Tofe: Is this syntax correct? https://github.com/webOS-ports/imlibpurpleservice/blob/master/inc/LibpurpleAdapter.h#L37 Aug 13 18:18:31 Looks weird to me without any concatenation marker in between APP_PATH and "plugins" ? Aug 13 18:18:50 Then again I don't know C++ :P Aug 13 18:19:21 nizovn: On my TP4G I get a screen with blue vertical lines when I switch the screen back on Aug 13 18:19:32 yup, writing "a" "b" results in the string "ab" (not sure if it's the preprocessor or the compiler itself) Aug 13 18:20:01 It's also valid in C Aug 13 18:20:56 Tofe: OK didn't know that, learned something new Aug 13 18:24:11 String literals placed side-by-side are concatenated at translation phase 6 (after the preprocessor). That is, "Hello,"  " world!" yields the (single) string "Hello, world!" (from http://en.cppreference.com/w/cpp/language/string_literal ) Aug 13 18:27:00 I'm used to the way JS and some other languages do it ;) Aug 13 18:30:18 Tofe: https://github.com/webOS-ports/messaging-accounts/pull/8 Aug 13 18:30:37 I did this quickly since filmor offered to make the python templates stuff work, so we don't need to manually do it Aug 13 18:30:42 So did this quickly for now so they work Aug 13 18:36:30 Herrie: one question about these templates: is it normal we have ids ending up like "aim.aim", "icq.icq", etc? Aug 13 18:38:02 Tofe: Probably not :S Aug 13 18:38:46 It's all dynamic, so it doesn't matter much, but it looks weird Aug 13 18:40:25 Yeah it does Aug 13 18:40:37 I can imagine it conflicts with templateId somehow Aug 13 18:40:39 But it shouldn't Aug 13 18:41:11 I'll update them and kick off a new nightly Aug 13 18:41:19 Want to do some more ICQ test? Aug 13 18:41:23 Since you now have config? Aug 13 18:42:25 well, I still don't see the configs in the db Aug 13 18:42:32 Herrie: but after that it reboots? Aug 13 18:42:41 though it has to work a bit better, because jabber works Aug 13 18:44:30 nizovn: No it was stuck with me Aug 13 18:44:49 Needed to hold center+power for 30s Aug 13 18:56:15 Herrie: ok, it looks like the same issue as on my TP Aug 13 18:56:44 i think it's quite important/critical. need to think about it.. Aug 13 18:59:07 nizovn: Anything in logs? Aug 13 18:59:52 Tofe: I kicked off a new nightly with all changes :) Aug 13 19:12:37 Herrie: yes, looks like kernel bug https://bpaste.net/show/fdeb30821520 Aug 13 19:21:09 nizovn: Yeah but we didn't touch kernel in like forever Aug 13 19:21:18 So it must be in combination with some other change Aug 13 19:21:24 Maybe enabled ALS? Aug 13 19:27:02 not sure, maybe we have this bug for a long time Aug 13 19:43:06 nizovn: ALS should be easy to test by disabling it in the luna.conf Aug 13 19:44:30 Let me try that now Aug 13 19:47:39 That seems to help at least on first try Aug 13 19:47:43 Let me try a few more times Aug 13 19:47:58 I'll boot a few times and see if it keeps behaving Aug 13 20:01:47 nizovn: No it's not ALS it seems. It still crashes. Aug 13 20:02:03 It does seem a bit random. It doesn't happen every time, but majority of times it does. Aug 13 20:04:18 Herrie|Laptop: ok, thanks **** ENDING LOGGING AT Sun Aug 14 02:59:58 2016