[07:06] <mzanetti> tsdgeos: good morning
[07:07] <mzanetti> tsdgeos: I had to merge trunk at this one. mind checking/approving? https://code.launchpad.net/~mzanetti/unity8/quicklist-unset-model-before-closing/+merge/181517
[07:09] <tsdgeos> mzanetti: checking
[07:09] <tsdgeos> morning
[07:09] <tsdgeos> mzanetti: technically i only need to check you merged right, no? :D
[07:09] <mzanetti> tsdgeos: yeah :)
[07:11] <tsdgeos> mzanetti is the name of the branch "wrong"?
[07:11] <tsdgeos> it's called unset before closing
[07:11] <tsdgeos> but you close before unsetting, no?
[07:12] <mzanetti> tsdgeos: ah yeah... I had it the other way round at first
[07:12] <mzanetti> tsdgeos: caused some issues.
[07:13] <tsdgeos> oka
[07:22] <tsdgeos> mzanetti: so we got a whole lot of autoland fails still :-/
[07:22] <mzanetti> tsdgeos: it's not a whole lot any more, but yeah, too many, I agree
[07:23] <mzanetti> tsdgeos: from what I see this are the main issues:
[07:23] <mzanetti> autopilot-tests for unlocking sometimes freezes when swiping away the greeter
[07:23] <mzanetti> one of the builders causes a nullpointerexception somewhere in jenkins
[07:23] <mzanetti> and at times, the unit tests for lvwph randomly fail
[07:24] <tsdgeos> yeah, we need 5.1 for lvwph
[07:24] <tsdgeos> mzanetti: you set https://code.launchpad.net/~nick-dedekind/unity8/indicators.menu-items-cleaning/+merge/181410 to approved
[07:24] <tsdgeos> but it's been merged already, no?
[07:24] <tsdgeos> isn't it r249?
[07:24] <mzanetti> there was something weird, yeah...
[07:25] <tsdgeos> ok, i'll edit the MR manually and set it as merged
[07:25] <mzanetti> done
[07:25] <jamesh> sil2100: hi.  Has there been any progress on getting mediascanner into main?
[07:25] <mzanetti> ah... reminds me...
[07:25] <mzanetti> is didrocks in vacation currently?
[07:26] <sil2100> jamesh: working on that ;)
[07:26] <sil2100> mzanetti: yes, for 2 weeks
[07:26] <mzanetti> sil2100: I found a situation where commits get past the releasing process without being tested
[07:27] <sil2100> mzanetti: how? What's happening?
[07:27] <mzanetti> sil2100: if something gets merged by the upstream merger in between the release job starts and ends, it gets merged and then the release job commits the release commit message on top of that
[07:27] <mzanetti> sil2100: check out unit8's history
[07:28] <mzanetti> sil2100: there is a commit with a rather long commit message. starting with "tweak launcher folding"
[07:28] <jamesh> sil2100: I guess the same goes for unity-scope-mediascanner? :)
[07:28] <mzanetti> sil2100: it doesn't show up in the changelog because it landed after the release jenkins started, but before it comitted the changelog
[07:30] <sil2100> mzanetti: ok, this situation indeed might happen, sadly we don't have a way currently to not make it happen - but the good thing is that it's not really being released without testing
[07:30] <sil2100> mzanetti: let me check the lp:unity8 history to make sure
[07:30] <mzanetti> sil2100: do the tests run again after jenkins posts the changelog?
[07:32] <sil2100> mzanetti: from what I see on lp:unity8, there's not really any problem - the process looks like this that the 'changelog' and the revision that the snapshot is being taken is actually created before the testing starts, during the prepare job
[07:32] <mzanetti> sil2100: yeah. but at that point it also does the checkout to run the tests
[07:33] <mzanetti> sil2100: so I'm quite positive that this commit was not tested by the release jenkins
[07:33] <sil2100> mzanetti: it's tested by the next release, look:
[07:33] <sil2100> http://bazaar.launchpad.net/~unity-team/unity8/trunk/revision/253
[07:33] <sil2100> mzanetti: it just didn't appear in the changelog because it wasn't released
[07:33] <mzanetti> sil2100: sure.. it'll be tested by following released. that's true
[07:33] <sil2100> mzanetti: the snapshot was made from 249
[07:34] <sil2100> mzanetti: your commit was 250, so it went to the next release - so it's all safe and nothing goes 'unnoticed'
[07:34] <mzanetti> sil2100: yeah... it creates a snapshot at 249 and tests this
[07:34] <mzanetti> then 250 comes in
[07:34] <mzanetti> then the release jenkins finishes testing and commits its changelog bump to 251
[07:35] <mzanetti> 251 gets released because of that, containing 250 - untested
[07:35] <sil2100> mzanetti: no no no
[07:35] <sil2100> It's not
[07:35] <sil2100> mzanetti: look, jenkins says clearly: Releasing 7.81.3+13.10.20130826.4-0ubuntu1 (revision 249 from lp:unity8).
[07:36] <sil2100> mzanetti: when releasing it doesn't take lp:unity8 directly, it only uses the commits it prepared during the prepare job
[07:36] <sil2100> mzanetti: it creates the source package only once, when doing the snapshot
[07:36] <sil2100> mzanetti: 250 didn't land in this release
[07:37] <sil2100> mzanetti: what jenkins does is - it prepares the packages in the prepare job (the changelog is then prepared), builds and tests the packages, if all is ok, it commits the 'Releasing ..." commit to trunk and releases the *exact* packages it used for testing
[07:38] <mzanetti> sil2100: ok... understood it now
[07:38] <mzanetti> thanks for explaining.
[07:39] <mzanetti> I was watching it live yesterday (to debug our issues in jenkins) and saw those commits coming in in that order... made me think this must be an issue
[07:39] <sil2100> mzanetti: if there were some commits made in the meantime - well, they don't get released, since the source package got created during snapshotting - those commits that got into trunk in the meantime will be picked up by the next release, thanks to the (revision <num> from lp:unity8) <- this tells the build bot which was the last commit it used
[07:39] <mzanetti> sil2100: I was thinking that the commit to the changelog is the important one
[07:39] <sil2100> mzanetti: no problem ;) I guess we can't do anything not to let those things get 'out of order'
[07:39] <sil2100> mzanetti: but luckily it's safe
[08:19] <tsdgeos> mzanetti: https://jenkins.qa.ubuntu.com/job/generic-mediumtests-runner-mako/266/testReport/junit/unity8.shell.tests.test_lock_screen/TestLockscreen/test_can_unlock_passphrase_screen_Native_Device_/ failing again :-/
[08:19] <tsdgeos> any clue why?
[08:19] <mzanetti> tsdgeos: I know it will strike back
[08:20] <mzanetti> tsdgeos: no. I have absolutely no clue and haven't been able to reproduce it locally in hours
[08:20] <mzanetti> s/know/knew/ ^^
[08:20] <tsdgeos> weeeeeeeeeird
[08:33] <tsdgeos> stupid tablet ran out of battery while flashing
[08:33] <tsdgeos> hope nothing bad happened
[08:34] <mzanetti> tsdgeos: seems this phone produced just one failure. the next run on it was a good one again
[08:34] <mzanetti> tsdgeos: you can't break the device by flashing it with phablet-flash
[08:34] <tsdgeos> good stuff then :D
[08:39] <sil2100> jamesh: I need to consult the MIR for mediascanner a bit since mediascanner is using A LOT of typical universe-only packages
[08:40] <sil2100> jamesh: since gstreamer1.0-plugins-bad and gstreamer1.0-plugins-ugly we can't get into main
[08:41] <jamesh> sil2100: I'm in a standup right now.  Can I get back to you in 20 minutes?
[08:41] <sil2100> jamesh: are those required? Are those used in the code extensively?
[08:41] <sil2100> jamesh: ok
[08:41] <tsdgeos> dednick: ping
[08:46] <dednick> tsdgeos: pong
[08:46] <tsdgeos> dednick: just commented in https://code.launchpad.net/~nick-dedekind/unity8/indicator.multi-icon/+merge/181862
[08:47] <mhr3> sil2100, do we know why are the ap tests for unity failing?
[08:48] <dednick> tsdgeos: hm. maybe i shouldnt be using gu then.
[08:49] <tsdgeos> not sure
[08:50] <tsdgeos> what are you trying to achieve?
[08:51] <dednick> tsdgeos: it's the width between indicator icons. 0.5 is a bit too close, 1 is a bit too far.
[08:51] <tsdgeos> dednick: maybe round the 0.7 gus?
[08:51] <sil2100> mhr3: not yet
[08:52] <tsdgeos> the problem is that qml "knows" how to paint in 5.6 pixels
[08:52] <tsdgeos> but that will look mega-crap on some displays
[08:52] <dednick> tsdgeos: maybe should just use dp ?
[08:52] <tsdgeos> and it's also affecting the rendering speed since needs to do "some for this pixel and some for this other pixel"
[08:52] <mhr3> tsdgeos, btw sorry for not replying yesterday, we had public holiday here
[08:52] <tsdgeos> dednick: maybe, but then dp is not scaled, no?
[08:52] <dednick> tsdgeos: right. antialiasing.
[08:53] <tsdgeos> mhr3: sure, no worries
[08:53] <tsdgeos> dednick: i mean if you use dp, won't it look "too close" in a tablet?
[08:54] <dednick> tsdgeos: probably on retina it will be super close.
[08:54] <tsdgeos> dednick: so maybe the rounded gu?
[08:54] <tsdgeos> seems a bit lame, but....
[08:54] <greyback> how about mixing the two: units.gu(0.5) + units.dp(2)
[08:55] <tsdgeos> greyback: is dp scaled?
[08:55] <tsdgeos> i thought it was "always 1 pixel"
[08:56] <greyback> tsdgeos: it is scaled to 2, for high DPI screens
[08:56] <tsdgeos> ah
[08:56] <tsdgeos> dednick: maybe that works then?
[08:56] <dednick> tsdgeos: i'll see how it looks
[08:57] <dednick> galaxy nexus should provide a high enough gu
[08:57] <tsdgeos> greyback: i'm trying to test https://code.launchpad.net/~gerboland/unity8/fix-sidestage/+merge/182212 but my battery keeps running out
[08:57] <tsdgeos> greyback: is that the fix for "starting a side stage app with nothing on the main stage makes the side stage app not be shown"?
[08:58] <greyback> tsdgeos: see attached bug. Side stage apps were completely broken
[08:59] <tsdgeos> so someone ignored my bug report and created a new one
[08:59] <greyback> tsdgeos: as in, when you launched the app, you saw the side-in animation, but then you get black. The app was not allowed to shine through the shell
[08:59] <tsdgeos> yeah
[08:59] <tsdgeos> what i said
[08:59] <tsdgeos> days ago
[08:59] <tsdgeos> https://bugs.launchpad.net/unity8/+bug/1210079
[09:00] <greyback> tsdgeos: I thought I got it irrespective of if a minstage app open or not
[09:00] <greyback> but thinking about it you're probably right,
[09:00] <jamesh> sil2100: hi.  As far as the gstreamer dependencies go, the I know -bad and -ugly are not required for mediascanner to run.  I'll have to check what is actually needed for the test suite
[09:01] <tsdgeos> greyback: so dupe bugs? or?
[09:02] <greyback> tsdgeos: I've sorted it
[09:02] <tsdgeos> ah you already set the branch
[09:02] <tsdgeos> oka
[09:02] <sil2100> jamesh: please do, I'm working on getting the other deps into main in the meantime - we'd probably need to get rid of the gstreamer1.0-libav, gstreamer1.0-plugins-bad, gstreamer1.0-plugins-ugly build-deps, as I'm sure we won't force anyone to let them into main
[09:02] <tsdgeos> greyback: now is when we get some shit for ignoring this since 8 August :D
[09:03] <greyback> tsdgeos: :)
[09:35] <tsdgeos> greyback: did we got anywhere we can merge https://code.launchpad.net/~aacid/unity-mir/application_manager_tests ?
[09:37] <greyback> tsdgeos: nope, I didn't have chance to look into what's going on. I'm guessing we need to set up those tests to run on actual device
[09:37] <tsdgeos> ok
[09:55] <tsdgeos> greyback: ping
[10:01] <greyback> tsdgeos: pong
[10:04] <tsdgeos> greyback: should i be able to "move" (i.e. uncollapse) a side stage app if it's the only app open?
[10:04] <tsdgeos> err uncollapse -> collapse
[10:04] <greyback> tsdgeos: no
[10:04] <greyback> it should be locked
[10:04] <tsdgeos> okii
[10:04] <tsdgeos> then it works :D
[10:04] <greyback> yay :)
[10:30] <tsdgeos> dednick: not sure i understand https://code.launchpad.net/~nick-dedekind/unity8/indicator-location/+merge/182331
[10:31] <tsdgeos> dednick: so they changed the "default" file for indicators?
[10:32] <dednick> tsdgeos: there are old backends and new backends. Currently byu default we used the old backend compatible UIs. But now, every indicator we add is going to be compatible with new backend UIs. so i changed the deafult qml file we use.
[10:33] <dednick> tsdgeos: but at the same time, i added the override for the location indicator so that it gets a "friendly" title.
[10:33] <tsdgeos> dednick: where does the title come from otherwsie?
[10:33] <tsdgeos> dednick: and how do we i18n that title?
[10:34] <dednick> tsdgeos: at the moment if just uses the indicator name by default (eg indicator-datetime) if not handled in unity8. It's something that needs sorting out.
[10:35] <dednick> *it just uses
[10:35] <tsdgeos> dednick: so we are not providing i18n facilities for it either?
[10:37] <dednick> tsdgeos: nope
[10:38] <tsdgeos> dednick: because we'll have a better solution so not worth it? or?
[10:39] <dednick> tsdgeos: i'm not sure if we have a solution at hand, but this is not the final one for sure.
[10:39] <tsdgeos> ok
[10:39] <tsdgeos> not sure how important is i18n at this point
[10:39] <tsdgeos> i know paulliu was working on it for a while
[10:39] <tsdgeos> paulliu: you there? was your i18n push for a tradeshow or something?
[10:46] <dednick> tsdgeos: i just respaced those icons to 0.5
[10:46] <dednick> tsdgeos: with the addition of the location indicator we would be running out of space again anyway, and it wasnt much diff to 0.7 anyway
[10:47] <tsdgeos> ok
[10:47] <dednick> probably going to need a better solution anyway. we've still got loads of icons to go :(
[10:50]  * greyback_ not fond of 0.7 gu, it means we're not using the pixel grid that the whole shell UI placement should adhere to
[10:51] <nic-doffay> greyback_, ping
[10:52] <greyback_> nic-doffay: pong
[10:52] <nic-doffay> greyback_, are you familiar with the Icon class at all in the SDK? Particularly where the assets are stored which it loads by name?
[10:53] <nic-doffay> "image://gicon/%1".arg(icon.name)
[10:53] <greyback_> nic-doffay: not off the top of my head. Wellark or larsu probably know more than I about it
[11:06]  * greyback_ moving to office, back in 30
[11:08] <tsdgeos> hmmm
[11:08] <tsdgeos> can i link a static lib into another static lib?
[11:10] <tsdgeos> good old google
[11:11] <tsdgeos> so it seems "you can't"
[11:19] <tvoss__> tsdgeos, you can :) you just need the right linker magic
[11:19] <tvoss__> tsdgeos, see the magic whole-archive option approach here: http://bazaar.launchpad.net/~phablet-team/platform-api/trunk/view/head:/src/ubuntu/hybris/CMakeLists.txt#L19
[11:19] <tsdgeos> tvoss__: yeah well, that's exactly what's failing :D
[11:20] <tvoss__> tsdgeos, what does it say?
[11:20] <tsdgeos> my ua_url_dispatcher_session symbol ends up in src/ubuntu/hybris/libubuntu_application_api.so.1.0.0 but not in src/ubuntu/mirserver/libubuntu_application_api_mirserver.so.1.0
[11:21] <tsdgeos> even if both are linking to the .a
[11:21] <tsdgeos> i think it's because in one it's before the whole-archive
[11:21] <tsdgeos> and in another after
[11:21] <tvoss__> tsdgeos, ah, it's working, but the mirserver part is not pulling it in ...
[11:22] <tsdgeos> my question is why
[11:22] <tsdgeos> since it seems it should :D
[11:23] <tvoss__> tsdgeos, the issue is here http://bazaar.launchpad.net/~phablet-team/platform-api/trunk/view/head:/src/ubuntu/mircommon/CMakeLists.txt#L41
[11:23] <tvoss__> tsdgeos, lacking the whole-archive thingy
[11:23] <tsdgeos> i added it there
[11:23] <tsdgeos> but didn't seem to help
[11:23] <tsdgeos> let me try again
[11:35] <Cimi> mzanetti, ping
[11:37] <mzanetti> Cimi: pong
[11:37] <Cimi> mzanetti, was wondering what's the launcherDragArea and the EdgeDragArea
[11:38] <Cimi> the difference between the two
[11:40] <mzanetti> Cimi: one is for dragging in, the other for dragging out
[11:40] <tsdgeos> tvoss__: so, no i did not do it before, i did it on the laptop and then compiled on the phone \o/, but now that i do it on the phone doesn't work either, seems too much stuff is getting inside the whole-archive, let me paste
[11:43] <tsdgeos> tvoss__: http://paste.ubuntu.com/6032225/
[11:44] <tsdgeos> seems that for some reason ${UBUNTU_APPLICATION_API_LINK_LIBRARIES} is getting expanded to "a lot" ../application/location/libubuntu_application_location.a -lubuntu-location-service -ldbus-1 -lubuntu-location-service -ldbus-1 -lpthread -lboost_system ../application/url_dispatcher/libubuntu_application_url_dispatcher.a -lurl-dispatcher -lgio-2.0 -lgobject-2.0 -lglib-2.0 -lhybris-common
[11:44] <tsdgeos> and thus we get the dupe symbols
[11:44] <paulliu> tsdgeos: i18n for indicators? I thought I fix an issue about i18n in the past in the datetime. Yes for a trade show in China.
[11:44] <paulliu> tsdgeos: Already got merged I think.
[11:45] <tsdgeos> paulliu: well, it got broken again according to dednick :D
[11:45] <tsdgeos> Cimi: you say it should be fixed with the current code in that branch?
[11:46] <Cimi> tsdgeos, it works for me
[11:46] <tsdgeos> Cimi: because it still goes thorugh here, see calculator for example
[11:46] <paulliu> tsdgeos: Argh. I'll check...
[11:46] <Cimi> tsdgeos, mmm
[11:46] <Cimi> tsdgeos, I forgot my ubuntu phone at home, cannot test
[11:46] <paulliu> dednick: Do you have a screenshot?
[11:47] <dednick> tsdgeos: nothing was broken that wasnt previously.
[11:47] <dednick> paulliu: ^
[11:47] <tsdgeos> dednick: sure
[11:47] <tsdgeos> dednick: for some value of "previosuly"
[11:47] <tsdgeos> no?
[11:47] <tsdgeos> i mean it may have been working when paulliu fixed it and then broke again (not in this MR of course)
[11:47] <dednick> tsdgeos: well there was never i18n for indicators as far as i know
[11:48] <tsdgeos> ah
[11:48] <tsdgeos> ok
[11:48] <dednick> paulliu: ^ ?
[11:48] <tsdgeos> tvoss__: any hint?
[11:48] <paulliu> dednick: For the datetime there is.. So you can get Chinese date-time.
[11:49] <dednick> paulliu: i c. well that's up to the backend now. if they send a chinese label then that's what we display
[11:50] <dednick> paulliu: we are talking about titles. I dont think they are translated at the moment are they?
[11:50] <dednick> eg "Date and Time", "Battery", "Messaging"
[11:51] <paulliu> dednick: yes.. It is from the backend. And datetime strings is based on the locale. for example, LC_TIME..
[11:51] <paulliu> dednick: ah, that is not yet translated.
[11:51] <paulliu> dednick: I wonder that is not i18n yet.
[11:54] <dednick> paulliu: dont worry about it. it needs to be fixed anyway
[11:54] <paulliu> dednick: yeah.
[11:56] <larsu> nic-doffay: don't use image://gicon (we're in the process of deprecating it)
[11:57] <nic-doffay> larsu, yeah chatting to seb128 about in #sdk
[11:57] <nic-doffay> larsu, not on this server though :P
[11:58] <larsu> nic-doffay: I'm not in there to read the scrollback. I trust seb128 to give you the right advise though ;)
[11:59] <seb128> larsu, I basically pointed them to https://code.launchpad.net/~ubuntu-sdk-team/ubuntu-ui-toolkit/unity-theme-icon-provider/+merge/180805 and to ubuntu-themes to add the icon
[12:00] <larsu> seb128: thanks. Still not merged, though :(
[12:01] <larsu> Wellark: what's up with that ^^? I thought you needed it urgently?
[12:01] <seb128> larsu, somebody needs to change the top status
[12:01] <seb128> larsu, saviq mhr3 and you approved, why not doing that?
[12:01] <seb128> I guess somebody from the sdk team needs to review/push the trigger?
[12:02] <larsu> seb128: they blocked it on not having tests. Wellark said he'd write some, but I can't see any in the MR
[12:02] <seb128> larsu, try to ping the #sdk guys about it
[12:02] <mhr3> seb128, yea, as larsu said, tests missing
[12:02] <seb128> urg, still
[12:02] <mhr3> sdk guys wanted that
[12:03] <seb128> Wellark, ^ weren't you supposed to add some test?
[12:03] <mhr3> or convince sdk guys that it's not worth it :)
[12:03] <larsu> so the sense of urgency that people communicated to me two weeks ago was greatly exaggerated :P
[12:04] <larsu> Wellark wasn't feeling well yesterday, don't know if he's back yet
[12:04] <larsu> mhr3: well, the tests they'd be happy with are really not worth it :)
[12:04] <mhr3> but we want that branch anyway, otherwise there's no support for themed icons with fallbacks
[12:05] <larsu> seb128: I'll wait for Wellark to reply in case he has a half-finished branch on his machine. Otherwise I'll write those tests.
[12:05] <mhr3> well unless you count things like "image://gicon/.%20GThemedIcon%20foo%20bar
[12:05] <seb128> larsu, thanks
[12:05] <larsu> mhr3: we don't count that :)
[12:06]  * larsu certainly doesn't
[12:06] <greyback_> mhr3: please tell me that doesn't actually work
[12:08] <larsu> greyback_: for your own sanity, stop thinking about that (and don't try it)
[12:08]  * greyback_ steps away
[12:09] <larsu> haha
[13:03] <tsdgeos> greyback_: https://code.launchpad.net/~gerboland/unity8/u8m-osk/+merge/181887 ¿?
[13:04] <greyback_> tsdgeos: yeah I needed to land it, else if you used Mir image, once you opened the OSK, you could do nothing else except type
[13:04] <tsdgeos> ok
[13:07] <mzanetti> dednick: hey
[13:08] <dednick> mzanetti: howdy
[13:08] <mzanetti> dednick: how does that work in the indicators?
[13:08] <mzanetti> there is one dbusmenumodel which holds the icons in the panel, right?
[13:08] <mzanetti> and where do the tabs get their information from? not the same model?
[13:09] <dednick> mzanetti: there is the IndicatorsModel, which tells both the panel and the tabs "where to get their content"
[13:09] <dednick> mzanetti: and the UnityMenuModel, which tells each item within the Panel and Tabs "what is the content"
[13:10] <dednick> ie the Indicators model tells us the dbus path etc, and the UnityMenuModels connects to each dbus path
[13:11] <dednick> mzanetti: but we need to connect to the dbus before we can tell if the indicator is visible or not.
[13:11] <mzanetti> dednick: does that mean that we always have all tabs, regardless if a given system service is not installed?
[13:11] <dednick> mzanetti: yes, at the moment.
[13:12] <mzanetti> somewhat defeats the purpose of the whole thing, doesn't it?
[13:12] <dednick> mzanetti: somewhat the purpose of this branch :)
[13:13] <mzanetti> yeah well.. seems to be a workaround for an architectural problem deeper down in the stack
[13:13] <dednick> mzanetti: hang on. no, the service is deemed to be installed if the indicator file exists.
[13:14] <dednick> only the ones with indicator files will appear in the tabs
[13:14] <dednick> or panel
[13:14] <mzanetti> ok... at least something :)
[13:14] <dednick> the visibility is more than just "being able to connect to dbus"
[13:15] <mzanetti> hmm... still don't like the fact that we're abusing the model for this... but don't have a better idea right now
[13:15] <dednick> for example, the messaging indicator in future probably wont be visible unless we have unread messages
[13:15] <dednick> but it still needs to be connected to be able to make that determination
[13:16] <dednick> mzanetti: yeah, i didnt really like the fact that i was using setData.
[13:16] <dednick> mzanetti: i have another solution in mind, but it's a bit unqmlish.
[13:16] <mzanetti> dednick: could the panel have a list property controlling all the visible tabs?
[13:17] <dednick> mzanetti: yeah, that could be a better solution.
[13:17] <mzanetti> tsdgeos: you happy with this?
[13:18] <tsdgeos> sorry
[13:18] <tsdgeos> wasn't reading
[13:18]  * tsdgeos reads
[13:18] <mzanetti> no problem
[13:19] <dednick> mzanetti, tsdgeos: although that wouldnt really work nicely with the filtermodel.
[13:19] <dednick> as it operates on roles...
[13:19] <tsdgeos> mzanetti: yeah, you need a model role
[13:19] <mzanetti> d'oh
[13:19] <tsdgeos> that's why i suggested an intermediate identitymodel
[13:20] <tsdgeos> that adds the extra role
[13:20] <tsdgeos> but it's a lot of work
[13:20] <dednick> tsdgeos: identityModels dont add roles do they? they just transform current ones
[13:20] <tsdgeos> dednick: you can add more, that's not a problem
[13:21] <tsdgeos> dednick: the other "easy" solution is creating your own proxymodel and reimplementing acceptsrow
[13:21] <tsdgeos> but of course that's also more work
[13:22] <mzanetti> wait... shouldn't this behavior be controlled by the service? I mean... the service adds and removes all kind of stuff in the UI. shouldn't it also be able to determine when an entry is "active"?
[13:22] <mzanetti> which is when it is displayed and the tab is there
[13:23] <mzanetti> dednick: ^
[13:26] <dednick> mzanetti: it is. the visiblity comes from the service. but you need to make the connection (ie create the tab page) to determine if it's visible. Plus we can't change tab visibility.
[13:29] <mzanetti> dednick: I mean, this setData(isVisible) should happen within the service
[13:29] <mzanetti> ah... doesn't work again
[13:29] <mzanetti> crap
[13:30] <mzanetti> I don't know... if alberts approach works fine for you... otherwise I guess we have to stick with what you have now
[13:30] <dednick> mzanetti: heh. it does. but it has to go through the panel items as they are what is connected to the service. The IndicatorsModel has no connection to the service
[13:30] <mzanetti> yeah
[13:31] <mzanetti> dednick: standup
[13:31] <greyback_> nic-doffay: you too
[13:32] <sil2100> tedg: hi! https://bugs.launchpad.net/ubuntu/+source/indicator-messages/+bug/1217131 ;)
[13:32] <sil2100> tedg: (just poking)
[13:32] <tedg> sil2100, Yes, I know, looking at it.  Unfortunately it happens much more often on the builders than locally :-(
[13:33] <greyback_> kgunn: joining us?
[13:34] <sil2100> ;/
[13:35] <dednick> mzanetti: my potential solution was to have an intermediate model which connected to UnityMenuModel for row that was added to get the visibility.. but it's quite a bit of work.
[13:35] <dednick> like tsdgeos solution, but more pimping.
[13:36] <kgunn> Cimi: curious, you connecting vga, hdmi or display port ? (mir on projectors)
[13:36] <Cimi> kgunn, VGA
[13:36] <Cimi> kgunn, but it behaves very weirdly...
[13:37] <kgunn> Cimi: gpu ?
[13:37] <Cimi> kgunn, intel
[13:38] <Cimi> kgunn, test 1.1, 1.2, 1.3 seems to work
[13:38] <Cimi> kgunn, cannot do 1.4 and 1.5
[13:38] <Cimi> on scenario 2, 2.1 starts with a weird "mirror mode" with one screen longer than the other
[13:38] <Cimi> mirror displays however is off
[13:39] <nic-doffay> greyback_, doh got totally carried away.
[13:39] <greyback_> nic-doffay: we're still there
[13:39] <nic-doffay> greyback_, on me way
[13:46] <Cimi> scenario 3, I have crashes
[13:46] <mzanetti> sil2100: can you help me finding an issue in the release jenkins tests setup?
[13:47] <Cimi> I can disable primary screen, but then when I switch the projector off, it goes back to lightdm
[13:47] <sil2100> mzanetti: what's up?
[13:47] <mzanetti> sil2100: http://reports.qa.ubuntu.com/smokeng/saucy/image/3770/unity8-autopilot/306767/
[13:48] <mzanetti> sil2100: on Galaxy Nexus, all unity8 tests fail to open /dev/uinput
[13:48] <mzanetti> sil2100: works fine on Nexus 4
[13:49] <mzanetti> sil2100: can you point me to the scripts that do the setup before running the tests?
[13:49] <sil2100> mzanetti: did you make sure uinput is there and that the user permissions are set-up correctly?
[13:50] <sil2100> mzanetti: let me point you to the otto scripts that do the job, since there is some pre-config thing we're doing
[13:52] <sil2100> mzanetti: take a look here for instance: http://bazaar.launchpad.net/~otto-dev/otto/testsuite_autopilot-unity/view/head:/target-override/usr/local/share/otto/setup-hooks/10-autopilot
[13:53] <mzanetti> sil2100: and which is the jenkins job that produces the results for this page?
[13:53] <mzanetti> http://reports.qa.ubuntu.com/smokeng/saucy/image/3770/unity8-autopilot/306767/
[13:54] <sil2100> mzanetti: hah! This might sound bad, but I have no idea ;) I think you'll have to ask someone from the QA team, gema would probably know best
[13:55] <mzanetti> sil2100: ok. thanks
[13:55] <sil2100> mzanetti: since once it's out of the daily-release process I'm maintaining, I don't really coordinate the path
[13:56] <mzanetti> sil2100: same here... still we need to figure why unity fails. I just hoped you'd know more than me. But no problem. I'll find someone
[14:12] <kgunn> greyback_: ping
[14:12] <greyback_> kgunn: pong
[14:18] <mzanetti> asac: ping
[14:18] <mzanetti> kgunn: I've found the issue
[14:19] <asac> mzanetti: hi
[14:19] <mzanetti> asac: http://bazaar.launchpad.net/~ubuntu-test-case-dev/ubuntu-test-cases/smoke-touch-apps/revision/80
[14:19] <kgunn> mzanetti: awesome!!!
[14:19] <mzanetti> this commit broke it
[14:19] <mzanetti> asac: because it removes the call to unlock_screen
[14:19] <mzanetti> thich contains the chmod 666 to /dev/uinput
[14:19] <mzanetti> s/thich/which/
[14:19] <asac> mzanetti: cool. btw, we think we fixed the uinput problem for real
[14:19] <mzanetti> even better
[14:20] <asac> e.g. adding phablet to autopilot group
[14:20] <asac> which makes that unnecessary... also fixing the udev rules :)
[14:20] <mzanetti> +1
[14:20] <asac> so lets see what happens in 1-2 hours
[14:20] <asac> then we should see results coming in
[14:25] <mzanetti> asac: to be precise, in this one: http://bazaar.launchpad.net/~ubuntu-test-case-dev/ubuntu-test-cases/smoke-touch-apps/revision/80/unity8-autopilot/setup.sh
[14:25] <mzanetti> the NO_UNLOCK=1 is the culprit
[14:35] <mzanetti> nic-doffay: time for a review? https://code.launchpad.net/~mzanetti/unity8/fix-1213153/+merge/182406
[14:41] <larsu> dednick|lunch: apparently qmenumodel is not building on i386 (tests for qmenumodel are failing: https://bugs.launchpad.net/ubuntu/+source/qmenumodel/+bug/1217335). Do you still depend on QMenuModel (the class) or can we remove it?
[14:41] <nic-doffay> mzanetti, on it
[14:42] <mzanetti> mterry: I assume because you got dragged away for the html thingie, the demo branch is not fixed/ready yet, right?
[14:43] <mterry> mzanetti, demo should be fixed
[14:44] <mterry> mzanetti, stupid Q_SLOTS/Q_INVOKABLE mixup (I put Q_INVOKABLE after 'public' instead of in front of method prototype)
[14:44] <mterry> mzanetti, there's a qmltest that fails on jenkins but works locally. Just pushed a possible fix for that, but shouldn't block testing
[14:44] <mzanetti> mterry: hehe... and I've seen in the last version you still have a "pubic:" too much :P
[14:45] <mterry> mzanetti, you kidding me?  How does that compile?
[14:46] <mterry> oh, I don't have pubic!
[14:46] <mzanetti> public
[14:47] <mzanetti> stupid typos...
[14:47] <mzanetti> mterry: line 819 for example
[14:48] <mzanetti> mterry: not a real problem... but not needed either
[14:48] <mzanetti> 1262 too
[14:48] <mterry> mzanetti, ah, I see what you meaj
[14:48] <mterry> *mean
[14:48] <mzanetti> mterry: and 1609 there is still the slot thingie
[14:49] <mterry> mzanetti, 1609 needs to be I thought, for the QtDBus auto-binding stuff to work,
[14:49] <mzanetti> mterry: we really should define that API in lp:unity-api... would fail to compile until all of them are aligned
[14:49] <mzanetti> ah ok
[14:58] <dednick|lunch> larsu: the messaging still using qmenumodel
[15:00] <larsu> dednick: ah okay. So I guess I should have an actual look why the tests fail, then :)
[15:02] <dednick> larsu: fail on amd64 as well :/ works for me...
[15:02] <larsu> dednick: you have qt 5.1?
[15:02] <dednick> larsu: no
[15:04] <Wellark> seb128: why am I seeing this? http://paste.ubuntu.com/6032941/
[15:05] <Wellark> that's on todays touch image on my nexus 3
[15:05] <Wellark> I must be crazy or something but it states that the installed version of HUD is 13.10.1+13.10.20130816.1-0ubuntu1
[15:07] <Wellark> instead of that 13.10.1+13.10.20130822.4-0ubuntu1 which we worked so hard to fix critical bugs
[15:07] <larsu> Wellark: did you see our discussion about the theme icon provider from earlier today?
[15:07] <Wellark> larsu: nope
[15:07] <Wellark> larsu: quick recap please
[15:09] <larsu> Wellark: did you get a chance to work on the tests for that?
[15:10] <larsu> Wellark: I can't see any on the MR and I remember you needed the theme provider desperately two weeks ago
[15:10] <Wellark> larsu: doing them today after we get this critical HUD problem sorted out
[15:10] <Wellark> larsu: well, we all need it
[15:11] <larsu> Wellark: ah cool. Thanks. Yes, other people have been asking abou this, which is why I'm pinging you right now :)
[15:11] <Wellark> in the mean time I fixed upstream QIcon
[15:11] <larsu> ah, very cool
[15:11] <Wellark> and HUD
[15:11] <Wellark> and HUD
[15:11] <Wellark> and HUD
[15:11] <larsu> lol
[15:12] <Wellark> larsu: so yeah. unit tests to get that MR forward coming today
[15:12] <Wellark> or I will lock my self to bathroom and cry
[15:12] <larsu> haha, please don't :)
[15:13]  * larsu hears that is not good for mental stability
[15:13] <Cimi> mzanetti, you said the launcher jumps in with my launcher branch, how?
[15:13] <Cimi> mzanetti, you mean the shadow suddenly appears?
[15:13] <mzanetti> Cimi: no
[15:14] <mzanetti> Cimi: the EdgeDragArea has a minimumMovement
[15:14] <mzanetti> Cimi: which means, you need to drag for half a grid unit or so before the drag gesture is recognized
[15:15] <mzanetti> Cimi: once the gesture is recognized, the launcher shows up, already having moved for that distance...
[15:15] <mzanetti> Cimi: I think the easy fix for that is to make it visible onMousePress already
[15:15] <mzanetti> Cimi: because then the shadow will appear when you put the finger down and the rest happens as before
[15:16] <Cimi> mzanetti, but there is no onMousePress
[15:17] <mzanetti> Cimi: DirectionalDragArea::Status
[15:18] <Cimi> mzanetti, recognized?
[15:18] <mzanetti> recognized is when a drag is recognized
[15:18] <mzanetti> that's too late
[15:18] <Cimi> so what?
[15:21] <mzanetti> Cimi: how about Undecided?
[15:22] <Cimi> mzanetti, probably better opacity: 0 when is waitingfortuouch or no status?
[15:30] <mzanetti> Cimi: not only...
[15:30] <mzanetti> Cimi:  it also needs opacity = 1 when the launcher is revealed
[15:30] <mzanetti> Cimi:  but you're getting close, yes :)
[15:32] <Cimi> mzanetti, implying 1 all other cases
[15:36] <dednick> larsu: just upgraded to 5.1 . still passes :(
[15:36] <dednick> well. :)
[15:37] <larsu> dednick: let's close the bug as WORKSFORME :)
[15:38] <dednick> larsu: lol
[15:40] <larsu> dednick: ah! "QXcbConnection: Could not connect to display"
[15:41] <larsu> I guess we jut need a xvfb-run somewhere in there...
[15:41]  * larsu wonders why this worked before
[15:46] <tedg> sil2100, So larsu found a fix for the indicator-messages issue.  Can we kick off the stack manually to see if it's all green?
[15:50] <seb128> tedg, he's hosting a vUDS track, not sure he has time for IRC
[15:50] <seb128> tedg, let me have a look
[15:50] <tedg> seb128, Ah, okay, thanks!
[15:52] <seb128> tedg, build triggered
[15:52] <tedg> seb128, Thanks /me crosses fingers!
[15:57] <Cimi> mzanetti, opacity: (root.state == "" && dragArea.status [15:57] <mzanetti> Cimi: looks reasonable. did you make sure that it doesn't disappear too early? afaik the state changes to "" before the animation is done with hiding it
[15:58] <Cimi> mzanetti, could do x == -width instead root.state == ""
[15:59] <mzanetti> Cimi: yeah, that seems a good one
[15:59] <Cimi> mzanetti, you think we need a behaviour?
[16:00] <mzanetti> hmm... I'd add one.
[16:00] <mzanetti> but UbuntuAnimation.FastDuration
[16:01] <Cimi> mzanetti, do they already work?
[16:01] <Cimi> mzanetti, ubuntuAnimation
[16:01] <mzanetti> Cimi: also, need to check if that doesn't conflict when you drag in the laucnher very fast
[16:01] <Cimi> mzanetti, it's why I was asking
[16:01] <mzanetti> Cimi: yes, UbuntuAnimation works fine
[16:04] <Cimi> mzanetti, I pushed, if you could test on your phone...
[16:07] <mzanetti> Cimi: ok. not sure I'll make it today
[16:07] <mzanetti> Cimi: but I'll review tomorrow
[16:21] <dednick> larsu: ping
[16:22] <larsu> dednick: yep
[16:23] <dednick> larsu: is the only diff between g_object_unref & g_object_clear that it nulls the pointer after?
[16:24] <larsu> dednick: no, it also checks for NULL (this was more of an FYI then something you have to fix)
[16:25] <dednick> larsu: ok. thanks
[16:26] <larsu> Mirv: I have a patch that makes the qmenumodel tests work for me. Can you please test it? (branch is attached to the bug)
[16:27] <dednick> larsu: hm. did it just fail outright for you then?
[16:28] <larsu> dednick: when sshing into a vm :D
[16:28] <dednick> larsu: ah :)
[16:28] <larsu> I like using vms when adding ppas like that
[16:29] <larsu> and sshing seemed like the easiest way to get a no X situation
[16:29] <dednick> larsu: indeed.
[16:35] <seb128> bregma, hey, is anyone looking at unity failing publication because too many tests fail?
[16:37] <bschaefer> seb128, bregma is out until tomorrow, and we are trying to look at all the failures...but not a lot of stuff has changed in unity it self the last few days
[16:37] <bschaefer> seb128, hoping to focus on all the AP failures today...
[16:38] <seb128> bschaefer, ok, great, thanks
[16:38] <bschaefer> seb128, np! thanks for poking us!
[16:40] <Mirv> larsu: tested. added my approve, works now both locally and in the 5.1.1 PPA with your branch!
[16:40] <larsu> Mirv: awesome! Exactly the message I want to hear on my eod :)
[16:40] <larsu> dednick: ^^ can you approve it too, please?
[16:49] <dednick> larsu: sure
[16:52] <dednick> larsu: done.
[16:53] <dednick> larsu: my fixes are in UnityMenuAction branch as well.
[17:59] <TheMuso> is there anybody around from the unity 8 team who is able to jump into http://summit.ubuntu.com/uds-1308/meeting/21947/client-1308-touch-1404-a11y/ and join the discussion?
[18:02] <TheMuso> tvoss: ^^ Are you able to join, or could you point others who may be in a better position to discuss the situation?
[18:05] <Dash__> Hello, how can I deactivate the animation when the dash opens ?
[18:05] <tvoss> TheMuso, gimme5, with you then
[18:06] <TheMuso> tvoss: Sure, hangout URL is https://plus.google.com/hangouts/_/ff15e632070786351f22a797e5fcf553212c43a8?authuser=0
[18:22] <bschaefer> mhr3, quick fix will be landing soon!
[18:35] <kgunn> greyback_: just wondering when you phlash are you using -b ?
[18:35] <kgunn> wonder if that's a diff
[18:35] <greyback_> kgunn: not usually, unless things get very confused
[18:36] <kgunn> greyback_: specifically what image are on atm
[18:36] <kgunn> ive gone back and forth several times...pending+mir got lots of rendering glitches on dash
[18:38] <greyback_> kgunn: nexus4, yes I've heard of rendering glitches unfortunately
[18:38] <kgunn> ;)
[18:38] <kgunn> greyback_: but these aren't just in the panel area....this is why i thot i had no assetts....they are there, just weird
[18:39] <greyback_> kgunn: can you send me a screenshot/photo of the phone?
[18:40] <kgunn> greyback_: holy crap...it just recovered...what the what
[18:40] <kgunn> looks awsome now
[18:40] <kgunn> hmmm.....i was measuring framerate...killed that...then it got better....
[18:42] <greyback_> very odd
[18:42]  * greyback_ hopes that doesn't happen again
[18:43] <kgunn> greyback_: ah...weird..."stop unity" unity failed to stop...but, i ran unity again on top....
[18:43] <kgunn> so i had like 2 unitys ?
[18:43] <greyback_> kgunn: yep, it's on my todo list to prevent that
[18:44] <kgunn> greyback_: just checking...password for su phablet is "phablet" right ?
[18:44] <greyback_> kgunn: yep
[18:44] <kgunn> greyback_: this definitely worked earlier...but now unity is being stubborn
[18:44] <greyback_> kgunn: any strange output in your shell?
[18:44] <cyphermox> hey
[18:45] <cyphermox> would someone be so kind as to review this very simple merge request? https://code.launchpad.net/~mathieu-tl/unity-lens-applications/libcolumbus/+merge/182467
[18:45] <cyphermox> I'd be so very grateful :)
[18:45] <kgunn> phablet@ubuntu-phablet:/$ su phablet
[18:45] <kgunn> Password:
[18:45] <kgunn> phablet@ubuntu-phablet:/$ stop unity
[18:45] <kgunn> stop: Unknown job: unity
[18:45] <kgunn> phablet@ubuntu-phablet:/$
[18:45] <seb128> cyphermox, can do
[18:45] <cyphermox> seb128: thanks
[18:45] <kgunn> greyback_: unknown job unity?
[18:46] <greyback_> kgunn: "unity8"
[18:46] <kgunn> greyback_: you know...i was just thinking...did i accidently delete the 8 in my notes?
[18:46] <greyback_> kgunn: :D
[18:49] <TheMuso> greyback_: Whats your launchpad user ID?
[18:50] <greyback_> TheMuso: "gerboland"
[18:50] <TheMuso> greyback_: Thanks.
[18:50] <greyback_> TheMuso: thank you
[18:58] <greyback_> kgunn: everything ok?
[18:59]  * greyback_ had to go
[18:59] <greyback_> has
[18:59] <greyback_> o/
[19:02] <sil2100> \o
[19:15] <om26er> mzanetti, did the icon size in the launcher change recently? they look a bit smaller to me with today' image
[19:43] <mzanetti> om26er: yes. they are a bit smaller
[19:44] <om26er> mzanetti, they look to small  in comparison to the dash icons, probably something for design to consider
[19:44] <om26er> *too
[19:44] <mzanetti> om26er: this is what I got from design
[19:45] <mzanetti> om26er: actually it was a bug on my side that they were bigger. all the design specs had the smaller ones, right from the beginning
[19:45] <om26er> mzanetti, it could possibly be that my eyes got used to the old size afterall.
[19:45] <om26er> the continued use of the next few days will tell if they look weird still
[19:46] <om26er> mzanetti, offtopic: the quicklists appear on the launcher, shall i raise a bug for that ?
[19:46] <mzanetti> om26er: yes, if it turns out to cause issues I'm sure the size will be increased again
[19:48] <mzanetti> om26er: https://bugs.launchpad.net/ubuntu-ui-toolkit/+bug/1204466
[19:48] <om26er> mzanetti, ah - cool
[19:54] <mzanetti> om26er: did you see this before? http://s-jenkins:8080/job/generic-mediumtests-runner-saucy/2287/console
[19:56] <om26er> mzanetti, no, never saw that one. try a restart for that job ?
[19:56] <mzanetti> om26er: there are 4 of this in a row now :/
[19:58] <om26er> something is broken with that VM maybe ?
[22:25] <Eric> Dumb question: I want to try unity. How do I get it?
[22:26] <bschaefer> Guest50120, well what are you on right now?
[22:28] <Guest50120> Ubuntu 10.04.
[22:29] <bschaefer> Guest50120, hmm well you'll either have to upgrade to 12.04, or partition your current hard drive with 12.04 or 13.04 to avoid messing up your 10.04 version
[22:30] <Guest50120> Ah.
[22:31] <bschaefer> theres always this fun online tour: http://www.ubuntu.com/tour
[22:31] <bschaefer> but, it wont be like the real thing :)