| Cimi | morning | 07:06 |
|---|---|---|
| Cimi | starting earlier today so I can watch italy later on :D | 07:06 |
| Cimi | Saviq, I am doing tests | 07:26 |
| Cimi | Saviq, found more bugs fixing them | 07:26 |
| Cimi | (some conditions where the layout was breaking) | 07:26 |
| tsdgeos | Saviq: taaaaaaaaags | 07:31 |
| Cimi | hah | 07:32 |
| Saviq | tsdgeos, damn :| | 07:34 |
| tsdgeos | no idea how they got to my unity8 | 07:34 |
| Saviq | tsdgeos, I do | 07:35 |
| tsdgeos | seems they are not upstream | 07:35 |
| Saviq | tsdgeos, they were, yesterday | 07:35 |
| Saviq | tsdgeos, or well, a few days before | 07:35 |
| tsdgeos | but somehow i ended up infecting swipe_down_close_no_remove_dash | 07:35 |
| Saviq | tsdgeos, Kaleo's branch | 07:35 |
| tsdgeos | cleaning up now | 07:35 |
| Saviq | tsdgeos, I will do some checking again | 07:35 |
| Saviq | before the next landing | 07:36 |
| tsdgeos | oki | 07:36 |
| Saviq | ooh 5.3 is a landeded | 07:37 |
| Saviq | crap | 07:38 |
| mzanetti | facundobatista: hey, was on holiday yesterday | 07:54 |
| mzanetti | facundobatista: if you have an utopic machine now, you can branch lp:unity8 and run ./build.sh in there. Then you'll have a "cardtool" somewhere in the builddir | 07:55 |
| mzanetti | facundobatista: ping when you're around I can give you better instructions then | 07:55 |
| Saviq | tsdgeos, hmm we seem to be missing some "don't animate height" on the VJ | 09:57 |
| Saviq | tsdgeos, I did http://paste.ubuntu.com/7673731/ | 09:58 |
| Saviq | AAARGh | 09:58 |
| tsdgeos | Saviq: i think i missed half of what you said | 10:00 |
| tsdgeos | i only got | 10:00 |
| tsdgeos | <Saviq> tsdgeos, hmm we seem to be missing some "don't animate height" on the VJ | 10:00 |
| Saviq | tsdgeos, https://code.launchpad.net/~aacid/unity8/dashviews_integration/+merge/222192/comments/537179 | 10:00 |
| tsdgeos | <Saviq> tsdgeos, I did http://paste.ubuntu.com/7673731/ | 10:00 |
| Saviq | tsdgeos, and there's a small inline comment on the doxy comment | 10:01 |
| tsdgeos | ok | 10:01 |
| * tsdgeos reads | 10:01 | |
| tsdgeos | oh come on, got disconnected again :/ | 10:01 |
| Saviq | tsdgeos, no, you're still here | 10:01 |
| Saviq | wow | 10:01 |
| mzanetti | Saviq: known troubles with QtFeedback? | 10:02 |
| Saviq | <tsdgeos> oh come on, got disconnected again :/ | 10:02 |
| Saviq | <Saviq> tsdgeos, no, you're still here | 10:02 |
| Saviq | <-- tsdgeos has quit (Read error: Connection reset by peer) | 10:02 |
| Saviq | tsdgeos, you knew it before it happened ;) | 10:02 |
| Saviq | mzanetti, context? | 10:02 |
| mzanetti | plugin cannot be loaded for module "QtFeedback": The plugin '/usr/lib/x86_64-linux-gnu/qt5/qml/QtFeedback/libdeclarative_feedback.so' uses incompatible Qt library. (5.3.0) | 10:02 |
| Saviq | mzanetti, looks like you didn't upgrade completely | 10:03 |
| mzanetti | yeah... but apt doesn't agree on that | 10:03 |
| tsdgeos | yeah it's kind of weird | 10:03 |
| tsdgeos | i was disconnected when i wrote that | 10:03 |
| tsdgeos | then reconnected | 10:03 |
| tsdgeos | and client quite | 10:03 |
| tsdgeos | no idea why | 10:03 |
| Saviq | mzanetti, you using a mirror? | 10:03 |
| Saviq | mzanetti, you probably upgraded in-between migrations from proposed | 10:03 |
| mzanetti | looks like... | 10:03 |
| mzanetti | qtsensors coming in noe | 10:03 |
| tsdgeos | Saviq: 4-indented ? you mean with tabs? or 4 spaces? | 10:05 |
| Saviq | tsdgeos, 4 space | 10:05 |
| Saviq | tsdgeos, like my patch did ;) | 10:05 |
| Saviq | tsdgeos, I'll prep one for the other places where that happens | 10:05 |
| Saviq | but separate MP | 10:06 |
| Saviq | tsdgeos, you know what I meant for resize-on-creation? | 10:08 |
| tsdgeos | Saviq: hmmm, not really | 10:10 |
| tsdgeos | can't see anything wrong after applying your patch | 10:10 |
| Saviq | tsdgeos, scroll fast | 10:10 |
| Saviq | tsdgeos, expand first category, scroll past it, scroll back up | 10:10 |
| Saviq | tsdgeos, you should see it not being expanded (which it should) and animating height (which it shouldn't) | 10:11 |
| tsdgeos | erhmmmm | 10:12 |
| tsdgeos | something's very wrong | 10:12 |
| tsdgeos | i'm just not getting the VJ :D | 10:12 |
| Saviq | tsdgeos, did you build? ;) | 10:12 |
| tsdgeos | i did | 10:12 |
| Saviq | tsdgeos, did you build your VJ branch? | 10:13 |
| tsdgeos | yeah | 10:13 |
| tsdgeos | http://paste.ubuntu.com/7673799/ | 10:13 |
| Saviq | tsdgeos, well, it might be grid in fact, it's not like it's easy to discern ;) | 10:14 |
| tsdgeos | well i replaced the carousel too | 10:14 |
| tsdgeos | and still get it :D | 10:14 |
| Saviq | tsdgeos, so maybe my bug is in trunk ;) | 10:14 |
| tsdgeos | it may | 10:14 |
| tsdgeos | i can't repro though | 10:14 |
| Saviq | tsdgeos, anyway there's a few template defs above as well | 10:15 |
| Saviq | ah but that's for special | 10:15 |
| tsdgeos | ah i changed the wrong ones | 10:15 |
| tsdgeos | ... | 10:15 |
| tsdgeos | silly me | 10:16 |
| tsdgeos | yeah, why is it doing that? | 10:17 |
| * tsdgeos checks | 10:17 | |
| tsdgeos | Saviq: ok, should be better now, can you check? | 10:32 |
| Saviq | tsdgeos, yeah, checking | 10:37 |
| Saviq | tsdgeos, it needs 4 spaces *more* than normal text ;) | 10:39 |
| tsdgeos | ¿?¿? | 10:39 |
| * tsdgeos is confused | 10:39 | |
| Saviq | tsdgeos, normal text is * foo | 10:39 |
| Saviq | the ascii art | 10:39 |
| Saviq | needs to be * foo | 10:40 |
| tsdgeos | so i'm missing 1 space still? | 10:40 |
| Saviq | tsdgeos, yes | 10:40 |
| Saviq | tsdgeos, make -C builddir doc | 10:40 |
| Saviq | and builddir/doc/unity8/class_responsive_vertical_journal.html | 10:40 |
| tsdgeos | uh oh | 10:40 |
| tsdgeos | do you guys get http://paste.ubuntu.com/7673886/ on make testShell? | 10:41 |
| Saviq | tsdgeos, looks like it, yes :| | 10:41 |
| tsdgeos | http://nooooooooooooooo.com/ | 10:41 |
| Saviq | tsdgeos, yeah, same trace | 10:42 |
| tsdgeos | this crash in valgrind seems veeeeeery similar to the other one we were having | 10:44 |
| * tsdgeos builds with debug again | 10:44 | |
| Saviq | tsdgeos, ACK on VJ | 10:45 |
| tsdgeos | \o/ | 10:45 |
| Saviq | MacSlow, there's still inline comments in https://code.launchpad.net/~macslow/unity8/combo-button-support/+merge/221499 that need action (reply at least) | 10:49 |
| MacSlow | Saviq, I'm on those... | 10:49 |
| Saviq | MacSlow, ah ok, thought you were done with that one, too | 10:49 |
| MacSlow | Saviq, was surprised I didn't see them (all) before. | 10:49 |
| Saviq | MacSlow, yeah, they're not easily visible indeed | 10:49 |
| MacSlow | Saviq, the other three branches seem fine | 10:50 |
| Saviq | MacSlow, and then if you change, the diff changes and so you need to switch to the previous diff to be able to see them | 10:50 |
| MacSlow | Saviq, yeah... by now I'm aware of that new LP-diff-comment feature | 10:50 |
| MacSlow | Saviq, after all that new way is nicer for comments... especially for the commenter | 10:51 |
| Saviq | MacSlow, indeed, well, as submitter you can reply to those, so that's good, too | 10:51 |
| Saviq | there's stuff to improve for sure | 10:52 |
| Saviq | but it's a start | 10:52 |
| tsdgeos | Saviq: one for the lazyimage test https://code.launchpad.net/~aacid/unity8/fix_lazyimagetest_53/+merge/223896 | 10:53 |
| tsdgeos | then there's the multigreeter one | 10:53 |
| tsdgeos | in which html behaviour has changed | 10:53 |
| tsdgeos | someone needs to check if we're fine with that change or not | 10:54 |
| tsdgeos | mterry probably | 10:54 |
| tsdgeos | and then the crash i'm investigating | 10:54 |
| tsdgeos | i'll create a bug for mterry and the multigreeter test failures | 10:54 |
| Saviq | tsdgeos, great, thanks | 10:54 |
| Saviq | k folks, taking second half of my holiday o/ | 11:05 |
| Cimi | Saviq, you off now? | 11:08 |
| kgunn | Cimi: i'm pretty sure this was fixed in our latest release, can you double check it ? | 11:18 |
| kgunn | https://bugs.launchpad.net/mir/+bug/1319907 | 11:18 |
| ubot5 | Ubuntu bug 1319907 in Mir "Last shell frame isn't sent to the display" [High,New] | 11:18 |
| Cimi | kgunn, phone dead at the moment and without carousel | 11:19 |
| Cimi | kgunn, let me ask mzanetti | 11:19 |
| Cimi | mzanetti, do you remember the bug with the carousel? | 11:19 |
| kgunn | Cimi: i can do it actually | 11:19 |
| Cimi | ok | 11:19 |
| Cimi | kgunn, thanks! | 11:19 |
| mzanetti | Cimi: hmm... which one? | 11:19 |
| mzanetti | ah... that one | 11:19 |
| mzanetti | haven't seen it any more lately | 11:19 |
| Cimi | mzanetti, flick on carousel, no overlay | 11:19 |
| Cimi | kgunn, ^ | 11:19 |
| kgunn | thanks | 11:19 |
| Cimi | I'll reopen it in case | 11:20 |
| kgunn | Cimi: for future...how to reproduce ? add like 10 music files or something ? | 11:20 |
| Cimi | kgunn, yes, just flick | 11:20 |
| mzanetti | kgunn: yep, just drop some music or videos on the phone | 11:20 |
| mzanetti | and carousels start appearing | 11:20 |
| Cimi | kgunn, quite almost every time the overlay wasn't shown | 11:20 |
| Cimi | no title/song | 11:20 |
| mzanetti | that said, I think I saw the one in the recent apps category again lately, where whe thought it would be related | 11:21 |
| mzanetti | but I didn't really investigate, so... take it with caution | 11:21 |
| mzanetti | can't reproduce it right now... | 11:22 |
| kgunn | mzanetti: Cimi yeah...seems pretty solid, i can't repro | 11:31 |
| mhr3 | jhodapp, got a u8 crash with 5.3 coming from media-hub http://paste.ubuntu.com/7674074/ sorry for crap debug, don't have all the symbols, but fwiw it should be uploaded to daisy | 11:35 |
| === MacSlow is now known as MacSlow|lunch | ||
| Cimi | tsdgeos, I refactored my card emblem as saviq asked https://code.launchpad.net/~cimi/unity8/card_emblems_single_container/+merge/223899 | 11:37 |
| Cimi | tsdgeos, I tested a bit, seems fine | 11:39 |
| Cimi | tsdgeos, I am happy to get it merged and wait for bugreports if there are | 11:39 |
| Cimi | there are so many nested cases that there is a chance something is broken | 11:39 |
| Cimi | but I tested most | 11:40 |
| tsdgeos | Cimi: i'll be away next week, and i'm having a look at a qt crash right now, maybe you should look for someone else to review | 11:40 |
| Cimi | ok Saviq then | 11:40 |
| tsdgeos | Cimi: did the new infographics already got merged? | 11:42 |
| Cimi | tsdgeos, no | 11:42 |
| Cimi | tsdgeos, unity side they work | 11:42 |
| Cimi | tsdgeos, but camera app doesn't populate the service | 11:43 |
| Cimi | tsdgeos, so we wait next week to talk with pete who is on holiday | 11:43 |
| Cimi | tsdgeos, basically you can manually add infographics to the backend and you see them in unity... but there is something broken on the backend | 11:44 |
| tsdgeos | oki | 11:45 |
| === alan_g is now known as alan_g|lunch | ||
| jhodapp | mhr3: yeah, we weren't ready to switch to qt 5.3 | 12:04 |
| === MacSlow|lunch is now known as MacSlow | ||
| facundobatista | mzanetti, saw your instructions from some hours ago; I'm currently installing stuff in this Utopic machine for build unity, thanks! | 12:41 |
| === alan_g|lunch is now known as alan_g | ||
| === _salem is now known as salem_ | ||
| mzanetti | dpm_: kgunn: hey, one of you should move the meeting to a different time slot. I don't mind which one :) | 13:40 |
| mzanetti | talking about reminders weekly and kevin/michael 1:1 | 13:40 |
| dpm_ | mzanetti, I can move it, sorry, didn't want to step on your 1:1 | 13:41 |
| mzanetti | dpm_: no... kevin stepped on the reminders meeting | 13:41 |
| dpm_ | aha! :) | 13:41 |
| mzanetti | dpm_: but I figured we're moving the reminders one often enough to probably move it for real | 13:41 |
| dpm_ | mzanetti, yeah, the last weeks have been quite hectic with other things and that's meant some movement there. Let me change the time for the one this week and then I'll find a better weekly time for the one next week | 13:42 |
| kgunn | dpm_: appreciate it....i got a lot of guys...so shuffling is a challenge of all the 1on1's | 13:43 |
| dpm_ | kgunn, no worries | 13:44 |
| dpm_ | mzanetti, ok, moved this one to Monday, as Riccardo pinged me earlier he might not be able to make it today | 13:45 |
| mzanetti | dpm_: works for me | 13:54 |
| dpm_ | ok, thanks | 13:58 |
| kgunn | greyback: i had a moment to tinker, export renderer_timings & when i tried to relaunch using | 14:25 |
| kgunn | QT_QPA_PLATFORM=mirserver | 14:25 |
| kgunn | it aborted | 14:25 |
| kgunn | am i doing it wrong ? | 14:25 |
| greyback | kgunn: need to also set MIR_SOCKET=/run/mir_socket | 14:25 |
| greyback | kgunn: if you pulled today, the PPA is broken again, since Qt5.3 landed | 14:26 |
| greyback | working on fixing it now | 14:26 |
| kgunn | greyback: ack... sorry man | 14:27 |
| greyback | kgunn: np | 14:27 |
| * kgunn wonders about qt5.3 so close to rtm | 14:27 | |
| greyback | kgunn: I'll let you know when PPA back up and running | 14:27 |
| greyback | the gcc transition also didn't help | 14:27 |
| kgunn | yeah :-/ | 14:28 |
| tsdgeos | kgunn: i just discovered a few crashers too :/ | 14:28 |
| tsdgeos | in qt5.3 | 14:28 |
| kgunn | olli_: ^ to keep in mind for discussions about how upstream migrations occur | 14:29 |
| kgunn | greyback: mirserver doesn't seem to exist... | 14:33 |
| kgunn | Available platform plugins are: eglfs, kms, linuxfb, minimal, minimalegl, offscreen, ubuntu, ubuntumirclient, ubuntumirserver, xcb. | 14:33 |
| kgunn | i tried the obvious ubuntumirserver | 14:33 |
| kgunn | Failed to connect to "/run/user/32011/ubuntu-keyboard-info" after 10 failed attempts | 14:34 |
| mterry | kgunn, if you have time today, you could also be my welcome-wizard guinea pig. Not urgent, still waiting on some test script changes to be reviewed. But might be a nice Friday activity | 14:35 |
| kgunn | mterry: is there something available now ? | 14:35 |
| kgunn | i'm taking off for the day at lunch | 14:36 |
| mterry | kgunn, just install ubuntu-system-settings-wizard and run 'adb shell rm /home/phablet/.config/ubuntu-system-settings/wizard-has-run' to reset state | 14:36 |
| greyback | kgunn: you'd better just hold on until I have the PPA working again | 14:36 |
| mterry | kgunn, again, not urgent though | 14:37 |
| kgunn | greyback: my apologies...i'm a distraction, just ignore me...note: i'm not on the ppa | 14:38 |
| greyback | kgunn: I'm confused, what are you testing? Trunk? | 14:39 |
| kgunn | greyback: yep...just straight devel-proposed image | 14:39 |
| kgunn | again..just ignore me | 14:40 |
| greyback | kgunn: ok, sorry, "mirserver" threw me off - that's PPA only. | 14:40 |
| greyback | kgunn: QT_QPA_PLATFORM=ubuntumirserver and MIR_SOCKET=/run/mir_socket should be enough | 14:40 |
| kgunn | greyback: yeah that worked...but failed on keyboard | 14:41 |
| greyback | kgunn: those "Failed to connect" messages are normal, as maliit-server hasn't been started (it's usually started by unity8's upstart job) | 14:41 |
| kgunn | maybe i didn't wait long enough | 14:42 |
| paulliu | Cimi: hi. Can you help me review this branch? https://code.launchpad.net/~paulliu/unity8/lp1330957/+merge/223779 | 14:43 |
| Cimi | paulliu, sure | 14:43 |
| paulliu | Cimi: not a large one but fixes some FAILED in testCard | 14:43 |
| cwayne | Saviq: any chance of the vertical journal fixes landing today? | 14:43 |
| kgunn | cwayne: i think Saviq is off... tsdgeos might know | 14:44 |
| Cimi | paulliu, is this a typo? headerLeftAnchorHasMagin | 14:44 |
| tsdgeos | cwayne: it's approved | 14:44 |
| tsdgeos | cwayne: https://code.launchpad.net/~aacid/unity8/dashviews_integration/+merge/222192 | 14:44 |
| kgunn | tsdgeos: do you just need a silo to land it ? | 14:44 |
| tsdgeos | cwayne: but it needs a lander | 14:44 |
| tsdgeos | not sure if Saviq has something prepared or not | 14:44 |
| greyback | kgunn: but you're not starting the unity8 job, you're starting unity8 manually. So no dependent processes (like maliit, indicators) won't autostart. | 14:44 |
| kgunn | tsdgeos: i'll check and get a silo if not... | 14:45 |
| tsdgeos | oki | 14:45 |
| paulliu | Cimi: No. The original code are using that variable. Or I should change all of them? | 14:45 |
| Cimi | paulliu, you can skip the fix | 14:46 |
| Cimi | paulliu, I will add it in my emblems branch | 14:46 |
| greyback | kgunn: you could run "initctl set-env QSG_RENDER_TIMING=1" and then "start unity8" as usual, and the ~.cache/upstart/unity8.log file will have the timings printed there | 14:46 |
| Cimi | paulliu, the entire fix I mean | 14:46 |
| paulliu | Cimi: ok. because I'll rebase my another branch about the attribute UI on your emblems branch. | 14:46 |
| Cimi | paulliu, or you can do that | 14:46 |
| greyback | but that means *every* Qt process will do the same, unless you unset that right after | 14:46 |
| greyback | kgunn: ^ | 14:46 |
| Cimi | paulliu, but I think is easier for you if you leave it to me | 14:46 |
| Saviq | kgunn, no, no silo - we could land stuff that's there in u8 queue, unity-notifications could use a landing, too | 14:47 |
| Cimi | paulliu, that code is quite awful to see (that js) | 14:47 |
| paulliu | Cimi: OK, maybe I should do that. Because this bug fixing branch is smaller. | 14:47 |
| paulliu | Cimi: yours are large. | 14:47 |
| Cimi | paulliu, I mean I can add the fix to my branch | 14:47 |
| paulliu | Cimi: ah. got it. Please do. | 14:48 |
| Cimi | paulliu, but you can keep fixes for the card tests | 14:48 |
| paulliu | Cimi: actually I need to fix the margin. | 14:49 |
| Cimi | paulliu, ok fix it then | 14:49 |
| paulliu | Cimi: ok. | 14:49 |
| Cimi | paulliu, but please rename the variable | 14:49 |
| paulliu | Cimi: ok. | 14:50 |
| kgunn | Saviq: you know time off means off :)...but since you're on, do you mean you want me to create a silo and land everything that's approved+ready for unity8+unity-notifications ? | 14:52 |
| kgunn | (sorry parse error on your last post) | 14:52 |
| Saviq | kgunn, yeah | 14:52 |
| kgunn | ack | 14:53 |
| kgunn | cwayne: just an fyi...might not land until monday...looks like ci train is unmanned for the moment | 15:06 |
| kgunn | so suspect i won't get a silo until late afternoon | 15:06 |
| cwayne | kgunn: ack, thanks for the update, monday should be fine | 15:07 |
| === dandrader is now known as dandrader|afk | ||
| mhr3 | mhall119, mind running your nice script to update the scopes docs? | 15:18 |
| mhr3 | mhall119, or is it running as a cron job somewhere? | 15:18 |
| mhall119 | mhr3: no cron yet, the changes are in trunk? | 15:18 |
| mhr3 | mhall119, in U even, yes | 15:18 |
| mhall119 | I'm still on trusty, so doing it from the branch is easier than from the package | 15:19 |
| Saviq | paulliu, hey, charles fixed the failure in indicator-session, but unity8 still didn't log out for him | 15:27 |
| Saviq | paulliu, I kicked a rebuild of everything in silo 008 | 15:28 |
| charles | saviq, I sent paulliu a mail about it on (wednesday?) | 15:28 |
| Saviq | charles, yes, seb128 fwd'ed it back to me | 15:29 |
| Saviq | charles, so I don't have a reply, if there was any :) | 15:29 |
| mhall119 | mhr3: please check http://91.189.92.89/api/scopes/sdk-14.10/ | 15:29 |
| charles | paulliu, are you able to trigger a logout in unity8 via dbus-send? | 15:29 |
| charles | I'd like to decouple the testing of the indicator-session parts and the unity parts | 15:29 |
| Saviq | paulliu, you could test on the phone btw, if your desktop doesn't run unity8 | 15:30 |
| Saviq | paulliu, you just need to run the dbus command, or you could even export the UNITY_INDICATORS_PROFILE="desktop" into upstart to get the session indicator on the phone | 15:31 |
| mhr3 | mhall119, +1, we moved some stuff to a new unity::scopes::experimental namespace, but it's actually good that it's not in the docs, discouraging use of classes from there was why we did it | 15:33 |
| seb128 | Saviq, there was no reply | 15:33 |
| mhall119 | mhr3: do any old classes need to be removed? remember that the import script never deletes things, only adds and updates | 15:33 |
| mhr3 | mhall119, hm, even if they were you wouldn't be linked from anywhere, no? | 15:34 |
| mhr3 | they wouldn't be* | 15:34 |
| mhall119 | mhr3: if you moved things from a supported namespace into experimental, they would still be in the old location | 15:35 |
| mhr3 | mhall119, right, but there wouldn't be any pages linking to that now | 15:35 |
| mhr3 | mhall119, or wait the index page is hardcoded, right? | 15:36 |
| mhall119 | no | 15:36 |
| mhall119 | http://91.189.92.89/api/scopes/sdk-14.10/ is generated from the API website's data | 15:37 |
| mhr3 | where does it get the categorization from? | 15:37 |
| mhall119 | mhr3: for scopes, I hand-crafted them | 15:37 |
| mhall119 | because the source docs don't have that data | 15:37 |
| mhr3 | so it is kindof hardcoded?! | 15:38 |
| mhall119 | for QML docs, I can get that from the sources | 15:38 |
| mhall119 | mhr3: only the categories | 15:38 |
| mhall119 | and those are hardcoded in the importer, not in the api-website | 15:38 |
| mhr3 | mhall119, ok, nonetheless, it looks good to me | 15:38 |
| mhall119 | ok, I'll push to production | 15:39 |
| === dandrader|afk is now known as dandrader | ||
| paulliu | Saviq: yes, I can do the logout by dbus-send | 15:41 |
| Saviq | charles, ↑ | 15:41 |
| mhall119 | mhr3: please verify http://developer.ubuntu.com/api/scopes/sdk-14.10/ | 15:41 |
| paulliu | charles: Can't you logout with dbus-send? | 15:41 |
| charles | paulliu, can you paste to me the recipe you're using for that? | 15:42 |
| paulliu | charles: I think tsdgeos also tested the dbus-send already. | 15:42 |
| paulliu | charles: https://bugs.launchpad.net/unity8/+bug/1302213 | 15:42 |
| ubot5 | Ubuntu bug 1302213 in Unity 8 "API to bring down the session" [Critical,In progress] | 15:42 |
| charles | paulliu, no it didn't work for me when I tried dbus-send from the command line. | 15:43 |
| paulliu | charles: the unity8 didn't kill itself? | 15:43 |
| charles | paulliu, I'm happy to re-test, could you paste the invocation you used, so I can repeat it hee? | 15:44 |
| mhr3 | mhall119, oh wait one leftover - "Deprecated List" | 15:45 |
| mhr3 | mhall119, should be gone now | 15:45 |
| mhall119 | mhr3: the full page? | 15:46 |
| mhr3 | mhall119, yep | 15:47 |
| mhall119 | ok, gone | 15:47 |
| mhall119 | mhr3: removed from staging as well, so it won't cause confusion on a future update | 15:48 |
| mhr3 | mhall119, awesome, thx | 15:48 |
| paulliu | charles: ok. wait. | 15:53 |
| paulliu | charles: first, bzr branch lp:~paulliu/unity8/logout and then ./run_on_device.sh (you might want to adb shell install some build-deps first). | 15:56 |
| Saviq | paulliu, just upgrade from silo 008 | 15:56 |
| Saviq | https://launchpad.net/~ci-train-ppa-service/+archive/landing-008 | 15:56 |
| charles | paulliu, thanks. I'll try that here | 15:56 |
| Saviq | still building latest unity8, but shouldn't be a problem (and should finish soon) | 15:57 |
| paulliu | charles: And source this shell script. http://paste.ubuntu.com/7675317 | 15:57 |
| Saviq | paulliu, if you ssh onto the device, that should be there in your env already | 15:57 |
| paulliu | charles: And use qdbus com.canonical.Unity /com/canonical/Unity/Session com.canonical.Unity.Session.RequestLogout | 15:57 |
| paulliu | Saviq: ah. ok | 15:57 |
| Saviq | or use sudo -u phablet -i | 15:57 |
| charles | well, I was testing on desktop since indicator-session doesn't have a profile on the phone | 16:00 |
| charles | unless I'm misunderstanding what I'm needing to test here... | 16:00 |
| paulliu | charles: Another way to test on desktop is use KDE, and then run ./run.sh, qdbus will also trigger a kill of unity8. | 16:01 |
| paulliu | charles: I'm thinking if it is because I didn't modify debian/*.install. The plugin might not be installed. | 16:10 |
| paulliu | charles: I'll check the silos | 16:10 |
| paulliu | charles: hmm, no, there is the plugin. | 16:11 |
| paulliu | charles: I'm testing silos right now. But I'll need some time for that. The network to ports and ppa isn't stable. | 16:14 |
| paulliu | charles: Some Taiwan sites are under attack now. | 16:15 |
| === salem_ is now known as _salem | ||
| charles | paulliu: okay, on my laptop I'm logged into a unity8 session | 16:22 |
| charles | paulliu: I ssh in from my desktop and run that script to get DBUS_SESSION_BUS_ADDRESS set | 16:23 |
| charles | (nice script, by the way. I'd been sourcing the upstart file, but this is more convenient ;) | 16:23 |
| paulliu | charles: try "qdbus com.canonical.Unity". It should prints all the objects | 16:24 |
| charles | paulliu: doesn't exist. I'm wondering if an update I've run since Wednesday overwrote the silo | 16:25 |
| charles | paulliu: 5 minutes :) | 16:25 |
| paulliu | charles: what? Session might not be there, but there should be some other objects there already. | 16:26 |
| paulliu | charles: I now only have 5000 B/s to ports and ppa. | 16:27 |
| paulliu | charles: Cannot get anything. | 16:27 |
| seb128 | charles, paulliu: I'm installing that ppa to try here | 16:28 |
| charles | paulliu: to answer your question, this is what I see: | 16:29 |
| charles | $ qdbus | grep Unity | 16:29 |
| charles | com.canonical.Unity.WindowStack | 16:29 |
| charles | com.canonical.Unity.Launcher | 16:29 |
| charles | com.canonical.UnityGreeter | 16:29 |
| paulliu | charles: hmm. So Session isn't there. I'll check. | 16:29 |
| charles | paulliu: but I suspect I'm not installed right from silo 8, so as I said, give me a minut :) | 16:30 |
| paulliu | charles: ok | 16:30 |
| charles | seb128, thanks :) | 16:30 |
| seb128 | charles, paulliu: the ppa doesn't work on utopic unity8-mir-desktop session :/ | 16:31 |
| charles | seb128, do you mean when trying to log out through indicator-session, or when manually sending the RequestLogout call over the bus via dbus-send / qdbus? | 16:32 |
| seb128 | the indicator | 16:34 |
| seb128 | tryin qtdbus now | 16:34 |
| charles | seb128, thanks | 16:34 |
| charles | if that works, maybe there's still a different bug remaining in indicator-session | 16:35 |
| kgunn | mterry: only thing i noticed is that the delay between the wizard and the spinner is kinda big... | 16:35 |
| kgunn | long enough that as a user i wanted to start punching buttons | 16:36 |
| seb128 | charles, trying to get qdbus to work first... | 16:37 |
| charles | seb128, you have qdbus5 installed? | 16:37 |
| * charles is waiting on apt-get upgrade :) | 16:38 | |
| seb128 | charles, installing it | 16:38 |
| seb128 | thanks | 16:38 |
| seb128 | "qdbus com.canonical.Unity /com/canonical/Unity/Session com.canonical.Unity.Session.RequestLogout" doesn't do anything to unity8 | 16:40 |
| seb128 | charles, paulliu: ^ | 16:40 |
| paulliu | seb128: ok. | 16:41 |
| paulliu | seb128: I'll check. I'm still on apt-get update. | 16:41 |
| paulliu | seb128: what does "qdbus com.canonical.Unity" say? | 16:43 |
| seb128 | charles, paulliu: unity8-mir.log has "Signal QQmlEngine::quit() emitted, but no receivers connected to handle it." | 16:44 |
| seb128 | displayed every time the dbus request is emitted | 16:44 |
| seb128 | paulliu, http://paste.ubuntu.com/7675560/ | 16:45 |
| seb128 | paulliu, is the "no receivers connected" error helping? what code is supposed to be the receiver? | 16:51 |
| paulliu | seb128: let me check. | 16:56 |
| greyback | that usually means in the main.cpp where the QQmlEngine is created, that no slot was connected to the QQmlEngine::quit signal to actually call QGuiApplication::quit() | 16:56 |
| paulliu | greyback: yes. I think I did that already when you told me. | 16:57 |
| paulliu | greyback: But might be cleaned after some trunk merge. I'll fix it. | 16:58 |
| greyback | paulliu: you've a better memory than I do. I don't remember saying that :) | 16:58 |
| charles | :) | 16:58 |
| seb128 | paulliu, https://launchpadlibrarian.net/178057888/unity8_7.89%2B14.10.20140618-0ubuntu1_7.89%2B14.10.20140620-0ubuntu1.diff.gz is what we got in the ppa upload | 16:58 |
| seb128 | ups, sorry | 16:59 |
| seb128 | launchpad seems confused | 16:59 |
| === alan_g is now known as alan_g|EOW | ||
| paulliu | Saviq: I push a new commit to logout branch based on seb128 and greyback's comments. | 17:04 |
| paulliu | Saviq: add QObject::connect(view->engine(), SIGNAL(quit()), application, SLOT(quit())); in main.cpp | 17:05 |
| seb128 | Saviq, charles, paulliu: I'm kicking a rebuild for unity8 so it picks up that change | 17:07 |
| seb128 | paulliu, do you know how that code got dropped/why you had it in your local version and no in the branch? | 17:08 |
| paulliu | seb128: My local branch also dropped sometime after merge trunk. But I don't know why it still works on all my devices. Even on phone. | 17:09 |
| seb128 | ok, let's see if that rebuild makes it work for us | 17:09 |
| === _salem is now known as salem_ | ||
| seb128 | Saviq, charles, paulliu: ok, works now with the unity8 signal connect change | 17:39 |
| charles | seb128: \o/ | 17:40 |
| seb128 | well "works", it logout without asking for confirmation | 17:40 |
| Saviq | finally | 17:40 |
| Saviq | seb128, yeah, that comes later | 17:40 |
| seb128 | but I guess that's what that change is supposed to do? | 17:40 |
| seb128 | k | 17:40 |
| seb128 | Saviq, so feel free to land the silo | 17:40 |
| charles | seb128: right, i-session tries to pull up the dialog, but when it gets a failure response, that's treated as confirmation now... | 17:40 |
| Saviq | seb128, Monday ;) | 17:40 |
| charles | :) | 17:40 |
| charles | seb128: thanks for being the scratch monkey | 17:41 |
| seb128 | Saviq, yeah, I was thinking the same/going to add "if you feel adventurous" ;-) | 17:41 |
| seb128 | charles, no worry! | 17:41 |
| Saviq | seb128, I'm even on holiday today ;) | 17:42 |
| === dandrader is now known as dandrader|lunch | ||
| seb128 | charles, btw did you see my ping about reviewing cyphermox's bluetooth setting changes? do you think you would have review slot for that? | 17:42 |
| seb128 | Saviq, stop being on IRC then, go enjoy your extended w.e ;-) | 17:42 |
| charles | seb128, no sorry I didn't see that ping. Yes of course I'll take a look | 17:43 |
| charles | (cyphermox: ^) | 17:43 |
| seb128 | charles, thanks | 17:43 |
| charles | seb128: are there any others you want me to look at besides https://code.launchpad.net/~mathieu-tl/ubuntu-system-settings/discoverability-annd-fixes/+merge/223955 | 17:49 |
| seb128 | charles, no, that's the one, thanks! | 17:49 |
| cyphermox | aye | 18:04 |
| === dandrader|lunch is now known as dandrader | ||
| === dandrader is now known as dandrader|afk | ||
| === salem_ is now known as _salem | ||
| === dandrader|afk is now known as dandrader | ||
| === _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!