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

=== Guest95146 is now known as mfisch
=== mfisch is now known as Guest57100
=== duflu_ is now known as duflu
=== jono is now known as Guest27980
Saviqmzanetti, I wonder if we should actually bump UNITY_API_MINOR in your branch... it's not like we're affected/affecting anything with the launcher api07:51
mzanettiSaviq: I don't mind tbh... Just did it because you asked for it07:51
Saviqmzanetti, no, not that07:52
greybackmorning folks07:52
mzanettihey greyback. good morning07:52
Saviqmzanetti, I meant here http://bazaar.launchpad.net/~unity-team/unity-api/trunk/view/head:/include/unity/shell/launcher/CMakeLists.txt07:52
Saviqhey greyback07:53
greybackmzanetti: Guten Morgen! I had a 1-on-1 German lesson yesterday. Aside from poor grammar, I'm not terrible!07:53
mzanettiSaviq: you mean that set(VERSION 2) ?07:53
greybackSaviq: eh oh07:53
mzanettihaha07:53
Saviqmzanetti, yup07:53
Saviqgreyback, eh oh what?07:53
mzanettiSaviq: so what now? bump it or not? (I still don't mind at this stage of the project)07:54
Saviqmzanetti, yeah, bump the launcher-specific one, not UNITY_API_VERSION07:54
mzanettiack07:54
Saviqs/VERSION/MINOR/07:54
Saviqmzanetti, we're going for the app id in the LauncherModelInterface, btw?07:56
greybackSaviq: https://www.youtube.com/watch?v=1wyW7uaXV8E07:57
Saviqgreyback, ah07:57
Saviqgreyback, it was translated here into "hey-o" or similar07:58
Saviqand it's the most psychedelic thing EVER07:58
greyback:D07:59
mzanettiSaviq: yeah sure.07:59
Saviqall of Teletubbies look like the whole thing is a recording of someone's acid trip...07:59
* mzanetti remembers how he had to spend 3 months in a room with like 200 teletubbies08:05
Saviqmzanetti, that explains things...08:05
* mzanetti starts crying08:05
* greyback hands mzanetti some acid, thinks it might help08:10
mzanetti:D08:13
Mirvdoes someone know the Qt bug Florian referred to me? https://bugreports.qt-project.org/browse/QTBUG-30246 "QML Text Component ignores alpha component of color property"08:19
Mirvcould you test a build with two patches for it with apt-add-repository ppa:canonical-qt5-edgers/qt5-beta2 on device?08:19
MirvKaleo_: I'm trying to find someone to test those patches, now built successfully ^08:21
Mirvbut someone more likely awake around now :)08:21
SaviqMirv, I'll try it out in 10, ok?08:23
MirvSaviq: excellent, thanks08:24
Mirvfiled a bug https://bugs.launchpad.net/ubuntu/+source/qtdeclarative-opensource-src/+bug/1198830 for it08:24
ubot5Ubuntu bug 1198830 in qtdeclarative-opensource-src (Ubuntu) "QML Text Component ignores alpha component of color property " [Undecided,New]08:24
MirvI'm not seeing anything wrong on my device with that updated, but would like to have another confirmation. preferrably by someone who knows where the bug would be seen fixed.08:25
dpmmorning Saviq. We've got a bug in the calculator app that is caused by the shell killing the process on exit without notification. I'm not sure what the best project to add a bug task for might be. Do you have any suggestions? It's bug 118829208:31
ubot5bug 1188292 in Ubuntu Calculator App "Saved calculations are lost when the app is closed" [Undecided,Confirmed] https://launchpad.net/bugs/118829208:31
Saviqdpm, it's something the app lifecycle will handle - apps will be given some grace period to save their data / serialize their state08:32
Saviqdpm, after that they're stopped and the process is not guaranteed to be started again08:32
dpmSaviq, gotcha. Do you know where the bugs for app lifecycle are tracked? Is it platform-api?08:34
Saviqdpm, I don't think there's any place yet, as the whole thing isn't built yet08:34
Saviqgreyback, any ideas ↑08:34
Saviqmzanetti, here's a breakdown of a qmluitests run http://pastebin.ubuntu.com/5854765/08:35
Saviqmzanetti, so it's 16 mins to build, 3 mins to test08:36
mzanettiSaviq: yeah...08:36
mzanettiSaviq: there are lots of things that could be tweaked. but it takes time I don't really have right now to do such stuff08:37
Saviqmzanetti, I'm not trying to drop this work on oyu08:38
greybackdpm: please log in "qtubuntu" package for now08:39
Saviqveebers, you'll be interested, too ↑↑ (pastebin)08:41
Saviqveebers, it's a breakdown of a unity-phablet-qmluitests-saucy job08:42
Saviqveebers, which suggest we're wasting time building over and over again08:42
SaviqIMO we should only build once per CI run / arch08:43
Saviqnow we're building 2 times in -ci, once in mediumtests-builder, once in qmluitests, and for qmluitests we're even building in VM, which takes 16 minutes instead of 6 (unity8-saucy-i386-ci)08:46
* greyback kills unity-panel-service again to stop it using a whole cpu core08:51
didrocksgreyback: do you have indicator-network or indicator-clients installed?08:58
Saviqdidrocks, indicators-client is a runtime dep of unity8, so yeah, probably08:59
didrocksSaviq: i wonder if it doesn't try to communicate with u-p-s and maybe there is a mismatch09:00
SaviqMirv, +109:02
=== zequence_ is now known as zequence
greybackdidrocks: network.09:04
greybackdidrocks: that the culprit?09:04
greybackby which I mean indicators-client-plugin-network09:04
didrocksgreyback: try removing it and restarts, it thowed u-p-s crazy and that's why I remove it from daily release a month and half ago09:04
didrocksnot sure if it was fixed09:04
dpmthanks Saviq, greyback, added a task for qtubuntu for that bug ^09:05
greybackdidrocks: thanks for the tip!09:05
greybackdpm: thank you09:05
didrocksgreyback: please keep me posted09:05
MirvSaviq: thanks09:05
dednickmzanetti: do you know what autopilot does with the console output when it's running?09:12
mzanettidednick: it should print it at the beginning of the results output09:12
dednickmzanetti: only error's, or all output?09:14
mzanettidednick: all output09:14
mzanettidednick: doesn't work?09:14
dednickmzanetti: hm. not getting anything09:14
dednickhttps://jenkins.qa.ubuntu.com/job/generic-mediumtests-runner-saucy/711/testReport/indicators_client.tests.test_power/TestDisplayMenus/test_auto_bright_switch_with_mouse_/09:14
dednickmzanetti: ^09:14
mzanettidednick: hmm... process-stdout: empty attachment09:15
mzanettidednick: strange... should be in there...09:15
dednickmzanetti: i've never really figured out how to capture qt output though09:15
mzanettidednick: also, the "Loading testability driver" which you can see in the process-stderr is actually a qDebug()09:15
dednickmzanetti: it doesnt seem to go to stdout09:16
mhr3didrocks, did the rebuild work on friday?09:25
didrocksmhr3: no, we still had issues, but discovered it was some regressions due to other components in proposed09:26
mhr3didrocks, which components?09:26
didrocksmhr3: so, it has been rerun without those09:26
didrocksmhr3: was quite late, didn't do the dpkg diff, but we can do those now :)09:26
mhr3didrocks, i'm just wondering if one of those components was something dbus-y09:27
didrocksmhr3: http://10.97.0.1:8080/job/autopilot-saucy-daily_release/345/, with all updates09:29
didrocksmhr3: http://10.97.0.1:8080/job/autopilot-saucy-daily_release/347/ with only the stack + xorg09:29
didrocksmhr3: and the diff: http://paste.ubuntu.com/5854882/09:30
mhr3ETOOMANY, we really need nice way to get some general dbus stats on AP runs09:32
didrocksmhr3: the thing we can be sure of is that we need to find where the regression comes from, or we'll stuck now for unity09:32
mhr3didrocks, indeed09:32
didrocksat least, we are lucky to get such a diff to know that something here makes everything screwing :p09:33
didrocksmhr3: but TBH, apart from the new udev from systemd…09:36
didrocksthe only diff with patch refresh is this Add 0000-upstream-keymaps.patch: Backport latest keymap fixes from09:36
didrocks    upstream, as we won't be able to update to newer upstream releases anytime09:36
didrocks    soon09:36
mhr3didrocks, hmm, yea seems to be most likely09:38
mhr3i mean udev09:39
didrocksyep, as soon as we'll have anything merged into unity, we'll get the confirmation, but I'm a little bit afraid that will be the next issue09:40
mhr3could be an interaction of udev + something and that something might get fixed :)09:41
mhr3so called wishful thinking development :P09:41
didrocksmhr3: the diff from earlier in the morning on Friday shows the same: http://paste.ubuntu.com/5854902/09:42
mhr3theoretically could be apparmor too if it decided to disallow dbus traffic by default :)09:43
* greyback bbiab09:45
didrocksmhr3: from the changelog, it's only related to sdk + apache transition09:47
mhr3mzanetti, could you look at https://code.launchpad.net/~mhr3/unity8/use-dee-filtermodel/+merge/171846/comments/38781409:51
mzanettimhr3: done09:54
mhr3mzanetti, thx09:56
mhr3Saviq, top ack on https://code.launchpad.net/~mhr3/unity8/use-dee-filtermodel/+merge/171846 pls?09:56
CimiSaviq, so the meeting I supposed to have last friday moved today at 310:04
CimiSaviq, I had a look at unity, so far I think we should make10:05
Cimitile, ratingstarts, dash bar, panelseparatorline, maybe notifications10:05
Cimia little bit themeable10:05
Cimimaybe panel10:05
CimiSaviq, there's also to say that we might want to move ratingstars into the sdk10:05
Cimiand some js utils files10:06
SaviqCimi, yeah, stars should probably have light & dark mode, so yeah, SDK and theming makes sense here10:20
Saviqmhr3, done10:20
dednickSaviq: ping10:44
Saviqdednick, pong10:44
tsdgeosSaviq: greyback: i'm going to use http://paste.kde.org/~tsdgeos/791234/ to highlight the need of being able to tell a gridview the parts of it that are exposed and thus should care about it's delegate creation, what you think?10:44
dednickSaviq: hi. apparently some of the new indicator file formats are in already, and it's causing conflicts with indicators-client because it's expecting a "phone" profile. But the new indicators dont have that yet.10:45
dednickSaviq: i think this is the problem with the AP tests10:45
Saviqtsdgeos, yeah, looks good10:46
dednickSaviq: i think the test system may be installing both indicator-power and indicator-battery10:46
greybacktsdgeos: +1 from me10:47
Cimimzanetti, did you have time to look at the calendar?10:49
Saviqdednick, having a look10:49
dednickSaviq: how does one actually have a look at the test env?10:50
Saviqdednick, doesn't look like it https://jenkins.qa.ubuntu.com/job/generic-mediumtests-runner-saucy/711/console10:50
Saviqdednick, I just go through that log and see what it installs10:50
dednickSaviq: what about the default installed packages?10:51
mzanettiCimi: did you update the merge proposal?10:51
Saviqdednick, there shouldn't be any10:51
Cimimzanetti, y10:51
Saviqdednick, it should be a minimal installation10:51
mzanettiCimi: so no, I didn't yet. but I have time now10:51
Saviqdednick, if you want a look, mzanetti can get you to ssh in10:51
dednickSaviq: i see. in which case indicator-battery isn't being installed anyway10:52
dednickSaviq: do i need to add that to the recommended packages for unity8?10:52
Saviqdednick, yeah, no of them are installed - if they should be, we should Recommend them10:52
mzanettiSaviq: https://code.launchpad.net/~mzanetti/unity-api/launcher-api-pinning/+merge/173064/comments/38786410:52
Saviqdednick, or for unity8-autopilot10:52
mzanettiSaviq: and this is updated accordingly too: https://code.launchpad.net/~unity-team/unity8/launcher-backend/+merge/17318910:52
Saviqmzanetti, k, will look10:52
mzanettidednick: let me know if you need some support with jenkins10:53
mzanettiCimi: looking at it now10:53
Saviqdednick, but yeah, probably makes sense to add them to Recommends for unity810:53
Saviqdednick, instead of indicators-client and the plugins10:53
Saviqmzanetti, shouldn't ::pin(appid) be ::pin(appid, index) already? maybe with a default of -1 in case it's not passed?11:04
mzanettiSaviq: hmm... ok. /me changes...11:05
mzanettigive me a sec11:05
Saviqmzanetti, k11:05
Saviqmzanetti, "Request removal an item from the model." +of11:05
=== MacSlow is now known as MacSlow|lunch
Saviqmzanetti, I wonder if "applicationId" would be better, too... but not really sold - we're talking "apps" everywhere anyway11:07
Saviqmzanetti, also, triggerQuickListAction - would triggerAction be enough do you think?11:17
Saviqmzanetti, also, I think we used "invokeAction" in Notifications11:17
* Saviq looks11:17
Saviqmzanetti, we have NotificationInterface::actionInvoked(QString id)11:19
mzanettiSaviq: yeah. the difference is that each Notification has a actionInvoked() slot while in the launcher there's only one for all. So I kinda prefer more descriptive names11:20
mzanettiI can change it to quickListActionInvoked() tho11:20
Saviqmzanetti, yeah, let's keep them as consistent as possible11:21
mzanettiack11:21
mzanettiSaviq: both pushed11:30
Saviqmzanetti, cheers11:30
Saviqmzanetti, any idea about http://pastebin.ubuntu.com/5855131/ ?11:33
mzanettiSaviq: apt-get install autopilot-touch11:33
mzanettiSaviq: don't ask me why that is not integrated or a dependency or a recommends...11:34
Saviqmzanetti, same thing, autopilot-desktop was installed11:34
Saviqmzanetti, installed autopilot-touch, same, removed -desktop, same11:34
mzanettiSaviq: you need both for unity8 I think11:35
Saviqmzanetti, still, same11:35
mzanettianyways... if that doesn't fix it... let me think11:35
mzanettiSaviq: hmm. it really looks like he can't create the touch input stuff11:36
Saviqmzanetti, I'm doing a minimal VM for autopilot, maybe I'm missing python-evdev or something11:37
mzanettiSaviq: python-evdev should be a dependency of autopilot-touch I think11:37
Saviqmzanetti, yeah, it's there11:37
mzanettiCimi: fails to start because of ItemStyle.class assignments11:41
Saviqmzanetti, you need to update launcher-backend for the new interface11:45
mzanettiSaviq: on it11:45
Saviqmzanetti, k11:45
mzanettiSaviq: wait...11:45
mzanettiSaviq: launcher-backend?11:45
Saviqmzanetti, https://code.launchpad.net/~unity-team/unity8/launcher-backend/+merge/17318911:45
mzanettiSaviq: ah... the branch11:45
mzanettiSaviq: I thought you mean the real backend11:45
mzanettiSaviq: but yeah, I'm on it right now11:46
Saviqmzanetti, k11:46
Saviqmzanetti, while you're at it:11:46
Saviq78+ unity8-private | unity8-launcher-impl,11:46
Saviq79+ unity8-launcher-impl-0.2,11:46
Saviq87+Provides: unity-launcher-impl,11:46
Saviq88+ unity-launcher-impl-0.211:46
Saviqmzanetti, this needs to be the same (unity-launcher-impl)11:46
Saviqmzanetti, and the -0.2 part needs to equal the VERSION you set in the branch for unity-api11:47
mzanettiSaviq: I'm not entirely sure what this should do11:53
mzanettiSaviq: can you explain please?11:54
Saviqmzanetti, unity-launcher-impl-2 == this package provides the implementation of the Launcher API version 211:54
Saviqthe Launcher API version is defined in unity-shell-launcher.pc (we'll automate extracting it later)11:54
Saviqmzanetti, Depends: unity8-private | unity-launcher-impl, unity-launcher-impl-2 means:11:55
Saviqwe depend on *an* implementation of the Launcher API version 2, of which the default implementation is in unity8-private now11:55
mzanettiSaviq: hmm.. I would read it as "depend on unity8-private or unity-launcher-impl AND unity-launcher-impl-211:56
Saviqmzanetti, yeah, that's the same, really11:57
mzanettiSaviq: why not just unity-launcher-impl-2?11:57
mzanettiSaviq: if unity8-private provides that, that one will be selected11:57
Saviqdidrocks, why was that ↑?11:57
Saviqdidrocks, it's some dpkg trickery11:57
Saviqmzanetti, ↑11:57
=== hikiko is now known as hikiko|lunch
didrockslet me backlog to get the context :)11:59
mzanettididrocks: the question is just why to use "unity8-private | unity-launcher-impl, unity-launcher-impl-2" instead of just "unity-launcher-impl2"12:00
=== alan_g is now known as alan_g|lunch
mzanettididrocks: given that unity8-private provides unity-launcher-impl-212:00
didrocksmzanetti: to ensure we are getting the right version of the API installed12:01
didrocksso that you can't update unity8-private for instance12:01
didrocks(implementing a new version of the API)12:01
didrockswithout the package that adapats on this new API12:01
didrocksotherwise, you would be able to update unity8-private for instance12:01
Kaleo_Mirv: thanks, did you find somebody?12:02
didrockswithout the shell which is compatible with the new "api 2" of unity-private12:02
mzanettididrocks: hmm.. ok... still seems a bit weird but I gess its just how it works12:03
mzanettiSaviq: updated all branchtes btw12:03
Saviqdidrocks, you wouldn't be able to upgrade unity8-private, if it only provided unity-launcher-impl-312:04
didrocksmzanetti: it's just a question of enforcing the right dep :)12:04
Saviqdidrocks, and unity8 depended on unity-launcher-impl-212:04
didrocksSaviq: right, hence the ", unity-launcher-impl-2"12:04
Saviqdidrocks, I think the question is why do we need "unity-launcher-impl"12:04
didrocksah12:04
mzanettiyeah12:04
didrocksIIRC, you told that this launcher implementation can be implemented by another component, right?12:05
didrocksthan unity8-private12:05
mzanettididrocks: could be in the future, yes12:05
Saviqdidrocks, yes12:05
didrocksso you need to be able to either install unity8-private12:05
didrocksor this alternative implementation12:05
Saviqdidrocks, yeah, but why not "unity8-private | unity-launcher-impl-2" directly?12:05
* mzanetti still doesn't see why just "unity-launcher-impl-2" wouldn't be enough...12:06
Saviqdidrocks, the other component is still required to Provide: unity-launcher-impl-$VERSION12:06
didrocksSaviq: otherwise, you loose the "force having the same implementation version"12:06
didrocksthat we just discussed12:06
didrocksso, let's say you update unity8-private to unity-launcher-impl-312:06
didrocksthen, we release that12:06
didrocksI just apt-get update and apt-get install unity8-private12:06
didrocks"unity8-private | unity-launcher-impl-2" is matched then12:07
Saviqdidrocks, right!12:07
didrocksand I'm screwed as I have and unity-launcher-impl-3 implementation for unity8-private12:07
mzanettiyeah.... so why use unity8-private all12:07
didrocksand a unity-launcher-impl-2 for unity812:07
mzanettiat all12:07
didrocksmzanetti: you still need to "hint" you want to install one12:07
Saviqmzanetti, it can't only be a virtual one, right, didrocks?12:07
didrocksvirtual packages are not installed12:07
didrocksSaviq: yep :)12:08
Saviqthat12:08
mzanettiahhh... ok... that was the missing link12:08
mzanettiok. everything clear now12:08
didrocksmzanetti: you can't install a virtual package, which one to pick if multiples of them? :)12:08
Saviqdidrocks, thanks12:08
didrocksyw!12:08
Saviqdidrocks, random ;)12:08
didrockswe all know it always work! ;)12:08
mzanettididrocks: I would have assumed that all providers of unity-launcher-impl-2 would show up and update-alternatives or the like would ask me which one I want12:09
Saviqmzanetti, update-alternatives is unrelated :)12:09
Saviqmzanetti, since you can't have two of unity-launcher-impl-2 installed at the same time12:09
Saviqmzanetti, as they conflict12:09
mzanettilike for example phonon backends12:10
Saviqmzanetti, and yeah, that would happen12:10
Saviqmzanetti, but then you couldn't just go "apt-get install unity8"12:10
mzanettiSaviq: right12:10
didrocksmzanetti: doesn't work well when you try to spin an iso :)12:10
Saviqmzanetti, 'cause you'd have to go "apt-get install unity8 unity8-launcher-implementation unity8-notifications-implementation ..."12:11
Saviqmzanetti, the idea is that we have the default impl12:11
mzanettiok... I start to understand reasonings behind those things12:11
Saviqmzanetti, but the seed can override12:11
Saviqand we Depend: default | some_other, version_x12:12
MirvKaleo_: yes, saviq tested it, I now proposed those for upload together with the other merge I mentioned12:13
Kaleo_Mirv: great12:15
Kaleo_Mirv: do you know how long it takes from now to having it in the ouch image?12:16
Kaleo_+t12:16
didrocksSaviq: exactly, the default is always the first one that apt will pick if we don't have one already seeded/installed12:17
MirvKaleo_: seb128 promised to upload it already earlier, so I believe we can ping him also now on this updated proposal https://code.launchpad.net/~timo-jyrinki/ubuntu/saucy/qtdeclarative-opensource-src/merge_continued/+merge/17344812:19
MirvKaleo_: it'd be in tomorrow's builds with luck then12:20
Cimimzanetti, comment it12:22
mzanettiCimi: yeah. I removed it from 5 places or so until it starts up12:22
Cimimzanetti, it's weird cause I remember to comment everything here locally...12:22
seb128Mirv, yeah, it's on my todo but I got busy, I wouldn't say no if didrocks or the day patch pilot would handle it ... will try to have a look today otherwise12:22
mzanettiCimi: but then the calendar just says white12:22
didrockssame, I'll try to get to it if I get a chance12:23
mzanettiCimi: oh... I had trunk for some reason... works with your branch12:23
Kaleo_Mirv: great12:23
Cimimzanetti, ah ok :)12:23
Mirvyeah, I'd have qtbase and now qtpim uploads tested and ready as well, but I'll wait until someone gets to that qtdeclarative first :)12:29
=== MacSlow|lunch is now known as MacSlow
=== hikiko|lunch is now known as hikiko
=== _salem is now known as salem_
Saviqtvoss, TheMuso, I only just noticed the invite, won't be able to be there before 15 mins12:58
tvossSaviq, ack :) themuso, you about?12:58
=== alan_g|lunch is now known as alan_g
=== cking_ is now known as cking
tvossmterry, ping13:13
mterrytvoss, i just guessed on a time for you13:13
mterrytvoss, should have thought of pinging you first  ;)13:13
tvossmterry, the time is okay-ish, but: a day ahead would help me :) Can we move the meeting to Tuesday?13:14
mterrytvoss, sure13:14
tvossmterry, thx :)13:16
tvossSaviq, TheMuso does not seem to be around, will reschedule the meeting13:16
=== jhodapp|afk is now known as jhodapp
Saviqguys, I gotta skip standup, got a guy coming to fix my cool13:23
paulliumhr3: I found that there's no resulttype in your branch? How do I get that value?13:30
paulliumhr3: should I add that by myself in categoryresults.cpp?13:33
mzanettidednick: standup13:33
mzanettiSaviq: will you join?13:33
greybackCimi: standup?13:34
Saviqmzanetti, <Saviq> guys, I gotta skip standup, got a guy coming to fix my cool13:34
mzanettinic-doffay: standup13:34
greybacknic-doffay: standup?13:34
Saviqgreyback, nic's out13:34
greybackSaviq: ta13:34
greybackkgunn: any link to XMir debug doc?13:38
Saviqdednick, conflict on indicators-client branch13:45
kgunngreyback: https://code.launchpad.net/~kgunn72/mir/add_xmir_debug_guide/+merge/17326013:46
greybackkgunn: thanks13:46
kgunngreyback: only a start....if you have something to add...feel free :)13:46
dednickSaviq: ta13:48
mhr3paulliu, hardcode it to 0, there's no need for dash nor scopes to know13:53
greybackkgunn:  noticed a few typos, hope you don't mind13:55
* greyback really needs to eat something today13:55
kgunngreyback: aboslutely...let me have it13:56
greybackkgunn: added as MR comment.13:56
kgunngreyback: great...go eat13:56
* greyback eating13:57
=== Guest57100 is now known as mfisch
tvossSaviq, ping14:00
=== mfisch is now known as Guest86601
Saviqtvoss, pong14:42
* Saviq has a new mobo and heatsink... still hot as hell...14:42
tsdgeosSaviq: get some AC ¿ :D14:52
Saviq:P14:52
Saviqdednick, http://bazaar.launchpad.net/~unity-team/unity8/indicators-client/revision/101 should be a separate MR against trunk15:05
dednickSaviq: yeah. i thought it probably should. i'll revert it15:06
Saviqmzanetti, yeah, leave it in15:30
Saviqmzanetti, just noticed you took it off the MR, last response from me just went out15:31
mzanettiSaviq: no, not on purpose... just clicked "reply" instead of "reply all"15:31
Saviqmzanetti, k15:31
Saviqmzanetti, you're actually *clicking*? ;)15:31
mzanettihaha15:31
mzanettiyes, sometimes I do15:31
mzanetti:D15:31
Saviqmy left hand is just used to Ctrl+Shift+r already15:32
mzanettihmm... its actually even easier in kmail... its just "a"15:33
* mzanetti tries to remember15:33
nic-doffaySaviq, re the scrolling container. Would it make sense to make this a separate SDK or something specific to the option selector? If it's the latter I can just move it out...15:38
nic-doffay*separate SDK component.15:39
Saviqnic-doffay, it shouldn't be specific for the option selector, it should be generic for any component inheriting the Expandable15:41
Saviqnic-doffay, which right now in the SDK would only be the Option Selector15:41
Saviqnic-doffay, but the messaging entries in the indicators would be another15:41
nic-doffaySaviq, cool, I'll chat further to the sdk guys about the specifics.15:41
Saviqmzanetti, don't we need something like "type" role in QuickListModelInterface?15:50
Saviqmzanetti, like for the separator at least15:50
mzanettiSaviq: afaik we won't have such things in the first iteration. Thats because I removed all those properties again15:51
Saviqmzanetti, not even separators? please make sure that's the case15:51
mzanettiSaviq: however, my proposal had a groupId15:52
Saviqdednick, https://jenkins.qa.ubuntu.com/job/generic-mediumtests-runner-saucy/726/? same error again15:52
mzanettiSaviq: I think that could paint separators between different groups15:52
Saviqdednick, it did install indicator-battery this time15:52
pstolowskibschaefer: morning!15:53
bschaeferpstolowski, hello!15:53
Saviqmzanetti, I think that'd not be flexible enough15:53
mzanettiSaviq: yeah... probably right... just thought a bit more about it15:53
mzanettiSaviq: so yes, in the long run we'd need that15:53
pstolowskibschaefer: thanks for another look at my MP! can you top-approve today?15:54
Saviqmzanetti, and IIRC that's how it is now - you can just add a separator anywhere you like15:54
bschaeferpstolowski, yup! any changes?15:54
* bschaefer digs up MP15:54
pstolowskibschaefer: nope. https://code.launchpad.net/~stolowski/unity/no-click-preview-for-apps/+merge/17278715:54
Saviqmzanetti, k, let's get to it when we need i15:54
Saviqt15:54
bschaeferpstolowski, awesome, approved :)15:54
pstolowskibschaefer: thanks!15:55
bschaefernp!15:55
Saviqmzanetti, is there fewer entries in the launcher now?15:56
pstolowskididrocks: hey, the most hated behavior about to be disabled, it's ready to land ^ :)15:57
dednickSaviq: hm. thats weird. succeeded earlier.15:57
mzanettiSaviq: hmm... shouldn't be15:57
dednickSaviq: maybe a instability rather than "not working"15:58
Saviqdednick, it didn't run the indicators_client suite15:58
didrockspstolowski: \o/15:58
Saviqdednick, we only enable that suite manually (with "Rebuild") now15:58
dednickSaviq: ah15:58
didrockspstolowski: autopilot tests are compatible?15:58
Saviqdednick, as otherwise we'd just fail15:58
mzanettiSaviq: no, its the same as the released version15:58
mzanettiSaviq: the new animation makes it look a bit less indeed15:58
Saviqmzanetti, k, was tricked15:58
Saviqmzanetti, ah there is a new animation15:58
Saviqmzanetti, like the fact that the last and first two items fold ~together?15:59
mzanettiSaviq: no... I mean the one with the fading out icons... as opposed the stack I had in the first revision15:59
Saviqmzanetti, right15:59
Saviqmzanetti, is it on purpose then that the first and last two items fold almost together?15:59
mzanettiSaviq: yep16:00
Saviqfeels inconsistent to me...16:00
Saviqbut hey16:00
mzanetti:D16:00
tsdgeosmeh, the range for delegate creation is harder than i expected16:01
tsdgeosdumb optimist me16:01
Saviqtsdgeos, :/16:01
tsdgeosSaviq: it's not ultra hard16:02
tsdgeosbut there's more parts of the code that expect that if nothing changed you don't need to care about culling/unculling the items16:02
tsdgeosand now i do16:02
Saviqtsdgeos, right16:02
tsdgeosi'm almost there16:02
mzanettiultra hard :)16:02
tsdgeosbut still need to fix a few small issues with my example qml code16:03
Saviqmzanetti, 178+ * Note: If an item is not contained in the launcher yet, calling this without an index16:04
Saviq179+ * will fail to pin the item.16:04
Saviqmzanetti, should just pin at the end16:04
Saviqmzanetti, is what happens now16:04
mzanettiSaviq: ah ok...makes sense16:04
* mzanetti fixes16:04
Saviqmzanetti, 186+ * Note: In case the according application is running, it will only16:05
Saviq187+ * disappear once the application is closed. In this case, this16:05
Saviq188+ * operation just unpins the item.16:05
dednickSaviq: runs from installer on my box. no idea why it's failing16:05
Saviqmzanetti, that's too much, I'd say - it should really only say "it might be delayed for different reasons"16:05
mzanettiSaviq: fixing16:05
Saviqdednick, do you want access to a machine today to test, or tomorrow?16:06
pstolowskididrocks: I think so, but you know whom to attack if there are new failues.. ;)16:06
dednickSaviq: yeah, that would be usefull16:06
didrockspstolowski: hehe, sure!16:06
=== tvoss is now known as tvoss|dinner
Saviqdednick, you want to try and tackle it today?16:07
dednickSaviq: yeah16:07
Saviqfginther, could we have a jenkins machine unhooked for dednick to debug some autopilot issues?16:07
fgintherSaviq, sure, i386 or amd64?16:08
Saviqdednick, ↑?16:08
dednickfginther: amd64 please16:08
Saviqdednick, fginther I'd suggest i386 :)16:08
Saviqdednick, as you can then get http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-saucy/817/artifact/work/output/*zip*/output.zip16:08
Saviqdednick, and test on the packages the builder came up with16:09
dednickSaviq: ok.16:09
Saviqdednick, btw, if you look at https://jenkins.qa.ubuntu.com/job/generic-mediumtests-runner-saucy/726/console16:09
Saviqdednick, the lines prefixed with +16:09
Saviqdednick, are usually the command lines run16:09
fgintherdednick, ps-saucy-server-i386-3 is all yours16:09
Saviqmzanetti, what's the IP to naartije? naartjie? naartje?16:10
Saviqdednick, so if you follow those, you should be able to get to where jenkins fails16:10
mzanettiSaviq: .0.616:11
Saviqdednick, so you can ssh to ubuntu@10.97.0.616:12
Saviqdednick, and then from that ps-saucy-server-i386-3 should resolve16:12
Saviqfginther, it's down, though, is it?16:12
Saviq$ ssh ps-saucy-server-i386-316:12
Saviqssh: connect to host ps-saucy-server-i386-3 port 22: No route to host16:12
dednickSaviq: ya, getting that as well16:13
Saviqmzanetti, for 186+ * Note: In case the according application is running, the actual removal of the16:16
Saviq187+ * item will be delayed16:16
fgintherdednick, now try it16:16
Saviqmzanetti, I meant that "in case .... is running" is a detail I don't think makes sense to mention there16:16
Saviqmzanetti, i.e. it's the backend's decision when will it remove the item16:16
mzanettiSaviq: ah. sorry16:16
mzanettiyeah, changing16:16
Saviqmzanetti, don't be16:16
dednickfginther: yep, working now16:17
mzanettiSaviq: ok with this: * Note: The actual removal of the item might be delayed in certain circumstances.16:17
Saviqmzanetti, +116:18
Saviqmzanetti, QuickListModelInterface::roleNames creates the list every time, too16:18
Saviqmzanetti, 289+} // unity16:18
Saviq290+} // shell16:18
Saviq291+} // launcher16:18
Saviqmzanetti, reverse16:19
Saviqmzanetti, why do we have MockLauncherModel::pin, ::requestRemove, ::quickListActionInvoked, ::findApp, if we're not testing it in lp:unity-api?16:21
Saviqmzanetti, I mean why do we have them implemented16:21
Saviqsame for MockQuickListModel16:21
mzanettiSaviq: there needs to be something so that the test if the function exists succeed. But yeah... the logic in the mocks is probably not that useful...16:22
Saviqmzanetti, yeah, just { } should be enough16:22
Saviqmzanetti, there's an indent issue in test_quicklist_model_roles_data() with the brackets16:22
Saviqmzanetti, and yeah, do we actually want test_pinning and test_remove there16:23
mzanettiSaviq: yeah... that would need to go if the mock code goes away16:24
Saviqmzanetti, I know it's tempting to put them there :)16:24
Saviqmzanetti, yeah, but those tests ideally should not test behaviours16:25
Saviqmzanetti, as they can be different per implementation16:25
mzanettiSaviq: yeah... agreed16:25
Saviqmzanetti, I will remove those from notifications probably16:25
=== boiko_ is now known as boiko
Saviqmzanetti, do you want to push it through today/tonight? or shall we wait until tomorrow morning just in case?16:26
mzanettiSaviq: just in case of what?16:27
Saviqmzanetti, in case they both don't merge before daily release takes over16:27
mzanettiSaviq: ah... hmm... if you think tomorrow would be safer, I'm fine with that...16:28
Saviqmzanetti, what I mean is "do you want to go away and have your evening or do we want to push it through and you'll be around in case something needs tweaking?"16:28
Saviqmzanetti, your call16:28
mzanettiSaviq: I'm fine here16:28
Saviqmzanetti, ok, so for unity-api I'm +1 after the last few tweaks16:30
Saviqmzanetti, same for launcher-backend it seems, just a few last tweaks remaining16:30
=== jhodapp is now known as jhodapp|lunch
mzanettiSaviq: ok. I think everything fixed and pushed16:32
Saviqmzanetti, ah, http://pastebin.ubuntu.com/5855854/16:34
mzanettidammit16:35
Saviqmzanetti, _data() indentation is inconsistent with brackets, but I'll let it slide ;)16:37
mzanettiSaviq: can fix it, np16:37
Saviqk16:37
mhr3Saviq, btw now that scopes have the formfactor prop, could you guys set it to somewhere?16:38
mhr3to something...16:38
Saviqmhr3, k, will do16:38
Saviqmhr3, it's on the Scope class, right?16:39
mhr3Saviq, yep16:39
Saviqmhr3, k16:39
mhr3Saviq, should affect the apps lens at least16:39
mzanettiSaviq: ok, now it should be ok...16:42
Saviqmzanetti, k, here's my last comment for launcher-backend https://code.launchpad.net/~unity-team/unity8/launcher-backend/+merge/173189/comments/38809416:45
Saviqmzanetti, I've approved unity-api, will be back in 1/2 hr to see what's what and make sure unity8 lands16:45
Saviqmzanetti, thanks!16:45
=== jhodapp|lunch is now known as jhodapp
fgintherdednick, still using that VM?18:49
=== tvoss|dinner is now known as tvoss|test
=== tvoss|test is now known as tvoss
dednickfginther: yep19:12
fgintherdednick, ok, just checking19:13
dednickfginther: i'll let you know when i'm done19:13
fgintherdednick, thx19:13
kgunnmterry: ping19:44
mterrykgunn, h19:46
mterryi19:46
kgunnmterry: hey...just curious...wrt https://wiki.ubuntu.com/Touch/Testing/Mir19:47
dednickfginther: ping19:47
kgunni know you are a N7 user19:47
kgunnany caveats there ?19:47
fgintherdednick, hello19:47
dednickfginther: hi. do you know how to turn on the debug logging in autopilot?19:47
kgunne.g. "this doesn't work on N7" :)19:47
mterrykgunn, yes, those are instructions I was following to try to get N7 to work, (and theoretically would let it work), but when I got to the end, nothing showed on screen after all19:48
fgintherdednick, 'autopilto run -v -v test-suite' I think19:48
mterrykgunn, so "this doesn't work on N7, but it should and I don't know why not"19:48
dednickfginther: thanks, that did it19:49
dednickSaviq: figured out the AP issue. the indicators-client app is starting at a negative x/y which is screwing with the pointer. Dont know why it's starting at that position though :(20:42
Saviqdednick, ugh!20:42
Saviqdednick, I thought the window was quite small in the videos...20:42
Saviqbut wouldn't have guessed20:43
dednickSaviq: yeah. me too. i've been screwing around with object introspection for the last few hours before i found that out.20:44
Saviqdednick, is the window set up the same way as in main.cpp for unity8?20:44
dednickexcept for the explicit size, i think so.20:46
dednickI'm trying it with explicit sizing and see how it goes.20:46
dednickSaviq: have to check it in detail some more20:46
dednickSaviq: AP specifies an explicit size for the window in unity8. Also the grid size.20:47
Saviqdednick, right, that might be what's helping (we need that for testing for different devices)20:47
Saviqdednick, but obviously it should work fine without20:47
dednickSaviq: i think unity8 is also started frameless.20:48
Saviqdednick, yeah, but that shouldn't be an issue20:48
dednickSaviq: but yeah. not sure why either would change anything. it's not position related, it's all about size.20:49
Saviqveebers, ping20:50
veebersSaviq: pong20:51
Saviqveebers, hey, we're trying to get a job like so https://jenkins.qa.ubuntu.com/job/generic-mediumtests-saucy/815/parameters/?20:51
Saviqveebers, working with two test suites (note there's unity8 and indicators_client)20:52
Saviqveebers, dednick just found out the window in indicators_client is laid out at negative geometry20:52
Saviqveebers, which makes the tests fail 'cause the input can't reach the right coords20:53
Saviqveebers, would you have time to investigate? or even have a ready-made solution?20:53
veebersSaviq: I can have a look today and see what I can find20:53
Saviqveebers, one thing dednick's not doing in the indicators_client suite is that he's not setting the geometry explicitly in autopilot20:53
Saviqveebers, which we're doing for unity8 (which tests fine20:54
veebersSaviq: sure, cheers that gives me a good starting place20:54
Saviqveebers, so that could probably be one solution - but then it'd be surprising if that's really required20:54
Saviqdednick, and you man go away20:55
veebersSaviq: I've been meaning to look closer at the geometry/setup of the tests, so this will be the start :-)20:55
Saviqveebers, yup, thanks20:55
veebersSaviq: nw20:56
dednickveebers, Saviq: ta20:57
veebersHi dednick :-)20:58
dednickveebers: Hi :)20:59
dednickveebers: just ran a test on jenkins, looks like setting explicit geo in qt fixes the issue20:59
dednickhttps://jenkins.qa.ubuntu.com/job/generic-mediumtests-runner-saucy/735/testReport/junit/indicators_client.tests.test_battery/TestDisplayMenus/test_auto_bright_switch_with_mouse_/20:59
dednickit got further20:59
veebersdednick: ah ok. I'll look into the setup and need for the geo etc. today20:59
dednickveebers: thanks21:00
=== salem_ is now known as _salem
Saviqveebers, one more question - I'm trying to build a minimal VirtualBox VM we could use for autopilot / UI tests that would not take your host over21:25
Saviqveebers, everything seems almost fine except that autopilot errors out with http://pastebin.ubuntu.com/5856659/21:25
Saviqveebers, any idea what I'm missing21:26
Saviq?21:26
Saviqveebers, I could get you a more complete and verbose log from autopilot if it'd help21:27
dednickSaviq: now indicators-battery-service is seg-faulting :)21:34
Saviqdednick, right21:34
dednickSaviq: which means i'm going to bed21:34
dednick:)21:34
Saviqdednick, go to sleep! veebers is having a look at the autopilot issue21:34
dednickSaviq: ok. the remaining issue i think is because indicators-battery-service is not running on jenkins (it crashes on startup), so the indicator menu isn't being populated.21:35
dednickveebers: ^21:36
Saviqdednick, you sorted out the geometry?21:36
Saviqdednick, by explicitly sizing in autopilot?21:36
dednickSaviq: setting it explicitly fixes, but not auto21:36
Saviqdednick, right21:36
dednickSaviq: the fix is in a debug branch at the moment. https://code.launchpad.net/~nick-dedekind/unity8/indicators-client.jenkins-debug/+merge/17356321:37
Saviqdednick, k21:38
dednickfginther: i'm done with the VM.21:38
fgintherdednick, thanks21:39
=== Trevinho_ is now known as Trevinho
veebersSaviq: that's odd, I'll take a look at it now21:47

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