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

=== jamesh_ is now known as jamesh
EricPoeSetting up deja-dup in Ubuntu 14.4. When attempting to connect to my SSH box, deja-dup claims "SSH program unexpectedly exited" Ideas? Know where this is logged?03:33
EricPoeI can ssh into my SSH box as the same user without a problem.03:36
tsdgeosCimi: so what about that issue with carousel?08:17
tsdgeosdednick: are you doing https://code.launchpad.net/~saviq/unity8/cmake-plugin-cleanup/+merge/218171 as Saviq asked or shall i jump in?08:17
dednicktsdgeos: ah. i can do.08:18
tsdgeosdednick: tx08:19
Cimitsdgeos, I confirm is with your branch08:20
tsdgeosok08:26
tsdgeosSaviq: so https://codereview.qt-project.org/#change,85437 sent upstream, i'll work in patches to our 5.2.1 packages that use it instead of delegate range and patches to unity8 to use displayMargin instead of delegateRange08:30
tsdgeosMirv: ping08:42
Mirvtsdgeos: hangout pong08:43
dednickSaviq: that plugin cmake branch. I can't seem to run unity8 on desktop when I removed unity8 system installed folders. "Need to use QMirServerApplication"08:43
dednickSaviq: works on trunk though08:44
tsdgeosMirv: i need to do a patch for our qt 5.2 packages to bring one of the patches we have closer to upstream, what's your preferred way of me doing that change?08:45
Mirvtsdgeos: LP bug with info on what to do inside current debian/patches/08:45
tsdgeosMirv: oki, tx08:45
Mirvtsdgeos: is that ^ something that'd be nice on 5.3 RC packages, and would it help with the empty scopes? (have you tried the RC packages on device?)08:45
tsdgeosMirv: yes08:46
tsdgeosMirv: but on the 5.3 packages we'd need a somewhat different patch, i'll work on that too08:47
Mirv\o/ I'll kick that in, it'd be nice to have something demoable of 5.3 next week08:47
Mirvtsdgeos: doesn't the codereview on targeted to 5.3 work?08:47
tsdgeosMirv: 5.3 is only for "BIG AMAZING VERY URGENT" bugs now only08:48
tsdgeosso i'm targetting what we miss from upstream to 5.3.108:48
tsdgeosso plan is08:48
tsdgeosbring to our 5.2 packages what would end up in 5.3.1 if they accept my patch08:48
tsdgeosand then add to our 5.3 packages my small patch08:48
tsdgeosso it's all in sync08:49
Mirvtsdgeos: yeah, I just mean that if I add https://codereview.qt-project.org/#change,85437 to 5.3 RC is that joy?08:49
tsdgeosand we can move unity8 to and from our 5.2/5.3 packages08:49
tsdgeosMirv: no, i need to update unity8 too08:49
tsdgeosMirv: to use that new property, that's why i want to backport htat new property to 5.2 so i can also change unity808:49
Mirvah, ok right. I can build that meanwhile in the 5.3 test PPA and wait until there's something for unity8 (I can manually patch too the test package)08:49
tsdgeosand then all is nicely aligned08:49
Mirvsounds good08:50
greybackdednick: QMirServerApplication from unity-mir, unity8's main.cpp creates it. The unity-mir QML plugin depend on it (they are teh ones printing that message)08:58
Saviqtsdgeos, awesome09:00
Cimitsdgeos, fixed xvfb tests09:12
tsdgeosSaviq: so was discussing with Cimi if the shadow for the card carousel we should add it to the carousel code or to the card code09:15
tsdgeosSaviq: what's your opinion?09:15
Cimitsdgeos, in cardCarousel :)09:15
Saviqtsdgeos, since it's only ever used in carousel09:15
Saviqtsdgeos, I'd put it in CardCarousel, or Carousel itself even09:15
tsdgeosok09:16
Saviqtsdgeos, it depends on what kind of items we say we support in the Carousel09:16
CimiSaviq, carousel no, since it should stay abstract09:16
SaviqCimi, maybe yeah09:16
Saviqhaving a shadow might be abstract no less, though09:16
CimiSaviq, but in the branch I made it quite abstract inside cardCarousel09:16
Saviqif that's "part of the carousel"09:16
CimiSaviq, but it's not a proper shadow09:17
CimiSaviq, made with shaders09:17
Saviqlol ;)09:19
mhr3Saviq, when will we have new header in dash? :)09:25
karnisince we're on the subject - font color on the card overlay is not legible on some backgrounds. even if someone points I may have written that code (yes, I may have), we have to improve it. often I see dark grey font on dark overlay. don't remember if the font color is dynamic in that widget.09:26
Saviqkarni, I think the only place where it's not readable is when the image itself is not opaque (which is the case for the "unknown" album image)09:41
Saviqmhr3, we need to look at it indeed, the only problem is you suddenly lose the ability to quickly switch between scopes (at least until we get designs and implement the bottom edge behaviour in the dash)09:42
mhr3Saviq, not really a problem with 4 scopes, and with 10+ it was a problem anyway :)09:42
Saviqmhr3, true, true09:43
Saviqmhr3, but was still easier with the swipable header09:43
Saviqis09:43
mhr3right09:43
karniSaviq: you've got mal09:43
karni*mail09:43
Saviqkarni, don't show that to designers :P09:44
karniheh :) but, do you agree we have a small problem there?09:44
Saviqkarni, that depends, I don't think we should have overlay and then summary below, though09:45
karniI think solid white in overlay mode would be much better.09:45
Saviqkarni, it is solid white, until you turn on background AFAICT09:45
karniSaviq: well. I showed a couple options to our product manager, and he liked it :D if we can't do that, the shell shouldn't allow it. (FWIW I like the looks, too)09:46
Saviqkarni, the first thing that springs to eyes is the double shape09:46
Saviqkarni, that really shouldn't happen, and yeah, I agree we should prevent that, but right now we don't have any validation in place, really09:46
karniToo bad, I liked it. I wonder if we could somehow disable the bottom part of ubuntu shape when the summary is present ;D09:48
Saviqkarni, that was the design indeed - art was only shaped at the top, with header and summary below it, within a single shape with a background09:48
karniSaviq: what did you mean when you said "until you turn on background" - which background? I'm not sending any colors09:49
karniSaviq: ah I see, very cool then09:49
Saviqkarni, well, how did you get the white background behind the whole card then?09:49
Saviqhmm do we enable t ourselves? /me looks09:50
karniSaviq: you tell me :D http://paste.ubuntu.com/7467065/09:50
karniSaviq: no idea09:50
karniIt just looked nice as it were, I didn't change colors.09:51
Saviqkarni, ok, so I think you should disable the overlay, it wouldn't look nice with the image as designed (square at the bottom) anyway I'd say09:51
Saviqkarni, you could use a mascot there probably09:52
karniSaviq: I'll send screenshots to product manager and let him know what's up09:52
karniSaviq: well, the point is, we wanted large images there :) you know, more colors and candy09:52
karniso I won't replace art for a mascot in this case09:53
Saviqkarni, well, sure, keep the large image09:53
karniSaviq: what did you mean by mascot then?09:53
karniit's optional, right?09:53
Saviqkarni, a mascot, if you had it, would break the whole white space below the image09:53
karnioh09:53
Saviqbecause indeed title + subtitle + summary just one under the other will probably not look that great09:53
karniI get your point now09:54
karniit's an RSS feed, so.. no mascot candidate :( (site icon wouldn't make sense, all posts from single source)09:55
Saviqkarni, heh, now I looked at tryCard we have exactly the same layout you've shown, just no mascot ;)09:56
Saviqkarni, the weird thing is, though, that the overlay *is* white, and has correct margins09:57
karniSaviq: i noticed if there's no mascot, right margin is 0, if mascot is there, margins are correct09:58
Saviqkarni, yeah, I disabled mascot09:58
Saviqkarni, are you using latest trunk btw?09:58
karniSaviq: also, just found a bugzzie: if you have 'overlay': 'false', it still shows the overlay. you have to *remove* the 'overlay' key for the overlay to go away. would you like me to file a bug?09:58
Saviqkarni, again, not the case here09:59
karniSaviq: you'll kill me now ;P - I flashed on Mon or Tue. I'll do it now.09:59
Saviqkarni, yeah, the whole card mechanism got reworked since then09:59
karniSaviq: ok, sorry to take your time. I'll re-check that now.09:59
* karni takes note to go back to re-flashing at least each morning10:00
=== dandrader is now known as dandrader|afk
Saviqkarni, no, just use trunk on your desktop for developing scopes ;)10:07
Saviqkarni, only you'd need to upgrade to utopic, but it's about time for that anyway isn't it!10:07
mhr3Saviq, noticing there's more flicker when searching now, did you see that?10:12
Saviqmhr3, maybe because it's faster! ;)10:14
Saviqmhr3, let me see10:14
Saviqmhr3, AFAICT it's indeed because it's faster, the delegates get destroyed and recreated quicker10:16
mhr3Saviq, but it feels like there's an extra frame now10:17
mhr3Saviq, i see it when doing "setting" -> "settings" and back10:17
Saviqmhr3, well, you give us a complete new set of models10:18
mhr3Saviq, but i always did :)10:19
Saviqmhr3, maybe it's because the mascot goes away and back in only when loaded10:19
Saviqmhr3, right, we're lazy-loading the shapes now10:19
Saviqmhr3, so the text shows up before the shape does10:19
mhr3ok, that explains that10:19
Saviqmhr3, although we should've catered for it10:19
Saviqor maybe not... we don't know whether the mascot is going to be there or not..10:20
* mhr3 waits for Saviq to mention diffs again10:20
Saviqoh right!10:20
Saviqif you didn't reset all our models, all would be fine!10:20
Saviqmhr3, I was thinking about the departments...10:21
=== dandrader|afk is now known as dandrader
Saviqarrrgh10:21
Saviqmaybe not10:21
Saviq;(10:21
Saviqwe had a plan! and they broke it :/10:22
mhr3Saviq, i'll get to properly think about them once i finish this docs crap10:25
Saviqmhr3, pfft!10:26
Saviq/food10:26
Saviqmzanetti, btw, what's the status of silo 005? any ETA?10:31
Saviq/reallyfood10:31
mzanettigreyback: ^10:31
greybackSaviq: working on a fix for a bug there10:32
mzanettiSaviq: well, still that one issue in unity-mir to fix, although I'm not sure any more if its really an issue of that branch. I think I saw the same with the promoted image yesterday night10:32
mzanettibut it was really late :D10:32
Saviqgreyback, mzanetti, just asking whether I should hijack the silo and land other changes in there, or maybe land other changes separately, or wait?10:32
* greyback doesn't have an opinion10:33
Saviqok, food first, then let's see how we are10:34
Saviqlast unity8 promotion took 16h :(, we need to fix that10:34
mzanettiok. I'll test in the meantime where the issue is introduced10:34
mzanettigreyback: Saviq: ok... launching something via url dispatcher seems to be broken in the latest images already10:44
greybackmzanetti: I can't /really/ land my changes until that's fixed tho. We can't reliably test10:45
mzanettiyeah. I agree10:45
mzanettigreyback: so you're saying you want to fix that in the same branch, right?10:46
mzanettigreyback: assuming its indeed an issue in unity-mir10:46
greybackmzanetti: unity-mir hasn't changed in some time afaik10:46
mzanettiok10:47
greybacklast change was >2 weeks ago, so I doubt unity-mir is to blame (this time)10:47
mzanettigreyback: any easy to verify that?10:47
mzanetti+way10:48
greybackmzanetti: am just reflashing to repro it now10:48
=== MacSlow is now known as MacSlow|lunch
greybackmzanetti: just flashed phone, it opened settings from the indicators fine11:00
mzanettigreyback: really...11:01
mzanettiI tried with the indicators and also launching the browser from tagger after scanning a barcode11:01
mzanettigreyback: which image did you flash?11:01
mzanettiwhich channel11:01
karniSaviq: yes, it's about time to upgrade :)11:02
greybackmzanetti: devel-proposed11:02
greybackimage 3011:02
mzanettigreyback: strange... same here. but I really can't open settings from the indicators11:03
mzanettithink there might be something in the home directory that breaks it?11:03
greybackmzanetti: any REJECTED messages in ~/.cache/upstart/unity8.log?11:03
mzanettigreyback: nope, not a single message printed11:04
greybackmzanetti: that's also not right...11:05
mzanettigreyback: well, not a single line when tapping the entry in the indicators11:05
mzanettiunity8 does print other stuff11:05
greybackah ok11:05
mzanettiI can try to wipe it completely...11:06
tsdgeosMirv: Saviq: any idea how we get a silo with https://bugs.launchpad.net/ubuntu/+source/qtdeclarative-opensource-src/+bug/1319777 ?11:06
ubot5Ubuntu bug 1319777 in qtdeclarative-opensource-src (Ubuntu) "Remove ItemViews delegateRange patch and add one that brings us closer to upstream feature for 5.3" [Undecided,New]11:06
greybackmzanetti: there are url dispatcher logs in that directory too11:06
mzanettigreyback: ah, that's interesting: ** (process:1431): WARNING **: Unable to parse SQL to find url11:07
mzanettiok, /me wipes the phone completely11:09
Cimitsdgeos, is it possible to access model rules without using listview, gridview etc etc?11:09
mzanettiCimi: repeater11:10
Cimimzanetti, but I need one of those, right?11:10
Cimimzanetti, I mean model[index] or so won't work?11:10
mzanettiCimi: usually we add get() methods to the models11:10
mzanettiCimi: but if the model doesn't have them no, you can't access it without a Repeater or *View11:11
Cimimzanetti, maybe we can add it11:12
Cimipete-woods, can we add get methods to the infographics model?11:12
mzanettiif the items are QObjects, make a "MyItem* get(int index) const"11:13
mzanettiotherwise I guess "QVariant get(int index, int role) const" would do11:13
pete-woodsCimi: okay, sure, gimme an example of this pattern and I'll add those methods11:14
tsdgeosi hate quilt ^_^11:14
tsdgeoswell debian packaging in general11:15
* tsdgeos just compiles and copies stuff over11:15
Cimimzanetti, can you guide pete-woods on adding those methods?11:15
mzanettipete-woods: if the items are QObjects, make a "MyItem* get(int index) const", otherwise I guess "QVariant get(int index, int role) const" would do11:15
pete-woodsmzanetti: okay, really that simple then11:16
mzanettipete-woods: yeah, just make them Q_INVOKABLE and that's it11:16
pete-woodsmzanetti: cool, thanks, I'll get on that11:16
mzanettipete-woods: http://bazaar.launchpad.net/~unity-team/unity-api/trunk/view/head:/include/unity/shell/launcher/LauncherModelInterface.h11:17
mzanettipete-woods: line 10811:17
Cimipete-woods, thank you, that way I can play with the interface more freely11:18
Saviqtsdgeos, the usual, we talk to Mirv11:18
mzanettipete-woods: that's an example where the items are QObjects (which means QML can just use it like this: Model.get(5).roleName11:18
pete-woodsmzanetti: cool, thanks, in my example they are just going to be QStrings11:18
mzanettigreyback: yep. after wiping the device it works again11:20
* mzanetti adds back silo 511:20
Saviqtsdgeos, we'd need the unity8 patch with it, though11:20
tsdgeosSaviq: sure, working on that11:20
Saviqtsdgeos, Q: we're not creating any delegates outside of the view (like currentBuffer), do we?11:21
Saviqerm11:21
SaviqcacheBuffer11:21
tsdgeosSaviq: tried to compile the thing properly but gave up and i'm just copying the .so :D11:21
tsdgeosSaviq: cachebuffer is still taken into account11:21
Saviqtsdgeos, I mean with delegate ranges it isn't is it?11:21
tsdgeosit's added at a latter stage11:21
Saviqtsdgeos, ah11:21
tsdgeosit should11:22
Saviqtsdgeos, but we're not setting it in our views are we?11:22
Saviqah well, it's set to like 320 by default?11:22
tsdgeosthere's the default value there i understand11:22
Saviqright11:22
Saviqok then11:22
tsdgeoswhich is not really helpful with high dpi11:22
Mirvtsdgeos: maybe it'd be easiest to get tested if I'd land it together with qtpim that's also being planned11:23
tsdgeosMirv: maybe :D11:25
Mirvtsdgeos: so what do I call the patch "Q", what do I disable/remove from current patches etc... or would you rather do a merge request towards  lp:~kubuntu-packagers/kubuntu-packaging/qtdeclarative-opensource-src while at it?11:25
tsdgeosMirv: as the bug says, you remove qtquick_delegate_creation_range_itemviews.patch and add Q11:25
Mirvtsdgeos: and the "QQuickItemView-QQuickPathView-Fix-creation-of-delega.patch" stays as is?11:26
tsdgeosyou can call it qtquick_displayMargin.patch11:26
tsdgeosMirv: that's unrelated yes11:27
Mirvok, that was the only uncertainty then. and yes, it seems to apply.11:27
tsdgeos:)11:29
Mirvtsdgeos: ok, aiming https://code.launchpad.net/~timo-jyrinki/kubuntu-packaging/qtdeclarative_update_delegate_patch_lp1319777/+merge/219686 towards the silo (when I get one) so that I could then test qtpim+qtdeclarative at the same test run11:30
tsdgeosMirv: cool :)11:30
tsdgeosi'm on the unity8 side11:30
tsdgeosMirv: will give you a unity8 branch asap11:30
Mirvtsdgeos: with 5.2, there's however no need on simultaneous unity8 update, is there?11:30
tsdgeosMirv: yes there is11:31
Mirvtsdgeos: oh, ok, so adding _that_ to the landing too then11:31
Mirvand marking it's not yet ready actually too11:31
mzanettiSaviq: greyback: rebuilding packages in silo 5. now that url-dispatcher works again I'm confident we can get this through soon11:31
tsdgeosMirv: untiy8 is using the "old" property, since we're switching to the new "upstream one" we need to update unity8 code too11:31
Mirvcool, thanks albert11:31
mardygreyback: hi! Do I need to add a changelog entry to https://code.launchpad.net/~mardy/unity-mir/signonui-with-oxide/+merge/216845, since there is no bug linked?11:32
MirvI added "here-be-unity8" as a placeholder to the MP field11:32
tsdgeosthank you :)11:32
greybackmardy: I have no idea. CI train generates the changelog automatically, no? But from what, I don't know11:36
mardygreyback: OK, I'll try leaving it without one, we'll see11:36
greybackmardy: yep11:37
asacsil2100: ^^ you know how the merge back changelog is generated? guess something for our non-existing FAQ :P11:38
Saviqmzanetti, ok, will wait, thankx11:38
sil2100mardy, greyback: so, no need for a changelog entry - citrain generates changelog entries from commit messages :)11:39
sil2100mardy, greyback: so as long as you have commit messages that sound alright, everything should be fine11:39
greybacksil2100: good to know, thanks11:42
=== alan_g is now known as alan_g|lunch
=== dednick is now known as dednick|lunch
mardysil2100: thanks11:43
* greyback goes to reboot router11:53
=== _salem is now known as salem_
=== MacSlow|lunch is now known as MacSlow
mhr3tsdgeos, pushed the docs to -api, pls check12:08
Saviqand now I need to plough through it :P12:15
=== jhodapp|afk is now known as jhodapp
=== greyback is now known as greyback|food
=== dandrader is now known as dandrader|lunch
=== tedg is now known as ted
=== alan_g|lunch is now known as alan_g
=== ted is now known as tedg
tsdgeosSaviq: you doing it?13:03
kgunnSaviq: tsdgeos ...a lot of good feedback on the perf improvement, kudos to you guys13:03
Saviqtsdgeos, the review?13:03
tsdgeos:)13:03
Saviqtsdgeos, gotta do something else first13:03
tsdgeosSaviq: mhr3's docs thing13:03
Saviqtsdgeos, I think this just needs a review now13:03
tsdgeosoki13:03
Saviqtsdgeos, so unless you can make mzanetti review it, I'm it13:03
tsdgeoshe he13:03
mzanettireview what?13:04
mzanettitsdgeos: ^13:04
tsdgeosmzanetti: https://code.launchpad.net/~unity-team/unity-api/unity-shell-scopes-api/+merge/21937913:04
mzanettithat's long :D13:05
mzanettiok, I guess I can review that13:05
=== greyback|food is now known as greyback
kgunnlooks like app shutdown fix is ready, needs another set of eyes for a double check...13:18
kgunnhttps://code.launchpad.net/~gerboland/unity-mir/fix-upstart-closed-apps2/+merge/21872113:18
=== dandrader|lunch is now known as dandrader
=== pete-woods is now known as pete-woods-lunch
=== boiko_ is now known as boiko
Cimidamn allergy14:11
=== mhr3_ is now known as mhr3
Saviqdednick, standup14:30
=== alan_g is now known as alan_g|tea
=== alan_g|tea is now known as alan_g
tsdgeosMirv: can you add https://code.launchpad.net/~aacid/unity8/useDisplayMargin/+merge/219709 to that new silo for Qt?14:57
Saviqdednick, fixed cmake cleanup14:57
Saviqgreyback, FWIW, if only webapps got exec()'ed and not forked, we wouldn't have this problem...15:08
Saviqgreyback, so maybe that'd be a better thing to fix?15:08
greybackSaviq: true, but we do, so gotta deal with it15:08
Saviqgreyback, well, we can make sure that webapps don't fork15:09
Saviqgreyback, I mean that the main process needlessly changes PID, that's the main problem right?15:09
Saviqgreyback, which suggests webbrowser forks, and exits just after15:09
greybackSaviq: actually I don't think that would fix the issue. It's mainly that unity-mir cannot tell that a session, which comes from a process spawned by the webprocess, is part of that webprocess15:09
Saviqgreyback, well, browser and gmail behaved fine?15:10
greybackSaviq: instead it hacks around it by simply accepting any session that is a QtWebProcess.15:10
Saviqgreyback, yeah, but... once the container is forked, doesn't the main PID of the upstart app change? does upstart know about it?15:11
greybackSaviq: I'm not certain tbh15:12
Saviqgreyback, empirically, gmail was special because its main process was "webapp-container" directly15:12
greybackSaviq: but yes, the PID change does make life harder15:12
Saviqgreyback, so for now we need to enforce that PID doesn't change15:12
greybackbut I don't think fixing it will make all unity-mir's problems go away15:13
Saviqgreyback, same we did when people had bash scripts, made them exec15:13
Saviqgreyback, but will let us not have to carry this workaround for now15:13
greybackSaviq: right15:13
Saviqgreyback, when the fact that they fork and not exec is unnecessary in any case15:13
* Saviq finds out what upstart things about it15:13
Saviq*thinks15:14
Saviqgreyback, hmm the PID doesn't change apparently:15:17
Saviq$ start application-click APP_ID=net.sawicz.michal.jakdojade_example_0.1.115:17
Saviqapplication-click (net.sawicz.michal.jakdojade_example_0.1.1) start/running, process 1219915:17
Saviqphablet  12199 87.2  2.1 250396 40504 ?        Ssl  17:16   0:03 webapp-container --webapp --enable-back-forward...15:17
Saviqunless it changes quickly enough that upstart notices15:17
tsdgeosSaviq: in https://code.launchpad.net/~aacid/unity8/use-unity-api/+merge/219222 you mention "Needs to depend on libunity-api-dev (>= 7.81)"15:18
tsdgeosSaviq: shall i do that in https://code.launchpad.net/~unity-team/unity-api/unity-shell-scopes-api/+merge/219379 ?15:19
tsdgeosah you commented15:19
tsdgeos:D15:19
Saviqtsdgeos, ;)15:19
Saviqgreyback, I just worry about WTH does webbrowser and gmail behave fine15:19
greybackSaviq: I'm more than a little confused by that tbh15:20
Saviqgreyback, could you see that it indeed starts two sessions (or more, actually)?15:20
mhr3Saviq, eek! forking good, exec bad!15:24
=== gatox is now known as gatox_lunch
Saviqmhr3, not yet15:24
mhr3always! :P15:24
Saviqmhr3, not until we can group sessions from multiple processes under the same app15:25
Saviqmhr3, and not until they can't escape it, for that matter :P15:25
greybackSaviq: starting Jakdojade, 3 sessions are created. Not sure if they're unique. Need more time to dig15:25
mhr3Saviq, cause we have thousands of apps doing that now? :)15:25
Saviqmhr3, "doing what"?15:25
mhr3Saviq, forking after launch15:26
Saviqmhr3, not on phone we don't15:26
Saviqgreyback, ktx15:27
Saviqmhr3, except for browser/webapp/something, which behave somewhat weirdly15:27
mhr3Saviq, so we want to be able to group them properly, while that isn't really necessary at this point, but throwing away the massive speed improvements that fork brings15:27
Saviqmhr3, I'm not sure what you're here after ;)15:27
mhr3priorities15:28
Saviqmhr3, you mean like launch boosting and such?15:28
mhr3yes15:28
Saviqgreyback, btw, it's probably a lower prio task, if you tell me you don't want to spend time on it now, don't15:28
Saviqmhr3, we can't do that right now either ;)15:28
mhr3doesn't mean we should close those doors15:29
Saviqmhr3, I think you're misunderstanding :)15:29
Saviqmhr3, it's a current limitation of unity-mir that it only expects a single session under the main upstarted PID15:29
Saviqmhr3, launch boosters will obviously need something special anyway15:30
Saviqmhr3, to inject that PID into an upstart / systemd job15:30
Saviqmhr3, but I'm not talking about "closing doors"15:30
mhr3indeed, i just saw the exec == good and my alarms went off15:30
Saviqmhr3, I'm just talking about apps (which are all currently exec'ed anyway - until we get a launch booster at least)15:31
Saviqmhr3, that break out of app management because they fork and upstart / unity-mir loses track15:31
Saviqmhr3, where there is no benefit to them forking in that case15:32
Saviqmhr3, 'cause the process from which they forked was _just_ exec'd a cycle agi15:32
Saviqago15:32
mhr3Saviq, just saying we shouldn't care, let them escape if we can launch them fast for rtm :)15:32
Saviqmhr3, we can't let them escape, 'cause we lose track of them15:32
mhr3if they launch fast... meh15:33
mhr3priorities! :P15:33
elopiokgunn: are you coming to the meeting?15:33
Saviqmhr3, we lose track of them, do you get it? DO YOU GET IT? ;)15:34
Cimipete-woods-lunch, there are conflicts, launchpad says15:34
Saviqmhr3, unless upstart tells us that this forked PID is app foo15:34
Cimipete-woods-lunch, can you push it to the silo after that?15:34
Saviqmhr3, we'll reject it15:34
mhr3Saviq, i have a branch for that ;)15:34
Saviqmhr3, we have holes poked to allow web renderer processes, signon UI and maliit15:35
mhr3Saviq, http://bazaar.launchpad.net/~mhr3/unity-mir/authenticate-via-apparmor/revision/21615:35
Saviqmhr3, right, sure, that'd be one way15:36
mhr3Saviq, any other concerns? ;)15:37
Saviqmhr3, the HACK part in there :P15:37
mhr3it allows apps to launch fast!15:38
mhr3priorities! :D15:38
Saviqmhr3, UPSTART15:38
mhr3will go away15:38
mhr3finally!15:38
Saviqmhr3, so will support for non-upstarted apps :P15:38
Saviqanyway, otp15:39
mhr3Saviq, you just need to see the magic where apps actually launch instantly and you'll close one eye ;)15:40
Saviqmhr3, port us to systemd, inject those PIDs into systemd jobs and we'll be fine!15:42
=== pete-woods-lunch is now known as pete-woods
tsdgeosmterry: if you remove the qscreen include i'll approve https://code.launchpad.net/~mterry/unity8/split/+merge/21314915:45
mterrytsdgeos, oh duh thanks15:46
pete-woodsCimi: fixed the conflict, I wish people would stop pushing things to distro without going through the CI train15:47
mhr3Saviq, and maybe you'll close both eyes :)15:48
greybackSaviq: somehow oxide based webbrowser only opens 1 Mir session. Whereas QWebProcess-based browsers open multiple sessions15:48
Saviqgreyback, ah so that's the difference...15:49
Saviqgreyback, ok, let's just get this in15:49
greybackSaviq: +115:49
mterrytsdgeos, done15:49
tsdgeostx15:50
=== jhodapp is now known as jhodapp|lunch
pete-woodsSaviq: could you trigger a rebuild on the libusermetrics silo when you get a moment?15:51
CimiSaviq, I cannot use crossfadeImage for infographics, will implement a new animation15:53
CimiSaviq, just tested with crossfadeimage, if I have image A and B15:53
Cimiwhat happens between the transition to image B15:53
CimiA.opacity is always 1.015:53
Cimithen B from 0 becomes 115:54
Cimiwhen B.opacity is 1, A.opacity turns 015:54
Cimiwhat we want here is a crossfade where A.opacity goes from 1 to 0 and B.opacity from 0 to 1 at the same time15:55
CimiI am not sure we want to add this to the SDK15:55
tsdgeosCimi: can you please open a bug about the thing you foudn in the carousel so i remember for monday?15:55
Cimitsdgeos, sure15:55
Cimitsdgeos, assigning you?15:56
Cimior subscribing?15:56
tsdgeosCimi: assign me15:56
Saviqgreyback, and QWebProcess-based things will be phased out in time15:57
Saviqgreyback, so yeah, let's not put work in there any more15:57
greybackSaviq: so I expect15:57
Saviqpete-woods, just camera-app?15:57
pete-woodsSaviq: it's just libusermetrics, the camera app shouldn't have changed15:58
Saviqpete-woods, there's a newer version in distro, so I'll kick both then15:58
Cimitsdgeos, https://bugs.launchpad.net/unity8/+bug/131990715:58
ubot5Ubuntu bug 1319907 in Unity 8 "Carousel overlay not always shown" [Undecided,Confirmed]15:58
pete-woodsSaviq: I've just pushed an updated version of the camera app, so at least the versions should match15:58
Saviqpete-woods, it would merge trunk anyway15:59
Saviqpete-woods, ah but yeah, if there was some kind of stuff like that15:59
Saviqpete-woods, it probably would've complained15:59
SaviqCimi, I think the "style" of the crossfade should be an option15:59
SaviqCimi, but a crossfade where they both run at the same time goes to background (black) in the middle, that's why it was made this way16:00
SaviqCimi, basically, it doesn't work for opaque images16:00
SaviqCimi, in case of infographics we might indeed need them to crossfade like you said16:01
SaviqCimi, but I think the same component (CrossFadeImage) should be used, just configured differently16:01
Saviqgreyback, ok, I'll look at the branch first thing tomorrow16:02
mhr3mhall119, ping?16:11
Saviqmzanetti, you can invoke rowCount() with no arg to get... the row count ;)16:11
tsdgeosmhr3: mzanetti did some comments on the unity-api changes, i'm out until monday but if you want to comment/act on them great, otherwise i'll take them when i'm back on monday16:11
mhr3checking16:12
=== gatox_lunch is now known as gatox
mhall119mhr3: pong16:23
mhr3mhall119, haven't heard back from you about the docs, and they don't seem to be updated, anything still missing?16:23
mhall119mhr3: no, yesterday I had to update my scripts to get the new UITK docs uploaded because they were broken16:24
mhr3mhall119, thomas would like to see the mention of server scopes removed from the current docs (which happened in our trunk and is released in U)16:26
mhr3any chance to get that update to duc?16:26
mhall119mhr3: removed from the 14.04 docs, or just not included in 14.10 docs?16:28
mhr3mhall119, both16:29
mhall119was it in 14.04?16:29
mhall119http://developer.ubuntu.com/api/scopes/sdk-14.04/16:29
mhr3yes16:29
mhr3and it shouldn't16:29
mhall119then shouldn't it be in the docs for 14.04?16:29
mhall119heh, answered before I could ask16:30
mzanettiSaviq: right... still I think a count property would be better in that case. if not for technical reasons, at least to align it with the rest of this and the other apis16:30
mhr3mhall119, basically, just update the docs on duc from the current U pkg16:31
mhr3both 14.04 and 14.1016:31
mhr3we'll deal with sru-ing stuff, and figuring out whether it's really necessary16:32
=== alan_g is now known as alan_g|EOD
mhall119mhr3: you want the 14.10 docs uploaded into the 14.04 docs space?16:36
mhr3yes16:37
mhr3mhall119, ^16:37
mhall119that feels wrong16:44
mhall119I'll do it, it's your stuff and you guys have to support it, but it feels wrong16:44
Saviqmhr3, can we get rid of formFactor from scopes API?16:51
kgunnmhall119: btw, mir had the same problem16:55
kgunnwrt loading up "new" docs16:55
kgunnjust sharing..16:56
kgunngreyback: Saviq in your opinions, what are we lacking in terms of being able to upstream our qtubuntu (qt-mir) backend? as i recall there was some interest in this from upstream16:57
Saviqkgunn, I believe greyback referred to that idea in a "NOOOOOO not yet, it's ugly!" manner16:57
Saviqkgunn, but QtCS will be a good place to spark a discussion on this16:58
greybackkgunn: it needs significant cleaning up (still lots of SF code in there). We also need to talk with them to see their opinion on the ubuntu specific bits16:58
mhr3Saviq, hm, i'm unaware of any scopes actually using it, so theoretically we could, but it is in the api16:58
kgunnok, something to take up post Qt comp16:58
mhr3Saviq, why don't you want to pass it?16:58
Saviqmhr3, https://docs.google.com/a/canonical.com/presentation/d/1K1oV4vMc-FduKUNYYO62zPUCU5WMb74zjer8KzYzhLg/edit16:59
mhr3Saviq, formFactor: "with-keyboard"? :)16:59
Saviqmhr3, don't get me started on "with-keyboard", we just managed to get rid of the need for that in UITK today ;)17:01
mhr3Saviq, imo we should keep it, it might be interesting for some (mostly server) scopes, but we can say in docs that whatever is there shouldn't matter for most use cases17:01
mhall119mhr3: in a CC meeting now,  but that's on my list to do when it's over17:01
mhr3mhall119, thx17:01
Saviqmhr3, well, we won't *know* the form factor17:01
mhr3Saviq, hm... ok that sucks17:02
Saviqmhr3, so sending it will be rather difficult17:02
Saviqmhr3, basically because it's an abstract and stupid classification17:02
mhall119mhr3: where are server scopes mentioned currently?17:02
mhr3Saviq, maybe we can still send something? that is perhaps more ambigous17:02
greybackkgunn: re silo6 and the platform-api update, I've installed it and it seems to work fine. Which surprises me, as I was expecting the ABI change to break stuff! For safety a qtubuntu rebuild might be needed, just in case.17:02
mhr3mhall119, in the big tutorial page17:03
Saviqmhr3, what we *could* send is the 'usage scenario' as described below, and we'll probably need to send more (like device name or something)17:03
mhall119ah, the wordpress side of things, ok17:03
Saviq(below in the slide deck)17:03
mhr3will check the doc properly tomorrow, need to run now17:04
kgunngreyback: ack i'll add that17:07
=== beidl_ is now known as beidl
code_glitchhey there! is there anyone who knows a thing or two about unity/compiz with a little time available?17:52
dobeyanyone around that knows anythinga bout unity-scope-tool?18:17
dobeybecause i'm getting the error aobut Runtime.ini and i don't know how to get around it :(18:18
code_glitchnope, im looking for someone who knows something about metacity/compiz/xlib and unity myself...18:18
=== jhodapp|lunch is now known as jhodapp
=== tedg is now known as ted
kgunngreyback: i'm gonna land the papi resize change....looks good...i found a couple of bugs but seem unrelated18:59
kgunn(e.g. copy/paste prompt has gone completely awol now)18:59
greybackkgunn: ok, thanks!19:00
greybackted: ping19:34
tedgreyback, Howdy19:35
greybackted: hey, quick question: when UAL starts an app, it sets some vars in its environment. Where is the file that sets those?19:35
tedgreyback, http://bazaar.launchpad.net/~indicator-applet-developers/upstart-app-launch/trunk.14.04/view/head:/helpers.c#L49319:36
tedgreyback, Those are the big ones, but there's also the path ones, is that what you're looking for?19:37
greybackted: hmm, not what I need. Any idea what would set the MIR_SOCKET?19:37
SaviqAlbertA, "USC to use default values when no session is active." I don't think that's correct...19:38
tedgreyback, I'm pretty sure the Unity8 upstart job sets it.19:38
Saviqgreyback, ted, it does19:38
AlbertASaviq: what should it use?19:38
greybackted: Saviq got it, thanks19:38
Saviqgreyback, ted, well it sets it to unity8's socket19:38
Saviqfor clients19:38
Saviqgreyback, ted, it's set to u-s-c's client socket by... something else19:38
SaviqAlbertA, sure, default values yes, but not "if no session is active", but "if no session was ever active" instead19:39
tedSaviq, I believe that's the touch-session init script19:39
Saviqted, probable yes19:39
AlbertASaviq: ok19:39
SaviqAlbertA, and greeter should not have its own values19:39
tedBut considering greyback's asking about applications, I bet he wants Unity8's socket :-)19:39
apwlarsu, hey ... you seem to have touched notify-osd recently, just found a bug wherein appended notifications go completely missing (bug #1319244) I have a fix for it, but no idea how one goes about handling things in the CI magic19:40
ubot5bug 1319244 in linux (Ubuntu) "iostat: Cannot open /proc/stat: Cannot allocate memory" [Medium,Confirmed] https://launchpad.net/bugs/131924419:40
greybackted: Saviq: no I'm good, I know where to go from here, thanks!19:40
* greyback had guessed it was set by some user login script19:40
apwlarsu, damn that is bug #131998319:40
ubot5bug 1319983 in notify-osd (Ubuntu) "notify-osd loses pending bubble if append occurs when bubble is not yet visible" [Undecided,New] https://launchpad.net/bugs/131998319:40
SaviqAlbertA, flow would be, IMO: phone boots up, defaults are applied, but as soon as you make a session active, values from the session are read and stored in u-s-c until they're overwritten by that, or any other session19:40
AlbertASaviq: ok make sense19:40
SaviqAlbertA, I didn't yet reply to your email, but re: power button - I don't think there needs to be any special communication between shell and usc19:42
SaviqAlbertA, basically the shell would react after 2s, usc potentially after 5s, lower levels (kernel or hardware) after 10s19:42
SaviqAlbertA, this way, after 2s the power off dialog would be displayed in shell or greeter, after 5s usc would issue a clean shutdown, after 10s hardware would shut down forcefully19:43
SaviqAlbertA, no need for comms between layers19:44
AlbertASaviq: I see, ok19:44
SaviqAlbertA, and in case shell, usc don't respond, we still have a fallback down under19:44
AlbertASaviq: sure it makes sense. So I'll remove the dbus signal from USC and change the threshold19:45
SaviqAlbertA, and actually, it's not that it should check the timer on _UP19:45
SaviqAlbertA, but as soon as the timer triggers it should react19:45
AlbertASaviq: yeah that's how I have it implemented19:45
SaviqAlbertA, so that user sees a reaction19:45
SaviqAlbertA, ok, I described it wrong in the email19:45
darklight_ChrisTownsend, have you had a chance to give it a look ?20:32
kgunnSaviq: in case you're still on, is unity8 AP having issues on the latest image ?20:34
kgunngreyback: ^ i might have spoke too soon about landing...20:35
kgunngoing to reflash and test the straight image20:35
ChrisTownsenddarklight_: No, not yet.  It's next on my queue, but I'm working on another issue where windows sometimes jump around workspaces when disconnecting/connecting an external monitor.20:35
ChrisTownsenddarklight_: Once I either fix that or get tired of working on it, I'm gonna look at the reset settings issue.20:35
darklight_ChrisTownsend,  oh i heard of that, just so that I don't bother you too much what's the right bug where I can follow the progress ?20:36
ChrisTownsenddarklight_: The reset setting issue?20:37
Saviqkgunn, I've seen a few flaky tests recently in our CI20:37
Saviqkgunn, last I landed was over a day ago20:38
* Saviq flashes20:38
darklight_ChrisTownsend, yep20:38
ChrisTownsenddarklight_: Hmm, I know there a few different bugs entered for this issue.  I'm going to triage those bugs and dup any I find.  Honestly, I don't know which bug will be The One yet.20:40
darklight_ChrisTownsend, ok, I'll bother you every once in a while till then :P20:41
ChrisTownsenddarklight_: That's fine:)  Hopefully I will have the bugs straightened out by the next time you poke me.20:42
Saviqkgunn, smoke is green for unity8 as well, and they didn't report any flakiness there20:42
SaviqChrisTownsend, on windows jumping around workspaces... I noticed that if I get an urgent on launcher and press it, the window (sometimes?) gets moved to my external screen20:47
SaviqChrisTownsend, I'm not sure it's 100%, but it's happened often enough that I noticed20:48
ChrisTownsendSaviq: Is the window maximized?20:48
SaviqChrisTownsend, yes, was about to mention20:48
ChrisTownsendSaviq: Yeah, many, many issues with maximized windows in Compiz:-(20:48
SaviqChrisTownsend, :|20:49
Saviqlet20:49
Saviq's fix that in unity8 then!20:49
ChrisTownsendSaviq: Indeed!20:49
robrumterry, hey, had to rebuild telephony-service in silo 2 again due to another landing of that component: https://ci-train.ubuntu.com/job/landing-002-1-build/64/console please land this silo soon ;-)20:51
mterryrobru, naw, don't want to20:51
mterry:)20:51
robru:-P20:51
Saviqmterry, you actually (almost) got ACK from Albert on this didn't you? ;)20:54
Saviqmterry, just to confirm: AP tests will pass in there will they?20:54
mterrySaviq, yeah for the root unity8 branch.  But there are fixes elsewhere now I'm waiting on (fallout from upstart fiddling with lightdm)20:54
mterrySaviq, AP test pass yeah20:54
Saviqmterry, cool, so sounds like we could get it before Malta?20:55
mhall119mhr3: stil around?20:55
mterrySaviq, this is the problem with this branch -- the longer it stays out, the more other pieces of the system break it  :)20:55
mterrySaviq, I desperately want that20:55
Saviqmterry, I can imagine20:55
mterrySaviq, I don't want any conversations in Malta to have the phrase "once split greeter lands, this will be different"20:55
Saviq:)20:55
Saviqmterry, let's talk tomorrow on where others can help20:56
Saviqmterry, and let's get this done early next week20:56
mterryk20:56
mhr3mhall119, i am now20:57
mhall119mhr3: I don't see the word "server" on the scopes tutorial, but I do see it on the overview page20:58
mhall119http://developer.ubuntu.com/scopes/overview/20:58
mhall119is that what you meant?20:58
mhall119mhr3: also can you check the API docs I just uploaded to http://91.189.92.89/api/scopes/sdk-14.10/20:59
mhr3mhall119, i meant here http://developer.ubuntu.com/api/scopes/sdk-14.04/index/20:59
mhall119ah, in that case can you confirm that http://91.189.92.89/api/scopes/sdk-14.10/index/ is how you want it?20:59
mhr3mhall119, ehm, nope looks old20:59
mhall119oh, you know what, I apt-get downloaded on my box, which is trusty, you want the ones from utopic don'tyou21:00
mhr3indeed21:01
mhall119hmmm, how do I easily get a package from another release21:02
mhr3good question21:02
mhr3mhall119, https://launchpad.net/ubuntu/+archive/primary/+files/libunity-scopes-doc_0.4.5%2B14.10.20140513-0ubuntu1_all.deb21:03
mhr3mhall119, looks like you could build that url with the output from rmadison21:11
larsuapw: thanks! I'll have a look at it tomorrow. Usually we prepare merge request of a bzr branch - but don't worry about that if you don't plan to do that often21:27
mhall119mhr3: please check http://91.189.92.89/api/scopes/sdk-14.10/ again21:29
mhr3mhall119, yep, that looks good21:32
Saviqkgunn, just ran latest proposed test suite, all OK21:35
kgunnSaviq: damn...it's working here for me too....21:37
kgunnmmm21:37
kgunngonna try that package once more...but not looking good21:37
apwlarsu, sounds good thanks, i have a couple of test sequences i can use to confirm when you have an update ...21:44
=== beidl_ is now known as beidl
larsuapw: can you post/attach them to the bug as well please?21:44
larsuwe'll need them if we want to SRU it (not sure if that's worth it yet)21:44
mhall119mhr3: please verify http://developer.ubuntu.com/api/scopes/sdk-14.10/ too21:46
apwlarsu, done ... the bug is pretty significant if you use x-canonical-append:true, which most of the IM integrations do21:46
apwlarsu, as you can completely lose whole sets of notifications21:47
larsuapw: ah okay. Likley SRU worthy then21:52
larsuI'll update the bug for that tomorrow and get it into the right hands21:53
larsuthanks!21:53
apwlarsu, great thanks, spent a very confused day trying to make use of it, and it took me a while to realise i wasn't doing it wrong ...21:56
=== jhodapp is now known as jhodapp|afk
=== broder_ is now known as broder
mhr3mhall119, looks fine as well23:20

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