=== _salem is now known as salem_
=== salem_ is now known as _salem
=== tvoss is now known as tvoss|exercise
mzanettigood morning07:05
didrockshey mzanetti07:06
MCRHi. Will Compiz be the standard WM for 13.10 ?07:06
didrocksMCR: yeah, Unity + Compiz will still be the default until Unity Next is up to parity for a desktop experience07:07
didrocksMCR: which realistically won't be the case for 13.1007:07
MCRdidrocks, thanks for the answer - TBH I do not think anything could replace this combination anytime soon...07:08
MCRat least not without a ton of regressions07:08
didrocksMCR: we'll, we are only talking about Unity default experience07:08
didrocksnot all the compiz crazyness07:08
didrocksand that's our goal to not regress for this07:08
MCRyeah, sure07:08
hyperairanything that doesn't have ezoom doesn't deserve to be called a replacement!07:08
didrockshyperair: ezoom was never supported in Unity btw07:09
didrocksso we can only do better there :)07:09
hyperairdidrocks: i know, which was why i brought it up.07:09
hyperairdidrocks: but compiz contains ezoom, and unity+compiz works with ezoom well enough.07:09
didrockshyperair: "well enough" != enough IMHO07:10
MCRdidrocks, it works perfectly ;)07:10
hyperairdidrocks: where does ezoom fall short?07:10
didrocksif we really want it to be effective, you need to be able to zoom in the UI07:10
didrocksMCR: no it doesn't07:10
didrocksyou can't zoom the shell07:10
hyperairzoom in the UI?07:10
didrocksyou can't zoom the dash07:10
hyperairoh that07:10
didrocksso not "well" supported07:10
hyperaircome to think of it, why did unity need to be a window manager? couldn't it just have been a gnome-panel thing?07:11
hyperairi mean a gnome-panel drop-in replacement07:11
didrockshyperair: because you need deeper integration to control windows states07:11
hyperairdidrocks: what window states does unity control?07:11
didrocksthat's why GS and Unity are in the wm process07:11
hyperairyeah but what window states does unity control, exactly?07:12
didrockshyperair: if maximized or not, where the windows are, which workspace, how many of them07:12
hyperairmost of that is outside of the unity plugin07:12
didrockshyperair: no, it's not, look at the unity code07:12
didrocksbecause unity needs to know about it07:12
hyperaircompiz already had logic for which workspace a window is in07:12
hyperairso it's more of querying07:12
hyperairrather than controlling?07:13
didrockswith some control to07:13
didrocks"move to ws X"07:13
didrocksor move this windows from ws X to Y07:13
hyperairwasn't there some nice compiz dbus api?07:13
MCRdidrocks: well, Compiz needed until 2013 to learn how to unsnap a horizontally maximized window - in fact Compiz in Raring still doesn't know about it...07:13
didrockshyperair: "nice" isn't the definitive word07:13
didrockshyperair: the API is never stable07:13
hyperairdidrocks: heh well, it could have been improved?07:13
didrockshyperair: and always poking dbus is not an option :)07:14
hyperairand stabilized?07:14
hyperairah yes, true.07:14
didrockshyperair: people tried, in metacity, mutter, compiz07:14
MCRTry this: right click on maximize button -> now drag the titlebar07:14
didrockshyperair: everybody failed07:14
didrocksthere is maybe a reason :)07:14
hyperairmaybe dbus is just slow.07:14
didrocksMCR: I'm not interested in knowing where compiz fails, I know there is a lot of cases ;)07:14
hyperairbut for all the caching of pixbufs and what other fancy stuff unity does that causes its memory to swell like that...07:15
hyperairi'm not seeing the speed improvements that should have come with it07:15
hyperairgnome shells07:15
hyperairGS's "dash" is much faster than unity's07:15
didrockshyperair: it does a lot less07:15
didrockslike not showing as many icons, no previews…07:15
didrocksyou can't really compare the dash :)07:15
hyperairthere's this very annoying bug where i key in stuff, it types out my query *slowly*, and then it spins for a while, and stops spinning without refreshing.07:15
hyperairthen i have to press another key, wait for the spinner, press another key, and then it'll work.07:16
didrockshyperair: did you report a bug about it?07:16
hyperairdidrocks: yes, well, super+a should be as fast as GS.07:16
hyperairdidrocks: hmm, i don't recall07:16
hyperairmaybe i didn't.07:16
smspillazhyperair: RGBA GLX is slow07:16
smspillazthat's why it had to be a shell plugin07:16
hyperairsmspillaz: GS also uses RGBA, does it not?07:16
MCRsmspillaz, hi.07:17
smspillazhyperair: the shell runs in process07:17
hyperairsmspillaz: yes?07:17
MCRsmspillaz - great you have set up a 0.9.10 PPA - but it is empty :(07:17
hyperairsmspillaz: as does unity.07:17
hyperairsmspillaz: so why is the dash so goddamned slow?07:17
hyperairi type super+a and it takes forever to search for whatever i type in07:17
MCRhyperair, it is already fast here07:17
hyperairMCR: you running on an SSD?07:18
hyperairsee, therein lies the problem.07:18
MCRbut it got faster and faster each version07:18
smspillazhyperair: dash blurs probably07:18
hyperaireveryone uses a damn SSD and says "oh my god, it's so blazing fast"07:18
hyperairsmspillaz: disabled.07:18
MCRthe blur is heavy on the hardware07:18
didrockshyperair: you are talking about the dash search07:18
didrocksnot the UI07:18
hyperairdidrocks: yes.07:18
didrocksand this is not in the process07:18
didrocksand goes through dbus btw :p07:18
hyperairdidrocks: the blurs exacerbated the issue by having a horribly slow UI response07:18
didrocksso what you are asking, putting the shell not in the wm is something totally separate07:19
smspillazhyperair: *shrug* Trevhino and I were looking into something the other day as to why the dash was slowing down. We think its a set of draw calls somewhere that's quite heavy, but its going to require a bit more time to consider07:19
smspillazhyperair: we've figured out that the slow dash isn't usually CPU bound07:19
hyperairsmspillaz: oh yay, finally.07:19
hyperairlike how MCR and the rest of the SSD users are seeing blazing fast dash searches07:19
=== jono is now known as Guest50668
MCRsmspillaz, I fixed all issues in my MPs btw (except screenshot) and am waiting for approvals...07:21
MCRsmspillaz, also I would like to apply as Compiz maintainer...07:21
smspillazMCR: I think we can arrange that07:21
MCRgreat, number 1 + 2 ? ;)07:22
smspillazMCR: yeah07:22
smspillazI'll get on to both ASAP, pretty busy this weekend07:22
MCRCool, thx07:22
smspillazhyperair: if you know of any good OpenGL profiling tools let me know. I haven't reallly been able to find any other than gDEbugger and its a pain to get that to work07:23
hyperairsmspillaz: i think there was a mention of something in phoronix that valve used..07:23
smspillazhyperair: it has a high price tag07:23
smspillazhyperair: if you go to their website it says "contact us about pricing details"07:24
smspillazthat's code for "this is several thousand per seat and we only negotiate enterprise contracts"07:24
MCRsmspillaz, do you have an ETA for the new PPA ? - I am eager to test it (altough most of 0.9.10 runs here already, ofc)07:24
smspillazMCR: PPA of what ?07:25
smspillazcompiz trunk?07:25
MCRhttps://launchpad.net/~smspillaz/+archive/compiz-dev yes07:25
smspillaz5 minutes07:25
hyperairsmspillaz: oh dear, that sucks.07:25
hyperairsmspillaz: i suppose you've looked through all of these: https://www.opengl.org/wiki/Debugging_Tools ?07:28
smspillazI don't recall coming across that07:29
MCRsmspillaz - great, the availability of this PPA should be announced also to get more testers, but I am sure you've planned that already ;)07:29
smspillazdunno the best place to announce it really07:29
MCRPlanet Compiz ?07:30
hyperairthe ayatana mailing list?07:33
hyperairunity-design now, iirc07:33
MCRIt could also be added to all the bugs that are fixed in 0.9.10, so people could immediately test those and report any issues07:33
MCRI could do that07:34
dufluMCR: zoom only works on things because those things are X windows that Compiz can stretch. That's not true for Unity elements :(07:54
MCRhi duflu :)07:55
MCRsmspillaz, compiz in the PPA is still named 0.9.9 - not sure if that is correct, should be 0.9.10, no ?07:56
dufluOn the other hand, you could just figure out the tranform matrix for the zoom (including translation) and pass it through to Unity/Nux. It could work07:56
MCRduflu, I know that this UI zoom was requested and would make much sense for visually impaired people, but it is not my main priority at the moment07:57
MCRfirst all window managing stuff needs to be rock solid imho07:57
MCRno more windows jumping around workspaces or maximizing on wrong workspaces for example07:57
MCRwe are (finally) almost there...08:01
Saviqtsdgeos, the plot thickens08:04
Saviqtsdgeos, https://bugreports.qt-project.org/browse/QTBUG-3073008:04
tsdgeosso there's probably an internal object somewhere that is sometimes getting magically converted to a qobject and sometimes not08:05
tsdgeosit seems08:06
Saviqtsdgeos, what's more, the type of the singleton gets registered automagically, and in the global scope08:08
Saviqtsdgeos, that's why I had to rename the singleton08:08
didrockshey sil2100! do you feel better?08:08
tsdgeosSaviq: brrr08:10
tsdgeosSaviq: i have http://paste.kde.org/~tsdgeos/728060/ working now :-)08:10
Saviqtsdgeos, yay08:10
tsdgeosjust need to get it merged :D08:10
tsdgeosit's two lines of code08:10
tsdgeosbut ...08:10
sil2100didrocks: hi! I overslept by accident ;p08:10
didrockssil2100: seems you needed it then! :-)08:11
seb128sil2100, hey, happy friday ! do you feel better than the other day ?08:12
sil2100didrocks: I got an e-mail about some apparent lockscreen errors from veebers in the 100scopes branch, might be that the fix I made is not enough - will look into that now08:12
dandradermzanetti, have you seen this before? https://jenkins.qa.ubuntu.com/job/unity-phablet-ci/637/testReport/junit/%28root%29/qmltestrunner/FilterGrids__test_clicked_signal/08:13
sil2100seb128: hi! Happy Friday indeed - I feel mostly much better now08:13
didrockssil2100: hum, I wonder if we shouldn't finish the hud stuff first08:13
didrockssil2100: so, what would I tell, regarding your email is to disable the failing test for armfh right now08:13
sil2100didrocks: that as well - you got me e-mail?08:13
didrocksyeah ;)08:13
sil2100Oh, ok08:13
didrockssil2100: and powerpc is FTBFS, so maybe the same08:13
didrockssil2100: just write it in the spreadsheet as well :)08:13
didrockssil2100: then, I think we should try to rerun stack per stack08:14
didrocksin head08:14
didrocksand see if autopilot behaves good :)08:14
mzanettidandrader: yes. let me check08:14
didrocksmaking sense?08:14
sil2100Ok, will do that - I just didn't want to disable the failing test because it was just failing because of the loading error08:14
sil2100But let's do that not to waste time08:14
mzanettidandrader: or better: let me Deebug it :D08:15
mzanettimorning nic08:16
didrockssil2100: yeah, we have tracks of what's wrong, let's try to get all the stacks running first :)08:16
mzanettidandrader: really weird. can't reproduce this locally08:20
Saviqmzanetti, there's a bunch of qmluitests that failed yesterday with that for every ui test08:34
Saviqmzanetti, looks like GL accel broke on the VMs or something08:34
mzanettiSaviq: that's already fixed08:35
mzanettiSaviq: the current red ones are merge conflicts08:35
Saviqmzanetti, I meant the shader issue dandrader mentioned08:35
mzanettiSaviq: and I'm currently Deebugging the yellow ones08:35
Saviqmzanetti, great, thanks08:36
SaviqI'm not touching anything then08:36
mzanettiSaviq: I'm not really sure how my commit passed CI tho and then broke all following ci runs...08:37
Saviqmzanetti, I'm not entirely sure it's related08:38
Saviqmzanetti, as it's just your filtergrid that fails08:38
Saviqmzanetti, but the rest report the shader errors just as well08:38
dandradermzanetti, but what about the missing Dee module and "tst_FilterGrids.qml:91: TypeError: Cannot set property 'model' of null"08:38
mzanettidandrader: I hope it'll be fixed by this https://code.launchpad.net/~mzanetti/unity/phablet-deebug/+merge/15977308:38
tsdgeosSaviq: https://codereview.qt-project.org/#change,5424408:38
mzanettiSaviq: but that filtergrids test must have been passing at some point08:38
tsdgeosSaviq: meanwhile yesterday i also fixed a bug dpm reported to me (no clue why) that is stalled because i haven't fixed it for windows and mac :-/ https://codereview.qt-project.org/#change,5421408:39
mzanettitsdgeos: I think for QtQuick its RGBA, not ARGB08:40
Saviqtsdgeos, nice one, thanks08:40
Saviqmzanetti, it's ARGB08:40
sil2100didrocks: https://code.launchpad.net/~sil2100/hud/disable_source_test_and_tweak_timeouts/+merge/15977608:41
tsdgeosmzanetti: nope, it's not08:41
dandraderSaviq, I didn't get the InputFilterArea issue. So the removal of the fakes from the shell code made that thing stop working (when dealing with real apps)?08:41
didrockssil2100: waiting for the diff :)08:41
tsdgeosmzanetti: http://paste.kde.org/~tsdgeos/728096/ that's the code they use08:42
didrockssil2100: what about powerpc FTBFS?08:42
Saviqdandrader, yes, because it relied on the "can I import Ubuntu.Application" bool08:42
mzanettitsdgeos: its Qt.rgba()... thats why I thought so08:42
didrockssil2100: it's the timeout being bigger?08:42
Saviqdandrader, which you removed (and rightfully so, we just need to get rid of that wrapper)08:42
tsdgeosmzanetti: yep08:42
sil2100didrocks: it was a ekhm, timing issue... there's that one problem we have when tearing down the HUD connection08:42
dandraderSaviq, ah, got it08:42
Saviqdandrader, and implement a noop filter in the fake plugin08:43
Saviqdandrader, and again, DON'T YOU SLEEP!?08:43
dandraderSaviq, yes08:43
sil2100It's really irritating, since there's not much we can do than wait, we might want to redesign somehow in the future maybe...08:43
dandraderSaviq, climbing this aftenoon :)08:43
didrockssil2100: hum, is hudtest(test-source test-source.c test-source.xml) only with platform api?08:44
dandraderSaviq, I would rather say the opposite. If I log on at 5am. you're online. At 5pm, you're online as well ;)08:44
Saviqdandrader, details08:44
didrockssil2100: commented08:46
sil2100didrocks: good catch, no need to disable it on non-platform-api systems08:49
didrockssil2100: ping me back :)08:49
sil2100didrocks: pushed and build-tested... but the first push went to some different branch, grrr08:52
* sil2100 hates when the default push branch is some old branch08:53
Saviqtsdgeos, and you guys complain about whitespace checks, "possible spelling error"? lol!08:53
didrockssil2100: approved :)08:54
didrockssil2100: ok, so now some stacks in head are running08:54
didrockssil2100: so, if you look, some have issues to fix, even before autopilot08:55
didrockssil2100: see he yellow indicator stack? it seems something is out of sync08:55
didrockssil2100: there is also qtvideo-node in the head/media stack which needs fixing08:56
didrocks(don't look at the mediaplayer-app, I fixed it)08:56
sil2100Looking looking08:56
didrockssil2100: phone and apps stacks are running autopilot, we'll see the result soon :)08:56
sil2100Oh, since I didn't see those blinking, so thought it's over already ;p But I'll check the indicator stack08:57
Saviqis anyone else's PA hogging the CPU constantly since yesterday?08:57
didrockssil2100: yeah, it's a "view", not a realtime status08:57
didrockssil2100: it's showing "latest finished state" :/08:57
dandraderthe worst thing about javascript is that it fails silently08:57
Saviqdandrader, yeah, undefined == false is the worst08:58
Saviqmzanetti, http://s-jenkins:8080/job/unity-phablet-qmluitests/448/console seems to hang08:59
tsdgeosSaviq: at least that spelling stuff is immediate, doesn't take 30 min like our whitespace check08:59
Saviqtsdgeos, if you had tested locally ;)08:59
tsdgeosyes, yes, i know, execute them locally :D08:59
mzanettiSaviq: *grrr*09:00
tsdgeosSaviq: well, i saw your whitespace error too ;-)09:00
mzanettiSaviq: I just requested 2 additional VM's09:00
Saviqtsdgeos, indeed09:00
mzanettiSaviq: before they go online I'll use one of them to debug this more09:00
Saviqtsdgeos, but I'm not complaining, am I :P09:00
Saviqmzanetti, k09:00
tsdgeosSaviq: not only that, now i have to wrap the commit message at 72 cols09:06
didrockssil2100: ok, it's merged, I'm relaunching the hud stack with only rebuilding hud FYI09:06
tsdgeoslet's pretend we live in 197509:07
tsdgeosSaviq: and they're not going to accept it for 5.1 so that sucks09:08
sil2100didrocks: ok09:09
Saviqtsdgeos, I gathered as much09:09
Saviqtsdgeos, but it's fine, we have good processes for distropatching, I think09:09
sil2100didrocks: btw. the indicator stack... it seems all the jenkins stuff is month-old09:10
Saviqtsdgeos, hmm, shouldn't we also support #ARGB?09:11
didrockssil2100: did you look at the branches in the config?09:11
didrockssil2100: and did we got new commits?09:11
tsdgeosSaviq: don't see the need to be honest09:11
tsdgeosdoes anyone ever use that format?09:11
Saviqtsdgeos, #RGB is supported, so09:12
tsdgeosi know09:12
* sil2100 is still noobish here09:12
tsdgeosmy guess is that this is just because they need to support xpm09:12
sil2100didrocks: yes, I checked and there were commits to the indicator.cfg09:12
sil2100didrocks: but the last run of for instance cu2d-indicators-head-1.1prepare-indicator-datetime is 22 days old09:12
Saviqtsdgeos, anyway, QML doesn't support #ARGB09:12
Saviqtsdgeos, so that's that09:13
sil2100didrocks: and that's why indicators is yellow, since it was executed when indicator-datetime was still using the raring 13.04 branch09:13
didrockssil2100: indeed, there is no time assigned09:14
didrockssil2100: I think nobody redeployed the stack09:14
didrockscyphermox: seems that you never redeployed the head/indicators.cfg stack. Doing it now ^09:14
* sil2100 wanted to use the 'redeploy' word but wasn't sure if it's the right one09:15
didrocksnice catch sil2100 ;)09:15
didrocksheh :)09:15
didrockssil2100: basically I looked at the head configuration09:15
didrockssil2100: and didn't see the timer09:15
Saviqtsdgeos, typo09:16
Saviqtsdgeos, not ARBG but ARGB09:16
sil2100I think it should indeed show the 'last run' timer in the main 'Head' view09:16
tsdgeosSaviq: where?09:16
Saviqtsdgeos, commented inline, qcolor.cpp09:16
didrockssil2100: yeah, that's the issue to play with jenkins to control the whole chain…09:16
Saviqactually no, qcolor.cpp09:17
sil2100didrocks: you fixed up the media stack problems already, yes?09:19
didrockssil2100: not the qtvideo-node, as said09:21
sil2100What also confuses me in jenkins is that there is a Last Success column and Last Failure column, but sometimes I would simply need a 'Last Run' one - here I have to check both columns to get the definite answer on when the build was executed09:21
didrockssil2100: mind looking at it?09:21
sil2100didrocks: aye!09:21
didrockssil2100: agreed it's confusing09:21
Saviqmzanetti, did you see the jobs seem not published anymore? following the links in jenkins messages gives out 40409:35
mzanettiSaviq: no... didn't see that yet... still trying to fix the broken test09:36
Saviqmzanetti, k09:36
Saviqmzanetti, do you know who do I ping in absence of mmrazik about that?09:36
mzanettiSaviq: I guess me09:36
mzanettiSaviq: in the afternoon fginther or sergiusens too09:37
Saviqmzanetti, k09:37
mzanettiSaviq: I'll check it out as soon as the test is fixed09:37
Saviqmzanetti, thanks09:37
Cimilaunchpad is super slow here, only to me?09:42
mzanettiCimi: works fine here09:43
mzanettiCimi: maybe you're super fast today :)09:44
Cimioh definitely not :)09:44
CimiI mean via browser, pages take a minute09:44
CimiI'm in Italy btw for the weekend, arrived here late night09:44
Cimimaybe we don't like launchpad xD09:45
Cimidednick, do you have any update on the categories branch you didn't push?09:45
dednickCimi: nearly there09:46
mzanettiCimi: for me internet is in general very slow in italy... Might be related to the mountain area I come from09:46
Cimimzanetti, all the rest of internet is fine09:46
Saviqyeah, internets don't deal with mountain well ;)09:46
Cimijust launchpad got slo09:46
Saviqthey get tired09:47
dandraderSaviq, have you mentioned pulseaudio eating CPU all the time?09:48
Saviqdandrader, yes09:48
dandraderI'm getting the same now09:48
seb128dandrader, Saviq: what version of the package do you have?09:48
Cimiok chrome is slow, safari is fast09:48
seb128dandrader, Saviq: https://launchpad.net/ubuntu/+source/pulseaudio/1:3.0-0ubuntu6 is supposed to fix it09:49
Saviqseb128, 1:3.0-0ubuntu509:49
CimiDNS prefetch?09:49
seb128Saviq, upgrade09:49
Saviqseb128, trying09:49
dandraderseb128, 1:3.0-0ubuntu09:49
seb128dandrader, same, upgrade09:49
Cimimaybe I'm running uk dns09:49
sil2100didrocks: the problem with qtvideo-node is that the source package name is different from the branch name...09:50
sil2100didrocks: since qtvideonode-plugin is the src package, while the branch is lp:qtvideo-node09:50
sil2100At least hm09:51
Saviqseb128, hmm no upgrade available yet09:51
dandraderSaviq, https://code.launchpad.net/~dandrader/unity/phablet_remove_fakes_from_qml/+merge/158370 should be good to go know, although jenkins is having some difficulties09:51
Saviqdandrader, yeah, mzanetti is on it09:51
seb128Saviq, change mirror?09:51
seb128Saviq, or use directly archive.ubuntu.com09:51
Saviqseb128, archive.ubuntu.com09:51
Saviqseb128, yeah, I'm using archive directly09:52
sil2100didrocks: maybe we should change lp:qtvideo-node to lp:qtvideonode-plugin ?09:52
seb128Saviq, hum, 0ubuntu6 was uploaded yesterday afternoon, it should be available09:52
seb128Saviq, sudo apt-get update?09:53
dandraderSaviq, odd I got the pulseaudio upgrade09:53
* Saviq disables apt-cacher09:53
didrockssil2100: what's the project name?09:53
* dandrader reboots to try it out the new stuff (including new kernel)09:54
sil2100https://launchpad.net/qtvideo-node <- on LP that's like this09:54
didrockssil2100: I would rather go the other around, change the source package name to map to the project name09:54
didrockssil2100: as it's what was decided for opening the project09:54
didrockssil2100: also, can you please add some bootstrap commit message?09:54
dandraderwell, so far so good. seb128 thanks for the tip!09:56
seb128dandrader, great, thanks for confirming the fix ;-)09:56
Saviqseb128, yup, helped, thanks!10:01
seb128Saviq, yw!10:01
* Saviq wonders why apt-cacher-ng broke10:01
sil2100didrocks: in case of changing the source package name... should I change all previous changelog entries, or just the most recent? ;)10:02
sil2100Will do!10:02
didrockssil2100: most recent is fine10:02
=== tvoss|exercise is now known as tvoss
mzanettiSaviq: so, what exactly is not published?10:10
Saviqmzanetti, actually, it seems to have just taken time10:11
Saviqmzanetti, yeah, it's fine10:11
smspillazMCR: which ones needed review ?10:19
smspillazMCR: I don't see anything that's not marked resubmit, and the only other one has comments by me but no follow up10:19
MCRsmspillaz, https://code.launchpad.net/~compiz-team/compiz/0.9.10/+activereviews10:20
MCRSam, all that have been resubmitted (Resubmit: 1)10:21
smspillazMCR: hmm, I thought when you marked something "resubmit" it meant you were going to resubmit it10:22
smspillazin any case I can have another look10:22
MCRand if you have a solution to fix the screenshot thingy with fbos, you could take over the screenshot branch10:22
smspillazMCR: we'll do it separately10:22
MCRI have set the screenshot MP to WIP10:23
smspillazMCR: hm? I meant we'll do the fbo thing in a separate mp10:24
MCRyeah, I understood10:24
MCRbut I have not yet applied your suggestions10:24
MCRin my MP implementing the overlay color selector for screenshot10:25
smspillazMCR: oh, actually quesiton about the cube gears proposal10:25
MCRsure - go ahead10:25
smspillazdo we call glDepthMask (GL_FALSE); anywhere ?10:25
smspillaz(can't see it from the diff10:25
MCRI do not think so10:26
smspillazMCR: oh, actually10:26
smspillazMCR: never mind10:26
smspillazMCR: I meant to ask - we're calling glDisable (GL_DEPTH_TEST);10:26
smspillazaren't we?10:26
Saviqdednick, can you take a look at https://bugs.launchpad.net/touch-preview-images/+bug/1170465 please10:27
ubot5Launchpad bug 1170465 in touch-preview-images "[network-indicator] wrong design when asking for the AP password" [Undecided,Confirmed]10:27
Saviqdednick, it's gonna be in indicators-client network plugin, I expect10:28
dednickSaviq: sure10:28
smspillazMCR: BTW, removing the <precision> tags in https://code.launchpad.net/~mc-return/compiz/compiz.merge-fix1167933.2-KEY_MOVE_INC-is-hardcoded/+merge/158767 was largely unrelated to the point of the original proposal10:29
smspillaztry to avoid doing stuff like that, its confusing :)10:29
smspillazits fine, just that it didn't belong there10:29
MCRI did find out it does not work for ints, just floats10:29
MCRso I removed it10:29
MCRbut you are right10:29
smspillazMCR: that's fine, just do it separtely10:29
smspillaz(in future)10:29
MCRshould have been separated - yes10:30
MCRmy main problem ;)10:30
MCRsmspillaz - yes I did disable  GL_DEPTH_TEST in gears10:32
MCRline 28510:32
MCRsry - it was already there10:32
Saviqtsdgeos, can you comment on https://bugs.launchpad.net/touch-preview-images/+bug/1170550 please10:32
ubot5Launchpad bug 1170550 in touch-preview-images "Searching for music restarts the shell on manta" [Critical,Confirmed]10:33
smspillazMCR: I've commented on the three marked resubmit10:33
smspillazacked two10:33
didrockssil2100: any issue? That should have taken 5 minutes at max. I can do it otherwise ;)10:34
Saviqtsdgeos, actually I can't seem to crash it here on manta10:34
MCRsmspillaz, I see 4 acked ?10:34
tsdgeoswell, it's all a matter of luck10:34
tsdgeosit's about the order stuff gets constructed/destroyed10:34
tsdgeosthat may leave some things dangling10:35
smspillazMCR: ah, sorry about that. my brain is not working today10:35
smspillazI reviewed *five* and acked *four*10:35
MCRsmspillaz, anyway thanks - no problem ;)10:35
Saviqdandrader, could you tackle https://bugs.launchpad.net/touch-preview-images/+bug/117049510:36
ubot5Launchpad bug 1170495 in touch-preview-images "Search results on apps not obvious" [Undecided,New]10:36
Saviqdandrader, my plan would be: create a mockapp.lens in demo-assets10:37
Saviqdandrader, and use the same approach we have in DashHome to assign the relevant category results10:37
Saviqdandrader, the mockapp lens would have to be invisible, though10:38
tsdgeosSaviq: commented in https://code.launchpad.net/~saviq/unity/phablet.unity-test-module/+merge/15941010:38
Saviqdandrader, so frequent and available apps would come from the mock lens, and installed ones from the real lens10:39
sil2100didrocks: uuh, sorry, pushed it and didn't request a merge ;p Damn10:39
didrockssil2100: ah, so I guess you are looking at the autopilot results now? :)10:39
dandraderSaviq, after I'm done with this Jenkins autopilot error about missing InputfilterArea10:40
mzanettidandrader: https://code.launchpad.net/~mzanetti/unity/phablet-deebug/+merge/15977310:41
mzanettidandrader: this should fix the problem you're having10:42
tsdgeosSaviq: adding the comment on https://bugs.launchpad.net/touch-preview-images/+bug/1170550 too10:42
Saviqdandrader, k10:42
ubot5Launchpad bug 1170550 in touch-preview-images "Searching for music restarts the shell on manta" [Critical,Confirmed]10:42
MCRsmspillaz, needs information, please: https://code.launchpad.net/~mc-return/compiz/compiz.merge-composite-cleanup/+merge/15875710:42
MCRsmspillaz, foreach is now in brackets, just if is not...10:43
sil2100didrocks: there seemed to be an ssh connection refused for nvidia during the autopilot job, did utah fail or something?10:43
sil2100During the apps AP10:43
mhr3sil2100, is it possible that your disable-lock-screen fix doesn't work 100%?10:44
dandradermzanetti, that's one problem. but there's also this https://jenkins.qa.ubuntu.com/job/generic-mediumtests-runner/1168/testReport/junit/qml_phone_shell.tests.testconfigurations/TestNexus10/test_hide_hud_dragging/10:44
sil2100mhr3: there is a possibility, yes - Chris saw some issues yesterday10:44
mhr3sil2100, cause we merged trunk into the 100scopes branch and yesterday the ap tests run and half of them failed cause the screen is locked10:45
sil2100It might be re-enabled too early10:45
sil2100I will be looking into that later today10:45
mhr3sil2100, thx10:45
tsdgeosSaviq: shall we do a new release? last one is from a week ago10:45
mzanettidandrader: but this seems to be caused by your branch, no?10:45
Saviqtsdgeos, yeah we will, soon10:46
Saviqtsdgeos, we might want to wait for the fixes to the bugs from http://iso.qa.ubuntu.com/qatracker/milestones/268/builds10:46
dandradermzanetti, yes. that's what I'm working on10:46
didrockssil2100: not sure, did you ask on #qa?10:47
didrocksmhr3: the screen locker is a side effect anyway10:48
tsdgeosSaviq: ok, if we have them in the pipeline sure :-)10:48
didrocksmhr3: it's because some tests stuck, and nothing happened for 10 minutes10:48
mzanettidandrader: you need help on that?10:48
didrocksmhr3: you do have sil2100's fix for the HUD tests, right?10:48
dandradermzanetti, I think I got it. seems like a missing install(...) entry in CMakeLists.txt. but thanls10:48
sil2100Right, I think we'll need to port that to 100scopes10:48
mzanettiSaviq: should I then try to fix one of those bugs?10:50
Saviqmzanetti, look at https://bugs.launchpad.net/touch-preview-images/+bug/115750810:50
ubot5Launchpad bug 1157508 in touch-preview-images "Keyboard not showing up for wifi password entry on Nexus 10" [High,Incomplete]10:50
mzanettiSaviq: ack10:50
Saviqmzanetti, you have manta, right?10:50
mzanettiN10, yes10:51
mzanettidandrader: that branch passes in jenkins now. it just posted a needs fixing because the job hung. I retriggered it manually and it works now. so once you approve qmluitests should strat working again10:53
tsdgeosdednick: what's test_unity_object for in https://code.launchpad.net/~unity-team/unity/phablet.fake-unity-plugin/+merge/158865 ?10:53
dednicktsdgeos: eh. it was just a test object when i first created the plugin10:54
dednicktsdgeos: i'll remove10:54
dandradermzanetti, once I approve what?10:54
mzanettidandrader: https://code.launchpad.net/~mzanetti/unity/phablet-deebug/+merge/15977310:54
sil2100mhr3, didrocks: actually, my HUD fix won't help here, since as Chris mentioned a different test is breaking everything - it seems Dash crashed really strangely in unity.tests.test_shopping_lens.ShoppingScopeTests.test_preview_works_with_shopping_scope10:56
didrocksah, interesting…10:56
dandradermzanetti, why tests/qmltests/plugins/Dee/CMakeLists.txt isn't needed anymore?10:57
sil2100And some *strange* introspection errors for intel as well, huh?10:57
dandradermzanetti, because Dash test now points directly to  ${CMAKE_CURRENT_SOURCE_DIR}/plugins ?10:58
mzanettidandrader: yes10:58
dandradermzanetti, and tests do not get installed anyway, right?10:59
mzanettidandrader: exactly10:59
tsdgeosdednick: i'm also a bit uneasy that the Fake Lens api is a bit different from the Real Lens api10:59
tsdgeoshow does that work?10:59
dandradermzanetti, now we only need jenkins to agree with my verdict11:00
dednicktsdgeos: only implements the API it's needs for the tests that are written.11:00
mzanettidandrader: yeah... I'll keep an eye on it. and as soon we get the new VM's I'll use one to debug the hanging qmluitests further11:01
tsdgeosdednick: sure, i'd undersrand it being a subset, but it has new api too11:01
tsdgeosQ_PROPERTY(QString name READ name WRITE setName NOTIFY nameChanged)11:01
tsdgeosis in the fake one11:01
tsdgeosbut not in the real one11:01
tsdgeosah wait11:02
dednicktsdgeos: yes it is.11:02
dednickalthough no write11:02
tsdgeosit is there11:02
tsdgeossearch fail :D11:02
mhr3didrocks, sil2100, but yes we have all sil2100's fixes11:03
tsdgeosso there's a bit of new api to inject the fake stuff11:03
tsdgeosthat's fine11:03
mhr3the only thing we're missing from trunk is manuel's fix and changelog bumps11:03
tsdgeosdednick: ok, works for me too once you remove that fake object11:03
MCRsmspillaz, done ;)11:04
dednicktsdgeos: done11:05
Saviqtsdgeos, I'll assign you to the dash crash bug11:14
sil2100mhr3, didrocks: so, from what I preliminarily see in the failing tests - my screen-lock disable fix is working to some extent, but still there's a gap that casues the screen lock to happen11:21
didrockssil2100: I think the interesting part is rather: what tests was stuck for 10 minutes to have the screen lock enabled?11:21
sil2100Since for as long as a test is going, the screen lock is not able to appear - one of the broken tests was hanged for 12 minutes and the screen lock did not happen, but what is interesting what happened then - then the test started to do it's clean up, it cleaned up all the environment preparing to finish (also, re-enabled the screen lock again) and AGAIN hanged up afterwards for another 12 minutes!11:23
didrockssil2100: interesting, do you have any idea if it's always the same tests, or why autopilot is doing that hangup?11:24
sil2100didrocks: yes, I'm trying to figure that out as well, since it looked like Unity started breaking down, the dash being unresponsive and glitchy11:24
sil2100There were some graphical glitches seen on the movie as well - not sure now what happened exactly, I only have some traces in the logs11:25
didrockssil2100: but you are seeing the gap on logs, I guess11:26
didrockssil2100: yeah, it's reproducing some times, I don't know at all how we can get the needed infos11:27
sil2100didrocks: still looking for the cause - me and Chris noticed that there's a query error appearing every 30 seconds during the first 12 minute hang up11:28
sil2100Another strange thing is that this is the nvidia failure - intel fails differently11:28
* sil2100 is confused, needs coffee11:28
didrocksoh interesting11:29
nic-doffaySaviq, mzanetti more QML q's if you have the time.11:30
Saviqnic-doffay, hit me11:30
nic-doffayI'd like to do a Sequential animation with a state.11:30
nic-doffaySaviq, ^11:30
Saviqnic-doffay, you mean when changing between states, you want a SequentialAnimation in a transition11:31
Saviqnic-doffay, something like http://pastebin.ubuntu.com/5721295/ ?11:33
nic-doffaySaviq, yeah that looks spot on, thanks. I basically need to perform a transition between states and a transformation at the same time.11:37
nic-doffaySaviq, what's the best method to store a state in a variable to access at a later stage?11:49
Saviqnic-doffay, usually a property somewhere11:51
Saviqnic-doffay, depends on the use caes11:51
mzanettiSaviq: there was a second bug regarding the password field misaligned, right?11:51
Saviqmzanetti, yes11:51
nic-doffaySaviq, just store it as a string?11:51
Saviqnic-doffay, what's the use case?11:51
Saviqmzanetti, launchpad.net/bugs/117046511:52
mzanettiSaviq: ack. thats related... my MP will fix that too11:52
Saviqdednick, ^11:52
Saviqmzanetti, when I didn't get OSK on password entry, it wouldn't come up anywhere for the shell, is that what you're experiencing?11:53
mzanettiSaviq: no... its because the layout is messed up, so tapping the textfield focuses also something else which prevents the osk from popping up11:54
Saviqmzanetti, so there's two issues I'd say11:54
Saviqmzanetti, make sure you get OSK in the messaging menu and on search, for example11:54
nic-doffaySaviq, every dot has an initial variable state it needs to transition back to after the first transition. This state is set externally by the data which will be received from the backend.11:54
mzanettiSaviq: ok11:54
mzanettiSaviq: works fine for me11:55
Saviqmzanetti, it's not 100% reproducible11:55
Saviqmzanetti, try after first flash, for example11:55
Saviqmzanetti, or after rebooting11:56
mzanettiSaviq: the one on the network password was 100% reproduceable11:56
mzanettiSaviq: and right now I freshly booted and its fine...11:56
Saviqmzanetti, ok, let's see your fix, we'll try11:56
Saviqnic-doffay, then that initial state should be stored in a property on the dot11:56
Saviqnic-doffay, and the transition should happen on a different property11:56
nic-doffaySaviq, brilliant, thanks.11:57
Saviqnic-doffay, so the value stored initially is never changed, so no need to think about it11:57
mzanettiSaviq: https://code.launchpad.net/~mzanetti/indicators-client/fix-network-password-layout/+merge/15980511:58
Saviqmzanetti, why not implicitHeight?12:00
mzanettiSaviq: doesn't work12:00
Saviqmzanetti, ok, checking12:00
mzanettiSaviq: maybe something in the surrounding Menu {} element that only works with real height12:00
sil2100didrocks: argh, libappindicator failed on powerpc, making the indicator stack red on build (FTBFS because of -Werror=deprecated-declarations)12:00
Saviqmzanetti, but I do think you fixed dednick's bug and not your own ;)12:00
didrocksSaviq: not only powerpc12:01
sil2100didrocks: should we fix that or disable powerpc from Architecture: ?12:01
didrockssil2100: ^12:01
sil2100Ah, not only?12:01
mzanettiSaviq: well... the keyboard pops now up all the time12:01
didrockssil2100: IIRC? I saw a FTBFS somewhere else12:01
Saviqmzanetti, yeah, will test12:01
sil2100didrocks: right! There's one later on for amd64, looking at that one...12:01
didrockssil2100: yep :)12:02
didrockssil2100: better to fix the deprecation if needed12:02
sil2100Ah, the same error, ok, so let's fix that ;p12:02
didrocksyep ;)12:02
didrocksneeds a hand?12:02
sil2100For now I'm branching the branch, will try doing it now12:03
didrocksok ;)12:03
* mzanetti -> food12:05
sil2100Test building12:05
=== alan_g is now known as alan_g|lunch
Saviqmzanetti, if you freshly flash your N10, can you type the password in straight after the first boot?12:10
=== dandrader is now known as dandrader|lunch
mzanettiSaviq: let me reflash12:11
mzanettiSaviq: you cant?12:14
Saviqmzanetti, nope, only after a reboot12:14
Saviqmzanetti, so very difficult to reproduce12:14
mzanettiSaviq: ok... but thats the one where the osk doesn't popup everywhere I guess12:14
Saviqmzanetti, and must really be some aace12:14
Saviqmzanetti, yeah12:14
Saviqmzanetti, race12:14
Saviqmzanetti, I imagine the shell starts before the OSK comes up12:15
mzanettiSaviq: yeah... what I fixes is really the osk on the network password input12:15
Saviqmzanetti, and it never tries to reconnect12:15
mzanettiSaviq: problem is, by the time you can log in to investigate, everything is already settled12:16
Saviqmzanetti, exactly12:16
Saviqdandrader|lunch, "install TARGETS given target "InputFilterAreaQmlFile" which is not an executable, library, or module."12:17
mzanettihmpf... battery died during flash :/12:17
Saviquh oh12:17
Saviqmzanetti, can you add a FIXME there that it should be implicitHeight, please12:18
mzanettiSaviq: just wondering. what would be the advantage of using implicitHeight?12:18
Saviqmzanetti, mostly style12:19
Saviqmzanetti, components should not define their own dimensions12:19
Saviqmzanetti, and implicit* are there to show the default dimensions12:19
sil2100didrocks: still didn't do the test build, because I needed to install a package but I still have dist-upgrade running - and my chroot is still installing build-deps ;p But the merge request is ready and looking sane:12:20
mzanettiSaviq: I think I know what the issue is12:23
mzanettiSaviq: the implicitHeight is not supposed to change at runtime12:23
Saviqmzanetti, that can't be true12:23
Saviqmzanetti, Columns have implicit height12:23
Saviqmzanetti, based on their children size12:23
Saviqwhich obviously can change12:24
mzanettiSaviq: hmm... ok... then not...12:24
didrockssil2100: I've updated the spreadsheet for the hud, feel free to complete12:24
sil2100didrocks: ok, thanks!12:26
=== alan_g|lunch is now known as alan_g
mzanettiSaviq: added the FIXME12:31
mzanettiSaviq: did you remove the link to the osk bug?12:32
Saviqmzanetti, yes12:32
mzanettiso the question arises, why? :D12:32
mzanettiSaviq: ^12:33
mzanettiit does fix that bug, doesn't it?12:33
Saviqmzanetti, did I remove the wrong link?12:33
Saviqmzanetti, the bug is more generic ;)12:34
mzanettiSaviq: "Keyboard not showing up for wifi password entry on Nexus 10"12:34
mzanettinot sure...12:34
Saviqmzanetti, ok, it should be12:34
Saviqmzanetti, "OSK not showing up in shell on first boot on Nexus 10"12:35
mzanettiimho a different one... but don't wanna argue about this now12:35
=== Saviq is now known as Saviq|mtg
mzanettias there really is a bug that prevents it in the network setup... regardless how often you boot12:35
=== olli__ is now known as olli
Saviq|mtgmzanetti, I can access the pass entry just fine in pristine image12:37
Saviq|mtgmzanetti, assuming the OSK actually works12:37
mzanettiSaviq|mtg: didn't work here12:37
mzanettiSaviq|mtg: that said. because its because of the squeezed layout there might are some pixels where you can indeed trigger it12:38
sil2100didrocks: test build done12:38
mzanettiSaviq|mtg: FYI: can't reproduce the first-boot-no-osk thing12:40
mzanettiSaviq|mtg: works fine here12:40
mzanettiin people search and greeter and everywhere12:40
Saviq|mtgmzanetti, yeah, race12:40
sil2100didrocks: https://code.launchpad.net/~sil2100/libappindicator/fix_ftbfs_deprecated_func/+merge/159809 <- once you have a free moment12:47
didrockssil2100: and approved!12:51
didrocksthanks to you :)12:52
=== dandrader|lunch is now known as dandrader
didrockssil2100: do you know about the ssh connection what was the issue?12:52
=== _salem is now known as salem_
didrockssil2100: btw, run 115 has some AP result12:54
dandraderit's amazing that to build hud I need dbus installed12:54
dandrader(build its package)12:54
dandradermake its tough to build its package from within a chroot12:55
sil2100didrocks: will look at that, just will have some hangouts in a moment12:58
didrockssil2100: great!13:04
tsdgeosSaviq|mtg: can we milestone stuff like https://bugs.launchpad.net/touch-preview-images/+bug/1170550 so it shows in https://launchpad.net/~aacid/+upcomingwork ?13:08
ubot5Launchpad bug 1170550 in touch-preview-images "Searching for music restarts the shell on manta" [Critical,Confirmed]13:08
Saviq|mtgtsdgeos, we'd need a milestone for the touch images project13:13
Saviq|mtgtsdgeos, which there aren't any13:13
tsdgeosok :/13:13
tsdgeosanyway i guess i can include that in my head in the same place as the carousel-listview work13:13
dandraderSaviq|mtg, fixed that dumb mistake. but now that I have a chroot to try out packaging such things shouldn't pass onward anymore :)13:17
Saviq|mtgdandrader, yeah, sbuild helps a lot her13:17
cyphermoxdidrocks: what about head/indicators.cfg ? I deployed it when indicator-network was added13:30
=== Saviq|mtg is now known as Saviq
cyphermoxdidrocks: yesterday I couldn't release bamf / libappindicator in time btw, the publish step didn't work for some reason13:32
didrockscyphermox: did you use -U?13:32
didrockscyphermox: it seems that it didn't run for the past 28 days13:32
cyphermoxwhen I run cu2d-update-stack -dU always13:32
didrocksand there was no schedule in the head13:32
didrockscyphermox: weird, I did the same and it's scheduled now13:33
didrocksanyway, no worry, on the publisher, do you have the run numbers?13:33
didrockscyphermox: it's possible than the "manual publication forcing" has a bug13:34
didrockscyphermox: those changes can be SRUed, isn't it?13:34
cyphermoxthat's how it looked, but now the results aren't even the same as they were13:34
cyphermoxyeah, all can be SRUd13:35
didrockscyphermox: let me quickly look at the code :)13:36
didrockscyphermox: indeed13:36
didrockscyphermox: I screwed up :p13:37
didrockssorry about that13:37
didrocksthe force doesn't override the force manual publishing13:37
cyphermoxyeah that was builds 19 and 2013:38
didrockscyphermox: fixed in rev 30513:39
didrocksand deployed13:39
dandraderSaviq, ok, jenkins approved the MP13:46
dandraderSaviq, it's no you now13:46
Saviqdandrader, yup, already on it13:48
Saviqmzanetti, if you want to take the apps lens search bug from dandrader, that would probably work, too - and you have some more experience with the demo-assets package and stuff13:50
=== alecu_ is now known as alecu
Saviqdednick, are you going to investigate the implicitHeight issue then, or are we just going to go with mzanetti's fix then?13:51
dandradermzanetti, Saviq fine by me. I was just starting on it13:51
mzanettiSaviq: do you know in which repo I can find the tablet-services file?13:52
mzanettito at least hot-fix it a little13:52
Saviqmzanetti, lp:ubuntu-session13:52
* mzanetti tries13:53
Saviqmzanetti, yeah13:53
mzanettiSaviq: and that number in the beginning is so to say a sleep n before the binary gets spawned?13:54
Saviqmzanetti, I believe so13:54
Saviqrsalveti, right ^?13:54
mzanettiok... the reviewer will tell me if not :D13:54
rsalvetiSaviq: yup13:55
Saviqdandrader, approved!13:55
dandraderSaviq, wow, I didn't think I would live to see this day! :P13:55
Saviqdandrader, it was a 2k diff after all ;)13:55
* dandrader goes to rebase his "close apps from dash" branch on top of it13:57
mzanettirsalveti: I hope this should help with this: https://code.launchpad.net/~mzanetti/session-manager/lauch-shell-later/+merge/15983213:58
mzanettirsalveti: I verified that maliit comes up successfully and is operating. so its really a race condition on the first boot13:59
dednickSaviq: i can if you think it is a good use of time.13:59
mzanettirsalveti: we have an idea for the proper fix, but that's not going to happen today13:59
mzanettidandrader: have a link to the bug for me?14:00
dandradermzanetti, https://bugs.launchpad.net/bugs/117049514:01
ubot5Launchpad bug 1170495 in touch-preview-images "Search results on apps not obvious" [Undecided,Confirmed]14:01
Saviqdednick, we'll have to fix/dive into it anyway, so your call on whether what you're doing now is higher prio14:02
mzanettidandrader: cheers14:02
dednickSaviq: i'll just get mu current work finished, then take a look. I think Cimi is waiting on me.14:03
Saviqdednick, yeah, thought so14:03
rsalvetimzanetti: hm, that will add 2 seconds for the shell14:04
rsalvetimzanetti: what would be the proper fix for it?14:04
mzanettirsalveti: modify the Qt maliit plugin: In case the connection to maliit-server fails during startup, it needs to listen to onServiceRegistered() on D-Bus and if malitt-server becomes available, connect to it14:05
mzanettirsalveti: in here: https://qt.gitorious.org/qt/qtbase/blobs/606c21526aa4183946076e7784eea14b563a03f6/src/plugins/platforminputcontexts/maliit/qmaliitplatforminputcontext.cpp14:05
mzanettiline 54414:05
Cimidednick, does variable != undefined work?14:07
Cimidednick, in my previous code, only != "undefined" did14:07
dednickCimi: should work14:08
Cimidednick, you verified it works?14:08
CimiI had to put "" because wasn't working on my code14:08
Cimi(when I did for the carousel)14:09
dednickCimi: i've never seen "undefined"14:09
Cimidednick, look at my code :-D14:09
Cimiok though, fine14:09
Cimiwhen I had undefined things, I had to check the string14:09
dednicki did a search on the code tree for it ;)14:09
Cimiwas old code indeed14:10
rsalvetimzanetti: right14:10
rsalvetimzanetti: are you planning on doing such change?14:10
rsalvetimzanetti: also, how critical is to get the workaround in, which bug will it fix?14:10
mzanettirsalveti: https://bugs.launchpad.net/touch-preview-images/+bug/115750814:10
ubot5Launchpad bug 1157508 in touch-preview-images "Keyboard sometimes not showing up on first boot after flashing on Nexus 10" [High,Confirmed]14:10
mzanettirsalveti: tmoenicke will do the proper fix14:10
mzanettirsalveti: beginning of next week14:11
Cimimzanetti, if we find a test where code could be simplified using _data14:11
Cimimzanetti, shall I mark it as needs review or no? :)14:11
mzanettirsalveti: regarding the question hwo critical it is... I would say not critical. But opinion may differ from others. the bug is mentioned here: http://iso.qa.ubuntu.com/qatracker/milestones/268/builds14:12
Cimilike this https://code.launchpad.net/~nick-dedekind/unity/phablet-tests-dashcontent/+merge/15945914:12
Cimiat least the last test14:12
mzanettirsalveti: which creates visiblity on it :/14:12
mzanettiCimi: yeah... the last one here should definitely be a _data() test imo14:13
Cimimzanetti, not sure about a couple extra here14:13
Cimimzanetti, like test_positioned_at_beginning_signal14:13
mzanettiCimi: if one of them fails its way easier to find which one, because it will be split into separate test cases14:13
Cimiyour opinion as expert?14:13
rsalvetimzanetti: right, let me check with folks then, mind updating the bzr merge package changelog to include the bug number as well?14:14
mzanettirsalveti: sure. no problem14:15
rsalvetimzanetti: awesome, thanks14:15
mzanettiCimi: yeah... I think the test_*_signal could be aggregated... but I'm not saying thats a must have14:15
Cimidednick, so, it's up to you really14:16
Cimiread my chat14:17
dednickCimi: ok. i'll take a look14:19
Cimidednick, it's not required, the test works the same, it depends how you want to spend your friday afternoon :)14:19
dednicki fancy a drink14:20
Cimigo for it, I cover you ;)14:20
Ciminobody will ever know :D14:20
=== dandrader is now known as dandrader|afk
dednickCimi: pushed unity fake categories14:29
nic-doffaySaviq, another QML related question. I can't seem to find info on this. All I need to do is sequentially trigger one state after another. What's the best way to achieve this?14:29
Cimidednick, awesome thx14:31
didrockshey mterry! not sure what's your plan today, but it seems we have all autopilot tests failing, do you have a minute to look at this? (didn't get the time to look at that yet): for instance, run 115 on the generic job14:32
didrocksmterry: it's the      import Ubuntu.HUD 0.1 as HUD14:32
didrockson gallery-app and webbrowser-app14:32
mterrydidrocks, ah good, maybe their merges will go through then14:33
didrocksmterry: sorry, not following you :)14:34
didrocksmterry: btw, apparently, it's better for you to not get outside I heard today14:34
mterrydidrocks, I've had merges for the hud transition ready for over a week, but they couldn't land because CI didn't have new hud yet14:34
mterrydidrocks, yeah  :)14:34
didrocksfginther: can you help there? ^14:34
didrocksmzanetti: maybe you have something as well ^ :)14:35
mterryThe jobs just need to be kicked14:35
didrocksmterry: do you think it's using the right ppa this time? :)14:35
fgintherdidrocks, mterry, I'll kick them again.14:35
fginthermterry, the phone-app was building yesterday, but failed the tests14:36
mzanettididrocks: ?14:36
Saviqnic-doffay, you could add a PropertyAction {} that would change the state to the next one14:37
Saviqnic-doffay, but the question is if you actually need the in-between states at all14:37
didrocksmzanetti: see the backlog for some context, transition to HUD2, I think you had some dent on this for autopilot :)14:37
nic-doffaySaviq, what do you mean by in between states?14:37
Saviqnic-doffay, are we talking about QML states here at all?14:38
nic-doffaySaviq, yeah.14:38
Saviqnic-doffay, ok, so if you defined the states, in the transitions14:39
Saviqnic-doffay, you can put a SequentalAnimation { ... PropertyAction { }14:39
Saviqnic-doffay, that will mean that after everything else in the SeqAnim has finished, the state would get changed to the next one14:39
Saviqnic-doffay, but the real question is if you need the separate state at all14:39
Saviqnic-doffay, i.e. if it's just meant to be as a step in the transition, you can use sequential animations14:40
Saviqnic-doffay, without keeping it in a state14:40
nic-doffaySaviq, they should be different states. But for the inital animation I need to run through two of them.14:41
Saviqnic-doffay, then yeah, sounds like the PropertyAction {} approach will do14:41
nic-doffayCheers Saviq !14:41
Saviqnic-doffay, you can also look at ScriptAction {} if PropertyAction won't do14:41
Saviqi.e. if you only want to do it on the first transition14:42
Saviqnic-doffay, I do remember, though, that there is some limitation around that14:42
Saviqnic-doffay, i.e. it might be that you can't change the state property in a transition14:43
nic-doffaySaviq, giving a SequentialAnimation with a PropertyAnimation a go quickly, I'll let you know if it doesn't perform...14:43
Saviq'cause that could result in a weird state14:43
Saviqwhere you didn't yet reach the new state (transition has not finished)14:44
Saviqand changed the state14:44
Saviqnic-doffay, so you might need to use the running property of Transitions14:44
Saviqand act in « onRunningChanged: if (!running): »14:45
fginthermterry, I've kicked off new ci builds for your phone, webbrowser and gallery hud branches14:46
dednickCimi: i've udpate the dashcontent test to use data. but only the last one.14:47
Cimiok fine14:47
didrocksfginther: https://code.launchpad.net/~mterry/cupstream2distro-config/move-camera/+merge/159494 mind having a look?14:47
=== dandrader|afk is now known as dandrader
fgintherdidrocks, looking14:48
didrockssil2100: is https://code.launchpad.net/~sil2100/cupstream2distro-config/qtubuntu_camera_additions/+merge/158931 still needed?14:49
nic-doffaySaviq, this is what I'm attempting to do at the moment.14:53
sil2100didrocks: let me check14:53
nic-doffayI can't really put it in a transition because I don't want this behaviour repeated every time a state changes.14:53
sil2100didrocks: I think mterry's branch is more advanced14:54
sil2100So I'll mark mine invalid14:54
didrocksok ;)14:54
didrocksthanks sil210014:54
mterrysil2100, my branch has robot arms14:54
sil2100It has all those fancy hooks and configurations14:55
tsdgeosSaviq: ok, so the shell part for the highlighting of results is up and ready, but we don't want it merged until we get the Qt support and the backend support, right?14:55
mzanettiSaviq: what would you say is the "proper" fix for the search thingie? just hide everythig but "Installed apps" if searchtext.length > 0 ?14:55
davmor2Hey guys in autopilot is there a way to get a list of applications it knows about, I assumed it just used the .desktop name for the app but obviously not14:56
tsdgeosmzanetti: dandrader is working on that, no?14:59
mzanettitsdgeos: no... Saviq asked us to hand it over to me15:00
sil2100Oh noes, Francis dropped out15:00
mzanettibetter make sure we're not duplicating15:00
tsdgeosmzanetti: i'd say that the fix he proposes in the bug is creating a proper-ish lens so that the "Recently executed" stuff goes away if there's no match15:00
tsdgeoslike happens in the rest of places15:01
davidcalledidrocks, heya. I'm looking for a launchpadlib way to add affected projects to bugs, any idea? I don't find anything in the doc.15:05
didrocksdavidcalle: it's quite easy15:06
didrocksdavidcalle: look at lp:unify15:06
mzanettitsdgeos: yeah... but I meant for a workaround for today15:06
didrocksdavidcalle: http://bazaar.launchpad.net/~didrocks/unify/trunk/view/head:/unify/bugshandler.py#L51115:06
didrocksdavidcalle: you can look some lines above for the definition of "component_to_open"15:07
davidcalledidrocks, perfect, thanks!15:07
didrocksdavidcalle: ah, even better: http://bazaar.launchpad.net/~didrocks/unify/trunk/view/head:/unify/bugshandler.py#L17315:07
Saviqnic-doffay, not PropertyAnimation but PropertyAction15:09
nic-doffaySaviq, seems to be doing the job atm!15:09
Saviqnic-doffay, ok15:10
Saviqmzanetti, I commented on the bug15:10
mzanettiSaviq: yeah... don't understand what you're trying to say15:10
Saviqmzanetti, just move the hardcoded things out to a (visible=false) mockapps.scope15:10
Saviqmzanetti, and use the same approach we have in DashHome15:11
sil2100fginther: hello!15:11
sil2100fginther: I know you're probably terribly busy right now...15:11
Saviqmzanetti, to use the data from the mockapps.lens for the two categories15:12
Saviqmzanetti, then, bind the search query on mockapps.lens to the page header search query15:12
fginthersil2100, what's up?15:12
sil2100fginther: but regarding for instance https://code.launchpad.net/~mterry/gallery-app/hud1/+merge/158471 - I saw you did some work to enable daily-next for the CI15:12
Saviqmzanetti, I might be overcomplicating things15:13
fginthersil2100, yes I've enabled daily-next for a few jobs so far.15:13
mzanettiSaviq: I guess so :D15:13
sil2100fginther: but I saw in one place that there was an error in executing the hook script15:13
sil2100Trying to find it now ;/15:13
sil2100I had it just a moment agi!15:13
dandradertsdgeos, would you like to claim this review? https://code.launchpad.net/~dandrader/unity/phablet_close_apps_from_dash/+merge/15984515:15
Saviqmzanetti, you could add/remove the categories from categoryListModel indeed15:15
sil2100fginther: it was for the phone-app hud1 branch... -> https://jenkins.qa.ubuntu.com/job/phone-app-raring-i386-ci/54/console15:16
sil2100fginther: but I'm a bit confused...15:16
dandradertsdgeos, thanks!15:17
* tsdgeos pressed the claim button15:17
sil2100Anyway, we'd need the phone-app and gallery-app merges for switching to HUD 1.0 merged somehow15:17
Saviqmzanetti, yeah, I think for now just get into the SortFilterProxyModel15:18
Saviqmzanetti, and change filterRegExp and invertMatch accordingly15:18
fginthersil2100, the phone-app is building now, but it's failing the autopilot tests. one moment, I'll grab the link15:18
Saviqmzanetti, DashApps.qml:10315:18
mzanettiSaviq: purrfect... that's exactly what I started15:18
fginthersil2100, https://jenkins.qa.ubuntu.com/job/generic-mediumtests-runner/1132/15:19
tsdgeosdandrader: should the [x] go away if i launch a different app?15:20
tsdgeosseems it should, no idea what design said or if you have any design :D15:20
dandradertsdgeos, I don't know15:20
fginthersil2100, gallery-app and webbrowser-app failed because of missing dependencies in the quantal builds15:21
tsdgeosdandrader: do we have a design point of contact for this?15:21
dandradertsdgeos, I mean, i didn't put any code to change the close mode when that happens and this feature has no design yet15:21
tsdgeosi see15:21
fginthersil2100, I'm going to try again with the old ppas add15:21
tsdgeosok then, let them file bugs :D15:21
sil2100fginther: ah, I see that they're failing because HUD 1.0 is not available15:21
dandradertsdgeos, it's explained in the commit description15:22
tsdgeosdidn't read it :D15:22
tsdgeoswas just quick testing15:22
fginthersil2100, the autopilot tests run on quantal, is that the cause?15:23
Saviqtsdgeos, did you interface with Mirv to get QColor distropatched?15:24
sil2100fginther: probably, uhhh, why did the tests run on quantal btw.?15:24
tsdgeosSaviq: not yet since there's no agreement upstream yet on the final form of the patch, was waiting on that15:24
fginthersil2100, that what we had to support when starting to run those tests15:24
Saviqtsdgeos, ok15:24
sil2100fginther: does the generic-mediumtests-runner job only run on quantal?15:24
sil2100Ah, ok ;)15:25
fginthersil2100, yes15:25
tsdgeosSaviq: i'd prefer to have something approved upstream for 5.2 and then put it in our packages of 5.1/5.0.2/whatever, that ok?15:25
fginthersil2100, we have requested some resources to start the transition to raring15:25
Saviqtsdgeos, perfect15:25
sil2100Ok, hmmm15:25
sil2100fginther: would you mind if we approve the merges globally without the generic-mediumtests-runner passing?15:26
fginthersil2100, that's not my call. Please ask mzanetti.  Note that if we do that, we won't be able to run the medium tests on any future MPs15:26
sil2100fginther: indeed, but without those branches we're blocked with our stacks on the other hand15:27
mzanettisil2100: tried that... if you do it we can throw away autopilot tests within a month. noone will fix them15:27
fginthersil2100, awesome, deadlock15:27
sil2100Since we already have HUD 1.0 in the daily-next PPA, so all tests are failing there15:27
tsdgeosdandrader: nitpick, witdth -> width15:28
sil2100fginther, mzanetti: can't we get HUD 1.0 into some quantal PPA that could be used by CI?15:28
fginthermzanetti, can the medium tests run on raring (assuming we had a box)? Would that unlock us?15:29
mzanettisil2100: sure. if we have a ppa we just need to add a hook to the job15:29
tsdgeosdandrader: also i'd say the "// was bla bla" is not really needed, we can use bzr log :-)15:29
mzanettifginther: I requested the raring vm today15:29
mzanettifginther: assuming all our scripts and autopilot works fine on it, we should be able to just switch, yes15:29
sil2100Need to check what we need to get the new hud 1.0 on quantal15:30
mzanettitsdgeos: you know perhapy? ^15:30
fginthersil2100, mzanetti, it's quite easy to add a ppa to the build if we can get hud in 115:30
tsdgeosmzanetti: hmmm, what's the real question?15:30
mzanettitsdgeos: they seem to have issues with installing the HUD stuff15:31
mzanettisil2100: can you explain more? tsdgeos is our HUD guy15:31
=== jhodapp is now known as jhodapp|lunch
sil2100Ok, so hm...15:31
sil2100Well, I'm thinking if it makes sense anyway15:31
sil2100tsdgeos: would it be hard to get the latest lp:hud (1.0) working on quantal ? Does it have any hard dependencies on raring?15:32
dandradertsdgeos, I have to say what was its previous value (which is cleaner) to help in the explanation of the current one15:32
dandradertsdgeos, otherwise the other sentence would be left without context15:32
sil2100mzanetti, fginther: since I'm wondering if it makes sense to bother with quantal building, since right now even quantal autopilot coverage for lp:gallery-app is pointless for us15:32
sil2100Since lp:gallery-app is anyway now ->raring15:32
tsdgeosdandrader: sure, i meant to kill both :D15:33
dandradertsdgeos, but them someone will want to change it back again without knowing the problem with it15:33
sil2100So even autopilot test results for quantal for lp:gallery-app and lp:phone-app seems pointless, as it doesn't show the state of the app in the current release15:33
sil2100There should be separate branches for quantal anyway15:34
dandradertsdgeos, general rule15:34
tsdgeossil2100: lp:hud is not going to work with the shell anyway15:34
dandraderwhen some code is weird (as with work-arounds), you have to explain it15:34
tsdgeosdandrader: well i'd say we should have a test then :D15:34
tsdgeosdandrader: i don't trust your general rules, they say you have to wrap at column 72 ;-)15:34
dandradertsdgeos, how am I going to test for a warning message that is printed on stdout?15:34
mzanettihmm... I guess we will have a raring VM by monday15:34
mzanettithat means I could start trying to run mediumtests on it15:35
tsdgeosdandrader: it doesn't cause any other problem?15:35
dandradertsdgeos, I don't know15:35
dandradertsdgeos, wrapping long lines is a very widely used convention. I didn't invent it15:35
fginthermzanetti, I can help with some testing on my end15:35
tsdgeosdandrader: i know15:36
tsdgeosthere's lots of old conventions i don't trust :D15:36
dandradertsdgeos, it's old, but not obsolete15:36
mzanettifginther: I think the biggest issue is to update scripts in lp:ps-qa-tools to be able to set up machines on raring15:37
mzanettifginther: once we have those and can set them up I don't expect much troubles15:37
dandradertsdgeos, you could just as well not trust the age-old convention of indenting your code consistently15:38
tsdgeosi could15:38
dandradermixing tabs with spaces, etc15:38
tsdgeosbut my good sense made me decide it's ok15:38
fginthermzanetti, I'll setup a local VM and check them out15:38
tsdgeosdandrader: otoh we do mix tabs and spaces in poppler :D15:38
tsdgeoshate it15:39
sil2100didrocks: what say you?15:39
dandradertsdgeos, and compiz, if I'm not mistaken15:39
dandradertotally crazy15:39
tsdgeosnever understood the reason for it15:40
sil2100didrocks: I mean, I think it would be wise to get those two merges to lp:gallery-app and lp:phone-app in, even though they would basically break quantal autopilot CI for those branches15:41
sil2100didrocks: since at least we'll get the stack AP tests to work15:41
sil2100(the HUD 0.1 -> 1.0 switch)15:41
Saviqmterry, about the moc things, it built (and worked) just fine here without the symlinks, how can I reproduce what you're seeing?15:41
sil2100Ok, brb, need to drive my girl somewhere, be back soon15:41
tsdgeosdandrader: the  ApplicationManager::stopProcess implementation in tests/mocks/Ubuntu/Application/ApplicationManager.cpp doesn't seem to match what we do in reality15:43
tsdgeosi mean, when killing an app, if you have another one it'll replace it15:43
tsdgeosnot have m_mainStageFocusedApplication = 0;, no?15:43
tsdgeosor maybe you do15:43
tsdgeosand then get a focus in?15:44
dandradertsdgeos, well, the resulting behavior is the same15:45
tsdgeosit's just the mock part15:46
mterrySaviq, hmm15:46
tsdgeoswe can fix it later if we find out doesn't really adjust what the non-mock does15:46
tsdgeosthat's the problem with mocks15:46
mterrySaviq, i took out that stuff and I got the symbol error15:46
tsdgeosyour test depends on it being correct :D15:46
Saviqmterry, let me try again15:46
Saviqmterry, it's conflicting again, btw ;)15:48
tsdgeosdandrader: on test_enterTerminationMode() can you add the compares that check the termination mode is still enabled after the first mouseRelease ?15:48
mterrySaviq, you are killing me15:49
mterrySaviq, everyone stop changing trunk15:49
Saviqright now!15:49
dandradertsdgeos, !?15:49
tsdgeosdandrader: now you do : press, wait, check that termination is enabled, release15:50
tsdgeosi'm askign you to15:50
dandradertsdgeos, mousePress+wait == long press15:50
tsdgeospress, wait, check that termination is enabled, release, check that termination is still enabled15:50
tsdgeosi understand that15:51
dandradertsdgeos, you mean adding another check after the first release and before the second press15:51
dandradertsdgeos, ok, I can do it15:51
mterrySaviq, merged from trunk15:52
Saviqmterry, cheers15:52
fginthermterry, reviewed: https://code.launchpad.net/~mterry/cupstream2distro-config/move-camera/+merge/15949415:52
dandradertsdgeos, done15:53
mterryfginther, updated15:54
tsdgeosdandrader: also those 800 look a bit ugly (because of the arbietrarity of 800), do you think we could do a while with a qsignalspy against pressAndHold?15:55
tsdgeosnot sure it's much better15:55
tsdgeosbut kills the 80015:55
dandradertsdgeos, you mean waiting until pressAndHold gets emitted?15:56
Saviqmterry, yeah, it was working for me probably because the links were still there15:56
tsdgeosdandrader: yep15:56
dandradertsdgeos, it's worth trying15:56
tsdgeosdandrader: actually you may as well remove the 800 and let tryCompare do the waiting, no?15:56
tsdgeosit is waiting for 5sec by default15:57
dandradertsdgeos, I would like to make it explicit that I'm "pressing and holding"15:57
tsdgeosso that should be enough15:57
tsdgeosdandrader: add a comment :D15:57
Saviqmterry, I really would rather have the headers simply included in our tree for now, then15:57
tsdgeosor do the spy thing15:58
tsdgeosand trycompare on it15:58
Saviqmterry, instead of the link (and we could then stop pretending we actually link to liblightdm)15:58
mterrySaviq, we don't pretend we link right now15:58
dandradertsdgeos, it will fail it the state is changed onPressed()15:58
Saviqmterry, yeah, but we depend on the -dev15:58
dandraderI mean, it will pass15:58
dandraderactually the current test already does15:59
mterrySaviq, sure.  Is that so bad?15:59
tsdgeosdandrader: true15:59
nic-doffaySaviq, out of interest, I tried a PropertyAction as you suggested because the PropertyAnimation is overkill, but this doesn't appear to work: https://pastebin.canonical.com/89592/15:59
Saviqmterry, the process of building leaves artifacts in the source tree15:59
tsdgeosdandrader: so add the spy for pressAndHold ?15:59
dandradertsdgeos, something like that. I'm not sure yet how to do it15:59
Saviqnic-doffay, that's because you're setting the value to "pointer" and then to "unfilled" straight away16:00
Saviqnic-doffay, and you can't animate strings16:00
mterrySaviq, hmm...  I don't like hard copying it for the mock either...  unless we add a test that fails if the system version and the local one are out of sync.  but we'd still have to dep on the -dev package then16:00
fginthermterry, approved. thanks!16:00
mterrySaviq, but at that point, we're going out of our way to avoid a symlink16:00
mterrySaviq, is there an option we can pass to moc to make it smarter about weird header situations?  I couldn't find any16:01
tsdgeosdandrader: should be "easy", no? just create a  SignalSpy { signal: "pressAndHold" }set the target to the tile and do a tryCompare(spy.count, 1) ?16:01
Saviqnic-doffay, that's why you shouldn't use PropertyAnimation on string properties16:01
Saviqmterry, ok, I'll have a fresh look on Monday16:02
dandradertsdgeos, I have also to check that it doesn't change the terminatio mode before that signal is emited16:02
nic-doffaySaviq, PropertyAnimation was working perfectly, but PropertyAction doesn't.16:02
dandradertsdgeos, so maybe tryCompareFunction() will do16:02
Saviqnic-doffay, that's only because PropertyAnimation has a duration16:02
Saviqnic-doffay, so it only sets the value to the new one after its duration16:02
tsdgeosdandrader: will add that as needs fixing in the MR so we don't forget16:02
nic-doffaySaviq, so that would be the best method to use then?16:03
tsdgeosneed to run now16:03
dandradertsdgeos, ok.16:03
mzanettitsdgeos: hurry!16:03
dandradertsdgeos, hey hey16:03
Saviqnic-doffay, if you put a PauseAnimation between the PropertyActions, it will work16:03
dandradertsdgeos, one sec!16:03
dandradertsdgeos, does signalspy have that bug that it doesn't reset its count?'16:03
didrockssil2100: discussed with sergiusens about it16:03
dandradertsdgeos, or is it something else?16:03
didrockssil2100: so basically, what we can do:16:03
tsdgeosdandrader: it may, i've seen we do .reset() in some places16:03
Saviqnic-doffay, but really, if you have transitions between the "pointer" and "unfilled" states16:03
nic-doffaySaviq, ok brilliant thanks.16:03
didrocksquantal will have a final release today16:03
tsdgeos.clear() i  mean16:03
didrockssil2100: then, we can switch on Monday to HUD 216:04
tsdgeosdandrader: ↑16:04
didrocksfor everything16:04
didrocksmzanetti: FYI ^16:04
nic-doffaySaviq, I have no transitions between them.16:04
tsdgeosdandrader: mzanetti you had this problem, right?16:04
Saviqnic-doffay, you should put the PropertyAction changing the state at the end of the transition to "pointer"16:04
didrockssil2100: making sense?16:04
didrockssil2100: so don't break the apps right now, let's do that on Monday16:04
nic-doffaySaviq, but then that would apply to every state change which I don't want.16:04
Saviqnic-doffay, can you push the whole code somewhere? I think there's some misunderstanding going on16:04
nic-doffayYeah no problem.16:04
mzanettidandrader: yes... there is a bug in signalSpy. however its a bit weird... as long as you keep the same signalName and target I think it works. but once you assign it to another signal or so the clear() breaks16:04
mzanettitsdgeos: ^16:05
dandradermzanetti, ah, ok. good enough for my usage then16:05
tsdgeosmzanetti: one day we'll have to report that bug too ;-)16:05
dandraderor even better: fix it and put your name in Qt's hall of fame :)16:05
mzanettididrocks: ack. I hope the raring VM is ready by monday so I can start with moving mediumtests16:05
didrocksmzanetti: yeah, that would be needed ;)16:06
didrocksmzanetti: let's cross fingers!16:06
mzanettitsdgeos: dandrader: nah... already in there...16:06
dandraderme too :)16:06
mzanettilets others have some fun too16:06
mterrySaviq, actually...  maybe it's not so important that we always stay in sync with the headers16:06
mzanettiindeed... doesn't sound too complicated to fix...16:06
Saviqmterry, yeah, that's what I thought16:07
Saviqmterry, since we just fake the whole interface - it's rather the opposite16:07
mterrySaviq, I can make a commit that drops the links, and thus drops the need for liblightdm in this merge16:07
Saviqmterry, if the headers changed, we would break even if we're not really using it16:07
Saviqmterry, yeah, please, just copy the headers as they currently are16:08
Saviqmterry, and when the time comes we just yank them out16:08
mterrySaviq, no, we'll still need them for the mock plugin16:08
mterrySaviq, for the mock plugin, I'd like to still use the real plugin's code, and just swap out the liblightdm bits16:09
mterrySaviq, so that we can test the plugin's logic (like the realname->name conversion016:09
Saviqmterry, yeah, we'll work it out16:09
mzanettiSaviq: ubuntu-phone mailing list. mail from Alberto :)16:13
mzanettiSaviq: https://code.launchpad.net/~mzanetti/unity/phablet-hide-static-apps-while-search/+merge/15985716:27
Saviqmzanetti, yeah, saw that ;)16:30
Saviqmzanetti, I mean the email16:30
Saviqmzanetti, "invertMatch ?" bad :P16:31
Saviqmzanetti, either do the direct condition16:31
Saviqmzanetti, or have a separate bool prop16:31
Saviqmzanetti, I know it's slightly less performant, but much more readable ;)16:32
mterrySaviq, btw, updated the branch16:38
mterryfginther, links I'm seeing in MPs like http://jenkins.qa.ubuntu.com/job/unity-phablet-raring-i386-ci/545/console are giving me 404s16:40
fginthermterry, hmmm16:41
Saviqmterry, yeah, thanks16:42
fginthermterry, this might take a bit to figure out16:43
nic-doffaySaviq, when you have a moment: https://code.launchpad.net/~nicolas-doffay/unity/infographics16:45
nic-doffayCheck Dot.qml line 3516:45
nic-doffayI'm attempting to Do a PropertyAction on the visible property which isn't working.16:45
nic-doffayEverything else is good.16:45
mzanettiSaviq: pushed a FIXME... sorry. this is all I can do now - have to run. I'll fix it properly next week.16:51
Saviqmzanetti, have fun!16:52
mzanettiSaviq: btw... using the expression directly is why I was feeling stupid... you'll understand when you see the FIXME16:52
mzanettiSaviq: diff is updated16:52
mzanettiok. then.. have a good weekend!16:55
=== alan_g is now known as alan_g|EOW
=== jhodapp|lunch is now known as jhodapp
Saviqmzanetti, yeah, it doesn't reevaluate when invertMatch is changed...18:01
=== salem_ is now known as _salem
=== _salem is now known as salem_
=== salem_ is now known as _salem

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