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

=== Wellark is now known as 64MABGBII
=== hggdh_ is now known as hggdh
=== karni is now known as Guest6481
=== Wug is now known as Wug[bang]
=== Wug[bang] is now known as wug
=== wug is now known as 13WABRQ3V
=== xnox is now known as 16WAAKRPZ
=== Daviey- is now known as Daviey
=== ubuntulog2 is now known as 16WAAK6WE
=== shiznix is now known as Guest76975
=== shiznix_ is now known as shiznix
=== 13WABRQ3V is now known as Wig
=== Wig is now known as Wug
=== Wug is now known as 13WABRQ3V
=== sam113101 is now known as YOURBESTFRIEND
=== tsdgeos is now known as 77CABBB2X
=== duflu is now known as 45PABCG24
tsdgeosyay, everyone is back!08:38
tsdgeosSaviq: we have not commited yet the network manager fix thing?09:00
Saviqtsdgeos, no, CITrain is stuck due to stowstorms09:01
Saviq*snowstorms09:01
tsdgeosok09:01
Saviqtsdgeos, but for real they were fighting a regression, so there's a strike of the CITrain crew09:01
Saviqtsdgeos, should start happening again this morning09:01
tsdgeosok09:02
tsdgeosso my rebuilds of my branches were worthless09:02
tsdgeosshould have checked earlier09:02
Saviqtsdgeos, unfortunately, yes09:02
Saviqtsdgeos, I'd have kicked them if it landed09:02
Saviqtsdgeos, karni already ACK'ed it, but I'd like you to have a look at https://code.launchpad.net/~saviq/unity8/card-tool/+merge/204093 too09:02
* tsdgeos clicks the link09:03
=== vila is now known as 14WAB1TAR
Saviqtsdgeos, unfortunately had to make it width: 0; height: 0; clip: true to hide it, otherwise it would not lay the thing out correctly09:04
tsdgeosSaviq: visible: false?09:04
Saviqtsdgeos, even worse09:04
Saviqtsdgeos, Row/Column stop working09:04
Saviqtsdgeos, as they ignore visible: false children09:04
tsdgeosguess it kind of makes sense09:04
Saviqtsdgeos, yeah, kind of does, but I think it should work when in data09:04
Saviqtsdgeos, so put a FIXME there to investigate09:05
Saviqtsdgeos, also, for tests, on first look, it feels like cheating that I compare values to the real displayed one, but OTOH that's what it's meant to do ultimately, so IMO it's better than hardcoding some expected values that we'd have to adapt every time theme/visuals change09:07
tsdgeosSaviq: the brief for09:08
tsdgeos+ type:real \brief Width to be enforced on the card in this configuration.09:08
tsdgeosseems a bit weir09:08
tsdgeosd09:08
tsdgeossince it seems you continue the description on the line below09:09
Saviqtsdgeos, hmm, yeah, and it'll show when you click "More..." in the doc, what else do you have in mind? all in \brief?09:10
tsdgeoskill brief?09:10
tsdgeos:D09:10
Saviqhmm /me tries how that would behave09:11
tsdgeosi mean brief it's supposed to be one liner, and not sure we can condense the two lines in one only09:11
tsdgeosbut this is me just being silly,09:11
tsdgeosfeel free to ignre09:11
tsdgeosSaviq: didrocks: why is https://code.launchpad.net/~saviq/unity8/move-upstart-timeout/+merge/202041 not top approved? because of the evil mako failing tests?10:01
Saviqtsdgeos, not really, rather because of lack of review from our side10:01
Saviqtsdgeos, let me kick a CI rebuild10:01
tsdgeosSaviq: want me to have a look then?10:02
Saviqtsdgeos, sure, yeah10:02
=== davmor2_ is now known as davmor2
Saviqtsdgeos, I think the test would be if you SIGSTOP'ed unity8 - how long would it still live past "stop unity8"10:02
Saviqtsdgeos, or you could SIGSEGV unity8 and "stop unity8" at the same time to see if the stack trace will be ok (but then again we know that it will, already)10:03
Saviqtsdgeos, so only really check that the package is built correctly10:04
Saviqtsdgeos, http://upstart.ubuntu.com/cookbook/#kill-timeout for reference10:04
=== ubot5` is now known as ubot5
tsdgeosCimi: any real reason for https://code.launchpad.net/~unity-team/unity8/unity8.previews_textSummary/+merge/204177 to depend on the audio preview?10:08
tsdgeosor is it just an artifact of how you created the code?10:08
Cimitsdgeos, nope10:09
Cimitsdgeos, sharing same dirs10:09
Cimitsdgeos, this branch is WIP10:09
tsdgeosok, can you mark it as such then? :.)10:10
=== jasoncwarner__ is now known as jasoncwarner
=== karni is now known as Guest56874
Saviqsil2100, hey, any word on resuming CITrain service? Has the snow been shovelled off the tracks? Or can we maybe help there?10:24
=== zequence_ is now known as zequence
tsdgeosSaviq: back to https://code.launchpad.net/~saviq/unity8/move-upstart-timeout/+merge/202041 not sure what difference should i see with and without the patch, can you repeat/elaborate?10:34
Saviqtsdgeos, it's just moving the kill timeout from unity8-autopilot package to unity810:35
tsdgeosahhh10:35
Saviqtsdgeos, upstart kills hanging processes after 5s if you tell it `stop unity8`10:35
Saviqtsdgeos, that caused exit stacktraces to be corrupted, 'cause apport didn't have the time to collect all the data10:36
Saviqtsdgeos, so "kill timeout 30" tells it to only SIGKILL after 30s past SIGTERM10:36
sil2100Saviq: hi! So, I guess today we'll resume doing landings - we observed many more failures in many different AP tests, but these don't seem to be related to application changes10:37
sil2100Saviq: more like either autopilot changes or upstart-app-launch10:37
Saviqsil2100, mhm10:37
sil2100Saviq: me and robru are looking into that, once we have an answer we'll resume everything (tm)10:37
SpaghettiCatHello, does any know why Unity chose to place min/max/close window-buttons on the left?10:52
SpaghettiCatI can't find any info on google10:53
SaviqSpaghettiCat, because when your window is maximized, the buttons go into the top bar and would conflict with the indicator icons10:54
SpaghettiCatSaviq: What are the indicator icons?10:55
SaviqSpaghettiCat, the menus on the right in your top panel10:55
SpaghettiCatOh I see, thanks for the answer10:57
SaviqSpaghettiCat, if you want, you can bring them back to the right via org.gnome.desktop.wm.preferences gsettings key, but they will stay on the left for maximized state, for the reason above10:58
SpaghettiCatNo, that will get really confusing10:59
SaviqSpaghettiCat, indeed10:59
tsdgeosSaviq: please add the new boilerplate at top of https://code.launchpad.net/~saviq/unity8/move-upstart-timeout/+merge/202041 so i can approve11:06
Saviqtsdgeos, yup11:06
Saviqtsdgeos, done11:07
MirvSaviq: tsdgeos: any time for the Qt 5.2 stuff? the screen blanking problem is at least annoying, since if one forgets to run the always on command etc one needs to reboot again since it seems impossible to unblank the screen11:39
SaviqMirv, I'm afraid we're stuck on MWC requirements11:40
SaviqMirv, also, it's really weird how Qt could influence that, when it's powerd alone that handles all of that stuff by calling ioctls directly11:41
SaviqMirv, unity8 is only informed about the current status, neither unity8, unity-mir nor Mir itself is part to any of this process11:42
MirvSaviq: oh, ok :( I was kind of assuming there is something like that going on. there are also the 5 failing unity8 tests, but let's see when QA gets automated testing going on.11:43
SaviqMirv, elopio was looking at those for us11:43
SaviqMirv, for powerd, try pinging rsalveti when he's around, I think they took over powerd from sforshee11:44
MirvSaviq: funny yes that there is a difference, I don't know what's going on. but I'll add powerd to the bug.11:44
MirvSaviq: ok, thanks for the pointer.11:45
sil2100thostr_: ping11:59
thostr_sil2100: pong11:59
tsdgeosSaviq: so katie said she wanted that when you click on one of the items of the image gallery it "maximizes" on screen, but there's no design for it (i.e. does it scretch, or fit and the rest of the screen is black, or...) so do i do what i feel better, don't do that yet or ask and idle? I'd vote for "not doing" it for the moment since it's not *critical* to have the preview and do it after we have something that shows a preview using the new components12:11
tsdgeoscomments?12:12
Saviqtsdgeos, later12:18
sil2100Mirv: boing12:22
sil2100Mirv: https://code.launchpad.net/~sil2100/kubuntu-packaging/qtbase-opensource-src_5.0.2_qapp_fix/+merge/20447112:22
Mirvsil2100: boing12:33
Saviqtsdgeos, to clarify, "later" was meant to say "we'll do the zooming later"12:43
tsdgeosyep12:51
tsdgeosgot it12:51
tsdgeosi added the MR12:51
tsdgeoslunch!12:51
=== alan_g is now known as alan_g|lunch
=== shiznix is now known as Guest12734
=== Guest56874 is now known as karni
om26er_MacSlow, hi!13:57
om26er_MacSlow, I have to test a few scenarios related to notify-osd like tapping on 'accept' button on it opening the dialer-app (or decline button). do you think its fine to put those tests in unity8 ?13:58
Saviqom26er_, can we, at that moment, consider creating a lp:ubuntu-test-suite project or so, that will hold the tests crossing multiple other projects?14:01
=== dandrader is now known as dandrader|afk
Saviqom26er_, these kind of integration tests don't really belong to any of the projects in particular, since it will span unity8, unity-notifications, dialer-app most probably14:01
om26er_Saviq, right creating a new source is a bit of work, but if we agree to that as a patch then I can work on creating a small package14:02
Saviqom26er_, it's also about ownership / maintenance, we (unity8 team) can't own / maintain all such high level tests, just because u8 is the project that "binds" it all together14:03
Saviqom26er_, not sure what you meant there14:04
tsdgeoslarsu: man you were at fosdem? i didn't see you :D14:04
tsdgeosdidrocks: and i never found time to come and greet you either14:05
tsdgeosso much stuff to do!14:05
* Saviq missed FOSDEM this year again ;?14:05
Saviq;/14:05
om26er_Saviq, I meant that if we agree that creating a new source to hold inter-app integration tests then I can work on creating a new package. I have a meeting in an hour with my team so I will ensure we figure out responsibility/maintainance there (and get it agreed from different stake holders)14:06
Saviqom26er_, great, please let me know of the outcome14:06
om26er_Yes, will do.14:07
=== fginther` is now known as fginther
Saviqom26er_, I think you guys will like that approach, too, as we'll be owning the hooks you need to do the integration tests (emulators / helpers etc.) - that I don't want to throw your way14:08
Saviqom26er_, but then you will own the actual tests14:08
Saviqom26er_, and won't need to "ask" us whether we agree to have such a thing with our code :)14:08
tsdgeosSaviq: yes, you did miss it :D14:09
Saviqtsdgeos, that wasn't a question really ;)14:09
om26er_Saviq, right, I don't mind maintaining those tests and it sounds like we are already coming to an agreement ;)14:10
tsdgeosSaviq: i know i know14:11
Saviqom26er_, violent agreements are the best kind!14:11
karniSaviq: I'll attend standups only when time allows, schedule is pretty filled here @Orlando sprint14:16
Saviqkarni, of course, don't worry14:16
=== dandrader|afk is now known as dandrader
=== 14WAB1TAR is now known as vila
=== ssweeny` is now known as ssweeny
tedgom26er_, Could you make them a directory here?  https://code.launchpad.net/~ubuntu-testcase/ubuntu-manual-tests/trunk14:20
om26er_tedg, automated tests in "manual-tests" ?14:21
om26er_;)14:21
larsutsdgeos: yes! Didn't know you were there as well :/14:24
=== boiko_ is now known as boiko
didrockstsdgeos: oh, didn't see you either… ETOOMANYPEOPLE14:47
=== Zhenech_ is now known as Zhenech
dandraderalan_g|lunch, so to add my own messages on top of the regular ones in the mir socket I've to extend ProtobufSessionCreator and ProtobufMessageProcessor14:57
dandrader?14:57
dandraderoh, wrong channel14:58
=== alan_g|lunch is now known as alan_g
alan_gdandrader: if you look in tests/acceptance-tests/test_protobuf.cpp for exampe code you'll see that you need to extend mfd::MessageProcessor and mf::ProtobufSessionCreator15:00
tsdgeosSaviq: address comments at https://code.launchpad.net/~aacid/unity8/image_gallery_preview_widget/+merge/204499 (i think :D)15:01
dandraderalan_g, ah, so the idea is to wrap ProtobufMessageProcessor, not extend it15:04
alan_gdandrader: sure, reuse by composition is alway to be preferred to reuse by inheritance15:04
dandraderalan_g, ok, sounds good15:06
dandraderSaviq, would you consider https://bugs.launchpad.net/ubuntu-terminal-app/+bug/1257791 more important than having a working clipboard?15:06
* dandrader trying to prioritize work15:06
ubot5Launchpad bug 1257791 in ubuntu-keyboard "Enter and backspace broken in terminal" [Undecided,In progress]15:06
karniHey guys, where does unity*7* put its logs?15:07
karnitsdgeos: You'll probably know ↑ :)15:07
karnifailed ascii arrow up15:08
tsdgeos↑↑ here a few for you :D15:08
tsdgeoskarni: never been much involved in unity7, so don't really know, sorry15:08
karniyeah, I got something like 'FG' xD heheh15:08
karnitsdgeos: that's fine15:08
karni.cache/upstart/unity7.log contains hardly anything15:10
davmor2dandrader, Saviq: I think they are on an even par, clipboard kinda important but terminal hasn't worked for an age and if you don't have a computer to hand you are screwed if you need to run a command15:10
karniSaviq: Might you know where unity7 keeps its logs?15:10
Saviqkarni, if it's upstart-launched, ~/.cache/upstart/, otherwise ~/.xsession-errors.log IIRC15:33
karniSaviq: correct, talked to dobey. Thank you!15:33
karniSaviq: If you need any reviews, hit me15:33
Saviqkarni, I think we're good for now, if you have time I'd rather you rebase the center-title branch15:35
karniSaviq: ack15:37
Saviqtsdgeos, why do we want to put "count" as a top-level prop of the gallery?15:41
tsdgeosSaviq: seemed to me something useful for outside users, maybe not15:42
Saviqtsdgeos, outside users won't know, TBH, the PreviewWidget interface is all outside users should be concerned with, don't you think?15:42
Saviqtsdgeos, and the test... I'd drop the count = 3, it gets too close to testing the bindings / ListView, IMO15:43
tsdgeoswell, as said, that's because you know it's a listview15:44
tsdgeosbut if we were to change it to something else15:44
tsdgeosi'd still want a 3 as count15:44
tsdgeosbut since you don't think we need to make sure count is exposed15:44
Saviqtsdgeos, fine, but grab it from a findChild, don't expose on the widget itself15:45
tsdgeoswe don't need to test it15:45
tsdgeosgone15:45
Saviqtsdgeos, also, docs++ please?15:45
tsdgeosSaviq: why would it need docs?15:45
tsdgeosgiven "the PreviewWidget interface is all outside users should be concerned with"15:45
tsdgeosi don't see why one would need docs15:45
tsdgeosunless you mean for the future ourselves15:46
tsdgeosand not for the class users15:46
Saviqtsdgeos, of course15:46
Saviqtsdgeos, just a short note "this displays images in a horizontal list, URIs for the images are expected in widgetData["sources"]" or such15:46
Saviqtsdgeos, and yeah, docs are for future us, future us + new team additions, contributors etc.15:47
Saviqtsdgeos, always easier to read a sentence than code15:47
Saviqor well15:47
Saviqthat's maybe on overstatement15:47
Saviqdepends on the sentence and the code15:47
tsdgeosit's 50 lines of code15:47
tsdgeosbut sure15:47
Saviqtsdgeos, I know, I know, just trying to push for some consistency, sorry :)15:48
Saviqtsdgeos, one sentence won't hurt15:48
Saviqtsdgeos, it's also documentation that scope authors might reach (or at least one that will be copied to docs for scope authors)15:51
tsdgeosSaviq: do we want the placeholderScreenshot thing as we have it now?15:54
Saviqtsdgeos, good question, we might just hide it if there's no sources...15:55
Saviqtsdgeos, leave it be for now15:55
Saviqtsdgeos, I'll ask around15:55
tsdgeosok15:55
tsdgeosi asked Cimi for some short docu too in the audioplayer preview15:57
=== salem_ is now known as _salem
=== xnox_ is now known as xnox
=== _salem is now known as salem_
karniubuntu-trusty + demo-stuff = music scope won't show up. I'll install another scope and see if that shows up.16:15
mhr3karni, when did you upgrade demo ppa last time?16:20
mhr3karni, cause i pushed onlinemusic there a few hours ago (so it's binary-compatible with latest scopes-api)16:20
mhr3although i don't even know if it wasn't16:21
karnimhr3: minutes ago, clean flash + demo-stuff install16:21
mhr3anyway, try upgrading :)16:21
mhr3oh16:21
karniI'll try upgrading if that helps16:21
karniI'm instaling unity8 build-deps atm16:21
mhr3karni, apt-cache policy unity9?16:21
mhr38 rather16:21
karnimhr3: http://paste.ubuntu.com/6867938/16:22
mhr3hm, looks good16:23
mhr3restart scope-registry16:23
=== balloons is now known as Guest54791
karnimhr3: http://paste.ubuntu.com/6867941/16:23
karnimhr3: ack16:23
tsdgeosSaviq: mhr3: pstolowski: the "UX spec" lists "ACTION" preview widget that can be combo/button/progress but then on the json structure definition i don't see how to create one or the other. Is the json spec focused on button only for the moment?16:23
karnimhr3: I'm running this on nexus 4. "restart: Unknown job: scope-registry"16:24
tsdgeoskarni: as phablet user?16:25
sil2100mhr3: hello! Are unity-scopes-api and unity-scopes-shell used already in unity8?16:26
sil2100mhr3: or are those all related to the new scopes API?16:26
=== dandrader is now known as dandrader|lunch
* dandrader|lunch -> lunch16:27
karnitsdgeos: yes, phablet user16:29
tsdgeosok, no idea then16:29
karniI'll investigate one I have build-dep for unity8 installed16:30
kklimondahmm, unity in 14.04 seems to have HUD key hardcoded? I can't change it from the keyboard->shortcuts16:34
sil2100Saviq: ok, so there is a really big chance that unity8 caused a regression of installing apps16:35
sil2100Saviq: give us a sign once you're up16:36
sil2100Saviq: I mean, back16:36
sil2100;)16:36
sil2100Saviq: we're still bisecting, but it seems to be the main culprit right now16:36
kklimondaah, it actually works after restarting unity16:36
pstolowskitsdgeos, JSON doc has an example of "actions": [{"label": null, "icon": null, "id": null}]"16:37
tsdgeospstolowski: yes and how does that tell me if it's a combobox, a button or a progress bar?16:38
sil2100Saviq: ok! So, we confirmed - unity8 regressed!16:40
karniSaviq: do you have the design drive URL (could you PM it to me?)16:40
karnithe one with new design dash screen design16:41
sil2100Saviq: so, it seems 7.84+14.04.20140128-0ubuntu1 regressed click app installation16:42
sil2100Saviq: we need this fixed ASAP16:42
pstolowskitsdgeos, ah, sorry, I misread your question. yes, the doc doesn;t cover all those16:43
mhr3tsdgeos, we have "actions" widget and "progress" one, actions can be combo if it contains multiple actions, progress is too special so it's completely separate16:43
mhr3karni, are you running that in the context of user session?16:43
sil2100Saviq: the regression looks as following (easy to reproduce) - in case you didn't see the backlog in -ci-eng:16:43
mhr3sil2100, no, they aren't used16:44
sil2100mhr3: thanks16:44
karnimhr3: what is a context of a user session? adb shell - I can run either root or phablet, none worked16:44
karnimhr3: sorry, hangout with Victor and Joe16:44
cwayne1user session == sudo -iu phablet16:45
cwayne1AIUI16:45
mhr3karni, ^ yea, that16:45
sil2100Saviq: so, the case: find a click app that's not installed, open it up in the dash, install it - you will see that after it's done installing, unity8 is not showing the info for the installed application but instead moves to the first application in the scope16:45
karnitnx16:45
sil2100davmor2: did you fill in a bug for this ^ ?16:46
karnicwayne1: mhr3: tried, restarted scope-registry, nothing showed up. I'll reboot the device as soon as I can.16:46
davmor2sil2100: I am currently16:46
sil2100davmor2: thanks!16:48
davmor2sil2100: https://bugs.launchpad.net/unity8/+bug/127583216:49
ubot5Launchpad bug 1275832 in Unity 8 "REGRESSION: Installing apps open the first click app once installed rather than their own app" [Undecided,New]16:49
mhr3karni, tail ~/.cache/upstart/scope-registry.log ?16:53
sil2100tsdgeos: hi! You if Saviq will be around today still?16:53
sil2100Saviq, tsdgeos: sadly, we'll have to revert16:55
karnimhr3: sorry, focus on the hangout, but here's the paste: http://paste.ubuntu.com/6868113/16:56
tsdgeossil2100: i am16:57
mhr3karni, it's saying that everything is working16:57
tsdgeossil2100: what's wrong?16:57
mhr3karni, minus the phone not having network16:58
=== balloons is now known as Guest51231
karnimhr3: ok, it does, I guess reboot will help (though, I guess it should requery after network appeared on the phone)16:59
mhr3karni, it should... but it won't16:59
tsdgeossil2100: so you opened a bug 8 minutes ago and you're going to revert the stuff because of it?16:59
ubot5bug 8 in Launchpad itself "Translator forums/means of communication" [Low,Won't fix] https://launchpad.net/bugs/816:59
karnimhr3: :D16:59
tsdgeosi am totally out of understanding the policies of this place every day more and more16:59
didrockstsdgeos: the revert is uploaded, we pinged Saviq and kgunn, we can't wait more. We'll have to again finish at 9PM just because of it. So yeah17:00
tsdgeosdidrocks: this is insane honestly, what's the point of opening a bug 8 minutes ago and then deciding to revert the code anyway?17:01
didrockstsdgeos: revert is not a punition and we don't touch your trunk17:01
ubot5bug 8 in Launchpad itself "Translator forums/means of communication" [Low,Won't fix] https://launchpad.net/bugs/817:01
tsdgeosyou may as well not open the bug at all17:01
didrockstsdgeos: this regression is costing us 8 hours17:01
didrockstsdgeos: well, it's to track it upstream17:01
sil2100tsdgeos: we were poking for Saviq, who is our point of contact, and we cannot block it too much anymore17:01
tsdgeosdidrocks: did you tell saviq 8 hours ago?17:02
didrockstsdgeos: hum?17:02
didrockstsdgeos: we discovered it an hour ago17:02
tsdgeosah17:02
didrocksfetching for it17:02
sil2100tsdgeos: we poked on IRC about it already, the bug just got filled 8 minutes ago but we push on it since longer17:02
didrocksdoing the investigation ourselves17:02
tsdgeosyou mean it's costting you 8 hours because that's how much creating an image takes17:02
tsdgeosright?17:02
didrockstsdgeos: upload publication + image creation + test results, yeah17:03
tsdgeosi understood it had been found 8 hours ago17:03
didrocksno, sorry if I wasn't clear17:03
tsdgeossil2100: it'd be cool to know to what package you're reverting17:04
tsdgeos7.84+14.04.20131128.2-0ubuntu1 ?17:04
sil2100tsdgeos: old one... ;/ reverting to 7.84+14.04.20131220-0ubuntu1...17:04
=== jono is now known as Guest16175
sil2100tsdgeos: since 7.84+14.04.20140128-0ubuntu1 introduced the regression17:05
tsdgeosyes17:05
sil2100tsdgeos: so that's a LOT of changes17:05
tsdgeosare you guys really sure it's unity8 fault? because yes there are a lot of changes, but at first sight i can't find anything that touches how we launch apps, most of the code we added is even being used17:07
Saviqsil2100, didrocks, tsdgeos, am back17:08
sil2100Saviq: \o/17:08
sil2100tsdgeos: yes, we confimed it this way:17:08
sil2100tsdgeos: we installed 153 (everything working there), then installed the 7.84+14.04.20140128-0ubuntu1 version of unity8 and suddenly it's broken - no other packages changed17:09
tsdgeosok, that seems pretty clear17:10
tintoumhr3: https://code.launchpad.net/~tintou/libunity/removed-gee/+merge/204540 Here it is :)17:26
mhr3tintou, you're awesome17:33
=== salem_ is now known as _salem
mhr3tintou, one thing i'm oblidged to ask though - did you sign the cla?17:33
tintoumhr3: no I didn't… yet :), tell me how to do17:35
tintoumhr3: I'm signing here http://www.canonical.com/contributor-license-agreement/submit17:36
mhr3tintou, http://www.canonical.com/contributor-license-agreement/submit17:36
mhr3yep17:36
mhr3tintou, anyway, i'll still need to review it properly, expect a lp mail :)17:37
tintoumhr3: okay, tell me and I correct what is needed,I've signed the agreement17:39
mhr3tintou, did you put libunity as the contact?17:39
tintoumhr3: I did put your launchpad-id17:39
mhr3oh right, it's supposed to be a person not lp project17:40
mhr3cool17:40
=== dandrader|lunch is now known as dandrader
=== alan_g is now known as alan_g|EOD
larsuSaviq: is there a way tell qml to completely construct an object before other objects are constructed?18:14
larsuit seems like the order in which onCompleted is called for objects in the same file is random18:14
Saviqlarsu, nope, they're all constructed at the same time18:14
Saviqlarsu, what's your use case?18:15
larsuSaviq: that sucks :/18:15
Saviqlarsu, well, that's declarative for you, it's all there, at the same time, always18:15
larsuSaviq: a GSettings object gets its onCompleted called before a sdk checkbox18:15
larsus/before/after18:15
larsuwhich makes the checkbox animate when it is first shown18:16
larsubecause gsettings-qml creates the actual GSettings object in componentCompleted18:16
Saviqlarsu, that's probably a thing to fix in the checkbox18:16
larsuSaviq: hm, how would you do that? Add some way to make it not animate?18:17
Saviqlarsu, I don't think you're the cause of that behaviour, really18:18
larsucheckbox.animated: gsettings.ready?18:18
Saviqlarsu, as properties are set on the object after it gets created18:18
larsuno...18:18
larsuat least, I don't think so according to the testing we just did18:18
=== Guest51231 is now known as balloons
Saviqlarsu, oh well, actually18:19
Saviqlarsu, depends18:19
Saviqlarsu, if it's a value, it'll be there when created18:21
Saviqlarsu, if it's a binding, it will only get evaluated after18:21
Saviqlarsu, so the checkbox gets created with false, and that changes when the binding is evaluated18:21
Saviqlarsu, so yeah, it goes false → true in that case, and that's something you should be able to prevent - and yeah, an "animated" property would be a possibility18:22
larsuSaviq: it always works if I bind it to a property that is not from the GSettings object18:24
larsualso, moving the GSettings object below the checkbox makes the order turn around18:25
larsuin which case the animation also doesn't happen18:25
Saviqlarsu, sure, but that's not something you can rely on18:25
larsuya, of course not18:25
larsuI'm just trying to figure out what's going on :)18:25
Saviqlarsu, so the only solution is to make it so that animation is inhibited until the initial value is retrieved from gsettings18:26
larsuright18:26
larsuI could also create the GSettings object when the property is set18:27
larsuinstead of in componentCompleted18:27
larsubut that's a bit shitty: it might depend on other properties (like state)18:27
Saviqlarsu, but again, that might not solve your issue anyway18:29
Saviqlarsu, because the creation order is not deterministic, so if your onCompleted is called _after_ checkbox is already created, there will be an animation anyway18:29
larsuSaviq: hm okay. Who can I ping to add an animated property?18:35
Saviqlarsu, bug on ubuntu-ui-toolkit, assigned to ubuntu-sdk-team18:36
larsucool will do, thanks18:36
karniSaviq: Can we queue this up or there was something wrong with it? https://code.launchpad.net/~unity-team/unity8/unity8-fix-mascot/+merge/20396718:42
Saviqkarni, it's queued up18:42
Saviqkarni, row 40 in the CI train spreadsheet18:43
karniShould have checked, thank you Saviq18:43
karniSaviq: could you remind me the name of branch that had this new grey'ish background?18:44
karniSaviq: found it18:45
karnisorry :)18:45
Saviqkarni, don't be18:45
karni:)18:46
=== _salem is now known as salem_
Saviqdidrocks, FWIW identified the issue, fix+test on the way18:56
didrocksSaviq: great, thanks18:58
* didrocks still does more QA work…18:58
=== jono is now known as Guest82971
=== AlanBell_ is now known as AlanBell
Saviqmterry, you around?19:21
mterrySaviq, yup19:21
Saviqmterry, could you do https://code.launchpad.net/~saviq/unity8/fix-preview-updates/+merge/204559 please19:22
* mterry looks19:22
Saviqmterry, making sure that the bug it's meant to fix is gone19:22
=== salem_ is now known as _salem
SaviqCimi, bzr blame to the rescue ;)19:29
* mterry flashes fresh nexus419:29
Saviqmterry, thanks19:29
SaviqCimi, re: docs, comments _are_ docs, but to extract them into readable docs outside of code, they need to adhere to some standards19:31
darklight_Is it possible to finally have the shorcuts customizable in 14.04 ?19:32
SaviqCimi, and Albert meant the actual AudioPlayer.qml file - just put a line or two about what the component does, and what kind of input it expects (array of objects in the form...)19:32
Saviqdarklight_, which shortcuts do you mean?19:33
darklight_Saviq, the ones that are hardcoded such as the meta+[1..9] to launch applications19:33
Saviqdarklight_, I assume there's a bug about that, and unless it's marked as In Progress / Fix Committed / Fix Released, nothing is happening in that direction19:34
darklight_Saviq, https://bugs.launchpad.net/unity/+bug/1022743 since 2012 and in the past year it's just been pushed back and back, honestly I think it would be the right time to address it with an lts release coming up and it shouldn't even be incredibly diffcult to fix at least give an option to turn them off19:36
ubot5Launchpad bug 1022743 in unity (Ubuntu) "Hardcoded Unity shortcuts responsible for several problems" [Low,In progress]19:36
darklight_not to mention it is inconsistent since it's possible to set harcoded shortcuts for other things (eg workspace switching) and it won't work19:37
darklight_and it could possibly create accessibility issues19:37
Saviqdarklight_, commenting on the bug would get you a bigger audience than here, and one that might want to/have time to help fix it19:38
Saviqdarklight_, I doubt Canonical will put any effort in that, as simply it's not getting us any closer to our targets19:38
Saviqdarklight_, we will definitely accept contributions, though19:39
Saviqdarklight_, and spend time reviewing/helping with any patches that are submitted19:39
karniSaviq: if you have a small-card grid with only 2 items - is it intentional they're left aligned, instead of spread throughout horizontal space equally?19:39
Saviqkarni, yes19:39
darklight_Saviq, well to be candid a design with hardcoded shortcuts should not have been ever approved in the first place so while I understand how it must be boring to fix something like that whoever approved it in the first place should take responsability for it imo19:40
karnitnx19:40
Saviqkarni, because the layout is configured thus19:40
karniSaviq: yeah, I mean it could be |  X X  | instead of | X X   |19:40
Saviqkarni, no, because then you have 3 items, and suddenly stuff's shifting19:40
karnissweeny: ^ re: weather19:41
karnissweeny: so if theres just 1 small tile, it'll be left aligned anyway.19:41
karnissweeny: sorry, that was to Saviq ^19:41
karnissweeny: but previous '^' was FYI19:41
karnissweeny: that might not align with what Joe would like to see19:41
Saviqkarni, ssweeny, if the scope knows that it will only come up with one or two results, it should adapt its configuration to match, I'd say19:41
Saviqdarklight_, also, since the bug is assigned and In Progress for MC Return, trying to get a progress update from him would be the best first step19:42
Saviqdarklight_, it's not like that, we're not writing that software for sheer correctness19:42
karniSaviq: ssweeny: ignore me. I had to make other scopes use small cards, but we're good with medium size card for weather.19:42
darklight_Saviq, is he on irc ?19:42
Saviqdarklight_, no idea, sorry19:42
Saviqdarklight_, [...] sure, we try, but when we need to cut corners, we do19:43
Saviqdarklight_, having perfect, customizable projects is a great goal to have, but then that doesn't necessarily get you any closer to the real goal of having more users19:43
Saviqdarklight_, "Patches welcome" is possibly not a nice thing to say, but it is, in fact, the truth19:44
Saviqdarklight_, unity7 is in low maintenance mode and fixing that bug does not get us anywhere in the grand scheme of things19:45
darklight_Saviq, I'm not trying to get into an argument here but considering it's the last lts release before unity 8 it might be worthed to polish it as much as possible rather than already shifting the focus to something that is still far from being mature, playing the game of the grand scheme of things is dangerous if there's more focus to the future rather than the reality of the present19:50
Saviqdarklight_, but that particular thing is polishing one part that only a handful of people care about, it's a perfect example for community contribution19:51
Saviqdarklight_, and don't worry, there's plenty of more important polish happening on unity7 in parallel with focusing on the next thing, which, to be ready in any foreseeable future, needs plenty of focus, too19:51
darklight_Saviq, I'll try adding a comment to the bug hopefully someone will pick it up, unfortunately I don't have the skills to work on a proper solution19:54
=== dandrader is now known as dandrader|afk
mterrySaviq, so I flashed and was going to test your fix.  But I can't even get another app to install..  is that a different known bug?20:01
Saviqmterry, "can't get to install"?20:01
Saviqmterry, you sure?20:01
Saviqmterry, search for it in the apps scope (give it a few secs, too)20:01
mterrySaviq, no?  I click on the "install" button but nothing seems to happen20:01
mterrywill wait longer20:01
Saviqmterry, well, that should only take a second or two20:02
* Saviq flashes20:02
mterrySaviq, never gets to progress bar20:02
Saviqmterry, latest trusty-proposed?20:02
mterrySaviq, yeah20:03
mhall119Saviq: ping20:04
Saviqmhall119, pong20:04
mhall119Saviq: hey, so we want to show off some core apps converged for tablet at MWC20:05
mhall119I'm told we'll have a 2013 Nexus 7 at MWC20:05
mhall119my question is, will Unity 8 run in landscape orientation on the Nexus 720:05
=== _salem is now known as salem_
Saviqmhall119, we need to make it happen, but yes, that's the plan20:06
Saviqmhall119, https://blueprints.launchpad.net/unity8/+spec/client-mwc-unity820:06
Saviqand bug #1275750 in particular20:06
ubot5bug 1275750 in unity-mir "Shell UI needs rotating on nexus7(2013)" [Undecided,New] https://launchpad.net/bugs/127575020:06
mhall119what's the earliest you think that will be done?20:06
mhall119so we can start testing those apps on it20:07
Saviqmhall119, it'll be late, next week at the earliest20:07
mhall119that'll be perfect20:08
mhall119thanks20:08
=== dandrader|afk is now known as dandrader
=== salem_ is now known as _salem
=== _salem is now known as salem_
=== kgunn is now known as Guest438
=== boiko_ is now known as boiko
mterrySaviq, just a follow up, neither rebooting nor applying your branch fixes the "can't even install an app" problem I was seeing21:12
Saviqmterry, yeah, wouldn't have an effect on that21:18
Saviqmterry, am flashing now, downloading SLOOOW21:18
mterrySaviq, yeah I figured not, but figured I'd want the changes anyway once this is fixed  :)21:18
Saviqmterry, right, let's see if I can reproduce, then - QA folks did not report that issue21:19
mterryhrm, maybe my u1 account is borked on the device21:20
Saviqmterry, that'd be good to know (and report indeed)21:21
* mterry is not sure how to check that21:22
Saviqkenvandine, can you help ↑?21:22
Saviqkenvandine, how to gather info about a potentially broken u1 account on the phone?21:22
* mterry looks in system settings and it doesn't look right21:23
mterryLooks like it recognizes that a U1 account exists, but doesn't have an attached email21:23
kenvandineSaviq, use account-console to print info21:23
mterrySome previous install must have created this half-account21:23
kenvandinethere was a bug related to that i think21:23
Saviqmterry, ↑21:23
kenvandinei think remove the account and add again21:23
kenvandineor ask dobey21:23
mterrykenvandine, account-console via adb?21:24
kenvandinejust from shell21:24
mterrykenvandine, right, but via adb shell?21:24
kenvandinei think you need to install account-plugin-tools21:24
mterrykenvandine, done.  account-console crashes for me after complaining about accessing dbus, so sounds like it wants the user dbus session.  Let me get that21:25
kenvandinemterry, but i don't know if it'll tell you anything useful21:25
Saviqmterry, kenvandine, seems they're there:21:25
Saviq$ account-console show 121:25
Saviqaccount: id 1, enabled, provider: ubuntuone21:25
Saviq  Global settings:21:25
Saviq    CredentialsId: 1 (<class 'int'>)21:25
Saviq    auth/mechanism: password (<class 'str'>)21:25
Saviq    auth/method: password (<class 'str'>)21:25
Saviq    enabled: True (<class 'bool'>)21:25
Saviq    name: michal@sawicz.net (<class 'str'>)21:25
Saviq  Settings for ubuntuone21:25
Saviq    enabled: True (<class 'bool'>)21:25
kenvandinemterry, ^^21:25
kenvandineaccount-console list21:26
kenvandineto get the id21:26
kenvandinethen account-console show ID21:26
mterrySaviq, just from adb?21:27
Saviqmterry, as phablet21:27
mterryaccount: id 1, disabled, provider: ubuntuone21:27
mterry  Global settings:21:27
mterry    auth/mechanism: password (<class 'str'>)21:27
mterry    auth/method: password (<class 'str'>)21:27
mterry  Settings for ubuntuone21:27
mterry    enabled: True (<class 'bool'>)21:27
Saviqmterry, so `sudo -u phablet -i` recommended21:27
mterrykenvandine, ^21:28
kenvandinemterry, yeah... that is missing a bunch of info21:28
kenvandinecan you remove it?21:28
mterrykenvandine, sure can try21:28
kenvandinei think there might be a problem though... like updating existing click packages21:28
kenvandineafter re-adding21:29
kenvandinei don't recall... but i think dobey knew what to do21:29
Saviqdefinitely worth a bug on click / click scope  and such21:29
mterrykenvandine, Saviq: removing that account let me get further21:29
Saviqand online accounts as well21:29
kenvandinei think there was a bug, that was fixed... iirc21:29
kenvandineit was in ubuntuone-credentials21:30
kenvandinei think21:30
mterrykenvandine, now this is fun.  After removing, I can't enter the Accounts settings panel21:31
mterryclicking it doesn't do anything21:31
kenvandineugh...21:31
kenvandinetry rebooting first21:31
kenvandinemight be a process waiting to finish...21:31
mterrySaviq, anyway.  I manually confirm the bug and your fix with my U1 troubles out of the way21:40
mterrykenvandine, so you believed the u1-credentials bug to be fixed and released?  i.e. I should file a new bug?21:40
kenvandinemterry, i think it was... but it was a while back21:41
kenvandineso maybe regression...21:41
=== jasoncwarner__ is now known as jasoncwarner
=== sam113101 is now known as YOURBESTFRIEND
Saviqmterry, care to review the branch itself, please? ack, ideally?22:06
Saviqmterry, (remember about the checklist, too) :)22:06
mterrySaviq, ah yes.  Let me take care of some food stuff here, brb22:07
Saviqmterry, sure, it's not like anything will happen before tomorrow, would just like to get it ready22:07
=== salem_ is now known as _salem

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