=== olli_ is now known as olli [05:54] Saviq: something fishy in Unity8 compilation are actually dependencies installation https://bugs.launchpad.net/unity8/+bug/1212944 [05:54] Launchpad bug 1212944 in Unity 8 "unity 8 FTBFS (during package installation)" [Critical,New] [06:57] Mirv, hmm, looks like usermetricsservice configuration failed [06:59] Mirv, "Setting up usermetricsservice:i386 (1.1.1+13.10.20130815.2-0ubuntu1) ... [06:59] Failed to open connection to "system" message bus: Failed to connect to socket /var/run/dbus/system_bus_socket: No such file or directory" [07:00] Mirv, but that shouldn't be a build dependency anyway [07:03] Mirv, it seems http://bazaar.launchpad.net/~unity-team/libusermetrics/trunk/revision/125 is the culprit [07:03] Mirv, it has a .postinst that requires the system bus to be there [07:03] Saviq: yeah, it shouldn't fail installing in case it's not [07:04] Mirv, yup [07:13] hi sil2100 [07:15] Mirv, are you doing something about it or are we waiting for pete-woods? [07:21] Saviq: not doing at the moment and haven't filed a bug yet as I've other urgencies ongoing [07:21] Mirv, I've added affects: libusermetrics to the other bug [07:21] Mirv, so I'll own that, then [07:21] maybe sil2100 can look at it now that he's here, after he catches up :) (thankfully he's back to help!) [07:21] :) [07:22] thank you [07:27] Saviq, btw could we get someone to help with the running apps renderer? it doesn't support regular scope results model... when i was doing the category overrides branch i had a quick look at it, but it was getting too much into the ui bits for my taste [07:28] Saviq, touching ResponsiveFlowGrid... eek :P [07:29] mhr3, not sure we want to spend time on it when the Mir-backed recent apps scope should be working soon [07:30] Saviq, but that's why we need it, it needs to support the regular scope model [07:30] mhr3, ah, got it now [07:31] Saviq, not super high on the list, but once the mir bits are in, we'll want to use it [07:31] mhr3, yeah, will do [07:34] Saviq, off topic what's the current way of getting new image on the phone? phablet-flash ubuntu-system? seems to take too long compared to what was there a week ago :/ [07:34] mhr3, cdimage-touch [07:34] mhr3, otherwise you get read-only / [07:38] at least someone knows these things :) [08:01] Saviq: what do i use to test https://code.launchpad.net/~saviq/unity8/fix-generic-preview/+merge/179932 ? [08:01] tsdgeos, search for something in home [08:01] tsdgeos, and long-press [08:01] tsdgeos, description (if long) should be wrapped and formatted [08:01] tsdgeos, wikipedia is usually useful [08:02] tsdgeos, http://ubuntuone.com/0y3o0lPEaWsVBEP6gdiJdd http://ubuntuone.com/4AGFEchm7D1QqVvDtrbmZb [08:02] tsdgeos, we seem to have triggered an issue in LVWPH :/ [08:02] ah yes [08:02] there was this bug you said the other day in which it stuck in the middle? [08:03] is this that one? [08:03] tsdgeos, yeah [08:04] hmm [08:05] tsdgeos, and then something related to category expansion - go to apps, search for something, clear the search, click the "Dash plugins" category to expand it [08:05] tsdgeos, stuff breaks badly :/ [08:05] tsdgeos, like first of all, the limiting filter gets confused and doesn't actually "pass through" the items [08:06] tsdgeos, and then the whole thing just goes all over the place :/ [08:06] hmmm [08:06] works here? [08:07] what is really weird is that i can't vertically move stuff in the home scope until i expand files & folders [08:08] which kind of makes sense but doesn't at the same time :D [08:08] tsdgeos, i think the limiting proxy doesn't handle few cases - for example removal of one of the visible items, which pushes one of the invisible items to become visible [08:09] and vice versa [08:09] may be [08:09] pushing a new item in the first position, which should remove the last visible one [08:10] sup everyone? scopes still getting you down? [08:11] dednick, it's all your fault! [08:11] dednick, hope you feel down as well now :P [08:12] um. what did i do? [08:12] pete-woods, https://bugs.launchpad.net/unity8/+bug/1212944/comments/1 [08:12] Launchpad bug 1212944 in Unity 8 "unity 8 FTBFS (during package installation)" [Critical,In progress] [08:12] dednick, nothing in particular, we just want you to feel bad too, so you don't feel excluded [08:13] oh. thanks :) [08:13] anytime [08:13] i'm sure i had something to do with it anyway. [08:13] Saviq: thanks for pointing that out! [08:13] pete-woods, well, yeah, you caused FTBFS for unity8 ;) [08:16] Saviq: I don't understand why dbus wouldn't be there - it's a dependency [08:16] but what the hey, I'll make the thing less sensitive [08:16] pete-woods, it's not started, you don't want dbus on the build machines [08:16] Saviq: okay, I suppose that's fair! [08:19] Saviq: there's some weirdness in wrapping+richtext, look for elephant and preview the one in the Info category, see how text gets wrapped too early, are we ok with that [08:19] ? [08:21] tsdgeos, good catch, let me see [08:22] Saviq: but it seems like a qt bug ::/ [08:22] i've added a rectangle on the label and it's covering the whole width [08:22] dednick, i think we broke something quite horribly - http://10.97.0.1:8080/job/autopilot-saucy-daily_release/1027/ [08:23] ouch [08:26] sil2100: hi. What do we need to do next to get mediascanner into the archive? [08:26] tsdgeos, I wonder if it's the line that's causing that [08:26] maybe [08:26] Saviq: anyway it0s an improvement over what we have, so going to aprove it [08:27] and I'll log a bug about it, ok? [08:27] mhr3: told you i did something [08:31] mhr3: yeah, i think i can see what's going on. it is my branch [08:32] ...although [08:34] mhr3: no, dont know what's happening [08:34] tsdgeos, hmm, the interesting thing is... should \n be considered as newlines in richtext anyway? [08:34] dednick, give me a sec, standup [08:35] Saviq: i think not [08:35] that's not how html behaves [08:35] tsdgeos, exactly [08:35] MacSlow, ping [08:36] nic-doffay, yes [08:36] MacSlow, do you have experience with running autopilot tests? Remember you mentioning it in standup a while ago. [08:36] nic-doffay, I do have some... what do you want to run? [08:37] There's an issue with one of my branches and I need to sort it out. [08:37] MacSlow, https://jenkins.qa.ubuntu.com/job/generic-mediumtests-saucy/2286/? [08:37] tsdgeos, ah, I think it's caused by the fact that font size is set [08:37] It's related to the mouse pointer now that the OptionSelector has been added. [08:37] Saviq: weird [08:38] tsdgeos, aactually! [08:38] tsdgeos, check this out http://paste.ubuntu.com/5992013/ [08:39] tsdgeos, it doesn't get re-laid-out if you make the window bigger [08:39] brr [08:39] jamesh: hi! I'll be poking someone to sponsor it today into the archive [08:39] nic-doffay, so you want to only run that "test_can_select_listview" locally for debugging the issue?! [08:39] jamesh: since the packaging is more or less ready now [08:39] sil2100: thanks! [08:40] sil2100: any chance you could take a quick look at lp:unity-scope-mediascanner too? It is a lot simpler, and I think I've got most things right [08:40] mhr3, pstolowski so... richtext... can we nl2br the text? will unity7 be fine with
s? [08:40] Saviq: only if you make it smaller :/ [08:40] tsdgeos, exactly [08:40] Saviq: same with 5.1 actually [08:40] * tsdgeos goes on crying [08:41] MacSlow, ideally yeah. [08:42] nic-doffay, you just cd to unity8/tests/autopilot ; autopilot list unity8 ... and then look for the test you're interested in... [08:42] nic-doffay, then just "autopilot run " [08:42] nic-doffay, will be some looooong name [08:42] Saviq: do you report a bug to Qt? [08:43] tsdgeos, yeah, will do, trying for a workaround now [08:43] Saviq: you can replace the Label with Text to drop Ubuntu.Components [08:43] nic-doffay, sorry... make that toolkit not unity8 :) [08:43] tsdgeos, doesn't matter [08:44] Saviq: i mean for the bug report in Qt [08:44] tsdgeos, right, of course [08:44] MacSlow, great thanks! [08:45] jamesh: sure thing! [08:45] nic-doffay, should be... cd ubuntu-ui-toolkit/tests/autopilot ; autopilot run ubuntuuitoolkit.tests.gallery.test_gallery.GenericTests.test_can_select_listview [08:46] wtf is up with libusermetricoutput? it keeps on conflicting on my system. [08:47] dednick: the thing that gives data for the welcome screen infographic [08:47] every time i do an upgrade, it buggers up the install and i get a half upgraded system that is totally borked. [08:47] something with the conf file. [08:47] weird [08:48] works fine here [08:50] dednick, maybe you have some ppa enabled that you should not? [08:51] Saviq: http://ppa.launchpad.net/ubuntu-unity/next/ubuntu ? [08:51] dednick, yeah, not needed / wanted [08:51] dednick, for unity8 you don't need any PPAs now [08:51] dednick, so anything you have, you need to know that you want it :) [08:51] Saviq: cool. thanks [08:58] tsdgeos, ugh, can you edit the description in bugreports? [08:59] not sure [08:59] Saviq: why one? [08:59] "Provide more info" [08:59] of course! [08:59] tsdgeos, one what? [08:59] which one i mean [08:59] t [08:59] mhr3: hm. everything looks in terms of searching in the videos... [09:00] tsdgeos, https://bugreports.qt-project.org/browse/QTBUG-33020?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel [09:00] tsdgeos, fixed already [09:00] ah ok [09:01] mhr3: i wonder if something is going wrong with counting the number of items in the categories... [09:02] tsdgeos, pushed a workaround [09:03] oh wow :D [09:03] tsdgeos, actually, need to do one more [09:03] tsdgeos, yah! [09:04] tsdgeos, I knew you'd love it ;) [09:06] tsdgeos, pushed s/\n/
/g [09:06] +\ [09:06] oka [09:07] Saviq, yea, you better do that in the shell, it's coming from the scopes that way [09:07] mhr3, ok, un-TODOing, then [09:07] (and we don't want to fix each and every scope) [09:08] mhr3, yeah, what I though [09:08] t [09:08] tsdgeos, need to re-approve, won't land otherwise [09:08] yep [09:08] let me test it works :D [09:09] tsdgeos, but then it won't land anyway... [09:09] pete-woods, can we expect a fix for libusermetrics sometime soon? :) [09:09] dednick, i guess the category isn't "fully" removed and ap picks up an old one that has the same name and is empty now? [09:09] sil2100, is libusermetrics part of unity8's stack? (it should be, I think...) [09:09] Saviq: it's committed already [09:09] pete-woods, ah [09:09] pete-woods, thanks [09:09] np [09:10] sil2100, Mirv can we release libusermetrics? [09:10] mhr3: hm. maybe. i guess it would be set to non-visible! dang. didnt think about that [09:10] stupid categories [09:11] dednick, but there's a call to ClearCategories() shouldn't that handle it? [09:11] mhr3: it should [09:11] Saviq, sil2100: running indicators stack, can you sil2100 look it goes through? [09:11] Mirv: ok [09:12] well that's a bugger. builddeb failed because of one test... sigh [09:12] and yes for some reason it's in the indicators stack [09:13] dednick, let's corner Trevinho next week and make him fix it, he's in london [09:13] mhr3: hehe. fix our bug! [09:13] will probably need to lock him on the toilet with his laptop [09:13] oh wait... [09:15] mhr3: yeah, it's multiple categories. i can reproduce [09:15] mhr3: just removed SetVisible(false) ... [09:16] dednick, so why aren't they cleared? [09:16] :/ [09:16] i will find out promptly [09:17] oh... the initial setup is called before all the categories are added, isn't it? [09:17] so it adds them twice the first time [09:18] it should clear them every time it switches model. [09:18] unless the bind isnt working correctly.... [09:30] mhr3: right. so apparently sigc::bind doesnt work with sigc::hide. [09:31] mhr3: and http://pastebin.ubuntu.com/5992130/ [09:31] i'm getting 2 application categories... [09:31] dednick, that's fine [09:32] mhr3: why would that be fine? [09:32] dednick, one has id "global", the other "apps" [09:32] you're printing the name, it's the same [09:33] as you can see the index is different [09:35] mhr3: yeah, i thought it was just adding 2 apps at different indexes [09:35] cool. it's my bad then [09:36] it's adding duplicates because the bind for "bool add" isnt working [09:36] rather retardedly [09:37] Mirv, hey [09:37] dednick, but your log says "false", doesn't that mean it works? [09:37] mhr3: after fixing yes [09:37] ok [09:37] mhr3: was always true [09:38] * mhr3 loves when compiler / library doesn't work [09:39] jamesh: one thing about mediascanner - would you mind if I change the MEDIASCANNER_VERSION variable in CMakeLists.txt to the proper version? [09:39] sil2100: that sounds fine. [09:43] mhr3: https://code.launchpad.net/~nick-dedekind/unity/duplicate.categories/+merge/180511 [09:44] * dednick endevours to actually test code he merges [09:44] om26er: hello [09:45] Saviq: shall we get the indicators show on the road? https://code.launchpad.net/~nick-dedekind/unity8/remove.battery.time.indicators/+merge/180375 [09:45] dednick, not yet, we still can't release unity8... [09:45] Mirv, the qtorganizer5-eds project that you setup yesterday didn't have a source code branch target, but I just saw someone added that [09:45] dednick, need to wait for unity to release first [09:46] Mirv, thanks anyway :) [09:46] om26er: yes, that "someone" was me as well :) [09:46] after I noticed there was some branch now to set as such [09:46] dednick, at least libusermetrics should not be killing us anymore [09:46] dednick, how about using standard sigc::mem_func(this, ClearCategories), and do the forced add at the end? will be more readable [09:46] Mirv, cool, thanks [09:46] sil2100, Indicators finished, yellow though (packaging change) [09:46] om26er: so it's now just lp:qtorganizer5-eds like it should be [09:46] Saviq: needs +1 from core-dev [09:47] Saviq: if you see any around, please get, for http://pastebin.ubuntu.com/5992152/ [09:47] Mirv, can you +1 this https://code.launchpad.net/~om26er/cupstream2distro-config/ubuntu_keyboard_release_daily/+merge/180442 ? [09:47] Mirv, who's "core-dev"? [09:47] mhr3: yeah, dont know why still doing func for a single line... [09:47] Saviq: https://launchpad.net/~ubuntu-core-dev/+members [09:47] Saviq: yes, we're poking around about that [09:47] Mirv, its to enable daily release for ubuntu-keyboard, I got the packaging reviewed from mterry yesterday, he made a few improvements [09:47] Saviq: it's the deal we have, need an ack from one always [09:47] sil2100, ok thanks [09:48] Mirv, understood [09:48] preferably in 12 minutes before the next cycle starts :) I wonder how long ogra's meeting is [09:48] :D [09:48] dednick, if i had this comment yesterday we'd save ourselves quite some trouble :) [09:48] om26er: ok, looking [09:51] Saviq, we need vim in the default touch image, i can't stand vi :P [09:51] mhr3, https://code.launchpad.net/~ubuntu-core-dev/ubuntu-seeds/ubuntu-touch.saucy [09:51] ;) [09:51] need to apt-get it everytime... terrible user experience [09:51] mhr3, and ssh [09:51] +100 :) [09:51] :| [09:51] Saviq: got this http://paste.ubuntu.com/5992173/ [09:52] just starting and unlocking [09:52] Without didrocks and seb128 it's so much more complicated to get things released :| [09:52] tsdgeos, https://code.launchpad.net/~mhr3/unity/only-valid-filters [09:52] At least until ken and mathieu pop up later [09:52] OH ZE FRENCH! [09:52] Saviq: but that is merged already [09:52] and i just got it [09:52] VER AR THEY WHEN YOU NEED THEM? [09:52] tsdgeos, not released [09:52] ah wait [09:52] it's on unity [09:52] not unity8 [09:52] tsdgeos, yes [09:52] true-ish [09:52] tsdgeos, and not released [09:53] Saviq: they are enjoying 15th august holidays? :D [09:53] tsdgeos, sounds like it indeed ;) [09:53] mhr3, what's the deal with the Unity stack? any ETA? [09:54] Saviq, yea, we have a fix now, should land soonish [09:54] Saviq, so... 4hours? [09:55] mhr3, daily starts in 5 mins, will it land by then? ;) [09:55] 5 minutes leeeft! [09:55] erm quaternourly [09:55] Mirv: did you find anyone?! [09:55] no [09:55] :/ [09:55] maybe if you allow dednick to push the fix directly [09:55] sil2100, ^? [09:56] sil2100: yes, now [09:59] Mirv: \o/ [10:00] Saviq: so, indicators published, so the daily-release run that will start in a minute should already have the fixed libusermetrics, so unity8 should pass now [10:00] Saviq, getting a weird error assigning a deligate to a ListView, mind giving your thoughts? [10:00] sil2100, probably won't, as we still need the unity fix [10:00] sil2100, and unity won't publish before https://code.launchpad.net/~nick-dedekind/unity/duplicate.categories/+merge/180511 [10:00] nic-doffay, details, please [10:01] Saviq, https://pastebin.canonical.com/95961/ [10:01] First line is the error. [10:01] You'll see the model is just an int for now. [10:01] And the delegate is the option selector. [10:01] nic-doffay, you need to wrap in Component { } [10:01] nic-doffay, if you don't assign to "delegate" directly [10:01] nic-doffay, but really [10:02] nic-doffay, in that case you don't need "delegate: " at all [10:02] nic-doffay, IIRC the child of ListView is the delegate by default [10:03] nic-doffay, or maybe not, and anyway it's better to be explicit [10:03] nic-doffay, also it's recommended that you put the Component { OptionSelector { } } outside of ListView { } [10:03] nic-doffay, see http://qt-project.org/doc/qt-5.0/qtquick/qml-qtquick2-listview.html#details [10:05] Saviq, cool. Thing is it's still only displaying one selector now. [10:05] Even though the model is clearly set as 3. [10:05] nic-doffay, Component { id: foo OptionSelector { } } ListView { model: 3; delegate: foo } [10:05] Mirv, is that change enough for daily release ? [10:06] Saviq, yeah that's what it looks like atm. [10:06] nic-doffay, that creates 3 selectors for sure [10:07] nic-doffay, code, please? [10:07] Saviq, https://pastebin.canonical.com/95962/ [10:08] The cos recommended not manipulating contentY/X directly so I'm redoing the scrolling. [10:08] there's something broken in the filter grids too :-/ [10:08] Saviq, that's the reason for redoing it with a model is correct because it can be provided from the backend. [10:09] i can easily reproduce this [10:09] FilterGrid Model 26 [10:09] FilterGridLimited Model 6 [10:09] FilterGridLimited Items 13 [10:09] Saviq, at least that's how I feel atm. [10:09] why the view has 13 items if the model only has 6? [10:09] eh! tellme! why! [10:09] mhr3: again. https://code.launchpad.net/~nick-dedekind/unity/duplicate.categories/+merge/180511 [10:10] woops. guess i missed daily [10:11] Saviq: any idea when unity8 release will be? [10:11] nic-doffay, your ListView doesn't have enough height [10:11] nic-doffay, to show more than one [10:12] om26er: not yet, the stack config needs to be 'redeployed', and it can't be done while the current run is running. me or sil2100 will do that in eg. two hours or so. [10:12] nic-doffay, height: contentHeight [10:12] Mirv, that's fine [10:12] Mirv, last review https://code.launchpad.net/~om26er/cupstream2distro-config/ci_for_qt-eds/+merge/180515 [10:12] dednick, thx, approved [10:13] Mirv, adds CI for qtorganizer5-eds [10:13] nic-doffay, you should avoid manipulating ListView.contentY directly, btw [10:13] om26er: ok, will check [10:14] nic-doffay, if possible, use positionViewAtIndex() [10:14] Saviq, when I add that there are errors about a binding loop. [10:14] Saviq, yeah that's what I'm experimenting with atm. [10:14] Or positionViewAtEnd [10:14] I was planning to see what worked best. [10:14] nic-doffay, AtEnd will position at end of the whole ListView [10:14] nic-doffay, so AtIndex is what you most probably want [10:15] Saviq, then I'll just position it at the last index using the other function. [10:15] Saviq, but back to the height issue. [10:15] nic-doffay, but anyway, just anchor your ListView's bottom to the parent, that's what you need anyway in the end, no? [10:15] Saviq, I'm not sure what you mean. [10:16] nic-doffay, somehow you need the ListView to have a height [10:16] nic-doffay, it doesn't assume a height by default [10:16] Saviq, you mean use the anchor instead of setting it to the contentHeight? [10:16] nic-doffay, contentHeight is the height of all the items in the ListView [10:17] nic-doffay, it's rarely the right thing to use [10:17] nic-doffay, it was just a quick'n'dirty approach from my side to show what's the issue [10:17] nic-doffay, I don't know what's the height you need [10:17] other than != 0 [10:19] Saviq, setting the bottom anchor was def the best way. [10:19] ta [10:19] nic-doffay, told you so ; [10:19] ;) [10:27] Saviq, tsdgeos, mzanetti: Is there a _lean_ way to have a single method (providing the "reply" for the snap-decision dialogs) exposed on DBus using QDBus without the need to deal with xml-files and qdbusxml2cpp? [10:28] MacSlow: well, you don't need the xml stuff [10:28] MacSlow: that's just a helper to generate the stub. If you think you're confident enough with D-Bus and QtDBus, feel free to write it yourself [10:29] MacSlow: see http://bazaar.launchpad.net/~mir-team/unity-mir/trunk/revision/17 on how to expose stuff using qtdbus [10:29] mzanetti, I have what I want working in Python (as proof of concept) and now want to get it into the notification-backend [10:29] tsdgeos, looking... [10:30] MacSlow: what's the problem with writing down the D-Bus interface in xml and then generating a Qt class you can inherit from? [10:31] mzanetti, nothing really... just wanted to avoid touching the build-steps [10:33] MacSlow: you shouldn't call the qdbus2xml in the build steps anyways [10:34] MacSlow: only manually once, and then committing the resulting .cpp/.h files. It's really just a helper to generate the class for you out of a D-Bus interface description [10:36] +1 [10:37] mzanetti, I tend be used to hook such things up into the build-scripts... won't do it [10:37] Saviq, btw... were you ok with my last summary on the extended snap-decisions? [10:38] MacSlow, well, I'm still not sure it will provide us enough flexibility... [10:38] MacSlow, as I mentioned earlier, I don't think sending an "updated" notification with the whole list every time the list of AP changes [10:38] MacSlow, will provide for good UX [10:39] Saviq, I rather want a green light so I don't have to redo it again [10:39] MacSlow, that's why I was opting for the MenuModel approach [10:40] MacSlow, sending *some* things through as hints I'm ok with [10:40] MacSlow, but the list of APs with their signal strength... I'm really not sure that's going to cut it [10:41] MacSlow, and as for "replying" to the notification - a single action in the MenuModel that you'd submit the data to would be good enough, too [10:41] MacSlow, although I remember you wrote that there's issues with invoking an action with parameters / arguments? [10:41] Saviq, correct [10:41] i.e. not supported yet in UnityMenuModel? [10:42] Saviq, if that's not in there yet I'd be stuck [10:43] MacSlow, and since larsu isn't here, I doubt it's "there" [10:43] Saviq, who'd add that? Do I have to patch it then?! [10:43] MacSlow, afaik larsu is working on UMM [10:43] but he's on holidays [10:44] * Saviq checks [10:44] or on debconf [10:44] hmm nothing in cadmin [10:45] Saviq, pressure is pretty high for these extended snap-decisions... so I guess I've to further swim into unfamiliar territory to get it done [10:45] MacSlow, I know [10:45] MacSlow, in absence of the "submit action with parameters" [10:46] MacSlow, we can go with the custom interface you're doing now [10:46] MacSlow, unless you want to dig into UMM [10:46] jamesh: a review for you, last one before getting it into the archive I guess? https://code.launchpad.net/~sil2100/mediascanner/version_bump/+merge/180531 [10:46] MacSlow, but really I'd rather not send the list of APs as hints, but just point at a MenuModel [10:47] MacSlow, and have that MenuModel updated [10:47] Saviq, can we briefly chat about the custom interface (which is really small... just one call) after the stand-up? i want to avoid following a path that's not going to be ack'ed [10:47] MacSlow, I think a(ss) is good enough [10:48] MacSlow, i.e. [('username', 'foo'), ('password', 'bar')] [10:48] [('ap', 'whatever-ap-id-wanted')] [10:48] can someone do quick review (typo fix)? https://code.launchpad.net/~nick-dedekind/unity8/indicators.network.password/+merge/180530 [10:49] dednick, there's a bug for it, right? [10:49] dednick, can you link? [10:49] ^ fixes network password dialog not appearing. [10:49] ah [10:52] Saviq: done [10:53] dednick, yup, thanks === MacSlow is now known as MacSlow|lunch [10:55] Mirv: since jamesh seems to be out, could you review and approve this one? It's a typical packaging change more-or-less: https://code.launchpad.net/~sil2100/mediascanner/version_bump/+merge/180531 [10:56] Saviq: what is the status of unity8 release? thostr_ is pretty eager we get this indicator show on the road. [10:56] jamesh: btw. when building mediascanner on my system, sometimes some unit-tests are failing [10:56] jamesh: usually a re-run helps, but still [10:57] dednick, thostr_, we should be able to go through upstream merger again now that libusermetrics is fixed [10:57] dednick, thostr_, now we're only waiting for a unity release that fixes a libunity-core crash [10:58] Saviq: howcome we need to wait for that before removing the battery tests? [10:58] dednick, because we can't get through release at all [10:58] Saviq: or is it just removing the service files that is the issue? [10:58] dednick, so even if we merge your things [10:58] dednick, we can't release [10:59] dednick, (now we can merge, btw, we couldn't until usermetrics was fixed...) [11:00] sil2100: opening a tab for that too... :) [11:01] Saviq: just fyi, we're going to be doing the sound indicator as well. even though chewie sound will still be running. [11:01] dednick, k [11:03] sil2100, any idea when will libusermetrics get into the archive? [11:04] Saviq: it's in already, at least the one with the dbus script failing [11:04] https://launchpad.net/ubuntu/+source/libusermetrics [11:05] sil2100, ok, probably pl mirror isn't there yet [11:05] Saviq: probably... [11:06] Mirv: I think we also encountered a silly race as well, since the latest indicator stack is in manual publishing because it wants to re-release the change that already got released [11:06] Mirv: probably because it wasn't yet synced completely when indicators re-ran [11:06] sil2100, hey which ppa contains the staging packages that are tested for daily release ? [11:06] Mirv: so let's just ignore the indicators stack for this run [11:07] om26er: ppa:ubuntu-unity/daily-build is what you want [11:07] sil2100, thanks [11:07] om26er: but each stack only uses the packages it needs from this PPA [11:07] om26er: i.e. there is no dist-upgrade being done from this PPA during testing [11:08] om26er: just selected, needed packages are being upgraded from there [11:08] sil2100, ok, I just want to grab the latest unity8 from there [11:08] om26er: just remember that daily-build is very risky ;) [11:08] sil2100, not even the main mirror is "there" yet :P [11:08] sil2100, I am not adding the ppa, just downloading the debs ;) [11:09] Saviq: pff ;p It got in 25 minutes ago, so I would expect it to be in already! [11:09] sil2100, indeed ;) [11:09] sil2100: ok, makes sense, there are always such delays [11:10] sil2100: that's why we normally never run the manual runs anymore [11:11] sil2100, ah, jenkins picked up the new version, fortunately [11:11] jeez, something's still adding qt5-proper to qmluitests... [11:12] mzanetti, can you have a look ↑ [11:12] Saviq: sure [11:18] greyback: hi! Does unity-mir have some/any integration tests? Is it possible to do some? [11:19] sil2100: hey, it's totally possible to do some, I want to start at that next week when we've the basics in place. [11:20] but for now, I've been a bad boy ;) [11:20] Saviq, did anyone notice that dns resolving is blocking the rendering thread? [11:20] mhr3, huh? [11:20] Saviq: with bregma offline, who else could we ping bug #1212945 about? [11:20] bug 1212945 in Unity "A surge of failing autopilot tests in unity" [Critical,New] https://launchpad.net/bugs/1212945 [11:20] Mirv, I believe that's fixed already, right mhr3, dednick ↑? [11:21] ah, there's a check run now running so if that's so, nice and we should have the result maybe in 0.5h [11:21] Mirv, we're just waiting for it to land and release [11:21] Mirv, doubt it, it's not merged yet [11:21] mhr3, dednick, why isn't the unity fix merged yet ↑? [11:21] :o [11:21] Saviq: the last commit is from 9 hours ago, so not [11:22] Mirv, http://s-jenkins:8080/job/unity8-autolanding/ [11:22] wrong [11:22] http://s-jenkins:8080/job/unity-autolanding/ [11:22] Mirv, it's landing [11:22] no idea. it's been approved by ci... [11:22] if it's this, it's just jenkins delay https://code.launchpad.net/~nick-dedekind/unity/duplicate.categories/+merge/180511 [11:22] Mirv, yes [11:22] just linked the bug [11:23] dednick, yeah, it's being merged [11:23] greyback: hmm, since I would feel a bit reluctant to enable daily-release for unity-mir before it has any integration tests [11:23] greyback: but on the other hand, FF is close so we'd need to have it released at least once... [11:23] thanks dednick [11:24] sil2100: Unfortunately next week is best I can do [11:24] if the merge happens in the next 20mins or so, I'd vote for rerunning unity stack only (next cycle in 2.5h, it'd be ready for the unity8 run in theory by then) [11:24] Mirv, +1 [11:24] Mirv, but it will run it anyway, no? or is unity not a dependency of unity8? [11:25] Mirv, or if nothing changes, there will be no run of the unity stack? [11:25] Saviq: based on this cycle unity is not marked as dependency, ie unity8 finishes before unity has finished [11:25] interesting... [11:26] Saviq: so unless manually run, the next cycle unity would be (possibly) published but unity8 not still [11:26] * Saviq needs to talk to didrocks ;) [11:26] by all means :) [11:27] Saviq, i'd know cause my internet decides to not work from time to time [11:28] mhr3, and unity8 gets stuck does it? [11:28] yep [11:28] greyback: let's wait till next week then, I'll discuss it with Mathieu as well [11:29] mhr3, only place where unity8 accesses the interwebs... remote scopes? [11:29] sil2100: ok [11:29] Saviq, while you are reporting bugs, here is one more :) bug 1213038 [11:29] bug 1213038 in Unity 8 "running apps thumbnail becomes black if dash is scrolled down" [Undecided,New] https://launchpad.net/bugs/1213038 [11:29] om26er, dupe ;P [11:29] om26er, assuming I can find it ;) [11:30] om26er, bug #1193099 [11:30] right, its been a while. someone else must have [11:30] bug 1193099 in Unity 8 "Unity: thumbnails for running apps are not cached" [Medium,Triaged] https://launchpad.net/bugs/1193099 [11:30] Saviq, easy to see with the click scope [11:30] Saviq, let me make a vid [11:30] man, if I only had searched for 'thumbnails' would have found that [11:31] om26er, no worries, I had to remember to search for "cached" ;) [11:34] mhr3, unity8 crashes if I try to use smart scopes like: 'music: that song' should I file a bug for that ? [11:38] anyone here not using xmir? Does gnome-screenshot work for you? All I get is black screengrabs [11:43] Mirv, sil2100 unity landed, can we restart Unity stack plz? [11:43] tsdgeos: fancy a review? https://code.launchpad.net/~mzanetti/unity8/dnd-and-quicklists/+merge/176653 [11:43] Saviq: sil2100: will handle [11:43] greyback, yeah, works here [11:43] Saviq: the previous build is still running, but I think I could abort it and re-run [11:44] Saviq: thanks for checking. Will log [11:44] sil2100: yes, that's what I'm doing [11:44] Saviq: is unity the only thing which needs re-building? [11:44] Mirv: ah! [11:44] sil2100, afaik, yes [11:44] :) [11:44] Mirv: sorry, I misunderstood to as "sil2100 will handle" [11:44] Mirv: while it was more like a declaration 'I will handle it', right? [11:44] sil2100: ooops! :) I should be more precise :) [11:44] Confusing Friday! [11:45] sil2100: indeed! [11:45] mzanetti: wow that one's big :D [11:45] if only confusing [11:45] tsdgeos, big? measly 1.2kLOC [11:45] mzanetti: sure i'll handle it if noone does [11:45] ;) [11:45] tsdgeos: 1/3 of it is moving code [11:45] but yeah. the rest is still quite tricky stuff [11:45] definitely the most tricky thing I did with QML so far [11:45] still struggling to understand why we get the LVWPH stuff with holes at the top [11:46] sometimes [11:46] holes? [11:46] mzanetti, oops, got a loop of "LauncherPanel.qml:357: TypeError: Cannot read property 'height' of undefined" [11:46] mzanetti, after having dragged an item past the top [11:46] Saviq: right. I know this one [11:47] * tsdgeos gets some food, let's see if that helps clearning the mind [11:47] Saviq: thing is, if you drag an item outside the list where the items don't have delegates yet, it switches content and suddendly your delegate gets destroyed [11:47] mzanetti: btw you might be interested that I've (and upstream has) resolved qtlocation build problems there used to be and we're on track for a good new snapshot (added with Ubuntu plugin as well). OSM plugin included too. [11:47] Saviq: only thing would be to set cachebuffer to infinite which is not what I wanted [11:48] Saviq: so what I did is to make the Launcher's margins at least big enough so you can't cause this when running unity fullscreen [11:48] you still can break it when running windowed tho [11:48] mzanetti, got it [11:48] Saviq: LauncherPanel.qml line 105 is the comment that explains it [11:48] Mirv, sil2100 can you allow "greyback" the right to rebuild jobs in s-jenkins? [11:49] Mirv: wohooo! that's awesome stuff! [11:49] Saviq: I think Francis will have to pop up to do that sadly ;/ [11:49] Mirv: I'll probably check it out this weekend [11:49] sil2100, ok [11:50] I'll be off for an hour for lunch [11:50] mzanetti, enjoy [11:52] mzanetti: saucy build at ppa:ubuntu-sdk-team/ppa [11:52] greyback: i get black screenshots sometimes.. with xmir. [11:54] popey: only sometimes? https://bugs.launchpad.net/mir/+bug/1213047 [11:54] Launchpad bug 1213047 in Mir "[xmir] gnome-screenshot only gets black image" [Undecided,New] === alan_g is now known as alan_g|lunch [11:55] greyback: yes [11:56] greyback: maybe related to the fact I get desktop crashes now and then, might be black after that's happened [11:56] popey: very strange. You mutter a secret incantation perhaps? Maybe mention that in the bug [11:56] popey: for me I've just freshly booted [11:56] i press a magical key which has strange glyphs on it greyback [11:56] it's called "PrtSc" [11:56] * greyback reels back in horror [11:56] I know, right?! [11:56] non Mac user, gah! [12:01] greyback: please run apport-collect 1213047 [12:01] screenshot has worked fine here so far [12:01] but probably same as popey ie sometimes black, I just haven't used it much === MacSlow|lunch is now known as MacSlow [12:03] Mirv: "No additional information collected" [12:03] greyback: :P [12:04] Saviq: still crashing ? https://jenkins.qa.ubuntu.com/job/generic-mediumtests-runner-saucy/1959/testReport/unity8.shell.tests.test_lock_screen/TestLockscreen/test_can_unlock_passphrase_screen_Desktop_Nexus_4_/? [12:04] dednick, yes, Unity still not released [12:04] Saviq: ah right... [12:04] dednick, we just restarted Unity stack [12:05] Saviq: i keep on thinking of unity8 as unity :) i realise now what you are meaning "waiting for release" [12:05] :) [12:06] dednick, we (unity8) are not even a dependant stack (of unity) for some reason, need to talk to didrocks when he's back [12:07] my labels don't wrap... [12:07] I put wrapMode: Text.WordWrap [12:07] no joy.. [12:08] nevermind, was width/height [12:09] greyback: not that I know anything, but could you try again? I added the Ubuntu package there as well in addition to upstream project, maybe that guides it [12:10] Cimi, I agree, there should be a warning when you use Text with wordwrap and no explicit height [12:10] Cimi, same for ListView, for example === dednick is now known as dednick|lunch [12:11] Mirv: I need to do an update? [12:11] Mirv: all I get now is: "Package mir not installed and no hook available, ignoring" [12:31] greyback: mmkay, like I said not that I know anything, but at least that was progress, dunno how to go from there though [12:31] greyback: when I filed a bug against unity-system-compositor, apport-collect worked [12:32] Mirv: ok. I'll update, might help [12:32] om26er: oh right.. since we don't have didrocks and we lack the required information, we can't publish any new source packages like ubuntu-keyboard before Monday [12:33] greyback: it might be something to bring up on #ubuntu-mir, that bugs filed against 'mir' don't collect meaningful data [12:34] Mirv: ack [12:43] Mirv, oh? it was really needed to land today, both of them :/ [12:44] om26er: so far no-one including archive admins have not been aware what's the 'whitelist' didrocks adds new cu2d source packages to [12:46] om26er: so even if we did get them built in the daily release system, we couldn't release them into saucy archives [12:46] Mirv, ok, I get it. [12:46] I do think for future the "art" shall be transferred to more people [12:47] om26er: yes, that has been a topic today and yesterday, that art needs to be shared :) [12:49] om26er: the 'misc' stack has been now redeployed however, so the ubuntu-keyboard will be tried to be build in the next run in a little over hour [12:49] it will then stay unreleased like the poppler-qml-plugin in sdk stack which was added yesterday [12:50] Mirv, cool, we are slowly making progress :) there was also the eds plugin, i'd assume that's included as well [12:51] om26er: ah yeah, that's there too now, in the stack (http://10.97.0.1:8080/view/cu2d/view/Head/view/Misc./) [12:54] greyback, I retargeted unity-mir bp to ubuntu to inherit the milestones [12:54] greyback, and moved all items under ubuntu-13.08, feel free to move them around if needed === _salem is now known as salem_ [12:55] greyback, this way it shows up in +upcomingwork [12:55] /we need milestones for unity8 [12:55] Saviq: ack [13:05] kenvandine: ! === dednick|lunch is now known as dednick [13:21] sil2100, is that looking good enough for publishing http://10.97.0.1:8080/job/autopilot-saucy-daily_release/1040/ ? [13:22] Saviq: so far so good, let's wait for ati [13:22] sil2100, \o/ [13:23] Saviq: if ati fails, I'll force the publishing anyway [13:23] There's not much time left though, so we might have the same problem as with libusermetrics ;/ But oh well [13:23] tsdgeos, are you doing mzanetti's launcher dnd review in the end? or fighting with FilterGrid / LVWPH? [13:24] Saviq: having a look at the LVWPH "jump" [13:24] tsdgeos, ok, I'm taking on the launcher, then [13:25] oka [13:35] Saviq: standup! [13:35] Saviq: we need two core-dev acks again, unity would be ready otherwise, 25mins of time [13:38] Saviq: correction, one needed, libunity http://pastebin.ubuntu.com/5992746/ [13:39] mterry: doesn't nested-mir just create a mir surface, why does compositor care? [13:40] mzanetti: prep for presentation on Qt devdays [13:40] right [13:42] greyback, it had something to do with the compositor not registering sub mir servers as sessions? Like, I need to be able to switch between the greeter and the user sessions, and the compositor doesn't seem capable of that yet [13:42] mterry: as you're up, would it be possible for you as a core-dev to review / give an ack for pastebin.ubuntu.com/5992746/ / libunity packaging change? [13:43] mterry: ok, forget that, ogra approved, thanks [13:43] Saviq: published! [13:43] mterry: hmm, ok [13:43] sil2100, yay [13:44] great, at least one step of progress. with luck unity8 will be green after the next cycle as well, and Apps probably too [13:44] Saviq: it will probably take some time for it to sync up everywhere, but at least we have more probability that it'll be around for the next run [13:45] Saviq: deosnt look like it's been released yet though [13:45] dednick: it is, now, 15s ago (pushed) [13:45] dednick, probably just not synced yet === alan_g|lunch is now known as alan_g [13:45] dednick, https://launchpad.net/ubuntu/+source/unity [13:46] refresh refresh :) [13:46] for a couple of minutes at least [13:46] dednick, so what do I do to test that the new indicators work? [13:47] Saviq: just install them. they should pop up. "may have to reboot phone" [13:47] dednick, indicator-{datetime,power,sound}? [13:48] Saviq: yep [13:48] Saviq: and prepare to run out of panel space [13:48] dednick, ;) [13:49] Saviq: can remove the other indicator files as it pleases (com.canonical.indicator.time, com.canonical.indicator.battery, com.canonical.settings.sound) [13:49] sil2100, another quarternourly in 10 mins or in 2:10? [13:49] In 10 minutes [13:50] Saviq: that's why we were so crazy about getting that approved and published [13:50] sil2100, right, looking at fast-tracking https://code.launchpad.net/~nick-dedekind/unity8/remove.battery.time.sound.indicators/+merge/180375 then [13:51] good night everyone :) [13:52] Mirv, o/ [13:52] dednick, sound slider doesn't seem to work on the device? [13:52] Saviq: as in changing volume or enabled? [13:52] now there https://launchpad.net/ubuntu/+source/unity/7.1.0+13.10.20130816.3-0ubuntu1 , soon in proposed pocket, just on time [13:52] dednick, changing volume [13:52] um. [13:53] Saviq: is the chewie one working? [13:53] dednick, not, indeed [13:55] Saviq: i wonder if it's because we're running parallel with chewie... [13:56] Saviq: mute works only on new one... [13:56] dednick, indeed [13:56] dednick, fast-tracked merging your branch [13:56] dednick, let's hope it makes it in time ;) [13:57] Damn, still no libunity in proposed/release [13:57] Saviq: we need to change the seed as well. otherwise we wont have any indicators :) [13:57] dednick, that's fine, as soon as we actually make a release ;) [13:58] ci approved. [13:59] Saviq, can you ping me when you have a moment to chat about the filters back end a bit more. [13:59] Saviq: wow. merged already! [13:59] nice [14:00] mzanetti, ping [14:00] mzanetti, no pressure just wondering if you had a look or not [14:00] at the autopilot thing [14:01] dednick, well, CI didn't *really* approve - I made it approve ;) [14:01] Saviq: yeah, i saw that [14:01] Cimi: it should be fixed [14:02] Cimi: since Wednesday I think. Let me check the state of the MR [14:13] Saviq: can you see if https://code.launchpad.net/~aacid/unity8/lvwph_weirdness fixes the "big hole on the middle when running in fullscreen" you get? [14:13] tsdgeos, sure, checking [14:13] i think it's a problem with the flickable code [14:13] but still need to read it a bit more [14:15] tsdgeos, yes, that fixes the fullscreen issue (not the others) [14:15] Saviq: hmmm [14:16] Saviq: the other is the one that you scroll a lot and gets somehow stuck? [14:16] tsdgeos, +1 [14:16] i wish i could repro that :/ [14:16] tsdgeos, it also makes the categories "consistently" scroll $section_header_height up [14:16] on first input [14:16] Saviq: that's good, no? :D [14:17] tsdgeos, s/on first input/on showHeader/ [14:17] tsdgeos, when I launch and start scrolling to the side [14:17] tsdgeos, it scrolls the view a height of the section header up [14:17] tsdgeos, like showHeader() is doing something weird [14:18] you're going to have to send me your comptuer :D [14:18] Saviq: that happens all the time? [14:18] tsdgeos, on first start [14:18] tsdgeos, ;) [14:18] what do you mean on first start? [14:18] tsdgeos, I'm going to Barcelona in a week, remember? :D [14:18] next time you start it, it doesn't happen? [14:18] tsdgeos, rephrase: "on start" [14:19] ok [14:19] tsdgeos, the first time I swipe to the sides [14:19] so every time you start it [14:19] tsdgeos, yes, that also makes the "stuck in the middle" issue not happen [14:19] tsdgeos, i.e. if I first drag to the sides [14:20] tsdgeos, can't get it to be stuck in the middle [14:20] interesting [14:20] tsdgeos, so showHeader() effectively prevents that from happening [14:20] well, i guess i can write some code to output the state of the list [14:20] and then try to repro it here [14:21] it's not much [14:21] but let me finish with the fullscreen tihing first [14:26] Saviq: is there a daily release in progress for unity8? [14:28] dednick, quarternourly, yes [14:28] dednick, http://10.97.0.1:8080/view/cu2d/view/Head/view/Unity8/ ;) [14:28] dednick, you can also see in https://launchpad.net/~ubuntu-unity/+archive/daily-build [14:29] dednick, as it's actually building in that PPA [14:29] thanks. was looking for that dash [14:38] Saviq: for the same bug, can you check https://code.launchpad.net/~aacid/unity8/lvwph_weirdness2 ? [14:38] it makes me more confortable as a bugfix [14:39] tsdgeos, this seems to fix more! [14:39] tsdgeos, doesn't get stuck anymore [14:40] tsdgeos, nor does it scroll on showHeader [14:40] goodie [14:40] so 1 fix to rule them all? [14:40] :D [14:40] tsdgeos, almost everything [14:41] tsdgeos, only remaining one is related to the filtering [14:41] ah yes [14:41] the ghost entries? [14:41] or another one? [14:41] yes [14:41] i don't think that's LVWPH itself [14:41] most possible [14:41] it's the filter [14:41] do you have a screenshot? just to be sure we're talking about the same thing? [14:41] tsdgeos, tricky to get a screenshot [14:42] tsdgeos, but I end up with 3 rows of items in "Dash plugins" [14:42] ok, let me try to get one [14:42] tsdgeos, but only two of them are "inside" the ListView [14:42] tsdgeos, so only visible with overshooting [14:42] tsdgeos, uh oh [14:42] anyone knows what did we break so that screen grabbing doesn't work anymore? [14:43] tsdgeos, xmir? [14:43] afaik no [14:43] tsdgeos, and now the header got stuck in the middle for me... [14:43] greyback, ↑↑↑ [14:43] greyback, might not be xmir in the end [14:43] Saviq: :( [14:44] * tsdgeos takes a picture :D [14:44] tsdgeos: all you get is black screen? [14:44] tsdgeos, http://ubuntuone.com/2GGkY941W8HV03IMpr6HrB [14:44] yep [14:44] greyback: ↑↑ [14:44] tsdgeos: darm, ok [14:44] Saviq: wooooa [14:44] tsdgeos, relatively easy to repro: go to apps, expand dash plugins, search, unsearch, search quickly [14:45] how...?? [14:45] tsdgeos, or generally search quickly [14:45] Saviq: that with the new code [14:45] right? [14:46] tsdgeos, yes [14:46] tsdgeos, not a regression, same in trunk [14:46] oh [14:47] well, at least is not worse because of the fix :D [14:49] * greyback feels dirty using QRegularExpression [14:51] rotfl [14:51] mzanetti, https://code.launchpad.net/~mzanetti/unity8/dnd-and-quicklists/+merge/176653/comments/408385 [14:51] greyback, what is so bad about regexes :D [14:52] Saviq: they always feel a bit like magic, they work, but if not careful will bite me in the ass [14:53] sil2100, ;( unity8 check crashing, do we know if the new libunity-core got there? [14:53] *get [14:54] sil2100, should we just force it in our dependencies/ [14:54] ? [14:54] Saviq: the ghost thing would be http://i.cubeupload.com/lOEq65.jpg right? [14:54] like the extra row in Music [14:54] Saviq: ah, shit, wait, we'll probably have to re-run with check-with-whole-ppa [14:54] tsdgeos, yeah, sounds right [14:54] Saviq: give me a moment [14:54] sil2100, thanks [14:54] ok, yeah that's filtergrid or something below [14:54] i'll have a look too [14:54] but let's fix LVWPH first [14:54] +1 [14:55] tsdgeos, it's a huge improvement already [14:55] Saviq: it's in proposed, so it should be fetchable [14:55] Saviq: i need a unittest though [14:55] not sure how to create one [14:55] sil2100, yeah, but we've not requested the new version explicitly [14:55] sil2100, so it used whatever was there already? [14:55] Saviq: thanks. [14:55] tsdgeos, and a quick workaround for the ghost ones is a "clip: true" away [14:55] mzanetti, cheers [14:55] Saviq: sure :D [14:55] ;) [14:57] olli_, ping [14:57] Saviq: the thing is that we're not explicitly upgrading libunity when running unity8 tests, since we rely on what's on the iso - the iso will be upgraded tomorrow, so until tomorrow, we need to explicitly tell the unity8 stack - build but first do a dist-upgrade [14:57] Saviq: that's what I'm doing now [14:57] sil2100, ok, that works [14:57] Saviq: let's see how it goes now [14:57] sil2100, could we maybe do that by default? [14:57] sil2100, we're depending on unity anyway [14:57] Saviq, pong [14:57] sil2100, so not upgrading it feels error-prone [14:58] Saviq: I guess it might be a thing to consider, right [15:03] sil2100, looks green! [15:05] \o/ [15:05] Yessss [15:05] Saviq: the future runs might be red though again, but hm, maybe we'll consider adding the libunity dep there [15:07] sil2100, yup [15:12] Saviq, so I need the wifi thing, let me copy as much as possible from indicators and propose a plugin for the system settings? [15:12] Cimi, talk to seb first, he has a plan for that [15:12] Saviq, he's not online... [15:12] Cimi, and I don't know exactly what it is [15:13] Cimi, I know, you won't be, soon, either [15:13] Cimi, review something, or do something else, just don't do that, which can be in a completely wrong direction [15:13] ahah ok [15:19] Saviq: wait, unity8 isn't completely green ;p [15:19] Saviq: we need to publish it! [15:19] sil2100, already pung ogra about it in #ubuntu-touch [15:19] although might not have been explicit enough [15:20] sil2100, so you go :) [15:21] Maybe he's busy, so I'll also poke kenvandine [15:22] yup [15:24] Saviq: to be honest i've no clue on how to create a test for that bug i just fixed :-/ [15:24] tsdgeos, I can imagine... [15:25] * tsdgeos keeps thinking [15:25] tsdgeos, maybe a higher-level one, then? [15:25] Saviq: took me a while but I make the folding sequential to the expanding. [15:25] mzanetti, *made? [15:25] yes [15:25] Saviq: will fix/answer your comments now [15:25] mzanetti, cool [15:27] Saviq: that's what i'm trying .D [15:27] tsdgeos, I get it that's it's a tricky one - as it probably incorporates changing the model under the LVWPH? [15:28] so it's effectively a race? [15:28] it's kind of a race yes [15:28] but on a test i control everything [15:28] so it's ok [15:29] i just need to reproduce it :D [15:32] Saviq, https://code.launchpad.net/~mhr3/unity8/hide-ratings/+merge/180596 [15:33] Saviq, see what have you done? me doing qml... eek :P [15:33] hahaha... [15:33] HAHAHA... [15:33] HAHAHAHHAHAHAHAHAHAHAHAHA! [15:33] your evil masterplan worked [15:33] * tvoss_ hugs mhr3 [15:33] welcome to the dark side [15:34] tvoss_, i hope there are cookies at least === jhodapp is now known as jhodapp|lunch [15:36] mhr3, yeah, it's nice over here :) [15:36] mzanetti, updates? [15:37] mhr3, https://code.launchpad.net/~mhr3/unity8/hide-ratings/+merge/180596/comments/408421 [15:38] Saviq, why not? [15:38] new property? [15:39] mhr3, 'cause properties on top-level components are something like an API [15:39] mhr3, and they use memory [15:40] are all those signal connections cheaper? [15:40] although.. you need to do those anyway [15:41] Cimi: yeah, its fixed and Jenkins approved it [15:41] Cimi: there was another crash which should be fixed by now by another merge [15:41] mzanetti, we need reviews then :) [15:46] Saviq, to set the full name, is it ok to use command line tool like chfn ? [15:47] Cimi, there's probably a library for that [15:48] Cimi, and seb will know what it is :) [15:48] ok [15:49] Saviq, do you want to remove also the canBeRated prop? [15:50] mhr3, I'd rather get rid of it completely indeed ;) [15:50] mhr3, but that would result in 4 comparisons instead [15:51] mhr3, so at least move it down somewhere [15:51] mhr3, so it's not exposed on the root component [15:51] Saviq, i was trying to group the 4 components as a single item, but that just broke the layout [15:52] mhr3, yeah, let me have a look actually [15:53] mhr3, so we're hiding "Rate this" and "Add a Review"? [15:53] Saviq, also the reviews [15:53] "Comments" [15:53] mhr3, so effectively everything below the description? [15:53] yep [15:53] mhr3, should be easier than that [15:53] * Saviq looks [15:55] mhr3, btw, you broke tst_AppPreview with that [15:55] of course i did [15:59] mhr3, something along the lines of http://paste.ubuntu.com/5993239/ [15:59] mhr3, but it's not correct yet [15:59] but you'll get the drift [16:00] hmm, childrenRect... that's the secret [16:02] Saviq, the layout is still a bit broken [16:03] Add a review is on top of some other label [16:05] mhr3, that's why I said "it's not correct yet" [16:06] i thought it's because of the test still not passing :) [16:09] Saviq: replied [16:16] Saviq: so i have a ultra-high level test [16:16] Saviq: http://paste.ubuntu.com/5993321/ [16:16] what you think? [16:17] it's not awesome [16:17] but it's something [16:19] Saviq, tsdgeos we need reviews here :) https://code.launchpad.net/~unity-team/unity8/unity8.background-gsettings-fix/+merge/179884 [16:19] mzanetti, said the bug with autopilot is fixed [16:19] without , [16:24] tsdgeos, looks good enough :) [16:31] * mzanetti -> EOD [16:31] o/ [16:31] o/ [16:33] Saviq: so https://code.launchpad.net/~aacid/unity8/lvwph_weirdness2/+merge/180613 is with the test [16:34] tsdgeos, ok, will check it out later [16:34] * Saviq EOD [16:34] o/ [16:34] \o/ [16:35] o/ [16:38] sil2100, ping [16:45] fginther: ping === jhodapp|lunch is now known as jhodapp [16:50] * greyback eow [16:50] bye all === alan_g is now known as alan_g|EOD === bschaefer_ is now known as bschaefer [20:50] smart scopes in Unity 8 rock almost as much as you guys! === salem_ is now known as _salem