/srv/irclogs.ubuntu.com/2013/08/16/#ubuntu-unity.txt

=== olli_ is now known as olli
MirvSaviq: something fishy in Unity8 compilation are actually dependencies installation https://bugs.launchpad.net/unity8/+bug/121294405:54
ubot5Launchpad bug 1212944 in Unity 8 "unity 8 FTBFS (during package installation)" [Critical,New]05:54
SaviqMirv, hmm, looks like usermetricsservice configuration failed06:57
SaviqMirv, "Setting up usermetricsservice:i386 (1.1.1+13.10.20130815.2-0ubuntu1) ...06:59
SaviqFailed to open connection to "system" message bus: Failed to connect to socket /var/run/dbus/system_bus_socket: No such file or directory"06:59
SaviqMirv, but that shouldn't be a build dependency anyway07:00
SaviqMirv, it seems http://bazaar.launchpad.net/~unity-team/libusermetrics/trunk/revision/125 is the culprit07:03
SaviqMirv, it has a .postinst that requires the system bus to be there07:03
MirvSaviq: yeah, it shouldn't fail installing in case it's not07:03
SaviqMirv, yup07:04
jameshhi sil210007:13
SaviqMirv, are you doing something about it or are we waiting for pete-woods?07:15
MirvSaviq: not doing at the moment and haven't filed a bug yet as I've other urgencies ongoing07:21
SaviqMirv, I've added affects: libusermetrics to the other bug07:21
SaviqMirv, so I'll own that, then07:21
Mirvmaybe sil2100 can look at it now that he's here, after he catches up :) (thankfully he's back to help!)07:21
Saviq:)07:21
Mirvthank you07:22
mhr3Saviq, 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 taste07:27
mhr3Saviq, touching ResponsiveFlowGrid... eek :P07:28
Saviqmhr3, not sure we want to spend time on it when the Mir-backed recent apps scope should be working soon07:29
mhr3Saviq, but that's why we need it, it needs to support the regular scope model07:30
Saviqmhr3, ah, got it now07:30
mhr3Saviq, not super high on the list, but once the mir bits are in, we'll want to use it07:31
Saviqmhr3, yeah, will do07:31
mhr3Saviq, 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
Saviqmhr3, cdimage-touch07:34
Saviqmhr3, otherwise you get read-only /07:34
mhr3at least someone knows these things :)07:38
tsdgeosSaviq: what do i use to test https://code.launchpad.net/~saviq/unity8/fix-generic-preview/+merge/179932 ?08:01
Saviqtsdgeos, search for something in home08:01
Saviqtsdgeos, and long-press08:01
Saviqtsdgeos, description (if long) should be wrapped and formatted08:01
Saviqtsdgeos, wikipedia is usually useful08:01
Saviqtsdgeos, http://ubuntuone.com/0y3o0lPEaWsVBEP6gdiJdd http://ubuntuone.com/4AGFEchm7D1QqVvDtrbmZb08:02
Saviqtsdgeos, we seem to have triggered an issue in LVWPH :/08:02
tsdgeosah yes08:02
tsdgeosthere was this bug you said the other day in which it stuck in the middle?08:02
tsdgeosis this that one?08:03
Saviqtsdgeos, yeah08:03
tsdgeoshmm08:04
Saviqtsdgeos, and then something related to category expansion - go to apps, search for something, clear the search, click the "Dash plugins" category to expand it08:05
Saviqtsdgeos, stuff breaks badly :/08:05
Saviqtsdgeos, like first of all, the limiting filter gets confused and doesn't actually "pass through" the items08:05
Saviqtsdgeos, and then the whole thing just goes all over the place :/08:06
tsdgeoshmmm08:06
tsdgeosworks here?08:06
tsdgeoswhat is really weird is that i can't vertically move stuff in the home scope until i expand files & folders08:07
tsdgeoswhich kind of makes sense but doesn't at the same time :D08:08
mhr3tsdgeos, 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 visible08:08
mhr3and vice versa08:09
tsdgeosmay be08:09
mhr3pushing a new item in the first position, which should remove the last visible one08:09
dednicksup everyone? scopes still getting you down?08:10
mhr3dednick, it's all your fault!08:11
mhr3dednick, hope you feel down as well now :P08:11
dednickum. what did i do?08:12
Saviqpete-woods, https://bugs.launchpad.net/unity8/+bug/1212944/comments/108:12
ubot5Launchpad bug 1212944 in Unity 8 "unity 8 FTBFS (during package installation)" [Critical,In progress]08:12
mhr3dednick, nothing in particular, we just want you to feel bad too, so you don't feel excluded08:12
dednickoh. thanks :)08:13
mhr3anytime08:13
dednicki'm sure i had something to do with it anyway.08:13
pete-woodsSaviq: thanks for pointing that out!08:13
Saviqpete-woods, well, yeah, you caused FTBFS for unity8 ;)08:13
pete-woodsSaviq: I don't understand why dbus wouldn't be there - it's a dependency08:16
pete-woodsbut what the hey, I'll make the thing less sensitive08:16
Saviqpete-woods, it's not started, you don't want dbus on the build machines08:16
pete-woodsSaviq: okay, I suppose that's fair!08:16
tsdgeosSaviq: 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 that08:19
tsdgeos?08:19
Saviqtsdgeos, good catch, let me see08:21
tsdgeosSaviq:  but it seems like a qt bug ::/08:22
tsdgeosi've added a rectangle on the label and it's covering the whole width08:22
mhr3dednick, i think we broke something quite horribly - http://10.97.0.1:8080/job/autopilot-saucy-daily_release/1027/08:22
dednickouch08:23
jameshsil2100: hi.  What do we need to do next to get mediascanner into the archive?08:26
Saviqtsdgeos, I wonder if it's the line that's causing that08:26
tsdgeosmaybe08:26
tsdgeosSaviq: anyway it0s an improvement over what we have, so going to aprove it08:26
tsdgeosand I'll log a bug about it, ok?08:27
dednickmhr3: told you i did something08:27
dednickmhr3: yeah, i think i can see what's going on. it is my branch08:31
dednick...although08:32
dednickmhr3: no, dont know what's happening08:34
Saviqtsdgeos, hmm, the interesting thing is... should \n be considered as newlines in richtext anyway?08:34
mhr3dednick, give me a sec, standup08:34
tsdgeosSaviq: i think not08:35
tsdgeosthat's not how html behaves08:35
Saviqtsdgeos, exactly08:35
nic-doffayMacSlow, ping08:35
MacSlownic-doffay, yes08:36
nic-doffayMacSlow, do you have experience with running autopilot tests? Remember you mentioning it in standup a while ago.08:36
MacSlownic-doffay, I do have some... what do you want to run?08:36
nic-doffayThere's an issue with one of my branches and I need to sort it out.08:37
nic-doffayMacSlow, https://jenkins.qa.ubuntu.com/job/generic-mediumtests-saucy/2286/?08:37
Saviqtsdgeos, ah, I think it's caused by the fact that font size is set08:37
nic-doffayIt's related to the mouse pointer now that the OptionSelector has been added.08:37
tsdgeosSaviq: weird08:37
Saviqtsdgeos, aactually!08:38
Saviqtsdgeos, check this out http://paste.ubuntu.com/5992013/08:38
Saviqtsdgeos, it doesn't get re-laid-out if you make the window bigger08:39
tsdgeosbrr08:39
sil2100jamesh: hi! I'll be poking someone to sponsor it today into the archive08:39
MacSlownic-doffay, so you want to only run that "test_can_select_listview" locally for debugging the issue?!08:39
sil2100jamesh: since the packaging is more or less ready now08:39
jameshsil2100: thanks!08:39
jameshsil2100: 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 right08:40
Saviqmhr3, pstolowski so... richtext... can we nl2br the text? will unity7 be fine with <br/>s?08:40
tsdgeosSaviq: only if you make it smaller :/08:40
Saviqtsdgeos, exactly08:40
tsdgeosSaviq: same with 5.1 actually08:40
* tsdgeos goes on crying08:40
nic-doffayMacSlow, ideally yeah.08:41
MacSlownic-doffay, you just cd to unity8/tests/autopilot ; autopilot list unity8 ... and then look for the test you're interested in...08:42
MacSlownic-doffay, then just "autopilot run <testname>"08:42
MacSlownic-doffay, will be some looooong name08:42
tsdgeosSaviq: do you report a bug to Qt?08:42
Saviqtsdgeos, yeah, will do, trying for a workaround now08:43
tsdgeosSaviq: you can replace the Label with Text to drop Ubuntu.Components08:43
MacSlownic-doffay, sorry... make that toolkit not unity8 :)08:43
Saviqtsdgeos, doesn't matter08:43
tsdgeosSaviq: i mean for the bug report in Qt08:44
Saviqtsdgeos, right, of course08:44
nic-doffayMacSlow, great thanks!08:44
sil2100jamesh: sure thing!08:45
MacSlownic-doffay, should be... cd ubuntu-ui-toolkit/tests/autopilot ; autopilot run ubuntuuitoolkit.tests.gallery.test_gallery.GenericTests.test_can_select_listview08:45
dednickwtf is up with libusermetricoutput? it keeps on conflicting on my system.08:46
tsdgeosdednick: the thing that gives data for the welcome screen infographic08:47
dednickevery time i do an upgrade, it buggers up the install and i get a half upgraded system that is totally borked.08:47
dednicksomething with the conf file.08:47
tsdgeosweird08:47
tsdgeosworks fine here08:48
Saviqdednick, maybe you have some ppa enabled that you should not?08:50
dednickSaviq: http://ppa.launchpad.net/ubuntu-unity/next/ubuntu ?08:51
Saviqdednick, yeah, not needed / wanted08:51
Saviqdednick, for unity8 you don't need any PPAs now08:51
Saviqdednick, so anything you have, you need to know that you want it :)08:51
dednickSaviq: cool. thanks08:51
Saviqtsdgeos, ugh, can you edit the description in bugreports?08:58
tsdgeosnot sure08:59
tsdgeosSaviq: why one?08:59
Saviq"Provide more info"08:59
Saviqof course!08:59
Saviqtsdgeos, one what?08:59
tsdgeoswhich one i mean08:59
tsdgeost08:59
dednickmhr3: hm. everything looks in terms of searching in the videos...08:59
Saviqtsdgeos, https://bugreports.qt-project.org/browse/QTBUG-33020?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel09:00
Saviqtsdgeos, fixed already09:00
tsdgeosah ok09:00
dednickmhr3: i wonder if something is going wrong with counting the number of items in the categories...09:01
Saviqtsdgeos, pushed a workaround09:02
tsdgeosoh wow :D09:03
Saviqtsdgeos, actually, need to do one more09:03
Saviqtsdgeos, yah!09:03
Saviqtsdgeos, I knew you'd love it ;)09:04
Saviqtsdgeos, pushed s/\n/<br />/g09:06
Saviq+\09:06
tsdgeosoka09:06
mhr3Saviq, yea, you better do that in the shell, it's coming from the scopes that way09:07
Saviqmhr3, ok, un-TODOing, then09:07
mhr3(and we don't want to fix each and every scope)09:07
Saviqmhr3, yeah, what I though09:08
Saviqt09:08
Saviqtsdgeos, need to re-approve, won't land otherwise09:08
tsdgeosyep09:08
tsdgeoslet me test it works :D09:08
Saviqtsdgeos, but then it won't land anyway...09:09
Saviqpete-woods, can we expect a fix for libusermetrics sometime soon? :)09:09
mhr3dednick, 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
Saviqsil2100, is libusermetrics part of unity8's stack? (it should be, I think...)09:09
pete-woodsSaviq: it's committed already09:09
Saviqpete-woods, ah09:09
Saviqpete-woods, thanks09:09
pete-woodsnp09:09
Saviqsil2100, Mirv can we release libusermetrics?09:10
dednickmhr3: hm. maybe. i guess it would be set to non-visible! dang. didnt think about that09:10
dednickstupid categories09:10
mhr3dednick, but there's a call to ClearCategories() shouldn't that handle it?09:11
dednickmhr3: it should09:11
MirvSaviq, sil2100: running indicators stack, can you sil2100 look it goes through?09:11
sil2100Mirv: ok09:11
dednickwell that's a bugger. builddeb failed because of one test... sigh09:12
Mirvand yes for some reason it's in the indicators stack09:12
mhr3dednick, let's corner Trevinho next week and make him fix it, he's in london09:13
dednickmhr3: hehe. fix our bug!09:13
mhr3will probably need to lock him on the toilet with his laptop09:13
mhr3oh wait...09:13
dednickmhr3: yeah, it's multiple categories. i can reproduce09:15
dednickmhr3: just removed SetVisible(false) ...09:15
mhr3dednick, so why aren't they cleared?09:16
dednick:/09:16
dednicki will find out promptly09:16
mhr3oh... the initial setup is called before all the categories are added, isn't it?09:17
mhr3so it adds them twice the first time09:17
dednickit should clear them every time it switches model.09:18
dednickunless the bind isnt working correctly....09:18
dednickmhr3: right. so apparently sigc::bind doesnt work with sigc::hide.09:30
dednickmhr3: and http://pastebin.ubuntu.com/5992130/09:31
dednicki'm getting 2 application categories...09:31
mhr3dednick, that's fine09:31
dednickmhr3: why would that be fine?09:32
mhr3dednick, one has id "global", the other "apps"09:32
mhr3you're printing the name, it's the same09:32
mhr3as you can see the index is different09:33
dednickmhr3: yeah, i thought it was just adding 2 apps at different indexes09:35
dednickcool. it's my bad then09:35
dednickit's adding duplicates because the bind for "bool add" isnt working09:36
dednickrather retardedly09:36
om26erMirv, hey09:37
mhr3dednick, but your log says "false", doesn't that mean it works?09:37
dednickmhr3: after fixing yes09:37
mhr3ok09:37
dednickmhr3: was always true09:37
* mhr3 loves when compiler / library doesn't work09:38
sil2100jamesh: one thing about mediascanner - would you mind if I change the MEDIASCANNER_VERSION variable in CMakeLists.txt to the proper version?09:39
jameshsil2100: that sounds fine.09:39
dednickmhr3: https://code.launchpad.net/~nick-dedekind/unity/duplicate.categories/+merge/18051109:43
* dednick endevours to actually test code he merges09:44
Mirvom26er: hello09:44
dednickSaviq: shall we get the indicators show on the road? https://code.launchpad.net/~nick-dedekind/unity8/remove.battery.time.indicators/+merge/18037509:45
Saviqdednick, not yet, we still can't release unity8...09:45
om26erMirv, the qtorganizer5-eds project that you setup yesterday didn't have a source code branch target, but I just saw someone added that09:45
Saviqdednick, need to wait for unity to release first09:45
om26erMirv, thanks anyway :)09:46
Mirvom26er: yes, that "someone" was me as well :)09:46
Mirvafter I noticed there was some branch now to set as such09:46
Saviqdednick, at least libusermetrics should not be killing us anymore09:46
mhr3dednick, how about using standard sigc::mem_func(this, ClearCategories), and do the forced add at the end? will be more readable09:46
om26erMirv, cool, thanks09:46
Saviqsil2100, Indicators finished, yellow though (packaging change)09:46
Mirvom26er: so it's now just lp:qtorganizer5-eds like it should be09:46
MirvSaviq: needs +1 from core-dev09:46
MirvSaviq: if you see any around, please get, for http://pastebin.ubuntu.com/5992152/09:47
om26erMirv, can you +1 this https://code.launchpad.net/~om26er/cupstream2distro-config/ubuntu_keyboard_release_daily/+merge/180442 ?09:47
SaviqMirv, who's "core-dev"?09:47
dednickmhr3: yeah, dont know why still doing func for a single line...09:47
MirvSaviq: https://launchpad.net/~ubuntu-core-dev/+members09:47
sil2100Saviq: yes, we're poking around about that09:47
om26erMirv, its to enable daily release for ubuntu-keyboard, I got the packaging reviewed from mterry yesterday, he made a few improvements09:47
MirvSaviq: it's the deal we have, need an ack from one always09:47
Saviqsil2100, ok thanks09:47
SaviqMirv, understood09:48
Mirvpreferably in 12 minutes before the next cycle starts :) I wonder how long ogra's meeting is09:48
Saviq:D09:48
mhr3dednick, if i had this comment yesterday we'd save ourselves quite some trouble :)09:48
Mirvom26er: ok, looking09:48
mhr3Saviq, we need vim in the default touch image, i can't stand vi :P09:51
Saviqmhr3, https://code.launchpad.net/~ubuntu-core-dev/ubuntu-seeds/ubuntu-touch.saucy09:51
Saviq;)09:51
mhr3need to apt-get it everytime... terrible user experience09:51
Saviqmhr3, and ssh09:51
mhr3+100 :)09:51
sil2100:|09:51
tsdgeosSaviq: got this http://paste.ubuntu.com/5992173/09:51
tsdgeosjust starting and unlocking09:52
sil2100Without didrocks and seb128 it's so much more complicated to get things released :|09:52
Saviqtsdgeos, https://code.launchpad.net/~mhr3/unity/only-valid-filters09:52
sil2100At least until ken and mathieu pop up later09:52
SaviqOH ZE FRENCH!09:52
tsdgeosSaviq: but that is merged already09:52
tsdgeosand i just got it09:52
SaviqVER AR THEY WHEN YOU NEED THEM?09:52
Saviqtsdgeos, not released09:52
tsdgeosah wait09:52
tsdgeosit's on unity09:52
tsdgeosnot unity809:52
Saviqtsdgeos, yes09:52
tsdgeostrue-ish09:52
Saviqtsdgeos, and not released09:52
tsdgeosSaviq: they are enjoying 15th august holidays? :D09:53
Saviqtsdgeos, sounds like it indeed ;)09:53
Saviqmhr3, what's the deal with the Unity stack? any ETA?09:53
mhr3Saviq, yea, we have a fix now, should land soonish09:54
mhr3Saviq, so... 4hours?09:54
Saviqmhr3, daily starts in 5 mins, will it land by then? ;)09:55
sil21005 minutes leeeft!09:55
Saviqerm quaternourly09:55
sil2100Mirv: did you find anyone?!09:55
mhr3no09:55
Saviq:/09:55
mhr3maybe if you allow dednick to push the fix directly09:55
mhr3sil2100, ^?09:55
Mirvsil2100: yes, now09:56
sil2100Mirv: \o/09:59
sil2100Saviq: so, indicators published, so the daily-release run that will start in a minute should already have the fixed libusermetrics, so unity8 should pass now10:00
nic-doffaySaviq, getting a weird error assigning a deligate to a ListView, mind giving your thoughts?10:00
Saviqsil2100, probably won't, as we still need the unity fix10:00
Saviqsil2100, and unity won't publish before https://code.launchpad.net/~nick-dedekind/unity/duplicate.categories/+merge/18051110:00
Saviqnic-doffay, details, please10:00
nic-doffaySaviq, https://pastebin.canonical.com/95961/10:01
nic-doffayFirst line is the error.10:01
nic-doffayYou'll see the model is just an int for now.10:01
nic-doffayAnd the delegate is the option selector.10:01
Saviqnic-doffay, you need to wrap in Component { }10:01
Saviqnic-doffay, if you don't assign to "delegate" directly10:01
Saviqnic-doffay, but really10:01
Saviqnic-doffay, in that case you don't need "delegate: " at all10:02
Saviqnic-doffay, IIRC the child of ListView is the delegate by default10:02
Saviqnic-doffay, or maybe not, and anyway it's better to be explicit10:03
Saviqnic-doffay, also it's recommended that you put the Component { OptionSelector {  } } outside of ListView { }10:03
Saviqnic-doffay, see http://qt-project.org/doc/qt-5.0/qtquick/qml-qtquick2-listview.html#details10:03
nic-doffaySaviq, cool. Thing is it's still only displaying one selector now.10:05
nic-doffayEven though the model is clearly set as 3.10:05
Saviqnic-doffay, Component { id: foo OptionSelector { } } ListView { model: 3; delegate: foo }10:05
om26erMirv, is that change enough for daily release ?10:05
nic-doffaySaviq, yeah that's what it looks like atm.10:06
Saviqnic-doffay, that creates 3 selectors for sure10:06
Saviqnic-doffay, code, please?10:07
nic-doffaySaviq, https://pastebin.canonical.com/95962/10:07
nic-doffayThe cos recommended not manipulating contentY/X directly so I'm redoing the scrolling.10:08
tsdgeosthere's something broken in the filter grids too :-/10:08
nic-doffaySaviq, that's the reason for redoing it with a model is correct because it can be provided from the backend.10:08
tsdgeosi can easily reproduce this10:09
tsdgeosFilterGrid Model 2610:09
tsdgeosFilterGridLimited Model 610:09
tsdgeosFilterGridLimited Items 1310:09
nic-doffaySaviq, at least that's how I feel atm.10:09
tsdgeoswhy the view has 13 items if the model only has 6?10:09
tsdgeoseh! tellme! why!10:09
dednickmhr3: again. https://code.launchpad.net/~nick-dedekind/unity/duplicate.categories/+merge/18051110:09
dednickwoops. guess i missed daily10:10
dednickSaviq: any idea when unity8 release will be?10:11
Saviqnic-doffay, your ListView doesn't have enough height10:11
Saviqnic-doffay, to show more than one10:11
Mirvom26er: 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
Saviqnic-doffay, height: contentHeight10:12
om26erMirv, that's fine10:12
om26erMirv, last review https://code.launchpad.net/~om26er/cupstream2distro-config/ci_for_qt-eds/+merge/18051510:12
mhr3dednick, thx, approved10:12
om26erMirv, adds CI for qtorganizer5-eds10:13
Saviqnic-doffay, you should avoid manipulating ListView.contentY directly, btw10:13
Mirvom26er: ok, will check10:13
Saviqnic-doffay, if possible, use positionViewAtIndex()10:14
nic-doffaySaviq, when I add that there are errors about a binding loop.10:14
nic-doffaySaviq, yeah that's what I'm experimenting with atm.10:14
nic-doffayOr positionViewAtEnd10:14
nic-doffayI was planning to see what worked best.10:14
Saviqnic-doffay, AtEnd will position at end of the whole ListView10:14
Saviqnic-doffay, so AtIndex is what you most probably want10:14
nic-doffaySaviq, then I'll just position it at the last index using the other function.10:15
nic-doffaySaviq, but back to the height issue.10:15
Saviqnic-doffay, but anyway, just anchor your ListView's bottom to the parent, that's what you need anyway in the end, no?10:15
nic-doffaySaviq, I'm not sure what you mean.10:15
Saviqnic-doffay, somehow you need the ListView to have a height10:16
Saviqnic-doffay, it doesn't assume a height by default10:16
nic-doffaySaviq, you mean use the anchor instead of setting it to the contentHeight?10:16
Saviqnic-doffay, contentHeight is the height of all the items in the ListView10:16
Saviqnic-doffay, it's rarely the right thing to use10:17
Saviqnic-doffay, it was just a quick'n'dirty approach from my side to show what's the issue10:17
Saviqnic-doffay, I don't know what's the height you need10:17
Saviqother than != 010:17
nic-doffaySaviq, setting the bottom anchor was def the best way.10:19
nic-doffayta10:19
Saviqnic-doffay, told you so ;10:19
Saviq;)10:19
MacSlowSaviq, 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:27
mzanettiMacSlow: well, you don't need the xml stuff10:28
mzanettiMacSlow: 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 yourself10:28
tsdgeosMacSlow: see http://bazaar.launchpad.net/~mir-team/unity-mir/trunk/revision/17 on how to expose stuff using qtdbus10:29
MacSlowmzanetti, I have what I want working in Python (as proof of concept) and now want to get it into the notification-backend10:29
MacSlowtsdgeos, looking...10:29
mzanettiMacSlow: what's the problem with writing down the D-Bus interface in xml and then generating a Qt class you can inherit from?10:30
MacSlowmzanetti, nothing really... just wanted to avoid touching the build-steps10:31
mzanettiMacSlow: you shouldn't call the qdbus2xml in the build steps anyways10:33
mzanettiMacSlow: 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 description10:34
Saviq+110:36
MacSlowmzanetti, I tend be used to hook such things up into the build-scripts... won't do it10:37
MacSlowSaviq, btw... were you ok with my last summary on the extended snap-decisions?10:37
SaviqMacSlow, well, I'm still not sure it will provide us enough flexibility...10:38
SaviqMacSlow, as I mentioned earlier, I don't think sending an "updated" notification with the whole list every time the list of AP changes10:38
SaviqMacSlow, will provide for good UX10:38
MacSlowSaviq, I rather want a green light so I don't have to redo it again10:39
SaviqMacSlow, that's why I was opting for the MenuModel approach10:39
SaviqMacSlow, sending *some* things through as hints I'm ok with10:40
SaviqMacSlow, but the list of APs with their signal strength... I'm really not sure that's going to cut it10:40
SaviqMacSlow, and as for "replying" to the notification - a single action in the MenuModel that you'd submit the data to would be good enough, too10:41
SaviqMacSlow, although I remember you wrote that there's issues with invoking an action with parameters / arguments?10:41
MacSlowSaviq, correct10:41
Saviqi.e. not supported yet in UnityMenuModel?10:41
MacSlowSaviq, if that's not in there yet I'd be stuck10:42
SaviqMacSlow, and since larsu isn't here, I doubt it's "there"10:43
MacSlowSaviq, who'd add that? Do I have to patch it then?!10:43
SaviqMacSlow, afaik larsu is working on UMM10:43
Saviqbut he's on holidays10:43
* Saviq checks10:44
Saviqor on debconf10:44
Saviqhmm nothing in cadmin10:44
MacSlowSaviq, pressure is pretty high for these extended snap-decisions... so I guess I've to further swim into unfamiliar territory to get it done10:45
SaviqMacSlow, I know10:45
SaviqMacSlow, in absence of the "submit action with parameters"10:45
SaviqMacSlow, we can go with the custom interface you're doing now10:46
SaviqMacSlow, unless you want to dig into UMM10:46
sil2100jamesh: a review for you, last one before getting it into the archive I guess? https://code.launchpad.net/~sil2100/mediascanner/version_bump/+merge/18053110:46
SaviqMacSlow, but really I'd rather not send the list of APs as hints, but just point at a MenuModel10:46
SaviqMacSlow, and have that MenuModel updated10:47
MacSlowSaviq, 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'ed10:47
SaviqMacSlow, I think a(ss) is good enough10:47
SaviqMacSlow, i.e. [('username', 'foo'), ('password', 'bar')]10:48
Saviq[('ap', 'whatever-ap-id-wanted')]10:48
dednickcan someone do quick review (typo fix)? https://code.launchpad.net/~nick-dedekind/unity8/indicators.network.password/+merge/18053010:48
Saviqdednick, there's a bug for it, right?10:49
Saviqdednick, can you link?10:49
dednick^ fixes network password dialog not appearing.10:49
dednickah10:49
dednickSaviq: done10:52
Saviqdednick, yup, thanks10:53
=== MacSlow is now known as MacSlow|lunch
sil2100Mirv: 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/18053110:55
dednickSaviq: what is the status of unity8 release? thostr_ is pretty eager we get this indicator show on the road.10:56
sil2100jamesh: btw. when building mediascanner on my system, sometimes some unit-tests are failing10:56
sil2100jamesh: usually a re-run helps, but still10:56
Saviqdednick, thostr_, we should be able to go through upstream merger again now that libusermetrics is fixed10:57
Saviqdednick, thostr_, now we're only waiting for a unity release that fixes a libunity-core crash10:57
dednickSaviq: howcome we need to wait for that before removing the battery tests?10:58
Saviqdednick, because we can't get through release at all10:58
dednickSaviq: or is it just removing the service files that is the issue?10:58
Saviqdednick, so even if we merge your things10:58
Saviqdednick, we can't release10:58
Saviqdednick, (now we can merge, btw, we couldn't until usermetrics was fixed...)10:59
Mirvsil2100: opening a tab for that too... :)11:00
dednickSaviq: just fyi, we're going to be doing the sound indicator as well. even though chewie sound will still be running.11:01
Saviqdednick, k11:01
Saviqsil2100, any idea when will libusermetrics get into the archive?11:03
sil2100Saviq: it's in already, at least the one with the dbus script failing11:04
sil2100https://launchpad.net/ubuntu/+source/libusermetrics11:04
Saviqsil2100, ok, probably pl mirror isn't there yet11:05
sil2100Saviq: probably...11:05
sil2100Mirv: 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 released11:06
sil2100Mirv: probably because it wasn't yet synced completely when indicators re-ran11:06
om26ersil2100, hey which ppa contains the staging packages that are tested for daily release ?11:06
sil2100Mirv: so let's just ignore the indicators stack for this run11:06
sil2100om26er: ppa:ubuntu-unity/daily-build is what you want11:07
om26ersil2100, thanks11:07
sil2100om26er: but each stack only uses the packages it needs from this PPA11:07
sil2100om26er: i.e. there is no dist-upgrade being done from this PPA during testing11:07
sil2100om26er: just selected, needed packages are being upgraded from there11:08
om26ersil2100, ok, I just want to grab the latest unity8 from there11:08
sil2100om26er: just remember that daily-build is very risky ;)11:08
Saviqsil2100, not even the main mirror is "there" yet :P11:08
om26ersil2100, I am not adding the ppa, just downloading the debs ;)11:08
sil2100Saviq: pff ;p It got in 25 minutes ago, so I would expect it to be in already!11:09
Saviqsil2100, indeed ;)11:09
Mirvsil2100: ok, makes sense, there are always such delays11:09
Mirvsil2100: that's why we normally never run the manual runs anymore11:10
Saviqsil2100, ah, jenkins picked up the new version, fortunately11:11
Saviqjeez, something's still adding qt5-proper to qmluitests...11:11
Saviqmzanetti, can you have a look ↑11:12
mzanettiSaviq: sure11:12
sil2100greyback: hi! Does unity-mir have some/any integration tests? Is it possible to do some?11:18
greybacksil2100: hey, it's totally possible to do some, I want to start at that next week when we've the basics in place.11:19
greybackbut for now, I've been a bad boy ;)11:20
mhr3Saviq, did anyone notice that dns resolving is blocking the rendering thread?11:20
Saviqmhr3, huh?11:20
MirvSaviq: with bregma offline, who else could we ping bug #1212945 about?11:20
ubot5bug 1212945 in Unity "A surge of failing autopilot tests in unity" [Critical,New] https://launchpad.net/bugs/121294511:20
SaviqMirv, I believe that's fixed already, right mhr3, dednick ↑?11:20
Mirvah, there's a check run now running so if that's so, nice and we should have the result maybe in 0.5h11:21
SaviqMirv, we're just waiting for it to land and release11:21
SaviqMirv, doubt it, it's not merged yet11:21
Saviqmhr3, dednick, why isn't the unity fix merged yet  ↑?11:21
sil2100:o11:21
MirvSaviq: the last commit is from 9 hours ago, so not11:21
SaviqMirv, http://s-jenkins:8080/job/unity8-autolanding/11:22
Saviqwrong11:22
Saviqhttp://s-jenkins:8080/job/unity-autolanding/11:22
SaviqMirv, it's landing11:22
dednickno idea. it's been approved by ci...11:22
Mirvif it's this, it's just jenkins delay https://code.launchpad.net/~nick-dedekind/unity/duplicate.categories/+merge/18051111:22
SaviqMirv, yes11:22
dednickjust linked the bug11:22
Saviqdednick, yeah, it's being merged11:23
sil2100greyback: hmm, since I would feel a bit reluctant to enable daily-release for unity-mir before it has any integration tests11:23
sil2100greyback: but on the other hand, FF is close so we'd need to have it released at least once...11:23
Mirvthanks dednick11:23
greybacksil2100: Unfortunately next week is best I can do11:24
Mirvif 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
SaviqMirv, +111:24
SaviqMirv, but it will run it anyway, no? or is unity not a dependency of unity8?11:24
SaviqMirv, or if nothing changes, there will be no run of the unity stack?11:25
MirvSaviq: based on this cycle unity is not marked as dependency, ie unity8 finishes before unity has finished11:25
Saviqinteresting...11:25
MirvSaviq: so unless manually run, the next cycle unity would be (possibly) published but unity8 not still11:26
* Saviq needs to talk to didrocks ;)11:26
Mirvby all means :)11:26
mhr3Saviq, i'd know cause my internet decides to not work from time to time11:27
Saviqmhr3, and unity8 gets stuck does it?11:28
mhr3yep11:28
sil2100greyback: let's wait till next week then, I'll discuss it with Mathieu as well11:28
Saviqmhr3, only place where unity8 accesses the interwebs... remote scopes?11:29
greybacksil2100: ok11:29
om26erSaviq, while you are reporting bugs, here is one more :) bug 121303811:29
ubot5bug 1213038 in Unity 8 "running apps thumbnail becomes black if dash is scrolled down" [Undecided,New] https://launchpad.net/bugs/121303811:29
Saviqom26er, dupe ;P11:29
Saviqom26er, assuming I can find it ;)11:29
Saviqom26er, bug #119309911:30
om26erright, its been a while. someone else must have11:30
ubot5bug 1193099 in Unity 8 "Unity: thumbnails for running apps are not cached" [Medium,Triaged] https://launchpad.net/bugs/119309911:30
mhr3Saviq, easy to see with the click scope11:30
mhr3Saviq, let me make a vid11:30
om26erman, if I only had searched for 'thumbnails' would have found that11:30
Saviqom26er, no worries, I had to remember to search for "cached" ;)11:31
om26ermhr3, unity8 crashes if I try to use smart scopes like: 'music: that song' should I file a bug for that ?11:34
greybackanyone here not using xmir? Does gnome-screenshot work for you? All I get is black screengrabs11:38
SaviqMirv, sil2100 unity landed, can we restart Unity stack plz?11:43
mzanettitsdgeos: fancy a review? https://code.launchpad.net/~mzanetti/unity8/dnd-and-quicklists/+merge/17665311:43
MirvSaviq: sil2100: will handle11:43
Saviqgreyback, yeah, works here11:43
sil2100Saviq: the previous build is still running, but I think I could abort it and re-run11:43
greybackSaviq: thanks for checking. Will log11:44
Mirvsil2100: yes, that's what I'm doing11:44
sil2100Saviq: is unity the only thing which needs re-building?11:44
sil2100Mirv: ah!11:44
Saviqsil2100, afaik, yes11:44
sil2100:)11:44
sil2100Mirv: sorry, I misunderstood to as "sil2100 will handle"11:44
sil2100Mirv: while it was more like a declaration 'I will handle it', right?11:44
Mirvsil2100: ooops! :) I should be more precise :)11:44
sil2100Confusing Friday!11:44
Mirvsil2100: indeed!11:45
tsdgeosmzanetti: wow that one's big :D11:45
Saviqif only confusing11:45
Saviqtsdgeos, big? measly 1.2kLOC11:45
tsdgeosmzanetti: sure i'll handle it if noone does11:45
Saviq;)11:45
mzanettitsdgeos: 1/3 of it is moving code11:45
mzanettibut yeah. the rest is still quite tricky stuff11:45
mzanettidefinitely the most tricky thing I did with QML so far11:45
tsdgeosstill struggling to understand why we get the LVWPH stuff with holes at the top11:45
tsdgeossometimes11:46
mzanettiholes?11:46
Saviqmzanetti, oops, got a loop of "LauncherPanel.qml:357: TypeError: Cannot read property 'height' of undefined"11:46
Saviqmzanetti, after having dragged an item past the top11:46
mzanettiSaviq: right. I know this one11:46
* tsdgeos gets some food, let's see if that helps clearning the mind11:47
mzanettiSaviq: 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 destroyed11:47
Mirvmzanetti: 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
mzanettiSaviq: only thing would be to set cachebuffer to infinite which is not what I wanted11:47
mzanettiSaviq: so what I did is to make the Launcher's margins at least big enough so you can't cause this when running unity fullscreen11:48
mzanettiyou still can break it when running windowed tho11:48
Saviqmzanetti, got it11:48
mzanettiSaviq: LauncherPanel.qml line 105 is the comment that explains it11:48
SaviqMirv, sil2100 can you allow "greyback" the right to rebuild jobs in s-jenkins?11:48
mzanettiMirv: wohooo! that's awesome stuff!11:49
sil2100Saviq: I think Francis will have to pop up to do that sadly ;/11:49
mzanettiMirv: I'll probably check it out this weekend11:49
Saviqsil2100, ok11:49
mzanettiI'll be off for an hour for lunch11:50
Saviqmzanetti, enjoy11:50
Mirvmzanetti: saucy build at ppa:ubuntu-sdk-team/ppa11:52
popeygreyback: i get black screenshots sometimes.. with xmir.11:52
greybackpopey: only sometimes? https://bugs.launchpad.net/mir/+bug/121304711:54
ubot5Launchpad bug 1213047 in Mir "[xmir] gnome-screenshot only gets black image" [Undecided,New]11:54
=== alan_g is now known as alan_g|lunch
popeygreyback: yes11:55
popeygreyback: maybe related to the fact I get desktop crashes now and then, might be black after that's happened11:56
greybackpopey: very strange. You mutter a secret incantation perhaps? Maybe mention that in the bug11:56
greybackpopey: for me I've just freshly booted11:56
popeyi press a magical key which has strange glyphs on it greyback11:56
popeyit's called "PrtSc"11:56
* greyback reels back in horror11:56
popeyI know, right?!11:56
greybacknon Mac user, gah!11:56
Mirvgreyback: please run apport-collect 121304712:01
Mirvscreenshot has worked fine here so far12:01
Mirvbut probably same as popey ie sometimes black, I just haven't used it much12:01
=== MacSlow|lunch is now known as MacSlow
greybackMirv: "No additional information collected"12:03
Mirvgreyback: :P12:03
dednickSaviq: 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
Saviqdednick, yes, Unity still not released12:04
dednickSaviq: ah right...12:04
Saviqdednick, we just restarted Unity stack12:04
dednickSaviq: i keep on thinking of unity8 as unity :) i realise now what you are meaning "waiting for release"12:05
Saviq:)12:05
Saviqdednick, we (unity8) are not even a dependant stack (of unity) for some reason, need to talk to didrocks when he's back12:06
Cimimy labels don't wrap...12:07
CimiI put wrapMode: Text.WordWrap12:07
Cimino joy..12:07
Ciminevermind, was width/height12:08
Mirvgreyback: 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 it12:09
SaviqCimi, I agree, there should be a warning when you use Text with wordwrap and no explicit height12:10
SaviqCimi, same for ListView, for example12:10
=== dednick is now known as dednick|lunch
greybackMirv: I need to do an update?12:11
greybackMirv: all I get now is: "Package mir not installed and no hook available, ignoring"12:11
Mirvgreyback: mmkay, like I said not that I know anything, but at least that was progress, dunno how to go from there though12:31
Mirvgreyback: when I filed a bug against unity-system-compositor, apport-collect worked12:31
greybackMirv: ok. I'll update, might help12:32
Mirvom26er: 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 Monday12:32
Mirvgreyback: it might be something to bring up on #ubuntu-mir, that bugs filed against 'mir' don't collect meaningful data12:33
greybackMirv: ack12:34
om26erMirv, oh? it was really needed to land today, both of them :/12:43
Mirvom26er: so far no-one including archive admins have not been aware what's the 'whitelist' didrocks adds new cu2d source packages to12:44
Mirvom26er: so even if we did get them built in the daily release system, we couldn't release them into saucy archives12:46
om26erMirv, ok, I get it.12:46
om26erI do think for future the "art" shall be transferred to more people12:46
Mirvom26er: yes, that has been a topic today and yesterday, that art needs to be shared :)12:47
Mirvom26er: 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 hour12:49
Mirvit will then stay unreleased like the poppler-qml-plugin in sdk stack which was added yesterday12:49
om26erMirv, cool, we are slowly making progress :) there was also the eds plugin, i'd assume that's included as well12:50
Mirvom26er: ah yeah, that's there too now, in the stack (http://10.97.0.1:8080/view/cu2d/view/Head/view/Misc./)12:51
Saviqgreyback, I retargeted unity-mir bp to ubuntu to inherit the milestones12:54
Saviqgreyback, and moved all items under ubuntu-13.08, feel free to move them around if needed12:54
=== _salem is now known as salem_
Saviqgreyback, this way it shows up in +upcomingwork12:55
Saviq/we need milestones for unity812:55
greybackSaviq: ack12:55
sil2100kenvandine: !13:05
=== dednick|lunch is now known as dednick
Saviqsil2100, is that looking good enough for publishing http://10.97.0.1:8080/job/autopilot-saucy-daily_release/1040/ ?13:21
sil2100Saviq: so far so good, let's wait for ati13:22
Saviqsil2100, \o/13:22
sil2100Saviq: if ati fails, I'll force the publishing anyway13:23
sil2100There's not much time left though, so we might have the same problem as with libusermetrics ;/ But oh well13:23
Saviqtsdgeos, are you doing mzanetti's launcher dnd review in the end? or fighting with FilterGrid / LVWPH?13:23
tsdgeosSaviq: having a look at the LVWPH "jump"13:24
Saviqtsdgeos, ok, I'm taking on the launcher, then13:24
tsdgeosoka13:25
greybackSaviq: standup!13:35
MirvSaviq: we need two core-dev acks again, unity would be ready otherwise, 25mins of time13:35
MirvSaviq: correction, one needed, libunity http://pastebin.ubuntu.com/5992746/13:38
greybackmterry: doesn't nested-mir just create a mir surface, why does compositor care?13:39
greybackmzanetti: prep for presentation on Qt devdays13:40
mzanettiright13:40
mterrygreyback, 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 yet13:42
Mirvmterry: 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:42
Mirvmterry: ok, forget that, ogra approved, thanks13:43
sil2100Saviq: published!13:43
greybackmterry: hmm, ok13:43
Saviqsil2100, yay13:43
Mirvgreat, at least one step of progress. with luck unity8 will be green after the next cycle as well, and Apps probably too13:44
sil2100Saviq: 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 run13:44
dednickSaviq: deosnt look like it's been released yet though13:45
Mirvdednick: it is, now, 15s ago (pushed)13:45
Saviqdednick, probably just not synced yet13:45
=== alan_g|lunch is now known as alan_g
Saviqdednick, https://launchpad.net/ubuntu/+source/unity13:45
Mirvrefresh refresh :)13:46
Mirvfor a couple of minutes at least13:46
Saviqdednick, so what do I do to test that the new indicators work?13:46
dednickSaviq: just install them. they should pop up. "may have to reboot phone"13:47
Saviqdednick, indicator-{datetime,power,sound}?13:47
dednickSaviq: yep13:48
dednickSaviq: and prepare to run out of panel space13:48
Saviqdednick, ;)13:48
dednickSaviq: can remove the other indicator files as it pleases (com.canonical.indicator.time, com.canonical.indicator.battery, com.canonical.settings.sound)13:49
Saviqsil2100, another quarternourly in 10 mins or in 2:10?13:49
sil2100In 10 minutes13:49
sil2100Saviq: that's why we were so crazy about getting that approved and published13:50
Saviqsil2100, right, looking at fast-tracking https://code.launchpad.net/~nick-dedekind/unity8/remove.battery.time.sound.indicators/+merge/180375 then13:50
Mirvgood night everyone :)13:51
SaviqMirv, o/13:52
Saviqdednick, sound slider doesn't seem to work on the device?13:52
dednickSaviq: as in changing volume or enabled?13:52
Mirvnow there https://launchpad.net/ubuntu/+source/unity/7.1.0+13.10.20130816.3-0ubuntu1 , soon in proposed pocket, just on time13:52
Saviqdednick, changing volume13:52
dednickum.13:52
dednickSaviq: is the chewie one working?13:53
Saviqdednick, not, indeed13:53
dednickSaviq: i wonder if it's because we're running parallel with chewie...13:55
dednickSaviq: mute works only on new one...13:56
Saviqdednick, indeed13:56
Saviqdednick, fast-tracked merging your branch13:56
Saviqdednick, let's hope it makes it in time ;)13:56
sil2100Damn, still no libunity in proposed/release13:57
dednickSaviq: we need to change the seed as well. otherwise we wont have any indicators :)13:57
Saviqdednick, that's fine, as soon as we actually make a release ;)13:57
dednickci approved.13:58
nic-doffaySaviq, can you ping me when you have a moment to chat about the filters back end a bit more.13:59
dednickSaviq: wow. merged already!13:59
dednicknice13:59
Cimimzanetti, ping14:00
Cimimzanetti, no pressure just wondering if you had a look or not14:00
Cimiat the autopilot thing14:00
Saviqdednick, well, CI didn't *really* approve - I made it approve ;)14:01
dednickSaviq: yeah, i saw that14:01
mzanettiCimi: it should be fixed14:01
mzanettiCimi: since Wednesday I think. Let me check the state of the MR14:02
tsdgeosSaviq: 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
Saviqtsdgeos, sure, checking14:13
tsdgeosi think it's a problem with the flickable code14:13
tsdgeosbut still need to read it a bit more14:13
Saviqtsdgeos, yes, that fixes the fullscreen issue (not the others)14:15
tsdgeosSaviq: hmmm14:15
tsdgeosSaviq: the other is the  one that you scroll a lot and gets somehow stuck?14:16
Saviqtsdgeos, +114:16
tsdgeosi wish i could repro that :/14:16
Saviqtsdgeos, it also makes the categories "consistently" scroll $section_header_height up14:16
Saviqon first input14:16
tsdgeosSaviq: that's good, no? :D14:16
Saviqtsdgeos, s/on first input/on showHeader/14:17
Saviqtsdgeos, when I launch and start scrolling to the side14:17
Saviqtsdgeos, it scrolls the view a height of the section header up14:17
Saviqtsdgeos, like showHeader() is doing something weird14:17
tsdgeosyou're going to have to send me your comptuer :D14:18
tsdgeosSaviq: that happens all the time?14:18
Saviqtsdgeos, on first start14:18
Saviqtsdgeos, ;)14:18
tsdgeoswhat do you mean on first start?14:18
Saviqtsdgeos, I'm going to Barcelona in a week, remember? :D14:18
tsdgeosnext time you start it, it doesn't happen?14:18
Saviqtsdgeos, rephrase: "on start"14:18
tsdgeosok14:19
Saviqtsdgeos, the first time I swipe to the sides14:19
tsdgeosso every time you start it14:19
Saviqtsdgeos, yes, that also makes the "stuck in the middle" issue not happen14:19
Saviqtsdgeos, i.e. if I first drag to the sides14:19
Saviqtsdgeos, can't get it to be stuck in the middle14:20
tsdgeosinteresting14:20
Saviqtsdgeos, so showHeader() effectively prevents that from happening14:20
tsdgeoswell, i guess i can write some code to output the state of the list14:20
tsdgeosand then try to repro it here14:20
tsdgeosit's not much14:21
tsdgeosbut let me finish with the fullscreen tihing first14:21
dednickSaviq: is there a daily release in progress for unity8?14:26
Saviqdednick, quarternourly, yes14:28
Saviqdednick, http://10.97.0.1:8080/view/cu2d/view/Head/view/Unity8/ ;)14:28
Saviqdednick, you can also see in https://launchpad.net/~ubuntu-unity/+archive/daily-build14:28
Saviqdednick, as it's actually building in that PPA14:29
dednickthanks. was looking for that dash14:29
tsdgeosSaviq: for the same bug, can you check https://code.launchpad.net/~aacid/unity8/lvwph_weirdness2 ?14:38
tsdgeosit makes me more confortable as a bugfix14:38
Saviqtsdgeos, this seems to fix more!14:39
Saviqtsdgeos, doesn't get stuck anymore14:39
Saviqtsdgeos, nor does it scroll on showHeader14:40
tsdgeosgoodie14:40
tsdgeosso 1 fix to rule them all?14:40
tsdgeos:D14:40
Saviqtsdgeos, almost everything14:40
Saviqtsdgeos, only remaining one is related to the filtering14:41
tsdgeosah yes14:41
tsdgeosthe ghost entries?14:41
tsdgeosor another one?14:41
Saviqyes14:41
tsdgeosi don't think that's LVWPH itself14:41
Saviqmost possible14:41
Saviqit's the filter14:41
tsdgeosdo you have a screenshot? just to be sure we're talking about the same thing?14:41
Saviqtsdgeos, tricky to get a screenshot14:41
Saviqtsdgeos, but I end up with 3 rows of items in "Dash plugins"14:42
tsdgeosok, let me try to get one14:42
Saviqtsdgeos, but only two of them are "inside" the ListView14:42
Saviqtsdgeos, so only visible with overshooting14:42
Saviqtsdgeos, uh oh14:42
tsdgeosanyone knows what did we break so that screen grabbing doesn't work anymore?14:42
Saviqtsdgeos, xmir?14:43
tsdgeosafaik no14:43
Saviqtsdgeos, and now the header got stuck in the middle for me...14:43
Saviqgreyback, ↑↑↑14:43
Saviqgreyback, might not be xmir in the end14:43
tsdgeosSaviq: :(14:43
* tsdgeos takes a picture :D14:44
greybacktsdgeos: all you get is black screen?14:44
Saviqtsdgeos, http://ubuntuone.com/2GGkY941W8HV03IMpr6HrB14:44
tsdgeosyep14:44
tsdgeosgreyback: ↑↑14:44
greybacktsdgeos: darm, ok14:44
tsdgeosSaviq: wooooa14:44
Saviqtsdgeos, relatively easy to repro: go to apps, expand dash plugins, search, unsearch, search quickly14:44
greybackhow...??14:45
Saviqtsdgeos, or generally search quickly14:45
tsdgeosSaviq: that with the new code14:45
tsdgeosright?14:45
Saviqtsdgeos, yes14:46
Saviqtsdgeos, not a regression, same in trunk14:46
tsdgeosoh14:46
tsdgeoswell, at least is not worse because of the fix :D14:47
* greyback feels dirty using QRegularExpression14:49
Saviqrotfl14:51
Saviqmzanetti, https://code.launchpad.net/~mzanetti/unity8/dnd-and-quicklists/+merge/176653/comments/40838514:51
Saviqgreyback, what is so bad about regexes :D14:51
greybackSaviq: they always feel a bit like magic, they work, but if not careful will bite me in the ass14:52
Saviqsil2100, ;( unity8 check crashing, do we know if the new libunity-core got there?14:53
Saviq*get14:53
Saviqsil2100, should we just force it in our dependencies/14:54
Saviq?14:54
tsdgeosSaviq: the ghost thing would be http://i.cubeupload.com/lOEq65.jpg right?14:54
tsdgeoslike the extra row in Music14:54
sil2100Saviq: ah, shit, wait, we'll probably have to re-run with check-with-whole-ppa14:54
Saviqtsdgeos, yeah, sounds right14:54
sil2100Saviq: give me a moment14:54
Saviqsil2100, thanks14:54
tsdgeosok, yeah that's filtergrid or something below14:54
tsdgeosi'll have a look too14:54
tsdgeosbut let's fix LVWPH first14:54
Saviq+114:54
Saviqtsdgeos, it's a huge improvement already14:55
sil2100Saviq: it's in proposed, so it should be fetchable14:55
tsdgeosSaviq: i need a unittest though14:55
tsdgeosnot sure how to create one14:55
Saviqsil2100, yeah, but we've not requested the new version explicitly14:55
Saviqsil2100, so it used whatever was there already?14:55
mzanettiSaviq: thanks.14:55
Saviqtsdgeos, and a quick workaround for the ghost ones is a "clip: true" away14:55
Saviqmzanetti, cheers14:55
tsdgeosSaviq: sure :D14:55
Saviq;)14:55
Saviqolli_, ping14:57
sil2100Saviq: 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-upgrade14:57
sil2100Saviq: that's what I'm doing now14:57
Saviqsil2100, ok, that works14:57
sil2100Saviq: let's see how it goes now14:57
Saviqsil2100, could we maybe do that by default?14:57
Saviqsil2100, we're depending on unity anyway14:57
olli_Saviq, pong14:57
Saviqsil2100, so not upgrading it feels error-prone14:57
sil2100Saviq: I guess it might be a thing to consider, right14:58
Saviqsil2100, looks green!15:03
sil2100\o/15:05
sil2100Yessss15:05
sil2100Saviq: the future runs might be red though again, but hm, maybe we'll consider adding the libunity dep there15:05
Saviqsil2100, yup15:07
CimiSaviq, 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
SaviqCimi, talk to seb first, he has a plan for that15:12
CimiSaviq, he's not online...15:12
SaviqCimi, and I don't know exactly what it is15:12
SaviqCimi, I know, you won't be, soon, either15:13
SaviqCimi, review something, or do something else, just don't do that, which can be in a completely wrong direction15:13
Cimiahah ok15:13
sil2100Saviq: wait, unity8 isn't completely green ;p15:19
sil2100Saviq: we need to publish it!15:19
Saviqsil2100, already pung ogra about it in #ubuntu-touch15:19
Saviqalthough might not have been explicit enough15:19
Saviqsil2100, so you go :)15:20
sil2100Maybe he's busy, so I'll also poke kenvandine15:21
Saviqyup15:22
tsdgeosSaviq: to be honest i've no clue on how to create a test for that bug i just fixed :-/15:24
Saviqtsdgeos, I can imagine...15:24
* tsdgeos keeps thinking15:25
Saviqtsdgeos, maybe a higher-level one, then?15:25
mzanettiSaviq: took me a while but I make the folding sequential to the expanding.15:25
Saviqmzanetti, *made?15:25
mzanettiyes15:25
mzanettiSaviq: will fix/answer your comments now15:25
Saviqmzanetti, cool15:25
tsdgeosSaviq: that's what i'm trying .D15:27
Saviqtsdgeos, I get it that's it's a tricky one - as it probably incorporates changing the model under the LVWPH?15:27
Saviqso it's effectively a race?15:28
tsdgeosit's kind of a race yes15:28
tsdgeosbut on a test i control everything15:28
tsdgeosso it's ok15:28
tsdgeosi just need to reproduce it :D15:29
mhr3Saviq, https://code.launchpad.net/~mhr3/unity8/hide-ratings/+merge/18059615:32
mhr3Saviq, see what have you done? me doing qml... eek :P15:33
Saviqhahaha...15:33
SaviqHAHAHA...15:33
SaviqHAHAHAHHAHAHAHAHAHAHAHAHA!15:33
mhr3your evil masterplan worked15:33
* tvoss_ hugs mhr315:33
tvoss_welcome to the dark side15:33
mhr3tvoss_, i hope there are cookies at least15:34
=== jhodapp is now known as jhodapp|lunch
tvoss_mhr3, yeah, it's nice over here :)15:36
Cimimzanetti, updates?15:36
Saviqmhr3, https://code.launchpad.net/~mhr3/unity8/hide-ratings/+merge/180596/comments/40842115:37
mhr3Saviq, why not?15:38
mhr3new property?15:38
Saviqmhr3, 'cause properties on top-level components are something like an API15:39
Saviqmhr3, and they use memory15:39
mhr3are all those signal connections cheaper?15:40
mhr3although.. you need to do those anyway15:40
mzanettiCimi: yeah, its fixed and Jenkins approved it15:41
mzanettiCimi: there was another crash which should be fixed by now by another merge15:41
Cimimzanetti, we need reviews then :)15:41
CimiSaviq, to set the full name, is it ok to use command line tool like chfn ?15:46
SaviqCimi, there's probably a library for that15:47
SaviqCimi, and seb will know what it is :)15:48
Cimiok15:48
mhr3Saviq, do you want to remove also the canBeRated prop?15:49
Saviqmhr3, I'd rather get rid of it completely indeed ;)15:50
Saviqmhr3, but that would result in 4 comparisons instead15:50
Saviqmhr3, so at least move it down somewhere15:51
Saviqmhr3, so it's not exposed on the root component15:51
mhr3Saviq, i was trying to group the 4 components as a single item, but that just broke the layout15:51
Saviqmhr3, yeah, let me have a look actually15:52
Saviqmhr3, so we're hiding "Rate this" and "Add a Review"?15:53
mhr3Saviq, also the reviews15:53
mhr3"Comments"15:53
Saviqmhr3, so effectively everything below the description?15:53
mhr3yep15:53
Saviqmhr3, should be easier than that15:53
* Saviq looks15:53
Saviqmhr3, btw, you broke tst_AppPreview with that15:55
mhr3of course i did15:55
Saviqmhr3, something along the lines of http://paste.ubuntu.com/5993239/15:59
Saviqmhr3, but it's not correct yet15:59
Saviqbut you'll get the drift15:59
mhr3hmm, childrenRect... that's the secret16:00
mhr3Saviq, the layout is still a bit broken16:02
mhr3Add a review is on top of some other label16:03
Saviqmhr3, that's why I said "it's not correct yet"16:05
mhr3i thought it's because of the test still not passing :)16:06
mzanettiSaviq: replied16:09
tsdgeosSaviq: so i have a ultra-high level test16:16
tsdgeosSaviq: http://paste.ubuntu.com/5993321/16:16
tsdgeoswhat you think?16:16
tsdgeosit's not awesome16:17
tsdgeosbut it's something16:17
CimiSaviq, tsdgeos we need reviews here :) https://code.launchpad.net/~unity-team/unity8/unity8.background-gsettings-fix/+merge/17988416:19
Cimimzanetti, said the bug with autopilot is fixed16:19
Cimiwithout ,16:19
Saviqtsdgeos, looks good enough :)16:24
* mzanetti -> EOD16:31
mzanettio/16:31
Saviqo/16:31
tsdgeosSaviq: so https://code.launchpad.net/~aacid/unity8/lvwph_weirdness2/+merge/180613 is with the test16:33
Saviqtsdgeos, ok, will check it out later16:34
* Saviq EOD16:34
Saviqo/16:34
tsdgeos\o/16:34
greybacko/16:35
bschaefersil2100, ping16:38
greybackfginther: ping16:45
=== jhodapp|lunch is now known as jhodapp
* greyback eow16:50
greybackbye all16:50
=== alan_g is now known as alan_g|EOD
=== bschaefer_ is now known as bschaefer
mhall119smart scopes in Unity 8 rock almost as much as you guys!20:50
=== salem_ is now known as _salem

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