| === jhodapp is now known as jhodapp|afk | ||
| === duflu_ is now known as duflu | ||
| tsdgeos | is it me the only one having that i can't get a login to unity7 anymore? | 08:14 |
|---|---|---|
| tsdgeos | i select unity7 and then i get nothing :D | 08:14 |
| tsdgeos | need to start it from the vt | 08:14 |
| tsdgeos | and somehow my fonts are different :S | 08:15 |
| tsdgeos | back in a sec | 08:15 |
| tsdgeos | fonts are back to correct at least | 08:17 |
| Saviq | tsdgeos, works fine here, only issue is that I have to unlock twice upon resuming | 08:20 |
| tsdgeos | maybe something gets upset about me using kdm and not lightdm | 08:20 |
| tsdgeos | which would be totally sad, but you know how the future is, no more splitting of responsabilities, one process takes it all | 08:21 |
| === didrocks1 is now known as didrocks | ||
| === shiznix_ is now known as shiznix | ||
| didrocks | Saviq: hey, I remember sil2100 mentioning that the unity8 landing has a workaround for the current AP tests, right? | 09:04 |
| Saviq | didrocks, one of them, yes | 09:04 |
| didrocks | Saviq: you tested with latest and greatest, of course? :) | 09:04 |
| Saviq | didrocks, on Friday, so no :P | 09:05 |
| didrocks | Saviq: well, you dist-upgraded with 5.2? | 09:05 |
| Saviq | didrocks, of course | 09:05 |
| didrocks | so I guess that's fine, or do you want an additional try? | 09:05 |
| Saviq | didrocks, no, go for it please | 09:05 |
| Saviq | didrocks, we have one more fix in store that we'll land as soon as this is done | 09:06 |
| didrocks | ok, Mirv, mind giving a look at publishing the unity8 one? ^ | 09:06 |
| didrocks | thanks Saviq :) | 09:06 |
| didrocks | Saviq: I propose next landing embeed the icon theme changes as well | 09:08 |
| didrocks | so that one is done and gone | 09:09 |
| didrocks | (it's only unity8 + a seed change) | 09:09 |
| Saviq | didrocks, yup, ok | 09:09 |
| Mirv | didrocks: Saviq: yes, already started, I was thinking doing Unity8 AP tess manually first although if we're back to normal "we don't duplicate tests" then I can just release it :) | 09:09 |
| didrocks | Mirv: maybe we should do a double check as this testing happened on Friday, just in case, wdyt? | 09:10 |
| Saviq | Mirv, only unknown issue there was the crash, but I couldn't get to the.crash file | 09:10 |
| Mirv | didrocks: Saviq: will do, especially since I already enabled PPA:s etc so ready to run tests | 09:10 |
| didrocks | thanks Mirv :) | 09:10 |
| tsdgeos | Saviq: so basically kdm doesn't seem to want to start unity7 and lightdm itself doesn't even want to start in my computer ^_^ | 09:12 |
| tsdgeos | something broke a lot over this weekend :D | 09:12 |
| Saviq | tsdgeos, yikes | 09:12 |
| Saviq | dednick, hey, could you review https://code.launchpad.net/~unity-team/unity8/suru-switch/+merge/207991? should be small | 09:21 |
| tsdgeos | meh, everything here is so broken i get this | 09:27 |
| tsdgeos | tsdgeos_work@xps:~$ start scope-registry | 09:27 |
| tsdgeos | start: Unknown job: scope-registry | 09:27 |
| tsdgeos | MacSlow: so about that ghost notification we have there, could you check that we still need it with 5.2? | 09:29 |
| Saviq | tsdgeos, hmm sounds like you removed unity? | 09:30 |
| Saviq | tsdgeos, or parts of it.. | 09:30 |
| tsdgeos | Saviq: well i'm running it now | 09:30 |
| tsdgeos | so no :D | 09:30 |
| Saviq | tsdgeos, apt-cache policy libunity-scopes0? | 09:31 |
| Saviq | tsdgeos, that's what's shipping the scope-registry job | 09:31 |
| tsdgeos | Saviq: it's there | 09:31 |
| Saviq | tsdgeos, yeah, but what version? | 09:31 |
| tsdgeos | i guess that since my session kind of doesn't start properly | 09:31 |
| tsdgeos | and i've brought stuff up manually | 09:31 |
| tsdgeos | all is unhappy | 09:31 |
| MacSlow | tsdgeos, sorry... answered in the wrong channel :) | 09:31 |
| tsdgeos | Saviq: 0.4.0+14.04.20140312.2-0ubuntu1 | 09:31 |
| MacSlow | tsdgeos, yes... it the "PlaceHolder" for the synchronous notification (e.g. volume up/down) once we'll get a Design and UX for it. | 09:31 |
| Saviq | tsdgeos, that would complain about upstart being unavailable, not that it can't find the job... | 09:32 |
| Saviq | MacSlow, yeah, but maybe we don't need it when it's not used | 09:32 |
| Saviq | MacSlow, and we're not gonna use it on phone IIRC | 09:33 |
| MacSlow | Saviq, the moment we remove it, I bet I'll get a mockup for it ;) | 09:33 |
| Saviq | MacSlow, right | 09:33 |
| MacSlow | Saviq, and I would not want to even further hold back all the notification-related branches due to that | 09:34 |
| mhr3 | Saviq, can i count on silo with new-scopes today? | 09:55 |
| Saviq | mhr3, I think yes, I only have a small landing to do once the current silo for u8 gets in | 09:56 |
| Saviq | mhr3, which should be real soon | 09:56 |
| mhr3 | ok | 09:57 |
| dednick | Saviq: sure | 10:13 |
| mzanetti | Saviq: so... I was reading through the the 8k diff... Looks quite good. Only thing is the PreviewDelegateMapper. Given that this is a cleanup branch already, I tend to think we should already drop the file and replace it with the simeple if condition. | 10:24 |
| Saviq | mzanetti, you mean scopedelegatemapper? | 10:25 |
| mzanetti | err, yeah | 10:25 |
| Saviq | mzanetti, there should be no previewdelegatemapper any more | 10:25 |
| mzanetti | *delegatemapper, anyways | 10:25 |
| Saviq | tsdgeos, get rid of it ↑? | 10:25 |
| tsdgeos | Saviq: it's used/useful for the tests | 10:26 |
| mzanetti | ah... | 10:27 |
| mzanetti | well, again... no biggie... I was just thinking that if we don't clean it up in the cleanup branch, this file will be left stale in the tree eventually | 10:28 |
| tsdgeos | mzanetti: well, it's used, isn't it? | 10:28 |
| mzanetti | tsdgeos: it is right now, but could be replaced by a "scope == apps ? DashApps.qml : GenericScopeView.qml" | 10:29 |
| mzanetti | tsdgeos: and as DashApps will go away soon too... | 10:29 |
| tsdgeos | mzanetti: well it'll be stale when it's stale, not now ;) | 10:29 |
| mzanetti | ok | 10:30 |
| tsdgeos | Saviq: so i have the patch that kills DashFilterGrid.qml and moves it to CardFilterGrid.qml | 10:37 |
| tsdgeos | Saviq: commit to cleanup? | 10:37 |
| Saviq | tsdgeos, yeah | 10:37 |
| tsdgeos | done | 10:38 |
| tsdgeos | and i broke tests ^_^ | 10:40 |
| tsdgeos | done now | 10:43 |
| Saviq | mzanetti, thanks for catching the panel issue, fixing | 10:45 |
| mzanetti | np | 10:46 |
| Saviq | mzanetti, btw, trunk just got the prereq branches merged | 10:51 |
| mzanetti | Saviq: nice. thanks | 10:51 |
| Saviq | mzanetti, or actually, /me needs to push a button first | 10:52 |
| mzanetti | ack | 10:52 |
| Saviq | mzanetti, done | 10:55 |
| mzanetti | Saviq: is there a way to update the diff on LP without pushing something? | 10:58 |
| mzanetti | I guess resubmitting would do | 10:58 |
| Saviq | mzanetti, no need | 10:58 |
| Saviq | mzanetti, LP will notice | 10:59 |
| Saviq | (should) | 10:59 |
| Saviq | mzanetti, give it a few minutes | 10:59 |
| mzanetti | ok | 10:59 |
| Saviq | Mirv, hey, can we have a silo for row 55? | 11:09 |
| Saviq | dednick, re: icons being small, that's probably because you didn't actually get the suru icons, which don't have padding like the others had | 11:16 |
| dednick | Saviq: hm. i installed the theme... | 11:16 |
| Saviq | dednick, yeah, I can't get it to work on desktop here either | 11:17 |
| Saviq | dednick, and Icon is working, we still need the colourize, only issue is that they're not aspect-correct, but that was design-approved for the time being | 11:17 |
| Cimi | Saviq, tsdgeos you guys need help with new scopes? | 11:34 |
| Saviq | Cimi, we're good I think | 11:35 |
| Cimi | reviews? | 11:35 |
| Saviq | didrocks, can we get a silo for row 55 please? | 11:35 |
| Saviq | Cimi, there's plenty of branches in need of review in https://code.launchpad.net/~unity-team/unity8/trunk/+activereviews | 11:35 |
| didrocks | on porter, will get back to you afterwards | 11:35 |
| Cimi | Saviq, saw that, was wondering if you had one in particular | 11:36 |
| Cimi | I'll start | 11:36 |
| Mirv | Saviq: seems to be already | 11:36 |
| Mirv | Saviq: correction, not. so, now in landing-012 | 11:36 |
| Saviq | Mirv, thanks | 11:36 |
| Saviq | didrocks, unping | 11:36 |
| didrocks | good :) | 11:37 |
| Saviq | xnox, hey, did you manage to have a look at crossbuilding under Qt 5.2? the Qt cmake modules fail now due to mkspecs being in multiarch libdirs | 11:40 |
| tsdgeos | Saviq: are you merging trunk to new-scopes-clean-to-trunk and new-scopes-cleanup or should i? | 11:41 |
| Saviq | tsdgeos, I can | 11:41 |
| tsdgeos | Saviq: cool | 11:41 |
| === jhodapp|afk is now known as jhodapp | ||
| Cimi | tsdgeos, https://code.launchpad.net/~aacid/unity8/new_tabbar/+merge/210453 | 11:50 |
| Saviq | tsdgeos, mzanetti, the clipping of DashContent is about when we "zoom out" on right edge | 11:50 |
| tsdgeos | Cimi: what? | 11:50 |
| Cimi | tsdgeos, the -1 is for debugging purpose? | 11:51 |
| Cimi | tsdgeos, we can probably just use 0 | 11:51 |
| Saviq | tsdgeos, mzanetti, and in new-scopes about non-favourite scopes we open from the scopes scope | 11:51 |
| Saviq | hence the || scopeView.visible | 11:51 |
| tsdgeos | Cimi: why would you default to 0 as selectedIndex? | 11:51 |
| tsdgeos | -1 is a much better default | 11:51 |
| Saviq | we're actually *only* doing scale <= 1.0, never > 1.0 IIRC | 11:52 |
| Saviq | contentScale: 1.0 - 0.2 * disappearingAnimationProgress, so yeah | 11:52 |
| Cimi | as default, yes | 11:52 |
| Saviq | tsdgeos, mzanetti, there should be no need to clip the ScopeItem, though | 11:52 |
| Saviq | or well, bottom, so maybe we need it too | 11:53 |
| Saviq | yeah, we do | 11:53 |
| * Saviq reboots | 11:54 | |
| Cimi | tsdgeos, just curiosity, why you put connections outside sortfilterproxymodel? | 12:02 |
| Cimi | they don't work inside it? | 12:02 |
| Cimi | at least the connection of dashcontentlist | 12:03 |
| tsdgeos | Cimi: which connections? | 12:03 |
| tsdgeos | ah | 12:03 |
| tsdgeos | because they were there | 12:03 |
| tsdgeos | and i just changed them | 12:03 |
| Cimi | ok | 12:04 |
| Cimi | tsdgeos, isn't like a binding loop here? | 12:09 |
| tsdgeos | Cimi: because a changes b and b changes a? | 12:09 |
| Cimi | yes | 12:09 |
| tsdgeos | no | 12:09 |
| tsdgeos | because it's not a binding | 12:10 |
| tsdgeos | just javascript code | 12:10 |
| Cimi | when tabbar model changes cound, tabbbar model changes selected index, which will change dashcontestlist index, which has a connection to selected index of tabbar model | 12:11 |
| tsdgeos | yes | 12:11 |
| tsdgeos | and will change it to 3 | 12:11 |
| tsdgeos | so as it was 3 | 12:12 |
| tsdgeos | it will do nothing | 12:12 |
| tsdgeos | no? | 12:12 |
| Cimi | tsdgeos, I see now | 12:13 |
| Cimi | tsdgeos, was simple, but better to check before approving things :) | 12:13 |
| paulliu | Cimi: Actually I got the ZoomableImage from an ancient project of MeeGo. So should I re-factor it to the Components? Because LazyImage is also there. Not sure if we need that to be in the SDK. | 12:18 |
| Cimi | paulliu, it's fine if you used someone else code | 12:18 |
| Cimi | paulliu, but we need to test it | 12:19 |
| Cimi | paulliu, in any case Components | 12:19 |
| Cimi | paulliu, then possibly one day SDK | 12:19 |
| paulliu | Cimi: ok. got it. | 12:19 |
| mzanetti | meh... did a upgrade and now my opengl is slooooow | 12:20 |
| Cimi | mzanetti, you sure you're running opengl? :) | 12:25 |
| mzanetti | Cimi: yeah, if I turn it off, everything is fast again, but ofc no transparancy or shadows or anyhting | 12:26 |
| === MacSlow is now known as MacSlow|lunch | ||
| === alan_g is now known as alan_g|lunch | ||
| Cimi | can you retrigger? http://s-jenkins.ubuntu-ci:8080/job/unity8-ci/2469/rebuild | 13:07 |
| === lool- is now known as lool | ||
| mzanetti | Saviq: with new-scopes, is there supposed to be a scope that searches everything? Like the Home scope did? | 13:16 |
| Saviq | mzanetti, unity-scope-scopes | 13:16 |
| mzanetti | ah, the scopes scope | 13:16 |
| mzanetti | wah... no space left on device to install the scopes scope :D | 13:17 |
| Cimi | this space thing just made me waste half an hour | 13:18 |
| mzanetti | Saviq: tsdgeos: here's some more new-scopes feedback: https://code.launchpad.net/~unity-team/unity8/new-scopes-clean-to-trunk/+merge/209459/comments/498372 | 13:18 |
| Saviq | mzanetti, re [] in \n | 13:19 |
| Saviq | mzanetti, it's a Qt/Ubuntu font bug, nothing we can do about it | 13:20 |
| mzanetti | ah ok... if its known and under investigation its ok | 13:20 |
| === MacSlow|lunch is now known as MacSlow | ||
| mzanetti | Cimi: can you give some input on this? https://code.launchpad.net/~unity-team/unity8/new-scopes-clean-to-trunk/+merge/209459/comments/498379 | 13:35 |
| mzanetti | and perhaps fix it? | 13:35 |
| Cimi | mzanetti, old performance issues | 13:36 |
| Cimi | on maguro | 13:37 |
| Cimi | mzanetti, the 18 assumption, is to avoid caching shitload of pixels on higher density devices | 13:37 |
| Cimi | like the nexus 10 | 13:37 |
| mzanetti | Cimi: doesn't the N10 have 16px/gu only? | 13:38 |
| Cimi | mzanetti, manta is 20 | 13:38 |
| Cimi | mzanetti, can you retrigger this? https://code.launchpad.net/~aacid/unity8/new_tabbar/+merge/210453 | 13:40 |
| mzanetti | Cimi: I can give you permissions to trigger it :) | 13:40 |
| Cimi | mzanetti, I think I need my pgp key | 13:42 |
| Cimi | and I don't know where it is | 13:42 |
| mzanetti | Cimi: for the VPN? | 13:43 |
| Cimi | yep | 13:43 |
| Saviq | Cimi, unrelated to pgp btw ;) | 13:43 |
| Saviq | didrocks, so... I'm not sure what happened (maybe Qt 5.2?), but I can't get unity8 to load the suru icons... | 13:44 |
| * Saviq builds in saucy | 13:45 | |
| Cimi | Saviq, gpg | 13:45 |
| Saviq | Cimi, again, unrelated | 13:45 |
| Saviq | Cimi, it's a vpn key, nothing to do with gpg / pgp | 13:45 |
| Cimi | Saviq, thought we needed that key to have vpn | 13:46 |
| seb128 | Saviq, do you try on a desktop? | 13:46 |
| didrocks | Saviq: urgh? | 13:46 |
| didrocks | Saviq: the package isn't installed by default | 13:46 |
| Cimi | Saviq, at least this was a year ago | 13:46 |
| Saviq | Cimi, yes, but it's not pgp/gpg | 13:46 |
| didrocks | did you install it manually? | 13:46 |
| Saviq | didrocks, yes I did | 13:46 |
| Cimi | Saviq, which key then? | 13:46 |
| Saviq | seb128, yeah | 13:46 |
| Saviq | Cimi, vpn key | 13:46 |
| seb128 | Saviq, Qt got fixed to respect the xsettings, maybe that override your choice? | 13:46 |
| seb128 | Saviq, https://codereview.qt-project.org/#change,49387 | 13:46 |
| Cimi | Saviq, cnonical sends it encrypted and you need pgp/gpg to decrypt it | 13:47 |
| Saviq | seb128, sounds sane | 13:47 |
| Saviq | Cimi, ok, that explains things ;) | 13:47 |
| Cimi | Saviq, which I lost when the laptop totally broke last time | 13:48 |
| Saviq | Cimi, your pgp key is probably something worth making a backup of, then ;) | 13:48 |
| Saviq | seb128, where do I change the theme these days? | 13:49 |
| seb128 | Saviq, icon theme? gsettings set org.gnome.desktop.interface icon-theme | 13:50 |
| Cimi | Saviq, I did! | 13:50 |
| Cimi | Saviq, worth remembering where I put it :D | 13:50 |
| Saviq | Cimi, indeed ;) | 13:50 |
| dpm | hi pstolowski, for the Go scopes, what you've been looking at is to use Go bindings directly, or would a Go scope template be using a C++ library? The reason I'm asking is because it seems that if we're using pure Go cross-compilation would be fine, but it might get trickier if the Go scope template would need to reference C++ code | 13:53 |
| pstolowski | dpm, hey! the go bindings use scopes c++ lib, but for specifics please talk to jamesh | 13:54 |
| mzanetti | Saviq: the activityindicator in the new scopes preview looks a bit odd/misplaced imo. Is this designed like this? or should there perhaps be a dark overlay or something that makes it clear what this indicator is for? | 13:58 |
| Saviq | mzanetti, there is no design for it unfortunately, we're thinking it should go to the header maybe | 13:58 |
| Saviq | mzanetti, but we'll need design feedback for it | 13:58 |
| mzanetti | yeah, sonething... as it is right now its not really good imo | 13:59 |
| mzanetti | Saviq: should I mention that in the review or do we want to land it nevertheless and improve this later? | 13:59 |
| Cimi | mzanetti, what does the 404 error on autopilot on otto means? https://code.launchpad.net/~mterry/unity8/selectively-skip-nm-test/+merge/206985 | 14:00 |
| mzanetti | Cimi: probably not publised to the public jenkins yet | 14:01 |
| Cimi | mzanetti, so I have to wait? | 14:01 |
| Saviq | mzanetti, later | 14:01 |
| mzanetti | Cimi: yeah, you can use the time to search for you pgp key and connect VPN :P | 14:01 |
| Cimi | hah | 14:02 |
| Saviq | mzanetti, with the new header, hopefully | 14:02 |
| mzanetti | ack | 14:02 |
| mzanetti | Saviq: so should we ignore the header issues I discovered too? | 14:02 |
| Saviq | mzanetti, right... probably | 14:02 |
| mzanetti | or is that different | 14:02 |
| Saviq | mzanetti, well, not the fact it's not showing while | 14:02 |
| Saviq | whole | 14:02 |
| Saviq | mzanetti, but the fact it doesn't show on start, yes | 14:02 |
| mzanetti | ok. I'll leave the comments, you guys fix what you think can/should be fixed and reply for the rest. that works for me | 14:03 |
| Saviq | didrocks, seb128 so... whatever I do, whether we call QIcon::setThemeName or not, whether I set the system theme to "suru" or not... doesn't seem to affect the icons at all, WTH :/ | 14:03 |
| Saviq | did they change the whole docs layout *again*??? | 14:04 |
| mhr3 | tsdgeos, so... what to do about the dee-qt thing? | 14:04 |
| tsdgeos | mhr3: https://code.launchpad.net/~cjwatson/dee-qt/powerpc-pie/+merge/211308 | 14:04 |
| tsdgeos | mhr3: http://lists.linaro.org/pipermail/linaro-toolchain/2014-January/003942.html | 14:05 |
| tsdgeos | mhr3: broken/buggy linker it seems | 14:05 |
| mhr3 | tsdgeos, oh wow... | 14:05 |
| didrocks | Saviq: urgh, possiblity, did you ask the sdk guys? | 14:08 |
| seb128 | Saviq, same here it seems :/ | 14:08 |
| === alan_g|lunch is now known as alan_g | ||
| Saviq | didrocks, yeah, no response :P | 14:10 |
| * didrocks sigh | 14:10 | |
| dpm | thanks pstolowski | 14:13 |
| seb128 | Saviq, didrocks: modules/Ubuntu/Components/plugin/unitythemeiconprovider.cpp: QIcon::setThemeName("ubuntu-mobile"); | 14:14 |
| seb128 | changing that in the uitk to "suru" makes sure used | 14:14 |
| didrocks | hum | 14:14 |
| Saviq | right, that wasn't there before | 14:15 |
| didrocks | that was my MP? | 14:15 |
| seb128 | Saviq, didrocks: so it's our uitk overriding the theme | 14:15 |
| didrocks | ah | 14:15 |
| didrocks | in toolkit | 14:15 |
| didrocks | grrr | 14:15 |
| didrocks | I don't care enough of it right now, enough battle to pick… | 14:15 |
| === jono is now known as Guest78266 | ||
| tsdgeos | Saviq: does https://jenkins.qa.ubuntu.com/job/unity-phablet-qmluitests-trusty/1378/consoleFull mean the build was killed because it took more than 90 min? | 14:27 |
| Saviq | tsdgeos, they get stuck sometimes | 14:27 |
| Saviq | tsdgeos, normal run is ~50 mins | 14:27 |
| tsdgeos | Saviq: get stuck *where* ? | 14:28 |
| tsdgeos | i mean tryComprae fails after 5 seconds | 14:28 |
| tsdgeos | so how can it get stuck? | 14:28 |
| Saviq | tsdgeos, something, not sure, but saw it happen in jenkins a few times now | 14:30 |
| Saviq | tsdgeos, never outside of it... | 14:30 |
| tsdgeos | if someone could activate the timestamps as i asked :/ | 14:31 |
| Cimi | can I try to fix this binding loop? | 14:31 |
| Cimi | it makes reading logs impossible | 14:31 |
| Cimi | only with grep/search | 14:31 |
| tsdgeos | i think i had some fix somewhere in seemore | 14:31 |
| tsdgeos | let me see | 14:31 |
| === dandrader is now known as dandrader|afk | ||
| mzanetti | Saviq: acceptable? https://code.launchpad.net/~mzanetti/unity8/make-single-testfunction/+merge/205035 | 14:40 |
| mhr3 | Saviq, we'll be able to get rid of the dep on unity-core soon... yey! :) | 14:41 |
| mzanetti | I would have been in desperate need of this when doing the test fixes for 5.2 :) | 14:41 |
| Saviq | mhr3, :) | 14:43 |
| Saviq | mzanetti, "merge trunk... and do something else" really? :D | 14:43 |
| mzanetti | Saviq: obvisously not intentional | 14:44 |
| mzanetti | but yeah... happened :/ | 14:44 |
| tsdgeos | Cimi: i can't find if i fixed it, so sure give it a go :) | 14:44 |
| mzanetti | and I figured it to be easy enough to not have to undo stuff | 14:44 |
| Cimi | hah ok | 14:44 |
| Saviq | mzanetti, so you didn't make it work with ninja, just made it not break ninja? ;) | 14:44 |
| mzanetti | yeah | 14:44 |
| Saviq | mzanetti, one thing I'd be wary of is case sensitivity | 14:45 |
| mzanetti | couldn't find any way to do this. | 14:45 |
| Saviq | mzanetti, not sure "Ninja" will always be cased like this | 14:45 |
| mzanetti | hmm... ok. can do that. but so far I've only seen case sensitive generator names. probably I haven't seen all of them though | 14:46 |
| mzanetti | Saviq: pushed | 14:50 |
| Saviq | mzanetti, thanks | 14:50 |
| * mzanetti goes for a run. bbiab | 14:50 | |
| Saviq | mzanetti, OTOH I'd probably go for make to be the special case, not ninja | 14:50 |
| Saviq | mzanetti, this way we don't break other generators... | 14:51 |
| mzanetti | Saviq: hmm... fair point | 14:51 |
| Saviq | tsdgeos, can you follow up on https://code.launchpad.net/~macslow/unity-notifications/dont-ignore-placeholder/+merge/206950 please | 14:51 |
| tsdgeos | yes | 14:51 |
| mzanetti | Saviq: pushed. | 14:54 |
| Saviq | mzanetti, one last thing before you go :D | 14:54 |
| mzanetti | heh | 14:54 |
| mzanetti | hit me | 14:54 |
| Saviq | mzanetti, can https://code.launchpad.net/~unity-team/unity8/new-scopes-cleanup/+merge/209642 be put into clean-to-trunk? | 14:54 |
| Saviq | mzanetti, I don't want it to be a separate commit in trunk, so if you don't have any more issues with it, I'll merge it there | 14:55 |
| mzanetti | Saviq: hmm, yeah, I guess we can now... if you promise not to push any more other changes to that branch | 14:55 |
| mzanetti | Saviq: yeah. definitely merge them together in the end | 14:55 |
| Saviq | mzanetti, "that" → cleanup? | 14:55 |
| mzanetti | Saviq: no, the to-trunk one... I'd prefer no more "lets get this small thing in too" after it's >8k | 14:56 |
| mzanetti | Saviq: so if you still want to do small changes in the trunk one, lets only merge the 2 before going to trunk | 14:56 |
| Saviq | mzanetti, ok | 14:56 |
| mzanetti | they are quite hard to review already now | 14:56 |
| * Saviq will do a silo with both and reconfigure later | 14:56 | |
| mzanetti | ok | 14:57 |
| mzanetti | ok. will be back in less than an hour | 14:57 |
| tsdgeos | Saviq: we "killed" lp:~unity-team/unity8/new-scopes, right? | 15:01 |
| Saviq | tsdgeos, yes, it's abandoned | 15:01 |
| tsdgeos | Saviq: so i should "rebase" https://code.launchpad.net/~aacid/unity8/new-scopes_do_not_request_previews_on_activate/+merge/209461 to new-scopes-clean-to-trunk? | 15:02 |
| === jasoncwarner___ is now known as jasoncwarner | ||
| Saviq | tsdgeos, yeah please | 15:03 |
| Saviq | tsdgeos, and maybe we can get a test now? | 15:03 |
| tsdgeos | will try | 15:04 |
| tsdgeos | Saviq: against new-scopes-clean-to-trunk or new-scopes-cleanup? | 15:05 |
| Saviq | tsdgeos, your call | 15:05 |
| tsdgeos | i read something about you wanting to merge them, not sure if that happened already | 15:05 |
| Saviq | tsdgeos, not yet, we'll do that just before merging into trunk | 15:06 |
| tsdgeos | ok | 15:06 |
| Saviq | tsdgeos, so yeah, on top of cleanup, probably | 15:06 |
| Saviq | tsdgeos, so that you get all the test fixes already | 15:06 |
| tsdgeos | sure | 15:08 |
| tsdgeos | mterry: https://code.launchpad.net/~mterry/unity8/hide-greeter-on-focus-request/+merge/201817 | 15:15 |
| mterry | tsdgeos, will re-merge, thanks | 15:16 |
| tsdgeos | dednick: https://code.launchpad.net/~nick-dedekind/unity8/indicators.hint-interval/+merge/202104 | 15:17 |
| dednick | tsdgeos: ta | 15:17 |
| Cimi | tsdgeos, does it require some special package? it fails https://code.launchpad.net/~aacid/unity8/new_tabbar/+merge/210453 | 15:22 |
| tsdgeos | Cimi: | 15:23 |
| tsdgeos | * Are there any related MPs required for this MP to build/function as expected? | 15:23 |
| tsdgeos | https://code.launchpad.net/~tpeeters/ubuntu-ui-toolkit/tabsModelIndex/+merge/210380 | 15:23 |
| tsdgeos | Saviq: i'm confused, you asked me to test https://code.launchpad.net/~macslow/unity-notifications/dont-ignore-placeholder/+merge/206950 but you say you already have, no? | 15:35 |
| Saviq | tsdgeos, I wanted you to follow up, since you reviewed it already | 15:38 |
| tsdgeos | ok | 15:38 |
| tsdgeos | i'll test it again anyway | 15:38 |
| tsdgeos | almost there :D | 15:38 |
| Cimi | on the phone, indicator tells me it's connected but it's a LIE | 15:39 |
| Cimi | mzanetti, can you retrigger this as well? https://code.launchpad.net/~tpeeters/ubuntu-ui-toolkit/tabsModelIndex/+merge/210380 | 15:46 |
| mzanetti | Cimi: done | 15:46 |
| Cimi | thx | 15:46 |
| Cimi | mzanetti, all were built with qt 5.0 | 15:47 |
| Cimi | I think I lost my gpg | 15:47 |
| === dandrader|afk is now known as dandrader | ||
| === gatox is now known as gatox_lunch | ||
| Saviq | mzanetti, you back? | 15:54 |
| mzanetti | Saviq: yes | 15:54 |
| Saviq | mzanetti, quick look at https://code.launchpad.net/~saviq/unity8/undisable-hud/+merge/211344 please | 15:54 |
| Saviq | mzanetti, I jumped the gun on landing that... | 15:54 |
| Saviq | Friday afternoon brainfart | 15:54 |
| mzanetti | huh? | 15:54 |
| Saviq | mzanetti, I landed disable-hud | 15:54 |
| Saviq | mzanetti, need to revert | 15:55 |
| mzanetti | oh... | 15:55 |
| mzanetti | yeah. that should have gone with the SDK people | 15:55 |
| Saviq | mzanetti, worse than that, needs test scripts updated in smoketesting and such | 15:55 |
| mzanetti | Saviq: I thought that happened already | 15:55 |
| Saviq | mzanetti, in -ci, yes, not in smoke | 15:55 |
| Saviq | because, of course, there's 10 different ways to test and scripts... | 15:56 |
| mzanetti | ah, I see | 15:56 |
| mzanetti | approved | 15:56 |
| Saviq | mzanetti, anyway, yeah, Didier approved already | 15:56 |
| mzanetti | heh, thanks didrocks... you saved me 10 pushups | 15:56 |
| didrocks | mzanetti: what's this rule? :p | 15:57 |
| didrocks | like a punishement? | 15:57 |
| mzanetti | didrocks: I always forget to top-approve and decided to do pushup so I hopefully start remembering | 15:57 |
| didrocks | ahah | 15:57 |
| Saviq | didrocks, I told him to go exponentially from the initial 5, but he's a pussy ;P | 16:02 |
| didrocks | roh, worse than I. How is it possible? | 16:02 |
| MacSlow | tsdgeos, Saviq: Do you happen to know about this QWARN "QQmlComponent: Created graphical object was not placed in the graphics scene." that's what my notification qmltests spit out ever since the move to Qt 5.2. They also segfault frequently. | 16:06 |
| MacSlow | tsdgeos, Saviq: I'm already searching for some more info, but thought perhaps you already know about it and how to avoid it. | 16:06 |
| Saviq | MacSlow, that looks like things getting torn down too quickly, sounds like some waitForRendering() could help | 16:07 |
| tsdgeos | mterry: never seen it | 16:07 |
| tsdgeos | err | 16:07 |
| tsdgeos | mterry: sorry | 16:07 |
| Saviq | MacSlow, do you have a trace for the SEGV? | 16:07 |
| tsdgeos | MacSlow: never seen it | 16:07 |
| MacSlow | Saviq, not atm | 16:08 |
| tsdgeos | Saviq: ok, approved MacSlow's notification thing | 16:09 |
| Saviq | tsdgeos, thanks | 16:11 |
| Cimi | mzanetti, I changed and activated my pgp | 16:11 |
| Cimi | mzanetti, can I haz jenkins? | 16:11 |
| Saviq | Cimi, you want the VPN access details? mzanetti can't give that to you | 16:12 |
| Saviq | Cimi, only IS can | 16:12 |
| MacSlow | Saviq, best trace of the segfault I could get... http://pastebin.ubuntu.com/7108933 missing some -dbg packages still | 16:12 |
| Cimi | Saviq, ok | 16:13 |
| mzanetti | Cimi: yeah, once you have VPN to the QA lab, sign up a user account at jenkins | 16:13 |
| mzanetti | Cimi: once you have that I can grant you permissions to retrigger jobs | 16:13 |
| Saviq | yikes it's crashing *in* waitForRendering... that's interesting | 16:13 |
| MacSlow | tsdgeos, thx for the review btw | 16:14 |
| tsdgeos | dednick: anything missing for top approval of https://code.launchpad.net/~aacid/unity8/indicator_highlight_x_position/+merge/209400 ? | 16:20 |
| dednick | tsdgeos: nope | 16:25 |
| MacSlow | dednick, any idea about the failing unity8.indicators.tests.test_indicators.IndicatorTestCase.test_indicator_exists here -> https://jenkins.qa.ubuntu.com/job/generic-mediumtests-runner-mako/5830/console ? | 16:29 |
| tsdgeos | Saviq: anything in new-scopes that need my attention or should i go back to having a look at mterry's split? | 16:30 |
| Saviq | tsdgeos, mzanetti added a comment recently, did you go through it? | 16:31 |
| dednick | MacSlow: NoSuchProcess: no process found with pid 8773 | 16:31 |
| dednick | MacSlow: looks like unity crashed | 16:31 |
| Saviq | yup | 16:31 |
| tsdgeos | Saviq: no, let me read it | 16:32 |
| Saviq | MacSlow, rerun the job, it was the Qt 5.0 crash, we're on 5.2 now | 16:33 |
| MacSlow | Saviq, already retriggered it | 16:34 |
| === gatox_lunch is now known as gatox | ||
| === jhodapp is now known as jhodapp|doc | ||
| === dandrader is now known as dandrader|lunch | ||
| mhr3 | Saviq, extend-scope-tool needs some review love | 17:21 |
| === om26er_ is now known as om26er | ||
| === dandrader|lunch is now known as dandrader | ||
| === alan_g is now known as alan_g|EOD | ||
| === jhodapp|doc is now known as jhodapp | ||
| === dandrader is now known as dandrader|afk | ||
| === dandrader|afk is now known as dandrader | ||
| === dandrader is now known as dandrader|afk | ||
| robru | anybody around to help me troubleshoot a titlebar-menu issue in latest unity7? it stopped working recently (ie, no menubar integration is happening at all for me), not sure if I broke it or if there's been a regression or what | 21:39 |
| === dandrader|afk is now known as dandrader | ||
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!