=== bschaefer_ is now known as bschaefer [06:04] there is a regression in unity/compiz with latest updates ( +13.10.20130920-0ubuntu1) bug 1229540 [06:05] bug 1229540 in unity (Ubuntu) "[regression] Clicking launcher icon does not switch to application with multiple workspaces" [Undecided,New] https://launchpad.net/bugs/1229540 === jamesh_ is now known as jamesh [08:46] mzanetti: hey, who would be a good person to talk to re: the dash and scopes? I'm introspecting it and can't tell which scopes are 'enabled'. I.e. the social.scope details state enabled, but I can't see it on the phone (only have 4 options) [08:49] veebers: hmm... mhr3 perhaps [08:49] veebers: but let me have a look in the code [08:50] mzanetti: awesome, much appreciated [08:50] veebers: you should be able to get the amount of loaded scopes with dashContentList.count [08:51] veebers: and in the delegate you can get the scope.id I'd say [08:52] mzanetti: hmm, that might give me the count, but not which ones are enabled. I.e. I see the QQuickLoader with scopeId: "social.scope" has the properties: isLoaded(True), enabled(True), active(True) [08:53] but I can't tell from the introspected details if it's available or not (i.e. the social scope isn't available on the phone, I actually have Music, Home, Apps and Videos) [08:53] veebers: hm... isn't that enough? [08:53] veebers: no... but in that case it won't show up in the loader either [08:53] mzanetti: hmm, then why can't I swipe to it? perhaps my settings are borked? [08:54] veebers: huh? [08:54] so it shows up on the desktop, but you still can't swipe to it? [08:56] mzanetti: hmm, actually [08:56] mzanetti: sorry I think I've lead you and myself astray [08:56] I've gotten the logs between what's running on the phone and what I have running on my desktop :-\ [08:56] ok [08:56] mzanetti: really sorry about that. Perhaps I'll have an actually question tomorrow. But for now I got that _all_ wrong [08:57] veebers: come on... no worries about that [08:57] happens [08:57] mzanetti: cheers [09:11] Could anyone have a look at bug bug 1229540, please [09:11] bug 1229540 in unity (Ubuntu) "[regression] Clicking launcher icon does not switch to application with multiple workspaces" [Undecided,Confirmed] https://launchpad.net/bugs/1229540 [09:11] this is a recent regression [09:11] asac, ^ [09:15] hiya [09:15] did anyone else notice apps like primarily firefox and thunderbird being extra slow since last week? [09:16] Trevinho, bregma, other compiz hackers: ^ [09:30] jibel: thats on desktop? [09:30] asac, yes [09:35] jibel also filed bug 1229540, which I encounter as well [09:35] bug 1229540 in unity (Ubuntu) "[regression] Clicking launcher icon does not switch to application with multiple workspaces" [Undecided,Confirmed] https://launchpad.net/bugs/1229540 [09:35] dholbach, I confirm the slowness on intel too [09:36] it takes up to 3-4 seconds when clicking on a folder in thunderbird [09:36] especially transitions between windows, dash, alt-tab [09:38] it's difficult to measure responsiveness with an application that heavily relies on network [09:39] sure, this is more gut feeling than anything scientific [09:50] dholbach, jibel: https://bugs.launchpad.net/compiz/+bug/1228352 and https://code.launchpad.net/~townsend/compiz/fix-auto-vp-switch-0.9.10/+merge/186881 seem to be the "super doesn't work on other workspace" issue you discussed earlier [09:50] Ubuntu bug 1228352 in compiz (Ubuntu) "[regression] Alt-Tab for all viewports will not switch viewports when selecting a window on another viewport" [High,In progress] [09:51] it's fixed in trunk [09:51] so I guess we need another compiz landing [09:51] seb128, thank you [09:51] yw [09:52] thanks seb128! [09:58] pstolowski, hey, here's what should get you going with the invalidation https://code.launchpad.net/~unity-team/unity8/scope-isactive/+merge/187179 [09:59] pstolowski, feel free to just push to the same branch [10:02] Saviq: awesome, thanks! [10:02] mhr3: ^ [10:02] pstolowski, tested, too! ;) [10:05] Saviq, are all those test changes desired/required? [10:05] or mixed up branches? [10:05] mhr3, cleaned up a little [10:05] mhr3, if you look at it it's just moving stuff around [10:05] very well, just wanted to check [10:06] i'll merge it with my searchInProgress branch, don't want conflicts in it later [10:06] mhr3, sure, just do whatever you want with it [10:07] mhr3, treat as your own :D [10:07] Saviq, like `bzr rm *`? :) [10:07] mhr3, try doing that remotely on my copy! [10:07] hmm, challenge accepted! [10:07] don't forget you're running my code [10:07] :) [10:34] Saviq: https://code.launchpad.net/~mzanetti/unity8/switching-previews/+merge/186991 [10:34] mzanetti, WiP? [10:34] Saviq: not any more [10:35] mzanetti, could you please get design ACK for that? I feel, for one, that we need a bigger spinner [10:36] and/or maybe darken the background while we're waiting [10:36] Saviq: I noticed that ApplicationsFilterGrid is not used any more. and in general the Dash directory contains a lot of unused legacy stuff [10:36] mzanetti, you sure it's not used? [10:37] Saviq: yeah... just in one test [10:37] mzanetti, right, it's using the generic one [10:37] yep [10:37] mzanetti, well, generally there's quite some cleanup that needs to happen around that indede [10:37] indeed, even [10:38] yep. and I'm afraid testing of the whole scope thing is not good enough either [10:38] which is why my branch doesn't really contain a test for this [10:38] we'll really need to setup a test scope and have some tests talk to it [10:38] it would require a lot of mocking and [10:39] mhr3, +1 [10:39] yeah... what mhr3 said [10:39] mzanetti, btw, could we get a spinner on first preview open? [10:39] Saviq: isn't it there? [10:39] mzanetti, when you tap on "More suggestions", it just sits there [10:40] right... problem is, that this is triggered by a activate() on the item which is supposed to run the app [10:40] so in QML I don't have a way to know if there's a preview coming up [10:40] the scopes items would need an additional property "activatable" or whatever so it can trigger the correct signal [10:41] => the spinner on first preview only shows up onPressAndHold (as I know a preview is coming up) [10:41] mzanetti, well, or we just put up a spinner on activate() [10:41] mzanetti, and drop it when anything comes back [10:41] just on top of the other stuff? [10:41] would look quite bad imho [10:41] mzanetti, all taps should do previews (minus apps) [10:42] mhr3: in all scopes? [10:42] mhr3, minus Online Videos, too, atm [10:42] mzanetti, Saviq, hm? [10:42] mzanetti, in all afaik [10:42] mhr3, oups, it seems to do both, actually [10:43] mhr3, yeah, I tap on an item in Online, get a preview, and soon thereafter the web browser [10:43] Saviq: really... just tried here... I only get the browser onClick [10:43] Saviq, then it's broken [10:43] oh wait [10:43] its broken indeed. first click behaves correctly [10:43] the others not any more [10:44] morning guys I have a bit of an issue with the commercial apps that I am porting to Unity in that the installed apps seem to be showing up twice in the dash http://ubuntuone.com/3YJYgeuXb9jN9IYwkQwzLm [10:44] actually. only once you trigger a preview manually with longpress. [10:44] after that it's broken and does both [10:44] to saucy rather than unity even [10:44] mhr3: ^ [10:45] davmor2, known bug, in fixing [10:45] Saviq, it is? [10:46] mhr3, I saw a branch... [10:46] Saviq: fantastic do you have a bug number I can subscribe to so I can retest once it is fixed please [10:46] davmor2, where are the .desktop files? [10:46] mhr3, davmor2 actually bug #1225387 [10:46] bug 1225387 in unity-lens-applications "Multiple entries for each application" [Undecided,Fix committed] https://launchpad.net/bugs/1225387 [10:46] Fix committed [10:47] or released, even [10:47] mhr3: /usr/bin/applications [10:47] that one was because of click scope [10:47] so not relevant to this [10:47] ok /me shuts up [10:47] sorry /usr/share/applications even [10:48] Saviq: this is desktop commercial apps vs click apps [10:48] davmor2, can you pastebin output of `libunity-tool -s /usr/share/unity/scope/applications.scope -q moves - r`? [10:48] -r [10:48] not - r [10:49] mhr3: scope or scopes? [10:49] scopes indeed [10:50] mhr3: I'll just install libunity-tool first then it might work better then D'oh [10:50] -tools :) [10:51] is the pkg name [10:52] mhr3: http://paste.ubuntu.com/6149541 [10:53] are you fully updated? [10:53] anyone, once again [10:53] plus you're missing the "-q moves" [10:53] D'oh [10:54] mhr3, mzanetti, so we can then assume that, except for running and installed apps, everything else will give us a preview? [10:54] Saviq, you shouldn't assume, you should request a preview [10:54] mhr3: http://paste.ubuntu.com/6149559 [10:54] mhr3, oh? [10:54] hmm... I'm not really sure if that will be true forever [10:54] mhr3: I updated this morning let me reboot incase that fixes anything [10:54] mhr3, thought it was the scope that will come back with a preview on activation [10:55] Saviq: yeah. that's what's happening now [10:55] davmor2, nah, it's fine, it was the missing param [10:55] mzanetti, yeah, I though we'll be continuing with that [10:55] t [10:55] brb [10:55] Saviq, that's just an old "hack" [10:55] the ui should request the preview, then you get proper signals etc [10:55] Saviq: the problem with that is that we don't have a chance to know if a preview will come up or not [10:56] hence we can't really display the ActivityIndicator [10:56] that's why you should request a previe [10:56] w [10:56] if you do, you know it will come up [10:56] mzanetti, of course [10:56] mhr3: Right so I'm up-to-date and rebooted and it is still the same so phew at least you are not wasting your time :) [10:56] well... or an error [10:56] mzanetti, mhr3, I'm game with that [10:57] mhr3: but I'm still not convinced that we should do some if(name != applications) hack [10:57] mzanetti, better solutions welcome [10:57] somehow the scope backend should tell us if this is supposed to be launched or not [10:57] as everything else is dictated by the backend too [10:58] mzanetti, thing is we don't want to let the scopes decide [10:58] mzanetti, why should it care? it's ui thing whether it want to launch something or preview it? [10:58] mzanetti, apps are the exception [10:58] mhr3: So do you need a fresh bug for this issue? [10:59] davmor2, no duplication when you search? [10:59] mhr3: duplication does indeed go when I search [11:00] mzanetti, it's just that sometimes scopes will give you preview instead of directly launching it :) [11:00] mzanetti, but if you do request a preview you will get a preview [11:00] davmor2, can you search in the dash for "moves" and screenshot it? [11:01] so we're 100% sure that everything should always open a preview (except apps) [11:01] ? [11:02] mzanetti, that's what design told us [11:02] and that's what unity7 does now [11:02] mhm.... [11:02] mhr3: http://ubuntuone.com/5ke5YUabcgNHzf9RJo8ZLu [11:02] ok. in that case I'd say the activate() should never deliver a preview [11:02] mzanetti, sorry, can't guarantee [11:03] but this seems all a mess then imho [11:03] if the scope has the power to device if it'll open a preview or not. then it also should tell us that [11:03] if unity is the one to decide than activate() should do what it says it does [11:03] mzanetti, you can assume it doesn't, but unfortunately there are valid use-cases when an activation will bring up preview [11:03] but those are rare [11:04] yeah... and exactly that's why I thought the if(name == applications) hack is bad [11:04] because there are more exceptions [11:04] mzanetti, i agree, yet it's a ui exception, not a scope exception === MacSlow is now known as MacSlow|lunch [11:06] mhr3: what are those use cases where activate() should bring a preview? [11:06] davmor2, sorry, meant in the apps dash page [11:06] mzanetti, things that need payment [11:06] mhr3: apps? [11:07] anything that needs payment [11:07] apps music videos [11:07] you can't just "activate" those results [11:07] mhr3: assuming we would only call activate() on installed apps anyways [11:07] are there still installed apps that need to open a payment dialog before launching? [11:08] mhr3: http://ubuntuone.com/0sge41sSr0oGMViZFRLeuy [11:08] mzanetti, well right now we don't do payments anywhere in unity8 [11:09] sure.. but in the future... it somehow seems paymant for apps can only happen int he suggested apps, not in the installed ones. [11:09] but I might be wrong... people want to charge for everything at times :D [11:09] davmor2, ok one more pastebin then, do the libunity-tool with -q "" [11:10] mzanetti, heh, pay per use apps? :) [11:10] opening new business models... why not [11:10] I really hope not... but I'm sure iOS supports something like that already [11:10] anyways, in that case it probably should be an in-app payment mechanism anyways [11:11] anyway, there are (rare) cases where you might get a preview when asking a scope to activate something [11:11] i don't think it needs to be super polished right now, but it should support that [11:12] ie, i know spinner is hard in that case, so let's forget about it [11:12] ok... I'll change it then... but I warned you. this is getting messy [11:12] mhr3: just so I get this one right the libunity-tool command you asked be to run earlier yes not just libunity-tool -q "" [11:13] davmor2, yes, same as before but instead of -q moves do -q "" [11:14] mhr3: http://paste.ubuntu.com/6149619 [11:15] davmor2, libunity-tool -s /usr/share/unity/scopes/applications.scope -q "" -r [11:15] that one pls ^ [11:15] meh yeah sorry wrong paste [11:16] mhr3: http://paste.ubuntu.com/6149634 [11:17] mhr3: I forgot to hit copy after select all D'oh. it's been a long day and I only started at 11 :D [11:20] davmor2, eh, one more pls, libunity-tool -s /usr/share/unity/scopes/home.scope -q "" -r [11:21] pstolowski, mhr3 re: bug #1226514 - how does that work in unity7? [11:21] bug 1226514 in Unity 8 "[DASH] button in preview and scope icon do not show disabled state" [Undecided,Incomplete] https://launchpad.net/bugs/1226514 [11:21] Saviq, yes [11:22] Saviq, mzanetti, clicking a preview button can send back a "new" preview, in this case with updated button [11:23] creating a whole new preview? [11:23] mhr3: http://paste.ubuntu.com/6149656 [11:24] davmor2, hm, something is broken there, once again pls [11:25] i mean, until it's different :) [11:26] mhr3: http://paste.ubuntu.com/6149661 that is different [11:26] mzanetti: yes [11:27] i see... on installing click apps for example [11:27] davmor2, you installed those via the software center? [11:27] mhr3: I did [11:27] davmor2, did they fly into the launcher when you did? [11:27] trevinho, you broke stuff ^ [11:27] mhr3: they did [11:27] davmor2, did you unpin them after they did? [11:27] pstolowski: what happens if I install a click app, and while the progress bar fills in I request another preview? will the update to the other be cancelled? [11:28] mhr3, well, yeah - it doesn't in unity8, but works in unity7? [11:28] Saviq, right, sorry missed the "how" the first time :) [11:28] mhr3: I did otherwise I have an unusable launcher there are 960 apps to test [11:28] davmor2, perfect, can you note those things in a bug report pls? [11:28] mhr3: sure [11:28] mzanetti, not entirely sure how it works right now, but the idea was that "progress" would be an action that would get triggered on complete [11:29] mhr3: is it just against unity7 or something specific [11:29] davmor2, yes, unity7 [11:29] mzanetti, so if we moved away from that preview, we should stop it from sending any actions, but refresh when we go back [11:29] no worries [11:29] Saviq: I mean, once the progress bar reaches 100%, the preview changes to have 3 buttons instead of a progressbar [11:30] davmor2, also, it'll remain broken for you even after it's fixed, sorry :/ [11:30] Saviq: and from what I understood that happens with the same signal as a new preview would have been requested [11:30] mzanetti, yeah, that *should* be handled by the fact that reaching 100% triggers that action, and the scope comes back with a new preview [11:31] yeah... comes back with a new preview. what if the user switched over to another preview? that will be overridden with the old, completed one [11:31] mzanetti, that action shouldn't be triggered [11:31] mhr3: that's not a biggie I just didn't want it broken for end users, I'll be wiping my system and reinstalling once I get all the app installed hitting 900+ ppas in apt-get update takes several hours :D [11:32] mzanetti, unless you're thinking the action was already triggered, and we're just waiting for a response... [11:34] mzanetti, trigerring the preview progress is same as triggering any other preview button - they can return a new(/updated) preview and navigating away should cancel those requests [11:34] mhm... [11:34] mzanetti, pstolowski has a branch that does preview cancellation [11:35] well, in the backend... might need some extra hooking up [11:39] mhr3, mzanetti : yep.. and I just realized I don't have cancellations for preview actions. and interestingly, unitycore doesn't seem to support that :/ [11:39] mhr3: bug #1229681 [11:39] bug 1229681 in unity (Ubuntu) "Unity7: Application appear twice in the dash home page" [Undecided,New] https://launchpad.net/bugs/1229681 [11:40] mhr3: did you want any of the pastebins adding or are the steps to reproduce enough [11:42] pstolowski, Saviq: ok... I pushed to the branch. should hopefully do what it is expected to: https://code.launchpad.net/~mzanetti/unity8/switching-previews/+merge/186991 [11:43] as in only call preview except for installed apps (which triggers the activityindicator also onClick) [11:44] mzanetti, runningApps are handled separately, right? [11:44] Saviq: seems so, yes. not triggering any of this code [11:44] mzanetti, k [11:45] mzanetti, you actively disabling swiping in the list of screenshots when there's no overflow? [11:46] Saviq: yes [11:46] mzanetti, not sure we want that [11:46] can drop it... felt right to me [11:47] mzanetti, can you get design folk to look at the whole thing? [11:47] yeah, on it [11:49] mzanetti, right, so... if you switch between previews while they're loading, they get mixed up [11:50] mzanetti, same with the update you mentioned [11:50] mzanetti, is there a way to slow down autopilot moves? [11:51] davmor2, that's ok, i added a comment [11:51] Saviq: yeah... pstolowski is still working on the cancellation branch [11:51] it's super fast I cannot see why it's failing [11:51] mzanetti, right, that will help things [11:52] Cimi: not entireley sure how. I think dandrader changed it intentionally to jump instead of move [11:52] * dandrader reads backlog [11:52] mhr3: nice one, /me goes off to break something else now [11:53] or if someone wants to help me here.. [11:54] mzanetti, suppose you should merge it with pstolowski's branch then [11:54] Cimi, mzanetti , well, I don't recall anything about this s/move/jump story... [11:54] ~cimi/unity8/unity8.hud-2_hint-reveal-commit/ [11:54] hud tests are failing... [11:57] unity8.shell.tests.test_hud.TestHud.test_show_hud_button_appears [11:57] is failing [12:02] Saviq: merged it with pstolowski's branch. looks quite good now. [12:02] mzanetti, great [12:02] pstolowski: any particular reason you didn't propose your branch for merging yet? [12:08] mzanetti, I wonder if we should block on activate() in the preview - so that you can't switch away from a preview you've executed an action on... [12:09] mzanetti, or somehow match the preview coming back with the spot where it should show up... [12:10] Saviq, so did you have any idea which osk issue olli was referring to ? [12:10] Saviq: wait... so. for example I trigger install on a recommended app [12:10] kgunn_, no [12:10] mzanetti, yeah, and switch away [12:11] Saviq: the progress bar starts. ok. switching away while waiting to be installed [12:11] mzanetti, I'm not sure that actually works as intended atm [12:12] mzanetti, I think the preview does the switching internally - except when there's an error, for example [12:12] I'm afraid it doesn't.... I guess the biggest problem is that items can disappear from the model while the previews are open [12:12] well... actually... not really [12:12] mzanetti, well, they could in theory, but we can make sure they won't [12:12] mzanetti, i.e. it's us that control when search results are refreshed [12:13] kgunn_, all of the things you mentioned in your emails are out of our control [12:13] Saviq: us as in the ui? [12:13] mzanetti, yes [12:13] mzanetti, with https://code.launchpad.net/~unity-team/unity8/scope-isactive/+merge/187179 [12:14] Saviq: ok... so I switch away, item gets destroyed. I switch back and the preview gets requested again [12:14] Saviq: should be fine I'd say [12:14] mzanetti, it's not that [12:14] mzanetti, and it doesn't get destroyed - not straight away - only when you switch more than one item away [12:15] yeah... that's true. that can be fixed tho [12:15] mzanetti, cacheBuffer should be 0 btw [12:15] mzanetti, still [12:15] mzanetti, if you activate an action in preview 1 [12:15] mzanetti, and switch to preview 2 [12:15] mzanetti, right now the response for preview 1 will come to preview 2's spot [12:16] Saviq: that's what I asked before... pstolowski said it will be cancelled [12:16] Saviq: as soon as we trigger a new preview() request, any actions won't trigger the update any more === jhodapp|afk is now known as jhodapp [12:16] mzanetti, ok, let's see [12:17] seb128: didrocks: ping [12:17] * mzanetti setting cacheBuffer to 0 [12:17] mzanetti, what if you close the preview after having activated? [12:17] ChrisTownsend, hey [12:17] Saviq: hmm... that might be an issue indeed. [12:18] seb128: Hey, I have a Unity/distro related question for you. [12:18] seb128: Does Distro agree with this MP: https://code.launchpad.net/~jbicha/unity/recommend-telepathy-indicator/+merge/186392 === alan_g is now known as alan_g|lunch [12:18] Saviq: but tbh... I think its quite a bad idea that everything can ship a preview at any time and we have no clue where it came from and what it is [12:19] ChrisTownsend, that seems fine to me yes [12:19] seb128: Ok, thanks, then we'll approve it. [12:19] ChrisTownsend, thanks [12:20] ChrisTownsend, since you are online, jibel and dholbach were looking for you earlier, new compiz seems to have lag issues on intel (your name come next to most of the approved changes in the recent landing so I guess it's a problem for you) [12:21] seb128: Uhh... [12:21] mzanetti, yeah, I agree I'm not liking that cat'n'mouse play [12:21] Saviq: it should be a proper model, holding all items and data for all the stuff (initially empty). with a fetch(index) to actually fill in the data if remote content [12:21] seb128: Hardly anything went into Compiz in the last release, definitely nothing that would affect performance. [12:22] Saviq: that way specific data can be updated in a performant and predictable manner, instead of just overwriting the whole set of data of whatever the current page is [12:22] mzanetti, problem is you can't know what type of preview you'll get [12:22] Saviq: withing a category it should be always the same, no? [12:23] mzanetti, not necessarily [12:23] Saviq i don't disagree with you [12:23] mzanetti, a request for a preview for the same item might get you different preview type, depending on other conditions [12:23] just need to convince the our bosses :) [12:23] ChrisTownsend, ok, maybe that one is unity ... there was also the compiz/multiple workspace/focus issue that seems to be fixed in trunk [12:23] ChrisTownsend, http://irclogs.ubuntu.com/2013/09/24/%23ubuntu-unity.html#t09:15 [12:24] seb128: Yes, the workspace issue, I'm very familiar with:-( [12:24] Saviq: then we'd need a "previewType" property in the model data. [12:24] Saviq: that can change at any time [12:24] seb128: That is definitely fixed, but that should have zero performance impact. [12:25] ChrisTownsend, ok, maybe the performance stuff is another bug [12:25] seb128: Well, I have an Intel machine have not noticed any subpar performance. [12:25] jibel, ^ did you open a bug about the performance thing? [12:25] mzanetti, potentially, yes, we'd have to go through all of the use cases and see where we can go - but anyway, not this time [12:26] Saviq: yes. I agree... something for a cold december day :D [12:26] seb128: Also, new Firefox and Tbird was released recently. [12:26] ChrisTownsend, right, that could be an issue with those apps if that's specific to them [12:27] seb128: Ok, that seemed to be all that was mentioned. [12:28] seb128: I doubt they have Atom based machines, but performance on those chips is pretty bad right now. [12:28] seb128: Due to Mesa 9.2 [12:29] seb128: Anyways, I can only speculate right now, so I'll keep a look out for any poor performance related bugs. Thanks! [12:30] ChrisTownsend, thanks ;-) [12:31] seb128, no I didn't, the machine I tested on is a test machine that I don't use every day, and I don't have a good base for comparison. I'll ask dholbach to submit one as he seems most affected by this performance issue [12:32] jibel, thanks [12:41] mzanetti: only reason it's not MPed is because I was adding support for result row to it [12:42] pstolowski: ah ok... no worries then. I've merged the current state into my branch and set it as a prerequisite branch [12:43] pstolowski: seems to work quite well btw [12:43] pstolowski: one thing I'd need is to manually cancel stuff [12:44] pstolowski: assuming I open the preview for a click app, click in install, then close the preview. when the installation is completed I think it would open again because of the action stuff [12:45] MacSlow|lunch, when will the mp be for prompting pin dialog ? [12:47] mzanetti: hmm, why? I'm giving you a preview instance, and make QML the owner of it, so it will be gone? [12:47] pstolowski: didn't you guys tell me before that in that case new preview will be generated? [12:48] if that's not the case then we're fine [12:51] mzanetti: you mean this: mhr3 | mzanetti, trigerring the preview progress is same as triggering any other preview button - they can return a new(/updated) preview and navigating away should cancel those requests"? I'm not sure tbh [12:51] pstolowski: yeah... that one [12:57] pstolowski, mzanetti, you know in the ui when a preview is closed, at that point you should call something so that any pending requests to the scope (like waiting for response to action-activated) can be cancelled [12:57] something == something that you'll agree on with pstolowski :) [12:58] mhr3: yeah... that's what I asked pstolowski for :) [12:58] right, i'm just translating ;P [12:58] mzanetti, mhr3 : ok, got you.. will add that, though afaict we miss that bit on unity core side... [12:59] pstolowski, we do? don't we pass cancellable to those? === MacSlow|lunch is now known as MacSlow [12:59] mhr3: not to action activation in Preview::performAction() === alan_g|lunch is now known as alan_g [13:00] pstolowski, :/ we need to fix unity-core then [13:00] i hate touching unity-core, the whole thing is an abi break waiting to happen [13:01] kgunn_, as soon as I solved the exported MenuModel not showing up in the AP-tests... currently don't know what's causing this. [13:02] MacSlow, so does it work on the phone? i'm assuming you see the unexpected on the pc ? [13:05] kgunn_, yes... the issue is on the desktop-machine... I've not yet tried the ap-test on the phone yet. [13:05] greyback, thanks for the top approval help [13:06] kgunn_: only did my bit, hope everythings ok [13:06] MacSlow, it'd be interesting to see not only the AP test on the phone...but even just manual use case exercise [13:07] kgunn_, I can verify that today [13:07] olli_, around? [13:08] Saviq, yep [13:11] MacSlow, thanks...one more ques, is the back end there & ui tied in? === _salem is now known as salem_ [13:14] kgunn_, yes... for the MP only the autopilot-tests are missing (for pinunlock, password and user-auth) ... wifi-selection still has to be completed [13:15] MacSlow, oh yeah...i knew about wifi...but sim pin is the one we're gunning for [13:16] good to hear its all tied in [13:16] kgunn_, I was told by dednick, that the way signal-strength needs to be passed in a different way. [13:16] kgunn_, I think I already showed off the first three dialog-cases in screencasts already. [13:17] MacSlow, yeah...but can't tell if backend is there (at least i can't) [13:18] tedg: might be a good time for a full review of the simlock indicator-network branch of mine. [13:27] mhr3: Hey man, would you have a sec to chat about what we last talked on g+? I need to access a model backing a scope. While I can iterate over items, CategoryResults type seems unmutable, and you can't instantiate CategoryResults from QML. Any ideas how I could update a model backing a scope? [13:28] mhr3: Unless I can remove an item from a music (sub)scope, we can't integrate the OneAPI billing demo with the real music scope. [13:28] dednick, Ah, okay. Is the notification support in Unity trunk or do I need a branch? [13:29] karni, pls rewind, why do you need to change that model? [13:31] tedg: you'll need some branches [13:31] is there any plan to add some sort of "project window grouping"? (so i can minimize/restore/switch between groups) [13:32] tedg: lp:~nick-dedekind/unity-notifications/simunlock.dialog [13:32] tedg: lp:~nick-dedekind/unity8/simunlock.dialog [13:35] Saviq: standup? [13:36] dednick, indeed [13:37] Saviq: I'm doing the notes [13:37] greyback, yeah I see [13:38] mzanetti: I've updated my cancellation branch. i'm leaning towards MPing it and handling action cancellation separately (since it needs a change in unitycore, and this will delay entire thing) [13:40] pstolowski: fine with me [13:48] Cimi: so what exactly is the issue? [13:48] mzanetti, failing tests? :) [13:48] on the hud [13:48] where? [13:48] maybe it's math [13:48] mm doesn't seem like math [13:49] it's weird [13:50] Cimi: where is this happening= [13:50] ? [13:50] hud [13:50] ~cimi/unity8/unity8.hud-2_hint-reveal-commit/ [13:50] 12:54 | hud tests are failing... [13:50] 12:57 | unity8.shell.tests.test_hud.TestHud.test_show_hud_button_appears === dandrader is now known as dandrader|afk [13:54] greyback, Hey, are you planning on landing the upstart support for start/stop in a separate branch from focus/resume ? [13:54] greyback, Or is that all one mega-upstart-app-launch branch? [13:54] tedg: upstart will be one branch [13:55] separate branch for focus/resume [13:56] greyback, Okay, makes sense. [13:56] greyback, So I'm not blocking the first one without the focus/resume stuff today, but probably soon. [13:56] greyback, When will I block you? :-) [13:57] tedg: nope, you're not blocking me on the upstart stuff. That's me being slow getting tests working around it. [13:58] tedg: I don't think you will. From the mails you'll have the focus/resume stuff done pretty soon [13:58] greyback, Yeah, I'm hoping today. But more worried about landing, releasing, etc. [13:59] greyback, But wanted to make sure that wasn't an issue, so we could find a solution if it was. (i.e. release the API without backing or something) [14:01] tedg: you can land and release your stuff independently of my work - just tell me when so I can increment my package dependencies. [14:01] Cimi: but what's the issue? I mean, you changed the behavior. of course tests are going to fail. You need to update them [14:02] greyback, Sure, but you can't link to symbols that don't exist. So I need to get my part done first :-) [14:03] tedg: indeed. But I can manage locally with your proposed focus/resume API until you've that part ready [14:03] Cimi: it doesn't seem that complex after all [14:03] mzanetti, can we slow down the mouse? [14:03] tedg: so don't worry about me, I'm too far behind you just yet [14:03] Cimi: no... but I don't think you need that [14:03] mzanetti, code seems correct to me [14:03] Cimi: well, you can. but I don't know how out of the top of my head [14:04] Cimi: so. start with one test only. run this: autopilot run unity8.shell.tests.test_hud.TestHud.test_show_hud_button_appears [14:05] Cimi: the test will fail, telling you that the buttons opacity is not 1.0 as expected by the test in line 61 [14:05] Cimi: but instead it is 0.5 [14:05] which is what you have changed. so adjust the expected value to 0.5 [14:05] no? [14:05] should be 1 === dandrader|afk is now known as dandrader [14:07] Cimi: right... the swipe_coords don't seem to match any more [14:07] mzanetti, mmm maybe because of the bottomMargin [14:08] nope [14:08] doesn't make sense [14:08] Cimi: I added a sleep(10) before that check [14:08] Cimi: it seems to move the pointer only very little [14:09] Cimi: because it moves it to hud_show_button.y - hud_show_button.height [14:09] Cimi: did you change the height of the button item? [14:10] nope [14:23] mzanetti: my MP https://code.launchpad.net/~stolowski/unity8/cancel-previews/+merge/187249 [14:44] Cimi: so actually the issue seems to be that the button doesn't change its opacity to 1 indeed [14:44] mzanetti, yep but why? [14:46] Cimi: I have a feeling... let me verify.. will take 5 mins or so [14:46] mzanetti, could be that onDistanceChanged is not emitted === alan_g is now known as alan_g|tea [14:50] Cimi: I think its the usage of Behavior together with states... thats a bad thing... but there is another issue [14:50] Cimi: right now I see the button has 4 opacity values [14:51] Cimi: 0 (when its hidden) [14:51] Cimi: then there is one that looks like 0.25 (during the time it moves up/down) [14:51] Cimi: 0.5 when it reached its final place but does not contain the mouse [14:51] Cimi: and 1 when it contains the mouse [14:51] Cimi: hoewer, I can find only 3 of them in the code [14:52] mzanetti, those are internal [14:52] mzanetti, hudButton changes the real opacity [14:52] Cimi: yeah... I'm looking at the code. not the test [14:52] mzanetti, but from outside (what we are testing, bottomBar), it's correctly either 0.5 or 1 [14:52] as I believe the issue is actually there [14:53] Cimi: why doesn't hint have a opacity value set? [14:54] the state "hint" [14:54] so I'm quite sure thats the isse. if there is a state change while the behavior is still running the behavior overwrites the target state value again [14:54] Cimi: 1) make sure all states have opacity values defined [14:54] mzanetti, it does 0.5 [14:55] mzanetti, line 215 here [14:55] Cimi: 2) use a transition instead of behavior [14:55] Cimi: sorry... meant the state "reveal" [14:55] oh... it has extends hint [14:55] yep [15:01] mzanetti, it works with the mouse though... [15:05] this is really weird stuff indeed [15:07] mzanetti, might be onDistance [15:08] mhm... that looks suspicious indeed [15:09] Cimi: yeah... quite sure thats it [15:10] Cimi: the question is why though [15:11] dandrader: can you see why this failes with autopilot? http://paste.kde.org/p7e10ed32 [15:11] Cimi: hah... probably it doesn't even go to recognized [15:12] because the autopilot moves so fast that it's not detected to be a valid gesture [15:12] mzanetti, yeah [15:12] mzanetti, I was thinking that too [15:12] mzanetti, maybe autopilot is simulating a drag with move events too far apart [15:13] dandrader: yeah... I think at some point they changed it to only do 2 points. start and end [15:14] mzanetti, so it's going straight to commitDistance, skipping the step where it would hit the revealDistance check [15:15] mzanetti, well, if I'm not mistaken, I recall I solved at least two autopilot issues I had by adding more steps into autopilot's drags [15:16] Cimi:^^ [15:16] mzanetti, but it's not solved now :) [15:17] ? [15:17] Cimi: did you try what daniel suggested? [15:18] mzanetti, of course not [15:24] dandrader, adding those steps where? in the test case? [15:39] Cimi: yeah... in the testcase... but for some reason I can't make it work [15:39] gosh I hate autopilot === dandrader_ is now known as dandrader [15:47] Cimi: ok. one step closer [15:47] Cimi: this is the debug print from onDistanceChanged [15:48] DDA status: 0 [15:48] DDA status: 1 [15:48] DDA status: 2 [15:49] so it looks the dda is not the issue after all [15:56] fginther, ah, you beat me to it by a minute, have now cancelled my duplicate job in unity8-ci [15:56] fginther, I must say we've much improved on the stability of our tests now... === dandrader is now known as dandrader|lunch [15:57] Saviq, glad to hear that. [15:57] fginther, although I'm still seeing quite a bit of https://code.launchpad.net/~unity-team/unity8/scope-isactive/+merge/187179/comments/427167 [15:57] fginther, where all the tests on mediumtests-touch fail [15:57] (ignore qmluitests, that's valid failure) [15:58] fginther, I wonder if the 10s timeout is not enough there [15:59] Cimi: ok... there is something more weird [16:00] Cimi: I added debug prints in that onDistanceChanged and it turns out, the opacity is not related to the states at all [16:00] Cimi: seems you overwrite it somewhere [16:00] fginther, what's the issue with the runner for multiple suites? if it can be compressed to a sentence or two :) [16:00] Cimi: when the state goes to shown the opacity is still 0.5. then only when it has onmouseover it goes to opacity 1. still staying in the shown state [16:01] Cimi: I'd suggest you clean that mess up a bit... for example you add Behavior on opacity twice, once in BottomBar and once in HudButton etc... [16:01] Cimi: hopefully it'll be easier to debug afterwards. or might even work magically [16:02] mzanetti, i think your preview branch makes the music grid behave differently than the rest of them? [16:02] would be nice to fix :) [16:02] * mzanetti checks [16:03] Saviq, my theory is that we need a reboot between the two test suites. unity8 has special handling to restart the shell and possibly that is causing the app tests to fail [16:04] fginther, well, it should work without reboots, so let's make sure it does somewhen soon [16:04] fginther, we should be able to repro locally with otto ,right? [16:04] Saviq, otto still needs some work for touch [16:05] Saviq, we've been using om26er's scripts so far [16:05] fginther, so phablet-test-run? [16:05] Saviq, checking === alan_g|tea is now known as alan_g [16:07] Saviq, it's calling autopilot directly, it should be updated to use phablet-test-run [16:09] fginther, ok, I'll try and reproduce our latest failures tomorrow [16:09] fginther, it might be as easy as adding some delay [16:11] mzanetti, btw, there's a conflict in your split surfaces branch, when you have some free cycles (sic!) [16:14] Saviq: are you using that branch for something? [16:14] mzanetti, trying to run it on mir currently [16:14] ahok [16:14] mzanetti, I solved the conflict here, but there's some weirdness in launcher height [16:15] mzanetti, so might be I did something wrong [16:15] * mzanetti merges while waiting for compile run on phone [16:16] * Saviq just set up sbuild+ccache properly on manta, hopes to speed up ↑ [16:16] mzanetti, can get you a script for ↑ that does *almost* everything needed [16:16] won't that break every time you flash? [16:17] mzanetti, somewhat [16:17] mzanetti, I'm bind-mounting everything to /home/phablet [16:18] mzanetti, or almost everything [16:18] mzanetti, so it should be a case of `apt-get install sbuild` on flash, assuming home is preserved, which was not the case for me a few times now [16:18] mzanetti, but well, who needs flashing anyway, apt should be good enough for a build machine ;) [16:19] Saviq: well... I flashed countless times in the last 2 days for some reason :D [16:20] mzanetti, CRAZY! [16:20] Saviq: you know why [16:20] mzanetti, yeah, /me still needs to try and repro with the latest stuff [16:20] that's one of the reasons why I have the sbuild, so that I can build latest trunks and beat the heck out of it on the devices [16:21] Saviq: nah... I'm pretty sure what happened and pawel's fixes look really good [16:38] Saviq, any idea by chance what changed in the current touch-image that could cause gi/DBus/Python to no longer work... -> http://pastebin.ubuntu.com/6150866 [16:39] MacSlow, what if you run it manually? [16:39] MacSlow, the dbus-launch... command, I mean? [16:39] Saviq, oh... btw... I'm using the flash-target "ubuntu-system" as "cdimage-touch" didn't work on my GalaxyNexus... [16:39] MacSlow, it exited with code 1, so it sounds like an error dbus-launch removed [16:39] oh one sec [16:40] s/removed/returned/ [16:40] MacSlow, image shouldn't matter [16:40] Saviq, doh... -> "Autolaunch error: X11 initialization failed." === salem_ is now known as _salem [16:42] MacSlow, sounds like something in your env is b0rked [16:42] mzanetti, I didn't add it :) [16:42] Saviq, I hardly touched the settings after I flashed it [16:43] Cimi: not really sure how to answer that one :D [16:43] mzanetti, I just added a couple of states… you can see from the diff [16:43] Saviq, never mind... wasn't id=phablet... [16:43] MacSlow, not sure what to make of that, no idea how dbus-launch should behave... or what it requires... [16:43] Saviq, works now [16:43] MacSlow, right [16:44] Saviq, still not fully back yet... [16:44] MacSlow, fish are EVIL [16:45] Saviq, well... I tried a new sushi-place downtown... [16:45] muahaha [16:45] Saviq, looked very fancy and good... [16:46] if you really need to have fish... at least cook it before dude [16:46] :P [16:46] but then, I didn't see their shopping-habits or kitchen, which one usually takes for granted to be ok [16:46] mzanetti, are you sure of this opacity thing? [16:46] mzanetti, from what I see, there are two opacity [16:46] mzanetti, I'll stick to the known sushi-bars in the future [16:47] Saviq, password and user-auth dialogs work... [16:47] mzanetti, one is in bottom bar, and one in hudbutton [16:47] mzanetti, we are checking the one in bottom bar [16:47] MacSlow, cool [16:47] Saviq, simunlock doesn't... as the indicator-network is "in the way" [16:47] MacSlow, sure, MP away! [16:47] Cimi: yeah... the state "shown" has different opacity values, depending on the position of the mouse [16:47] Cimi: while it should only have the one defined in that state [16:48] mzanetti, it only has one [16:48] mzanetti, 1.0 [16:48] mzanetti, yay, we got two surfaces on Mir - only problem you can only see the "flickers" from the launcher one ;) [16:48] Cimi: yeah... in the code... but not for real [16:48] Saviq, the ap-tests aren't done/working yet... and I now have to sort out how to make the simunlock work with "indicator-network" being running in the bg [16:48] mzanetti, but I assume this can be fixed by surface type or something [16:48] Cimi: add a onStateChanged hanlder wnd print the current state [16:48] mzanetti, what you see is the internal opacity of hudbutton, doesn't count [16:48] MacSlow, just so you know, tests are needed, but the feature probably even more [16:49] MacSlow, so park the work on tests if you can, and let's just get the feature in and look at tests when it's done [16:49] Cimi: yes it counts... it makes the code unreadable and unmaintainable. and tests are so hard to debug that you can't do it yourself any more [16:49] Saviq, well at least let me try to sort out the now failing simunlock as kgunn (and pete-woods probably too) is urging to have it in image [16:50] MacSlow, yup [16:50] Saviq: hmm... its a standard surface... shouldn't flicker [16:50] Saviq: the only exception is alwaysontop [16:50] Saviq: ah... might ba that the alwaysontop is not working so its actually behind [16:50] mzanetti, yeah, but Mir probably doesn't handle everything, we need to set the surface type or something [16:51] Saviq: and you see it flicker because of the same reason you see apps shining trhough [16:51] mzanetti, yup [16:51] * kgunn cheers on MacSlow [16:51] * mzanetti too [16:51] * MacSlow wishes he was born a reno-air-race pilot atm [16:51] \o\ /o/ \o/ \o\ /o/ [16:52] can we change the channels topic? [16:52] * Saviq wonders who decided it's going to be cool to generate an ssh key per-ubuntu-device [16:52] * Saviq couldn't do password-login to some places and couldn't figure out why [16:53] mzanetti, I don't think it changes anything in terms of tests [16:53] dednick, can the simunlock UI be used (in the snap-decision) with the indicator-network running? [16:53] but it seems trying 6 keys in a row makes the other side say not nice things [16:53] mzanetti, anyway, we can rewrite the code another time... [16:53] mzanetti, I didn't write this [16:53] dednick, that's what's currently blocking it to work on the device [16:53] MacSlow, doesn't the network indicator trigger the sim dialog? [16:54] Cimi: thing is debugged this for 2 hours now. I'm at the stage where I would start cleaning up the code if this would be my branch [16:54] MacSlow: I had it working on the device using my branches [16:54] mzanetti, ok... [16:54] mzanetti, I can start a rewrite but I cannot see how it could change [16:54] Saviq, well in the end yes... I guess... not sure about that integration-detail... but for demonstrating/testing I'd like to stick to my stand-alone python-examples... as I know much better how to control them [16:54] MacSlow: to a somewhat limited extent. I had some issues with ofono bugs. which are in progress [16:55] Cimi: not saying you should rewrite [16:55] but there is something fishy with the states [16:55] mzanetti, I think it's only due to autopilot [16:55] mzanetti, what's wrong with the states? [16:55] Saviq, dednick: hm... that leaves me in an akward position for the MR [16:55] MacSlow: why? [16:56] mzanetti, I think the issue is that the mouse movement is not recognised [16:56] dednick, the example/test for the simunlock will never work on the device as is [16:56] Cimi: it is... I added debug prints for that [16:56] the state goes to shown [16:56] MacSlow: why not? [16:57] mzanetti, and then? [16:57] Cimi: still the opacity goes not to 1, even if the state is "shown" [16:57] why on earth? [16:57] dednick, doh... never mind... just fixed it [16:58] kgunn: you'll get you screencast in a few minutes... [16:58] mzanetti, nothing is overriding opacity [16:58] let me run it again [16:59] kgunn, Saviq, dednick, pete-woods: I'll do a MR for the three first ext. snap-decision dialogs then afterwards [16:59] :D [16:59] Saviq, I'll try to catch veebers later this night about my issues with ap-tests for the ext. snap-decisions, thus I can bring them in a later MR once I hopefully got them working [17:05] mzanetti, still same issue? [17:05] mzanetti, would be interesting doing the same with the mouse and see [17:06] Cimi: no... I was wrong about the state... this is the output when running the test: http://paste.kde.org/pa950fa89 [17:07] mzanetti, indeed it keeps stuck in hint [17:07] mzanetti, can you bzr diff | pastebinit ? [17:09] Cimi: yeah. doing one more run with more debug prints. one sev [17:09] sec [17:09] mzanetti, I have 10 mins then the gate closes and I'll lose my flight :) [17:10] Cimi: you're at the airport? [17:10] mzanetti, yes [17:10] mzanetti, in the lounge... [17:10] mzanetti, gotta go to the dentist tomorrow afternoon :) [17:11] Cimi: http://paste.kde.org/pc78df946 and http://paste.kde.org/p4e152d15 [17:11] mzanetti, it's a relative [17:11] Cimi: you fly to your dentist... interesting [17:11] lol [17:12] gotta renegotiate my salary I guess [17:12] :P [17:12] mzanetti, what's interesting [17:12] just joking [17:12] mzanetti, is that onDistanceChanged is emitted basically only once [17:13] mzanetti, I mean one, very rarely [17:13] mzanetti, it should print loads of them [17:13] mzanetti, the drag is definitely too far [17:13] Cimi: yeah... autopilot does freaking begin/end steps aparently [17:13] but i even tried to do it in a loop [17:13] with calling each pixel manually [17:14] mzanetti, we should add some function that splits the movements into more than one, using for [17:14] still nothing [17:14] lemme add it again [17:14] one sec [17:14] too far -> too fast I meant [17:14] actually the too far was correct [17:14] ok :) [17:14] but yeah... same thing somewhat [17:15] mzanetti, the status "shown" is triggered by onDistanceChanged, same thing for the others [17:15] mzanetti, so it kinda relies on that to work... === _salem is now known as salem_ [17:17] mzanetti, unless the asserts kinda stop the swipe? [17:19] I have to leave… I'll have a look when I land [17:20] "at least I'll see the light" [17:20] Cimi: ok... I'll do some more debugging. by now I'm curious whats going on [17:20] have a good flight [17:20] (aka, the sun) [17:20] thanks [17:20] haha [17:20] chat later [17:20] k === dandrader|lunch is now known as dandrader [18:03] * greyback eod === dandrader is now known as dandrader|afk === dandrader|afk is now known as dandrader [18:32] kgunn, Saviq: screencast (still uploading) and MRs put up... see related eMail [18:32] MacSlow, yeah, thanks! [18:32] MacSlow: thanks man [18:32] MacSlow, if you ask me, again, the feature is more important than the test [18:32] MacSlow, so wifi selection dialog it is [18:34] Saviq, ok... also more likely to be completed before the week ends... regarding the ap-test... phew... I really don't know yet [18:34] MacSlow, is ine [18:34] fine [18:34] we'll get there [18:35] think so too... see you folks tomorrow [18:35] o/ === jhodapp is now known as jhodapp|afk [19:36] mterry, "Note: You can only anchor an item to siblings or a parent." [19:36] mterry, at the end of http://qt-project.org/doc/qt-5.0/qtquick/qml-qtquick2-item.html#anchors.alignWhenCentered-prop [19:36] mterry, and yeah, I saw it was working fine [19:36] mterry, but let's just be correct for the sake of correctness :) [19:37] Saviq, maybe by "siblings or parents" they really mean "non-children" [19:37] mterry, not "parents", but "a parent" [19:38] Saviq, correct, but my point is the same. They may just be meaning "anywhere else in the widget tree is fine, just not below the anchor" [19:38] mterry, http://paste.ubuntu.com/6151579/ [19:38] test.qml:11:9: QML Item: Cannot anchor to an item that isn't a parent or sibling. [19:38] mterry, been there, done that [19:39] mterry, your thing doesn't go up with that warning with sheer luck ;) [19:39] mterry, I'd be keen to look if it actually is set through the createObject [19:39] Saviq, OK... how does mine even work then? [19:39] mterry, no idea ;) [19:39] mterry, been looking through the log to see the same failure, but didn't :) [19:40] Saviq, the previous code did the same thing, panel.indicators also isn't sibling/parent of the edge demo [19:40] mterry, yes it is, that's the first argument to createObject [19:42] Saviq, right, I forgot I placed each overlay object into its target object [19:42] jeez why is U1 so slow :( [19:42] Saviq, OK. I guess that's a simple fix to update the parent to be demo.indicators.content; weird that I was getting away with it [19:42] mterry, we could probably just go with the Overlay having anchors.fill: parent by default [19:42] mterry, and just create it as a child of whatever it's supposed to be part of [19:43] mterry, but it's fine like it is, too [19:43] ChrisTownsend, do you know when did the i386 failures start? [19:43] It seems like last night or so. [19:43] mhr3: ^^ [19:43] ChrisTownsend, rather, when was last successful build? [19:43] mhr3: Let me try to find that out. [19:44] Saviq, hmm, no I remember I ran into a problem with that elsewhere. Like topEdgeDemo wanted to be created in underlay otherwise there was input issues, but it wants to target the dash [19:44] but that's a sibling at least [19:45] Saviq, oh wait. so is demo.indicators.content, eh? Isn't that a sibling of the object I'm creating under demo.indicators? [19:47] mhr3: Ok, these times are Eastern US times: last good build was reported at 3:41 am this morning. First bad build was reported at 10:09am this morning. [19:48] eh, where's a tz convertor when you need one :) [19:48] mhr3: Yeah, really:) [19:49] mhr3: Also, those are the times I received emails from Jenkins, so the times could be approximate. [19:50] ChrisTownsend, you mean regular unity merge mails? [19:50] last one i got is 12hours ago... hm [19:50] mhr3: The emails from Jenkins CI jobs for MP's - either the initial CI or an automerge CI. [19:51] mhr3: You want me to forward them to you? [19:51] ChrisTownsend, this shows the times properly, right? [19:51] https://jenkins.qa.ubuntu.com/job/unity-saucy-i386-ci/ [19:52] and you're saying last success is 276, is that correct? [19:53] mhr3: This is the last good job: https://jenkins.qa.ubuntu.com/job/unity-ci/388/ [19:54] mhr3: This is the first fail job with the error I reported: https://jenkins.qa.ubuntu.com/job/unity-autolanding/309/ [19:55] mhr3: So yes, for i386, job 276 is the last good one. [19:56] mhr3: For failing on i386, this is the job: http://jenkins.qa.ubuntu.com/job/unity-saucy-i386-autolanding/265 [19:56] mhr3: I hope that helps [19:59] what would help is a i386 machine :/ [20:01] mhr3: We are trying to simulate the building in an i386 chroot. [20:01] so am i [20:01] but it's been a while that i updated it [20:01] mterry, ah! sibling it is ;) [20:02] will take a while [20:03] ChrisTownsend, but anyway, the test-service helper crashes, would be nice to get a stacktrace where does it happen [20:04] mhr3: Ok [20:04] mhr3, sooo i just finished building unity in i386 pbuilder...and there are the failures: [20:04] http://paste.ubuntu.com/6151673/ [20:04] bschaefer, stacktrace pls ^ [20:04] no scope problems :( [20:04] hmm, yea, that no good [20:05] bschaefer, do you have the ppas enabled? [20:05] mhr3, its not crashing here :( [20:05] nope [20:05] mhr3, that does make sense [20:05] bschaefer, enable them :) [20:05] :) [20:05] mhr3, daily build ppa right? [20:06] yep [20:06] cool, ill restart that then with that ppa... === salem_ is now known as _salem [20:44] bschaefer, updated my chroot, can't reproduce :/ [20:44] mhr3, mines building atm :( [20:44] mhr3, though jenkins always likes to be different... [20:45] mhr3, possibly you could get an i386 machine from qa? [20:51] mhr3, yup, mine didn't hit the issue either :( [20:55] bschaefer, mhr3: need i386 testing for something? [20:56] seb128, yeah, for this issue: [20:56] https://jenkins.qa.ubuntu.com/job/unity-autolanding/309/ [20:56] the i386 seems to be failing on scope tests [20:56] * bschaefer digs up bug ChrisTownsend made [20:56] https://bugs.launchpad.net/bugs/1229891 [20:56] Ubuntu bug 1229891 in Unity "Many Unity unit test failures on i386 CI due to some Scopes backend issue" [Critical,Confirmed] [20:57] does that require to build unity? [20:57] is that current saucy or trunk? [20:58] seb128, i believe saucy with daily build ppa, and yeah unity will need to build to run the tests [20:58] non trivial then [20:58] as the issue seems to be in libunity, which is causing unity unit tests to fail causing the automerger to fail :( [20:58] seb128: Yeah, it's also trunk. We are blocked on automerging because of this. [20:59] but we can't reproduce the problem in chroot... [20:59] let me see if I can reproduce locally [20:59] so basically building libunity trunk then unity trunk should be enough? [21:00] seb128, as long as you have an i386 machine :) [21:00] thats the real problem we are running into... [21:00] yes, I'm running i386 [21:00] cool, then yes that should be neough [21:00] enough* [21:01] * seb128 starts building [21:01] seb128, thanks! [21:04] bschaefer, libunity fails to build on tests here [21:04] seb128, hmm thats strange... [21:04] * bschaefer also doesn't know much about libunity [21:05] ../../test-driver : ligne 95 : 12106 Erreur de segmentation (core dumped) "$@" > $log_file 2>&1 [21:05] segfault [21:05] hmm [21:05] * bschaefer attempts to build libunity [21:07] seb128, this is just tests for libunity? [21:07] bschaefer, it's doing "bzr branch lp:libunity; cd libunity; bzr bd" [21:07] but "make check" in the build tree hits the same segfault reliably [21:07] i get a fun: [21:07] -a make test-nonrecursive; \ [21:07] sleep 1; [21:07] /bin/bash: line 1: -a: command not found [21:08] seb128, when I try make check...but i know mhr3 wanted a stracktrace [21:08] ChrisTownsend, but anyway, the test-service helper crashes, would be nice to get a stacktrace where does it happen [21:08] mhr3, hey [21:08] seb128, you've the daily build ppa? [21:08] no, current saucyt [21:09] hmm I think mhr3 mentioned the need for the daily build ppa, but that could have been for chroot only :( [21:09] well, I'm having the testsuit segfaulting in current saucy [21:09] that could be the same issue [21:09] which isn't good, yeah [21:09] seb128, would you be able to get a stacktrace and add a comment on that bug? [21:09] https://bugs.launchpad.net/bugs/1229891 [21:09] Ubuntu bug 1229891 in Unity "Many Unity unit test failures on i386 CI due to some Scopes backend issue" [Critical,Confirmed] [21:09] well, I'm trying [21:10] but running that .py by hand doesn't segfault [21:10] not sure how to run "make check" under gdb [21:10] hmm neither do i, unless you know where that file is... [21:11] no, I'm trying to read the makefile [21:11] possibly try to figure out which test suite is failing? in test/vala... [21:11] would be useful to have mhr3 around [21:11] very much so :) [21:11] FAIL: container-ownership.py [21:12] * bschaefer does not know much/anything about libunity :( [21:13] well thats in test/python... [21:13] right [21:17] seb128, i get the same [21:17] ../../test-driver: line 95: 6877 Segmentation fault (core dumped) "$@" > $log_file 2>&1 [21:17] FAIL: container-ownership.py [21:17] good [21:17] so you have something to debug (or mhr3 has something to debug since it's libunity) [21:17] welll then this must be un related to the i386 issue...as i have 64bit... [21:17] but it still needs to be fixed [21:18] seb128, right, would you still be able to install libunity and test unity out? [21:18] yes [21:18] just to see if you get unity unit-test failures? [21:18] thanks [21:18] * bschaefer tries it out as well [21:18] as Im not sure if i was using the right libunity [21:19] seb128, thanks again, I don't want to keep you up to late! [21:19] bschaefer, don't worry, I'm not really working, mostly chatting on the computer, builds can run on the side ;-) [21:20] seb128, :) [21:30] seb128, you're building it with latest vala [21:30] seb128, pbuilder does it correctly :P [21:31] mhr3, not, I'm using 0.20.1 [21:31] seb128, exactly, we use 0.18 to build libunity [21:31] cause there are issues in 0.20 [21:31] ok, current vala is 0.22 [21:31] just for info :p [21:32] ah, ok you win :P [21:32] ;-) [21:32] it should work with 0.22 iirc [21:32] should have the fix we need [21:32] vala sucks [21:32] yea, yea, c++ is awesome we all know that [21:33] mhr3, im glad you agree :) [21:35] bschaefer, sorry, didn't realize it needs a tag :P [21:35] mhr3, haha, i got it, but I suppose i needed a tag as well [21:35] mhr3, it's so great that I started that build 15 minutes ago and unity is still not half built [21:36] seb128, and that's only one of the great features ;) [21:36] mhr3, but wait theres more! [21:36] * bschaefer has watched to many infomercials [21:38] seb128, do you know if there are any other ppas the autolanding jobs use besides daily-build? [21:38] not that I know, why? [21:40] cause isn't using trunks for the stack, and the daily build ppa is... you know... daily [21:40] it's using* [21:58] bschaefer, mhr3: no test issue here :/ [21:59] on that note I'm calling it a day [21:59] bye [21:59] * bschaefer wonders if its jenkins falut [21:59] seb128, thanks! [21:59] fault* [21:59] seb128, thx for checking cu [22:00] bschaefer, is there a way to have jenkins run through a branch without actually approving it? [22:00] * mhr3 has an idea [22:03] mhr3, hmm there use to be [22:03] veebers, ping? [22:04] mhr3, if you link me a branch i can try to push it onto someone to test it [22:11] bschaefer, actually i think if you'd just approve it [22:11] mhr3, i can do that as well :) [22:11] but not top-approve it'd run [22:11] yeah, sounds like a good idea [22:11] let me prepare the branch [22:11] cool [22:15] bschaefer, https://code.launchpad.net/~unity-team/unity/get-some-debug/+merge/187377 [22:15] mhr3, done, and have a good night :) [22:16] it's not even midnight yet :P [22:16] unless you want to stick around for it to output some fun [22:16] :), thats true here as well! [22:17] oh wait, the change that happened is that we run on real hw now, right? [22:18] so... yey for uncovered threading issues maybe? [22:19] hmm im not sure, but its consistent? You would think it would be easy to reproduce else where though right? [22:19] as seb ran it on his machine (im assuming hw) [22:27] dunno, lets wait for the trace, but the error is pretty weird [22:27] plus only on i386... :/ [22:30] * bschaefer didn't actually look at the error yet ... [22:31] the stacktrace would be nice :) [22:31] will* [22:42] bschaefer: pong [22:52] veebers, unpong! [22:52] bschaefer: ^_^ ack [22:52] thanks :)