=== salem_ is now known as _salem [07:13] jamesh: hi! [07:13] sil2100: hi [07:14] jamesh: yesterday evening someone pointed out some problems with the mediascanner packages in -proposed, strangely in the socket it's failing to build due to flacky unit tests for amd64 and i386 ;/ [07:15] jamesh: https://launchpad.net/ubuntu/+source/mediascanner/0.3.93-0ubuntu1/+build/4889104 [07:15] jamesh: https://launchpad.net/ubuntu/+source/mediascanner/0.3.93-0ubuntu1/+build/4889106 [07:15] sil2100: yeah. I just noticed that about half an hour ago when talking to thostr_ [07:15] jamesh: I guess that it would pass after some re-tries, but they asked me to poke you and maybe fix it [07:15] I'm looking into it [07:16] sil2100: once we have that issue fixed, is there anything else needed to get mediascanner on to daily-release? [07:17] jamesh: I *guess* not, we usually require integration tests, but those will be provided by the scope side, right? [07:18] sil2100: yes. Currently the scope only has unit tests, but I have some ideas about how to do some simple integration tests [07:20] jamesh: ok then, I'll just propose one more branch with some small packaging things that the guys pointed out, and then we can bootstrap it and add it to daily-release - once the unit tests are fixed up [07:20] Same for the scope [07:21] sil2100: since your intial package, we discovered and fixed a class of bugs that were causing crashes on ARM [07:30] jamesh: excellent - so now, all that is left is the unit tests problems, and maybe this small branch [07:30] https://code.launchpad.net/~sil2100/mediascanner/packaging_minor/+merge/180983 [07:31] I prepare all things from the daily-release side [07:32] sil2100: I think the test suite bug triggered by the tests being run in parallel -- one of the test fixtures creates and deletes temporary files in a well known path relative to $PWD. [07:32] sil2100: It's probably worth fixing, but a short term fix would be to not run tests in parallel [07:33] jamesh: the workaround is fine enough with me [07:40] sil2100: ping [07:42] dednick: you anything to do with the battery indicator? [07:42] tsdgeos: depends on what's wrong. [07:42] i have a red icon and then when pulling the battery indicator it tells me i've 83% charged :D [07:43] tsdgeos: yeah. i've seen that [07:43] dednick: where do i report a bug? unity8? [07:43] tsdgeos: probably against indicator-power [07:43] Hi All, I'm using the latest hud=13.10.1daily13.08.16.1-101 build. I noticed that the HUD stopped searching the indicators quite a while ago. Is that a known regression? [07:44] divansantana: yes [07:44] Can't seem to find a bug report on it. [07:44] divansantana: probably not much people was using it :D [07:44] Is it planned to be put back for saucy release? [07:44] I find it extremely useful. [07:45] i think not, but not sure, seb128 do you remember what are the plans regarding hud searching indicators ? [07:45] Should I file a bug report? Or is it a decision that will not be implemented for some reason? [07:45] divansantana, it's a bug [07:45] I loved it to search diodon clipboard for stuff. And other indicators. [07:45] you guys rock thanks for the quick response. [07:46] seb128 is there a bug report that is open to the public where I can view it? Couldn't find it. [07:47] tsdgeos: pong! [07:47] divansantana, bug #1165420 [07:47] sil2100: so yesterday i added some unittests for unity-mir, but they fail since they need the platform-api thing to be able of working [07:47] bug 1165420 in Unity HUD "Unable to access the messaging menu from HUD" [Low,Confirmed] https://launchpad.net/bugs/1165420 [07:48] sil2100: gerry and me were wondering how does mir itself runs the tests and if you had any idea how it would work for unity-mir [07:48] tsdgeos, there is no real "plan", it should work, that's a bug [07:48] seb128: oh, i thought it was deemed an acceptable regression when introducing the new hud, sorry about that [07:49] tsdgeos: give me a moment === swook` is now known as swook [07:49] tsdgeos seb128. Thanks again. cherio [07:49] tsdgeos, I would not be surprised if ted though that, but as far as I'm concerned it should be fixed before release [07:49] sil2100: https://code.launchpad.net/~aacid/unity-mir/application_manager_tests/+merge/180898 for reference, though the jenkins links are gone [07:49] seb128: oka [08:03] sil2100: for disabling parallel tests, passing --max-parallel=1 to dh_auto_test should do it, right? [08:05] jamesh: should be enough [08:06] jamesh: just put a comment there on why we do such a thing [08:06] So that the distro guys don't skin us alive [08:08] mzanetti, hey, some warnings from launcher (when you click between items) http://pastebin.ubuntu.com/6005590/ [08:08] mzanetti, also "QObject::connect: Cannot connect QQuickShaderEffectSource:: to ShapeItem_QML_122::onImagePropertiesChanged()" [08:08] Saviq: I'll check it out [08:09] mzanetti, but that's UbuntuShape being tricked into using a ShaderEffectSource, I htink [08:09] *think [08:09] mzanetti, do we have designs for the quicklist? [08:09] Saviq: yes... looks quite different that what I have now [08:09] mzanetti, like shouldn't it show up next to the launcher? [08:09] Saviq: but I'd prefer to do the themeing in a separate branch [08:10] Saviq: yes. it should show up next to the launcher, but that's a limitation of the popover currently [08:10] Saviq: bug is reported [08:10] mzanetti, you can force the horizontal alignment, can't you? [08:10] I don't think... unless I missed something [08:10] ah [08:10] not [08:11] mzanetti, well, you *could* but it'd be pretty hacky [08:11] Saviq: how? [08:11] mzanetti, you'd need an object aligned to where you want the popover to be [08:12] mzanetti, and pass that to Popover.open() [08:12] and provide { 'pointerTarget': launcherItem } or similar [08:12] no... it still will be above or below as long as there is enough space [08:13] already tried playing around with the pointertarget [08:13] mzanetti, sure, but above/below could be below that positioner object [08:13] mzanetti, but it's probably not worth it [08:13] right... [08:17] mzanetti, for count emblems - the "12" one looks bad aligned, ideas? [08:18] Saviq: I noticed that too late night yesterday... really weird. as I do horizontalAlignment: parent.horizontalCenter [08:18] no clue yet what happens [08:18] mzanetti, I think you should just fill and center the text [08:18] mzanetti, although that will be tricky with eliding [08:18] that would cause a binding loop [08:19] well, not necessarily [08:19] yep... and I need to adjust parent's with bases on implicitWidth [08:19] based [08:19] well, I'll give it a shot [08:19] morning [08:19] hey gerry [08:20] yello [08:20] :) [08:20] yellow is both your nicks on xchat [08:22] greyback, the correct response is "OOo!" [08:22] jamesh: btw. how much time would it take to prepare integration tests for the mediascanner scope? At least 1-2 basic ones for now [08:23] Saviq: if you say so :) [08:23] sil2100, you're hooked up on integration tests, aren't ya! ;) [08:24] Saviq: very ;p [08:25] I've been paid by the autopilot guys to push on AP integration test everywhere [08:26] sil2100: are you after a simple test of integration between the media scanner and its scope, or a full-stack one? [08:27] I can probably get the first up and running fairly easily [08:28] mzanetti, btw, you might want to try paintedWidth of Text instead of implicitWidth [08:28] hmm or is that gone already... [08:29] greyback: https://code.launchpad.net/~aacid/unity-mir/noprivate/+merge/180989 [08:30] tsdgeos: thanks! [08:30] jamesh: just a simple test would be enough, using autopilot if possible [08:32] Saviq: ping [08:33] mzanetti, yeah, s/implicitWidth/paintedWidth/ and it's fine [08:33] mzanetti, interesting is that paintedWidth is not documented... [08:33] dednick, pong [08:33] greyback: any idea why https://code.launchpad.net/~gerboland/unity-mir/add-surface-attribute-updates/+merge/180813 failed? [08:34] in ci i mean [08:34] Saviq: did we ever remove the old indicators-client from archive? [08:34] tsdgeos: yep, forgot licences, just pushed the fix [08:34] works fine here [08:34] ah [08:34] dednick, was it ever *in* the archive? [08:34] dednick, right, still there [08:34] https://launchpad.net/ubuntu/+source/indicators-client [08:35] didrocks, how do we drop that from the archive ↑ ? [08:36] Saviq: we need to keep the indicators-client from unity8 though :/ and from the other. [08:36] dednick, "from the other"? [08:36] *drop the other [08:36] dednick, right ;) [08:36] dednick, yeah, that's fine [08:37] dednick, we just need to drop the indicators-client source package [08:37] Saviq: apparently it's causing some issue getting the indicator-messages published [08:38] Saviq: all plugins are uneeded now, for sure? [08:38] didrocks, yes, they're built into unity8 now [08:38] I hope if you ship the same files, you handled the Replaces: tag like for indicators-client-plugin-sound [08:38] didrocks, no, we're not shipping the same files [08:39] ok [08:41] mzanetti, hmm... height: units.gu(2) for the progress, is that correct? the asset is 1.5 GU high @30 [08:42] interesting… [08:42] mzanetti, and... re: "opacity: root.progress / 10" - shouldn't we just clip the filling to the correct width instead? [08:43] Could not find source 'indicators-client/None' in saucy . Exiting. [08:43] but blocked in proposed at the same time [08:43] Saviq: the mockup I have has a thicker progress bar... but its also doesn't match with the asset [08:43] mzanetti, ok, as long as you're consistent with the design [08:43] Saviq: yeah.. can try clipping [08:44] ah, it has always been in proposed only [08:44] Saviq: the paintedWidth has a strange effect [08:44] mzanetti, as long as you're not expected to have rounded corners on the right [08:44] it makes everything elide after 123 [08:44] grrrr [08:44] mzanetti, orly? [08:45] new indicator-messages sigtrap on start [08:45] didrocks, sil2100: ^ known issue? [08:45] mzanetti, indeed [08:45] * didrocks flushes [08:45] Saviq: https://launchpad.net/ubuntu/+source/indicators-client/+publishinghistory [08:45] seb128: not that I know of [08:45] shrug, TED [08:45] didrocks, thanks [08:45] can we revert indicator-messages? [08:45] that's clearly not ready to land to distro [08:45] sil2100: taking care of that? ^ [08:45] sil2100, ^ [08:46] sil2100: can you as well list that as a pb from the indicator stack? [08:47] sil2100, didrocks: can we revert? [08:47] the thing just sigtrap on start [08:47] that's ridiculous it landed [08:47] we should revert and rebuild, yeah, waiting for sil2100 [08:48] seb128: wasn't aware of that, good to know - noting down [08:48] Reverting last change [08:48] sil2100, thanks [08:48] sil2100: feel free to push to trunk and rerun the stack [08:48] "last change" is the port to gmenu [08:48] Saviq, ^ btw (not sure if you are one of the those who are waiting on the new indicator-messages) [08:49] We probably should force indicator upstreams to each write a simple integration test that would check if 'my indicator is running' [08:49] dednick, read up ↑ [08:51] seb128: which indicator? [08:51] oh. messages [08:53] yes [08:54] yeah, i wasnt even aware it had landed until about 10 minutes ago [08:55] i'm pretty sure i'm running it now... [08:55] didrocks, dednick: pushing it directly to trunk [08:58] larsu: ^ [08:59] The indicators stack is being re-ran [09:02] seb128: where did you see the issue? dekstop? [09:02] dednick, yes it his "g_menu_item_set_detailed_action: Detailed action name has invalid format" when you have tb email accoutns [09:02] it hits* [09:02] which sigtrap [09:03] Saviq: in this paste http://paste.kde.org/pd97d1643/ , would you think there is a difference in the size of the progressbar-fill ? [09:05] pete-woods: ^ [09:05] dednick: hi - I've been sent to investigate the indicators messages fault [09:05] seb128: awesome work! [09:05] mzanetti, that a trick question? ;) [09:06] mzanetti, probably shouldn't [09:06] Saviq: well, the answer is yes, but I don't understand why [09:06] btw, I love it that "khaki" is a defined color :D [09:06] Saviq: yeah... you immediately know who has seen the video about QtQuick.Controls :D [09:07] pete-woods, hey, "awesome work" for what? [09:07] seb128: for figuring out the source of the error so quickly - helps me try and fix it :) [09:07] Saviq: lasts 15 mins but it's well worth it: https://www.youtube.com/watch?v=_6_F6Kpjd-Q [09:07] not that I've ever looked at the code before :/ [09:07] pete-woods, ah [09:08] pete-woods, seb128: can confirm [09:08] pete-woods: (process:4001): GLib-GIO-ERROR **: g_menu_item_set_detailed_action: Detailed action name 'src.imap://nickd@mail.canonical.com/%5BGmail%5D/Important' has invalid format [09:11] dednick: is this on the desktop or phone? [09:11] pete-woods: desktop. i just installed the latest messaging indicator. [09:11] seb128, you want the pot files of the wizard merged or in the wizard folder' [09:11] ? [09:12] Cimi, I've no strong opinion on it, do you have a preference? [09:12] seb128, separate [09:12] Cimi, that works for me [09:12] seb128, in case we want to move the app somewhere else [09:13] seb128, it's easier to grab the potfiles [09:14] dednick: and where did you get that error log? [09:14] the upstart log? [09:14] Cimi, right [09:15] pete-woods, run /usr/lib/indicator-messages/indicator-messages-service [09:15] pete-woods, with a tb account configured [09:15] pete-woods, it's going to sigtrap on start [09:16] seb128: thanks! [09:18] pete-woods, yw [09:23] seb128, who knows a bit how translations work here with qml? [09:23] seb128, I see a i18n.cpp file and header [09:23] seb128, with initTr [09:23] is that required? [09:24] Cimi, just use i18n.tr("string") [09:24] ok thx [09:25] seb128, how do I generate the wizard.pot file? :) [09:25] Cimi, look at po/po.pro [09:25] Cimi, just copy/tweak that I guess [09:28] thanks larsu :-) [09:32] jamesh: any ETA for those basic-basic integration tests for mediascanner scope? ;) [09:33] sil2100: I'll try to get them finished tomorrow. It's 5:30pm here now [09:33] jamesh: also, do you have a branch for the parallel tests prepared, or should I push that to trunk? [09:33] sil2100: https://code.launchpad.net/~jamesh/mediascanner/no-parallel-tests/+merge/180986 [09:33] Awesome, see it got approved [09:33] Cool! [09:40] seb128, dednick: I can't seem to reproduce this error - I have tb configured with canonical's imap server and some new e-mails, I've got G_MESSGES_DEBUG=all, but I don't get any console errors [09:41] however the "Inbox" or "Important" entry in the menu for new messages doesn't seem to appear [09:41] pete-woods: i guess it depends on your mail setup. [09:42] pete-woods: i can get you a stacktrace if you like. [09:42] dednick: what's your "account name" in tb? [09:42] mine's just my e-mail address [09:43] maybe I should stick some naughty characters like : in it [09:44] pete-woods: it's just 'Canonical [09:44] dednick, ping [09:44] without the ' [09:44] MacSlow: pong [09:44] pete-woods, do you have unread emails? [09:45] seb128: yes [09:45] Do you happen to know what errors like "GLib-GIO-CRITICAL **: g_dbus_connection_signal_subscribe: assertion 'object_path == NULL || g_variant_is_object_path (object_path)' failed" point towards regarding using UnityMenuModel? [09:45] pete-woods: in your tb, do you have folders [GMail]/Important ? [09:45] pete-woods, maybe it's due to the folder name, the one hitting the sigtrap is "dx-indicators" there [09:46] * pete-woods creates new folder with dash in the name [09:46] dednick, I've been trying to add code from my stand-alone example to the notification-frontend (in unity8) and don't receive anything via the model... just these errors. [09:46] dednick, I double checked the dbus-name/path numerous times... there are no typos. [09:47] seb128, pot file has to be added to bzr? [09:48] seb128: that doesn't seem to be it [09:48] MacSlow: can you pastebin me your UnityMenuModel setup code? [09:48] dednick: yes, that's one of the folders I've been using [09:48] Cimi, yes [09:49] dednick, http://pastebin.ubuntu.com/6005842 [09:50] hello [09:51] pete-woods, let me look [09:52] how can i make the Unity launcher bar have a solid (i.e. non transparent i.e. opaque) color? [09:54] if you don't understand what i mean, just see this picture: [09:54] http://imageshack.us/f/196/screenshotfrom201204280.png/ [09:54] picture source: http://askubuntu.com/questions/126556/is-it-possible-to-set-the-unity-launcher-to-a-nice-solid-color [09:55] seb128: have reproduced it now :) / :( [09:55] pete-woods, what did you change? [09:56] seb128: I trashed the gconf settings for the indicator, re-created my tb account, added various folders, set up gmail auto-filters for those folders, then sent myself some e-mails [09:58] MacSlow: hm. not really sure about that. only thing i can think of is that the bus is appearing before you set a path. but i think it's supposed to be async, so not really sure why. [10:00] pete-woods, I guess the format of the action is invalid, could be the ":" in "src.imap:..." [10:00] MacSlow: so you can pick up your model in the unityqmlmenumodel.qml example when changing the params? [10:00] dednick, why would that be different between using UnityMenuModel in unity8 (as part of the notifications) and a stand-alone python-example pulling data from the menu-model? [10:01] dednick, I've my own qml-example (not python... sorry) [10:01] dednick, but I'll try to use unityqmlmenumodel.qml [10:01] MacSlow: it shouldnt be different [10:03] seb128: that's it, yes [10:03] dednick, yup... all items show up as expected in unityqmlmenumodel.qml [10:04] seb128: the source_id that tb is providing is that long string, and the indicator is creating an action with that name [10:04] MacSlow: sure you're using the right version? havent installed to usr/local or something? [10:06] dednick, sure... unless unity8 (trunk, compiled from sources) picks up something else than system-wide installed libqmenumodel [10:06] MacSlow: well it might have if you have one installed in /usr/local. errrr [10:07] pete-woods, right [10:07] dednick, certainly no libqmenumodel.so in /usr/local [10:08] MacSlow: how about different loders within /usr/lib ? [10:08] loders=folders [10:08] pete-woods, we had somewhat similar issues in the past, e.g http://bazaar.launchpad.net/~indicator-applet-developers/indicator-messages/trunk.13.10/revision/344 [10:10] pete-woods, https://bugs.launchpad.net/ubuntu/+source/indicator-messages/+bug/1180302 was the same error [10:10] Launchpad bug 1180302 in indicator-messages (Ubuntu) "hit SIGTRAP errors after glib 2.37 update" [High,Fix released] [10:11] dednick, only one version for sure... http://pastebin.ubuntu.com/6005879 [10:12] seb128, does the language plugin work? [10:13] MacSlow: have you installed the one you're testing the examples with? [10:14] MacSlow: for sanitys sake. i'm just trying to verify that the code you're testing is the code your using in unity8 [10:14] dednick, no... I only used it once in-tree (when it was not merged/released) [10:14] seb128: maybe you should fix this for me ;) that fix has been lost along the way [10:15] dednick, also just did a fresh compile of my unity8 branch [10:15] MacSlow: what's the installed package version of libqmenumodel [10:15] dednick, 0.2.7+13.10.20130813-0ubuntu1 [10:16] weird. [10:17] Cimi, define work? it changes the locale, but not on the device because polkit doesn't work (logind registration is broken until we get lightdm) [10:18] seb128, doesn't change on my pc... [10:18] pete-woods, how come you ended up owning that fix? ted should be doing that... [10:18] Cimi, did you restart your session? [10:18] seb128, nope [10:18] MacSlow: i dont supposed there's any way for me to easily check out your code and test it on my system? [10:18] seb128, but the entry goes back to US english [10:18] Cimi, that's why, locales are not dynamic under linux [10:19] seb128, this is a probem [10:19] seb128: http://www.urbandictionary.com/define.php?term=Muggins [10:19] dednick, I could push the current state... to my branches... although I don't like to push such broken state usually... [10:19] seb128, on the wizard we must change locale on the fly [10:19] dednick, one sec... [10:19] MacSlow: you could just create new "-broken" branches. [10:20] dednick, yeah... hold on [10:20] seb128, dednick: do we have a bug report for this regression? [10:21] pete-woods, just step out and wait for ted to be online in a few hours if you ask me... [10:21] Cimi, good luck with that, try talking to xnox about it, ubiquity does it [10:21] seb128: well it's fixed now, and the indicator actually turns blue :) [10:21] Cimi, you might need to exit the mainloop and re-enter it or something, fun [10:21] :-\ [10:21] pete-woods, so it was the same fix I pointed to apply? [10:22] seb128: it was the same principle, that class seems to have been heavily worked on since then, but as the fix was simple, it was easy to forward port [10:22] Cimi: there is no way to actually change the locale of all the application threads. So in ubiquity, we iterate across each label, ask gettext for new translation with new locale, and replace the strings. [10:22] pete-woods, great, glad I helped then ;-) [10:22] Cimi: it's ugly =/ [10:23] seb128: as am I, otherwise I would have spend 2 hours in the gmenu docs :) [10:23] xnox, and will be uglier with qml.. [10:23] seb128, does the entry change for you? [10:24] Cimi, what entry? [10:24] seb128, locale [10:24] seb128, if I change us english to italian, click confirm [10:24] it goes back to us english [10:24] I restart the app [10:24] us english [10:24] Cimi, does it change LANGUAGE or LANG in ~/.pam_environment [10:25] no [10:25] good luck debugging it then :p [10:27] seb128, works for you? [10:27] Cimi, I don't know, you are one of the 15 people pinging me since this morning I didn't manage to get anything done yet today out or replying to questions on IRC, I need to get some work done [10:27] Cimi, it worked last time I tried [10:28] Cimi, it goes through accountsservice, that should still work [10:28] if not, open a bug [10:28] if you open one, try as well if language-selector works to change locale [10:38] jamesh: I had to switch branches, so I'll re-do your parallel-fix and merge it in [10:42] bregma: hi! Before switching to the new compiz daily-landing, could you review/approve https://code.launchpad.net/~sil2100/compiz/remove_unity_support_0.9.10/+merge/179866 ? [10:42] bregma: last time CI failed, cannot see why, but this change we have released in saucy so I want to propagate it here too [10:44] bregma: once this is in, I'll enable compiz 0.9.10 for daily release again [10:45] didrocks: ^ fine with that? It has been dogfooded by me in the past, bregma said they also did that [10:47] bregma: there's an ABI break in compiz, right? I mean, we need to rebuild unity to work with compiz 0.9.10? [10:55] andyrock: ^ [10:55] ;) === MacSlow is now known as MacSlow|lunch [11:06] who provides RoleClickable? [11:06] libunity? [11:07] sil2100: fine with me :) [11:07] sil2100: maybe bump the unity version on latest compiz then? [11:08] ah, me has updates [11:08] * tsdgeos installs [11:09] didrocks: what do you mean? You mean bumping the compiz dependency in lp:unity? [11:09] sil2100: right [11:11] didrocks: will try, but need to make sure the lp:unity merge gets in then [11:11] yep ;) [11:16] sil2100, will review, and I don't believe there is an ABI break but andyrock can confirm [11:18] bregma, andyrock: thanks! [11:24] sil2100, there is an ABI break for what I know [11:24] i'm quite sure [11:25] andyrock: hmm, ok, then we have to resolve this somehow before we enable daily ;) === tvoss is now known as tvoss|lunch [11:30] mzanetti, couldn't we just have Item { width: foo * progress; clip: true; BorderImage { width: bar; anchors { top: parent.top; bottom: parent.bottom; left: parent.left } } } [11:30] mzanetti, for the clipping? [11:30] mzanetti, the -rightMargin seems somewhat scary [11:31] Saviq: sure. but its more to type and margins are said to be faster than absolute widths, no? [11:31] mzanetti, if they're changing, sure, but that would be static [11:31] mzanetti, and now you're recalculating the margins of the fill as well [11:31] andyrock, sil2100, all we should have to do is bump the Compiz build-depends in Unity and rebuild Unity, no? [11:32] mzanetti, or at least re-applying it [11:32] mzanetti, and it's much easier to read, too [11:35] mzanetti, re: centering the label, what you did is somewhat hackish, too, didn't find a better way? any idea what was the deal with the misaligned 12? [11:36] Saviq: No... don't know why the horizontalAlignment didn't work [11:36] didrocks: can I, in this case, do something like this or am I over-complicating things: push a 1:0.9.10~ manually dputted version of compiz to daily-build, then bump the compiz dep and request a merge, get it in and then roll the daily-release machinery with compiz enabled? ;) [11:37] didrocks: i.e. so that the lp:unity merge gets in properly [11:37] sil2100: hum, I think it's better to have one version of unity deliver [11:38] sil2100: then, you directly bump the version in trunk for lp:unity [11:38] (without MP) [11:38] and next daily release run will do what it needs [11:38] wdyt? [11:39] mzanetti, looks good here http://paste.ubuntu.com/6006104/ [11:39] mzanetti, could you try and see if you can reproduce on top of that example? [11:40] ok [11:40] didrocks: ok, sounds simpler ;p I just don't feel safe direct-pushing to lp:unity [11:41] sil2100: for trivial stuff like that, I think it's fine [11:41] sil2100: but I would really suggest having one unity delivery just before [11:41] and then one with just the compiz + unity rebuild [11:42] didrocks, I've had two questions about quarternourly release, please ping when you have a moment [11:42] didrocks: I'll try to make the unity stack green now during this tick that's running [11:42] Saviq: now is fine, I didn't restart yet on the system update [11:42] sil2100: good! [11:43] bregma, andyrock: when the next tick will be near, I'll prepare everything for the compiz switch - will give you a heads up once that's done [11:43] Since as didrocks mentioned, we want to have one last unity release before the switch [11:43] mzanetti: ping. see your "other" irc :) === hikiko is now known as hikiko|lunch [11:43] didrocks, a) why doesn't Unity8 stack depend on Unity? b) could we force unity upgrade for Unity8 stack? [11:44] Saviq: do you use some unity components? (apart from the scopes) [11:44] mzanetti: when you have the time, of course [11:44] Saviq: IIRC, you have your own copy on libunitycore [11:44] Wellark: yeah. I've seen it. Why can't you do the change in one commit? [11:44] didrocks, nope [11:44] sil2100, thanks [11:44] didrocks, we're using libunity-core [11:44] didrocks: unity8 uses libunity [11:44] or at least one review [11:44] Saviq: I wasn't refreshed on that :p [11:44] sil2100, -core [11:44] so yeah, unity8 will need to dep on unity [11:44] didrocks: ...-core [11:44] didrocks, it was always like that :) [11:44] ;) [11:44] which means, way less release I'm afraid [11:44] (seeing how flacky the unity tests are…) [11:44] didrocks, we never had a copy of libunity-core [11:45] Saviq: ok, I remember you copied unity [11:45] then, I thought you told me you won't need it [11:45] but I probably mixed it [11:45] didrocks, yeah, some miscomm [11:45] Saviq: so, the issue is that unity doesn't release often [11:45] it's often blocked on flacky tests [11:46] so if you dep on it, be prepared to have less releases, or we need finally someone on bregma's team looking at those tests [11:46] and cleaning to have the flackyness way down [11:46] didrocks, which might be a good thing [11:46] Saviq: agreed, not like if I never asked for it :p [11:46] didrocks, there's going to be more asking then ;) [11:47] didrocks, and if we see it's not working for us, we can always undo, can't we? [11:47] sure [11:47] sil2100: interesting in doing that? will be a nice additional to the FAQ as well :) [11:47] mzanetti: well, there is no reason why I couldn't squash them to just single commit. then again, each of the commits is self contained (except the unit test which needs all the previous ones) [11:47] sil2100: for stack dependency, it's just about adding it and then pulling on magners [11:48] didrocks, it's not like we're not looking at the tests trying to get them down, but the solution(s) appear elusive [11:48] Saviq: maybe let's wait for sil2100 to have unity green at least? [11:48] didrocks, sure [11:48] didrocks, it isn't something we need asap [11:48] didrocks, what about b) [11:48] bregma: the whole new compiz will fix everything I'm sure ;) [11:48] mzanetti: but if gerrit is more or less unable to clearly indicate that these changes belong together (except the hidden dependency chain) then I have no problems squashing them [11:48] Saviq: ok, I didn't really parse B though :p [11:49] I just would prefer smaller commits over a bigger one [11:49] didrocks: yeah, let's do that after this release [11:49] didrocks, for Unity8 check, *unity* wasn't upgraded to the latest available version [11:49] mzanetti: maybe I don't have the git^H^H^Hright mentality :) [11:49] didrocks, *unity* meaning libunity / unity / libunity-core etc. [11:50] Saviq: so the check failed, right? [11:50] didrocks, yes, because there was a crasher we were causing in libunity [11:50] ok, so it worked as intended :) [11:50] then we have an option for relaunching the check [11:50] which is "upgrading the whole ppa" [11:50] (it should be able to specify stacks to be more picky TBH) [11:51] We did that for one run [11:51] didrocks, ok, so that's the intended solution? was wondering if we can upgrade the whole ppa every time [11:51] not that it makes sense, just wondering [11:51] Saviq: no, because you don't detect ABI breakage [11:51] this is to detect that specific case, ABI breakage between 2 stacks :) [11:51] and we have that option to take "moar packages" [11:52] (but it needs to be more fine-grained than whole ppa, if only I had time for that :p) [11:52] (ideally, even automatically to define which stacks need to be released altogether) [11:52] * didrocks dreams… [11:52] anyway ;) [11:53] didrocks, ok, that works for me [11:53] Saviq: I wonder though if we shouldn't have one or two unity8 tests in the unity stack [11:54] to ensure unity doesn't break you (without bumping the libunitycore ABI version) [11:54] didrocks, sure, that could do [11:54] didrocks, we still need to improve our coverage on integration with scopes (from 0...) [11:55] didrocks, so those would probably be good candidates [11:55] Saviq: 0 is giving room from improvement :) [11:55] Saviq: ok, keep us posted! [11:55] for* [11:55] didrocks, indeed [11:55] Saviq: what is actually the difference of your example to my code? [11:56] mzanetti, ideally none, right? [11:56] thats exactly how I do it, except I need to set a maximum width for the eliding [11:56] mzanetti, yeah, that's why I wanted to look if we can reproduce [11:57] ah... you mean I should break your example [11:59] mzanetti, yes :) [11:59] mzanetti, here's a closer one even http://paste.ubuntu.com/6006165/ [11:59] still working [12:00] kgunn, you here already? [12:02] Saviq: http://paste.kde.org/p1c05777a [12:02] Saviq: interestingly it breaks the "1" here instead of the "12" [12:03] mzanetti, both 1 and 12 here [12:03] 12 looks good here [12:03] Saviq: I'm running at 18 px/gu [12:04] not sure if that's something to do with it [12:04] mzanetti, right [12:04] mzanetti, it does [12:05] mzanetti, so, Qt bug? [12:06] Saviq: I'd say yes === MacSlow|lunch is now known as MacSlow [12:07] Saviq: anyways, the emblems branch has been reviewed by jounih. He's happy with it as is === tvoss|lunch is now known as tvoss [12:09] I'll move over to do some theming :) [12:10] mzanetti, well, *I'm* not yet happy with the solution lol [12:10] mzanetti, or the fact that we don't know where it comes from [12:10] Saviq: which one? [12:10] hmm... [12:10] dunno. looks good to me [12:10] mzanetti, the "center a big label in a small Item to center the text" [12:12] Saviq: don't really see the problem... the "big" label is just to have a maximumWidth [12:12] Saviq: in the end we work with the implicitWidth which is the real one [12:12] mzanetti, the problem is we're drawing too many pixels ;) [12:13] do you think a label paints the transparent pixels? [12:14] mzanetti, they're definitely sampled [12:14] mzanetti, http://paste.ubuntu.com/6006201/ [12:14] mzanetti, smaller example - broken [12:14] mzanetti, but well, I can see what the deal is [12:15] Saviq: your last example is not broken here === alan_g is now known as alan_g|lunch [12:16] Saviq: if I change the text to "12" it breaks here too [12:16] dafuq [12:16] mzanetti, interesting, anyway, I'm thinking the cause is: anchoring to the sides of a parent whose width depends on implicitWidth [12:16] mzanetti, it doesn't obviously cause a binding loop [12:16] yeah... it does [12:16] mzanetti, but the changing width of the Text must be fucking the alignment somehow [12:17] hmm... [12:18] mzanetti, and paintedWidth has the same [12:18] mzanetti, although it does complain [12:18] test.qml:10:9: QML Rectangle: Binding loop detected for property "width" [12:19] mzanetti, ugh, obviously [12:19] mzanetti, it's eliding [12:19] Saviq: check out my last commit [12:19] mzanetti, 'cause it doesn't know whether to elide if it doesn't know the width, but it doesn't know the width if i doesn't know to elide [12:20] right... [12:20] Saviq: anyways, my last commit hopefully kills your concern about sampling too many pixels :D [12:21] mzanetti, that will actually cause the same loop === dednick_ is now known as dednick|lunch [12:21] mzanetti, even if we're not seeing this now [12:22] mzanetti, so indeed a static width might be the best we can do [12:22] mzanetti, 'cause it's the only way for it to be deterministic [12:22] mzanetti, please undo the last commit [12:22] Saviq: done [12:22] mzanetti, thanks [12:25] mzanetti, we will probably want to make the app icon a common component between launcher and apps scope [12:26] mzanetti, to keep the exact same visuals [12:26] mzanetti, but that's just FYI === swook is now known as swook` [12:26] I wonder if we want progress and count emblems in dash, btw (other than installation progress, that is) [12:27] no idea [12:29] Saviq: so, you think its good enough now? [12:29] mzanetti, yeah, just having a last look through [12:30] * mzanetti is eager to set more blueprint entries to DONE [12:30] :D [12:30] mzanetti, don't you miss anchors.fillHorizontally: parent, for example? [12:30] mzanetti, and/or anchors.fillBottom: parent [12:30] Saviq: where exactly? [12:30] mzanetti, no no [12:30] mzanetti, in QtQuick ;) [12:31] ah... [12:31] yeah... totally [12:31] mzanetti, I mean some shortcuts to all the anchoring [12:31] tons of stuff I could think of [12:31] fillBottom would anchor left, right, bottom === hikiko|lunch is now known as hikiko [12:31] yeah... exactly [12:31] horizontally left, right [12:31] etc. [12:31] once would need that so often [12:31] one would need that so often [12:32] yup [12:32] Saviq: also, if you want to have 3 same margins and only one other [12:32] I just tried if anchors { fill: parent; bottom: undefined } works [12:32] but no [12:32] mzanetti, yeah, and that [12:32] Saviq: it works by setting anchors.margins: 5; anchors.topMargin: 10 [12:32] but I think that's pure luck [12:32] mzanetti, not necessarily [12:32] mzanetti, otoh yeah, maybe ;) [12:33] Saviq: once the order of statements matter in QML there is something wrong [12:34] mzanetti, yeah, but you could go with priority [12:34] true [12:34] mzanetti, if you set the "more concrete" prop, it overrides the other one [12:34] regardless of the order [12:35] mzanetti, happroved [12:35] \o/ [12:35] didrocks: unity check job is green! The build job was red, but it was caused by a powerpc failure of libunity [12:35] didrocks: a rebuild fixed it [12:35] \o/ [12:35] didrocks: can I run publish? (first without force) [12:35] sil2100: sure [12:36] sil2100: how many failures for tests? [12:37] didrocks: around 13 [12:37] per-platform [12:37] ok, equally reparted? [12:37] btw. ati gone missing :o [12:37] didrocks: intel has more, but I guess that's because of some test flackyness [12:37] 8:01:32 didrocks | Mirv: FYI, I deprovisionned the ati machine for the Mir team to be able to test and debug [12:37] 8:01:42 didrocks | (so tests are running on both intel and nvidia only) [12:38] Ok ;) [12:38] sil2100: can we have bregma's team looking at them? [12:38] didrocks: sure thing, noting it down in the spreadsheet and poking them [12:38] mzanetti, did you see awe's messages yesterday? about being unable to find out the number of tries until you try? :D [12:38] (for SIM/PUK entry, that is) [12:38] Saviq: yeah [12:39] Saviq: didn't I send you the link to the bug report requesting your input? [12:39] mzanetti, don't think you did [12:39] hmmm... I was sure I did... anyways [12:39] 13 failures per platform with about half being one-off inexplicable unrepeatable failures is about par for the course [12:39] Saviq: https://bugs.launchpad.net/ubuntu/+source/ofono/+bug/1206941/ [12:39] Launchpad bug 1206941 in ofono (Ubuntu) "[ofono][rild] "Retries" property does not not work" [High,Confirmed] [12:40] mzanetti, checking how the N9 behaves [12:41] mzanetti, at least it did show the number of counts on SIM replace [12:41] Saviq: of course it does [12:41] mzanetti, same on reboot [12:41] Saviq: all phones do except android [12:41] Saviq: unfortunately we're piggypacking on the worst implementation [12:43] does anyone have compiz puking these messages into .xsession-errors over and over: compiz (opengl) - Error: FBO is incomplete: GL::FRAMEBUFFER_INCOMPLETE_ATTACHMENT (0x8cd6) ? [12:44] Saviq: also the fact that you can't verify if a PUK was entered correctly before changing the pin is quite bad imo [12:46] Saviq: the BB Z10 can read the remaining attempts too [12:48] mzanetti, yeah, it shows that no one normal uses the SIM PIN anyway ;) [12:49] Saviq: I wouldn't say so [12:49] mzanetti, do you? [12:49] (use) [12:49] Saviq: I don't use it because I have a prepaid card... but I would have a contract where the bill directly ends up on my bank account I sure would use the PIN [12:49] you don't, I don't, that's settled, then ;) [12:49] ... but if I would... [12:50] I do... [12:51] I think it is quite useful... if you loose the phone someone can call some porky hotline which might end up with thousands of euros of phone bill - depending on how fast you manage to reach someone that can lock down the card [12:52] I know what it's there for ;) [12:52] anyways... is rild a closed blob? [12:52] mzanetti, even if rild isn't [12:52] mzanetti, HAL is [12:53] mzanetti, and I expect this to be missing from the HAL if RIL didn't need it :/ [12:54] ok... so I guess the only thing we can do is not to show the label while we don't know the number... [12:56] someone has an android phone and could check how it behaves there? I still can't believe this isn't possible [13:03] https://code.launchpad.net/~mzanetti/unity8/launcher-less-warnings/+merge/181035 [13:05] Is there a Qt-equivalent to Gio? Or do I have to use Gio's API to export a menu-model? === _salem is now known as salem_ [13:13] sil2100, ping [13:16] MacSlow: does this help you? https://launchpad.net/libdbusmenu-qt [13:17] MacSlow: at least there is a class called DBusMenuExporter [13:17] tvoss: pong [13:17] mzanetti, I'll try [13:19] sil2100, I need help with the famous ati machine [13:19] sil2100, how can I restore the lxc configuration? [13:21] MacSlow, ping [13:22] MacSlow, got an n4? [13:24] tvoss: ah, hm, you can use lxc-start and lxc-attach for this [13:24] tvoss: first check the available containers by lxc-ls [13:24] (usually lxc-ls --fancy gives more info) [13:25] tvoss: you need root permissions for that, so sudo everytime [13:26] tvoss, nope... just the GalaxyNexus [13:29] mzanetti: do you have a nexus4 ? [13:29] tvoss: ^ [13:29] kgunn: I bought one myself, yes [13:29] kgunn: which I intended to use for dogfooding [13:30] MacSlow: nexus4 ? [13:30] MacSlow: oops...nvmd, just saw the scrollback [13:30] kgunn, nope... just the GalaxyNexus [13:30] kgunn, np [13:30] greyback, dednick|lunch, nic-doffay standup [13:31] Saviq, on my way. === dednick|lunch is now known as dednick [13:36] katie, Saviq: if you have time, I'd appreciate re-reviews of my demo branch. https://code.launchpad.net/~mterry/unity8/demo/+merge/177923 [13:36] mterry, yup, will do for sure [13:36] MacSlow: wow...impossible to decipher, very digitized voice....can you just update notes ? === alan_g|lunch is now known as alan_g === rachelliu_ is now known as rachelliu [13:48] mzanetti, what's happening with emblems? [13:49] Saviq: got a last minute design feedback [13:49] mzanetti, ok [13:49] Ah, shit, my merge didn't get merged again [13:49] Saviq: need to make the font bold and move the progress bar down half a gu [13:49] mzanetti, k [13:49] dednick, didrocks: if you don't mind, I'll do the switch on the next tick ;/ [13:50] sil2100: switch? [13:51] Saviq: this is it now: http://wstaw.org/m/2013/08/20/plasma-desktopD13243.png [13:51] Saviq: feel free to re-approve when you have time [13:52] mzanetti, done [13:52] cheers [13:52] Saviq, Hi! just an fyi, I am enabling autopilot tests for unity8 to run on touch devices during each MR - sounds fine ? [13:54] om26er, http://reports.qa.ubuntu.com/smokeng/ says YES! [13:54] sil2100: sure, don't risk! :) [13:55] tsdgeos: greyback .... if i wanted to help racarr in bisecting images, are these instructions still good ? [13:55] https://wiki.ubuntu.com/Touch/Testing/Mir [13:55] or...do you have a different/better crib sheet on how-to [13:56] kgunn: should be good yes [13:57] tsdgeos: ta [13:58] kgunn: yes, they are [14:05] mterry, sure.. === jono is now known as Guest90491 === greyback is now known as greyback|needsfo === greyback|needsfo is now known as greyback|food [14:12] seb128, https://bugs.launchpad.net/ubuntu-system-settings/+bug/1214417 [14:12] Launchpad bug 1214417 in ubuntu-system-settings "cannot change language" [Undecided,New] [14:12] Cimi, thanks [14:13] Saviq: not if you've seen it before: https://code.launchpad.net/~mzanetti/unity8/launcher-less-warnings/+merge/181035 [14:14] mzanetti, "selectedItem !== undefined && selectedItem !== null" selectedItem && should be enough [14:15] Cimi: you've been working with themes already. do you have an example for me of a themed app? [14:15] mzanetti, what do you need to do? [14:15] Saviq: does if(selected) check for undefined too? [14:15] mzanetti, sure [14:15] mzanetti, to completely theme popovers [14:15] mzanetti, best way is to use delegate [14:16] Cimi: actually only one... but I know what to do to have only one themed... so I'd need to start with generally creating a theme and loading it first [14:16] mzanetti, we don't need a theme [14:16] quicklists? [14:17] Cimi: yeah [14:17] we want popovers of the shell to remain the same [14:17] just the quick lists to chang4e [14:17] exactly [14:17] so you simply create a delegate for those popovers [14:19] mzanetti, http://bazaar.launchpad.net/~phablet-team/camera-app/trunk/view/head:/ZoomControl.qml [14:19] mzanetti, look at Slider [14:19] line 56 [14:19] style: ThinSliderStyle [14:19] so instead of using the style by the suru theme, uses this delegate [14:20] in the same folder of this file there is ThinSliderStyle.qml which is nothing else than the delegate [14:20] Cimi: can I inherit the existing style from Suru? I'd just need to change one property [14:20] nope [14:21] mzanetti, which property? [14:21] need to check. one moment [14:24] Cimi: hmm... I need to understand more how this works... but your pointers help a lot already [14:24] Cimi: I might come back to you a bit later [14:25] ok cool [14:27] greyback|food, tmoenicke is on holidays after all [14:29] Saviq: tried it out. selectedItem && is not enough [14:29] undefined !== null [14:30] mzanetti, http://paste.ubuntu.com/6006603/ [14:30] mzanetti, sure, undefined !== null [14:31] mzanetti, but (bool)undefined === (bool)null [14:31] === false [14:31] but but.. [14:31] I get LauncherPanel.qml:236: Unable to assign [undefined] to bool [14:32] mzanetti, that's because undefined && true === undefined, unfortunately [14:32] mzanetti, so you need to "cast" to bool [14:32] right. [14:32] can't we go with "!null && !undefined && something" ? [14:33] mzanetti, sure [14:33] mzanetti, !undefined === true [14:34] mzanetti, !!undefined === false ;D [14:34] ~ [14:35] andyrock: could you take a look at this? [14:35] andyrock: http://s-jenkins:8080/job/compiz-team-compiz-0.9.10-saucy-amd64-autolanding/4/console [14:35] andyrock: my merge is failing because of those 2 unit tests failing [14:36] andyrock: it's not related to my changes, as uh, I'm just removing a startup-patch === greyback|food is now known as greyback [14:40] Saviq: ack, thanks for finding out [14:40] greyback, bfiller offered to help with the review [14:40] greyback, so just push it up and we'll get it merged somehow [14:41] Saviq: it'll be one of the last things to approve, as the MR will break OSK on non-mir images [14:42] sil2100, I can't acccess s-jenkins [14:42] greyback, ah ok... could it not? i.e. could it detect runtime? [14:42] andyrock: ah [14:43] Wait... [14:43] Saviq: hmmm, maybe. I can have a look, once I've everything else ready [14:43] greyback, thanks === alan_g is now known as alan_g|tea [14:45] mzanetti, all rght? [14:46] Cimi: yeah... the Popover lacks some stuff to do this... but relatively easy patches [14:49] tsdgeos, https://bugs.launchpad.net/ubuntu-ui-toolkit/+bug/1179569 [14:49] Launchpad bug 1179569 in Ubuntu UI Toolkit "[panel] Unintentional swipes can close Panel (need threshold)" [Critical,Fix committed] [14:49] tsdgeos, we should be ready to enable dash bottom swipe [14:49] awesome [14:49] Cimi, you know the bottom bar goes away in the dash? ;) [14:50] Saviq, now I do :P [14:50] Cimi, in favor of tab-like navigation between scopes [14:50] Cimi, but it's rather on the tail of the TODO list so.. [14:50] ah ok.. [14:50] Saviq, regarding the HUD [14:50] Saviq, I have a blueprint [14:50] Cimi, yup [14:51] Saviq, got a link to do doc? [14:51] to the [14:51] Cimi, I don't think there is one... [14:51] Cimi, let me see if I can find something [14:51] * Cimi searches [14:54] Saviq, any joy? === alan_g|tea is now known as alan_g [14:57] Cimi, not yet [14:59] vesar, hey, can you brief Cimi on the changes to HUD button and transitions? [14:59] vesar, I can't find any spec for that [14:59] (or the presentation Oren showed in IoM) [14:59] vesar, tomorrow I'm in the office if you prefer [14:59] but I have time now === alecu_ is now known as alecu [15:04] Saviq, Cimi : Sure I can give you brief. Maybe better to do it tomorrow if you Cimi are going to be at the office then. [15:05] vesar, ok works for me [15:05] Saviq, Cimi: So what are the changes exactly you're planning to implement here? [15:06] vesar, the three TODOs for cimi in https://blueprints.launchpad.net/ubuntu/+spec/client-1303-unity-hud-2-ui [15:06] Saviq, Cimi: Because my understanding is that there is not full design ready for it yet. [15:06] Saviq, let me check the blueprint [15:07] Saviq, Cimi: ok those changes are ok to do now. No worries I brief you Cimi tomorrow. [15:08] vesar, cool thanks [15:12] Saviq, will you ignore me again if i ask about uri activation? [15:14] mhr3_, did I ignore you before? [15:14] yep :) [15:15] mhr3_, sorry [15:15] Saviq, so once again, do we need to do something special about http uris, or will those magically start working at some point on the phone? [15:15] mhr3_, lp:url-dispatcher [15:15] mhr3_, you'll need to put them through (lib)url-dispatcher [15:16] mhr3_, that will launch the corresponding app (based on url / mimetype) through upstart [15:16] shouldn't we implement some abstract qt thing to do that? [15:16] mhr3_, bug #1186556 [15:16] bug 1186556 in qtubuntu "Browser does not respond to Qt.openUrlExternally(link)" [Critical,Confirmed] https://launchpad.net/bugs/1186556 [15:16] mhr3_, and http://qt-project.org/doc/qt-5.0/qtgui/qdesktopservices.html#openUrl [15:17] yea, that's what we're using [15:18] so the plan is to fix the platform to handle this nicely [15:19] wfm [15:19] mhr3_, yes [15:21] mhr3_: Saviq: https://code.launchpad.net/~aacid/unity8/fix_qlimitproxymodel/+merge/181071 [15:21] i'm planning to add some comments to the new added code [15:21] tsdgeos, \o/ [15:21] with ascii art [15:21] otherwise i'll forget why all those ifs [15:21] soon [15:22] :) [15:22] but the test is good and fixes the problem here [15:23] tsdgeos, yup, fixes! [15:23] tsdgeos, woooo :) [15:27] Saviq, got time to hop in on sdk quickly? Would like your opinion on something! [15:31] Saviq: https://code.launchpad.net/~nick-dedekind/unity8/indicator.messaging-rename/+merge/181076 [15:31] should stop the duplicate messaging indicator. [15:39] dednick, approved [15:39] ah renames renames [15:40] Saviq: :) thanks [15:43] tedg: i'm still not getting an icon in for indicator-network [15:46] dednick, Hmm, is that not part of disabling the new one? [15:46] Old one? [15:46] I figured that was part of the name thing. [15:47] tedg: no. it's that actionstate issue. where the parameter type doesnt match the target or whatever [15:47] dednick, ? The icon on the panel? [15:48] tedg: the icon in the panel comes from the actionState of the root item, but there is no action state is those do not match. [15:49] tedg: http://pastebin.ubuntu.com/6006873/ [15:49] kgunn: i've been told to try image 19.2 to see if the nexus4 thing is fixed but noone seems to know how to tell me to install the said image with phablet-flash [15:50] dednick, So you need an initial state. [15:50] dednick, K, let's fix it :-) [15:51] tedg: also, i'm not sure the agent will even work at the moment with indicator-network. may have to set it back to using qmenumodel rather than unitymenumodel [15:51] tsdgeos: yeah...i'm following scrollback also [15:53] tsdgeos, still some weirdness, searching for "cad" shows empty "Dash plugins" category [15:53] tsdgeos, it gives one result in unity7 [15:53] bregma: just for the record: https://bugs.launchpad.net/compiz/+bug/1214459 [15:53] Launchpad bug 1214459 in Compiz "Failing GLibSourceDestroyIntegration unit tests on compiz/0.9.10" [High,In progress] [15:54] mhr3_: where are you searching? apps scope? [15:54] tsdgeos, yep [15:54] dednick, Hmm, then indicator-network may not work, eh? [15:54] sil2100, thanks [15:54] tedg: i'm changing it to use qmenumodel and seeing how it fairs. [15:55] yep, can repro [15:55] let me see === tvoss is now known as tvoss|afk [15:55] tsdgeos, seems to have broken the model completely after that [15:55] keeps being empty [15:56] Saviq: you still planning to do Implement QDesktopServices::openUrl ? [15:56] sil2100, shall I approve and deploy https://code.launchpad.net/~sil2100/cupstream2distro-config/add_ubuntu-ui-extras/+merge/181083 so that the package changes are merged in? [15:57] sil2100, ah, never mind. it's already set to merge, (it's in no-dailies/sdk.cfg) [15:57] mhr3_: what do you mean keeps being empty? [15:58] dednick: hi! Ah, ok, need to remove it from there [15:58] I mean [15:58] fginther: ^ [15:58] fginther: thanks for reminding me [15:58] tsdgeos, i mean that after it breaks with the "cad" search it doesn't show anything even if i change the search back to "" [15:58] weird [15:58] it does here [15:58] sil2100, :-) [15:59] tsdgeos, then mine is more broken [15:59] otoh i'm runnigna slightly modified code [15:59] fginther: are all the hooks that are in the no-dailies important for this branch? [15:59] fginther: I think I'll copy-paste them to the head sdk stack [15:59] sil2100, looking [16:00] tedg: No, it would seem that is indicator-network not compatible with qmenumodel. [16:00] dednick, Ah, I was afraid of that :-/ [16:00] tsdgeos, that could explain that [16:00] yeah [16:00] thostr_1, ^ seems we can't have indicator-network and the Unity8 secret agent. [16:00] tedg: wifi actions greyed out, and selecting access point doesnt actually do anything [16:00] but still should should something in cad [16:00] let me finish the documentation [16:00] and see what's up [16:00] tedg: think it has to do with the namespaces [16:00] dednick, That'd be my guess as well. [16:01] sil2100, yes, we'd like to keep the hooks and other parts of the config to keep generating test/coverage results [16:01] tedg: hm. maybe i should concentrate on switching the network code to using unitymenumodel then instead of messages. dont know which is higher priority. I'l check with thomas [16:02] fginther: ACK :) [16:02] dednick, I'd say messaging menu, but you can ask him. [16:02] dednick, I think we'd rather land all the indicator-network stuff together if it's not easy to make the old stuff work. [16:03] fginther: pushed the modified version with your hooks in [16:05] mhr3_: can you pull and see if you can still repro the dac thing? [16:06] sure [16:07] tsdgeos, ehm, yep, still here [16:07] weird [16:07] i'm not getting it anymore [16:07] tedg: i'm guessing that the new indicator-messaging is not going to work either. [16:07] did you really push? :) [16:07] but i have not done any change :D [16:07] tedg: in terms of actions. [16:07] so i was wondering if i had uncompiled stuff [16:08] or stuff started hating me [16:08] mhr3_: r129 [16:08] err [16:08] mhr3_: r219 [16:09] dednick, Won't when it uses the .indicator file it use Unity Menu Mode? [16:09] Model [16:09] tedg: messaging is not compatible with it yet. there's lots of custom stuff in unity8 which i'm in the process of switching over. [16:10] mhr3_: do you have that one? [16:10] tsdgeos, yep, but it's not fixed for me [16:10] :/ [16:10] dednick, Hmm, I can get SMS's on my phone and reply and clear them... [16:10] tedg: ie all the menu items need to switch from using qmenumodel to using unitymenumodel [16:10] mhr3_: at least the fact that when you search something it comes back? [16:10] tedg: using the new indicator-messages ? [16:11] dednick, Yeah [16:11] tsdgeos, nope, keeps being empty [16:11] ^_^ [16:11] tedg: how about 'clear all'? [16:11] dednick, Didn't try that one. I don't have a SIM right now to check though. [16:12] tsdgeos, although i find the whole concept odd, shouldn't limit number of results be a view thing? [16:12] limiting* [16:12] mhr3_: qt models are part of the view [16:13] if you assume that, everything gets clearer :D [16:13] tedg: i think replying uses custom action names, but the clear all is just a button action "indicator.remove-all" [16:13] Ah, okay. [16:13] view in a "strict" MVC world point of view [16:14] the real model is whatever libunity has, qt models are "view" [16:14] i'm a doing a shitty job explaining myself [16:14] but there's a great talk by kdab around that makes much more sense than i do probably [16:14] that's a strange concept [16:15] so what would be a view in that model? :P [16:15] part of the view too [16:15] i see, this is a recursive problem [16:16] Wellark: ping [16:19] mhr3_: but yes i think there's something weird going on [16:19] mhr3_: if i search dash i have 5 entries, is that normal? [16:19] oh maybe it is [16:20] tsdgeos, there's also some other issue - removing all items seems to try to load their icons [16:20] mhr3_: don't pin every weridness to me now :D [16:21] sorry, will give you more time to process the first issue next time :P [16:22] tsdgeos, 5 items in "dash plugins"? [16:22] when searching for "dash" [16:22] tsdgeos, you should have 40 :) [16:23] cause description of every plugin says "this is a dash plugin..." and descriptions are searched [16:23] well, almost every scope [16:24] yep, somethings wrong for sure, since da -> 6 items, dac -> 1 item, da -> 5 items [16:25] * tsdgeos hits himself [16:26] mhr3_: is there any easy way to just get the dash plugins thing in the apps scope? [16:27] tsdgeos, yes, the dash plugins results can be used to disable scopes :) [16:27] ¿? [16:27] so disable applications [16:28] ah [16:29] which btw shouldn't be possible, you're lucky we didn't fix it yet :P [16:31] tsdgeos, not if you want to take it over, I started talking with ricmm as to where to put it in platform-api [16:32] tsdgeos, but didn't really get a response [16:32] Saviq: oh not want to take it over, just wanted to know if i should move the blueprint to you :D [16:32] still busy with this limitthing [16:32] not wokring as it should yet [16:32] tsdgeos, move it to me, yes [16:34] fginther: ! [16:34] fginther: for ubuntu-ui-extras... could you disable mediumtests? Since it has no integration/autopilot tests in it [16:34] fginther: there were some, but those were empty example tests, not really tests [16:35] fginther: could you disable the tests and re-run so that https://code.launchpad.net/~ubuntu-sdk-team/ubuntu-ui-extras/ubuntu-ui-extras-no-ap-tests/+merge/181091 gets in? [16:35] fginther: do you have the power to redeploy a stack btw.? [16:36] fginther: since if the merge ^ gets in, I would be grateful if the SDK stack could be redeployed, so that the new package is ran on the next tick ;) [16:36] sil2100, I can't do a stack, sorry. But I can fix the merge bits [16:36] fginther: ok, then I'll redeploy tomorrow if anything [16:36] fginther: thanks! === alan_g is now known as alan_g|EOD [17:03] tedg: i've got indicator-network somewhat working with unity8 now. if you fix up the action state issue i think we should be ok. Only issue is that i can't get the individual wifi strengths for the access points. unitymenumodel doesnt support that :( [17:06] tedg: s/action state/parameter type [17:11] Saviq: you got a n4 right ? [17:13] mzanetti: pong [17:15] mterry: hey...can i bother you :) [17:18] kgunn, no, msm ordered me one on Monday (and I think that was the trigger for Rick's "no-more-hw-money" email ;)) [17:19] :)) [17:28] Saviq: care to take a look: https://code.launchpad.net/~gerboland/unity-mir/enable-OSK-surface/+merge/181109 === racarr is now known as racarr|lunch [18:58] dednick, Hmm, okay. Do you think it's something possible? [18:58] tedg: ? [18:59] tedg: regarding the strengths ? [18:59] dednick, Yes, sorry, reading backlog. [18:59] tedg: some work in unitymenumodel, but we're ok without it for now. [18:59] Some strengths, but not others? [19:00] tedg: we get the overall strength, but not the individual ones for each access point [19:00] they are "non-menu" actions [19:00] dednick, Ah, okay. [19:02] tedg: good news is that i've got the network indicator connection working with passwords now. [19:02] dednick, Yeah, that is good news! [19:02] Makes migrating much easier. [19:05] dednick, This looks correct, no? http://bazaar.launchpad.net/~indicator-applet-developers/indicator-network/trunk.13.10/view/head:/network/network-action-manager.vala#L62 [19:08] tedg: just having dinner [19:09] np [19:36] tedg: that's what's in trunk [19:36] dednick, Yes, but your diff added that? [19:36] dednick, Or it set the state to null? [19:37] parameter_type set to null [19:38] dednick, Ah, okay, I was reading it backward. [19:41] dednick, https://code.launchpad.net/~ted/indicator-network/initial-panel-state/+merge/181145 [19:42] tedg: cool. approved. [19:43] dednick, Great! I don't think the next merge window is for 3 hrs though. [19:43] tedg: doesnt matter. unity8 isn't ready anyway. === racarr|lunch is now known as racarr [20:00] tvoss_, when do you expect the location-controller branch to land? We're going to need it here real soon now for charles' branch. [20:02] tedg, tomorrow over the day [20:03] tvoss_, Not sure if the expression... does that mean Thursday? [20:03] tedg, nope, that's Wednesday [20:04] K, I thought "over the day" might mean "day after" [20:05] mterry_: nice branch! [20:06] mzanetti, :) [20:06] mzanetti, I love deleting TODO lines [20:06] mterry_: :) [20:07] mterry_: is this intended to always be used? or only when running in the greeter? [20:08] mzanetti, always [20:08] mterry_: can I test it on the desktop in a way it stores the config? [20:10] mzanetti, uh, I guess, yeah. You'll have to install the deb though, and maybe killall accounts-daemon [20:10] mzanetti, because we install some policykit and dbus interface files [20:10] I only ever tested on the phone, but no reason it wouldn't work on desktop [20:11] mzanetti, except... you'll have to run as the right user, if you are talking to your system accounts service [20:11] yeah, that's ok [20:13] mzanetti, so you might want to change the demo user to 'mzanetti' or some such [22:25] mterry_, you rock! (re: Bug #1214477) [22:25] bug 1214477 in Unity 8 "lock screen shown when on a phone call" [High,In progress] https://launchpad.net/bugs/1214477 [22:25] mterry_, I added some minor comments to the MR [22:26] Saviq, ok [22:26] Saviq, just fixed the test actually. will look into other stuff [22:26] mterry_, yeah, saw it was updating diff [22:27] mterry_, seems like your branch was one of the first to have tests ran on hardware :) [22:27] run? [22:27] Saviq, what is with having both Q_FLAGS and Q_DECLARE_FLAGS... not sure the docs make it obvious [22:28] mterry_, indeed, they don't [22:28] mterry_, Q_FLAGS takes two arguments, btw [22:28] mterry_, Q_FLAGS(FlagName, EnumName) IIRC [22:28] mterry_, that just takes care of typedef, really [22:28] Saviq, I'm seeing that... didn't get any errors, oddly [22:28] well, I'll do it the verbose way [22:28] mterry_, Q_DECLARE_FLAGS takes care of the arithmetic [22:29] mterry_, i.e. operators and stuff [22:29] mterry_, to have the whole QFlag machinery working, you need both [22:29] Saviq, they even go to the trouble of reversing the order of the arguments to the two macros, and using a comma in one and not the other. Thanks Qt! ;) [22:30] mterry_, indeed... [22:30] mterry_, it's not the best thing out there... [22:31] mterry_, actually, it seems I swapped the two: http://qt-project.org/doc/qt-5.0/qtcore/qflags.html#flags-and-the-meta-object-system [22:31] https://code.launchpad.net/~gerboland/unity8/typo/+merge/181167 <- typo fix [22:32] mterry_, or generally mixed what they do :D [22:32] mterry_, yay for good docs and self-explanatory names === salem_ is now known as _salem [22:34] well, $20M to go... [22:35] yeah, not bad though [22:36] pity, would've liked to work on something cutting edge, if you pardon the pun [22:58] mterry_, test_suspend fails locally still: [22:58] FAIL! : qmltestrunner::Shell::test_suspend() property mainStageFocusedApplication [22:58] Actual (): null [22:58] Expected (): [22:58] Loc: [/home/michal/dev/canonical/unity8/repo/tests/qmltests/tst_Shell.qml(139)] [23:03] mterry_, same in jenkins [23:06] Saviq, odd, I had it work for me.. [23:06] Saviq, will look tomorrow [23:06] mterry_, ok o/ [23:14] o/