/srv/irclogs.ubuntu.com/2014/03/28/#ubuntu-unity.txt

=== jhodapp|afk is now known as jhodapp
=== jhodapp is now known as jhodapp|afk
Saviqmzanetti, some things to fix on right edgehttps://code.launchpad.net/~unity-team/unity8/right-edge-2/+merge/213172/comments/50419108:28
Saviqhttps://code.launchpad.net/~unity-team/unity8/right-edge-2/+merge/213172/comments/50419108:28
davidcalleSaviq, ping08:57
Saviqdavidcalle, hey08:57
davidcalleSaviq, hi ! I have three bugfix branches for a silo+landing, will you have time for it today ? I still need to write the testplans.08:58
Saviqdavidcalle, yeah, I can tackle the silo, if you don't have testplans I'll have to rely on you to do the testing, though08:59
davidcalleSaviq, np, I'll still write the plans to future-proof these scopes, even if you rely on me for the testing (you won't like having to install all the latex packages in the world to test one of these :) )09:01
davidcalleSaviq, I've attached the three branches to https://bugs.launchpad.net/unity-scope-devhelp/+bug/1274669 (texdoc, calculator, devhelp scopes), will ping you when I have testplans09:05
ubot5Ubuntu bug 1274669 in libunity (Ubuntu) "scope-runner-dbus.py crashed with signal 5 in g_variant_new_va()" [High,Confirmed]09:05
Saviqdavidcalle, those are desktop-only are they?09:06
davidcalleSaviq, yes09:06
Saviqok09:07
Saviqdavidcalle, we might need to wait a little, we're low on silos09:07
davidcalleSaviq, np09:08
CimiSaviq, tsdgeos you mind if I arrive later on monday and start working from home? It's a week I'm waiting a parcel to arrive and I just received the notification it will be delivered Monday. I cannot reschedule it :\09:11
SaviqCimi, there's no clear plan for the sprint, so I think it's ok09:12
Saviqyou'll get a slap on the back, but you'll survive09:12
CimiSaviq, aha thanks :) I'll arrive asap09:12
CimiSaviq, I might work a lil today even if I booked holiday to compensate (I'm actually stubborn and I want to finish a branch)09:13
CimiSaviq, tsdgeos I have questions for bindings09:13
Saviqdavidcalle, building in silo 01309:14
Cimiwhen we work with properties that are set by a backend somewhere and async09:14
Cimifor example indicators09:14
davidcalleSaviq, thanks ! :)09:14
Cimifirse of all, I didn't really understand what Binding is09:15
SaviqCimi, you didn't, or you don't09:16
Saviq?09:16
CimiSaviq, don't09:16
CimiSaviq, apparently Binding is for C++, right?09:16
SaviqCimi, http://qt-project.org/doc/qt-5/qtqml-syntax-propertybinding.html09:16
Saviqhttp://qt-project.org/doc/qt-5/qml-qtqml-binding.html09:16
CimiSaviq, I looked already at those09:17
SaviqCimi, no, the Binding component is just as any other binding, only explicit09:17
SaviqCimi, and you get the additional when: property09:17
Saviqso09:17
Saviqproperty var foo: someOtherProperty09:17
Saviqis the same as09:17
SaviqBinding { target: something; property: "foo"; value: someOtherProperty }09:18
CimiSaviq, I was wondering when we need binding and when just (looking at the example) app.enteredText = myfield.text09:18
Cimiah ok09:18
SaviqCimi, when myfield.text can change, and you want it to update09:18
CimiSaviq, yes, but why using Binding and not the opposite?09:19
SaviqCimi, the above will set (value assignment) app.enteredText to the value myfield.text had at the time of the assignment09:19
Cimisorry, the other way09:19
SaviqCimi, when not to use a Binding?09:19
Saviqwhen you don't need it09:19
tsdgeosthat was pretty zen :D09:19
Saviq;D09:19
Saviqbut ture09:19
Saviqtrue09:19
SaviqCimi, you need to think what you want to achieve09:20
Cimihah09:20
SaviqCimi, think when that javascript will execute, and whether it's the only place where you want the two values to be updated09:20
CimiSaviq, so Binding is two ways?09:20
SaviqCimi, no09:20
CimiSaviq, is it possible to have something two ways?09:21
SaviqCimi, bindings are one-way only, if you want two ways, you need to handle onChanged on both and do assignment09:21
Cimiouch09:21
SaviqCimi, dednick told you 5 times already09:21
CimiSaviq, I needed a 6th time09:21
SaviqCimi, that he planned for a bidirectional "sync" component, just didn't get to it09:21
SaviqCimi, a Binding is exactly that09:21
SaviqCimi, it reacts to onChanged on the "source" property09:21
CimiSaviq, because yesterday I saw some code in system-settings that left me very confused09:22
SaviqCimi, and sets the value on the "target" property09:22
SaviqCimi, it's important also, that when you  have a complex binding (or Binding, or Qt.binding, they're all the same)09:22
Saviqsay property var foo: one + two / three * four09:22
Saviqif you move the thing to a function09:23
Saviqfunction bar() { return one + two / three * four }09:23
Saviqand var foo: bar()09:23
Saviqthat's not actually gonna work09:23
Saviqfoo will only get evaluated once on startup09:23
Saviqthings that are supposed to trigger the binding need to be explicitly called in the binding expression09:24
CimiSaviq, because I saw this http://bazaar.launchpad.net/~system-settings-touch/ubuntu-system-settings/trunk/view/head:/plugins/security-privacy/diagnostics/PageComponent.qml09:25
CimiSaviq, lines 66-6809:25
Cimiwtf09:25
Cimi:)09:25
SaviqCimi, that's ad09:26
Saviqbad09:26
CimiI wanted to rewrite this code better, but I didn't know where to start and if this code was correct at first09:26
SaviqCimi, first order of business would be ask the author about what they wanted to achieve09:27
CimiSaviq, connecting to DBUS says commit09:27
SaviqCimi, but yeah it looks like that's a poor man's sync09:27
CimiSaviq, evan wrote that09:27
CimiI guess he's not working on that anymore now, he's managing09:28
SaviqCimi, so yeah a Connections {} and onCheckedChanged would probably be better09:28
SaviqCimi, only problem there is feedback loop09:28
Saviqagain, that's what dednick wanted to solve09:29
SaviqCimi, feedback loop meaning: checkbox.checked = true → checkbox.onCheckedChanged: backend.checked = true → backend.onCheckedChanged: checkbox.changed = true → ...09:30
Saviqin the case of true this would get stop quite quickly09:30
Cimiyeah09:30
Saviqs/true/bool/09:30
Saviq'cause the values would sync rather quickly09:30
Saviqit's more difficult with anything arbitrary like numbers (sliders)09:31
Saviqby the time the backend responded with changed with the new value, another one could have been sent already09:31
Saviqsay slider is going from 0 to 0.309:32
Saviqslider.value = 0.1 → slider.valueChanged: backend.value = 0.1 → backend.valueChanged: slider.value = 0.1; slider.value = 0.2 → slider.valueChanged: backend.value = 0.2 → backend.valueChanged: slider.value = 0.209:33
Saviqetc.09:33
Saviqbut09:34
Saviqslider.value = 0.1 → slider.valueChanged: backend.value = 0.1; slider.value = 0.2 → slider.valueChanged: backend.value = 0.2; backend.valueChanged: slider.value = 0.1; things break09:34
Saviqif backend is late to the game09:34
CimiSaviq, and in qt there's no official solution to that?09:41
CimiSaviq, looks like a big issue, everywhere09:41
SaviqCimi, not everywhere, it's only a problem with remote backends where both sides can (often) change the same value09:42
Saviqand ones that are slow to respond09:42
SaviqCimi, it will reconcile anyway, it's a corner case what I described09:42
CimiSaviq, can't we block the ui until we get the value from the backend?09:43
SaviqCimi, you can get fired for even saying "block the UI", if you want ;)09:43
CimiSaviq, or more, block the slider until it is sync09:43
SaviqCimi, but anyway, a proper two-way sync component could be useful to reduce the impact, by throttling the changes, timing out, there's plenty of things that can happen09:44
Cimi+109:44
Cimi:D09:44
SaviqCimi, +1 on the "fired" part? :P09:47
Cimiahah09:47
SaviqCimi, but just to explain why not: we'd have to apply timeouts for backend going away for some reason, otherwise we could end up blocking indefinitely etc. - it's not worth it for a corner case09:48
CimiSaviq, it's not easy though, loading the UI then seeing values changing looks bad as well to be honest09:48
mzanettiSaviq: hmm... strange. AP was passing fine. will fix stuff09:49
SaviqCimi, make the backends respond faster :)09:49
Cimiideally I'd have UI super snappy09:49
Cimiand backend catching up09:49
Cimiso when I tap a checkbox, it is immediately switched on09:49
Saviqmzanetti, maybe something changed, if you say it's passing for you, I'll try again, but have seen really reliable failures on them09:49
Saviqmzanetti, let me list the tests09:50
Saviqmzanetti, hmm now that I look, it was always GenericScopeView not found... which is weird in its own right09:51
Saviq\o/, me used "its" correct09:51
mzanettiSaviq: ah that one... yeah... sucks09:51
mzanettiSaviq: seems qml now *sometimes* optimizes it away and sometimes doesn't09:51
mzanettihaven't been able to figure when exactly09:52
mzanettibut I changed it to only search by objectName09:52
mzanettibut then Alberts merge came in which changed it back - but still working reliably for me so I left it09:52
Saviqmzanetti, the 60 vs. 100 GUs for side stage caught you in cross fire - we changed it before MWC09:53
mzanettiheh, yeah09:54
Saviqdavidcalle, silo built, let me know what I should test09:56
Saviqdavidcalle, would be good to get reviews on the branches, too :)09:56
CimiSaviq, I might actualyl start writing this component today, to learn this stuff09:58
CimiI have enough!09:58
SaviqCimi, be sure to talk to Nick for that09:59
Cimiok09:59
CimiSaviq, he's off as well today09:59
CimiSaviq, we might all do next week09:59
Saviqyup09:59
davidcalleSaviq, yep, just need to find a victim for that, finding one asap10:00
davidcalleSaviq, Testplans done (links on mp), tests done (works on my machine(tm)), reviews incoming...10:26
Saviqdavidcalle, cool, will do the testing soon10:27
SaviqMacSlow, hey, any dice on the notification avatar bug?10:33
Saviqdavidcalle, hmm I'm only getting "see more scopes" for "latex:math"10:35
Saviqdavidcalle, ps doesn't show the scope running at all10:35
tsdgeoswhy i don't get the keyboard when running unity8 manually on the phone?10:35
tsdgeosi have the maliit-server running10:35
Saviqtsdgeos, it stops with the unity8 job, you sure it doesn't get stopped?10:36
tsdgeoswell i stopped unity810:37
tsdgeosand then started maliit-server10:37
tsdgeosupstart says it's there10:37
Saviqmhm10:37
Saviqdo you get it in apps?10:37
Saviqmight want to try to restart maliit when unity8 is already running10:37
tsdgeosaha10:37
davidcalleSaviq, oh sorry, the test is assuming you have logged out/in, I should make that clear. Just kill unity-scope-home and reopen the Dash, that will reload scopes.10:38
Saviqdavidcalle, right10:38
tsdgeoslet's try thta10:38
Saviqdavidcalle, better10:38
davidcalleSaviq, :)10:38
tsdgeosrestarting did the trick :)10:40
Saviqdavidcalle, ok, let me know when they're ACK'ed, am +1 on testing10:42
davidcalleSaviq, thanks, pstolowski is reviewing10:44
MacSlowSaviq, not looked into it today.. still with the boot-anim for the moment... but I'll get to it today (afternoon)10:51
MacSlowSaviq, this is the latest state I'm at there https://bugs.launchpad.net/unity8/+bug/1296777/comments/910:52
ubot5Ubuntu bug 1296777 in Unity 8 "avatar displayed incorrectly in notification" [High,Confirmed]10:52
davidcalleSaviq, branches reviewed11:05
mzanettielopio: hey11:47
mzanettielopio: can I tell AP to give me the first child of an object, regardless of anything?11:47
mzanettiSaviq: did you write this as a todo for me or just a general todo and the respective package owners are on it already? https://code.launchpad.net/~unity-team/unity8/right-edge-2/+merge/213172/comments/50419912:10
Saviqmzanetti, todo for you :)12:10
mzanettiSaviq: ack12:10
mzanettifinally got AP working again.12:11
mzanettisuch a pain that thing12:11
mzanettiwell, I hope so. still running the whole suite at once to make sure12:12
Saviqmzanetti, http://bazaar.launchpad.net/~phablet-team/ubuntu-touch-session/trunk/revision/163/debian/changelog12:21
mzanettiheh12:21
mzanettinice one12:21
Saviqugh, /me switched channels12:22
Saviqmzanetti, sorry12:22
mzanettinp12:22
=== jhodapp|afk is now known as jhodapp
Saviqdavidcalle, apparently scopes landed12:47
davidcalleSaviq, seen that, thanks a lot :)12:49
Saviqcheers12:50
=== MacSlow is now known as MacSlow|lunch
=== _salem is now known as salem_
elopiomzanetti: like get_children()[0] ?13:29
Saviqelopio, he did that, yeah13:30
elopioget_children_by_type is the method.13:30
Saviqelopio, can we get your eyes on https://code.launchpad.net/~unity-team/unity8/right-edge-2/+merge/213172 for the autopilot changes please?13:30
elopioSaviq: yes, I'll take a look after my current meeting finishes.13:31
Saviqelopio, thanks13:31
dandraderwhen loading a themed icon, should I pass as an Image.source "image://theme/phone-app" or "//theme/phone-app" ?13:46
Saviqdandrader, image://13:46
dandraderSaviq, ok, thanks13:46
Saviqdandrader, //theme would be a local file path13:46
CimiSaviq, cool got my parcel delivered right now, so I'll be there on time monday morning!13:51
Saviqhappy happy Cimi :D13:53
Saviqand now everyone tries to guess what's in the parcel Cimi was waiting on...13:54
CimiSaviq, pc stuff...14:04
SaviqCimi, of course, yes, we believe you :D14:04
elopioSaviq: I have some question about the changes. With mzanetti right?14:05
Saviqelopio, yeah, he's afk for now, so maybe leave them on the MP14:06
=== dandrader is now known as dandrader|afk
=== sil2100_ is now known as sil2100
=== MacSlow|lunch is now known as MacSlow
=== dandrader|afk is now known as dandrader
mzanettielopio: hey14:46
mzanettielopio: huhu! can you please fix your IRC hightlighting?14:49
elopiomzanetti: it works. I was walking the dog.14:57
mzanettielopio: ah :)14:57
elopiobut I'm about to start a meeting. mzanetti: will you be around in ~30 minutes?14:57
mzanettielopio: ok then. let me know when you're back :)14:58
mzanettiyes14:58
Saviqmzanetti, what do you mean "but its annoying to update this with every commit over and over again."?15:03
mzanetti:D15:03
Saviqmzanetti, we need to maintain the emulators, and those tests are meant to verify we are15:03
mzanettiyeah. but it didn't look like that check being that import15:04
mzanettiadding it back15:04
mzanettialso I think this is not really the proper thing to do15:04
mzanettiwe should rather see if the object we have can give us what we want15:05
mzanettiand not if its name matches something15:05
mzanettiSaviq: ^15:05
Saviqmzanetti, well, since we maintain the object, we know what it should return (and check what type it is, as well as what it gives)15:06
mzanettiSaviq: but there's a qml optimization in between which might change the name15:06
mzanettithat's what I mean. if I add/remove one property to DashApps, that optimization might change and it becomes a GenericScopeView15:07
mzanettijust as one example15:07
Saviqmzanetti, ah I know what you mean15:07
mzanettiwhile it still would have everything we need, the test fails because the name doesn't match15:07
Saviqmzanetti, basically recently if a sub-class doesn't have new props15:07
Saviqmzanetti, it does not appear as the sub-class, but as the base class still15:08
mzanettiyeah. but since 5.2 I think this has become a bit more flexible15:08
Saviqmhm15:08
mzanettiwith < 5.2 yes, it was exactly how you said15:08
mzanettinow it can be optimized even if it adds properties. but I still haven't found what exactly is the measurement15:08
mzanettiwhich is exactly this back and forth between GenericScopeView and DashApps15:10
mzanettievery time I merge trunk it changes I have the feeling :D15:10
Saviqmzanetti, then maybe your change is good indeed, but needs to be supplemented by something? like a check of objectName?15:11
Saviqmzanetti, and we might actually need to force it be a DashApps emulator15:11
mzanettiyeah... maybe. I'll put in the old check for now and discuss with elopio on how to make this more robust15:11
Saviqmzanetti, elopio will know more15:12
mzanettiSaviq: also, the one above (assert_scope_is_opened(scope_id) kinda implcitly checks if its the right scope already. another reason why I just dropped it15:14
Saviqmzanetti, yeah, but it's not just that15:15
MacSlowboiko, any news on the compile issue ran into with telephony-service (https://bugs.launchpad.net/unity8/+bug/1296777/comments/9)?15:15
ubot5Ubuntu bug 1296777 in Unity 8 "avatar displayed incorrectly in notification" [High,Confirmed]15:15
Saviqmzanetti, the DashApps emulator has additional members, does it not15:15
MacSlowboiko, is there a fix for it or do I need to work around it somehow?15:15
mzanettiSaviq: yeah...15:15
Saviqmzanetti, so if something expects the DashApps emulator, but gets the generic GenericScopeView, it will be a problem15:15
Saviqmzanetti, that's why that check makes sense15:16
mzanettiSaviq: thanks for fixing flo in the meantime15:22
Saviqmzanetti, no worries15:22
SaviqCimi, standup15:32
Saviqjosharenson, standup15:32
boikoMacSlow: well, it shouldn't be happening in the first place, but I unfortunately didn't have time to look into that yesterday15:33
MacSlowrsalveti, hey there... I'm just wondering if there's a library I could use (early in the boot-process, when $GRID_UNIT_PX isn't set yet by ubuntu-touch-session) to read it from /etc/ubuntu-touch-session.d/$(getprop ro.product.device).conf ... otherwise I do it manually with popen() and friends.15:34
MacSlowboiko, ok... I'll look into it myself later today again too15:34
boikoMacSlow: sorry, I can try to look into that later today, but I have some higher priority stuff to sort out15:35
rsalvetiMacSlow: we only have a library to call property_get (the getprop thing), but you'd need to read the file15:36
MacSlowboiko, ok15:36
MacSlowrsalveti, ok15:36
MacSlowrsalveti, which library provides the property_get()?15:39
rsalvetiMacSlow: libandroid-properties115:39
rsalvetithere's a -dev package for it15:40
MacSlowrsalveti, thx15:40
elopiomzanetti: ready. Sorry.15:44
mzanettielopio: hi :)15:44
mzanettielopio: so... for now DashApps is a DashApps again but I'm confident that'll change again15:45
mzanettielopio: so for the right edge branch I reverted my AP changes15:45
mzanettielopio: but I think we need to make this more generic/robust against QML optimziations15:45
elopiomzanetti: I got that from the backlog. This is so weird for me, how can you tell if QML will name a class that extends other after the file name, or after the other class?15:46
elopioit seems undeterministic for me. I had a bad time with CheckBoxes and Switches.15:46
mzanettielopio: with Qt 5.0 the criteria was if a class adds properties. However, that seems to have changed with 5.2 and I'm not exactly sure yet what's the trigger. would need some evaluation15:47
mzanettielopio: but: can't we just make sure the object has a method get_apps() or whatever instead of forcing it to be called DashApps?15:48
elopiomzanetti: this is what we want: https://bugs.launchpad.net/autopilot/+bug/121026015:48
ubot5Ubuntu bug 1210260 in Autopilot "forcing the emulator name to be the same as the qml type it's too restrictive" [High,Fix released]15:48
elopioit has just landed.15:48
elopiothe problem is linking a QML object with the helper object that provides all the methods for tests to use.15:48
mzanettiyeah... without fully understanding what's going on internally, the description sounds about right15:49
mzanettiyep. I can imagine it might be a bit tricky...15:49
mzanettielopio: another idea:15:50
mzanetticouldn't we add the get_apps() function to the class in QML, and then invoking that from AP directly?15:50
elopiomzanetti: I'll just need to ask them for documentation of how to use it :/15:50
elopiobut in theory, we could do what you want, just return the first children and if it matches objectName='bla', it will return the helper object we defined.15:50
elopioeven if the class names don't match.15:50
mzanettiwithout the abstraction layer15:50
elopiomzanetti: I think this is the same that you did for another branch, wasn't it?15:52
elopioyou moved the specific method to the base class, and then it would be on both. It doesn't matter which one autopilot finds.15:52
mzanettielopio: that's exactly this15:52
mzanettielopio: had to revert it because the QML name changed back after I merged trunk15:53
elopioright, now I understand the dejavu sense :)15:53
mzanettiand because it annoyed me quite a bit, I dropped that check for isInstanceOf15:53
elopio:D15:53
mzanettithinking that it would be implicitly checked for having apps anyways15:53
mzanettiwhich it does. but still leaves a bit more room for breakage, I agree15:53
elopiomzanetti: let me get with nuclearbob to understand how to use it. I have been annoyed by this for 6 months since I reported the bug.15:54
mzanettielopio: anyways, I think we're good for the right edge branch now.15:55
elopioI hope to be able to show you on monday a small branch that depends on the objectName, not the class name.15:55
Saviqmzanetti, btw, it's ok to --overwrite sometimes ;D15:55
mzanetti:D15:55
mzanettiSaviq: I try to avoid that on shared branches15:55
mzanettican break the other's working copy quite badly15:55
Saviqmzanetti, I know, but when it's a case like this is ok15:55
Saviqmzanetti, i.e. 3 mins after the other15:56
mzanettiyeah, true15:56
mzanettiI'll remember15:56
* Saviq kicks another build15:57
mzanettiaye15:57
elopiomzanetti: one more detail15:58
elopiothis is extremely unsafe: 2963+ return loader.get_children()[0]15:58
Saviqah, except that it won't work without the unity-mir thing...15:58
elopiomzanetti: what would you think of try to get one, and if an exception is raised, try to return the other.15:58
* Saviq needs to wait for that with testing15:58
mzanettielopio: not a fan of such things. but if it gets AP out of my way, feel free :D15:59
bschaeferkgunn, hey, any luck with that demo?15:59
elopioyou are hurting my feelings... :)16:00
elopiomzanetti: well, at this point, I don't understand why that change is needed. Is it because you want to avoid having to revert again?16:00
mzanettielopio: oh. ok... sorry for that16:00
mzanettiwon't happen again16:00
mzanettielopio: let me just fix something in unity-mir first16:01
elopiomzanetti: I'm joking. I don't like QtTest nor javascript, but I'll have to live with it just as you.16:01
elopioI'll get used to it, at some point16:01
kgunnbschaefer: man...been busy on lots of arch discussions this morning hadn't had a chance16:03
bschaeferkgunn, no worries :)16:03
bschaeferkgunn, ill poke mhr3 about it16:03
bschaefermhr3, would you happen to know how to get unity8 to read newly installed *.desktop files?16:04
kgunn....or any of the other guys here....16:04
kgunnhttp://bazaar.launchpad.net/~brandontschaefer/+junk/sdl2-demo-blocks/view/head:/data/blocks-demo.desktop.in16:04
* bschaefer remembers mhr3 doing all that fun stuff in unity716:04
kgunnmzanetti: tsdgeos ^ is that desktop file kosher16:04
Saviqbschaefer, unity8 doesn't read .desktop files itself16:04
Saviqbschaefer, it's unity-scope-click now16:04
bschaeferi see, so how does the dash pick up new *.desktop files?16:05
mzanettikgunn: looks ok16:05
bschaeferi had it working on unity8 for the desktop16:05
mzanettiSaviq: note that it does still read .desktop files in ApplicationManager and Launcher16:05
Saviqbschaefer, should be enough to just search, it doesn't monitor the dirs, htough16:05
bschaeferjust its failing for the phone16:05
Saviqbschaefer, both with new scopes?16:05
tsdgeos+1 looks reasonable16:05
Saviqmzanetti, yes, but on launch16:05
Saviqmzanetti, or so16:05
Saviqmzanetti, it does not "find" them16:05
bschaeferSaviq, hmm i tested it before the new qt5.216:05
Saviqto display all16:05
Saviqbschaefer, yeah, we switched to the complete new scopes now, new scope backends, new scope implementations16:06
mzanettikgunn: note that files with .in are templates that will be filled in when cmake runs16:06
bschaeferthe problem with the phone, is it doesn't show up at all under search16:06
Saviqbschaefer, you'd have to check with a recent unity816:06
mzanettikgunn: the {} stuff in there will be replaced by cmake and the .in at the end removed16:06
bschaeferSaviq, oo i see, well unity8 doesn't work atm on the deskop :)16:06
Saviqbschaefer, right... you can use unity-scope-tool to check, though16:06
bschaeferSaviq, from the phone?16:06
* bschaefer needs a phone16:06
Saviqbschaefer, no, desktop16:07
bschaefero desktop16:07
Saviqbschaefer, if you install unity-scope-click, unity-scope-tool16:07
bschaefersweet, let me test that out16:07
Saviqbschaefer, start scope-registry, start smart-scopes-proxy16:07
Saviq(or restart for good measure)16:07
Saviqbschaefer, and then UNITY_SCOPE_LIST="clickscope" unity-scope-tool should give you exactly what you would get on the phone16:07
bschaeferSaviq, awesome!16:08
mzanettiSaviq: actually, where are we going to install Unity.Application to? qt5/qml/ ?16:08
bschaeferSaviq, thanks! Ill give that a go now16:08
Saviqmzanetti, no16:08
mzanettiSaviq: or qt5/imports/ still, just dropping the Unity-Mir16:08
Saviqmzanetti, the path from unity-api.pc16:08
Saviqmzanetti, same as other shell-only modules16:09
=== salem_ is now known as _salem
bschaeferhmm empty, possibly i've not upgraded my machine in sometime16:11
Saviqbschaefer, did you make sure to (re)start scope-registry and smart-scopes-proxy?16:12
Saviqbschaefer, also, it's actually SCOPES, not SCOPE16:12
bschaeferSaviq, i had started them16:12
SaviqUNITY_SCOPES_LIST="clickscope" unity-scope-tool16:12
bschaeferthat could have been the issue :)16:12
Saviqbut regardless16:12
Saviqbschaefer, it would've shown you all16:12
bschaeferSaviq, with the S i see 2 now16:13
bschaeferoo more now16:13
Saviqhuh, there should only be one :D16:13
bschaeferSaviq, cool, now to actually install my demo16:13
bschaeferSaviq, i see 100+16:13
Saviqah, items :)16:13
Saviqthat's good16:13
Saviqbschaefer, I meant would've shown you all  scoeps16:13
bschaeferoo right sorry :)16:14
bschaefercool, thats a useful testing method16:14
bschaeferso lets say I don't see the desktop file, what should be done to get it to be read by scopes click?16:14
Saviqbschaefer, X-Ubuntu-Touch=True16:15
bschaefercapital T?16:16
bschaeferi have true16:16
Saviqbschaefer, well, probably doesn't matter16:16
bschaeferyeah16:16
* bschaefer was just hoping it was cap sensitive as i have =true haha16:16
Saviqbschaefer, and if you search for it, doesn't come up?16:16
bschaeferSaviq, i've to update and add the mir staging ppa :)16:17
bschaefersoo i've yet to test it, but its not coming up for kgunn on the phone16:17
mzanettiSaviq: pushed unity-mir install path16:17
Saviqmzanetti, same branch?16:17
mzanettiyeah16:17
Saviqk16:17
bschaeferSaviq, possibly kgunn needs to upgrade to the new scopes changes? (Not sure what version hes on)16:17
kgunnhttps://launchpad.net/~mir-team/+archive/staging/+packages16:17
kgunnbschaefer: i used latest image devel-proposed16:18
kgunnsorry distracted otp16:18
bschaeferkgunn, well that looks up to date to me :)16:18
bschaeferkgunn, no worries at all!16:18
* bschaefer is just waiting for upgrading to happen...16:18
Saviqhmm interesting16:20
bschaeferSaviq, i would assume its my desktop file, or how im installing it, but i was putting it in the same place as the other *.desktop files I saw16:20
Saviqbschaefer, not really16:21
bschaeferSaviq, http://bazaar.launchpad.net/~brandontschaefer/+junk/sdl2-demo-blocks/view/head:/data/CMakeLists.txt16:21
bschaeferthe installing16:21
Saviqbschaefer, I just copied one and changed its name16:21
Saviqbschaefer, should show up, but doesn't16:21
bschaeferSaviq, o strange! Scopes *uses* to have to be opened first16:21
bschaefernot scopes16:21
bschaeferbut files16:21
bschaeferbut thats an application, and thats my old unity7 understanding :)16:21
Saviqpstolowski, do you have an idea about the new click scope reading .desktop files (non-click)?16:21
Saviqpstolowski, seems searching for a newly added one or restarting the scope registry doesn't help16:22
Saviqmzanetti, FYI, I'd rather you have used a separate branch for that, but it's ok16:23
=== gatox is now known as gatox_lunch
bschaeferinteresting, thanks for testing Saviq!16:23
mzanettiSaviq: I can revert and do that if you want16:23
* Saviq reboots the phone16:23
Saviqmzanetti, nah it's ok16:23
mzanettithought it would make it easier if its in here16:24
mzanettiSaviq: what's the reason you'd prefer another branch?16:24
Saviqmzanetti, separate commit in history16:24
pstolowskiSaviq, all I know is click scope has a hardcoded list of non-click .desktop files for a dozen of apps16:25
Saviqmzanetti, hmm why SHELL_PLUGINDIR and SHELL_INSTALL_QML?16:25
Saviqmzanetti, don't they need to be in the same place?16:25
* Saviq checks unity-scopes-shell how it does it16:26
pstolowskiSaviq, http://bazaar.launchpad.net/~ubuntuone-hackers/unity-scope-click/trunk/view/head:/scope/click/interface.cpp#L5316:26
Saviqpstolowski, ouch16:26
Saviqbschaefer, ↑16:26
mzanettiSaviq: I've stolen this from unity-notifications16:26
mzanettilet me check if I copied something wrong16:26
mzanettino... looks ok.16:27
bschaeferSaviq, i see, hmm how do new ones get added? :)16:27
pstolowskiSaviq, no idea if there's anything else. checking16:27
mzanettiSaviq: yeah, I throught the same, why 2 dirs. but have used it as is because we have it like this in unity-notifications too16:27
bschaeferone thing i dont have in my desktop file is:16:27
Saviqmzanetti, one of the vars is absolute, the other is relative16:27
bschaeferstatic const std::string DESKTOP_FILE_KEY_APP_ID("X-Ubuntu-Application-ID");16:27
bschaefernot sure if thats 100% needed, but it looks like its checked for16:27
Saviqmzanetti, notifications is using it wrong, bug #127361716:28
ubot5bug 1273617 in unity-notifications (Ubuntu) "Client plugin should be installed into defailt import dir" [Undecided,New] https://launchpad.net/bugs/127361716:28
mzanettiaha16:28
mzanettiok, fixing16:28
Saviqmzanetti, check unity-scopes-shell instead16:28
mzanettiack16:28
Saviqmzanetti, http://bazaar.launchpad.net/~unity-team/unity-scopes-shell/trunk/view/head:/CMakeLists.txt#L5516:28
Saviqbschaefer, the new scope has a hardcoded list of non-click .desktop files it looks at...16:28
bschaeferkgunn, well if you realllly want to test the sdl2 demo out, edit a working desktop file :)16:28
Saviqbschaefer, doesn't matter what the kontents are...16:29
=== _salem is now known as salem_
Saviqkontents...16:29
bschaeferSaviq, i see, well i just see more then those *.desktop files16:29
kgunncool...16:29
bschaeferSaviq, its ok i understood16:29
bschaeferhaha16:29
Saviqbschaefer, the rest are click apps16:29
Saviqbschaefer, and taken from the web16:29
Saviqbschaefer, not local16:29
bschaeferSaviq, o i see!16:29
* bschaefer now understands what scope clicks are16:29
* Saviq works too much with KDE people... c → k16:30
bschaefer:)16:30
Saviqbschaefer, there are no "scope clicks" :D16:30
Saviqbschaefer, scope is a backend for displaying data in dash16:30
Saviqbschaefer, as in current unity16:30
Saviqbschaefer, click are the new package format we use for apps on the phone16:30
Saviqbschaefer, so "click scope" is a scope (like in a lens) onto click apps :)16:30
bschaeferSaviq, i think i meshed those 2 words together when i shouldn't have :)16:31
bschaeferi see16:31
bschaeferSaviq, so, there will be plans later on to incorporate non-click apps?16:31
bschaeferor is the switch to use, click apps only, besides the hard coded?16:31
Saviqbschaefer, can't see how we can not support standard .desktop files16:32
bschaeferyeah16:32
Saviqbschaefer, it's a shortcut, I'm sure16:32
Saviqbschaefer, fwiw it was working before16:32
Saviqbschaefer, it just wasn't ported to the new scope infra16:32
pstolowskiSaviq, bschaefer ok, I think that was it, there is nothing more to it. if app is not on that hardcoded list, it needs to provide X-Ubuntu-Application-ID etc.16:32
bschaeferSaviq, yeah i had it working on unity8 desktop16:32
Saviqpstolowski, yeah, it needs to be a click, basically :)16:33
bschaeferpstolowski, does the X-Ubuntu-Application-Id need to be unique?16:33
* bschaefer imagines that being hard to guarantee a hard coded ID to be unique between different projects desktop files16:33
Saviqbschaefer, that's why they're mangled on installation16:34
Saviqbschaefer, it's the installation process that puts that in there16:34
bschaeferooo i see, so thats not my job to put it in the desktop file :)16:34
Saviqnope16:34
bschaefercool, well at lease that issue i known :)16:34
pstolowskibschaefer, dunno about that id16:35
bschaeferSaviq, for now ill get kgunn to write over a hardcoded *.desktop file to point to the demo to test it out16:35
Saviqbschaefer, yeah16:35
bschaeferSaviq, pstolowski thanks for all the help!16:35
Saviqbschaefer, one of them should actually be free by now16:35
bschaeferyeah16:35
Saviqbschaefer, like gallery, for example16:35
bschaeferi think thats what i tested on before haha16:35
Saviqbschaefer, so just put it in as gallery for now16:36
Saviqbschaefer, or well, package it into a .click! :)16:36
Saviqjoin the cool boys :D16:36
bschaeferSaviq, is there documentation to do such a thing haha?16:36
Saviq(only you'll have to bring SDL with you ;)16:36
bschaeferyeah haha16:36
Saviqit's all BYOD16:36
bschaeferhaha16:36
Saviqat least until we make SDL part of the framework16:37
bschaeferwell if i get sometime (there needs to be more time in a day...)16:37
bschaeferyeah16:37
bschaeferSaviq, i suppose I can for my SDL demo to overrides the gallery, but it could also be manually edited (overriding seems mean)16:37
Saviqbschaefer, yup16:38
Saviqbschaefer, to install .debs you need to be writable anyway16:38
bschaeferkgunn, ill get a nice diff for you, to change your gallery*.desktop file16:38
bschaeferSaviq, yeah, but doesn't the gallery app still work and is used?16:38
Saviqbschaefer, fyi, you can also launch stuff from the console, you need to pass --desktop_file_hint=/path/to/a/.desktop/file16:39
bschaeferSaviq, i could also look at patching that hardcoded list for now16:39
bschaeferSaviq, o really?!16:39
Saviqbschaefer, gallery has been converted to click16:39
bschaeferSaviq, oo cool, so yeah i can do that then16:39
* Saviq 's plan to prep for sprint next week didn't work out it seems...16:40
bschaeferSaviq, so if you pass the --desktop_file_hint, unity8 starts it up and ill get access to the  precious mir server/socket?16:40
Saviqbschaefer, yeah, one thing, though16:40
Saviqbschaefer, if launching from console16:41
Saviqbschaefer, you need to get the correct mir socket16:41
Saviqbschaefer, initctl get-env --global MIR_SOCKET16:41
mzanettiok Saviq, now hopefully I got it right16:41
Saviqand export that16:41
bschaeferits in $XDG_RUNTIME_DIR?16:41
bschaefero16:41
bschaeferthats even better16:41
Saviqbschaefer, yeah, that's the user's socket, the /tmp one is for the system compositor16:41
Saviqto which unity8 connects as the session compositor16:42
bschaefercool, i thought i looked in /tmp and didn't find a socket16:42
bschaeferthen saw mir checked XDG_RUNTIME_DIR now, but that makes sense16:42
bschaeferSaviq, thanks again!16:42
Saviqmzanetti, yeah, looks better16:43
Saviqmzanetti, checked debian/*install ? ;)16:44
=== dandrader is now known as dandrader|lunch
mzanettiSaviq: looks ok :)16:44
Saviqk16:45
mzanettiah... not so sure.16:45
mzanettilet me verify16:45
Saviqmzanetti, nope16:45
Saviqmzanetti, usr/lib/*/qt5/* is wrong16:45
mzanettiyeah16:45
mzanettijust saw it16:45
mzanettiusr/lib/*/Unity/Application/* should do16:47
mzanettiSaviq: can it be that the libunity-api-dev has the wrong arch?16:49
mzanettishouldn't it be all?16:49
Saviqmzanetti, it installs the .la16:50
Saviqmzanetti, I think16:50
* Saviq checks16:50
Saviqmzanetti, it actually ships the libunity-api.so16:51
Saviqlink16:51
mzanettioh16:51
Saviqmzanetti, we could probably get rid of it16:51
Saviqmzanetti, in general -dev packages are arch-specific16:52
Saviqmzanetti, we could potentially make it all, but I'm thinking we'll rather make it into a library sooner than later16:52
Saviqto allow sharing between things between plugins, like enums and such16:52
mzanettimhm... I c16:53
mzanettiSaviq: ok. I think the install path works now too16:55
Saviqmzanetti, k16:55
Saviqmzanetti, nope16:55
Saviqmzanetti, you're missing a /unity8/16:55
Saviq/unity8/qml/ actually16:56
mzanettiwhere16:56
mzanetti?16:56
Saviqmzanetti, just execute the pkg-config line16:56
Saviqpkg-config --variable=plugindir unity-shell-api16:57
Saviqyou'll see where it's meant to be installed16:57
Saviqand then you'll see that the installation path needs to be16:57
Saviq/usr/lib/*/unity8/qml/*16:57
Saviq* isn't recursive16:57
mzanettiSaviq: ok, yes. but shouldn't that be caught by my line too?16:57
Saviqmzanetti, no16:58
mzanettiahhh16:58
mzanettiwb :)16:58
Saviqgrr ^W16:58
mzanettiaah, so * matches only 1 dir?16:58
Saviqmzanetti, yeah, it's a standard glob16:58
mzanettias in usr/*/*/*/Unity/Application would work  then?16:59
Saviqmzanetti, yes16:59
mzanettiok16:59
Saviqmzanetti, but it's better to be explicit16:59
mzanettisure16:59
mzanettiok, now then :D17:01
tsdgeosSaviq: i just pushed a fix for the polishItems thing, if you have time to try to reproduce the crash it'd be cool to have someone else try it17:02
tsdgeosi'm going eow now17:02
tsdgeossee you on london17:03
Saviqtsdgeos, o/17:03
SaviqDAMN that guy :D17:03
mzanettiwhat's the polishItems? the ones written by Saviq? :D17:04
Saviqpfft!17:04
Saviqmzanetti, you wanna join #ubuntu-devel17:05
=== salem_ is now known as _salem
=== _salem is now known as salem_
* greyback eow17:41
=== gatox_lunch is now known as gatox
=== dandrader_ is now known as dandrader
=== salem_ is now known as _salem
jhodappAnybody know why running a QtTest on a target device results in it trying to connect to Mir? I have this backtrace: http://pastebin.ubuntu.com/7169856/18:32
jhodappwhat am I missing (I am running Mir)18:33
jhodappit hangs trying to connect to Mir18:33
=== _salem is now known as salem_
=== dandrader_ is now known as dandrader|afk
=== gatox_ultra is now known as gatox
=== salem_ is now known as _salem
=== dandrader|afk is now known as dandrader
=== _salem is now known as salem_
=== dandrader is now known as dandrader|afk
=== dandrader|afk is now known as dandrader
=== salem_ is now known as _salem
mhr3jhodapp, you need to pass minimal as the platform iirc22:21
mhr3QT_QPA_PLATFORM=minimal22:22
jhodappmhr3, thanks, I also discovered QTEST_GUILESS_MAIN22:29
mhr3ah, maybe i should have been using that in the first place22:32
mhr3jhodapp, so thx :)22:32
jhodappmhr3, hehe, np :)22:39

Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!