=== tvoss is now known as tvoss|test === tvoss|test is now known as tvoss [07:02] good morning [07:10] mzanetti: do you have any ideas on who/where else to possibly ask? (About the notification issue Im having, you suggested #ubuntu-webapps on friday) [07:11] jussi: sorry. no idea [07:12] mzanetti: ok, thanks. Ill let you know if I actuall find a solutoin somewhere... [07:14] hey sil2100, how are you? [07:26] jussi: cheers [07:28] mzanetti: found the issue [07:28] mzanetti: [07:28] https://bugs.launchpad.net/ubuntu/+source/chromium-browser/+bug/1153137 [07:28] Launchpad bug 1153137 in webaccounts-browser-extension (Ubuntu Raring) "Please remove recommends on webaccounts-chromium-extension and unity-chromium-extension" [Wishlist,Triaged] [07:29] basically those were pulled into chromium by accident [07:29] and simply uninstalling them fixes the issue === zniavre__ is now known as zniavre [07:35] tsdgeos: good morning [07:35] hiho [07:36] tsdgeos: yeah, I just triggered a rebuild, given that it failed only one one of the builders [07:36] still weird... crash in gcc if I read it correctly [07:36] gcc? [07:36] perl it says, no? [07:37] yeah well, aynthing that compiles/interprets it. not our code myself [07:37] itself [07:38] actually in debhelper... never seen that before [07:38] lets see if it passes now [07:38] didrocks: hi! Fine ;) How about you? [07:39] didrocks: on Friday I tried to add the package to the stacks, I pushed a branch yesterday [07:39] sil2100: hi there, asking again, sorry, did the compiz fix for qt popups get merged? [07:39] didrocks: https://code.launchpad.net/~sil2100/cupstream2distro-config/android-audiosystem_add/+merge/158806 [07:40] tsdgeos: yes \o/ So it seems at least [07:40] sil2100: should it be in raring? or does it need releasing? [07:41] It's not in raring yet, I don't see the release commit for it [07:41] oki [07:41] But I think soon it should be in [07:43] sil2100: great! do you mind keeping the file ascii sorted for projects, please? [07:44] Ah! Ok ;) [07:44] But is the platform stack the right place? [07:44] sil2100: sounds good, I would have put it in platform or misc as there is no tests, but fine either way :) [07:44] sil2100: do you have time for a hangout for the next course of action then? [07:45] didrocks: can we have it in 30 minutes :) ? [07:45] sil2100: sure [07:46] tsdgeos: it passed [07:47] :-) [07:47] jussi: thanks [07:47] mzanetti: yw [07:48] mzanetti: print("**volume changed", volume, actionGroup.serverVolume) ? [07:49] tsdgeos: do'h... I thought I removed them all... will do so now [07:50] tsdgeos: done [07:51] oka, charging the phone, somehow it died even if it was turned off :S [07:52] good that 2A charger that came with the Nexus10 :D [07:52] charges phones fast [07:53] yeah [07:53] yeah... mine is still on [07:53] charged it mid-last week for the last time. was in standby here on my desk [07:53] say 32% remaining [07:54] weird [07:56] awesome, not weird :D [07:57] ah... no wifi connected. that helps of course [07:57] mzanetti: so i just moved the volume slider in overview to max [07:58] and it is still at min in the volume indicator :-S [07:58] tsdgeos: that seems a bug in the volume indicator then. because if you move it with alsamixer, the slider in overview moves around [07:59] seems there's something weird when it's at min and you move to max [07:59] otherwise the rest works "ok" [08:00] tsdgeos: I'll test [08:03] mzanetti: on the device seems that using the overview slider i can't go to 0 or to 100 [08:03] everything else, alsamixer updates to [08:03] but 0 or 100 alsamixer ignores [08:03] let's see the indicator [08:04] yeah, 0 and 100 don't work from overview and work from indicator [08:05] not sure that'd be "your fault" though [08:11] since your code doesn't seem to care about that [08:12] mzanetti: should serverVolume still be readonly? [08:13] tsdgeos: its now internal one.... [08:13] still could be... [08:13] true [08:13] let me fix [08:14] mzanetti: do you know where QDBusActionGroup comes from? [08:14] tsdgeos: no... it was there for the hardware volume buttons [08:14] tsdgeos: I just made it more guideline-compliant and reused it in the overviewpage [08:14] sure [08:15] wanted to know what qdbus calls it made [08:15] to prove it wasn't the ui fault that 0 and 100 don't work [08:15] Saviq: do you know where QDBusActionGroup comes from? [08:15] tsdgeos, sounds like qmenumodel [08:15] may be [08:15] tsdgeos: ah... yeah. I suspect it to be in there too. but haven't seen it before. even though it seems really useful and probably should be part of QtQuick itself [08:16] tsdgeos, https://code.launchpad.net/qmenumodel [08:16] mzanetti: seems like a gtk/glib adaptor thing, not sure makes sense :D [08:16] tsdgeos: oh... in that case no... but exactly the same that just wraps QDbus for qml... [08:16] yep [08:17] tsdgeos: I guess that would be QServiceFramework tho [08:17] Saviq: yeah it's there, tx [08:19] meh, can't call (sva{sv}) with qdbus [08:22] didrocks, I noticed that network and location stacks have an empty 'tests' parameter. What do they run in this case ? [08:23] jibel: they are doing the dist-upgrade I guess [08:23] jibel: but I'm waiting them to have components to have the list of real tests ;) [08:23] sil2100: it's 40 minutes ;) [08:25] didrocks, so we are provisioning 3 machines just to check installability/upgradeability of the packages, right? [08:25] jibel: indeed [08:25] which is better than nothing ;) [08:27] didrocks: ready ;) [08:27] Didn't want to hang-out in my pajamas [08:27] didrocks, hm, okay, seems suboptimal, couldn't it be tested before starting the provisioning, running something like piuparts and then only provision physical machines and run AP if it succeeds? [08:28] jibel: this is really temporary, we'll have by the end of the week I hope at least one AP to run [08:29] mzanetti: have you been able to reproduce the 0/100 problem? [08:29] tsdgeos: not yet. had to reflash the device. still on it [08:29] sil2100: https://plus.google.com/hangouts/_/7ab39ee5ed454250b930d481b69dd69b80ca8f50?authuser=0&hl=fr [08:29] i can't find how to pass a variant to gdbus either [08:30] all those uppercase letters on the OSK even when typing lowercase letters confuse the shit out of me :D [08:31] didrocks, ack, just doing some capacity planning to estimate our needs in physical hw and optimizing the resources we have already. === duflu_ is now known as duflu [08:33] jibel: after this hangout, do you want to discuss some strategy for this? [08:33] didrocks, sure thing [08:40] mzanetti: yeah the BB Z10 does the same, i find it not cool [08:40] because the BB playbook doesn't :D [08:40] they changed their mind from product to product [08:40] tsdgeos: stupidly copying apple crap [08:41] tsdgeos: I guess the biggest troubles is because I use maliit now for 2 years, but in a sane configuration [08:42] D: [08:42] what i like a "lot" about the BB10 is the inline suggestions in the keyboard [08:42] have you seen those? [08:42] tsdgeos: narf... no. I haven't seen anything bug a youtube video from the BB Z10 [08:42] I'd really like to get one into my hands [08:44] mzanetti: see www.youtube.com/watch?v=6Fusk03iTEI&feature=youtu.be&t=27s [08:45] the little words over the keys can be just typed by pressing they key and swiping up [08:45] yeah... this seems cool indeed [08:46] so eventually you can have lots of "next word" suggestions [08:46] if it adjusts to personal usage it can become quite efficient I guess [08:47] it does [08:47] or seems to [08:47] let me try to write a mail to some developer friends at rim :D see if they have a developer device around they don't need any more [08:47] at least here it started suggesting words quickly after the first messages [09:13] mardy: hey mardy ... umm i have noticed few other problems with ubuntu online accounts :'( [09:13] mardy: first of all doing async callback-based programming in gobject is a proper torture and big thumbs up for you guys to cope with that :-) [09:16] mardy: though what makes me really upset is that I can see the passwords in gnome-keyring without being prompted for my password - well, not really a fault of UOA, but it affects UOA drastically. [09:17] in comparison, firefox password manager will ask you for your master password (again!) before allowing you to see the passwords decrypted [09:18] luv: that's a "feature" of GNOME keyring: there is a master password, but the keyring is automatically unlocked when you login [09:18] well I am happy to help ... but given me trying to hack on UOA last few week didnt get very far, Im a bit down now. [09:18] luv: yes, that's what im saying [09:20] also, signon_identity_signout doesnt not do what we want ... so only possibility is to create a new identity, change account to that identity and delete the old one (which is fine ... if it didnt have to be coded in gobject async madness :-) ... it's about five lines of python ;-) ) [09:21] next problem ... even when using the existing "remove account" functionality it sometimes does not delete the associated identity - it happened to me with a made-up jabber account [09:21] mardy: well GNOME keyring has to be cleared patch to required your password even when you are logged-in because this a huge f*cking security problem for UOA if it doesn't (sorry for being that upfront about it) [09:22] s/cleared/clearly/ .. s/patch/patched/ .... :-) [09:25] (an obvious use-case - I am visting a friend, (s)he lets me use the computer, leaves to make a tea and I have their google/yahoo/ms account) [09:32] mzanetti: could you repro¿ [09:33] tsdgeos: yes [09:34] any clue what may be wrong? === alan_g is now known as alan_g|afk [09:34] tsdgeos: not yet... but there seems something fishy in my code indeed [09:34] https://code.launchpad.net/~sil2100/webbrowser-app/change_arch_to_list/+merge/158871 <- didrocks [09:35] mzanetti: you sure? mean you are just passing 0 and 1 to actionGroup.actionObject.updateState, no? [09:36] sil2100: looks good :) [09:36] sil2100: approved ;) [09:37] mardy: sorry for pointing out problems in solution you are working on .. but seriously ... it's like being able to easily see your linux password decrypted after you log in (if not worse) [09:38] luv: you can change your master password in the gnome keyring, and then it won't open automatically at login [09:39] luv: signon_identity_signout might be buggy, but it should do exactly what you want to do: clear the password and all stored tokens [09:40] mardy: well I can, and I can stop using UOA (as I do at the moment - it's the only issue which keeps me on 12.04, that's why I care) but what about those tens of millions of users who do not know about it! ... signon_identity_signout is not supposed to clear the passwords at least not according to the docs and comments in the code [09:40] i wish it was! [09:40] tsdgeos: yeah... I debugged a bit more... seems to be the QDBus thingie indeed. however, I found another issue in my code. so let me fix that before approving [09:41] oka [09:41] and again even if i change the master password - it does not help! [09:41] and add a unit test for that you found :D [09:41] i want to be able to use the keyring but not to see the passwords decrypted [09:41] just as other passwords managers do [09:42] luv: http://code.google.com/p/accounts-sso/source/browse/src/signond/signonidentity.cpp?repo=signond#370 [09:42] umm, i guess, let's get it covered on slashdot and it will make the gnome guys fix it ;-) [09:42] mardy: http://docs.accounts-sso.googlecode.com/git/libsignon-glib/html/SignonIdentity.html#signon-identity-signout [09:43] luv: oops, yes, the documentation is incomplete and misleading [09:43] dednick: i thought we were removing FIXME's not adding them :D [09:43] luv: I'll fix that [09:45] mardy: ok, I will make sure the gnome guys fix the bug in gnome-keyring :-) I guess a blog post linked from slashdot will do the job :-) [09:47] luv: I'm not sure it's a bug, but you can try [09:47] luv: the alternative is that you'll always be asked for the master password the first time an applications tries to ue the keyring [09:50] they will probably need to add a concept of a "privileged application" or something [09:50] your solution - to ask first time for every application trying to use it - is very good as well [09:51] umm, the dont' think it's a bug either ... that's why we need slashdot to tell them ;-) [09:51] luv: I don't know, I think it would be very annoying for the users [09:51] well, yeah, the fact that anyone can steal your password is not annoying though ;-) [09:52] people are not shouting at gnome devs only because they do not know it's that easy to "get in" [09:53] maybe ask every time an application wants to use the keyring + "privileged applcations" (application which are granted access just when you log in) ... would make everyone happy === alan_g|afk is now known as alan_g [09:57] tsdgeos: pushed... the 0/1 seems not to be my code [09:57] mzanetti: shall we open a bug? [09:58] against the shell itself [09:58] saying "this doesn't work", seems to be either q QGroupdbusaction or of the service handling the dbus call, investigate [10:00] tsdgeos: yeah... would make sense [10:00] mzanetti: you do it? or want me to? [10:00] tsdgeos: and we need a bug for the volume indicator tab not updating when the volume changes somewhere else [10:01] tsdgeos: feel free :) [10:06] mardy: but indeed, this is something that should be discussed with gnome-keyring team (unfortunately, gnome devs are bit peculiar when it comes to discussing stuff with outsiders ;-) ), so back to UOA talk ... If signon_identity_signout should remove the credentials from db - what is the difference between signon_identity_signout and signon_identity_remove then? [10:07] luv: _signout() only removes the password and the tokens, not all the metadata -- that is, the Identity record still remains valid [10:07] luv: _remove() removed the record completely, which means a new record has to be stored [10:07] for the accounts to work [10:07] good [10:08] luv: please try to see if that removes all the records from the keyring -- if not, please file a bug [10:09] mardy: cool, thanks a lot! [10:09] luv: back to the keyring issue: you should never lend the PC to someone else while you are logged in; Ubuntu has the "guest user" session exactly for that reason [10:10] mardy: yeah, that's what the gnome guys will say [10:10] it's just a cheap excuse - not how the real world works [10:11] luv: true, but it's just because users are not careful about security [10:11] if you give access to somebody to your session you either trust the person or don't care about giving access to the datas available there [10:12] luv: it's not just passwords, think of all your pictures, e-mails, documents; there might be plenty of stuff you don't want to share, other than passwords [10:12] well I know they WON'T be able to see my password saved in firefox password manager (even though they will be able to use them from that session) ... [10:13] how not? it's an option in firefox preferences [10:13] because firefox ask for the master password again(!) when you ask it to show you the password decrypted [10:13] luv: setting the master password is not mandatory -- I for instance don't have one [10:13] "display password" [10:14] yes, if you use a master password, it differentiates the two different kinds of access to the keyring [10:15] seb128: click it and it asks for your master password (if you use one) [10:18] luv, dunno what "master password" is and I guess I don't use one because it doesn't ask for anything [10:18] luv, so it's similar to the "set a keyring password different from your login password", it applies to few people who know/care enough to do opt in [10:19] mzanetti: https://bugs.launchpad.net/unity/+bug/1169127 [10:19] Launchpad bug 1169127 in Unity phablet "Setting volume to 0 or 1 (min/max) does not work when done thorugh the Overview Indicators Panel" [Undecided,New] [10:21] seb128: 1) almost everyone cares about it - people just don't know how easy it is to get in - so they blindly trust developers (and it's our job not to screw up). And nah, even if you set different password then your login it won't crack it! The moment you enter that different password to be used (for example by signond) it is unlocked and you do not need to enter it again to see the passwords. When using firefox you would need to ente [10:22] luv, IRC cut after "When using firefox you would need to ente" [10:22] "When using firefox you would need to enter it again" [10:22] i can make you a video of the difference when i get back home :-) [10:22] luv, but my point is that almost no user will know about the firefox master password or optin for one, so in practice if you hand your session to somebody you hand your firefox passwords [10:22] umm [10:22] luv, no need of a video [10:23] i think firefox asks actually [10:23] it ask "are you sure you want to display the passwords" [10:23] yes/no [10:23] but that's it [10:23] no i mean [10:23] tsdgeos: confirmed [10:24] first time you ask firefox to save a password, it tells you the implications of using a password manager (AFAIK) [10:26] well it doesn't ask for the master password because you don't use one (I do use one for gnome keyring altough it's same as my login password - it really makes no difference). Just think of being able to see your linux password decrypted easily after you log in. [10:26] and then saying - oh well, that's how it is, you need to lock your screen everytime - same thing [10:27] and think of all the security vulnerabilities which can get access to your account etc. [10:28] if somebody has access to your unlocked session, you loose [10:28] they can access your private documents [10:28] your emails [10:29] the websites you are logged on [10:29] no tehy cant :-) [10:29] your im logs [10:29] etc [10:30] they can't access my emails, neither im log - they would have to get access to my google account as well ;-) ... they can access only local files which which the local user have access to [10:30] maybe mess with a key logger or something [10:30] but that's completely different league [10:30] you tweak your setup, which is fine [10:31] but you said earlier than you want to solve the issue for all users [10:31] on a normal config your IM logs are in empathy->log [10:31] ok good [10:31] and they are not password protected [10:31] forget it [10:31] mzanetti: you said something about " and we need a bug for the volume indicator tab not updating when the volume changes somewhere else" [10:31] that works here [10:31] leave it completely open, fine :-) [10:31] mzanetti: what do you do to repro it? [10:32] hell, that's the original motivation to implemented the logout functionality ffs [10:32] tsdgeos: change it in alsamixer [10:32] that doesn't seem like a supported usecase :D [10:32] tsdgeos: dafuq... that doesn't work in Overview any more either [10:32] tsdgeos: it does work on desktop [10:32] sure [10:32] that I can see I'm logged in into my google account so I _know_ that the computer has access to it, so when I let anyone else use the computer I f*king log out (as I do with email) and as anyone with half-brain understands [10:33] luv, well, the base line is "don't give access to an unlocked device/session to somebody you don't trust" in any case, you can improve things sure but that's not a proper security [10:33] mzanetti: i'll let you file that one :-) [10:33] tsdgeos: ok [10:35] seb128: let's add a showpass command which will show your linux password decrypted after you log in (maybe a bit of pam hacking?) [10:35] passwords are really a different beast than emails and im logs [10:36] and should be protected (they plain text version) even if you have access to an unlocked device [10:37] (that's why I never use google account with android afterall - even though android doesnt go that far to show you your google password in plain text either!) [10:37] they are indeed [10:37] it still doesn't mean it's a good idea to hand an unlocked session to somebody you don't trust ;-) [10:38] and Im telling you - most people would do same if they knew how weak the security really is :'( [10:38] oh, of course, no disagreement there! it's absolutely not a good idea :-) [10:38] mzanetti: quick one, https://code.launchpad.net/~aacid/unity/2many_regexps/+merge/158881 [10:39] tsdgeos: lol... you should remove the others too [10:39] mzanetti: which others? [10:39] tsdgeos: just joking... (not a big fan of boost) [10:40] luv, btw there is work ongoing around the keyring: https://blueprints.launchpad.net/ubuntu/+spec/security-r-app-keyring [10:40] mzanetti: ah [10:40] seb128: cool, I hope that's get sorted soon [10:40] makes me really upset at the moment :-( [10:41] luv, the security team is working hard on apps isolation atm, hopefully we see good progresses in the next monthes [10:43] mornin' [10:44] hey kgunn [10:44] good morning [10:44] seb128: umm, great! you think the patches can make it upstream or is that ubuntu only? (not that it matters that much to me ;-) ) [10:45] it's unfortunate it didn't make it to raring. But totally ok, given how my deadlines end up ;-). [10:54] luv, I'm not sure how much of that will be taken upstream, I guess the code to restrict access should be upstreamable, not sure about the profiles since those are centered on apparmor and not all distributions use it [11:00] well, let's see how it turns out ... umm according to the document the funcionality used to be there and was removed :-S === MacSlow is now known as MacSlow|lunch [11:19] Saviq: I just checked out the *FilterGrids... seems like reusing tst_FilterGrid is not an option. you think its still worth to add tests given that they just use FilterGrid + Tile and both of them are tested already [11:19] ? [11:20] Saviq: ok... they define a clicked signal... that can be tested... I'll create something [11:24] fginther: ok to me to kill every old autopilot jobs which are != from generic? === alan_g is now known as alan_g|lunch [11:37] didrocks: https://code.launchpad.net/~sil2100/qtubuntu-sensors/arch_change_to_list/+merge/158892 [11:37] didrocks: should I also remove the any bit from platform-api package? [11:37] Since it currently builds for powerpc as well [11:39] sil2100: qtubuntu-sensors is the only one which doesn't build, right? https://launchpad.net/~ubuntu-unity/+archive/daily-build-next/+packages [11:39] Yes, that's why I'm asking abotu tplatform-api [11:39] Since maybe we want to get rid of powerpc at all :> [11:40] (saving disk space and everything) [11:40] sil2100: no, keep any for those [11:40] sil2100: no need to constrain when it's not needed (especially the day we are going to add one more arch) [11:41] Ok ;) [11:43] Saviq, Would you have time to check this one? It's been around for a while now: https://code.launchpad.net/~dandrader/unity/phablet_remove_fakes_from_qml/+merge/158370 [11:44] I wonder why jenkins didn't build it again [12:01] also the "close apps from the dash" relies on that MR [12:12] dandrader, yeah, I will [12:12] mzanetti, why not an option? [12:13] mzanetti, could we not build the FilterGrid test so that it would take them all in order and test? [12:13] Saviq, thanks! === MacSlow|lunch is now known as MacSlow [12:16] kgunn, where from did you get that the UShape mounts and bottom bars are DONE? [12:17] kgunn, loicm assigned it to himself, but it doesn't seem INPROGRESS yet, even [12:17] Saviq: sorry, he emailed me... [12:17] kgunn, that's fine [12:17] kgunn, just asking [12:18] Saviq: but fair question...."what is done"....i better check its all landed [12:21] didrocks: heya, do you know if jenkins jobs automatically time out if they take too long ? [12:21] fginther: ^ might know the answer to that one too [12:21] (sorry for the ping!) [12:24] smspillaz: fginther will know better than I [12:25] coolio, we shall see then [12:26] if not, its pretty easy to pass a flag to ctest to force tests that take longer than 60 seconds to fail, we're just hitting a condition in the xorg gtests in compiz that times out after 3000ms, and sometimes running under valgrind hits that [12:26] so I was just going to get rid of that 3000ms timeout since its not really useful for getting rid of long-running tests === alan_g|lunch is now known as alan_g === seb128_ is now known as seb128 === alan_g is now known as alan_g|afk === alan_g|afk is now known as alan_g [12:34] didrocks: where would you put the qtubuntu-camera* bits in the stack? It was in the qt stack in phablet, I added it to platform for now - but you think it's the right place? [12:34] Maybe media? [12:34] But it's a backend [12:39] I'm trying to build unity (./build -s) but it keeps failing in: [ 79%] Building C object tests/CMakeFiles/test-voice.dir/test-voice.c.o [12:39] Linking C executable test-voice. [12:40] Anybody any idea how to fix? The log is the same as here: http://pastebin.com/bND2DCbj [12:44] sil2100: I guess it's in the media one [12:44] sil2100: look for the google doc from stacks [12:50] Saviq, any idea^ [12:51] vesar, that looks like the hud is failing [12:51] vesar, try ./build_unity --clean [12:51] vesar, that will build the whole set in ../unity_build/ from scratch [12:51] vesar, you updated to raring, btw? [12:52] Saviq, no I still have 12.10 64-bit [12:52] vesar, you should upgrade to raring, we will soon stop caring about quantal [12:52] Saviq, ok. good to know. but can that cause the issue? [12:53] vesar, it could, in theory (/me tries to build in quantal) [12:56] smspillaz, nearly all of our jenkins ci and autolanding jobs are set to timeout. A few old jobs do not, but they will eventually be fixed. [12:57] fginther: great, thanks! === alan_g is now known as alan_g|afk === _salem is now known as salem_ === alan_g|afk is now known as alan_g === alan_g is now known as alan_g|afk [13:13] Saviq, got a confirmation from Albert that doesn't work in quantal anymore. [13:14] tsdgeos, what would we need to do for it to work? [13:15] Saviq: probably comment out the tests [13:15] is what i think that fails linking [13:15] Saviq: maybe even we can trigger that on the cmake "call" level [13:16] tsdgeos, right [13:16] * Saviq will try [13:16] Saviq: otherwise it's hard since we are targetting a given revision on the repo so even if we fix it later won't be able to use that revision since it introduces the hud-client2 lib [13:16] tsdgeos, we can always have an lp:hud/phablet branch ;) [13:17] sure [13:17] actually it's the branch we are using :D [13:17] it's called phablet already [13:17] :D [13:17] right ;) [13:24] sil2100, the last unity raring build had too many failures from autopilot tests on nvidia. I'm rerunning the job just to see if it was a fluke, but do any of the failures here make sense to you: http://10.97.0.1:8080/job/ps-generic-autopilot-release-testing/64/label=autopilot-nvidia/testReport/ ? === alan_g|afk is now known as alan_g [13:32] dednick_, standup? [13:38] sil2100, you don't hang out in #ubuntu-touch? [13:41] vesar, I'll have a fix for you in 5 [13:42] Saviq, ok. cool. Though I think I should upgrade to raring [13:42] vesar, yeah === dandrader is now known as dandrader|afk [13:54] mterry: ok, so, as I mentioned in the qtvideo-node, it looks all fine - I had problems building with earlier package versions, but I think it shouldn't be a problem? [13:54] (i.e. missing minimum versions of libplatform-api-headers etc.) [13:55] sil2100, yeah. I wasn't too worried about it because I didn't want to include versions that had the daily.build.next string; once we actually have these packages in the ubuntu-unity/next ppa it might make more sense to add versions [13:56] sil2100, but that merge failed because of the missing deps in the cu2d-config stack, eh? [13:58] tsdgeos, vesar https://code.launchpad.net/~saviq/unity/phablet.disable-hud-tests_add-regex-dep/+merge/158928 [13:58] mterry: yes, in the same way it happened on my system [13:58] i.e. libplatform-api-headers not new enough [13:59] Should we merge it in anyway? [13:59] sil2100, no...? let me look at the console log again [13:59] shadervideomaterial.cpp:17:30: fatal error: ubuntu/ui/config.h: No such file or directory [13:59] sil2100, oh right! [13:59] ubuntu/ui/config.h is from libplatform-api-headers - at least the https://jenkins.qa.ubuntu.com/job/qtvideo-node-raring-armhf-ci/5/console job! [13:59] sil2100, this is because https://code.launchpad.net/~mterry/cupstream2distro-config/videonode/+merge/158408 hasn't landed [13:59] sil2100, the -ci job is still using old phablet branch [14:00] Ah ;) [14:00] brrrr [14:00] Saviq: ok, going to take a while to get the quantal machine up [14:00] Things are getting out of hand slowly! Ok, so first let's get this merged in [14:00] tsdgeos, should've kept a clean snapshot ;) [14:00] i do [14:00] just cloning takes a while [14:00] no ssd here [14:01] a while ~10 min [14:01] sil2100, can we kick jenkins-bot to check for the commit message again? [14:01] tsdgeos, why cloning? just snapshot, work, drop the changes? [14:01] can virtualbox do that? [14:01] tsdgeos, yeah [14:01] mterry: I think if we trigger a rebuild it should re-check it (I think) [14:01] Doing that [14:01] sil2100, thanks [14:02] tsdgeos, you can even have multiple trees of snapshots now [14:02] Saviq: interesting, i'll investigate later [14:02] tsdgeos, cheers [14:02] sil2100, still though, we need to add the autopilot deps to the stack configs. The stacks are failing to pass checks because of that. Do you know what I mean by that? (the stack configs list all the deps of the autopilot packages explicitly) [14:02] i think for this time it's going to be faster just to clone :D [14:03] sil2100, I was going to have a look at that if you hadn't started === dandrader|afk is now known as dandrader [14:03] mterry: yes, I had a hangout about that with Didier today [14:03] mterry: I'm taking that one on me, just wanted to finish those small tasks first [14:03] sil2100, OK cool, thanks [14:05] Saviq, thanks! [14:22] fginther, in the qtvideo-node branch, the dropping of quantal was intentional [14:22] fginther, the head stacks are raring-only [14:23] fginther: pushed the modified version of https://code.launchpad.net/~sil2100/cupstream2distro-config/android-audiosystem_add/+merge/158806 [14:23] mterry: you're handling the addition of qtvideo-node to the head stack, yes? [14:26] * fginther is thinking [14:26] To anyone concerned - once we have all camera* branches ready, here's the stack addition: https://code.launchpad.net/~sil2100/cupstream2distro-config/qtubuntu_camera_additions/+merge/158931 [14:26] (so that there's no work duplication) [14:26] Probably some fixing will be needed later [14:27] sil2100, trying [14:27] dednick_, so basically from Lens I need searchQuery and name [14:27] mterry: ACK [14:27] dednick_, so I'll need to add some methods right? [14:28] dednick_, one get method for name, a write and get method for searchquery, and a q signal each [14:28] Cimi: yah. might want to separate the lens into a different source file as well. i just put it in with the lenses to save time. [14:28] Cimi: think you need to add Q_PROPERTY for each [14:29] dednick_, yeah exactly [14:29] dednick_, was implying the property [14:29] Cimi: ok :) [14:29] dednick_, shall I push there once I move into a new file? [14:31] Cimi: please === alan_g is now known as alan_g|tea [14:31] ok [14:36] hey Cimi. wanna do a review? https://code.launchpad.net/~mzanetti/unity/phablet-test-filtergrids/+merge/158941 [14:39] Trevinho: heya, do you happen to know if there's any udev events for VT switches ? === alan_g|tea is now known as alan_g [14:46] smspillaz: no, I've looked a little into it at the time, then we got a different solution.. [14:47] smspillaz: platform guys could be more aware than me, though :) [14:47] Trevinho: I haven't found anything thus far [14:48] Trevinho: if we don't really care about VT switching, we can just try to handle the resume case [14:48] https://code.launchpad.net/~jassmith/unity/unity.redraw-on-resume/+merge/95945 does it for the launcher, should we just expand that to the rest of the shell ? [14:48] smspillaz: that was handled using upower signals [14:48] smspillaz: however it was not fixing things that well [14:48] Trevinho: the problem is that its the best we've got -.- [14:49] smspillaz: ok, if that's fixing for everything I think we can accept it [14:49] I'll look into hooking those bits up [14:50] Trevinho: Actually, now that I think of it, it might be better to connect to the screensaver daemon and listen for unlocks [14:50] as well as resume from suspend [14:50] the reason being that I suspect the reason it didn't work so well was a race condition between us getting the signal and the driver trashing the framebuffers [14:51] listening for unlocks on the screensaver service at least guaruntees us that we're going to do a redraw when the user has typed in their pw [14:51] smspillaz: the problem with that was that it didn't work when user was disabling the lock-screen [14:52] smspillaz: and OEMs want to support that as well [14:54] Trevinho: right, so I was thinking of putting the redraw code on a timeout (5 seconds will do) and then for all other cases the lock screen signal should handle it [14:54] Trevinho: its just a race condition really, there's not a whole lot we can do about it [14:55] better yet, we can probably do it after the first swap after resume [14:56] so resume -> first swap -> QueueDraw everything [15:13] Saviq: what's the policy regarding new merges that introduce FIXMEs like https://code.launchpad.net/~nick-dedekind/unity/phablet-tests-menucontent/+merge/158562 ? Should we update the FIXME list? or? [15:13] tsdgeos, good point [15:14] tsdgeos, yeah, I'd say that's right [15:14] tsdgeos, fortunately they won't get lost, but makes sense to add them to the list, yes [15:14] oki [15:24] Trevinho: is there a way that I can tell nux "just QueueDraw everything" or do I need to expose a new method in WindowCompositor to do that ? [15:26] smspillaz: maybe at this point it's just better to include that into WindowCompositor, and using it in compizDamageNux as well [15:26] smspillaz: as there's not... [15:26] Trevinho: well, I'm poking around a bit to see what there is [15:26] there's "Draw" with "force_draw" dunno if that refers to the content or the presentation stage [15:27] * smspillaz would like to avoid breaking its encapsulation if possible [15:32] Saviq, updated https://code.launchpad.net/~dandrader/unity/phablet_remove_fakes_from_qml/+merge/158370. I bet something got merged in the meantime disabling use of "signals" keyword by Qt. [15:33] dandrader, if it wasn't before, it should've been ;) [15:33] dandrader, but yeah, we're doing -DQT_NO_KEYWORDS for compatibility reasons [15:35] dandrader, s/emit/Q_EMIT/, too [15:39] Saviq, done === dandrader is now known as dandrader|lunch [15:42] Trevinho: okay, just rebuilding nux now to give that a try [15:59] tvoss, do we have html of the development guidelines published somewhere? === dandrader|lunch is now known as dandrader [16:47] Saviq, nope, moved it to the top of my list [16:50] fginther: is it ok now? https://code.launchpad.net/~sil2100/cupstream2distro-config/android-audiosystem_add/+merge/158806 [16:51] sil2100, sorry, got sucked into something else. I'll take a look now [16:52] tvoss: ping [16:53] tvoss: when is Unity Next development going to become our main focus, will that happen before the client sprint, or afteR? [16:54] mhall119, I think we are already focusing it :) but best to ask kgunn and Saviq for concrete timelines === jasoncwarner__ is now known as jasoncwarner === alan_g is now known as alan_g|EOD [17:06] mhall119: tvoss: it will be in the distro during June [17:07] mhall119: its actually our main development focus now [17:18] kgunn: good to hear! So my next question is, what can I to do get more people in the community actively involved in it's development? [17:19] mhall119: actually, for low hanging fruit fixme/todos [17:20] mhall119: in this bp https://blueprints.launchpad.net/ubuntu/+spec/client-1303-unity-ui-iteration-0 [17:20] mhall119: there is a link to a google doc (completely public) [17:20] mhall119: which has a list of the todos/fixmes in the code that need to be addressed [17:21] mhall119: and some estimate of difficulty etc [17:21] mterry, didrocks: http://10.97.0.1:8080/job/ps-generic-autopilot-release-testing/label=autopilot-intel/68/artifact/results/artifacts/unity.tests.launcher.test_keynav.LauncherKeyNavTests.test_alt_f1_closes_hud%20%28Single%20Monitor%29.ogv [17:21] sil2100, ^ [17:21] from the last buggy run [17:21] screen is locked! [17:22] seb128: let me restate… "it's all your fault" [17:22] :p [17:22] he [17:22] we have an inhibit api [17:22] it's made to be used :p [17:22] kgunn: do we have any documentation on the code layout, different components and what they do, what functionality is implemented where, etc? Something to give new developers a understanding of the codebase as a whole? [17:22] seb128: tssss, that's overrated ;) [17:23] sil2100: thomi: is there a way that autopilot in his setup have something to launch the inhibit api for avoiding screenlocking when using autopilot? [17:24] mhall119: not really, its been in flux lately....but starting to settle....and since most of its qml....its not hard to figure out whats what [17:25] tvoss, thanks, I have a QML set in the works, but it's still some work away [17:26] didrocks, seb128: not sure, since I never used the inhibit API, would have to check how it's done [17:26] mhall119, yeah, we try to name the components after what they are [17:26] sil2100: quite easy, I have a small snippet [17:26] mhall119, and we are here all the time if people have questions [17:26] or try to be [17:26] sil2100: http://paste.ubuntu.com/5710940/ [17:27] sil2100: see test_inhibit() [17:28] Saviq, let's sync tomorrow on that topic [17:28] tvoss, k [17:28] sil2100: but I think this need to be at the autopilot level, like a facility [17:28] and called at the start of autopilot [17:29] didrocks: thanks! Let me test something in Ap [17:29] thanks ;) [17:30] seb128: good catch! [17:30] didrocks, thanks ;-) === dandrader is now known as dandrader|afk [17:33] Got a quick question, I used to be able to maximize a window and it would hide the controls and title-bar in the panel which was awesome but now using 13.04 for some reason the window no longer maximizes normally, is there a setting for this? [17:36] didrocks, sil2100: other (easier) option, could be to "gsettings set org.gnome.desktop.lockdown disable-lock-screen true" [17:36] that's what we do in e.g the guest session [17:36] sounds good to me :) [17:37] Ok, we'll do that if this will be too troublesome anyway === dandrader|afk is now known as dandrader [17:57] Saviq: kgunn: thanks [18:01] Man, AP killed my system so many times... [18:07] https://code.launchpad.net/~sil2100/unity/autopilot_disable_screen_lock/+merge/158989 [18:32] mterry: regarding https://code.launchpad.net/~mterry/qtvideo-node/arches/+merge/158405 [18:32] Soo... [18:32] mterry: we get a failure on quantal since there is no new libplatform-api-headers etc. available on quantal [18:33] mterry: and as you already know, we can't disable it in CI [18:33] So what I'm proposing: [18:33] We could merge it in as it is, but then every merge would fail CI on trunk there, so I would propose to push a new libplatform-api-headers package for quantal to the daily-next PPA [18:34] Same for libhybris (since I think this one also was problematic) [18:34] mterry: this way CI would pull the new packages and not fail on quantal anymore - what do you think? [18:35] sil2100, we need to just run the daily job again after changing the config to support quantal [18:35] sil2100, that way the new platform-api will be in the ppa for quantal [18:40] mterry: awesome [18:41] mterry: will you take care of it? [18:41] sil2100, I think fginther said he was going to do that [18:41] I mean, change the config to support quantal [18:42] mterry, fginther: thanks guys [18:46] mterry, I'd like a clarification. 'new platform-api will be in the ppa for quantal' what ppa are you referring to? [18:47] fginther, I'm talking about daily-next [18:47] Once we enable quantal for it [18:51] mterry, are you aware that daily-next is not used for building CI? [18:56] fginther, I guess not... :) Doh. What PPA do we use for CI? [18:57] mterry, it can vary per project, but for qtvideo-node, it's usingqt5-proper and phablet-team-ppa [18:57] mterry, we can change that, but I didn't want you to spend a lot of effort in supporting something that we may not be using w/o more changes [18:59] fginther, where is that specified? [19:00] mterry, by the "hooks" line [19:00] fginther, but we just landed my merge today that changed that [19:00] fginther, moved it into head/media.cfg [19:01] mterry, and I'm adding those back in my update [19:02] fginther, ah... but why drop daily-next from the list of PPAs when doing CI? [19:04] mterry, we don't drop it. Our CI tools just don't know to look at those 'ppa' and 'dest' values yet. [19:05] fginther, oh. Huh. So can we have a global hook that adds the daily-build-next PPA? It seems like that should be used for CI [19:05] For raring, I'd like the daily-build-next PPA to be self-sufficient (i.e. build without other PPAs) [19:05] I understand that for quantal, we'll need to have more stuff in there [19:05] mterry, yes, we need to figure out a way to do that and add it [19:05] But even then, we could just put it into the daily-build-next PPA like we've done in raring [19:05] there may even be a bug, let me look [19:06] I'd like to not add the phablet PPA if possible [19:09] mterry, I understand, let me try and address the ppa issue first. We haven't yet deployed these changes in the media.cfg, so the old ci jobs will continue to work for the time being (I know that doesn't help your qtvideo MP) [19:10] mterry, do we need to add both "ppa: ubuntu-unity/daily-build-next" and "dest: ubuntu-unity/next" when building CI jobs? [19:12] fginther, CI jobs don't have a dest, right? [19:12] mterry, I simply don't know the purpose of those two PPAs [19:12] mterry, we just need a source to pull in deps [19:13] fginther, daily-build-next is pre-validation. Dest is final, validated result. For example, for the raring stack, the dest is raring I believe [19:13] fginther, for pulling in deps, daily-build-next is your ppa [19:17] mterry, thx [19:18] Saviq: wondering, we may have missed 2 items on indicators [19:19] Saviq: "account settings" & cell radio [19:19] kgunn, at least cell radio is supposed to be handled by the network indicator [19:20] Saviq: that makes sense [19:20] kgunn, and by account settings you mean? [19:20] Saviq: account settings i suppose is the little fluffy cloud on my desktop panel [19:20] that's sync [19:20] I think we have that covered [19:21] Saviq: you are right we do....looking at Seb's list [19:21] Saviq: he called it account settings...i dont see a sync [19:21] kgunn, line 41 in the draft monthly plan [19:22] Saviq: right...if you look at the "services" tab....there is basically a duplicated structure that Jason & Seb are working on [19:23] kgunn, indeed [19:23] kgunn, so they're mostly stepping on thostr's toes rather than ours [19:24] kgunn, and also I'm not sure indicators === settings [19:24] or rather I expect settings(indicators) [19:25] Saviq: sure...they are different, yet linked [19:25] kgunn, yeah, I expect indicators to be a view into a subset of settings [19:32] kgunn, that's not to say we should ignore them, not at all [19:33] kgunn, at the very least we should look at the commonalities between them (will the indicators really be a subset of settings when it comes to used widgets, for example? how do we share them?) [19:33] Saviq: i'm putting a note in on their page for when we think panel ui & backend are done [19:33] on the month2month [19:34] kgunn, but I haven't seen indicators designs even as rudimentary as those on the mentioned wiki === salem_ is now known as _salem [23:33] slangasek: hey, still around? Thanks for the review, I'll deal with the easy bits though if you're around I can deal with some of the more complicated bits over IRC