[00:00] mas: you might be interested in http://design.canonical.com/2012/01/launcher-reveal-prototype/ and leaving your preferences in the comments [00:00] jono: shell or django? [00:00] mhall119, django [00:00] i've been contemplating it myself, i was thinking moving it to the bottom might help but shuttleworth keeps has some obsession with everything having to be to the left. it would also be nice to see some sort of feedback from the buttons in the launcher, like when the mouse is over them they glow or hover. they feel so "static" [00:01] jono: sure, give me a minute [00:01] mhall119, thanks! [00:02] oh thanks mhall, i'll leave some comments [00:03] mas: it's not so much wanting it on the left, it's wanting it to use up the dimension you have the most space in [00:03] mas, some hover or glow might be really good for button feedback. [00:03] I'd rather lose 48px of 1024, than 48px of 786 [00:05] If my opinion and experience with unity on corporate machines. The launcher hidding and revealing it self on left side confuses people. [00:06] I wonder my self, if the launcher be moved to bottom it could looks a lot like the win7 bar. But it would be a lot easier to switch betwen tasks. [00:06] hmm, i've heard that argument before about losing in the horizontal versus the vertical. not to say i disagree...however i never hear anybody talk about symmetry. it can sometimes be visually unpleasant. also the space argument doesn't really matter when you maximize as the launcher disappears anyway [00:08] mhall119, your observation makes sense in the logical point of view. But, from a usability vision, users are used to look at the bottom bar and find its running apps and menu. i think. [00:09] what has confused me about the entire argument is why the option of not being able to move it around to any side the user wants is not desired. that was something i always admired about *nix desktop environments. i rarely deviate from the default desktop, but sometimes one or two tweeks depending on the end user's workflow can make a world of difference [00:11] i mean even mac, the ultimate in controlling the end user's experience let you move the dock around [00:12] i wonder how RTL language users feel about it === htorque_ is now known as htorque [00:22] quit [00:23] bgois: it doesn't take that long to learn to look on the left [00:23] we shouldn't treat users as if they are incapable of learning a better experience [00:24] htorque: that's an interesting question [00:42] hi mhall119 [00:43] are we moving all ayatana chatter here?\ [00:44] what else what talked about on ayatana? [00:45] htorque: I'd love to see some research on RTL languages and launcher expectations [00:45] thumper: moving it here [00:45] we'll put in a redirect on #ayatana in about a week [00:45] mhall119: we should have a call next week [00:45] I'm not actually working today [00:45] thumper: yes please [00:45] 62 hours travelling home [00:45] just back this morning [00:45] eeew [00:45] sure, I'm pretty flexible, so whenever is good for you [00:46] mhall119: where are you based? [00:46] USA, east-coast [00:46] ah cool [00:46] just 3 hours different then [00:46] perhaps Monday afternoon your time [00:46] I assume you're west-coast then [00:46] that matches up with after lunch tuesday for me [00:46] or the middle of the atlantic [00:46] Nope [00:46] NZ [00:46] wait... [00:47] well three hours + one day [00:47] three hours ahead and a day behind [00:47] that still doesn't sound quite right, what time is it there now? [00:47] 13:47 [00:47] damn [00:48] ok, so 6 hours different [00:48] east and west mix up again [00:48] that sounds better [00:48] I do that all the time [00:48] no problem [00:48] you are in the same tz as lamalex, DBO, and jay [00:49] and I don't know any of them.... :( [00:49] you were ISD before? [00:49] yup, until the end of December [00:50] just looked you up on the directory [00:50] thumper: give me a ping monday morning (your time) and we'll setup a time for a call [00:50] I don't think we've met have we? [00:50] maybe in passing at UDS [00:51] I've not been to a UDS since Boston [00:51] ok,then not in passing at UDS [00:51] even then, it was a co-located launchpad sprint [00:52] you were in the launchpad team? [00:52] for 4.5 years [00:52] switched last April to DX [00:52] then it's very possible that I've bugged you about something before ;) [00:52] which is now product strategy [00:52] maybe [00:52] summit or loco-directory perhaps [00:53] * thumper has to head off for stuff [00:53] we'll chat next week === Justin____ is now known as Justinba1010 [01:41] hello [01:42] anyone here? === JoseeAntonioR is now known as JoseeAntonioR1 === JoseeAntonioR1 is now known as JoseeAntonioR [02:50] Hello. Is here the best place to give suggestions about Unity? === nick is now known as Guest94462 [03:28] Hi guys [03:28] I'm running natty here, I think I found a 'security' bug in Unity [03:28] Can someone confirme me if it happens on oneiric?? [03:28] Sure. [03:29] The bug is the following: [03:29] 1 - Press the super key (will launch unity 'default' view) [03:29] 2 - Wait to computer to block [03:30] block? [03:30] As in session block [03:30] As you can see, you can still do a one time usage of unity. [03:30] Ah so the lock screen? [03:30] yep [03:30] the lock screen dont appear directly, you can 'use' unity for a command [03:30] I think what happen is that the unity launcher stays on top of the gnome-lock-screen [03:31] and you can use unity one time, once it dissapears (the unity windows) it goes to the password of the lock screen [03:31] I'll see [03:31] Try it :) [03:31] Put your comp session lock < 2 min [03:32] and press the super :) [03:32] wait, and profit [03:32] you'll get the chance to explore unity hehe, once you get out of the unity screen the computer goes to the lock screen (session) [03:32] :~ [03:34] I don't know if this is a security bug, but IMO after the period that you specified (for instance, 2min) your computer should go automatically to the gnome-session screen with the pass. [03:37] Did you hold the super key, or press and let go? [03:38] my screen turned off and locked, it came back to the lock screen though, not unity [03:38] press and let it go [03:39] Yeah that's what I did, didn't happen here [03:39] Strange :~ === IdleOne is now known as pangolin [06:32] hi all [06:33] anybody here? [06:34] whois JanC JanC [06:39] hello [06:39] bye [06:52] hello [08:31] does anyone know how we sort the lenses in the lensbar in the dash? [08:34] kamstrup, preliminary look at 2d code says that we do whatever you guys give us [08:35] Saviq: yeah, and u3d also seems to take whatever comes out of the FilesystemLenses class [08:35] kamstrup, yup, same here it seems [08:35] but the sorting order in the dash seems to be stable, so there must be something sorting it [08:35] kamstrup, "m_unityLenses = new unity::dash::FilesystemLenses("/usr/share/unity/lenses"); for (unsigned int i=0; icount(); i++) [...]" [08:35] maybe it's really just sorting on inode accurence within the /usr/share/unity/lenses dir... [08:36] Saviq: which order are your lenses in? [08:36] i have home, apps, files, music [08:36] same here [08:36] any additional get added later [08:36] hmmm, so something is likely to sort it [08:36] s/likely/certain [08:37] yup [08:37] Saviq: AHA! vvvv [08:37] /FIXME: We don't have a strict order, but alphabetical serves us wonderfully for [08:37] // Oneiric. When we have an order/policy, please replace this. [08:37] ookies [08:38] in FilesystemLenses [08:38] yup, was just going there [08:39] except the app lens seems to be forced first [08:39] ok, I am actually implementing that policy in my home-lenses branch, so I'll put it on my todo [08:39] right [08:39] and we probably want the home lens to be first in Precise [09:05] Hi [09:22] hey greyback [09:27] mhr3, so your icon loading branch is great, but i think it needs tests - just one that loads a bunch of icons and makes sure they all come in [09:27] Saviq: morning [09:28] gord, i knew you'd said that [09:28] frck! [09:36] Saviq: I think I have addressed all your concerns in the shape tests branch. just pushed the last changes now [09:36] greyback, we need to set locale to en_something in the tests.. [09:37] otherwise I'll waste many more hours wondering what the frck [09:37] or at least require a "en_*" locale for the tests [09:38] Saviq: yep. but then we should also have some localization tests [09:38] Saviq: yeah. Getting tests running on every developers machine correctly is a pain. Why I want to use VBox [09:38] nerochiaro: agreed, and also needed for RTL tests [09:38] greyback, still, I've changed the locale on my vbox to check out RTL yesterday [09:39] greyback: Saviq: i'm also worried about some tests that will require resolution changes (for example to test the min size of the dash) [09:39] and since then I couldn't, for the life of mine, understand why the tests failed [09:39] (min size before it goes fullscreen by default) [09:39] Saviq: yep, my idea was to have a VBox Snapshot that everyone just uses and won't be changed [09:39] greyback, that's not safe [09:39] greyback, tests should work on virtually every machine [09:40] greyback: yeah, but what do you do for tests where a certain feature is for a configuration that is impossible to obtain on a certain machine ? [09:40] if we just have a snapshot that noone changes, the tests won't get enough coverage, tbh [09:40] Saviq: ^ [09:40] nerochiaro, example? [09:40] nerochiaro: you'll have to add checks to ensure the resolution is big enough for the dash not to be full-screen, and bypass the test with a warning maybe [09:40] Saviq: multimonitor tests ? [09:41] nerochiaro, true, what greyback said - just check whether the env is ready for the test [09:41] and bypass it [09:41] exactly what I want to do for locale [09:41] check if it's en_whateveryouguysuse.UTF-8 [09:41] and drop out of the test suite [09:41] or change the locale for the duration of the tests [09:44] nerochiaro: saviq: In the shape-tests branch i checked move the dash bit away from launcher [09:44] nerochiaro: saviq:actually to the second screen [09:44] nerochiaro: saviq: the dash is no more visible there [09:44] dyams, meaning multimonitor? [09:45] saviq: yes [09:45] dyams, we didn't yet do anything re multimonitor in -shell [09:45] dyams, but you need to remember there is no "dash" and "launcher" anymore [09:45] yep, no multimonitor support in shell for now [09:45] there's just a single window [09:45] which has both the dash and the launcher [09:46] so if you've moved the dash over screen, it will just go overscreen, not to the next screen [09:46] saviq: sure, but if i move the window bit away from launcher, then any portion of the dash falling outside the current screen is cut off [09:48] nerochiaro: saviq: Yes, I didn't expect it will work perfectly by just reposition it [09:48] nerochiaro: saviq: yet i didn't expect it will be clipped either. [09:48] dyams, what are you repositioning? the whole shell window or just the dash inside? [09:48] dyams, yes it will be clipped [09:48] our window is the size of your current screen's available geometry [09:48] saviq: I reposition dash only [09:49] if you go over that, it will get clipped, yes [09:49] dyams, there's simply no support gone into -shell yet [09:49] saviq: i see , height: screen.availableGeometry.height [09:49] width: screen.availableGeometry.width [09:50] saviq: Ok [09:50] nerochiaro, that fits into the merge-todo, do we have that there yet? [09:50] nerochiaro, at least getting on-par with trunk [09:51] nerochiaro, so you didn't manage to work with _NET_WORKAREA in the end? [09:51] Saviq: i don't think we have it yet [09:51] nerochiaro, ok I'll add it [09:52] Saviq: i decided not to, it was quicker this way and i'll also be forcing the panel to be there for the dash button tests, so i thought might as well do the same in that case [09:53] lazies :P [09:53] ok [09:53] Saviq: i'll add the test to verify the correct shape in absence of the dash when i add the ability for the shell window to correctly adjust in case the panel appears and disappears [09:53] k [09:55] nerochiaro: saviq: 1) Invoke dash, 2) invoke spread, [09:55] 3) Cancel the spread [09:55] Now dash is reappearing [09:56] moreover now dash is not closing either [09:56] didrocks: welcome ;) [09:56] hey didrocks [09:56] dyams: sounds like a bug in shell. let's add to the TODO in the wiki please [09:56] greyback: hey ;) Seems I need to add the autojoin here ;) the super + keynum seems to not work with the numpad in 2d, do you have a bug report on that? :) [09:57] didrocks: designers say it's not supposed to [09:57] dyams: ^^ [09:57] hum? [09:57] a lot of people marks the tests are failed because they try to use it [09:57] didrocks: greyback is correct [09:58] greyback: didrocks: actually super + numpad with launcher? Or with windows? [09:58] dyams: launcher [09:59] oh that will be the shortcut for maximinizing the window now [09:59] forgot about it [09:59] dyams, you need to run the spread from the shell branch [09:59] nerochiaro, ^^ [09:59] didrocks: https://bugs.launchpad.net/unity-2d/+bug/750514 [09:59] didrocks: Super + Numpad [1] [2]....[0] Is not supported by launcher [09:59] hey Saviq ;) [09:59] nerochiaro, dyams: works fine for me, but you need to run the spread from the shell branch [09:59] Saviq: oh, right, forgot that might be the issue [09:59] not the installed one [09:59] saviq: trying again.. [09:59] dyams: well, it was for Natty, but the design changed :) [10:00] didrocks: so no Super+numpad does same as Super+num ? [10:00] s/no/now [10:00] greyback: in 3D right [10:01] didrocks: oh they changed it..? Or is it a bug in 3D? [10:01] Saviq: let me know when you are done with the review, so i can pull it into another branch if it's ok, or fix what's wrong if not. thanks. [10:01] dyams: I can tell for sure it's not a bug in 3D and they ask for it as I spent time to support it :) [10:01] stop assuming things are bugs in 3d you guys :P [10:01] anyway, there is this new shortcut stuff where it should maximize/semi-maximize window now [10:01] nerochiaro, seems right already, I'm just verifying that the tests fail without the shell being shaped [10:02] didrocks: lemme crosscheck with the shortcuts gdocs once [10:03] didrocks: am consulting with design now :) [10:03] thanks :) [10:04] dyams: John showed me this gdocs, but I can't find it anymore [10:04] didrocks: hrer https://docs.google.com/a/canonical.com/document/d/1jqeKtIJwqLtl58Wk_fqjr9Rrgxn9zsouCYOo-cZsLSE/edit?authkey=CLGG9NkJ&hl=en_GB [10:05] dyams: thanks [10:06] dyams: hum, no, it's ctrl + alt in fact [10:06] so nothing on super + numpad [10:06] seems a lot of people expect it to do the same than super + keynum [10:06] didrocks: see the other channel :) [10:08] dyams: dust off that Super+keypad branch, it's going in :) [10:09] greyback: :) sure, I can do it... :) [10:11] greyback: should we wait for shell work or we merge it current trunk already? [10:13] dyams: as ultimately we'll be using shell, there's little point in merging it to trunk only. [10:13] dyams: see how easy it ports to shell [10:14] greyback: Ok, I'll work on shell branch [10:15] Saviq: I think adding features should be done to shell, if they don't apply cleanly to both branches. Otherwise we're making more work for ourselves [10:15] saviq: Yes, Spread works better if I launch it from shell itself [10:15] dyams, "better"? not "correct"? [10:16] saviq: but that didn't solve the issue either [10:16] greyback, true [10:16] greyback, if they fit nicely in both - then both [10:16] otherwise - shell it is [10:16] saviq: two issues i noticed. [10:16] Saviq: agreed [10:17] saviq: 1) Launcher won't stay open when spread is active, it disappears after a few seconds [10:17] weirds, should be forcedVisible, add to the wiki please [10:18] 2) after 3-4 attemps dash stays open and won't go away [10:18] saviq: sure [10:35] nerochiaro: You already added the Dash clipping issue in wiki? [10:35] nerochiaro: Or is it not required? [10:36] dyams, there's no "dash clipping" [10:36] that is not an issue [10:37] saviq: Ok [10:37] dyams, why would you move the dash itself anyway? [10:38] the dash is supposed to be stuck to the launcher, and proper multimonitor support will come later [10:38] Saviq: multi-monitor [10:38] dyams, we won't be moving it [10:38] we will have a dash per-screen [10:38] just as we will a launcher per-screen [10:38] Saviq: no [10:39] dyams, I know, it will just be shown on one screen [10:39] current screen [10:39] but that doesn't mean we will necessarily move the window around [10:39] saviq: Ok [10:40] saviq: then it is like shell per monitor. no? [10:40] dyams, yes [10:40] saviq: i got it :) [10:45] booyacacha mhr3, gord! nailed the home-lenses despite njpatel trying to trip me up with his tricks! :-D [10:45] woohoo [10:45] he's trixy [10:45] just adding some tests [10:45] then mp [10:46] kamstrup, you know omg already wrote about this stuff right? can't disappoint them now ;) [10:46] gord: indeed :-) that kinda upped the pressure a bit... [10:46] gord: as you'll learn soon enough, this was not exactly trivial to get right [10:46] I basically wrote an in-memory relational db :-) [10:47] oh dear, /me gets his finger over the rejected button [10:47] lol [10:47] comment: stop re-writing mysql every six months [10:47] boring [10:48] it would actually be interesting to rewrite the internal lens handling by using in-mem sqlite tables. Will be quite fast, and actually not too ugly [10:49] kamstrup, so... i suppose the the merge diff in counted in dozen thousands lines? [10:49] is counted in*? [10:52] indeed [10:53] nerochiaro, I'm pushing just the input shaping tests to lp:~saviq/+junk/shell-shapetests [10:53] nerochiaro, can you try them? I can't seem to be able to fail them even without input shaping.. [10:53] Saviq: how do you remove input shaping ? [10:53] nerochiaro, still pushing, though [10:54] nerochiaro, I just merged the commits that did the tests [10:54] not those that actually implemented the feature [10:54] nerochiaro, pushed [10:54] Saviq: on top of unity-2d-shell ? [10:54] nerochiaro, yes [10:54] nerochiaro, btw, -shell has stuff merged from trunk again this morning [10:55] i saw that [10:55] let me try [10:55] I freakin' can't fail those tests... wth [10:56] Saviq: checking. might be something wrong with waht i do [10:56] nerochiaro, btw that's why it's good to write the tests first, too ;) [10:56] otherwise you never know if they actually fail :] [10:57] or at least make it easy to only merge the tests and not the fix itself [10:57] brb [10:57] Saviq: you have a point [11:04] Saviq: they don't seem to fail here neither. i'm looking into why [11:05] Saviq: are you sure you picked up all my changes ? in your branch there's some commits missing it seems. i'm double checking i pushed all [11:06] Saviq: revision 934 (which i added this morning) is the one that should make the tests properly fail [11:07] nerochiaro, oh [11:07] nerochiaro, will check out again in a bit [11:07] Saviq: okies [11:07] Saviq: i double check and i pushed all of them. the last on top of my branch should be 935 now [11:19] greyback: Saviq: or anyone: panel buttons for dash submitted as an MR, for whoever wants to take it: https://code.launchpad.net/~unity-2d-team/unity-2d/unity-2d-shell-dash-panel-buttons/+merge/89414 [11:19] i'll bbiab [11:40] kamstrup, about matching the sorting of lenses in the Home Dash to the lens bar. Would it mean that the lens bar wouldn't be sorted alphabetically on the lens name anymore? [11:41] davidcalle: depends... :-) [11:41] davidcalle: it could be, but not if we let users override [11:43] i can imagine that in some languages the ordering isn't what we want [11:44] unless it's being sorted on non-localized name [11:44] kamstrup, anyway so what was the thing you wanted me to look at yesterday? [11:44] mhr3: some tweaks to the music lens [11:44] kamstrup, ok. I'm just going to make an Aardwark lens to mess with you guys. ;-) [11:45] Aardvark* [11:45] kamstrup, oh dear :P [11:45] is there a bug opened? [11:45] mhr3: when doing global search, we need to collate the Songs and Albums categories into one Music category (with albums first) [11:46] mhr3: it's in relation to the new homescreen, it's mentioned briefly there [11:46] mhr3: I talked with JohnLea about it [11:46] mhr3: I have a partial branch ready... hold on I'll upload it [11:46] Dee.Transaction comes to rescue us :) [11:49] mhr3: you can save yourself a bit of time if you start from lp:~kamstrup/unity-lens-music/home-lenses [11:59] nerochiaro, so here's what I'm doing: `bzr branch lp:~unity-2d-team/unity-2d/unity-2d-shell-shaped-testability-shapetests` [11:59] nerochiaro, `bzr merge -r 913..907 .` [11:59] there are two small conflicts [11:59] but that gets rid of the actual shaping code [11:59] Saviq: I don't understand why you do that merge [12:00] nerochiaro, I want to get rid of the actual shaping [12:00] Saviq: to test that the tests fail ? [12:00] I want to have shell with shape tests on top [12:00] yes [12:00] and they don't [12:00] Saviq: ok [12:00] nerochiaro, http://pastebin.ubuntu.com/ [12:00] facepalm [12:00] nerochiaro, http://pastebin.ubuntu.com/810617/ [12:01] Saviq: only one fails [12:01] hmm [12:01] Saviq: can you check that the function to compare window shape in the input_shaping.rb tests don't compare the sizes of the windows anymore ? [12:02] Saviq: if it does, then something is messed up again and the branch is not current [12:02] Saviq: but honestly to test that the tests fail if the shaping isn't there, i'd just remove the call for XShapeCombine... [12:03] even though i understand why you're doing it that way [12:03] standup [12:03] sure [12:03] i'm there already [12:03] Kaleo, you comin'? [12:14] nerochiaro, lp:~saviq/+junk/shell-shapetests [12:15] Saviq: looking into it now [12:27] Saviq: it looks like on my machine it actually hangs when comparing the images in one case, i'm trying to find out why [12:28] Saviq: hangs as in IM compare takes 100% cpu seemingly forever [12:29] hi all [12:29] nerochiaro, isn't that why you were comparing the dimensions of the images? [12:30] that imagick hanged when they were different? [12:30] and also, they shouldn't ever be different, should they? [12:30] I have a problem with Windows Theme.... Here is a screen shot with current look:http://imageshack.us/photo/my-images/209/zrzutekranu201201201327.png/ ... is there any posibility to change it?:/ [12:31] Saviq: no, i was comparing the sizes because IM returned 0 difference when the sizes were different, and instead i should've checked the error code. but i guess it's a good thing to have both the size check and the error code check [12:31] Saviq: they should be different in the case when you want the tests to fail: without shaping applied the size of the window is not fullscreen [12:32] Mart_ini, looks like your gnome-settings-daemon crashed, did you try to log out and back in? [12:32] yeah:/ [12:32] and didnt help:/ [12:33] even restart of whole system [12:33] Mart_ini, you can change the theme in gnome-control-cente [12:33] r [12:33] yes I can but it didnt work [12:34] Mart_ini, try in #ubuntu, here isn't really the place for that [12:34] There I have few options like Ambiance , radiance, high contrast but even one of it changes nothing [12:34] ok [12:34] there's more people there that will be able to help [12:35] no problem, thanks:) [12:35] nerochiaro, sure, but the test should only pass if the images are the same size _and_ the input shapes are correct, right? [12:36] nerochiaro, I removed the unlinks, the masks you generate look ok [12:37] except [12:37] let me just have one at a time [12:37] ah no, ok - there's four [12:37] launcher, launcher + dash, launcher + collapsed dash, fullscreen [12:38] Saviq: to your first question, yes [12:38] and yes 4 tests [12:38] nerochiaro, they're not the same size, though [12:38] only the fullscreen one is [12:39] the rest are only bounding-box sized [12:39] Saviq: let's mumble a moment please [12:39] i.e. launcher is only the size of the launcher, shouldn't it be a white rectangle with a black rectangle on the left? [12:39] sec [12:46] didrocks: will you have time today to talk about singlet and quickly, or early next week perhaps? [12:47] mhall119: early next week [12:47] ok [12:47] I'll ping you Monday then? [13:01] Saviq: i'm a moron,i was using verify() { somevalue } instead of verify_equal('true') { somevalue } [13:01] or verify_true() {} ? [13:01] or that [13:01] nerochiaro, okies, good thing we caught that [13:01] that would be a non-useful test [13:02] Saviq: yeah. so i'll fix that and put back the size check [13:02] Saviq: push and let you know [13:02] nerochiaro, thanks [13:02] Saviq: thanks to you [13:03] sometimes I'm being an a$$ for a reason :] [13:03] good [13:03] Saviq: generally you aren't one [13:03] yeah, generally [13:03] ;) [13:03] well we all have our moments of assitude ;) [13:11] Saviq: pushed fixes. hopefully for the last time ;) [13:11] nerochiaro, cool, checking now [13:16] nerochiaro, yup, much better! [13:17] Saviq: awesome [13:17] kamstrup: https://launchpad.net/unity-lens-sample contains some very old code, what's the plan for that project? [13:19] nerochiaro, http://pastebin.ubuntu.com/810671/ [13:19] launcher+dash failed [13:19] that's with shaping [13:22] wtf, everything worked here [13:22] Saviq: ^ [13:23] Saviq: can you post the pngs somewhere, like imgur.com ? [13:27] nerochiaro, yup [13:27] nerochiaro, looks like it's comparing launcher against launcher+dash [13:27] might be it's trying to get the shape too fast [13:28] nerochiaro, yup [13:28] Saviq: sleep in the middle ? [13:28] nerochiaro, ye [13:28] s [13:28] Saviq: are you pushing that one yourself ? [13:28] nerochiaro, no, push [13:29] I need to straighten up my branches here first [13:29] don't want to push something weird [13:30] Saviq: 2 secs sleep ok ? [13:30] nerochiaro, 0.5 was fine here [13:30] nerochiaro, but 2 is fine [13:31] i'll do 1 then, if 0.5 was fine [13:31] I wonder if we can actually use the verify()s timeout here somehow [13:31] no idea how that works, though [13:31] Saviq: better not risk for now [13:31] but yeah that won't help either [13:31] since what will we verify [13:31] the image is there [13:31] just not up to date yet [13:33] * Saviq needs to read about the verifies and stuff [13:33] Saviq: in any case it's a bit weird to do these kinds of tests because ultimately you're still bound by the speed of the machine [13:33] nerochiaro, sure, that's why the verify_ stuff helps [13:33] I'm not sure how it works, though - will it check every second or? [13:33] Saviq: yeah, but it works only for things you do in the SUT IIRC [13:33] ok [13:33] it's syncronized somehow [13:34] Saviq: pushed the sleeps for now [13:34] that's why I said /me needs to reda about that [13:34] nerochiaro, cool, checkinf [13:34] -f [13:34] +g [13:36] :D [13:36] nerochiaro, "4 tests, 8 assertions, 0 failures, 0 errors" yay [13:36] im running precise on a few systems.. including this one which i installed from a daily build [13:36] approving [13:37] anyway.. on this system.. i cant visually tell the difference between a focussed window and a non focussed one [13:37] they all appear to have focus even if they dont [13:37] is there a ccsm setting for that? [13:38] nerochiaro, on a related note, did you get an answer about the actual input shape of the dash? should it incorporate the shadow or not? [13:38] nerochiaro, as the images now very clearly show it's wrong how it is now :) [13:40] Saviq: i don't think i did. and i didn't go check unity3d yet [13:40] let me [13:40] Saviq: but this is what we had in unity-2d [13:40] Saviq: so please let's commit this, then fix it in another MR [13:41] nerochiaro, already merged [13:41] I was curious.. [13:41] nerochiaro, aaand anyway... [13:41] 3d behaves completely different - it takes the whole screen when dash is active [13:42] Saviq: it's always fullscreen ? [13:42] nerochiaro, when the dash is open there's no interaction with the windows below [13:42] * snadge cries [13:43] Saviq: eww [13:43] Saviq: well, we're not going back [13:43] nerochiaro, not going back, but we might need to revisit our shapes later [13:43] Saviq: but how do they exit ? clicking outside of the dash area should close it [13:43] like have a fullscreen mousearea behind the dash [13:43] nerochiaro, yes, the lighter border already does that [13:44] does what ? [13:44] exits the dash [13:44] clicking on the lighter border around the dash will exit it, as will clicking anywhere outside [13:44] Saviq: but that border doesn't have a drop shadow ? [13:44] nerochiaro, it does [13:44] http://askubuntu.com/questions/70071/wrong-window-focus-behaviour-in-unity-but-not-gnome-shell [13:45] snadge, sounds compiz-related [13:45] Saviq: so clicking on the shadow exits. then it's what we need to do in unity-2d too [13:45] yeah .. only one of my systems does it [13:45] greyback, -qt's dash corner has much bigger radius than unity's does, do you know which is correct? [13:45] regardless of the shaep [13:45] nerochiaro, yeah we need a mousarea covering the whole thing [13:45] but im trying to figure out which ccsm option would be related to window focus appearance [13:45] didrocks: hi! can you please make sure the old test case in bug 913569 works for you (reveal launcher bar, click on the ubuntu/dash launcher, don't move the mouse outside of the launcher bar area, press alt+f1 → launcher bar should be saturated)? it's not working here. [13:45] snadge, I'm not saying it's an option [13:46] no ubot? well: https://bugs.launchpad.net/unity/+bug/913569 [13:46] looks like its an fglrx problem [13:46] Saviq: that's two separate issues: one is the non-interaction with the stuff below, the other is what part of the dash should trigger the dash to close [13:46] nerochiaro, yes [13:46] Saviq: i'm focusing only on the latter question [13:46] nerochiaro, both minor issues for later [13:46] Saviq: indeed [13:46] i need to enable unity switcher apparently [13:46] htorque: oh interesting, I found another case then [13:46] snadge, tried `unity -reset` to reset your environment? [13:46] htorque: try right-click on bfb -> dash home [13:46] htorque: alt + F1 [13:47] it saturated then, isn't it? [13:47] greyback, dyams, Kaleo, tiagosh: input shaping merged, yay :) [13:48] nerochiaro, can you resubmit the buttons branch so that tarmac won't die? [13:49] Saviq: will do shortly [13:49] hmm.. actually that bug seems to describe the fix [13:49] saviq: very nice [13:50] Saviq: https://code.launchpad.net/~unity-2d-team/unity-2d/unity-2d-shell-dash-panel-buttons/+merge/89436 [13:50] nerochiaro, thanks [13:53] greyback, one thing I noticed - press super, alt+f1 - dash goes away [13:54] bug? feature? [13:55] saviq: Atleast I can say its not reported as bug so far :) [13:55] dyams, checking with unity [13:55] saviq: Ok [13:58] dyams, unity does not do that [13:59] dyams, I say we fix along with the shell work [13:59] dyams: Ok, sure [14:00] saviq:^^ [14:01] dyams, trying to talk to someone smart here? ;) [14:01] saviq: ;) [14:02] dyams, do you confirm that bug, though? [14:03] Saviq: dyams: nice find, but I'd like to check with design what is the right thing to do. [14:04] greyback, sure, another thing is keyboard nav between the dash and the launcher [14:04] * dyams greyback is right [14:04] * greyback only learns now that nick changes have to be done for each server [14:04] greyback, ? [14:05] Saviq: example? [14:05] greyback, go left from search entry [14:05] ok unity --reset doesnt help.. deleting my .compiz .config and .gnome2 dirs didnt help [14:05] Saviq: maybe it's xchat [14:05] greyback, I don't think it is, you can have different nicks per server [14:05] Saviq: rock [14:06] ...and a hard place [14:06] paper scissors [14:06] lizard spock? [14:06] greyback, you can't have two [14:06] paper _or_ scissors [14:06] but hey, you lost with yourself :D [14:06] Saviq: hmm, again I'd like to check. If you're editing text in that box, going left too many times to focus BFB might be confusing too [14:07] or won :P [14:07] greyback, depends how you look on it ;) [14:07] Saviq: yep, hence asking design ;) [14:07] greyback, that was re paper scissors [14:07] greyback, but sure, for tv we have going left from dash content [14:08] that's my answer to everything :) [14:08] lol [14:08] a good one, too [14:08] Saviq: yeah that consistency would be nice alright [14:08] greyback, except I hate it that I can't go left from search entry to launcher ;) [14:08] so yes, that needs design [14:08] JohnLea, we need you here :D [14:09] Saviq; where are you? [14:09] here [14:09] ;-) [14:09] JohnLea, like, on IRC ;) [14:09] JohnLea, two things: open dash, press alt+f1 [14:09] right now the launcher gets focus [14:09] but the dash disappears [14:09] (in unity other weird things happen, but dash does not disappear) [14:10] (that's just the first thing) [14:10] I have a bug ready that I can affect to ayatana-design, if you'd like [14:10] bug #? [14:10] none, right now [14:10] 'cause we weren't sure it's a bug [14:10] I can file and affect you guys [14:11] yup, ping me the bug number when you have filed it [14:11] thx [14:11] JohnLea, bug #919209 [14:11] I said I have it ready [14:13] Saviq: ooh you're kidding! https://bugs.launchpad.net/ayatana-design/+bug/919211 [14:13] Saviq: grrr, you & your efficiency [14:13] greyback, no that's fine [14:13] I only filed the dash, alt+f1 one [14:14] Saviq: yep just saw that, pfew [14:14] greyback, we need one the other way around, too [14:14] Saviq: yep [14:14] except that going right from the launcher gives you the menu [14:14] which is pointless [14:14] so it might be that we shouldn't have any keynav between launcher and dash [14:14] to have it consistent [14:15] just use super / alt+f1 to be sure where you'll end up [14:15] Saviq: yep, hence asking design ;) [14:15] yeyp [14:16] Saviq; the dash should hide, current behaviour in Unity2d is correct (but this needs fixing in unity3d) [14:17] gah... all my windows look like they have focus [14:17] JohnLea, ok, so that kind of solves part of #919211 [14:17] JohnLea: woo :) [14:17] greyback, I'll cook up a test for that :) [14:17] probably an fglrx issue.. sigh [14:18] snadge, try with radeon [14:18] if you can, that is [14:19] Saviq, greyback; take a look at bug #919209 now ;-) [14:19] take that! [14:19] mwahahahaha [14:19] ok [14:21] didrocks: yeah, but again: not when the mouse doesn't leave the launcher bar area [14:22] greyback, we do have a bug there, though - go super, alt+f1, then try to dismiss the launcher [14:22] htorque: yeah, I misread that part, I edited it, sorry :) [14:22] didrocks: thanks, and no problem! :-) [14:23] Saviq: Alt+F1 again, or Escape. You want to click away? [14:23] greyback, mine doesn't hide [14:23] Saviq: hmm, you running shell? [14:23] I have to alt+tab / click away to get out of the launcher [14:23] greyback, nope, oneiric [14:23] wrong [14:23] staging [14:24] greyback, it works if I just go alt+f1, alt+f1 or alt+f1, esc [14:24] but not if I go super, alt+f1, [esc,alt+f1] [14:24] Saviq: yep, confirmed. I can click away though [14:25] Saviq: logging bug [14:25] greyback, do you think that belongs to launcher or dash tests? [14:25] greyback: while testing { Super+Num_Key }, I assume that application launched successfully if the tile has atleast one pip [14:26] Saviq, grayback; also see bug https://bugs.launchpad.net/ayatana-design/+bug/869122 [14:26] ok getting somewhere now.. fglrx has the window focus bug.. but radeon does not [14:26] greyback: i.e to check the application is running status [14:26] Saviq: yep I know the cause. A bad bit of code from me :( [14:27] Saviq: launcher I'd say. Other apps can cause it [14:27] greyback, oks [14:29] JohnLea, apart from point 3. looks fixed in unity-qt [14:29] (see how I've sneaked the non-2d name there? ;>) [14:29] Saviq: oO [14:34] njpatel: comments on https://code.launchpad.net/~vanvugt/unity/fix-742544-alt-trunk/+merge/86779 [14:34] snadge: dri drivers do not have the no-damage-events-on-non-window-pixmaps bug anymore [14:34] dammit.. its an fglrx problem [14:34] smspillaz, jasons' work will fix this [14:35] smspillaz: Im using the latest drivers from amd :| [14:35] we can't just revert without having a proper fix [14:35] njpatel: for 4.0 ? [14:35] njpatel: ok, can you leave comment then ? [14:35] not for 4.0, we don't need this in 4.0 [14:35] the fix seems to make sense, always picking the "primary" monitor. unless I missed something [14:36] fglrx_8.920 aka catalyst 11.12 [14:36] installed with --buildpkg Unity/precise [14:36] greyback, so I'm thinking separate tests - one that verifies that the launcher behaves correctly when the dash is open (not caring about what the dash does) [14:36] and the other just for the dash [14:37] (two for the first case) [14:37] Saviq: yeah that's more comprehensive [14:37] - one for alt+f1, one for esc to dismiss [14:38] smspillaz: i cant find any references to that bug on google [14:38] smspillaz, you can't open the launcher when it's hidden on 4.0 if it's not on the left screen easily [14:38] that's why it picks 0 (it used to always pick primary) [14:38] jason's multimonitor work fixes that [14:39] greyback, hmm we actually don't have a test to just check whether launcher shows after dash has been invoked [14:39] I'll have one for that, too [14:39] Saviq: not yet, lots of things to test :( [14:39] greyback, yup [14:40] does http://blogs.gnome.org/carlosg/2012/01/20/multitouch-is-near/ use any uTouch technology? [14:40] Saviq: shell in a usable condition now? [14:40] greyback, yup [14:40] cnd, ^ [14:40] Saviq: ok, I'll focus more on that for tests in future [14:40] greyback, I'll do the four I mentioned [14:41] njpatel: ok, can you comment on the merge then ? [14:43] mhall119, cnd will be best to answer that, but he's hard asleep now, I'm afraid [14:44] when he wakes up, will it be Friday or Saturday? [14:44] Friday [14:47] mhall119, but from what I can see there won't be any utouch happening behind that [14:47] mhall119, all the work that's gone into X re multitouch while utouch was in the works will, of course [14:48] ok, so some of the work that Canonical did for uTouch is helping enable GTK+ touch [14:48] that was really my question [14:53] smspillaz, i did [14:58] njpatel: yep, saw, thanks [14:59] just so it didn't look dodge when I abstain for no reason XD [15:00] mhall119: cnd works upstream with XInput2 - some of XInput2 is used for the Gtk+ touch stack. you'll have to ask him on the specifics [15:08] greyback, do we want any tests that we know will fail in trunk submitted there? [15:08] greyback, the actual question is whether we want _any_ tests that fail in the suite? or do we keep them somewhere in a branch and only merge with the fix? [15:09] 'cause I can easily imagine (and not only imagine) people writing tests not being those that actually fix the issues [15:18] Saviq: can add tests that are skipped by defining them as "xtest something" instead of "test something" [15:18] so they can be enabled later [15:18] greyback, oh cool [15:18] greyback, is there a way to actually run them, too? [15:18] greyback: Saviq: any idea on how to test that the dash will go fullscreen when the resolution changes to resolution that's too small ? [15:19] nerochiaro, you could test the dash mode and its height / width? [15:19] Saviq: no unfortunately, it's just a little bit of syntactic sugar [15:19] greyback, ok nvm [15:19] Saviq: yes but how do i change the resolution in the test ? [15:19] nerochiaro, xrandr? [15:20] Saviq: to what resolution ? each system is different in what it does allow ;) [15:20] Saviq: and actually on my machine for some reason just right now xrandr isn't working [15:20] also, do we want to fuck around with resolution on people's machiens ? [15:20] nerochiaro, as greyback mentioned earlier - skip the test with a warning [15:20] nerochiaro, we can have them separate [15:20] greyback, what do you think? [15:21] I think fucking about with people's resolution is dodgy [15:21] my suggestion here is to test at whatever resolution we are right now and check the dash in the correct shape [15:21] Best detect in the test what resolution they have, and just return if it's not suitable [15:21] nerochiaro: yeah, I think that's ok [15:22] greyback: or test that we do the right thing at that resolution, knowing that maybe at another resolution we will fail [15:22] nerochiaro, btw, it doesn't, not in my VBOX at least [15:22] Saviq: doesn't what ? [15:22] nerochiaro, doesn't adapt [15:23] Saviq: no it doesn't, i'm working on it now :) [15:23] nerochiaro: exactly. I don't see how to change resolution safely [15:23] at 640x480 the launcher is resized, but the dash doesn't change modes [15:23] nerochiaro, ok [15:23] greyback, nerochiaro: maybe we could have an env var [15:23] that says "RUN_DISRUPTIVE_TESTS" [15:23] greyback: it would be great to be able to run the tests in an instrumented environment where ScreenInfo returns whatever we want it to return [15:23] Saviq: ^ [15:24] nerochiaro, that might make sense [15:24] so we would be able to fake changes in the resolution [15:24] nerochiaro, can't we hack ScreenInfo already? [15:24] nerochiaro, from testability [15:24] nerochiaro, just write to the properties [15:24] Saviq: hmm, it's a singleton class, not part of the object tree [15:24] doubt it can reach it [15:24] nerochiaro, oh :/ [15:25] Saviq: but aren't we running unity-2d-shell with -testability ? [15:25] nerochiaro, yes, we are [15:25] we can check that and use the fake ScreenInfo in that case [15:25] yup, we probably coudl [15:26] *could [15:26] but i'm not sure it's a great idea from another standpoint because it's just adding an extra code path _in_ the app under test [15:26] * nerochiaro got an idea [15:27] * greyback pondering how evil Xephyr would be for this [15:27] greyback, too evil [15:27] why ? [15:28] unless we decide to run everything under xephyr, that i [15:28] s [15:28] Saviq: just for resolution tests, why so evil [15:28] ? [15:28] dunno, I kind of hate xephyr for some reasony [15:28] -y [15:28] might have had issues with it before or something [15:28] nigthmares maybe [15:29] and simply unsure if we really need that [15:29] Saviq: i guess it's the same feeling i have for vbox ;) [15:29] nerochiaro, true, true [15:29] I think it's an option, but not a great one. It means installing more crap on the machine under test [15:29] surely running the tests under xephyr will be closer to the real system than vbox [15:30] nerochiaro, how so? [15:31] nerochiaro, as far as the system knows, vbox is just different hardware [15:31] as far as an application knows, xephyr is just a different X server/client/whatever [15:31] so the difference is just at a different level [15:32] can't we just skip the tests that are disruptive unless there's an env var set? [15:32] and those that we can't run (like when Ugo's xrandr is broken) [15:33] Saviq: sure, if you know xrandr is broken ;) [15:33] Saviq: yep, do-able [15:33] nerochiaro, how does your broken xrandr show it's broken? [15:33] Saviq: in general i agree, in this particular case i dunno even if xrandr works, how do we know which resolution to change to ? [15:33] it probably doesn't display the res or something? [15:33] Saviq: it just say it faield to change the res [15:33] might be i'm using nvidia drivers or something [15:34] nerochiaro, I'm on nvidia too [15:34] nerochiaro, just run xrandr, find one that fits, change [15:34] change back [15:34] +test in between [15:34] nerochiaro: this is the reason why VBox is good idea IMO. If something core is broken on your machine, then test will fail because of it . At least in a VBox instance, it should be a clean install & everything working [15:34] Saviq: "find one that fits" ? [15:34] nerochiaro, it will list you available resolutions [15:34] just find one that's low enough that the change will happen [15:35] if any of that fails, the test failed, but will tell you why [15:35] it [15:35] shit will happen, yes - tests will sometimes fail due to [15:35] greyback: Saviq: what i'm really thinking is that we should take a step back and define who are the tests meant for. if they are meant for any random user to run,it's one thing, if they are meant to be run on jenkins for verification before release,it's another [15:35] nerochiaro, they're primarily based for automated testing [15:36] but if we can make them work for user x then cool [15:36] greyback, unless I'm mistaken with that? [15:36] that's my priority list, yeah [15:37] nerochiaro: I do want test to run on your machine, without VBox. But there's only so much time I can spend thinking of every possiblt thing your machine might have different to work around [15:37] greyback: i agree. i'm not necessarily against vbox or xephyr [15:37] bbiab [15:38] nerochiaro: in fact, I prefer when they can run inside some virtualised thing, so I can do other stuff as they run [15:38] VBox has a headless mode too, which I want to play with in future [15:38] greyback, yeah I do the same, they just run away, I'm doing other things, when I get back I have a result ready [15:39] Saviq: yep, perfect, that's what I'm trying to get going [15:39] Amazing all the little things that can get in the way [15:40] yup [15:40] Saviq: nerochiaro: But I have to say, this week with you all using Testability, your comments about problems, stuff that's missing, etc, have been bloody useful [15:40] Great to get proper feeedback [15:40] feeeedback [15:41] greyback, for me it's awesome to see that we can actually test stuff [15:41] before your Testability talk I was scared as hell [15:41] easy peasy now [15:42] Saviq: it is nice, with a few painful exceptions. XDotool stuff needs a lotta loving (in progress) [15:42] sure, but we can already do a lot [15:42] and we're really pushing what the tool can do. It wasn't meant for testing a desktop environment [15:42] Yep, I'm pretty pleased with it [15:42] I love it how we can hack a test up, make it fail (well, error at this point :P) and the fix it, pass the test [15:42] done! [15:42] no f*cking about [15:43] Yep. It's a good way to work [15:46] that's what Ugo failed to do with the shaping work - he built tests on top of a working feature [15:46] it was difficult to actually see whether the tests fail when the feature was broken (or not there) [15:48] greyback, you sir need to settle on a nick, too :P [15:48] Saviq: eh? [15:48] you're greyback here, gerboland on LP [15:48] greyback, you got a MR waiting, sorry [15:48] Yeah, always meaning to get that changed [15:49] Saviq: np, looking [15:50] greyback, we (by we I mean you with a little help from your friends) need to define a process for working with tests [15:50] like whether we want separate branches for tests [15:50] Saviq: bug #919209 is a duplicate [15:50] submitted with xtest to disable them initially [15:51] didrocks, couldn't find one, sorry [15:52] bug #885304 [15:52] Saviq: I see. My concern is someone writing a disabled test will then forget to do the fix and enable the test. [15:52] So I want fix+test [15:52] greyback, well, the bug will be open all the time [15:53] so as long as someone is assigned to the bug... [15:53] greyback, fix+test has the added issue of not being able to test the test [15:53] or making it difficukt [15:53] s/k/l/ [15:53] Saviq: sure [15:53] exactly our issue with Ugo today [15:53] Yeah, that I saw [15:53] I spent several hours to verify that the tests actually fail without the fix [15:54] that should be easy as pie [15:54] True [15:54] I'm not requiring for people to go TDD [15:55] but a separate test / fix branch might be useful [15:55] and easy to work with, really [15:56] since they're separate enough, merging the tests into your fix branch to test would be very easy [15:56] problems arise when you need to fix your tests [15:56] but they're even bigger when you only have one branch where the history gets intertwined [15:57] Let me recap. You have a bug to fix. You first write a test for the fix in one branch [15:57] In second branch you do the fix. [15:57] You try to have both merged into trunk simultaneously [15:59] Test-only branch will cause Jenkins to lockout, unless test is disabled. [15:59] that's TDD, I'm not set on whether you should write the test first or the fix [15:59] Yeah, I'm pondering out loud ;) [15:59] that's irrelevant, really [16:00] but there are two approaches here [16:00] 1) you merge xtests and not tests [16:00] then you merge the fix, actually enabling the tests [16:00] Saviq: greyback: re [16:01] 2) merge the fix and the test before merging into trunk [16:01] 2) has issues with MRs [16:01] 1) does not [16:01] Yeah option 1 is what I'm thinking is good [16:02] two separate MRs - one for the test, the other for the fix - only thing we need to remember is that the test one needs to be merged first [16:02] at this point i'm actually starting to think that TDD might be a better option (not mandatory, just saying for me for some cases) [16:02] nerochiaro, !!!!! [16:02] D: [16:02] :D [16:02] ;) [16:02] Saviq: yep [16:02] nerochiaro: omg! [16:02] :) [16:02] yeah, shocking i know [16:03] I think he's been bitten by me being an a$$ today [16:03] and even more by me being an a$$ rightly so [16:03] by you being very rigorous, sir. that's differnet [16:03] nerochiaro: you have opinion on what we're discussing above. A test-driven workflow process [16:03] ? [16:03] gord, question about the views that displays stuff from lenses [16:03] gord, it seems like they just append the results, no matter the position in the model [16:04] gord, do they support inserting at proper position? [16:04] Saviq: nerochiaro whatever you two get up to in private has no business on this channel ;) [16:04] gord, like i have 5 results in the view already and then i want to insert something at pos 0 [16:04] greyback: i'm not much in favor of a tests branch and a feature branch, MR'd separately. adds lots of overhead imho [16:04] greyback, right, we should take that to #unity-qt [16:05] nerochiaro, yes, the overhead is definitely an issue [16:06] nerochiaro, but how else do you think we can make sure it's easy to verify the test itself? [16:06] i've gotta think about it a bit [16:07] Saviq: LP can have one branch depend on another. Might help prevent fix being merged before test [16:07] mhr3, are you talking about per category views? [16:07] greyback, yeah, but tarmac pukes with that [16:07] Saviq: ahh, I didn't know [16:07] if bzr made it easier to reorder the history i would suggest that when you commit you move them as the first commit of your MR [16:07] gord, yea [16:07] sure, if we were using git I would just rebase my fix on the test [16:08] but we're not :P [16:08] yeah, a man can dream right ;) [16:08] gord, oh right the model is then split to categories... so no straight mapping from the model positions... [16:09] mhr3, yeah, we go in order iirc, but the only signals the views get are row_added/row_removed so any changes are appended/removed [16:09] Saviq: if the commits about the tests are not mixed with the commits about the feature one can just revert the block of commits about the feature when reviewing,no ? [16:10] Saviq: what you did in my case [16:10] gord, ok then, i'll pre-sort it in the lens, that will surely be easier... [16:10] Saviq: so maybe we can either do the 2 branches thing or the grouping thing, and each dev choose which one they prefer [16:12] nerochiaro, meeting, will get back to you [16:12] nerochiaro: sorry, grouping thing? You mean merge test into fix before MR? [16:13] greyback: i mean that all commits that are not tests should not be mixed with the commits (or hopefully, single commit) that are tests [16:13] mhr3, wouldn't that hurt lenses with scopes with different latencies? As the result would have to wait for the slowest scope to pass results before being displayed? [16:13] greyback: so one can do "bzr merge -r X..Y ." where X and Y are the non-test commits, to revert the feature but keep the tests [16:14] greyback: i think it works well for small bugfixes [16:14] greyback: larger bugfixes might want the split branches [16:14] nerochiaro: true. I think it's valuable to keep them separate [16:14] nerochiaro: just trying to figure out best way so there's not too much overhead [16:15] greyback: yeah, and i think the overhead of splitting is ok with a large branch (like -shaped) but not ok with a small fix [16:15] say if a fix is bad, but the test is good, having them separate in trunk is great [16:15] greyback: so for the small fix we can just have one single branch with feature commits and test commits cleanly separated [16:16] davidcalle, well, yes, in some way [16:16] greyback: well, if the commits are not mixed, it's easy to merge just a part of them,as if they were from 2 separate branches [16:16] nerochiaro: I dunno, then we've got 2 policies which adds confusion [16:16] nerochiaro: sure [16:16] greyback: right. i just feel neither works well for all commits [16:16] greyback: for all MRs, sorry [16:17] davidcalle, i'll be actually sorting the results within one scope, so no big deal [16:17] nerochiaro: Yeah, I'm torn myself. I'd rather not the pain for every little bugfix [16:17] nerochiaro: but I don't like there being 2 policies either [16:17] davidcalle, but now that i think about it, the merge strategy might actually not work at all [16:18] greyback: frankly i wouldn't want to make any of these policies mandatory either [16:18] and kamstrup's gone already :/ [16:18] mhr3, why? [16:18] nerochiaro: it wouldn't be a policy if it's not enforced :) [16:19] nerochiaro: guidelines are a nicer term [16:19] davidcalle, cause you are sorting the model, but the dash won't care about the order of the model [16:19] greyback: right [16:19] it's appending latest results at the end [16:19] mhr3, indeed... [16:21] mhr3, can the Dash be changed in time for Precise? [16:21] that's a topic for next week's discussions :) [16:22] i suppose 2d has the same issue? [16:22] greyback, ^^? [16:22] greyback: bug #919226 is another duplicate you just opened of bug #919209 ? (which I signaled as a duplicate of bug #885304 ? [16:23] mhr3: yes I believe so [16:25] didrocks: 919226 != 919209 - it's a different bug [16:26] urgh, yeah [16:26] 919226 is the duplicate of bug #885304 [16:27] not 919209 [16:27] isn't it? [16:27] didrocks: in the code, it's actually a different issue [16:27] greyback, one can totally tell that you studied maths ;) [16:27] (919226 != 919209) [16:28] mhr3: 8 years of university baby! [16:28] greyback: can you please dup 919226 to 885304? [16:28] greyback: I'll undup 919209 and say "sorry" to Saviq :) [16:29] or are those you mean, "it's different bugs?" [16:29] didrocks: yep, 919226 != 885304 [16:29] didrocks: it's a problem determining which window to give focus back to [16:29] hum, weird ;) [16:29] ah ok [16:29] guys, straighten your maths up and let me know ;) [16:30] greyback, i'm just waiting while you get to a dupe and type 9098 == 9172 [16:30] didrocks: tho there should be a different bug it is a dup of, am searching [16:30] anyway 919209 is a different one as well, undupping [16:30] mhr3: I... just ... can't gaah [16:31] nerochiaro, problem with "grouping" is that after review you will fix the test, then you will fix the feature... and then you're not grouped anymore [16:31] greyback, so it'll be "~=" ? [16:31] greyback: do you mind if i rename tests/places to tests/dash ? we can finally get rid of that old name now [16:31] that's never gonna work [16:31] Saviq: ah, good point. keep thinking in terms of git [16:31] Saviq: feh [16:31] nerochiaro: please do! [16:31] awww [16:32] Saviq: but i really hate forcing separate branches even for small bugfixes [16:32] grrr [16:33] and bzr support for in-place branches ain't that great either so i have to keep them in two dirs, and rebuild all two times [16:33] bleh [16:33] nerochiaro, well, colo-branches work ~fine for me [16:34] it's the ~ that bites me at the wrong moments [16:34] unless I move the whole repo around, when it dies completely [16:34] or rather, bit me [16:34] well, ok, the only real issue I've had is moving the repo around [16:34] it's got absolute paths saved in there [16:34] Saviq: I am thinking for a bug fix, the overhead of 2 branches is too much. As long as test and fix are separate commits, it should be ok [16:34] which is :O [16:35] i dunno, it might have been me being stupid, but sometimes switching between branches confused it immensely [16:35] Saviq: problem is defining "just a bugfix" :) [16:35] i'll give it another go i guess [16:35] or just suck up the overhead [16:35] greyback, ok so maybe the guideline would be: every MR needs to have a clear way to get rid of the fix to be able to test the test [16:36] how you achieve that is the dev's responsibility [16:36] Saviq: yeah, that's my thinking [16:36] nerochiaro, yeah I can't understand why once `bzr switch trunk` works and once it does not [16:36] and then list out the recommended approaches [16:37] If you write something that needs more than 1-2 tests, then the tests should be separate [16:37] `bzr rebase` might be your friend there, too [16:38] aanyway, EOD for me guys [16:38] we'll talk on monday, have a great weekend [16:38] Saviq: thanks for good discussion [16:38] Have a good one! [16:39] have a good weekend [16:39] Saviq: one last thing, if you have a sec [16:39] nerochiaro, yup [16:40] do you know if when i overwrite a branch with --overwrite and you want to pick up what i did, but preserve the build in your local tree, how to do that ? [16:40] say you branched earlier, then i overwrote the branch, then you want to pick up my overwritten branch [16:41] but in the same dir as the previous branch [16:43] nerochiaro, `bzr pull --overwrite` should do, no? [16:43] Saviq: i didn't know it existed :) [16:43] :) [16:44] Saviq: because with push --overwrite is permissible, then we can relatively easily reorder the history git-style with bzr uncommit and a few helper scripts [16:44] s/with/if [16:45] and ask the reviewer to pull --overwrite again [16:45] nerochiaro, yeah sure, but I'm too afraid to do that in bzr [16:45] Saviq: i've been using uncommit a bunch and it's really sae [16:45] safe [16:45] but YMMV [16:45] we basically need a `bzr rebase -i` [16:47] Saviq: ok i'll think about it [16:47] Saviq: have a good weekend [16:47] cheers [16:53] greyback: that's funny, i'm fixing something where the test is actually longer than the fix [16:56] greyback: also the test depends on something that's in gsettings, do i hardcode these values in the test file or read them from gsettings [16:56] ? [17:13] nerochiaro: oops sorry, reading [17:13] greyback: np [17:13] yeah I can see lots of tests longer than the fixes ;) [17:13] Especially with my demand for a nice long comment at the top [17:14] Oh gsettings grr, another thingy on my list to figure out a proper test system for dconf stuff [17:15] nerochiaro: reading from gsettings risky? [17:16] greyback: didn't say risky, just wondering what i should do. in the end the test will mostly mirror the fix, since we can't change the resolution on the user's machien [17:18] greyback: i mean, i will read the min resolution, check what the user has, and test what the dash does. the dash will read the min resolution, check what the user has, and do something. i mean, it's pretty much the same ;9 [17:18] ;) [17:28] nerochiaro: lol ok so. Go for gsettings then [19:32] does someone use unity-2d? or is maybe a developer? [19:33] I want to know why unity-2d does not support the shortcuts for the lenses [19:33] like Super+a or Super+f [19:41] i use it [19:41] i dont know wht [19:41] why [19:42] but you can mail them to why they dont develop it [19:43] and I want to replace Metacity with Mutter [19:43] why ? [19:44] With Metacity I have problems with tearing [19:44] ok [19:44] did u try unity 2d 5.2 ? [19:44] its new [19:45] just 1 week release [19:45] It looks good so far. The only downside is, that the window decorations is not hidden at maximized windows [19:46] yes, I have Precise [19:46] good [19:46] enjoy it [19:46] and also u can report bugs in lunchpad [19:46] I can, and I do [19:47] good === kancerman_ is now known as kancerman [19:55] nava_, why do you use unity-2d? [21:11] Does unity 2d allow you to re-order icons? [21:12] I cannot seem to be able to drag the icon out. (using 2d for performance reasons) [21:23] mhall119, did you say that i should be able to add/remove filter options now? [21:23] kenvandine: on what? [21:24] a scope [21:24] whoops [21:24] completion fail [21:24] mhr3, ^^ [21:24] sorry mhall119 :) [21:24] ah, ok [21:24] no problem :) [21:25] kenvandine, that's right [21:25] mhr3, well it doesn't blow up if i do it [21:25] but the lens doesn't see it [21:26] nooo, dont tell me it doesn't work [21:26] mhr3, i wish i wasn't telling you that :) [21:26] kenvandine, could you just restart unity while your lens is up? [21:26] see what happens [21:26] i tried, restarting unity makes it show up [21:27] cause i have a feeling that at some point unity no longer cares about changes to the model [21:27] but that'd be a bug in unity [21:28] kenvandine, anyway now that it's restart does it react to changes in the filters? [21:28] i didn't try that, let me do that [21:29] actually, maybe it isn't unity's fault [21:29] i don't get the filter.changed signal [21:30] i connect to the signal before i add options, and get the signal when it is first populated [21:30] but later if i remove or add an option i don't get the signal [21:31] var filter = scope.get_filter ("account_id") as CheckOptionFilter; [21:31] filter.add_option (account.id, account.service + "/" + account.username); [21:31] is what i am doing [21:31] * kenvandine restarts unity [21:38] ok... that doesn't really help... i have an unrelated crasher when interacting with those filters :) [21:40] but am am not getting filter.changed signals after i initially populate them [23:26] hey guys [23:49] gday