**** BEGIN LOGGING AT Mon Aug 08 02:59:59 2016 Aug 08 06:02:48 morning Aug 08 06:03:08 My ICQ message arrives, though doesn't display in messaging app properly Aug 08 06:03:41 (There was an error receiving this message. Either you and 281884312 have different encodings selected, or 281884312 has a buggy client). Aug 08 06:04:05 Outgoing message seems treated as SMS instead of IM. I'll look into that. It's probably Messaging that needs updating for that :P Aug 08 06:54:38 Herrie: you can also do a push --force tofe/lastest:webOSports-master, after having done a push of the current master into another branch Aug 08 06:54:41 Morning! Aug 08 07:22:02 morning Aug 08 07:38:14 Tofe: Ah let me try that Aug 08 07:40:24 Ah seems we're harcoding SMS as serviceName currently, that for sure doesn't help :P Aug 08 08:02:32 Tofe: Hmmz the qtwebengine repo is quite big to clone :P Aug 08 08:03:05 Not sure I can do that here in office. In case you have access, feel free to do the forced push to webOS-ports/master from tofe/latest Aug 08 08:03:52 I made a backup of current webOS-ports/master Aug 08 08:59:39 Herrie|Pre3: I don't have access to my clone from office, so that'll have to wait till tonight Aug 08 09:00:06 Tofe: OK. I can do it then :P Aug 08 09:00:59 Because I have it at home too :P Aug 08 09:01:25 :) Aug 08 09:03:15 Just thought maybe you have access :P Aug 08 11:40:38 JaMa: morning Aug 08 11:42:10 Thnx for merging. We did some local upgrade to latest from upstream qtwebengine 5.6 branch and chromium. We can switch to proper 5.6.1 after the release in a few days. Aug 08 11:42:20 Probably Wednesday or so. Aug 08 11:58:20 oki Aug 08 11:58:33 do you want me to upgrade meta-starfish pin with script? Aug 08 11:59:05 meta-starfish? Aug 08 11:59:11 That's not our repo :P ? Aug 08 11:59:15 heh Aug 08 11:59:18 meta-smartphone Aug 08 11:59:48 JaMa: Not sure what you mean, bump layers.txt with all the changes? Aug 08 11:59:52 Or.... ? Aug 08 12:01:09 We're using HEAD for meta-smartphone in testing ;) Aug 08 12:02:29 yes, that's what I meant, we aren't using HEAD for stable branch, do we? Aug 08 12:03:10 No stable is commit for each layer Aug 08 12:09:43 what's the name for next release? Aug 08 12:10:54 Touba Aug 08 12:15:15 thanks, I've updated all 3 relevant branches in webos-ports-setup and added tags for tobio and touba release Aug 08 12:19:17 JaMa: Thnx Aug 08 15:54:19 Tofe: ping Aug 08 16:44:34 Herrie: pong Aug 08 16:47:34 Tofe: How can I get the Browser app to do proper full screen? Aug 08 16:47:48 I see we have some fullscreen mode in LNC but not sure how to call this from app? Aug 08 16:50:31 Not sure either, let me see Aug 08 16:51:45 How did legacy achieve this ? was there a dedicated LS2 call ? Aug 08 16:52:38 Not sure let me see Aug 08 16:53:22 "window.PalmSystem.enableFullScreenMode(true);" ? Aug 08 16:54:35 Of course PalmSystem isn't visible for a QML app (which may be a bug, we'll have to see) Aug 08 16:56:25 Anyway it's not yet implemented in our PalmSystem... I say, let's do it! we propose an LS2 call in luna-next, and we also call it from PalmSystem. Aug 08 16:56:44 Give me an hour or so Aug 08 16:59:27 Browser after my fix behaves pretty well but we should hide statusbar + gesture area as well. Or at least statusbar Aug 08 16:59:43 Tofe: Sure :) Aug 08 17:02:28 mmh... How do we get out of fullscreen, it the app doesn't respond ? Aug 08 17:14:24 That's a good question... Legacy used the gesture area or button I think... Aug 08 17:15:29 Let me see Aug 08 17:19:15 Ok we should hide both statusbar and notificationarea it seems: https://github.com/openwebos/luna-sysmgr/blob/1393f0af5dd8d9f0e9cc79627f4acb226f8c8d45/Src/base/SystemUiController.cpp#L1390 Aug 08 17:22:09 yup Aug 08 17:31:06 Tofe: I guess we could do a power press to exit full screen? Aug 08 17:31:17 We could do that Aug 08 17:32:49 Tofe: The LNC is indeed working with your change Aug 08 17:32:53 However the icons are too small Aug 08 17:33:00 About half the height they should be Aug 08 17:33:28 It could be because it's set onCompleted we'd need to hard specify the height upfront Aug 08 17:33:48 This is the dashBoard icon when it's minimized Aug 08 17:33:51 I'll play a bit Aug 08 17:43:35 I've got some bits of fullscreen working, but hiding status bar and notifications might take a bit more time Aug 08 17:47:01 Tofe: OK Aug 08 17:48:02 Herrie, we are going to need to do a LONG Full Stop the server that our infra is on (well GH will carry the development for a while) Aug 08 17:48:13 the builder will be up but NOT Jenkins Aug 08 17:48:35 so essentially about 16hrs of work needs to happen...date to be determined but NOT this week. Aug 08 17:49:55 ka6sox: OK no problem. We'll push a release out shortly Aug 08 17:50:00 Afterwards you can go ahead Aug 08 17:50:12 With shortly I mean today/tomorrow timeframe Aug 08 17:50:19 What you're planning to do? Aug 08 17:51:55 Ok, fullscreen seems to work Aug 08 17:52:00 Tofe: Nice :D Aug 08 17:52:07 I'll push that, so you can play with browser a bit Aug 08 17:53:26 Tofe: OK :D Aug 08 17:56:41 it's pushed on my tofe/fixes branch Aug 08 17:57:06 You'll have to use luna://org.webosports.luna/enableFullScreenMode {appId: , enable: } Aug 08 17:57:23 (enable is not a string, but a boolean) Aug 08 17:57:45 Tofe: Thnx... Can you have a look at this mini icon for Dashboards? I tried a few things but no luck. I suspect https://github.com/webOS-ports/luna-next-cardshell/blob/herrie/iconfinal/qml/Notifications/NotificationArea.qml#L215 or https://github.com/webOS-ports/luna-next-cardshell/blob/herrie/iconfinal/qml/Notifications/NotificationArea.qml#L223 is the issue Aug 08 17:57:59 You even have an example for free: https://github.com/webOS-ports/luneos-components/commit/0ddbcf2b45efe82c33a168cc6c419296e5779ea6 Aug 08 17:58:37 Herrie: I just checkout your branch and start on desktop ? Aug 08 17:58:46 Tofe: I tested on N4 Aug 08 17:58:52 Not sure how it behaves on desktop Aug 08 17:59:06 But it seems Dashboard minimized view only Aug 08 17:59:51 behaves correctly on desktop, I'll have to try on qemu (my N4 build isn't up to date) Aug 08 18:00:00 Tofe: OK Aug 08 18:00:09 It's not a blocked, but a sligth annoyance :P Aug 08 18:00:34 s/d/r s/sligth/slight Aug 08 18:00:54 They seem OK in Testr but not from email Aug 08 18:01:05 It actually might be the miniicon only that's an issue :s Aug 08 18:01:16 miniicon/smallicon that is Aug 08 18:06:34 mmh we have both anchors.fill and anchors.centerIn... That's not good. Let's just keep anchors.fill. in NotificationItem:40 Aug 08 18:07:38 Let me try that Aug 08 18:08:03 Nope same Aug 08 18:08:10 You're able to replicate in general? Aug 08 18:08:16 I can show some screenshot Aug 08 18:08:18 Didn't try yet :p Aug 08 18:08:22 but I'll do that now :) Aug 08 18:08:57 http://tinypic.com/r/2lt0aw4/9 Aug 08 18:09:02 It's really small Aug 08 18:09:32 I have one with a proper clock next to it now Aug 08 18:09:39 Let me put that so you can see Aug 08 18:10:13 http://tinypic.com/r/u1n5y/9 Aug 08 18:10:53 ok yes Aug 08 18:11:53 NotificationArea line 225, let's add width: height Aug 08 18:19:03 How do you create a miniicon notification with the email app ? Aug 08 18:22:29 Tofe: Add an account in Accounts app, wait for it to sync Aug 08 18:22:34 And new email notif Aug 08 18:24:26 Tofe: The width = height seems to work for the miniicon Aug 08 18:24:41 ok good, because my mail icon is the right size Aug 08 18:31:48 Now the minicon actually seems bigger a bit compared to the normal minimized dashboard one Aug 08 18:33:08 But could be just these icons somehow Aug 08 18:33:09 http://tinypic.com/r/6rkn08/9 Aug 08 18:33:39 no it's fine I think Aug 08 18:34:02 It's for sure better Aug 08 18:36:41 Tofe: PR updated & ready to merge :) Aug 08 18:36:47 I took into accoutn your comments Aug 08 18:39:56 ok Aug 08 18:40:39 Tested & I cannot find issues anymore :P Aug 08 18:40:43 For now Aug 08 18:41:03 merged Aug 08 18:42:19 Thnx Aug 08 18:42:33 I have one more for Testr soon and will kick off some release test images Aug 08 19:06:12 OK Testr seems fine too - PCM audio Aug 08 19:06:18 Not sure what's up with that Aug 08 19:12:00 Since you said it worked in browser. Anyway nothing too urgent Aug 08 19:13:39 Tofe: That git push you proposed doesn't work... Aug 08 19:14:22 oh :( Aug 08 19:15:14 Anyway will ask JaMa: Aug 08 19:15:16 JaMa: ping Aug 08 19:25:54 OK stable builds are running :D Aug 08 19:27:42 Tofe: I'll try and see if I can get IM sending working :P Aug 08 19:29:10 Shouldn't be too hard it seems Aug 08 19:38:52 Tofe: Seems we have a small issue still with imlibpurpleservice... Aug 08 19:39:19 Aug 08 07:50:19 mako ls-hubd[699]: [2016-08-08 07:50:19:658] [0x441ba000] [error] [imlibpurple.serviceApp]: updateSyncStateRecord: unknown serviceName type_icq. No syncState record created. Aug 08 19:39:43 Which might be the root cause of the serviceName not populating correctly further down the line. Aug 08 19:40:30 I guess this is the culprit: https://github.com/webOS-ports/imlibpurpleservice/blob/master/src/IMLoginState.cpp#L1464 Aug 08 19:42:01 Seems we had a few more in the legacy template, but it's hardcoded. A solution similar to the one you came up yesterday would be nice: https://github.com/wosigh/messaging-plugins/blob/master/3.0/Compiler%20Source%20Files/imlibpurpleservice-1.0/src/IMLoginState.cpp#L1392 Aug 08 19:43:20 And a few more that are nasty it seems: https://github.com/webOS-ports/imlibpurpleservice/blob/2b76134fd563ad6232bfdf715275d335fbad26ee/src/OnEnabledHandler.cpp#L146 Aug 08 19:44:16 So stuff might actually work in Messaging, just before it hits there it has issues and gets defaulted to SMS ;) Aug 08 19:46:05 Yes... It still needs some work... Aug 08 19:46:30 If I would only know C++ :P Aug 08 19:47:31 I can do it in the current way but not do it properly :P Aug 08 19:50:09 Then again this list shouldn't change that much.... Aug 08 19:54:24 I'll have a look to see how we can do it in a generic way, it possible Aug 08 19:59:13 Something generic would be nicer of course Aug 08 19:59:38 But seeing there will be 1-2 changes a year normally doing it hardcoded isn't the end of the world, though not very pretty Aug 08 20:00:31 well, yes, but that way we also ensure that the capabilityId are what we've put in Aug 08 20:01:06 Yes Aug 08 20:01:11 True ;) Aug 08 20:01:19 I'll just see if it's simple; if it isn't, we fallback on the if..then list Aug 08 20:01:32 There must have been a reason for the current way, or maybe just laziness :P Aug 08 20:01:47 Starts with 1 or 2 then gradually expands :P Aug 08 20:11:40 looks like it won't be that easy... Aug 08 20:14:13 Tofe: OK, I'll just hardcode it for now then? Aug 08 20:14:24 but wait -- we can have several capabilityProvider for one account ? Aug 08 20:15:14 Tofe: Theoretically yes: https://github.com/webOS-ports/org.webosports.service.contacts.carddav/blob/master/accounts-google/account-template.json#L37 Aug 08 20:15:22 However for IM that shouldn't be the case normally... Aug 08 20:15:36 I.e. if we assume we don't mix IM templates with other like legacy did Aug 08 20:15:45 Legacy had Google for email, contacts, calendar + Gtalk Aug 08 20:15:50 For example Aug 08 20:16:04 And AOL for AIM, Email, Contacts Aug 08 20:16:05 ah, yes, there's also a login state for email for instance Aug 08 20:16:54 Let's just hardcode it for the moment. Aug 08 20:17:31 We could say that the IM ones should always start with org.webosports.messaging as the ID Aug 08 20:17:37 That's something we can easily control Aug 08 20:18:36 Or there seem to be a "busAddress": "com.palm.imlibpurple" Aug 08 20:19:51 Tofe: I guess that comes from https://github.com/webOS-ports/messaging-accounts/blob/master/accounts/org.webosports.messaging.aol.aim/org.webosports.messaging.aol.aim.json#L14 Aug 08 20:19:56 But not sure Aug 08 20:20:32 Do you have the com.palm.account.syncstate db kind ? Aug 08 20:22:02 mmh there's no index for accountId + busAddress.. Aug 08 20:22:29 Let's just hardcode it. If we find something more elegant we can always put it in. Aug 08 20:24:16 Ah it's hardcoded from here: https://github.com/webOS-ports/imlibpurpleservice/blob/7f37f87aa2c4fd20afb14c3a7a9ce84c570afd64/src/IMLoginState.cpp#L1538 Aug 08 20:24:22 Tofe let me see Aug 08 20:25:26 Tofe: This is on 3.0.5: https://bpaste.net/show/f10f15602d76 Aug 08 20:25:44 I can add an index there if needed Aug 08 20:26:53 Why not Aug 08 20:28:57 I'll leave it here for tonight Aug 08 20:29:14 Tofe: OK :) Aug 08 20:29:20 Do we need a default? Aug 08 20:30:33 Tofe: { Aug 08 20:30:33 "id": "com.palm.account.syncstate:1", Aug 08 20:30:33 "sync": false, Aug 08 20:30:33 "indexes": [ Aug 08 20:30:33 { Aug 08 20:30:34 "name": "basic", Aug 08 20:30:34 "props": [{ "name": "accountId" }, { "name": "capabilityProvider" }] Aug 08 20:30:35 }, Aug 08 20:30:35 { Aug 08 20:30:36 "name": "withcollection", Aug 08 20:30:36 "props": [{ "name": "accountId" }, { "name": "capabilityProvider" }, { "name": "collectionId" }] Aug 08 20:30:37 }, Aug 08 20:30:37 { Aug 08 20:30:38 "name": "accountAndCollection", Aug 08 20:30:38 "props": [{ "name": "accountId" }, { "name": "collectionId", "default": null}] Aug 08 20:30:39 }, Aug 08 20:30:39 { Aug 08 20:30:40 "name": "accountAndBusaddress", Aug 08 20:30:40 "props": [{ "name": "accountId" }, { "name": "busAddress", "default": null}] Aug 08 20:30:41 } Aug 08 20:30:41 ] Aug 08 20:30:42 } Aug 08 20:30:42 Oops Aug 08 20:30:43 https://bpaste.net/show/cdbf96523011 Aug 08 20:30:50 You can put this in: /etc/palm/tempdb/kinds/com.palm.service.accounts/com.palm.account.syncstate Aug 08 20:31:00 And run configurator or reboot Aug 08 20:34:04 could we put the libpurple busAddress as a default? Aug 08 20:34:42 mmh wait, if we have these two defaults, it means the search using only accountId is undefined Aug 08 20:35:09 ...or it could be I don't understand what I'm talking about :) Aug 08 20:35:25 I'm a bit tired, let's see tomorrow Aug 08 20:35:31 gn8! Aug 08 20:35:43 Tofe: gn8! Sleep does miracles Aug 08 21:17:52 Herrie: pong? Aug 08 21:20:07 JaMa: Stuff seems broken after you did things today Aug 08 21:20:12 Cannot run testing & stable Aug 08 21:20:25 I just bumped the meta-webos-ports to include 2 more commits on stable Aug 08 21:20:36 testing seems 5.6.1 acting up Aug 08 21:20:42 Stable I'm not sure what would cause it Aug 08 21:21:10 Looks like some python bits but not sure Aug 08 21:21:17 Testing: http://jenkins.nas-admin.org/job/luneos-testing_mako/lastBuild/console Aug 08 21:21:35 Stable: http://jenkins.nas-admin.org/job/luneos-stable_mako/lastBuild/console Aug 08 21:21:40 Or maybe we're out of space again? Aug 08 21:22:58 Ran a testing build y'day and days before without issues Aug 08 21:24:18 I have a fix for the testing build Aug 08 21:24:36 but stable failure looks like undeclared pcre dependency from gdk Aug 08 21:25:38 the qtbase issue in testing needs small change to enable eglfs PACKAGECONFIG Aug 08 21:25:49 which I forgot to merge to meta-webos-ports/krogoth Aug 08 21:26:01 but that shouldn't be in the stable build yet Aug 08 21:26:20 JaMa: Both pcre and gdk don't ring a bell to me tbh Aug 08 21:26:21 so I'll just revert this meta-qt5 upgrade until stable is out Aug 08 21:26:30 JaMa: That sounds good Aug 08 22:46:09 Herrie, you have 29GB available. Aug 08 22:46:18 so you aren't out of space **** ENDING LOGGING AT Tue Aug 09 02:59:58 2016