/srv/irclogs.ubuntu.com/2014/05/13/#ubuntu-unity.txt

=== salem_ is now known as _salem
=== duflu_ is now known as duflu
SaviqCimi, coolz07:16
djbello1can anybody help? I had Mythbuntu installed and had a specific background image set. I have now moved to regular Ubuntu-desktop, and I have plenty of problems with the desktop. I finally got the lightdm unity-greeter working, but the background image is always the one I had previously. And when I go into the settings panel and try to set a different image, nothing changes at all. It's driving me nuts. I've already gotten rid of all hid07:28
djbello1And by moved I mean I installed the "ubuntu-desktop" package.07:29
djbello1I should also note that I had been running Mythbuntu 12.04 LTS for the longest time and just upgraded to 14.04 LTS today.07:31
Saviqdjbello1, it's probably best if you post this on askubuntu.com or as a bug07:35
Saviqor maybe a mailing list07:35
djbello1aw man.07:36
djbello1I'll give it a shot then.07:37
MirvSaviq: tsdgeos: are you ready for Qt 5.3? I still find it hard to understand which packages would need recompilation and which problems are clear bugs, but Unity8 starts now. scopes are clearly broken though despite the rebuilds. https://launchpad.net/~canonical-qt5-edgers/+archive/qt5-beta207:49
SaviqMirv, we probably are not07:49
Mirvbugs at https://bugs.launchpad.net/bugs/+bugs?field.tag=qt5.3 against qtubuntu*, unity-api, unity-scopes-shell. I've disabled tests in unity-scopes-shell, qtubuntu, qtubuntu-sensors because some test failed, so that might be telling.07:50
tsdgeosi can't make unity-api nor unity-scopes-shell tests pass here07:50
tsdgeosnot sure it is 5.3 related07:50
tsdgeosor that you need a special environment/knowledge07:50
Saviqtsdgeos, I just ran unity-api tests and they passed :?07:50
tsdgeosSaviq: really?07:51
Saviqtsdgeos, in your branch, no less07:51
Mirvtsdgeos: yep, same here in the PPA, which is why I filed the bugs and disabled tests to get builds going07:51
tsdgeosok, then maybe it was scopes-shell only07:51
tsdgeosSaviq: i didn't do 2013 -> 2014 nor drop authors because it's a verbatim c&p07:52
tsdgeosSaviq: but i will :)07:52
Saviqtsdgeos, yeah, let's clean it up in the process? if you're unsure about the docs, maybe we should get mhr3 to doc them?07:53
tsdgeosSaviq: i don't think i can really add the docs07:53
Saviqtsdgeos, yeah, that07:53
Saviqtsdgeos, resubmit as ~unity-team then?07:53
MirvSaviq: tsdgeos: darn, you're both not "there" next week yet. so, it'd be good to get opinions from you whether we would enjoy/need Qt 5.3 probably for this cycle, or if we need to stay at 5.2 (which would create problems in coordination with Kubuntu, that is, we might need to fork everything and block Debian autosyncs)07:53
tsdgeosok, will fix your comments and resubmit07:53
tsdgeosMirv: i'd prefer we switch at some point07:53
tsdgeosbut that's me07:54
tsdgeoswith my only-engineering pov07:54
SaviqMirv, I don't think we'd block the switch, no07:54
SaviqMirv, and I would like to switch, too, we'll look into what's happening this week07:54
Saviqtsdgeos, indeed -scopes-shell fails for me, too07:55
Mirvtsdgeos: Saviq: I added you rights to a doc that lists pros/cons. I was assuming we go for 5.3, but I'm now hearing more conservative thoughts and uncertainty.07:55
Saviqtsdgeos, ah wait, it complains about timeouts, maybe it requires the scope registry to work07:55
Saviqtsdgeos, heh, no that made it worse ;D07:56
tsdgeosSaviq: yeah07:57
tsdgeosnow it's 21 vs 107:57
tsdgeosor something :D07:57
Saviqtsdgeos, yeah, and failed in sbuild, too, we'll need mhr3 to get those going07:58
tsdgeosSaviq: yeah i had to comment the tests so i could dpkg-package07:58
tsdgeosand try my patch correctly07:58
tsdgeoswhich doesn't work btw07:58
MirvSaviq: feel free to edit your quote ;)07:58
tsdgeosthat's why is still in wip  ;)07:58
Saviqtsdgeos, you can always NOCHECK=1 when dpkg-buildpackage07:59
tsdgeosSaviq: ah07:59
Saviqor NO_CHECK=1? Mirv, which one is it?07:59
tsdgeosi thought that was package dependant07:59
tsdgeosand not build wise07:59
Mirvtsdgeos: Saviq: I tend to add an empty override_dh_auto_test: in debian/rules08:02
SaviqMirv, "extra work" is not an argument IMO, we'd need work to backport fixes from stable, and that's not necessarily less work than adapting to 5.3, which is a much less disruptive change08:02
MirvSaviq: correct, removing08:03
Mirvit's quite unrealistic to think 5.2 would be now bug free08:03
SaviqMirv, "Risks in quality" sounds like we deem 5.3 be of worse quality...08:03
tsdgeosMirv: it is totally not :D08:04
MirvSaviq: tsdgeos: well, it's risky in the sense that for example now we have tests failing and scopes not working :)08:05
tsdgeosand the more we find issues with it (specially in qml) will be harder to get proper fixes once they have moved to 5.3 stable08:05
tsdgeosseems we've been lucky and not having much of those lately thoguh08:05
SaviqMirv, sure, but that's not a "quality risk", that's just us not being ready for 5.3, 'cause we're using private headers here and there ;)08:05
Mirvbut as always, if the biggest hurdles can be won in relatively little time, most apps work etc then we can start validating it08:05
Saviqtsdgeos, indeed, you fixed them all!08:05
SaviqMirv, it all *builds*, that's like a huge improvement already ;D08:06
MirvSaviq: right, leaving that bug mention the only, it covers the problems as soon as problems are identified08:06
MirvSaviq: :D08:06
* Saviq tries it out08:06
dednickSaviq: hey. got a problem with the AccountsService dbus object in unity8. The volume indicator updates the accounts service with the user volume directly, and unity8 responds to property changes on the accounts service (background, demo-edges, etc). problem is, that calls into dbus properties seem to be blocking dbus "Get" calls.08:12
dednickSaviq: causing a lag when dragging around the indicator slider. whole UI blocks (which is pretty bad!)08:13
dednickSaviq: should be on a thread?08:13
Saviqdednick, and mterry's change in the indicator didn't help?08:13
dednickSaviq: yeah. that was one issue. this is a nother08:13
dednickSaviq: the stuttering is gone. but every second you are dragging the slider, there is a short UI block08:14
Saviqdednick, are any of the set operations synchronous?08:14
dednickSaviq: because of this: QDBusMessage answer = iface->call("Get", interface, property);08:14
dednickSaviq: the Get ops are sync08:15
Saviqdednick, any idea why that Get takes so long?08:15
dednickSaviq: no idea. theres a tiny block on desktop as well, but it's not as long as phone08:16
Saviqdednick, so... in any case yeah this should be wrapped in a lib (we shouldn't be calling dbus directly)08:17
Saviqdednick, and there the operations should be threaded indeed08:17
Saviqdednick, well, you could say our QML plugin is the lib, so we could just put it in a thread there as a band-aid08:20
dednickSaviq: yeah. ok08:20
Saviqmhr3, are unity-scopes-shell tests working for you?08:26
Saviqmhr3, this seems to be happening for me on 5.2 as well: bug #131892108:26
ubot5bug 1318921 in unity-scopes-shell "unity-scopes-shell fails testpreviewtest on Qt 5.3" [Undecided,New] https://launchpad.net/bugs/131892108:26
Saviqand to tsdgeos, too08:27
SaviqMirv, :| unity-api passes locally08:28
SaviqMirv, and it's a segfault on the builders08:28
Saviqnot good08:28
dpmmorning mhr3, I'm testing scopes translations and I've come across this untranslated string: "Preview" in here. Do you know where it comes from? I think it'd need to be marked for translation in the code, but I'm not sure where to file the bug against08:34
dpmmhr3, as in this screenshot: http://i.imgur.com/xTiwh2F.png08:34
MirvSaviq: those are the most annoying kind :(08:35
Cimitsdgeos, Saviq shadows rocking this way https://code.launchpad.net/~cimi/unity8/carousel-shadow/+merge/21923308:35
wtrbnew unity user here. Could anyone point me to how to enable horizontal mouse wheel scrolling?08:51
Saviqwtrb, there are some settings in the "Mouse and touch pad" section in the settings (top right "cog" icon → settings)08:54
Saviqwtrb, it doesn't have anything about scrilling horizontally with a mouse wheel, though...08:54
wtrbSaviq, thanks for the reply. Yes there's nothing there as you say. I was guessing I'd have to use some command line option, but not sure which one.08:57
Saviqwtrb, best to ask on askubuntu.com08:58
wtrbwill do, thanks08:59
mhr3dpm, it's in shell itself09:01
mhr3dpm, but that whole thing will go away09:02
dpmmhr3, what does exactly "will go away" mean and when?09:02
mhr3dpm, latest designs replaced it with the title itself09:03
greybackdammit why did my keyboard shortcuts get reset??09:06
mhr3greyback, cause you should be using hud!09:09
dpmmhr3, ok thanks. So I guess I should ask Saviq for the shell. Saviq, so I'm testing translations for a MAE demo next week, and I came across this "Preview" string that's untranslated. mhr3 tells me that it comes from the shell and it will go away. Depending on when, I'd like to find out whether it's worth fixing it for the demo or just wait for the new designs to land. What do you think? http://i.imgur.com/xTiwh2F.png09:09
* greyback shakes his fist at mhr309:10
Saviqdpm, mhr3, I think we should fix this in unity8 still, I believe this will involve a bit more to put the title in there (like we need to tweak the preview APIs to accommodate this)09:12
mhr3Saviq, result.title || "Preview"09:12
Saviqmhr3, think that's it? we don't want the scope to have more control over this?09:13
mhr3not unless says so09:13
mhr3design says so09:13
Saviqmhr3, well, yeah, OTOH I didn't see a signed-off design yet which does have the title in the header ;)09:14
mhr3Saviq, and you won't until after malta :)09:14
Saviqmhr3, yes, that's exactly why I think we should fix i18n for it09:15
SaviqMirv, did you carry the "delegate creation" patch to 5.3? that's why unity8 doesn't work well09:15
SaviqMirv, that == not finding the added property09:15
SaviqMirv, maybe we need to adapt the patch09:15
MirvSaviq: oh, no I didn't. it would need adaptation (adaptation welcome), on these quick packages I just commented out most of the patches since most of them where backports anyhow.09:17
SaviqMirv, you have a branch somewhere or should I just apt-get source and ping you with the adapted patch?09:18
MirvSaviq: just pushed, before you asked, to https://code.launchpad.net/~timo-jyrinki/kubuntu-packaging/qtdeclarative-opensource-src_qt530RC09:19
dpmmhr3, Saviq, so where do I best file the bug to request internationalization of that string in the meantime while the new designs have not landed?09:19
Saviqdpm, with unity8, let me just put up a branch, gimme 3 mins09:19
dpmawesome09:20
SaviqMirv, tsdgeos_, btw looks like scopes-shell failing tests are handled here https://code.launchpad.net/~unity-api-team/unity-scopes-shell/fix-tests-after-config-branch/+merge/21917709:20
tsdgeos_:)09:20
MirvSaviq: you'd need the orig tarball anyway from the PPA to the parent dir, and you may want to do bzr rm debian/*.symbols for builds. yes, ping welcome!09:20
MirvSaviq: the old patch is still there, just commented out in debian/patches/series09:20
SaviqMirv, got it09:20
karnimhr3: hey. what does "my-preview-one" stand for on the Scopes JSON structure definition doc? Can there be more than one preview?09:20
karnifor a single element09:21
mhr3karni, that's for not-yet-implemented early-preview support09:22
mhr3ignore when dealing with regular preview request09:22
mhr3s09:22
karnimhr3: okay, thanks09:22
karnimhr3: but I still need to send that id, right?09:22
karni(some id)09:22
mhr3no09:22
karniso I just push everything 1 lvl up?09:23
mhr3there also isn't any "previews" key09:23
mhr3so 2 lvls09:23
karniok, what should be the root key, "preview" ?09:23
Saviqdpm, hum, it's actually already there:09:23
Saviqhttps://translations.launchpad.net/unity8/trunk/+pots/unity8/en_GB/+translate?batch=10&show=all&search=preview09:23
Saviqdpm, which translation were you using, maybe it just misses this string?09:24
dpmSaviq, oh you're right, nice!, so it's probably not translated in my locale09:24
dpmlet me check09:24
mhr3karni, hm, dunno tbh, do a request on the rest-server and look at it :)09:24
mhr3karni, also, remote/rest scopes are kindof no-no, are you sure you should be implementing them?09:25
karnimhr3: what do you mean by no-no?09:25
dpmSaviq, indeed, that was the case, thanks for the correction, and sorry for the noise09:25
karnimhr3: there are things local scopes are not suitable for09:25
Saviqdpm, no worries09:26
karnilike attempting a search over RSS feed09:26
mhr3karni, i heard somewhere that most rtm scopes should be local ones09:26
karniwhat's rtm?09:26
mhr3karni, the final image09:26
mhr3the first one09:26
karniWell, I'll forward that feedback to Chris, thanks.09:27
Saviqtsdgeos_, , hum hum hum09:30
Saviqtsdgeos_, do you know "displayMarginBeginning" and "displayMarginEnd"?09:30
tsdgeos_Saviq: doesn't ring a bell, where?09:31
Saviqtsdgeos_, delegateCreation* conflicts with them https://github.com/special/qtdeclarative/commit/40ce9bb0d3a640e16f44ab5aaeb2d050386ffb0709:31
Saviqtsdgeos_, in 5.309:31
tsdgeos_because it does te same thing09:32
tsdgeos_^_ ^09:32
tsdgeos_obviosuly w00t has more pull than i do09:32
tsdgeos_or he just bypassed aalpert09:33
Saviqtsdgeos_, exactly... assuming they can be set to negative values09:33
Saviqtsdgeos_, but I see nothing to prevent that09:34
mhr3Saviq, could you take a look at https://code.launchpad.net/~mhr3/unity8/update-scope-tool/+merge/219024 pls?09:34
Cimi@unity + seb128 and whoever knows09:34
CimiI am back with my wifi issue that my phone says connected but it's not09:34
Cimihow do I fix this?09:34
CimiI keep rebooting but no luck so far09:34
tsdgeos_Saviq: we should investigate dropping it and if our patches have some other thing this one doesn't have09:35
seb128no idea, talk to cyphermox when it's not middle of the night anymore for him I guess?09:35
Cimiseb128, I am sure I am not alone in this, did anyone reported this issue?09:35
Cimiseb128, I think it got worse over the past weeks09:35
seb128Cimi, you better ask on #ubuntu-touch, ogra had routing issues iirc09:36
Cimiok09:36
Saviqtsdgeos_, indeed, could you please do that when you have a moment?09:36
Saviqtsdgeos_, this seems to be the actual commit into qtdeclarative https://qt.gitorious.org/qt/qtdeclarative/commit/a46312b3b5f97802b8a74e53a86ce4a57df320ef09:36
tsdgeos_Saviq: i'll write me a note somewhere09:36
tsdgeos_Saviq: bah aalpert did review it09:37
Saviqtsdgeos_, nasty09:37
* tsdgeos_ shuts up 09:37
tsdgeos_and works09:37
SaviqMirv, so... it looks like the patch we had to carry was ~implemented upstream... after having been rejected first...09:40
SaviqMirv, so we'll have to investigate whether it works for us as it is upstream and potentially drop that patch09:41
Saviqmhr3, any reason why you're interrogating pkg-config runtime instead of build-time? and why doesn't scope-tool depend on the -dev where that .pc file comes from?09:43
mhr3Saviq, cause unless you run it against non-installed scope it doesn't need it09:44
Saviqmhr3, mhm09:44
mhr3Saviq, and you won't run with non-installed scope unless you have -dev09:44
Saviqmhr3, unless someone sent you the .so and .ini files (click)09:44
Saviqmhr3, in which case we'd crash won't we09:44
mhr3Saviq, in which case you'll just install the click :P09:45
Saviqmhr3, touché, but still scary :P09:45
SaviqMirv, but other than that it seems to work fine09:46
SaviqMirv, so the only unknown issue would be the test failure+segfault in unity-api09:47
Saviq/food09:47
karniIs there a way to point unity-scope-tool at locally deployed server scopes?10:04
karnion trusty (I know, old school)10:04
Cimimhr3, I need your script for the disk space! :)10:07
davidcallekarni, ./usr/lib/x86_64-linux-gnu/smartscopesproxy/smartscopesproxy http://localhost:800010:08
davidcallewell, you need to stop smart-scopes-proxy first :)10:08
karnidavidcalle: lovely, thanks!10:08
davidcallekarni, np10:09
tsdgeos_Saviq: i see in unity-api some Interfaces also define rolenames and some other seems not, shall i add htem?10:12
tsdgeos_i think it makes sense, but then it's not "Interface" anymore10:12
mhr3Cimi, also, it's ogra's, not mine ;)10:13
tsdgeos_mhr3: your opinion on ↑↑↑ ?10:16
tsdgeos_and by define rolenames i mean10:17
tsdgeos_doing10:17
tsdgeos_    m_roles[MockScopes::RoleScope] = "scope";10:17
tsdgeos_    m_roles[MockScopes::RoleId] = "id";10:17
tsdgeos_    m_roles[MockScopes::RoleVisible] = "visible";10:17
tsdgeos_    m_roles[MockScopes::RoleTitle] = "title";10:17
tsdgeos_not defineing the Enums10:17
tsdgeos_should probably be part of the Interface10:17
tsdgeos_since without that it's "worthless"10:17
mhr3tsdgeos_, not sure i follow, so you mean the strings themselves should be defined in the interface?10:19
tsdgeos_mhr3: see http://bazaar.launchpad.net/~unity-team/unity-api/trunk/view/head:/include/unity/shell/launcher/LauncherModelInterface.h10:20
tsdgeos_mhr3: it has a constrcutor and implements  roleNames()10:20
tsdgeos_mhr3: that makes sure all the implementations expect the same names, so is good, but then it's not a pure interface anymore, so it's not that good10:21
tsdgeos_mhr3: i think i'd prefer having them though, what's your opinion?10:22
mhr3tsdgeos_, right, ultimately you don't even need the constructor, so it could be slightly cleaner10:22
mhr3tsdgeos_, but then the ids could clash in various implementations, could be an issue10:23
mhr3but maybe not10:23
tsdgeos_got lost :D10:23
mhr3i mean the interger ids10:23
tsdgeos_those are defined as part of the interface10:24
tsdgeos_that is fine10:24
mhr3Scope::Roles starts with 0, but maybe an actual implementation would use 0 for its own thing10:24
tsdgeos_hmmm, don't see it happening tbh10:24
tsdgeos_i'm more worried about the "strings" being the same in all the implementations10:25
tsdgeos_and users knowing which strings they can expect10:25
tsdgeos_since right now it's "guessing"10:25
tsdgeos_which string in qml gives me RoleScope ?10:25
tsdgeos_from looking at the interface i can't know10:25
tsdgeos_but if i add the rolenames mapping10:25
tsdgeos_i do10:25
mhr3tsdgeos_, ok, if the int id is not an issue, i don't see a problem with it... and as i said you don't need to keep it as private member, so can be simpler/cleaner10:27
tsdgeos_mhr3: how should i do it then if not using a private member?10:28
tsdgeos_want me to use a function static map?10:28
tsdgeos_s/map/hash10:28
mhr3tsdgeos_, just return a new map in roleNames()10:29
mhr3qml caches it anyway afaik10:29
tsdgeos_ok, hope that's not too slow :D10:29
mhr3tsdgeos_, Saviq told me as some point that that is actually the desired way10:30
tsdgeos_ok10:31
mhr3Mirv, where there some changes to qt in past 2 days?10:43
mhr3Mirv, we've seen some weird armhf failures and suddenly they're gone again10:44
Saviqtsdgeos_, in general it's only read once, there's no point in keeping a copy around when QML has to keep it anyway10:45
tsdgeos_ok10:45
Saviqtsdgeos_, and yeah, add rolenames, it still is an interface, only not in the programming-language sense (i.e. not a pure-virtual class)10:46
tsdgeos_added10:46
tsdgeos_doing the tests10:46
karniSaviq: any chance for icon actually surfacing in category header? have there been designs delivered? I'm asking, as I would love to be able to show an icon in the category header.10:50
karnior, the scope header, for that matter :)10:51
Mirvmhr3: past two.. no, I don't think so, aside from ricardo's x86 emulator related builds10:51
Saviqkarni, https://docs.google.com/a/canonical.com/document/d/1jeGVALlFH7KFEuu7DNPW1KhEhBFh1GhcT4ZsSxIRRqY/edit10:51
karniSaviq: thanks, reading10:51
Saviqkarni, there's also a weekly engineering/design meeting (in 9 mins actually)10:51
Saviqkarni, most of scope customization is missing from the spec10:52
karniaha10:52
mhr3Mirv, ok, something else then, ty10:52
Mirvmhr3: I don't always get to know myself if someone uploads a Qt module, so that's why I simply checked https://lists.canonical.com/archives/utopic-changes/2014-May/thread.html10:54
mhr3ah, useful :)10:55
Saviqtsdgeos_, fyi, I invited you (optional) to a weekly design sync on dash11:00
tsdgeos_ok11:00
Saviqtsdgeos_, it starts now if you'd want to join, but it's not necessary if you'd rather work11:00
tsdgeos_Saviq: yeah i'd just rather do all these billions mocks & Verifier {} for unity-api11:01
Saviqtsdgeos_, yeah... we need to look into what UITK does for this (like diff the plugindump or something...)11:01
tsdgeos_or that11:02
tsdgeos_but still let's finish this :D11:02
Saviqtsdgeos_, yeah, not now11:02
Saviqtsdgeos_, /nick tsdgeos?11:02
=== tsdgeos_ is now known as tsdgeos
CimiSaviq, where is QLightDM::UsersModel defined from the shell LightDM plugin?11:22
SaviqCimi, otp11:22
Cimiok11:22
=== jhodapp|afk is now known as jhodapp
=== MacSlow is now known as MacSlow|lunch
karnimhr3: thoughts? http://paste.ubuntu.com/7457016/ - this may be python specific, but I don't see how without line 7 the scope would know how to layout widgets in 3 columns, even if 5 and 7 are syntactically equivalent11:31
Wellarkdednick: this is ready for testing: https://code.launchpad.net/~kaijanmaki/unity8/indicator-root-state-icons-fix/+merge/21372711:38
Wellarksee the silo in the description.11:38
WellarkSaviq: I've removed the spurious tags. please check and mark as approved from your side --^11:40
Wellarkwhat is this unity-phablet-qmluitests-trusty jenkins job and why is it always failing in ci ?11:49
CimiSaviq, still otp?11:50
mhr3karni, hm... looks good to me11:50
mhr3davidcalle, ideas ^?11:50
karnihttps://bugs.launchpad.net/unity-scopes-api/+bug/131901911:50
ubot5Ubuntu bug 1319019 in unity-scopes-api "Scope preview column problem" [Undecided,New]11:50
karnisorry for bug title ;P11:50
SaviqCimi, back now11:51
mhr3marcustomlinson, do you remember how those are parsed ^?11:52
Saviqdednick, can you follow up on https://code.launchpad.net/~kaijanmaki/unity8/indicator-root-state-icons-fix/+merge/213727 please?11:52
CimiSaviq, I think I have the right stuff installed11:52
marcustomlinsonmhr3: looking11:53
CimiSaviq, http://paste.ubuntu.com/7457085/11:53
SaviqCimi, plugins/LightDM/UsersModel.{cpp,h}11:53
davidcallekarni, no idea, the onlinemusic scope from u-r-s works fine with two cols too, haven't tried three.11:53
CimiSaviq, I have UidRole11:53
SaviqCimi, but there's also a bunch of mocks11:53
CimiSaviq, but if in code I do11:53
CimiSaviq, QVariant data2 = QSortFilterProxyModel::data(index, QLightDM::UsersModel::UidRole);11:54
karnidavidcalle: two cols are no problem11:54
SaviqCimi, in plugins/mocks/LightDM/{full,single,demo...}11:54
CimiSaviq, make does error: ‘UidRole’ is not a member of ‘QLightDM::UsersModel’11:54
SaviqCimi, it's probably loading one of the mocks and not the one you've changed11:55
mhr3dpm, do you see any of these translatable? http://bazaar.launchpad.net/~ubuntuone-hackers/ubuntu-rest-scopes/trunk/view/head:/configs/scopes-info.yaml ?11:55
SaviqCimi, but anyway11:55
SaviqCimi, you just added it to the enum, did you also add it to roleNames()? and handled it in data()?11:55
CimiSaviq, I had everything11:55
CimiSaviq, https://code.launchpad.net/~cimi/lightdm/uid-bindings/+merge/21895811:56
SaviqCimi, just put a breakpoint in data() of the model, and see if that's actually what's called11:56
SaviqCimi, we're not actually using lightdm11:56
dpmmhr3, it seems that the descriptions are translatable, but none of the names are: https://translations.launchpad.net/ubuntu-rest-scopes/trunk11:56
SaviqCimi, we mock the same API11:56
SaviqCimi, until we're in split greeter11:56
CimiSaviq, I see11:56
CimiSaviq, path to our mocks?11:56
mhr3dpm, then that's the other part of the problem11:57
SaviqCimi, I wrote like 5 above11:57
CimiI don't have those11:57
SaviqCimi, in lp:unity811:57
Cimiin tests?11:57
SaviqCimi, there's plugins/LightDM/UsersModel.{cpp,h}11:57
dpmmhr3, ok thanks for investigating that, will file a bug11:57
CimiSaviq, I have those11:57
SaviqCimi, then there's the test mocks in tests/mocks/LightDM/*/UsersModel.{cpp,h}11:57
Cimiah tests11:58
Saviqactually UsersModelPrivate11:58
Saviq7 in total11:58
Saviqyou need to update all those11:58
CimiI am trying now11:59
Cimicompiling...11:59
mhr3dpm, pls reply to the thread with the bug# then12:00
Saviqmhr3, how do I try out the tool update?12:00
=== alan_g is now known as alan_g|lunch
dpmmhr3, done, thanks12:01
mhr3Saviq, branch for example lp:unity-scope-scopes, build it, and do `...-tool ./builddir/path-to.ini`12:02
CimiSaviq, ok works12:05
Saviqmhr3, it fails trying to aggregate, worked with the click scope12:05
CimiSaviq, thank you, didn't realise was using mocks and not qlightdm directly12:05
Saviqmhr3, but for future reference, what do I do to enable aggregation?12:05
SaviqCimi, nw, we can't use the real impl until split greeter12:06
mhr3Saviq, yea, there's an issue in -api where it can't combine regular server proxy with custom registry (which the tool is using)12:06
mhr3Saviq, i'll catch that up12:07
CimiSaviq, so I need to fake the uid then12:09
SaviqCimi, yup12:09
Cimiok12:09
CimiSaviq, in demo full12:09
CimiSaviq, can I fake them all to the same uid?12:09
SaviqCimi, do they have the same name?12:10
CimiSaviq, well infographics only work on 3201112:10
Cimiuid12:10
Cimiof phablet12:10
Cimithe user who is running the service12:10
SaviqCimi, except you shouldn't be using the real infographics but mock it12:12
SaviqCimi, as it was mocked before (but now it's rather simpler to mock)12:12
CimiSaviq, we were using real infographics12:16
CimiSaviq, when I took pics I saw new pic on my screen12:16
=== MacSlow|lunch is now known as MacSlow
* Cimi lunch break12:28
dednickSaviq: i thought it had been approved...12:34
dednickmaybe CI unapproved.12:35
Saviqdednick, I think it was, but your vote is still Needs Fixing ;)12:35
Saviqdednick, Wellark unapproved it12:35
dednickSaviq: ah. ok, well i've approved. can top approve Wellark ?12:36
=== alan_g|lunch is now known as alan_g
marcustomlinsonkarni: hey, I left a comment on that bug. make sense?12:47
karnimarcustomlinson: oh, totally! thank you. feel free to mark this invalid.12:48
marcustomlinsonkarni: cool, np :)12:49
Wellarkdednick_: we still need a ci train TestPlan checklist12:57
Wellarksorry, Reviewer Checklist12:57
Wellarkso we can land12:57
Wellark(not going to land this minute, still fighting the indicator side)12:58
Wellarkbut the unity8 branch can be tested already12:58
Wellarkno changes to the icon handling will happen anymore.12:58
pete-woodsSaviq: quick question, if I want to get the "latest" unity builds, do I need to upgrade to utopic, or is there a nice PPA or something for trusty?12:59
Saviqpete-woods, utopic12:59
pete-woodsSaviq: okay, thanks :)13:00
dednick_Wellark: ok. well, I've done the checklist13:07
Wellarkdednick_: thanks!13:09
Wellarkdednick_: now it's ready to land as soon as the other MP's in the silo are OK.13:10
Saviqgreyback, I say we're in violent agreement on shell rotation, but you were high when you've written the last email ;)13:38
Saviqgreyback, my only worry is simply that things won't reflow fast enough, that's why I was looking at ways to fake it a bit, but we might just as well take that on later...13:39
greybackSaviq: that's what I get for stopping the pills13:47
Saviqgreyback, :)13:47
=== _salem is now known as salem_
darklight_Saviq, any news on Uhttps://bugs.launchpad.net/ubuntu/+source/compiz/+bug/1305438 ? I also wrote to the ML as you suggested but nobody cared (or cared enough to answer at leasta)13:58
ubot5Ubuntu bug 1305438 in unity (Ubuntu) "hot corners stop working time to time" [Undecided,Confirmed]13:58
Saviqdarklight_, sorry, I'm not working on compiz at all, you'll have to be more patient13:59
darklight_Saviq, but it's not a compiz issue it's a unity issue13:59
Saviqdarklight_, I'm not working on unity (7) either ;)13:59
darklight_ok , do you know anyone I could bother ? :)14:00
Saviqdarklight_, the folks maintaining unity7 are mostly late US14:00
darklight_pacific time ?14:00
Saviqdarklight_, filing a bug is best, they'll get to it in due time14:00
darklight_Saviq, that is the bug it has already been filed14:01
Saviqdarklight_, yes, subscribe to it and arm yourself in some patience I'm afraid is the only thing I can recommend14:01
darklight_Saviq, well the thing is it has been broken for ~2-3 releases so I have a feeling they just don't care enough to fix it14:03
Saviqdarklight_, can't comment on that14:03
tsdgeosSaviq: https://code.launchpad.net/~unity-team/unity-api/unity-shell-scopes-api/+merge/219379 is ready for mhr3's docu i'd say14:10
tsdgeosSaviq: can you give it another review before he adds the docu?14:10
Saviqtsdgeos, yup, saw that, will do14:11
tsdgeosit's one of those reviews where it looks big because the test code is huge14:11
tsdgeoscompared to the code itself14:11
Saviqtsdgeos, yup14:11
Saviqmhr3, https://code.launchpad.net/~unity-team/unity-api/unity-shell-scopes-api/+merge/219379 is ready for your doc-love :)14:19
cimi_cannot connect to my ZNC, use Cimi_ for queries instead :)14:19
cimi_still receiving pings though14:20
=== alan_g is now known as alan_g|tea
mhr3Saviq, ouch, felt like slap in the face right after joining :P14:21
Saviqmhr3, should not have left! ;)14:21
mhr3heh14:22
=== alan_g|tea is now known as alan_g
=== dandrader is now known as dandrader|afk
tsdgeosmterry: i fixed the Dash test, can you quick review?14:40
cimi_paulliu, MacSlow my connection degradated at the last minutes, can you ping me your minutes and I add them14:40
mterrytsdgeos, yes!  let me look14:40
mterrythanks14:40
paulliucimi_: ok. I'll fill it on googledoc.s14:40
tsdgeosmterry: thankyou for noticing :14:40
tsdgeos)14:40
MacSlowcimi_, I'll update the notes myself... no worries14:40
MacSlowcimi_, done14:43
cimi_MacSlow, thanks14:44
cimi_Saviq, I don't understand if I have to fake the infographics on the shell or just for the tests14:45
Saviqcimi_, just for the tests14:45
cimi_Saviq, cool14:45
cimi_Saviq, it will be like a list of svg then, easy14:46
Saviqcimi_ yup14:46
cimi_ok14:46
tsdgeosdandrader|afk: what's missing for top approval of https://code.launchpad.net/~paulliu/unity8/logout/+merge/216373 ?14:50
paulliucimi_: done.14:51
cimi_paulliu, thanks14:51
tsdgeosmterry: should the tests  of https://code.launchpad.net/~mterry/unity8/split/+merge/213149 work?14:52
mterry tsdgeos, tests?  like autopilot and qmluitests?  Yes, except for the existing broken qmluitests14:53
tsdgeosmterry: ok, i remember some breakage on the autopilot unlock, is that gone?14:53
mterrytsdgeos, should be14:54
tsdgeosgood, let's see :=)14:54
cyphermoxcimi_: re your issues with connectivity; try the dnsmasq package from this PPA and let me know if it helps: https://launchpad.net/~mathieu-tl/+archive/build-tests/+packages14:54
cyphermoxoh wait, it was for the phone, this ppa doesn't build armhf14:54
cimi_cyphermox, indeed14:54
cimi_but thx14:54
cyphermoxI'll copy it to another14:55
mzanettiSaviq: ping14:55
Saviqmzanetti, pong14:55
mzanettiSaviq: I've entered details in the choo-choo spreadsheet14:55
mzanettiSaviq: however, it changed a bit since I last used it. what's the requestId?14:55
mzanettiSaviq: row 3214:56
Saviqmzanetti, you don't look past I14:56
mzanettiack14:56
Saviqmzanetti, it's an issue with the spreadsheet - they should be hidden14:56
mzanettiso everything should be fine then14:56
mzanettiSaviq: do I need to do anything to get a silo to test this?14:57
Saviqmzanetti, yup, but you need to wait for silo 008 to land14:57
mzanettiSaviq: right. not in a hurry with this14:57
mzanettiSaviq: once I got the silo, I'll test and move it on from there asap.14:57
Saviqmzanetti, and then I'll find out whether silo 009 wants to land or whether we should still ignore it for now14:58
mzanettiSaviq: I'm bugging designers on a daily basis to tell me how to proceed with the ui14:58
Saviqmzanetti, right edge you mean?14:58
mzanettiSaviq: no.. dual sim pin entry14:59
Saviqmzanetti, ah that, yeah14:59
mzanettiSaviq: I've got functionality mostly implemented, just no idea where to put the labels14:59
mzanettiSaviq: re right edge, vesa is back :)14:59
=== dandrader|afk is now known as dandrader
dandradertsdgeos, I checked the code but I didn't test it or even compiled it15:12
dandraders/checked/reviewer15:13
dandraderreviewed15:13
MacSlowmzanetti, will the dual sim-pin entry affect the fullscreen notification using the pinpad item?15:14
tsdgeosdandrader: can you say so on the review, it was unclear to me (or maybe you did and i read too fast :D)15:16
dandradertsdgeos, I said in my comment (there's a checklist there)15:16
=== salem_ is now known as _salem
tsdgeosdandrader: ok, my bad then, sorry15:16
Saviqgreyback, https://code.launchpad.net/~gerboland/unity-mir/fix-upstart-closed-apps2/+merge/218721/comments/52332415:20
cimi_tsdgeos, this was branch you wanted review? https://code.launchpad.net/~aacid/unity8/fix_tstDash/+merge/21929615:20
SaviqETOOMANYCIMIs15:21
tsdgeoscimi_: mterry was having a look at that one15:21
cimi_ok15:21
tsdgeoscimi_: https://code.launchpad.net/~aacid/unity8/remove_empty_dirs/+merge/219168 ?15:21
tsdgeosit should take you around 5 seconds to do that one15:21
cimi_Saviq, I am away from home and cannot access my home server :) I think the port mapping is fucked up15:21
cimi_pls use cimi_ for the rest of the week15:22
mzanettiMacSlow: yes, it will. I'll let you know if I need something from you15:22
MacSlowmzanetti, ok15:22
cimi_unless I hack on my own home network :)15:22
cimi_no idea how though15:22
greybackSaviq: you recall, had the webapp actually died after the first pkill?15:23
Saviqgreyback, yes it did15:23
Saviqgreyback, it's 100% reproducible - like when it resumes, it doesn't get "canberesumed = true"15:23
greybackSaviq: ok15:24
tsdgeosSaviq: upstart crashed again :S15:29
tsdgeosSaviq: should that bug you mentioned be already fixed?15:29
mhr3alecu_, what app uris is click using?15:30
mhr3alecu_, do they include version as app id or not?15:30
dandradertsdgeos, Saviq: the fault of my missing dash was in a silent javascript NaN situtation... deja vu15:31
Saviqtsdgeos, just did for me on the phone, too...15:31
Saviqtsdgeos, filed https://bugs.launchpad.net/ubuntu/+source/upstart/+bug/131908315:31
ubot5Error: ubuntu bug 1319083 not found15:31
tsdgeosSaviq: wrong bugno?15:32
Saviqtsdgeos, try again, is private still15:32
tsdgeosah15:32
tsdgeosok15:32
=== gatox is now known as gatox_lunch
alecu_mhr3: ? what uri are those? the uris for the actual .click package files, or the uris for package details from the webservice?15:33
Saviqgreyback, it obviously misses "go to webapp" after 315:34
Saviqgreyback, to resume it (the steps I mean)15:34
greybackSaviq: ahh, not so obvious :)15:35
Saviqgreyback, sorry, just read through the steps ;)15:35
mhr3alecu_, the ones that launch actual applications15:36
greybackSaviq: so you resumed the web app, then switched right back to the dash? Did you wait for hte webapp to appear?15:36
Saviqgreyback, it did15:36
Saviqgreyback, even waited for it to suspend15:36
greybackSaviq: ok, that helps, will dig15:36
Saviqgreyback, and now it just goes away every time :/15:37
Saviqafter upstart crashed and restarted my session... /me reboot15:37
Saviqs15:37
* greyback hates webaps15:37
greybackupstart kinda brittle these days15:39
cimi_Saviq, I think we need to add liblightdm-qt5-3-dev to build deps, no?15:46
cimi_and the non dev to depends15:47
alecu_mhr3: I see that the application:/// url includes the version15:48
tsdgeosmterry: so from your pov the split branch is "done"? or there's stuff to do?15:48
mhr3alecu_, mzanetti is asking to ideally drop it15:49
alecu_mhr3: it's actually: manifest.name + "_" + manifest.first_app_name + "_" + manifest.version + ".desktop";15:49
alecu_mhr3: we would need to change the click desktop hooks too15:49
mterrytsdgeos, two things: indicator team needs to fix syncing between sessions with indicator-messages.  And I'm investigating a new breakage with init scripts due to some ubuntu-touch-session changes.  But those are not the unity8 branch.  That I think is done (hopefully!  ;))15:49
Saviqcimi_, why? we're still not actually building against it or linking to it15:50
alecu_mzanetti: ^15:50
tsdgeosmterry: ok15:50
cimi_Saviq, well fresh system15:50
mzanettialecu_: yes, we're stripping them to the short app id (dropping _version)15:50
cimi_Saviq, apt-get build-dep unity815:50
cimi_Saviq, ./build.sh15:50
cimi_Linking CXX shared module libMockLightDM-qml.so15:50
cimi_/usr/bin/ld: cannot find -llightdm-qt5-215:50
mzanettialecu_: or rather, asking you to do so, so that we only have short appids in the shell15:51
cimi_first of all we have qt5-3 now iirc15:51
alecu_mzanetti: I'll ping the people doing the click hooks, and I can do the work on the click scope15:51
mzanettialecu_: awesome, thanks!15:51
Saviqcimi_, especially since that's a _mock_ you should not even think of solving that with the real library15:52
alecu_mzanetti: would you mind opening a bug for that?15:52
Saviqcimi_, which you can't access still until you're running split greeter15:52
mzanettialecu_: ack15:52
cimi_Saviq, anyway found the cmakefile that needs update15:52
alecu_mzanetti: https://bugs.launchpad.net/ubuntu/+source/unity-scope-click/+filebug15:52
cimi_Saviq, there is qt5-3 now15:52
mzanettialecu_: this is the one that triggered all this: https://bugs.launchpad.net/unity8/+bug/123975015:53
ubot5Ubuntu bug 1239750 in Unity 8 "Launcher items get "stale" after app upgrades" [High,In progress]15:53
mzanettialecu_: I might just add unity-scope-click as affects too15:53
Saviqcimi_, you should adapt all the mocks to do -3 now  then15:53
mzanettialecu_: the title is a bit misleading by now tho :/15:53
Saviqcimi_, and there's a debian/rules comment that should get updated15:54
cimi_Saviq, all done15:55
cimi_Saviq, sed ftw15:56
alecu_mzanetti: I think the title of that is perfect, and yes, I see how launcher items would get broken by this15:56
mzanettialecu_: ok, I can't add unity-scope-click to it for some reason. It says it doesn't use LP to track bugs15:57
alecu_mzanetti: right: we are tracking bugs in ubuntu's source package. Let me do it.15:58
mzanettiok, thanks15:58
Saviqo/ folks16:05
=== alecu_ is now known as alecu
cimi_Saviq, https://code.launchpad.net/~cimi/unity8/lightdm-bump/+merge/21939716:05
Saviqmterry, can you look at ↑ please16:06
mterrySaviq, cimi: shouldn't be needed -- we build our own qt5-2 version16:06
mterrySaviq, cimi: liblightdm bumped to 3 months ago, hasn't been a problem yet16:07
cimi_mterry, but doesn't work on my system16:07
mterrycimi, sounds like something isn't being pointed at our mock qt5-2 version16:07
cimi_was complaining about missing that16:07
cimi_mterry,  /usr/bin/ld: cannot find -llightdm-qt5-216:07
mterrycimi, right, I get that.  But sounds like maybe mock qt5-2 isn't built yet or isn't being pointed at right16:08
mterrycimi, if you need to make the changes you did, that means something is pointing at system liblightdm which shouldn't be the case.  you should always be pointing at our mock version16:08
cimi_ok16:09
cimi_mterry, don't know then, works for you?16:14
mterrycimi_, yeah.  How are you building?  Did you make any related changes?16:15
cimi_mterry, trying trunk16:15
cimi_mterry, new machine16:15
mterrycimi_, and building via ./build.sh?16:15
cimi_mterry, yes16:15
mterrycimi_, ok will give a fresh checkout a try16:15
cimi_mterry, also make you you don't have that library system wise16:16
cimi_mterry, spot any mistake here? http://bazaar.launchpad.net/~unity-team/unity8/new-infographics/revision/822#plugins/LightDM/plugin.cpp16:18
cimi_mterry, the change in tests/mocks/LightDM/CMakeLists.txt I bet16:19
alecuted: hi! are you the one maintaining the desktop-hook in https://launchpad.net/upstart-app-launch ?16:19
tedalecu, Yup16:19
cimi_Saviq, when you have time, looks like you removed the dependency there http://bazaar.launchpad.net/~unity-team/unity8/new-infographics/revision/822#plugins/LightDM/plugin.cpp16:19
tedalecu, Though I'm trying to get it to go away :-)16:19
ted(by default, not entirely)16:20
cimi_http://bazaar.launchpad.net/~unity-team/unity8/new-infographics/revision/822#tests/mocks/LightDM/CMakeLists.txt16:20
cimi_Saviq, is there a reason?16:20
alecuted: ah! so, I've been asked by mzanetti to fix the click scope for this: https://bugs.launchpad.net/ubuntu/+source/unity-scope-click/+bug/123975016:20
ubot5Ubuntu bug 1239750 in Unity 8 "Launcher items get "stale" after app upgrades" [High,In progress]16:20
mterrycimi_, trunk just built for me16:20
alecuted: I was thinking of either 1) changing the click scope and the desktop-hook to rename .desktop files to the short app name16:21
alecuted: or 2) making the click scope use short uris (appid://short instead of application:///long.desktop)16:21
tedalecu, I think that you should use appid://pkg/app/current-user-version, but also you should install your own desktop hook to manage your cache.16:22
alecuted: I don't currently have a cache16:23
tedalecu, That way you can do thinks like break apart the names into your search format on install instead of startup.16:23
alecuted: hmmm.... right.16:23
alecuted: is "current-user-version" just that const string?16:24
tedalecu, Yes16:24
alecugreat16:24
tedalecu, There's a UAL function to break apart an appid into its pieces, please use that :-)16:24
tedLong term, I think those desktop files should be the short ids because it'll be easier for legacy desktops. But since we don't have any yet, I haven't prioritized changing it much.16:25
alecuted: are hooks re-run on system updates? I'm trying to understand what would happen for people that already have apps installed in their phones.16:26
tedalecu, They're run when the user logs in after the system update. For encrypted homes.16:26
alecugreat16:27
tedalecu, At the start of each session there's a check to see if any hooks need to be run.16:27
=== gatox_lunch is now known as gatox
=== alan_g is now known as alan_g|EOD
cimi_#ubuntu-unity16:28
cimi_hah16:28
cimi_mterry, sorry I am back16:28
karniphablet-screenshot no worky? remote object '/tmp/mir_screencast_768x1280.rgba' does not exist16:29
=== dpm is now known as dpm-afk
mterrycimi_, I was just saying that trunk built for me16:34
cimi_mterry, yes16:34
mterrycimi_, are you using ninja?16:34
cimi_mterry, it looks like saviq in the attempt of removing usermetrics mock might had removed something important16:34
cimi_mterry, http://bazaar.launchpad.net/~unity-team/unity8/new-infographics/revision/82216:34
mterrycimi_, , but trunk still works (for me)16:34
cimi_mterry, indeed16:34
cimi_mterry, is that branch that doesn't16:35
mterrycimi_, oh!  trunk works for you?16:35
cimi_yes16:35
cimi_tried now16:35
cimi_will try later on16:36
dednick_mterry: ping16:56
mterrydednick_, heyo16:56
dednick_mterry: hey. so that other problem with the laggy slider; it's a blocking call in unity8 originating from the accountsservice plugin.16:57
dednick_mterry: the unity8 plugin is getting property updates for the volume change, and the the get/set dbus operations for other properties synchronous16:58
mterrydednick_, ooh, good tracking down.  That sounds like my fault again  :-/16:59
dednick_mterry: yeah :) I had a go at it this morning, but wasn't really sure about the best way to get it sorted. Possibly needs to go on a thread, or use async calls.16:59
mterrydednick_, if I recall correctly, that code is trying to be a qml plugin, and those property bindings don't do well with async code -- I assume that's why I wrote it like that17:00
dednick_mterry: there's a separate dbus & qml part to it.17:01
mterrydednick_, yeah but the qml part uses the dbus part17:01
dednick_mterry: yup17:01
mterrydednick_, I dunno, I'm just guessing why it may have been left as sync17:01
dednick_mterry: you want to take a look at it? or should i look more into it?17:05
mterrydednick_, it's my fault, I can look17:05
=== dandrader is now known as dandrader|lunch
=== _salem is now known as salem_
dednick_mterry: aiight.17:10
tedgreyback, What does unity-mir use the desktop file for?17:45
greybackted: read the application name, comment, icon, stage hint17:46
greybackas it supplies the shell with a model of running applications plus their metadata17:46
tedgreyback, Perhaps I'm a bit confused on the arch, but I'm surprised that unity-mir handles the user visible strings.17:46
greybackted: Unity-mir just handing shell a list of PIDs isn't very friendly to it, especially as unity-mir needs to open the desktop file to get startup data like destination-stage and splash-image17:49
greybackted: what's your concern?17:49
tedgreyback, More surprise, I was reading through the shortid branch.17:50
tedgreyback, I'm a little worried that running apps aren't tracked by full appid.17:50
tedgreyback, For instance, if you stop an app, you should be passing the full app id. Not the version that was just installed.17:50
greybackted: why? Are not these short appIds unique per user session?17:51
tedgreyback, No, you could upgrade a click while the session is running.17:52
greybackted: right, so the appId the launcher saved is now incorrect, as the version string changed17:52
greybackwhich is the core bug we're trying to work around17:53
tedgreyback, Correct, or you couldn't stop it.17:53
tedgreyback, So I think it's fine to start an app with the short id, but then you need to refer to that instance with the long from there on out.17:54
greybackted: Way the code works: shell asks unity mir stop app with short-appId. unity-mir asks upstart, what is the long appId for this short-appId, please close that17:54
greybackted: all conversation with upstart is using long-appId17:54
tedgreyback, Yes, so in that case we ask click for what is the current version for this user.17:55
greybackted ah I see what you mean17:55
greybackted: so you know the reason we've been trying short appId. Launcher saves the appId of pinned apps. If app updated, the appId stale17:57
tedgreyback, Yes, and I think it makes sense for the launcher to save the short id. I think you just need to translate, when running, once.17:57
greybackted:but how does launcher know that the appId changed, and so the desktop file it reads for its icon & name, has moved somewhere new, so it should load the new one?17:59
=== dandrader|lunch is now known as dandrader
tedgreyback, You should install a desktop hook for that. Then when it gets changed you'll get a signal.18:00
* larsu wonders why the app id would ever change...18:00
tedlarsu, appid has the version number in it18:00
larsuoh, that doesn't seem like a good idea to me18:01
larsubut then, I only barged into this dicussion and know nothing of the subject18:01
larsuso I'll trust you guys to do the right thing and continue with my evening :)18:01
tedHeh, it's good for some things bad for others.18:01
tedKeeping user preferences it's bad for. Managing security profiles it's good for.18:02
greybackted: I need mzanetti here to discuss this, I don't know the launchers difficulty with long appIds18:03
greybackted: but I will ask, what would be so wrong with having UAL support short appIds? UAL knows the installed & running apps, it can always figure out the right thing18:04
tedgreyback, Kinda yes, kinda no. It doesn't really know anything, it queries Upstart for information. And it's not very good at searching, it would be kinda expensive.18:04
tedgreyback, I'd imagine (and I haven't looked in detail) that at some point you're giving an object back to the launcher, no? Wouldn't it only need the id to create that object?18:06
greybackted: and I don't understand why the desktop file can't have a name without the version string, and placed in the user's local app directory. System apps similar, in /u/s/apps.18:06
greybackted: unity-mir not passing a fully read desktop file object to launcher, no. Launcher reads the desktop files it needs on it's own. Yeah, there's a bit of duplication re desktop files18:08
tedgreyback, I think it could. And really probably should. But I really only consider those desktop files for compatibility, not a long term solution. We need to move to caching the info on install of the app instead of reading the files at startup.18:08
greybackted: desktop files in freedesktop spec, what's wrong with them? They do the job nice & simply. If we want to build a cache of them, fine, but keep the plain text file around18:09
tedgreyback, Ah, okay. I wasn't thinking read desktop file as much as a UnityMirApplicationObject or some such.18:09
tedgreyback, Sure, keep it in the package. But no need to generate another one.18:10
cyphermoxCimi: the issue you had was with connecting to websites from the phone right? not necessarily with indicator icons and whatnot?18:10
greybackted: unity-mir does create an Application object, that the shell can use to read app specific info from (desktop file data part of that model alright). So in that way, you're right. But for apps that Unity-mir hasn't lauched, the Launcher itself has to read the desktop files18:11
tedgreyback, So as long as that object contains the long app id, could you call stop with the data in it?18:11
greybackted: keeping it in package doesn't follow freedesktop spec, and package update deletes it and re-creates it in a different directory.. That's problematic. Other shell will simply not use that18:12
greybackted:  that could be done18:12
greybackted: but frankly, this is a mess18:12
greybackted: short & long appIds??!18:12
greybackI don't like this solution at all18:13
tedgreyback, short to create, long stored and used.18:13
greybackted: I fail to see any reason that a shell cares about the version string in the app Id. As far as I'm concerned, it's an implementation detail of click packaging, that its utilities should mask from the shell18:14
greybackted: that's what I'm trying to do in unity-mir, hide version string entirely. I honestly don't think I should be doing that either18:14
tedgreyback, The shell cares because it's a critical part of our application lifecycle story, which is restrictive for really good reasons. To be restrictive you to be precise.18:14
tedhave to be18:15
greybackted: please give me an example of when shell needs to know this version string. that would satisfy me18:15
tedgreyback, To use the correct information about starting an app. One version could be sidestage and the next version not. You have the info on the wrong version it behaves inconsistently.18:17
greybackted: a user can have multiple versions of the same app installed??18:18
tedgreyback, They could start one, upgrade.18:18
tedHopefully only for a short time.18:19
greybackted: that's not having multiple versions installed simultaneously though18:19
tedgreyback, I think at some level you have to go from long id to short. Not everything needs to know long for sure. I think that unity-mir is a good place to do that.18:21
greybackted: so sudoku_1 is running. You update. sudoku_2 is new installed, but sudoku_1 is still running. shell asks UAL to stop "sudoku" <- tell me UAL/upstart could not figure out what needs to stop18:21
tedgreyback, It's more than a UAL thing. Let's say sudoku_1 doesn't have permission to send notifications but sudoku_2 does.18:22
tedgreyback, And really, most of UAL is a library. So the process that'd be figuring all that out would be unity-mir anyway :-)18:23
greybackted: I agree that there's a boundary where translation between long & short appIds has to happen. But I don't think unity-mir is the place. If you want someone else to implement a shell that uses UAL, they'll have to care about long & short appIds too, for no other reason than UAL not hiding those detail from them18:26
tedgreyback, So if I changed UAL to create an application object from the short id that you could then call stop/start on, would that be better?18:26
greybackted: upstart_app_launch_start_application() <- could it support both long & short appId?18:28
greybackpreferably just short appId - but we might need a transition period18:28
tedgreyback, Sure, but stop is the issue.18:29
greybackted: for the update reason?18:30
tedgreyback, Correct.18:30
tedgreyback, So if we could have a _create_application that returned an object with start and stop functions that would work.18:30
greybackted: let me work through the steps: shell asks sudoku to start, so unity-mir asks UAL to start sudoku. Internally UAL determines that sudoku is actually sudoku_1 and launches it18:31
greybackCase 1 - no update: shell asks to stop sudoku, so unity-mir asks UAL to stop sudoku. Internally UAL determines that sudoku is actually sudoku_1 and stops it. All good18:32
greybackCase 2 - an update happened, sudoku_1 -> sudoku_2. shell asks to stop sudoku, so unity-mir asks UAL to stop sudoku. Internally UAL does what to stop sudoku_1?18:33
tedgreyback, If we had an object we could track the Upstart instance.18:33
greybackted: you're proposing instead of just using strings to identify applications, when I start an app, UAL returns an object that unity-mir can operate on?18:35
greybackted: that's a nice idea18:35
greybackwould require a rather large refactoring of UAL, but it solves the problem18:36
greybackI would still very much like that UAL only refers to short appIds18:36
tedgreyback, Not too much, let me look at it. Guessing you want to EOD here soonish :-)18:37
greybackted: yeah it's late enough here18:37
greybackted: ok, let me know how much work you think it would be. mzanetti & I will consider if our shortAppId stuff will suffice in the short term or not (if yes, I'll try fixing that bug you observed)18:38
tedgreyback, Cool, I'll try to mock something up.18:39
greybackted: great, thanking you18:39
greybackted:  one last thing, what would be an objection to symlinking the desktop file in the click package to ~/.local/share/applications on install?18:42
tedgreyback, That doesn't really work as the Path isn't right. But I think we could do that to ~/.cache/unity/click-cache/18:43
tedgreyback, Then you don't need to search for them.18:43
tedgreyback, Unity-mir doesn't need all the aa-clickexec stuff to be right like other desktops not using UAL would.18:44
tedgreyback, Those could also have the shortid pretty easily as click already supports that.18:44
greybackted: the lack of searching would be great. But then, what is difference between ~/.cache/unity/click-cache/ and  ~/.local/share/applications ?18:45
greybackit's just an arbitrary directory. But one is part of a freedesktop spec and would help compatibility with other shells18:45
tedgreyback, The ~/.local/s/a would have the Path set to the click directory and the exec line changed, etc. Things that other folks need as well.18:46
tedgreyback, But I'd like to deprecate that on Unity8 only installs so we don't have to create the files in the user's home directory.18:46
greybackted: okay. it'll get confusing for people if they try to use utils they know like xdg-open on a unity8 desktop though and they don't work like expected...18:48
tedgreyback, It won't work anyway as we'll reject them starting without using UAL.18:48
greybackted: true. Though I don't look forward to using a desktop like that.18:49
tedgreyback, I'm thinking that we should have the terminal app create a trusted session on itself, and then pass that to apps. They'll end up overlayed on the terminal app, which will work for most things.18:50
Saviqkarni, yeah, I saw that, too (screenshot no worky), care to file a bug against Mir and phablet-tools?18:51
greybackted: let's see. Once we all start using it, we'll see quickly how practical that actually is18:52
SaviqCimi, you  mean I removed the "Infographic" type from the LightDM plugin? yeah, because UserMetrics has its own module now18:52
greybackSaviq: can I get your clarification on https://code.launchpad.net/~gerboland/unity-mir/fix-upstart-closed-apps2/+merge/218721/comments/523413 ?18:54
SaviqCimi, http://bazaar.launchpad.net/~unity-team/libusermetrics/file-based-infographics/files/head:/src/modules/Infographics/18:54
Saviqgreyback, I did "pkill -9 webapp"18:54
Saviqgreyback, so afaict it did kill just the webapp-container18:54
greybackSaviq: which didn't do anything for me :)18:54
Saviqgreyback, and shouldn't matter in any case18:54
Saviqgreyback, let me check gmail then18:55
Saviqgreyback, it might be specific to "old-style" webapps18:55
greybackSaviq: I'll try pkill again18:55
Saviqgreyback, try on jakdojade.pl - which is what I tried18:55
greybackSaviq: either way, I'll try18:55
Saviqgreyback, yeah, gmail behaves fine, jakdojade.pl does not, I wonder if it's the "--webapp" arg18:57
greybackSaviq: ok, well if I can repro, I can fix18:57
Saviqgreyback, so indeed I'm killing a child18:59
Saviqgreyback, jakdojade.pl uses webbrowser-app --webapp18:59
Saviqgreyback, while gmail uses webapp-container directly18:59
Saviqin .desktop Exec= lines18:59
greybackSaviq: grand, that helps18:59
Saviqgreyback, it might be wrong to use webbrowser-app, but that's what the SDK suggested AFAIR, so probably need to cater for that, too19:01
greybackSaviq: indeed. As it worked before (it did, right?) it must work now too. Just didn't realize gmail was different to other webapps, gmail was my go-to app for testing webapps19:02
kgunnSaviq: so, can you imagine we'd need to do any work to support push notifications? ...i woulda thot that would be backend (unity api team)work?19:02
kgunnam  i missing something19:02
Saviqwell, we never resumed anything, so it didn't work in that sense, it behaves fine except for the "can be resumed" use case19:02
Saviqkgunn, coop between us and API sometimes, but mostly API, yes19:03
Saviqkgunn, like we already added support for setting the launcher counters19:03
Saviqkgunn, https://code.launchpad.net/~ted/unity8/launcher-dbus/+merge/21591719:03
kgunnSaviq: right, like if there some data field we have missing...19:04
Saviqbut that was actually only API team by now since mzanetti exposed everything in the launcher backend before19:04
Saviqkgunn, we did discuss a "post office" concept today https://docs.google.com/a/canonical.com/drawings/d/1XpLCqGFo0_o6Sg_tDINigcxNq5VzAmbkktdobjwUvlQ/edit19:04
Saviqkgunn, to consolidate all feedback in a single API, for push and apps to use19:05
Saviqkgunn, JLenton's team will be incubating that in the push notifications backend for now19:05
Saviqted, don't peek!19:05
tedseb128, Heh19:06
tedSaviq, Heh19:06
tedseb128, Sorry19:06
Saviq;)19:06
Saviqkgunn, so as long as we expose all the entry points to the "endpoints" as indicated in that diagram, we're good19:07
kgunnSaviq: agreed, can't help but wonder/ask...whats the arch advantage of making that "post office" separate from notification backend?19:08
Saviqkgunn, you mean from push notification backend?19:08
kgunnyeah19:08
kgunnrealize there is a naming thing there...but19:08
kgunnlike app would use it too (so no longer just push)19:09
Saviqkgunn, well, we want to make local services/apps interface with the post office, too19:09
Saviqkgunn, and it's a feeling that push notifications backend should deal with just that - push notifications19:09
Saviqkgunn, for OEM replacement purposes if not anything else19:09
kgunnSaviq: got it...makes sense, push notif is more about server req/resp19:09
kgunnpost office more about getting ui elements updates19:10
Saviqkgunn, yeah, applying policy (like "don't bug me" mode) and such19:10
* kgunn remembers june is coming19:10
kgunnworries19:10
Saviqkgunn, oh yeah, post-that19:11
Saviqkgunn, that's why I say "incubate in push backend"19:11
kgunnok19:11
* kgunn worries less, until someone comes along and mandates it for june ;)19:11
Saviq;)19:12
kgunnjosharenson josharenson1 hey...which one are you :)...anyway, could you add the raw data to that sheet as well ? sorry if i didn't ask for that y'day19:12
=== josharenson1 is now known as josharenson
josharensonkgunn, yeah I'm still gathering it... itll be there by eod19:13
josharensonkgunn, the internet where I'm at is terrible and I'm downloading android/ubuntu images19:13
kgunnjosharenson: no worries19:14
Saviqkgunn, I'm thinking we should drop project bugs for unity8 (and other projects under our purview that basically only get released into Ubuntu)19:14
Saviqkgunn, it would require at least some of us to get into https://wiki.ubuntu.com/UbuntuBugControl19:14
Saviqkgunn, otherwise we lose triage rights19:14
Saviqkgunn, advantage being we wouldn't have to bug lists (unity8 and unity8 (Ubuntu))19:15
Saviqand LP managing the bugreports for us (closing on merge)19:16
Saviqs/to bug lists/two bug lists/19:16
kgunnSaviq: +119:16
kgunnits what people want i think19:16
kgunnpeople==other leaders in canonical19:16
kgunnSaviq: should we just send team mails out and then lock down the project bugs (disallowing any new entries)?19:18
Saviqkgunn, well, we need to get into BugControl first :)19:18
Saviqkgunn, I'm not there, for example19:18
* kgunn remembers there was some reason this wasn't totally clean....19:18
Saviqkgunn, yeah, you get triage rights for the whole of Ubuntu19:19
Saviqkgunn, so the barrier to entry is a bit steep19:19
kgunnSaviq: but also, if you do project milestones...like mir does, then you can't use those19:19
kgunnyour stuck with "ubuntu" as your project19:19
Saviqkgunn, oh yeah, this would only be about projects that only target Ubuntu19:19
kgunni think that's what it was19:19
Saviqkgunn, sure, once a project starts releasing into something other than Ubuntu, it makes perfect sense to maintain both19:20
Saviqkgunn, but most of our projects don't (and won't for at least some time)19:20
kgunnSaviq: should we just push for  whole team ?19:23
Saviqkgunn, over time, sure19:23
Saviqkgunn, not necessarily all at once19:23
kgunnSaviq: i seem to have rights already...have you tested yours ?19:25
Saviqkgunn, you sure? can you set the status to "triaged" for example?19:28
Saviqkgunn, or set me as assignee? or set importance?19:28
Saviqthere's a limited set of things you can do with distro bugs19:29
Saviqwhen you're not bug control19:29
kgunnSaviq: yeah...i just tested a bug that is _only_ filed against unity8(ubuntu) and i could do all those things19:30
kgunnwonder if i have it via inheritance19:30
Saviqmight be, I don't19:30
kgunn+1 to getting rights before changing anything19:31
kgunnprob should get gerry, terry, and zanetti rights as well...19:32
tedgreyback, http://bazaar.launchpad.net/~ted/upstart-app-launch/app-object/view/head:/libupstart-app-launch/ubuntu-app-object.h19:32
Saviqkgunn, I imagine mterry already is bugcontrol19:36
Saviqkgunn, what does https://launchpad.net/~ubuntu-bugcontrol say about yourself?19:37
mterryyeah I'm in that team19:37
kgunnSaviq: they even have greyback as having control...wow...they'll let anyone in19:39
Saviqkgunn, ;D19:40
=== dandrader is now known as dandrader|afk
=== beidl_ is now known as beidl
=== dandrader|afk is now known as dandrader
=== salem_ is now known as _salem
kristenbbhi, anyone here ?23:21

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