**** BEGIN LOGGING AT Thu Aug 28 02:59:58 2014 Aug 28 17:37:56 morphis: as for c++ side of #551 I need just to copy callback and post to another place?? Aug 28 17:39:59 nizovn: mainly, look at how the system manager service did it Aug 28 17:42:42 morphis: what to use instead of m_service? m_serviceHandlePublic? is there a typo that m_serviceHandlePublic=m_serviceHandlePrivate? Aug 28 17:43:36 you need to do that on the private bus Aug 28 17:43:41 so m_serviceHandlePrivate is right Aug 28 17:43:54 there are two ls2 buses Aug 28 17:43:57 private and public Aug 28 17:48:47 ok, just it was public in sysmgr. so https://github.com/webOS-ports/luna-appmanager/blob/master/Src/base/application/ApplicationManagerService.cpp#L8188 is not a typo? Aug 28 17:58:25 nizovn: possible Aug 28 18:04:49 morphis: maybe I'm wrong understand this. It should be public method on private bus? Aug 28 18:05:38 nizovn: give me some minutes Aug 28 18:05:42 will explain it then Aug 28 18:26:55 nizovn: I am ready now Aug 28 18:27:12 ok Aug 28 18:27:30 nizovn: you know dbus? Aug 28 18:27:47 no Aug 28 18:28:48 ok Aug 28 18:28:58 but you know about IPC in general? Aug 28 18:35:31 morphis:(reading about dbus)hm.. i used pipes, qt signal, i will find something if not understand you. Aug 28 18:36:16 nizovn: let me explain how ls2 works Aug 28 18:36:21 you have two buses Aug 28 18:36:27 a private and a public one Aug 28 18:36:37 normal applications and services on get access to the public bus Aug 28 18:36:49 privileged apps and services gets access to the private bus too Aug 28 18:37:02 so you have to look where you register your service methods Aug 28 18:37:08 either on the private or public one Aug 28 18:37:34 for internal system communication we use the private one has those things is something normal apps and services should not know about Aug 28 18:38:32 look at https://developer.palm.com/content/api/dev-guide/enyo/palm-service.html Aug 28 18:38:51 and https://developer.palm.com/content/api/dev-guide/mojo/services-overview.html Aug 28 18:39:45 so public methods means public bus and so for private? Aug 28 18:42:46 right Aug 28 20:02:05 morphis: you can check PR#2 in luna-appmanager Aug 28 20:06:37 sorry for questions, thank you for clarification Aug 28 20:06:53 nizovn: nothing to be sorry about :) Aug 28 20:06:57 thanks a lot! Aug 28 20:06:59 looks great Aug 28 20:07:50 nizovn: maybe http://issues.webos-ports.org/issues/555 is another one for you? Aug 28 20:09:55 morphis: ok, but don't I should work on enyo side of #551? Aug 28 20:10:47 ah right Aug 28 20:10:52 if you want please do Aug 28 20:11:25 ok, never tried enyo, but seems just to copy. Aug 28 20:12:28 what is the enyo part of that bug? Aug 28 20:12:32 or issue Aug 28 20:14:37 GodGinrai: it has to listen for a signal from com.palm.applicationManager instead of com.palm.systemservice Aug 28 20:14:51 should be only replace one string Aug 28 20:15:34 ah Aug 28 20:15:50 btw, morphis, that JSON in Bug #555 looks weird Aug 28 20:16:11 GodGinrai: why? Aug 28 20:16:52 shouldn't it be: { "params": { "summary": "Test Summary", "text": "Test Test" } } Aug 28 20:16:55 ? Aug 28 20:18:51 morphis: hm.. I thought it also need to move handleAppQuitStatus and AppQuitAlert Aug 28 20:20:13 possible Aug 28 20:20:30 GodGinrai: you mean without \"? Aug 28 20:21:04 morphis: yea. The JSON in the ticket is passing a JSON string as a string instead of part of the object Aug 28 20:21:27 which is why it is being escaped Aug 28 20:21:33 But that doesn't make sense Aug 28 20:21:39 because if you are using a JSON object already Aug 28 20:21:47 why would you pass a JSON string in it? Aug 28 20:22:11 as you supply it as a string to a underlaying object which will push it out on the bus Aug 28 20:22:16 we're in QML here Aug 28 20:22:23 things are a bit different then in enyo Aug 28 20:23:10 morphis: but even if you don't pass it as a string, whatever takes that string out and pushes it onto the bus should be able to just take obj.params and push that onto the bus Aug 28 20:23:25 (converting it to a JSON string *then* if a string is needed) Aug 28 20:24:08 GodGinrai: trust me, it currently has to work that way Aug 28 20:25:04 :\ I'm afraid I have a hard time believing that. Aug 28 20:25:28 things could be a lot better, yes Aug 28 20:26:05 give me at minimum 40 hours paid per week and I will implement such things better right from the beginning Aug 28 20:28:02 morphis: don't misunderstand me, I am not trying to undermine your effort. I am simply stating that placing the burden for passing the JSON strings on a developer rather than a function will lead to many more bugs. Aug 28 20:28:31 GodGinrai: want to fix it? :) Aug 28 20:29:43 morphis: want to? yes. Have time to? no. I have other things that I have been meaning to get done that are still unfinished. :\ Aug 28 20:30:59 I know that feeling .. Aug 28 20:33:08 morphis: ah, understood. if it's allowed to leave it in SystemManagerAlerts.js(not to move it and alerts to AppManagerService.js), then just one line. so it's allowed? Aug 28 20:33:32 nizovn: for sure Aug 28 21:22:18 morphis: PR#3 in luna-systemui. just asking... should it also be removed from systemservice? **** ENDING LOGGING AT Fri Aug 29 02:59:58 2014