/srv/irclogs.ubuntu.com/2013/01/30/#ubuntu-unity.txt

bschaeferMCR1, hmm strange, its alt+f5 for me one time, now its Ctrl+Super+Down, it seems like its random00:13
* bschaefer goes to check the default value in the generated code00:13
bschaeferhmm it seems to want to set it as       <default>&lt;Alt&gt;F5</default>00:14
* bschaefer wonders if its using the wrong schemas00:15
bschaeferMCR1, wait, i was looking at the wrong branch....mines getting generated to00:16
bschaefer      <default>&lt;Control&gt;&lt;Super&gt;Down</default>00:16
bschaeferwhich it shouldn't be...00:16
MCR1hmmm00:16
bschaeferMCR1, hmm im seeing it set to       <default>&lt;Control&gt;&lt;Super&gt;Down</default>00:17
bschaeferin the metadata/core.xml.in00:17
MCR1then you have to pull again probably00:17
* bschaefer 's fire fox crashed00:17
bschaefero nice, a new rev in the core.xml00:18
MCR1:)00:19
bschaeferMCR1, also add {}00:22
bschaeferto your if/else00:22
bschaeferits confusing to read when the cond ends and the statement begins00:23
MCR1yep, you are right00:23
bschaeferMCR1, usually always include a {} if you have an if/else00:23
bschaeferMCR1, usually always include a {} if you have an if/else00:25
bschaeferopps00:25
MCR1ok, ack00:25
* bschaefer hit up on the wrong window00:25
bschaefer:)00:25
MCR1Done - how is the functionality ?00:28
MCR1does it finally do what it is supposed to ?00:28
MCR1;)00:28
* bschaefer is having a window focus problem00:32
bschaeferMCR1, sweet, works00:33
MCR1phew00:33
MCR1togh job for one shortcut ;)00:33
MCR1*tough00:33
bschaefernow ill approve that and poke Daniel about the patch...though he wont have anytime besides to point me in the right direction00:33
bschaeferyeaah00:33
MCR1cool - thanx - I count on you00:34
* bschaefer has a bunch of stuff on the stack00:34
* bschaefer goes to finish some other work00:34
MCR1I'll do it if necessary...00:34
jibelhey, there's no auto upload of unity to the release the morning08:51
jibelthere are failures in autopilot for unity and indicators08:51
jibeland dbus-test-runner FTBFS (fatal warning, g_type_init deprecated)08:51
jibelsil2100, ^08:51
sil2100jibel: hi08:54
sil2100jibel: yes, we had a talk with fginther and mterry08:55
sil2100jibel: this week there was again some refactoring in the switcher and it broke all the switcher tests ;/08:55
sil2100jibel: it was pretty late yesterday so I asked them to give me time today in the morning for fixing it08:55
sil2100jibel: so I'm working on it now08:55
jibelsil2100, k, thanks08:57
smspillazbregma: the recent refactoring on SwitcherController doesn't make a whole lot of sense to me10:13
smspillazbregma: it seems like we're putting bits of buisness logic inside the interface that we provide to clients. Was that intended or am I reading it wrong ?10:14
luvrye: yo, morning - last night I tried to forward port the window list patch: http://killem.al/unity-612-launchericon-windowlist.patch10:15
sil2100smspillaz: hi10:16
luvthough, I didnt' get it to compile because i used a week old (or so) branch and it ended up with g++ segfault even without my patch, but i'm kinda confident it will work :-)10:16
sil2100smspillaz: I'm in the middle of fixing up switcher tests again, since bregma'a most recent changes removed the impl completely from introspection, breaking all switcher tests - do you intend on changing the switchercontroller again? Since I don't want to do the same work 3 times again ;p10:17
smspillazI don't know whats happening because I've seen a complete lack of communication from the PS team10:17
smspillazI suspect we should have a chat with bregma to figure what that was all about10:18
smspillazI appreciate the desire to make some methods nonvirtual, but doing that makes testing with other components later down the road a total PITA10:18
* sil2100 liked the SwitcherController when it was simple as in the beginning :(10:19
sil2100luv: thanks! I'll also try looking at it and testing it later10:19
smspillazsil2100: it was completely untestable10:19
smspillazthe original SwitcherController is why we have the sed script BS in unity10:19
sil2100smspillaz: would be good if all the unity devs could discuss the whole concept and decide on a final version, so that we can rewrite autopilot to accomodate the changes once and for all10:20
sil2100Since currently I'm most worried with the autopilot switcher tests being broken again, which is blocking daily releases10:21
MCR1JohnLea: Hi :) Proud to announce to you: bug 966099 is in the process to be fixed :)10:22
ubot5bug 966099 in Unity "Keyboard shortcut overlay says Ctrl+Super+Down "minimises" the current window, but it doesn't" [High,In progress] https://launchpad.net/bugs/96609910:22
MCR1JohnLea: (properly)10:22
seb128sil2100, fginther: hey, jibel said you are looking a indicators' stack test failures ... what is broken exactly? I would like to see the current libdbusmenu autoland ;-)10:23
JohnLeaMCR1; awesome! It will be super to get it fixed at long, long last ;-)10:23
MCR1JohnLea: Next one is bug 89074710:23
ubot5bug 890747 in gnome-settings-daemon (Ubuntu) "Keyboard shortcut - Ctrl Alt Del doesn't do what most people typing it would expect" [High,Triaged] https://launchpad.net/bugs/89074710:23
JohnLeaMCR1; cool ;-)10:24
luvsil2100: nice, thanks10:24
MCR1JohnLea: But for this we gotta remove a standard gnome-shortcut10:24
sil2100seb128: hi! Ok, let me check that in a moment, since I was looking into unity-release autopilot tests right now, but I'll check it and get back to you in a moment10:24
MCR1JohnLea: As currently gnome will propose to log you out10:24
JohnLeaMCR1; in this case that is ok10:24
MCR1JohnLea: so no change ?10:24
JohnLeaMCR1; yes to change, in this case it is ok to change the gnome shortcut so that Ctrl Alt Del brings up the Processes tab of System Monitor10:25
MCR1JohnLea: Ok, acknowledged. I yet have to find where this is hardcoded to fix it though... Last question:10:26
MCR1JohnLea: What about the additional shortcut to toggle a selected window between fullscreen/un-fullscreen10:27
MCR1JohnLea: It is Alt+Enter on most windows versions10:27
MCR1JohnLea: This is Fullscreen, not maximize10:27
MCR1JohnLea: Very useful - for media players for example, but it will also fullsceen/unfullscreen any other window10:28
luvumm, making methods nonvirtual ... premature optimizatio?10:28
JohnLeaMCR1: At the moment it is usually F11, see VLC, Chrome, Firefox, etc...10:29
MCR1JohnLea: Often it is F11, in apps like chromium -> then it will do the same like standard10:29
MCR1JohnLea: Yep10:29
JohnLeaMCR1; this is also a windows shortcut as well10:29
MCR1JohnLea: but media players won't work with F11...10:29
MCR1JohnLea: Well, it is not that important, but we do currently have NO! shortcut for Fullscreening/Unfullscreening in Ubuntu10:30
MCR1JohnLea: and it would be nice to have, I would say...10:30
JohnLeaMCR1; We can add Alt+Enter as long as it does not affect the existing F11 behaviour (which I suppose it shouldn't)10:31
MCR1JohnLea: It won't. It is tested. I got it running here since ages.10:31
MCR1JohnLea: The only problem we have, we already have in trunk10:31
JohnLeaMCR1; cool, can you ping me a bug # with *just* that change for me to ok?10:32
MCR1JohnLea: It is the possibility to invoke the HUD while fullscreened10:32
MCR1JohnLea: To reproduce: Hit F11 in Chromium and then Alt to invoke the HUD10:32
MCR1JohnLea: It will come up (the hud) but will be invisible and block F1110:33
MCR1JohnLea: So this should be fixed to avoid confusion10:33
MCR1JohnLea: I propose to forbid the HUD for fullscreened apps10:33
JohnLeaMCR1; there are a bunch of other bugs with full screen mode at the moment, for example you can't reveal the Launcher by pushing against the left edge in full screen mode10:33
MCR1JohnLea: but this is a design question10:33
MCR1JohnLea: Also design question10:34
MCR1JohnLea: What should be possible in Fullscreen mode ?10:34
JohnLeaIt should be possible to:10:35
JohnLea- reveal a auto-hidden launcher10:35
JohnLea- use Alt-Tab10:35
JohnLea- open the Dash10:35
JohnLea- open the Hud10:35
JohnLeaetc...10:36
JohnLeabasically all of these things are broken now10:36
MCR1urgh, - okay -> gotta think about that one first...10:36
MCR1could be complicated to solve, because of various reasons10:36
JohnLeayup, full screen mode is really broken atm10:37
MCR1JohnLea: I will prepare the bug for you, regarding Alt+Enter Solo10:37
JohnLeacool, thx!10:37
MCR1JohnLea: might take a while, do not hold your breath10:37
MCR1JohnLea: Thanx a lot 4 your time.10:39
=== larsu is now known as Guest15091
MCR1seb128: Hi :) I bet you know which package I have to manipulate to get rid of the hardcoded Ctrl+Alt+Del gnome shortcut to log out the user... Could you tell me ?10:51
sil2100hah, I actually see how to easily fix this now10:51
* sil2100 talks about the switcher tests to himself10:52
sil2100Don't mind me10:52
MCR1hehe10:52
seb128MCR1, is it hardcoded? I'm unsure, try gnome-settings-daemon or gnome-session...10:52
MCR1seb128: I did not find it in the settings-daemon, I'll try gnome-session... thanks10:53
MCR1sil2100: Btw, do not hate me - but my implementation needs a packaging change :-[10:54
MCR1sil2100: Forget it, just the fullscreen stuff needs it...10:54
seb128MCR1, gnome-settings-daemon/plugins/media-keys/gsd-media-keys-manager.c10:55
seb128        case LOGOUT_KEY:10:56
seb128                do_logout_action (manager);10:56
seb128                break;10:56
MCR1seb128: oh great - that is a big help - I must have missed it :)10:56
seb128MCR1, isn't what you are looking for?10:56
sil2100;)10:56
MCR1seb128: well, not exactly -> I am searching for the definition of LOGOUT_KEY10:56
seb128it's in plugins/media-keys/shortcuts-list.h10:57
seb128        { LOGOUT_KEY, "logout", NULL },10:57
MCR1seb128: Top. Thanks. Should be Ctrl+Alt+Del somewhere...10:57
seb128MCR1, gsettings: org.gnome.settings-daemon.plugins.media-keys logout '<Control><Alt>Delete'10:58
MCR1seb128: Last question: Where exactly is this defined, what source file writes this gsettings key ?10:59
seb128MCR1, the schemas/default value is in g-s-d: data/org.gnome.settings-daemon.plugins.media-keys.gschema.xml.in.in11:00
seb128MCR1, gnome-control-center allows you to rebind it in its keybinding ui afaik, otherwise not sure if anything writes it11:01
MCR1seb128: I am sorry for nerving, P E R F E C T ! ! !11:01
=== mmrazik is now known as mmrazik|lunch
MCR1seb128: Without you this would have taken me hours to find out...11:01
seb128MCR1, no worry, glad that I was able to help you, happy hacking ;-)11:01
MCR1thx11:01
sil2100Could anyone test and review https://code.launchpad.net/~sil2100/unity/autopilot_fix_switcher_again/+merge/145578 with latest trunk?11:12
sil2100If it also fixes switcher tests for them11:12
=== _salem is now known as salem_
sil2100seb128: looking into the failing tests, interesting11:17
seb128sil2100, hum, for some definition of interesting right? ;-)11:17
MCR1seb128: I now know how to perfectly fix bug 890747 without messing with gnome itself. I will upgrade the Compiz Gnome Compatibility plug-in to also deal with other gnome shortcuts (currently it just deals with showMainMenu, showRunDialog, CommandScreenshot, CommandWindowScreenshot and CommandTerminal)11:24
ubot5bug 890747 in gnome-settings-daemon (Ubuntu) "Keyboard shortcut - Ctrl Alt Del doesn't do what most people typing it would expect" [High,Triaged] https://launchpad.net/bugs/89074711:24
seb128MCR1, great!11:25
=== mmrazik|lunch is now known as mmrazik
MCR150% of credit goes to you, seb128 ;)11:25
MCR1My fix will also have the positive side-effect that all will be perfectly configurable CCSM style, so you will be easily able to modify everything to your needs if it won't suit you...11:27
MCR1Unfortunately (once again) I will have to deal with Ubuntu Compiz patches :P11:27
MCR1smspillaz: New plan by MCR: Upgrading the Gnome Compatibility Plugin ! :)11:28
ryeluv: are the changes only in launcher/ApplicationLauncherIcon.{h,cpp} ?11:33
* rye is building11:33
luvyup11:35
ryeluv: can't wait to see it built then :)11:36
luvwell, fingers crossed :-)11:39
=== mmrazik is now known as mmrazik|afk
smspillazMCR1: cool, I guess12:06
MCR1smspillaz: I guess (hope) too, that it will be cool ;)12:08
MCR1smspillaz: it will enhance compatibility with gnome, while upgrading customization possibility and helping us fix the Ctrl+Alt+Del issue...12:09
MCR1smspillaz: but I will keep the first upgrade simple and just add the logout command...12:10
=== mmrazik|afk is now known as mmrazik
ryeluv: ok, built and it added the menu! However, in case there are more than one window with the same title it would show one entry and focus the first one. Should there be separators before window list? Where should the list be located - under quit or somewhere in between12:21
ryeluv: but this is maaaajor usability improvement for me :)12:22
luvthanks .. right regarding the seperator:  http://killem.al/windowlist.ogv <- this is how it should work12:22
luvregarding the problem with one entry for more windows with same title - right, it handles it in ubuntu 12.04, so I will need to fix the patch against raring12:24
luvit really sucks, I can't get VBox guest additions running in raring so it's a nightmare using it12:24
luvthough, I'm happy it builds fine and mostly works! Could you make a screenshot of how the seperator is placed on raring - it'd be interested to see that.12:25
ryeluv: aha, in raring the window list is below all the other entries12:25
luvwell, if i had a fully working (virtual) box running raring, it'd be a matter of an hour getting all these details sorted out :-/12:26
ryeluv: at what point does vbox fail?12:27
luvwell there are two problems 1) can't compile unity because of g++ segfault (not vbox related), 2) the vbox - well - i can't install guest additions (tried both, the script coming with VBox and the packages in raring repo) - I am not sure why it doesn't install properly - I should investigate that.12:28
ryeluv: http://ubuntuone.com/5xzEdYJLFLcjntYP72siJ712:29
ryeluv: are you compiling unity inside vbox and get the segfault?12:29
luvyes, but somehow i don't think it is vbox related12:30
* rye installs vbox, why not12:30
luvit's not that hard to get g++ segfault with c+11 even on bare metal12:30
luvoh, i dont think you have to investigate there ...  i believe the segfault was caused by me not using latest trunk12:30
luvi used a week trunk out of convenience (I just ran apt-src install in raring)12:31
ryeluv: well, i built stuff for quantal/raring/precise in vm under kvm and so far i had no failures of the compiler, so it is a bit strange12:31
ryeluv: well, i have a unity-team/staging ppa where I get trunk versions packaged and i just do dpkg-buildpackage to get the debs12:32
ryeluv: are you using amd64 or i386 image?12:33
luvit'd be interesting if you could reproduce that segfault indeed ... it's amd6412:33
luvcould i just add unity-team/staging ppa to my raring install and use apt-src install and get latest trunk unity?12:34
* rye zsync the fresh iso12:34
ryeluv: yup, it will be re-built on every commit in compiz/unity/nux12:34
luvvery good12:35
luvyeah, here is an example g++ segfault12:44
luvhttp://pastebin.blesmrt.net/2881/12:44
luvg++ -std=gnu++0x c11segfault.cpp -o c11segfault12:44
luvsegfaults on g++ in ubuntu 12.04 atleast12:44
ryeluv: ok, that code does segfault on 12.04 but works on raring12:53
luvyeah, just tested w/ colleague and it is indeed fixed in gcc-4.712:59
sil2100https://code.launchpad.net/~sil2100/unity/autopilot_fix_switcher_again/+merge/145578 <- can anyone review and approve? Since we need this reviewed and in to fix the daily builds of unity13:08
=== MacSlow is now known as MacSlow|lunch
sil2100mterry: ^13:14
sil2100fginther: ^13:14
bregmasil2100, just approved it for you13:14
sil2100bregma: thanks :)13:15
bregmaleast I could do, given I caused all the trouble13:15
=== MacSlow|lunch is now known as MacSlow
=== dandrader is now known as dandrader|afk
=== dandrader|afk is now known as dandrader
ryeaha, vbox guest additions cannot be compiled now14:05
luvoh, good to get that confirmed, i will try to install raring on a usb drive when i get home14:08
ryevboxvideo module cannot be loaded because it cannot be built14:15
SquarismWHYY has ubuntu-unity per default occupied the majoryty of all CTRL/ALT/SHIFT + common keys. Do unity developers understand we use their window manager to run "application" that usually rely on keyboard shortcuts?!14:33
fgintherseb128, I'm looking at the indicators test failure, looks like a setup issue14:37
seb128fginther, thanks for looking at it14:38
fginthersil2100, do I need to do anything special to rerun the ps-indicators-autopilot-release-testing job?14:43
sil2100fginther: huh14:50
sil2100fginther: not sure, I usually just re-ran it through jenkins14:50
sil2100But hm, I did so today and it resulted in a failure ;/14:50
fginthersil2100, that's all I wanted to know, just in case I should start some other job14:51
fginthersil2100, the failure may be due to a recent utah update, that's why I'm rerunning :-)14:51
sil2100;( ;) Thanks!14:52
=== dandrader is now known as dandrader|lunch
SquarismStop hogging all damn hotkeys.. most IDE's are unusable on unity with factory settings15:33
Squarismi mean please... in particular stop using combinations of ctrl+alt+shift - theres a win/mac key that can be used for OS specific things15:34
ryeSquarism: what ctrl+alt+shift keys are grabbed?15:35
SquarismCTRL+ALT arrow keys for example15:36
ryeSquarism: well, that's been used to switch desktops for ages (as far as I remember, in gnome since 2006), ctrl+alt+shift+arrows - to move window to another workspace15:37
sil2100I remember ctrl+alt+arrows in Ubuntu since a long time, never had any problems with it myself, I always thought metacity used this keycombination by default as well15:37
ryeSquarism: in raring by default there will be only one workspace15:37
SquarismWell, that could be - but linux/ubuntu desktop was a "enthusiast" OS in those days15:38
SquarismTo gain market i would focus on catering to people who can develop the OS. Like i use Intellij - most popular ide for worlds most popular language15:39
Squarismand its hell15:39
Squarism...ie cater to developers of all sorts15:39
sil2100It's hard to have everyone satisfied, sadly15:40
ryeSquarism: well, what actions are triggered by ctrl+alt+shift+up/down/left/right (I tried to search but can't really find anything)15:41
ryeaha15:41
ryeSquarism: forward/back15:41
Squarismi really wanna use linux but when i need to reassign all damn keys its problematic... biggest problem though is that some "Super" keys arent modifiable (wo CCSM)... like CTRL+Super + Left/Right15:41
SquarismCTRL+ALT left/right is back/forward in intellij15:42
Squarismif i could reassign window swithcing to ctrl + super + arrows it would be dandy.. but that is non modifiable15:42
Squarism"CTRL+ALT left/right is back/forward in intellij" -> rye15:43
fginthersil2100, https://code.launchpad.net/~sil2100/unity/autopilot_fix_switcher_again/+merge/145578 is merged15:43
popeySquarism: reduce the number of desktops to 1 and then those key combinations will do nothing in Unity15:44
Squarismalso ALT+F7 and CTRL+ALT+F7 seems occupied by unity15:44
ryeSquarism: System Settings / Keyboard / Navigation has the Ctrl+Alt shortcuts15:44
popeyalso, when you say "most IDEs" you mean, one specific IDE?15:45
Squarismrye, i know.. but i cant ressagin them to Super combinations as many of them are unassignable15:45
Squarismwell i i know eclipse is shortcut heavy to.15:45
sil2100fginther: yay! Thanks!15:47
ryeerm, ctrl+super+down does not appear to work for restoring window15:47
SquarismI like unity as a whole.. and dont want anything else then flee from windows... but with unecessary hurdles like these i get a bit irritated15:47
Squarismrye: and ctrl+super + left/right arent modifieable15:48
ryeSquarism: that is a good point15:48
MCR1Squarism: There are a few key-combinations left, which are not configurable/hardcoded. We are working on fixing those as well.15:48
SquarismMCR1, thanx - id really appreciate that!15:49
ryeMCR1: there is ccsm config for ctrl+super left/right15:49
Squarismi read everywhere that CCSM isnt supported (ie can crash your desktop).15:50
MCR1bug 101753915:50
ubot5bug 1017539 in compizconfig-settings-manager (Ubuntu) "Unity Dash/Compiz Unity plug-in: Missing possibility to change the keyboard shortcut (Ctrl+Tab) to switch between lenses" [Low,Confirmed] https://launchpad.net/bugs/101753915:50
MCR1bug 102274315:50
ubot5bug 1022743 in unity (Ubuntu) "Hardcoded Unity shortcuts responsible for several problems" [Low,In progress] https://launchpad.net/bugs/102274315:50
MCR1^^ this one is partially fixed, but not fully yet15:50
dednickSquarism: unfortunately there are always going to be conflicts with key-bindings, we can only do so much without making crazyily complicated ones. Do you use multiple workspaces? Or can you change the bindings in intellij?15:51
MCR1Squarism: Unfortunately CCSM is not installed by default - yes...15:51
MCR1Squarism: *maybe* this will change again as stability has improved a lot lately15:52
Squarismdednick, i use multiple workspaces. One of the reasons i like unity/ubuntu15:52
ryeSquarism: CCSM is simply modifying the options that later read by compiz plugins (and unityshell as a plugin) and I personally rendered my unity session unusable several times with ccsm (or at other times got compiz to crash). But the config is stored in ~/.config/dconf/user and i usually reverted to removing that and relogin. OTOH I am running a bleeding edge version of ubuntu/unity15:53
ryewhich reminds me I wanted to crash compiz and report that15:53
MCR1hehe15:54
dednickhmm. suspend menu button looked suspiciously like the system settings menu. Think i need sleep.15:54
Squarismdednick, And as intellij is what I work with all days regardless of OS (sometimes on collegues machines, sometimes on client computer) - i prefer to use factory settings there15:55
=== jono is now known as Guest7503
dednickSquarism: Super+Ctrl+left/right work ok for me.15:57
Squarismdednick, to reassign?15:57
ryeMCR1: by hardcoded shortcuts - do you mean that they are hardcoded in the code or that they are exposed via CCSM only?15:57
Squarism...wo CCSM?15:57
dednicki just assigned it in the keyboard shortcuts section of system settings15:58
dednickoh wait...15:58
dednickno, i was right. it is working. I just assigned it to the move window to left/right workspace. But the same works for switch to left/right15:59
dednickSquarism: ^15:59
MCR1rye: unfortunately hardcoded in the code16:00
Squarismdednick, hmm.. on 12.04 it doesnt work (tested right now) and at home i run 12.10 and there its not listed (in shortcut settings)  atleast16:01
dednickSquarism: I'm on 12.10. You on latest updates?16:01
Squarismi installed it 2 weeks ago16:02
Squarismfiled a bug on launchpad about then also16:02
dednickSquarism: system settings -> keyboard -> shortcuts -> navigation . Not there?16:02
Squarismdednick, no.. Exactly16:03
Squarism"Semi-maximizes window" or whatever its called16:04
Squarismdednick, you have such a setting?16:04
dednickSquarism: er, no. that sounds like something to do with an on-screen keyboard.16:06
Squarism"Semi maximize" = maximize to occupy half-right or half-right screen16:07
dednickSquarism: http://ubuntuone.com/40X7ZumcAvJk9IUD4EdDId16:09
Squarismdednick, sure - i can do that too.. but the default function "semi-maximize" is left on too / conflicts .. and it seems it has precedance16:12
dednickSquarism: i see. Use Super + Alt then? :)16:15
fginthersil2100, now that your switcher branch has landed, should we restart the daily testing?16:17
Squarismcan one get involved in unity in some way?16:18
Squarismie to release a patch on a released verison so to speak16:18
sil2100fginther: yes, let's do that :)16:19
* sil2100 almost got the ibus tests fixed16:19
sil2100Almost16:19
sil2100I need to prepare something packaging related to dee now16:20
fginthersil2100, ok16:20
=== dandrader|lunch is now known as dandrader
Squarismwould appriciate a lead on where to start - ie fixing issues in unity16:21
sil2100Squarism: yes, contributions are most welcome :)16:21
sil2100Ok, so - which version of Ubuntu are you running?16:21
dednickSquarism: I'll point you at the standard website. I'm not sure about the process for community contributions (but I'm sure there is one). http://unity.ubuntu.com/getinvolved/16:22
Squarismsil2100, 12.04 and 12.1016:22
SquarismI presented a choice here http://unity.ubuntu.com/getinvolved/development/16:24
Squarismis shortcut stuff in unity, common components or common libraries16:25
sil2100Squarism: the shortcut mechanism is in compiz, but Unity shortcut definitions are in the unity source code16:25
sil2100Squarism: although some of the shortcuts you mentioned are not related to Unity but to other compiz plugins16:26
Squarismoh ok16:26
sil2100Squarism: but everything you need is defined in the bazaar branches of lp:unity and lp:compiz16:27
sil2100But those repositories have the latest changes for the upcoming release 13.0416:27
sil2100(bleeding edge)16:27
Squarismoh.. hot!16:27
sil210012.10 branches are in lp:unity/6.0 and lp:compiz/0.9.816:28
sil2100But the rule is that we prefer patches first being sent to trunk (bleeding edge) and then back-ported to the earlier releases16:28
Squarismtaking notes!16:29
Squarismhow does the typical unity developer work? I mean.. if u develop.. how do you view/test your changes?16:30
sil2100Squarism: it's maybe a *bit* complicated, but well, the code is rather well readable16:31
sil2100hmmm16:31
sil2100I usually build packages from source and test them on my system16:31
MCR1Squarism: Right now there is Ubuntu Developer Week on the #ubuntu-classroom channel16:32
MCR1Squarism: http://ubuntuclassroom.wordpress.com/2013/01/30/ubuntu-developer-week-day-1-is-over-whats-up-on-day-2/16:32
SquarismWill that require a reboot for every test then?16:32
sil2100Squarism: it depends on developers - some prefer building packages directly from trunk and installing by make install, but I always prefer building a test package by using bzr builddeb and then testing16:32
sil2100Since I can easily revert to an earlier version16:32
sil2100Squarism: nooo, you just need to restart compiz/unity16:33
dednickSquarism: smaller changes can be tested on standalone unity components (eg dash or launcher standalone apps). But with things that need system-wide integration like shortcuts, we have dev staging sets which use to test.16:33
sil2100Squarism: it's usually a unity --replace call16:33
sil2100We're welcome to all new, precious contributors16:34
SquarismI do alot of programming in my free time so why not16:34
dednickSquarism: http://unity.ubuntu.com/getinvolved/development/unity/ gives instruction on how to build a staging set. But it "may" be a bit out of date.16:35
Squarismwill need to get a working process up first though16:35
SquarismThanx guys!16:36
SquarismWill try to bite my teeth into this when my current project is done.. (week or two)16:36
rperiersil2100: What do you think about bug https://bugs.launchpad.net/unity/+bug/1108956 ? I meant, finally this is a real bug or not I am confused16:41
ubot5Launchpad bug 1108956 in Unity "The dash closes when trying to switch from Command lens to home lens" [Undecided,New]16:41
rperier(otherwise I will work on another bug, np)16:46
MCR1rperier: IMHO the problem here is that Super is used to open/close the Dash primarily and Dash Home does not have a specific shortcut like the other lenses...16:47
rperierMCR1: this is not a shortcuts problems, but a problem about the switch, the transition16:51
rperieris it acceptable to have the dash closed when pressing "super" from the command lens ?16:52
rperierif yes, np this is not a bug and it can be closed :)16:52
MCR1If you change the behaviour the bug would be: Super does not close the Dash when Command Lens is opened16:54
MCR1but I noticed that pressing Alt+F2 again it does not close, which it imho should...16:55
MCR1rperier: But I confirmed the other related bug Dash->Command Lens, which you already fixed. And I can confirm your fix as well. Top job !17:03
rperierMCR1: you're confusing me :D17:06
rperierthe code fix is valid, and not the bug ?17:08
* rperier needs a coffee17:08
MCR1rperier: imho you should think about implementing a toggle function for Alt+F2 first, otherwise you will not be able to directly close the command lens17:10
rperierah !17:10
rperierI see17:11
MCR1rperier: as Alt+F2 does not close the command lens at the moment (which imho is a bug)17:11
MCR1with your solution (which is nice) you would always HAVE to switch from command to home before being able to fully close everything with keys...17:12
rperierMCR1: it works here, I meant If I press ALT+F2 it opens the command lens, if I press ALF+F2 a second time, it closes it17:12
rperiermhhh... I see17:13
MCR1raring, latest trunk ?17:13
rperierraring from mirrors17:13
MCR1does not close here17:13
rperierokay so I need to add a toogle function for ALT+F2 (I agree now) and then my bug might be acceptable ?17:14
rperierthe correct solution would be:   switch from home.lens to command.lens , switch from command.lens to home.lens (both without closing the dash)  + toogle function for ALT+F2 ?17:15
rperierideally17:16
MCR1I am just a community member adding his 2 cents -> you got to assign the bug to Ayatana Design and set it to incomplete17:16
rperierok :)17:16
sil2100rperier: hi! Yes, I agree with MCR1 that it's best to poke design about this ;)17:44
sil2100rperier: you can also try quickly poking JohnLea when he's around, usually a bit earlier17:45
sil2100Since this seems like a small issue that can be commented on real quickly17:45
sil2100fginther: hm, strangeness - the ps-indicators-autopilot-release-testing finished, but only ati tests ran for build 9817:48
sil2100fginther: why do things like that happen?17:48
sil2100I see you started job 99, hope we have more luck with that17:48
fginthersil2100, there was a hang in the intel and nvidia so it was restarted18:03
fginthersil2100,  there was a machine resource issue which I'm told is now resolved :-)18:03
sil2100uuuu ;) Great to know, thanks!18:04
=== dandrader is now known as dandrader|afk
=== dandrader|afk is now known as dandrader
cwaynemhall119: when is the switch to that new api?18:56
mhall119cwayne: for scopes?19:00
cwaynemhall119: yar19:00
mhall119cwayne: nope, but soon19:01
mterryYay for the unity stack passing again19:03
* mterry looks at the dbus-test-runner failing because of g_type_init19:03
mterryah, tedg already merged that19:03
mterryfginther, hello!  So about crashes that happen during autopilot tests, that cause the remainder of the tests not to be run.  Can we change the jenkins logic to fail the build if autopilot or X crashes or some such?19:12
fginthermterry, yes, I think veebers was looking into implementing that19:13
fginthermterry, we discussed it yesterday19:13
mterryfginther, awesome, OK19:13
luvoh, I see my merged got merged into lp:unity even though it's not working properly yet ;-)19:15
luvs/merged/branch/19:15
luvwell, at least I have motivation to polish it out tonight :-)19:16
mterrybregma, a couple focus test failures: http://10.97.0.1:8080/job/ps-indicators-autopilot-release-testing/label=autopilot-ati/lastCompletedBuild/testReport/19:17
* bregma give a quick glance19:18
* bregma suspects https://code.launchpad.net/~sil2100/unity/autopilot_enhance_test_hud/+merge/14559519:21
mterrybregma, how would that fix the failures?  That just adds a new assert19:26
fginthermterry, just a theory, but the assert will wait 10 seconds for it to become true, that might be enough to fix the test if it's timing related19:37
fginthermterry, in this case, the test may be failing because the application window is not ready. the assert sil2100 added will fix that19:38
mterryfginther, hrm.  OK.  We'll see if it helps19:39
luvI see, it's just a proposal, missed that in the email. I was a bit scared that code would make it through.20:05
mterryfginther, could jenkins have some -dbg packages installed when running?  That way, when apport creates a .crash file, it would have symbols built-in20:08
mterryfginther, bryce can't get a good stack out of the Xorg crash20:09
fginthermterry, if we know the list of packages, we can add it20:10
fginthermterry, is that something that would normally be handled by apport-retrace?20:10
mterryfginther, yeah I guess.  But apport can do it upfront if dbg symbols are already around20:12
mterryfginther, we'd want at least xserver-xorg-core-dbg.  Asking bryce if there are other useful ones20:13
luvTrevinho: hi, im just reading your review, thanks for the comments. I dont fully understand the comment " At that point call it into GetMenus() putting the items you computed into the result list." under line 2220:22
luvalso, does it really matter if I use "for (auto const& w : Windows())" instead of 'for ( auto w: Windows() ) {" ... Window is just a 64bit integer anyway20:23
luvi am happy to change so it follows the project's way, indeed20:24
mterryfginther, can we start with at least: libdconf-dbg libdconf-dbus-1-dbg xserver-xorg-core-dbg20:24
mterryfginther, those seem to be related to the two crash files we have now20:24
fginthermterry, got it20:27
Trevinholuv: no, Windows() a list of WindowPtr... Yeah, copying a shared_ptr is not that much, but if we can save it it's better20:51
Trevinholuv: I mean, you should add a call into GetMenus() to ensure that you build your _window_menus_items, then you merge these with the results... But it's better if you do that into GetMenus20:52
mterryfginther, also libdrm2-dbg libgl1-mesa-dri-dbg  libgl1-mesa-glx-dbg20:56
mterryfginther, is it also possible to have jenkins run ubuntu-bug --save=PATH and upload PATH too (for logs and stuff when it crashes)20:57
mterryfginther, it can be run on a .crash file20:57
fgintherveebers, ^^20:57
veebersmterry, fginther yep I can add that too20:58
fginthermterry, we were seeing apport-retrace crash on the test machines20:59
fginthermterry, which is causing the unprocessed crashes, hopefully these other steps will work21:00
=== salem_ is now known as _salem
=== _salem is now known as salem_

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