/srv/irclogs.ubuntu.com/2013/07/02/#ubuntu-unity.txt

bschaeferdigital, im not actually that sure, as you are no longer using unity :)00:02
davidcalledidrocks, ping07:34
didrocksdavidcalle: pong07:35
davidcalledidrocks, ça va bien ?07:35
didrocksdavidcalle: ça va, et toi, les vacances bientôt? :)07:35
davidcalledidrocks, ça va :) Oui, dans quelques semaines... Et toi ?07:35
didrocksfin août, donc il y a le temps!07:36
davidcalledidrocks, hehe, en effet ;) Au fait, j'ai oublié ça hier https://code.launchpad.net/~davidc3/cupstream2distro-config/online-raring-to-saucy/+merge/17247607:37
didrocksdavidcalle: il n'y a pas d'importance en fait de ça, puisque cette stack n'est pas sous daily releases07:38
didrocks(tout ce qui est dans online)07:38
didrocksdavidcalle: tu peux même le supprimer en fait :)07:38
davidcalledidrocks, hum... J'ai l'impression que Jenkins ne peut pas trouver les build deps, ça ne dépend pas de ça, donc...07:39
didrocksintéressant07:39
didrocksdac07:40
didrocksdone!07:40
davidcalledidrocks, par hasard, est-ce que Jenkins utilise la distro du dernier changelog ?07:41
didrocksdavidcalle: possible, aucune idée pour la partie non daily release, ça a pas mal changé depuis mon premier prototype :)07:41
didrocksmais à l'époque, ça utilisait la distro du changelog07:42
davidcalledidrocks, ok, je vais regarder de ce côté là aussi. Merci ! ;)07:42
didrocksde rien!07:44
didrockspstolowski: thostr_: hey!07:55
pstolowskididrocks: hi!07:56
didrocksFYI https://bugs.launchpad.net/ubuntu/+source/unity/+bug/1189088/comments/2007:56
ubot5Launchpad bug 1189088 in unity (Ubuntu) "Unity Dash, left mouse button starts preview instead of directly the application" [High,Triaged]07:56
didrocksso I think we need to revert the behavior for the application lens07:56
didrocksas well as adapating surely some integration tests, right?07:56
thostr_didrocks: yes. and most probably extending the scope interface with a kind of activationBehaviour flag07:58
didrocksthostr_: right, would make sense07:59
didrocksdo you know how long we can expect this to take before landing?07:59
didrockswould be good to have the definitive action soon enough07:59
pstolowskididrocks: that's good news... the bad thing is it has been implemented in the dash globally07:59
didrocksargh08:00
pstolowskididrocks: so we need to make an 'exception' for apps in the dash... or implement something along the thostr_'s idea08:01
didrocksjust give me a rough approximation for landing those :)08:01
thostr_the fix shouldn't take that much time (hours)08:07
thostr_the challenge is predicting the integration and fixing the tests08:07
didrocksthostr_: right, we need a plan for that. Can you preparing one and get back to me once you get the estimate?08:13
Saviqpstolowski, thostr_ couldn't this work roughly how action activation works in previews? i.e. on activation you either come back with a preview (or a hint that a preview should be requested?), notice of scope-side activation or no-op (as it is now, right?)08:13
pstolowskididrocks: I'll implement the change across the stack today (libunity, unitycore, dash). not sure about AP test atm, I've a feeling it's simple change08:14
didrockspstolowski: ok, let's cross fingers! Thanks :)08:14
pstolowskiSaviq: it could... but then you need to change every single scope. so we can think of it as a 'hint' about how to handle activation click08:15
seb128pstolowski, you guys managed to make alt-f2 skip previews, isn't the "framework" already there?08:15
seb128or how is that different?08:16
Saviqpstolowski, right...08:16
pstolowskiseb128: for alt+f2 we added a special pseudo-schema "x-unity-no-preview:". no use for this case.08:17
seb128ok, I was mentioning it in case... ;-)08:18
seb128you probably know what you are doing, /me goes back to stuff he's doing :p08:18
pstolowskiseb128: that was a good question, it got me puzzled for a second and I had actually to lookup the sources ;)08:18
seb128;-)08:18
didrocksseb128: see, they don't know what they are doing OMG /o\ ^08:18
seb128haha08:19
pstolowskiSaviq, tsdgeos: are you ok with https://code.launchpad.net/~stolowski/unity8/activation-and-previews/+merge/172102 for the time being (I've just merged trunk as there was a text conflict)?08:29
Saviqpstolowski, I didn't look at the code much, tsdgeos your call - I'm ok with leaving LocalResult for now08:30
Saviqpstolowski, btw, can you add a TODO/FIXME around there?08:30
tsdgeospstolowski: i'll have another look onow08:31
pstolowskiSaviq: sure. I plan to do the change today/tomorrow08:31
Saviqpstolowski, the index approach looks sane, either with a "master" index role passed down from DeeListModel08:31
Saviqpstolowski, or maybe even saner with mapToSource()08:32
pstolowskiSaviq: do you think we are safe wrt to potential races? should be ok as long as we're single-threaded?08:32
Saviqpstolowski, think so, but just to be safe maybe the index role approach makes more sense08:33
Saviqwe did that somewhere already, yoo08:33
Saviqtoo08:33
Saviqpstolowski, I think in the Categories object08:34
pstolowskiSaviq: ok08:34
Saviqpstolowski, yeah, case RoleId: return QVariant::fromValue(index.row());08:34
pstolowskiSaviq: ok, nice, I didn't see that08:35
tsdgeosis testDragHandle failing for you too?08:52
tsdgeosSaviq: greyback: mzanetti: ↑↑↑08:52
* mzanetti tries08:53
tsdgeos   Actual   (actualFunc()): 10008:53
tsdgeos   Expected (expectedValue): 008:53
tsdgeos   Loc: [/home/tsdgeos_work/phablet/lolo/tests/qmltests/Components/tst_DragHandle.cpp(139)]08:53
mzanettitsdgeos: yep. failing here too08:54
* mzanetti wonders how that passed CI08:54
tsdgeosyeah :D08:54
tsdgeoswe were unlucky in that it passed CI08:54
tsdgeosand now it's blocking the rest of the stuff :D08:54
dednickhm. that passed when i tested before approve08:54
Saviqdidn't dandrader file an MP to fix?08:55
Saviqhttps://code.launchpad.net/~dandrader/unity8/fixDraHandleHint/+merge/17242608:55
Saviqit was probably my fault08:55
Saviq'cause I merged lp:unity8 and lp:unity/8.008:55
Saviqhapproved08:56
tsdgeosahhh08:56
tsdgeosmakes sense08:56
Saviqtsdgeos, dednick, mzanetti ↑08:56
mzanettiah ok08:56
dednickSaviq: ah ok. yeah, i remember that line there08:56
Saviqbzr f***ed up08:56
mzanettiSaviq: tsdgeos: still, the commit that introduces the failure was approved by PS Jenkins Bot08:57
Saviqmzanetti, nope08:57
mzanettiand also dednick and tsdgeos...08:57
Saviqmzanetti, there were two commits in parallel to lp:unity808:57
Saviqmzanetti, and lp:unity/8.008:57
Saviqmzanetti, to sync them, I merged one into the other08:57
mzanettihmm... so much for the history in bzr08:57
Saviqmzanetti, and pushed08:57
Saviqmzanetti, it's fine08:57
Saviqmzanetti, only I didn't test08:58
Saviq/mea culpa08:58
Saviqbut bzr didn't complain... :/08:58
tsdgeosquestion08:58
tsdgeoswhy https://code.launchpad.net/~stolowski/unity8/activation-and-previews/+merge/172102 CI passed ?08:58
dednickSaviq: would you be apposed to me creating cmake procedure for qmltypes files for the plugins? so we can get type info in qtcreator?08:58
Saviqdednick, no, I wouldn't be apposed :)08:59
Saviqdednick, assuming it doesn't take your whole day08:59
dednickSaviq: nope. should be quick08:59
Saviqdednick, yup08:59
mzanettidednick: yay. nice08:59
mzanettitsdgeos: oooops09:00
mzanettitsdgeos: you found a bad bug09:00
tsdgeoswhere? when? how?09:01
mzanettitsdgeos: the jenkins job still merges to lp:unity/8.0 before testing09:01
tsdgeoswops :_D09:01
mzanettifixed. tests may start failing now :D09:02
tsdgeosyay!09:04
tsdgeosnot!09:04
mzanettihuh?09:06
mzanettiSaviq: any news on the unity-api releasing?09:08
Saviqdidrocks, ↑?09:08
didrocksmhr3: hey, around?09:08
Saviqmzanetti, that's no bug, lp:unity8 == lp:unity/8.009:09
Saviqmzanetti, well, bug, maybe - but nothing tragic yet09:09
didrocksmzanetti: what about it? it's released and in the next ppa09:09
mzanettiSaviq: well, tests fail with unity8 but pass with unity/8.009:09
didrocksas the whole unity8 stack09:09
mzanettididrocks: ah ok... I thought we shouldn't use any ppas any more09:10
didrocksmzanetti: well, apparently, we can't push unity8 to distro right now09:10
didrocksto we have to use a ppa for releasing09:10
Saviqmzanetti, unity8 will remain in a PPA until mir integration is complete09:10
mzanettididrocks: ok. thanks09:10
mhr3didrocks, in hangout09:12
didrocksmhr3: there is just the start of a hang09:12
didrocksmhr3: on the ati machine09:12
didrocksmhr3: so run if you want to look at it :)09:12
Saviqdidrocks, are we merging all the qtubuntu and unity8 packaging changes today09:15
Saviq?09:15
didrocksSaviq: waiting for Mirv to deal with the unity stack to get libunity in distro and then, yeah :)09:16
Saviqdidrocks, cool09:16
didrocksmhr3: letting the -check running for you09:16
nic-doffaymzanetti, you have a moment?09:24
mzanettinic-doffay: meeting starting in 4 minutes09:26
mzanettinic-doffay: what up?09:26
ethana3Does anyone here have an always-displayed global menu bar working? I've installed various PPAs and they never seem to work.. I used to have this with ASD's global menu hack and gnome panel widget...09:26
ethana3oh, interesting.09:26
nic-doffaymzanetti, unknown issue I'm facing.09:27
nic-doffayQML related.09:27
mzanettinic-doffay: that's not exactly the most precise description of a problem...09:28
mzanettinic-doffay: have some code and a error message?09:28
nic-doffaymzanetti, it's in the gallery of the SDK.09:29
nic-doffay./gallery.sh09:29
nic-doffayThe templates are displayed using a delegate.09:30
nic-doffayEverything on the component is repeated, even though it follows the same trend as all other gallery components. So I must be missing something.09:30
nic-doffayIt's fine in a normal qmlscene.09:31
mzanettinic-doffay: not following09:32
mzanettinic-doffay: I've opened the gallery09:32
mzanettinow what?09:32
nic-doffaymzanetti, you'll need my branch.09:32
nic-doffaymzanetti, lp:~nicolas-doffay/+junk/option-selector09:32
nic-doffaymzanetti, to see the issue go to list items, then scroll down to option selector.09:33
nic-doffayThat's handled is ListItems.qml09:33
nic-doffay*in09:34
mhr3didrocks, eh, lost me scrollback, which one was it?09:34
mhr3my*09:34
didrocksmhr3: ati machine09:34
mhr3thx09:34
didrocksyw :)09:34
mhr3didrocks, can't attach to it again, so no good :/09:36
mhr3jibel, ^ ati dead09:36
jibelmhr3, ack, oom09:37
jibelcan I have that machine for 15min, I'll collect information to report a bug09:37
mhr3jibel, sure09:37
mhr3ehm, i mean, not my call :)09:38
mhr3but surely 15minutes won't kill anyone09:38
didrockseven more if needed :)09:38
didrocksnothing else to run09:38
m4n1shmhr3: when you are free can you have a look at #1196878 #1196800 and  #119682209:39
m4n1shyou know both the components well :)09:39
jibel[76766.931095] lxc-list[26171]: segfault at 0 ip b762c1a6 sp bfd80674 error 4 in libc-2.17.so[b75a7000+1ad000]09:42
jibelnot good09:42
mhr3m4n1sh, what's the benefit of moving those to libzg2?09:43
mhr3jibel, so, did i make it crash with the list call?09:44
mzanettinic-doffay: so. meeting has ended. checking out your branch now09:44
jibelmhr3, no compiz and recordmydesktop crashed before that09:45
jibelI'm disabling the node in jenkins, so it doesn't reboot under my feet09:45
tsdgeospstolowski: ping09:50
pstolowskitsdgeos: pong09:50
tsdgeospstolowski: the data of a unityPreview never changes?09:51
pstolowskitsdgeos: yes09:52
tsdgeosdamn yes to a negative question :D09:52
pstolowski:)09:52
tsdgeosyes as in "yes, it does change" or "yes, it does not change"?09:52
pstolowskitsdgeos: it doesn't change09:52
tsdgeosoki09:52
=== shiznix_ is now known as shiznix
tsdgeospstolowski: i was thinking that you could reuse a single signal then09:53
tsdgeosinstead of one per property09:53
tsdgeossince you're only going to emit it at setPreview(unityPreview) anyway09:53
tsdgeosno?09:53
m4n1shmhr3: moving everything to libzg209:55
m4n1shto avoid long term maintenance of libzg109:56
m4n1shsome fine day they will have to be moved09:56
mzanettiSaviq: can't install libunity-api from the next ppa. requires libboost 1.49 while we require libboost-1.5309:58
pstolowskitsdgeos: in therory yes, but doesn't NOTIFY protocol mandate that notification of every property follows <property>Changed naming?09:58
tsdgeosnot that i know09:59
tsdgeosbut i may be wrong09:59
tsdgeosafaik the qml side of things is all defined by the property name09:59
Saviqmzanetti, we shouldn't require boost at all, do we still?09:59
tsdgeosand the rest is just the c++ names of the "bindings"09:59
mzanettiSaviq: apparently... installing libunty-api-dev and then doing ./build -c removes libunity-api-dev again. because of conflicting boost10:00
mzanettiSaviq: oh... comes with libunity-core and libnux I guess10:00
Saviqmzanetti, yeah, probably10:00
Saviqmzanetti, still, needs fixing in unity-api10:01
Saviqmzanetti, MP incoming10:01
* Saviq hooked sbuild up to `bzr bd`, feels smug ;)10:04
tsdgeospstolowski: i mean http://paste.kde.org/~tsdgeos/787022/ in case i wasn't clear10:05
tsdgeosSaviq: mzanetti: ↑↑↑ this should work, right?10:05
Saviqtsdgeos, yeah, should10:05
pstolowskitsdgeos: yes, that's what I understood10:06
tsdgeoswhat you guys think about that?10:06
tsdgeosam i being silly?10:06
Saviqtsdgeos, pstolowski question is - do they actually ever change?10:07
Saviqshould they just be constant instead10:07
Saviqbecause you create a new Preview every time anyway?10:07
tsdgeosthey never change10:08
tsdgeosor so said pstolowski a few lines above10:09
tsdgeoswell, they change from empty to something10:09
tsdgeosand then they never change10:09
pstolowskiSaviq: in theory you could create a preview object with default ctor, and then do setUnityPreview10:09
pstolowskiin which case they would change10:09
tsdgeoswell, it's what you do, no?10:09
Saviqpstolowski, right10:09
pstolowskiyes, but only in factory method10:09
pstolowskiso it's not exposed to qml before that10:09
tsdgeosright10:10
Saviqpstolowski, tsdgeos so it's never possible that only one of those change10:10
Saviqpstolowski, tsdgeos in which case I'm fine with tsdgeos's thing10:10
pstolowskiSaviq: exactly10:10
Saviqpstolowski, tsdgeos only thing to make sure is to only emit the signal once10:13
Saviqpstolowski, so in setUnityPreview10:13
Saviqafter having updated all the members10:13
Saviqmzanetti, https://code.launchpad.net/~saviq/unity-api/any-boost/+merge/17252510:14
mzanettiSaviq: cheers10:15
pstolowskiSaviq, tsdgeos: ok10:15
mzanettidednick: you have a lot of "\No newline at end of file" in your MP10:15
tsdgeospstolowski: will give us a somewhat smaller binary ;-)10:16
dednickmzanetti: grr. ok, fixing now10:16
pstolowskilol10:17
* Saviq sees latestsnapshots10:17
didrocksSaviq: yep, let's wait for it to migrate to the release pocket to do the merges?10:17
Saviqdidrocks, yup10:18
didrocksSaviq: on the qtubuntu transition, they wanted to wait for the flipped image to be switched on though10:18
didrocksSaviq: and I think it's not there yet?10:18
Saviqdidrocks, yeah, might be10:18
didrocksSaviq: so we'll have to postpone the packaging merge until it's in, I'll ask rsalveti and sergiusens once they are around10:19
Saviqdidrocks, yup10:20
mhr3Saviq, unity on the phone is constantly using 25% cpu with yesterday's image, known?10:34
om26erI get as high as 46% continuous hog on Nexus 4 for unity810:36
Saviqmhr3, om26er is that with the flipped image?10:36
om26erSaviq, yeah, I have flipped image installed10:37
mhr3Saviq, i don't10:37
Saviqhmm I'm not seeing that...10:37
Saviqon 20130701.210:37
mhr3there's also 10% in surfaceflinger, so clearly something's repainting10:37
mhr3dunno how to give you more info10:38
Saviqmhr3, we'll try and reproduce10:38
mhr3this is even with screen off btw10:39
om26eron another note the NetworkManager was consuming 99% yesterday, had to kill the service. But that's clearly for someone else to figure.10:40
tsdgeoswoa, i'm getting 40% of unity8 in my desktop10:42
tsdgeosthat's bad :D10:42
mhr3uh oh, i just dist-upgraded and unity just crashes10:46
didrocksmhr3: apt-cache policy unity?10:48
mhr3  Installed: 7.81.210:49
didrocksah, unity810:50
didrocksas we just published a version to saucy, you frightned me :)10:50
mhr3didrocks, you're frightening me, you don't care about working system on the phone?! :P10:51
didrocksmhr3: not that, but as long as we don't run integration tests and if that passed tests, it would mean we have big issues in our infra :)10:52
Saviqtsdgeos, I can't reproduce, can you try and find out which commit introduced that?10:56
tsdgeosamy not be our fault10:57
tsdgeosi did a qml-debug thing on the phone10:57
Saviqtsdgeos, right10:57
tsdgeosand it has no repaint10:57
tsdgeosso may be some other thing10:57
tsdgeosi was going to run callgrind on it now10:57
Saviqok, I'm not seeing it anyway10:57
mlankhorstmodprobe nouveau10:57
mlankhorstoh figures10:58
Cimimzanetti, seems to work but all tests are screwed :D will have to dig but thanks11:01
mzanettiCimi: ok, cool11:01
pstolowskitsdgeos: updated my MP11:04
tsdgeosoki11:07
tsdgeosSaviq: gone back to r37 and i still get around 40% cpu usage when i'm at the videos scope11:07
Saviqtsdgeos, flipped or not?11:08
Saviqtsdgeos, and is that on your desktop still?11:09
tsdgeosSaviq: desktop yes11:11
Saviqtsdgeos, interesting11:11
tsdgeoscallgrind didn't gave me much11:11
tsdgeosgdb seems to imply there's something drawing all the time11:11
tsdgeossince everything i ctrl+c there's a renderer thread going on11:11
tsdgeosbut i may be wrong and the thread is just waiting11:11
tsdgeosanyways, if you guys don't get it let's forget it for the moemnt ;D11:12
Saviqtsdgeos, well, mhr3 and om26er seem to get it on devices11:13
Saviqtsdgeos, do you?11:13
tsdgeoslet me see11:14
=== MacSlow is now known as MacSlow|lunch
tsdgeosSaviq: no, can't repro on the phone11:22
Saviqtsdgeos, k, I'll have a shout in -touch, too11:23
=== alan_g is now known as alan_g|lunch
jibeldidrocks, mhr3 ati is back, but I couldn't collect data I needed. I killed the machine by removing the wrong modules. It's all yours, ping me when it crashes again11:55
=== MacSlow|lunch is now known as MacSlow
davidcalledidrocks, ping again. Who should I talk to for the Jenkins issue ? I now have both a saucy changelog and cupstream2distro switched to saucy, but the CI pass is still trying to build with raring (and obviously fails).12:38
didrocksdavidcalle: fginther should be your contact12:48
didrocksjibel: ok, thankx12:48
davidcalledidrocks, thanks12:48
=== alan_g|lunch is now known as alan_g
=== _salem is now known as salem_
tsdgeospstolowski: why setUnityPreviewBase + setUnityPreview instead a virtual setUnityPreview where children call the parent implementation too?13:06
pstolowskitsdgeos: because I didn't like to require subclass to call base class method in the overriden method to work correctly, I preferred to ensure this from the outside (the factory method)13:13
tsdgeosok13:15
tsdgeospaulliu: shall i top-approve https://code.launchpad.net/~stolowski/unity8/activation-and-previews/+merge/172102 or you want to have a look too?13:17
paulliutsdgeos: I've looked already. No special comments.13:18
tsdgeosok13:18
pstolowskithanks for reviewing!13:20
tsdgeosmterry: and is that "regression" accepted?13:25
mterrytsdgeos, it's only a regression if we consider the touch image to be just a demo.  Otherwise it's getting closer to actually working13:26
tsdgeossure13:27
tsdgeosbut still it'll look worse :D13:27
mterrytsdgeos, it'll look the same as when a user gets the device for the first time13:27
tsdgeosok13:27
mterrytsdgeos, which is what's happening when you install13:27
dednickdandrader: you need a review for panel drag handle branch?13:43
dandraderdednick, oh yeah13:43
dandraderdednick, a big diff, unfortunately13:44
dednickdandrader: indeed. i'll get on it13:44
dandraderdednick, thanks a lot!13:45
dandraderdednick, biggest changes are in the tests though13:46
dandradergreyback, I'm getting this error when running unity8+mir on the flipped saucy http://paste.ubuntu.com/5836289/14:01
dandradergreyback, how far did you get?14:01
dandradergreyback, btw, I've updated that wiki page14:02
greybackdandrader: not even that far :) I get crash14:02
greybackdandrader: there's no working Ubuntu.Application plugin yet. So you need to use hte fake one14:02
Saviqmzanetti, can I ask you to please look at https://code.launchpad.net/~nick-dedekind/unity/8.indicators-client/+merge/168022 from autopilot PoV?14:03
dandradergreyback, ah... it's weird the library is actually installed, so it should at least find and load it14:03
dandradergreyback, have you gone the ppa:phablet-team/mir way?14:03
greybackdandrader: no, I'm using the mir staging ppa. To my detriment I believe14:04
greybackbut I need to work on cutting edge14:04
Saviqdednick, are the autopilot tests actually run in the indicators-client branch?14:04
Saviqdednick, they're not part of the unity8 autopilot module, are they?14:04
Saviqmzanetti, should they be ↑?14:04
Saviqdednick, is there a custom target to run them maybe?14:05
Saviqdednick, I'm unable to get them to work, I'm afraid14:05
dednickSaviq: hm. apparently they're not.14:07
dednickSaviq: should i make them part of the unity8-autopilot package?14:08
dednickSaviq: or make a indicators-client-autopilot one?14:08
tsdgeosSaviq: is there an easy way to only load the dashhome?14:09
Saviqtsdgeos, it's a dconf setting14:11
dandradergreyback, ok it's working now!14:11
Saviqdednick, I think part of unity814:11
greybackdandrader: excellent!14:11
Saviqdednick, as that's the only autopilot package we're running in CI14:11
Saviqmzanetti, any comment on ↑?14:12
dandradergreyback, but doesn't mean much as it's using fake plugins, just like when you "./run" on the desktop14:12
greybackdandrader: yes I know. I'm working on a real application manager plugin so we get real behaviour back14:12
Saviqfginther, hey, is it correct that http://10.97.2.10:8080/job/unity-phablet-qmluitests-saucy/ is bound to ps-saucy-server-amd64-1 ?14:13
Saviqfginther, we're kinda blocked by mir on that one :/14:13
fgintherSaviq, I think I can help there, just give me a few minute14:14
fginthers14:14
Saviqfginther, thanks14:14
Saviqfginther, btw, would it be possible to have unity-8.0-autolanding and unity8-autolanding to block each other? i.e. so they don't run in parallel?14:26
dandraderSaviq, cant' we forget about  lp:unity/8.0 ?14:29
Saviqdandrader, we will, soon14:29
fgintherSaviq, maybe. I don't know of a simply way to do this off the top of my head, but it may be possible. Can we discuss a little later (an hour or so)?14:29
Saviqfginther, probably not worth it :)14:30
fgintherSaviq, ok14:30
Saviqdandrader, OTOH, we could just resubmit against lp:unity8 and be done with it...14:30
Saviqdandrader, I think I was just hoping we'll reduce the queue against lp:unity/8.0 quicker14:31
dandraderSaviq, exactly14:31
dednickSaviq: i've added indicator-client to the autopilot target. should now run with make autopilot.14:31
dednickand added to install.14:31
Saviqdednick, cheers14:31
Saviqdandrader, ok, doing14:33
Saviqdandrader, there's only 4 branches now14:33
Saviqdandrader, right, but... "lp:~unity-team/unity/unity8-packaging-cleanup is not mergeable into lp:unity8"14:34
* Saviq needs to pull/push14:34
mzanettiSaviq: sorry... have been eating14:36
Saviqmzanetti, don't be14:36
=== alan_g is now known as alan_g|tea
Saviqdandrader, ok, did it, if anything happens, I'll blame you :P14:39
Saviqdidrocks, resubmitted against lp:unity8 https://code.launchpad.net/~unity-team/unity8/packaging-cleanup/+merge/17257814:40
Saviqdednick, resubmitted against lp:unity8 https://code.launchpad.net/~unity-team/unity8/indicators-client/+merge/17258214:40
didrocksSaviq: ah great! let's wait from the phone fundation team for the greenlight :)14:40
dednickSaviq: ta14:41
Saviqdidrocks, yup, with that we'll be able to drop lp:unity/8.0 altogether14:41
didrocksSaviq: debian/unity8.install merge conflict btw :)14:41
Saviqdidrocks, will merge14:41
didrocksthx!14:41
dandraderSaviq, hahaha14:42
tsdgeoslol, someone using the LVWPH exposed a bug in QLimitProxyModelQML :D14:46
tsdgeostook me a while to trace14:46
=== alan_g|tea is now known as alan_g
Saviqtsdgeos, ;)14:49
Saviqtsdgeos, test, fix, ship it :)14:50
tsdgeoson it14:50
=== dandrader is now known as dandrader|afk
mzanettiSaviq: dednick: commented regarding autopilot tests14:57
Saviqmzanetti, yup, thanks14:57
Saviqmterry, uh, the meeting got pushed back to tomorrow, can you please ping sforshee to have a chat about the blanking?14:59
dednickmzanetti: ta. i'd prefer to tackle the further dev of autopilot in a future revision. This one is taking a bit long, and I'd like to get it in so we can link up with the new backends sooner rather than later.15:03
mzanettidednick: sure... It wasn't meant to add it here15:03
mzanettidednick: but as you guys asked me on the general opinion for the autopilot suite, I wrote it there as a comment15:04
dednickmzanetti: i'll fix up the targets. I'm sure i tried and the unity8 ap tests were run as well as the indicators_client.15:04
mzanettidednick: oh... I didn't do that15:04
mzanettidednick: I killed the unity8 one after a while15:04
dednickmzanetti: sure. the info is appriciated :)15:04
dednickmzanetti: yeah, it's a bit painfully long15:04
mzanettidednick: so no clue if the indicators one would have come after it15:04
mterrySaviq, sure...  I'm a bit confused why you prefer powerd's interface to Qt's, but I can talk to sforshee, sure15:05
dednickmzanetti: but yah, think we could benifit from splitting anyway15:05
Saviqmterry, no no, that's not it15:05
Saviqmterry, I just want to make sure we're doing the right thing at the right moment15:05
mterrySaviq, are you saying that in future, we will suspend before we blank the screen?15:06
Saviqmterry, i.e. backlight == 0 shouldn't necessarily mean that we should lock15:06
Saviqmterry, and I'm not even saying what you're doing is incorrect15:06
Saviqmterry, just that I don't have enough info to say that's the case15:06
Saviqmterry, i.e. backlight == 0 != blanking15:07
mterrySaviq, ah...  I thought it was by design that when the backlight is turned back on, the greeter shows15:07
Saviqmterry, it is15:07
tsdgeosSaviq: quick one https://code.launchpad.net/~aacid/unity8/qlimitproxyfiltersetsamemodeltwice/+merge/17258715:07
mterrySaviq, wait...  I get backlight == 0 != locking15:07
mterrySaviq, but I don't get backlight == 0 != blanking15:07
Saviqmterry, blanking (screen off) isn't necessarily the same as backlight == 015:08
Saviqmterry, as you can reach backlight == 0 with brightness control15:08
Saviqmterry, but that shouldn't result in locking15:08
mterrySaviq, hrm, fair15:08
mterrySaviq, OK, then without GNOME tech, I don't think we have the right signals yet...  I can put this off until the meeting tomorrow15:09
Saviqmterry, and IIUC the Qt API exposes the actual brightness/backlight control15:09
Saviqmterry, but I'd assume there should be something re: locking / screensaver, too?15:09
mterrySaviq, yes.  It still had a bug that it never emitted backlightStateChanged, so that was still good to fix, but we don't need to use it for this greeter thing15:10
Saviqmterry, yup15:10
mterrySaviq, as for locking/screensaver, I haven't seen it in Qt's API yet, but I'm not an expert15:10
=== dandrader|afk is now known as dandrader
Saviqmterry, so it's not there in the module you were looking at?15:11
mterrySaviq, there is QSystemScreenSaver, but it only exposes whether the screensaver is inhibited15:11
Saviqmterry, is this what you were looking at http://harmattan-dev.nokia.com/docs/cn/qtmobility/qsystemdisplayinfo.html#BacklightState-enum ?15:11
mterrySaviq, sure, I was going off http://doc.qt.digia.com/qtmobility/qtsysteminfo.html but same API yeah15:12
mterrythough that API is old, since the new API has a backlightStateChanged signal, but this page doesn't show that15:13
Saviqmterry, so, we definitely should expose those... but whether we should rely on it to lock the screen, /me doesn't know...15:14
Saviqmterry, we'll know by other means anyway (shell+mir will know that screen is being blanked)15:14
Saviqmterry, so yeah, let's pick that up tomorrow15:14
mterrySaviq, yeah, but it's a matter of exposing an API for it.  I hadn't thought we'd want to bother, but if we do want to treat them separately, yeah, we'll have to15:15
mzanettiSaviq: jenkins still uses the unity-build-next ppa. I guess its ok to switch it over to the "next" ppa15:21
Saviqmzanetti, I'm not sure we should use it at all, didrocks ↑?15:22
mzanettiI need the "next" one for libunity-api-dev15:22
didrocksyeah, you should use the next ppa for unity8 right now15:22
didrocks(unity8 stacks)15:22
mzanettiack, thanks didrocks15:22
didrocksuntil it's getting into distro15:22
didrocksyw!15:22
didrocks"next" is always safe anyway15:22
didrocksmterry: hum, is it possible to only restore one folder with deja-dup/duplicity?15:33
mterrydidrocks, yup, right click in nautilus15:39
didrocksmterry: no… don't tell me we have that level of integration? :-)15:39
mterrydidrocks, unfortunately no one expects it/looks for it15:40
didrocksmterry: testing, it looks awesome! :-)15:41
mterrydidrocks, you can even click in the empty bit of a nautilus folder and ask to see all missing files and restore them15:41
didrocksmterry: waow, I'm sold! :)15:41
Saviqmterry, we need integration with BackupPC ;D15:41
mterry:)15:43
jbichahi, I have two small MPs that have been pending for a few weeks:16:13
jbichahttps://code.launchpad.net/~jbicha/unity-lens-video/build-with-valac/+merge/16819916:13
jbichahttps://code.launchpad.net/~jbicha/unity-scope-home/build-with-valac/+merge/16827316:13
=== jhodapp is now known as jhodapp|lunch
Saviqmterry, FYI resubmitted as https://code.launchpad.net/~unity-team/unity8/libusermetrics/+merge/17261816:25
mterrySaviq, oh right, I forgot that was before the switchover16:25
Saviqmterry, that's ok, we just decided it's time to just move everything over16:25
pstolowskibschaefer: ping16:27
bschaeferpstolowski, pong16:27
pstolowskibschaefer: hey! fyi, I'm working on https://bugs.launchpad.net/ubuntu/+source/unity/+bug/1189088 if that's ok?16:27
ubot5Launchpad bug 1189088 in Unity "Unity Dash, left mouse button starts preview instead of directly the application" [High,In progress]16:27
bschaeferpstolowski, yup! I just was getting the bug ready to work on :)16:28
pstolowskibschaefer: in fact I've a fix, but struggling to test it atm16:28
bschaeferpstolowski, hmm yeah, getting a test for that could be hard...possibly an AP test that goes to the application lens16:29
bschaeferand you *should* be able to assert that an app start, or assert we are not in the dash and not in preview mode16:30
pstolowskibschaefer: I mean more fundamental problem - unity doesn't seem to work at all when compiled from src and run in vbox ;)16:30
bschaefero well16:30
bschaeferpstolowski, if you need me to test the branch out, give me a link :)16:30
pstolowski(runs fine in vbox otherwise)16:30
bschaeferhm so it crashes for you when compiled from src? Or is it just not running the compiled version?16:31
pstolowskibschaefer: I think it crashes (empty desktop on login). here is my branch lp:~stolowski/unity/no-click-preview-for-apps16:34
bschaeferpstolowski, awesome, Ill take a look :)16:34
bschaeferbut thats strange :(16:35
pstolowskibschaefer: million thanks, let me know if it works; in the meantime I'm recompiling again16:35
bschaeferpstolowski, cool, and thank you for the quick fix :)16:35
pstolowskibschaefer: it's not.. I've a very special setup :P where I compile in chroot, and deploy and test in vbox16:35
pstolowskibschaefer: so perhaps I messed something. this normally works with less complicated stuff, such as backend ;)16:36
bschaeferpstolowski, geez, that sounds a bit complicated to tests a branch :)16:36
bschaeferpossibly, we shall see, im recompiling :)16:36
mhall119tedg: does libhud-qt produce any API documentation?16:36
pstolowskibschaefer: btw, this fix needs to be tested in applications lens and in home scope (apps category)16:36
=== dandrader is now known as dandrader|lunch
bschaeferpstolowski, cool, tested that we can tell the difference in the home lens?16:37
* bschaefer hasn't looked at the diff yet16:37
pstolowskibschaefer: yeah, since home scope aggregates, so I had to do an extra check16:37
tedgmhall119, I think it did, but it's deprecated16:38
tedgmhall119, It'll be replaced by the Unity Actions API that Wellark is working on.16:38
bschaeferpstolowski, cool, ill test both :16:38
bschaefer:)16:38
mhall119tedg: Wellark: do we have a timeframe for when that will be ready?16:38
pstolowskibschaefer: thanks :)16:40
bschaeferpstolowski, looks like ill need a the newest versions of libunity?16:40
tedgmhall119, Soonish, not sure on the exact details.16:41
bschaeferopps, for some reason libunity-dev was uninstalled?16:41
mhall119tedg: in time for the SDK beta?16:41
tedgmhall119, I'm not sure when that is, so I can't comment :-)  My guess would be that Wellark does and would be trying for it.16:41
pstolowskibschaefer: you should be fine on saucy as long as you're up-to-date16:41
bschaeferpstolowski, the only thing I seem to be missing: --   package 'indicator3-0.4>=12.10.2' not found16:42
bschaeferwhich sounds a bit high number wise...16:42
=== alan_g is now known as alan_g|reboot
bschaefergeez, its not installed ... i've must have messed up my system a bit :)16:43
bschaeferpstolowski, sorry, doing some mir/1.14 xserver stuff and things just get removed automatically :). Now its building!16:44
=== jhodapp|lunch is now known as jhodapp
pete-woodsmterry: hi, just tried out your libusermetrics branch with the real usermetrics, and it works :)16:49
mterrypete-woods, nice  :)16:49
pete-woodsmterry: https://pastebin.canonical.com/93700/ it did need this patch to make the infographic animations work nicely, though16:49
pete-woodsmterry: I don't think they have ever been run with just a small amount of data before16:50
pstolowskibschaefer: yay, got it working; I've applied my patch ontop of src deb from saucy, rather than trunk16:50
pete-woodsmterry: when I say nicely, I mean for the second month data to appear at all16:50
bschaeferpstolowski, code wise the fix looks good, but it might be nice as somepoint to add this to the scope API, and say if any scopes wants to change left click behaviour it could make it easier...16:50
bschaeferpstolowski, and awesome!16:51
pstolowskibschaefer: funny you mentioned it. this is exactly the discussion we had this morning on standup. and we decided to opposite, to prevent that :)16:51
bschaeferpstolowski, well perfect :)16:52
bschaeferjust thought I should at lease mention that16:52
bschaeferpstolowski, also would you or mhr3 be able to look at these reviews?16:53
bschaefer<jbicha> https://code.launchpad.net/~jbicha/unity-lens-video/build-with-valac/+merge/16819916:53
bschaefer<jbicha> https://code.launchpad.net/~jbicha/unity-scope-home/build-with-valac/+merge/16827316:53
bschaeferthey look super simple...16:53
bschaeferbut i wasnt sure if you were wanting to just stick to 0.18 for valac or not16:53
pstolowskibschaefer: yes, I'll check them tomorrow morning, eod for now if that's ok?16:53
bschaeferpstolowski, sounds good. Have a good end of your day!16:54
bschaeferill email you letting you now how the branch goes haha16:54
bschaefer(even though yours works)16:54
pstolowskibschaefer: I think we want to avoid 0.20 for now, but will check with mhr316:54
bschaeferpstolowski, cool, thanks and cya :)16:54
jbichabschaefer: I believe vala-0.18 will be dropped from Ubuntu before 13.10 is released16:55
mhr3bschaefer, scopes should be fine with 0.20, we don't want libunity with 0.20... yet16:55
pstolowskibschaefer: thanks for looking at my branch, mind doing a proper review once I MP? (need to look at AP tests)16:55
bschaeferpstolowski, yes, I can do that :)16:55
pstolowskimhr3: ah, ok, good then16:55
pstolowskibschaefer: thanks16:55
bschaefernp!16:55
bschaefermhr3, which should make those scopes branches fine to merge16:56
bschaefermhr3, I just didn't want to merge that in without asking anyone :)16:56
mhr3bschaefer, indeed, if you have a moment and want to give them a spin when they're built with 0.20, feel free to ;)16:57
bschaefermhr3, yeah, I can build it with 0.20 and test it out16:57
bschaeferjbicha, thanks for the branches :)16:57
=== alan_g is now known as alan_g|EOD
Saviqfginther, btw, all the jobs for lp:unity/8.0 can be dropped17:36
fgintherSaviq, ok, I'll disable it17:38
Saviqfginther, btw, panda-4 seems flaky https://jenkins.qa.ubuntu.com/job/unity8-saucy-armhf-autolanding/17/console17:39
Saviqhttps://jenkins.qa.ubuntu.com/job/unity8-saucy-armhf-autolanding/16/console17:39
fgintherSaviq, thanks, we also just noticed that and took it out of the pool17:42
Saviqfginther, cheers17:42
mzanettiSaviq: is libunity-api to be released manually?17:51
dednickdandrader|lunch: review done for panel draghandle. few fixes needed.17:54
mzanettiSaviq: ah no... seems daily releasing has started on that one. So should be there tomorrow17:55
=== kdub is now known as kdub^lunch
=== ari_ is now known as arsson
Saviqmzanetti, if you're still here, any idea what changed to cause https://jenkins.qa.ubuntu.com/job/unity-phablet-qmluitests-saucy/337/?18:23
Saviqsuddenly we have 7 tests failing regularly in CI :/18:23
mzanettiSaviq: weird... no clue right now18:24
=== dandrader|lunch is now known as dandrader
Saviqmzanetti, ok, we'll tackle tomorrow18:25
=== kdub^lunch is now known as kdub
Saviqfginther, we actually have some failures in qmluitests that started this afternoon19:24
Saviqfginther, so no point in approving stuff that failed on the panda19:25
Saviqfginther, as it will fail there anyway, we'll tackle that early tomorrow19:25
fgintherSaviq, ok.19:25
kgunnmterry: ping19:26
=== dandrader is now known as dandrader|afk
kgunnmterry: nvmd...i'll catch you in 3019:26
=== dandrader|afk is now known as dandrader
kgunnveebers: just trying to get off the phone....20:02
veeberskgunn: ack20:02
kgunna minute or 220:02
mhall119hey guys, after an apt-get dist-upgrade on Saucy and a reboot, my indicators are out of order21:08
mhall119http://ubuntuone.com/2P9GfwQekCCN9uZFItClbS21:09
mhall119is that intentional or a bug?21:09
mhall119I'm having flash-backs to my Gnome 2.x days21:10
=== om26er_ is now known as om26er
=== salem_ is now known as _salem
dandradermhall119, I have the same issue21:41
fginthercyphermox, is this ready to approve and deploy? https://code.launchpad.net/~mathieu-tl/cupstream2distro-config/enable_indicator-network/+merge/17264321:57
cyphermoxfginther: yeah21:57
fginthercyphermox, great! done21:57

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