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

Kupiakos42Hello. I'm having trouble getting Wine applications to show their proper icons. When I start up a Windows application (e.g. msword), it shows up in the Unity sidebar as "Wine Windows Program Loader" and has the Wine icon instead of the Microsoft Word icon and title text. This does not occur in other Desktop Environments, such as Gnome Classic. Any Ideas?01:39
dufluKupiakos42: I thought we had a bug resembling your Wine problem but can't find it. Please log one using "ubuntu-bug unity"05:26
Mirvduflu: did you see bug #1103475 ?05:54
ubot5bug 1103475 in Compiz "compiz fails to start properly after update to 1:0.9.8.6+bzr3432-0ubuntu1~test1" [Undecided,Confirmed] https://launchpad.net/bugs/110347505:54
Mirvalthough, fginther already prepared a branch that fixes the issue so we'll just get that05:55
Mirvpossibly with that commit taken out sil2100 can proceed to doing the new quantal sru05:57
dufluYes, I saw that bug but had no immediate "upstream" explanation. More likely something distro changed ;)05:58
popeyduflu: fginther has identified the fix for bug 1060327 as the cause of bug 110347506:51
ubot5bug 1060327 in Compiz 0.9.8 "compiz crashed with SIGSEGV in compiz::opengl::bindTexImageGLX() from TfpTexture::bindTexImage()" [High,Fix committed] https://launchpad.net/bugs/106032706:51
ubot5bug 1103475 in Compiz "compiz fails to start properly after update to 1:0.9.8.6+bzr3432-0ubuntu1~test1" [Undecided,Confirmed] https://launchpad.net/bugs/110347506:51
popeyhe's built packages which i can test here shortly06:51
duflupopey: Yeah that's weird because it's a critical fix which we're using on raring06:52
popeyduflu: which also crashes in a vm06:56
popey(raring that is)06:56
duflupopey: You need updates to get the fix for raring I think... ?06:56
duflupopey: I meant the offending fix has been verified as working and fixing crashes for lots of VM users, on raring06:57
dufluSomething must be different for 0.9.8 in quantal06:57
popeyduflu: I need to verify raring06:58
popeyneed to go afk for 2 hours then will come back to it.06:58
duflupopey: Also nvidia has a very similar crash, different bug06:58
=== larsu is now known as Guest85736
didrockshey sil2100!08:37
sil2100didrocks: hey! ;)08:44
didrockssil2100: how are you? did you finish the administrative stuff for your car? :)08:44
sil2100didrocks: yes! Finally got my car back, after 2 months of pointlessness - I managed barely in time for the office to close08:45
didrocksgreat ;)08:45
didrockssil2100: do you have time to investigate something weird? see ps-unity-autopilot-release-testing and ps-indicators-autopilot-release-testing08:46
didrockswe have 110+ failures in one case and 22 in the othe08:46
didrocksother*08:46
didrocksso, apparently 25% of failures08:46
didrockscan you check we have a valid regression from the ppa?08:46
sil2100Oh shit08:46
didrocks(weird, because the indicator run is just the new indicator, without any new unity)08:47
didrocksI really wonder what happens?08:47
sil2100Let me fire up apview08:47
didrocksthanks08:47
popeyduflu sil2100 have tested with the package from fginther in virtualbox which has the offending code backed out, and it works08:52
sil2100hm, I need to think what could be the reason for all those failures08:52
didrockssil2100: is the video showing a correct behavior?08:52
duflupopey: Not just "works", which we know it works, but clearly and immediately resolves the bug?08:52
popeywell, doesn't crash compiz anymore08:53
dufluOk08:53
dufluThat's a shame for users of quantal VMs. That was the biggest issue for VM/LLVMpipe users08:53
sil2100didrocks: hah, wait one moment - since I wanted to say 'yes', but I just noticed that one of the failures is caused by an apport crash dialog ;p08:53
sil2100Let me check all of them08:54
didrocksah :)08:54
didrocksahah!08:54
didrocksthat can be yeah ;)08:54
sil2100didrocks: so, the indicators tests has only the new indicator packages installed, or are there other updates?08:55
didrockssil2100: only the new indicator packages08:55
didrockssil2100: not sure of the last run as I wasn't the one running it, but the run before for sure08:56
sil2100duflu: should we revert that fix for now then, or maybe leave it and wait for it to get properly fixed sooner or later?08:56
sil2100didrocks: most of the failures look like genuine failures, a mystery why they're so easily reproducible by a second run - some of them are strange, like the hud geometry one, I'm looking into it now08:59
didrockssil2100: thanks, keep me posted! this grows anxiety in me :)08:59
didrockssil2100: if only the unity ones were failing, I would understand because of https://code.launchpad.net/~didrocks/compiz/default-hsize-vsize/+merge/14446409:00
didrocksbut the indicator ones don't have unity…09:00
duflusil2100: If it's easy to prove "is broken and now fixed" then back it out of 0.9.8, sure. But I don't think you'll find it's broken at all in trunk09:00
=== larsu_ is now known as larsu
sil2100didrocks: indeed, as compiz shouldn't get updated there09:02
didrockssil2100: yeah, compiz is 1:0.9.9~daily13.01.21-0ubuntu109:03
didrocksso distro, not ppa09:03
veeberssil2100: I notice in the logs that this error (or similar) is repeated a lot: ** (autopilot:2012): WARNING **: failed to commit changes to dconf: GDBus.Error:org.gtk.GDBus.UnmappedGError.Quark._g_2dfile_2derror_2dquark.Code4: Failed to create file '/home/jenkins/.config/dconf/user.2Q7IRW': No such file or directory09:04
veeberssil2100: (this is re: the autopilot jobs)09:04
didrocksso would that be the cause of an apport popup making the test failing?09:06
sil2100veebers: hi! In which test's logs?09:07
sil2100veebers: since it would be a good explaination of why the HudAlternateKeybinding tests are failing, but I don't see that in the testlogs sadly...09:07
veeberssil2100: hello :-) I'm looking at the file 'ap_test_debug_log.txt' which is in the jobs results/artifacts/09:08
sil2100Ok, now I see the file09:09
sil2100veebers: strange thing, since it's for every application - autopilot, gedit, gcalc09:10
didrockslet's look what version of dconf we have09:12
didrockshum 0.15.2-0ubuntu309:13
didrocksit includes   * debian/patches/git_handle_empty_config.patch:09:13
didrocks    - include fix to create the config directory when needed as well09:13
didrocksI'm wondering if this one really fixed it09:13
didrocksveebers: you do have access to the machines, right?09:13
sil2100Is that a recent fix?09:13
didrockslike being able to log in09:13
didrockssil2100: yeah, yesterday evening, after the new version of dconf09:13
didrockssil2100: but it's included09:13
didrocksin this iso09:13
didrocksso I wonder…09:14
sil2100didrocks: since those messages don't appear in previous test-runs09:14
didrocks0.15.2-0ubuntu2 and 0.15.2-0ubuntu309:14
didrockssil2100: previous being?09:14
veebersdidrocks: yes09:14
didrocksthose which run?09:14
sil2100didrocks: for instance, in 83 build09:14
didrockssil2100: yeah, not sure if it's the real cause, but at least, it's an issue09:14
sil2100didrocks: build 83 had only 2 failures09:15
sil2100didrocks: who knows - might be related?09:15
didrocksveebers: not sure for how long you are still around, but it would be good to check on the machine if you have this .config/dconf directory :)09:15
sil2100veebers: good catch!09:15
veebersI'll jump on and check it out (if it hasn't been reprovisioned yet)09:16
sil2100Since the HUD geometries didn't change really, not sure if anything could have caused a failure there09:16
didrocksthanks veebers :)09:16
veebersdidrocks, sil2100: ls .config/  -> no dconf directory09:17
didrocksahah!09:17
didrocksso maybe seb's fix in 0.15.2-0ubuntu3 is still not enough09:17
didrocksand then, we are getting an apport dialog which is totally messing our results?09:18
didrockssil2100: wdyt? ^09:18
didrocksveebers: btw, those jobs should collect /var/crash if anything, that would be handy09:18
veeberssil2100: thanks, I generally take a look at that log file as I find it helpful09:18
veebersdidrocks: it should use apprt-retrace to grab anything in that folder09:19
veebersbut lately apport-retrace has been excepting/trackbacking09:19
didrocksveebers: apport-retrace is for retracing, isn't it? I meant, just archiving what we have in /var/crash :)09:19
veebersI'm considering just grabbing the whole directory instead (like you've just suggested)09:19
didrocksyeah09:19
didrockssounds good to me :)09:19
veebersdidrocks: Ah I see09:19
veebersyeah I'll add that first thing in the morning :-)09:20
didrocksveebers: meanwhile, what's in this dir?09:20
didrocksso that we can maybe debug our today's while you are not around :)09:20
veeberson the remote machine? nothing09:20
didrockslike putting the .crash file somewhere will be handy (people.canonical.com?)09:20
didrockshum09:20
didrocksthat's weird09:20
didrocksoh09:21
didrocksveebers: I guess apport-retrace killed them09:21
didrocksthat's why we don't get anything anymore09:21
didrocksI'll stay on the theory of dconf, and hoping I can reproduce that on a guest session09:21
veebersoh they tend to be left there09:22
veebersIt's just looking right now there is nothing09:22
didrocksweird, I thought this would crash09:22
didrocksseb128: hey!09:22
seb128lut didrocks09:22
didrockshow are you?09:22
seb128good, you?09:22
didrockswell, seems everything is against daily release! :-)09:23
didrocksYOU in that case ;)09:23
didrocksseb128: not sure when you arrive and if you are seeing our dconf discussion?09:23
seb128didrocks, just arrived, I forgot to start my IRC before starting on emails09:23
seb128what's up?09:23
sil2100veebers: good to know a log like that exists ;)09:23
didrocksseb128: so, we have 22 failures on the indicators and 110+ on the unity ones09:23
veebersdidrocks: Right I better leave for the night. Let me know if there is anything else I can do and I'll hit it in the morning09:23
didrocksso 25% :)09:23
didrocksveebers: yeah, thanks for hint! let's keep in touch tomorrow :)09:24
didrocksveebers: thanks again for looking at those :)09:24
didrocksseb128: we have a lot of:09:24
didrocksfailed to commit changes to dconf: GDBus.Error:org.gtk.GDBus.UnmappedGError.Quark._g_2dfile_2derror_2dquark.Code409:24
didrocksFailed to create file '/home/jenkins/.config/dconf/user.2Q7IRW': No such file or directory09:24
didrocksin the logs09:24
didrockshowever, we do have your latest dconf09:24
seb128ok, talk to desrt :p09:24
didrocks0.15.2-0ubuntu3, including debian/patches/git_handle_empty_config.patch then09:24
seb128I backported the fixes he told me to backport09:24
seb128that fixes guest session/liveCD for me09:25
seb128which were broken before09:25
seb128I guess it's just a warning09:25
seb128but your tests fail on any output?09:25
didrocksseb128: well, tests seem to fail everything this is set09:25
didrockslike if the new configuration isn't taken into account09:26
didrocksand we don't have a ~/.config/dconf09:26
seb128grah09:26
seb128of course in 0ubuntu3 I forgot to add the patch to the series09:26
didrocksseb128: "working on a guest" hum hum ;)09:26
veebersdidrocks: no worries, see you guys tomorrow09:26
didrocksseb128: I was trusting you!!! :-)09:26
didrocksveebers: see you tomorrow! you can go to bed with this light of hope ^09:27
seb128didrocks, yeah, well 0ubuntu2 was fixing the issue, 0ubuntu3 is just a follow up update I did quickly at midnight when I was supposed to be off for an hour :p09:27
didrocksseb128: revert! :-)09:27
sil2100;)09:30
didrockssil2100: oh, btw, I switched to one ws in compiz09:39
didrockssil2100: do you think that some tests will be impacted?09:39
sil2100didrocks: in overall?09:40
didrockssil2100: yep :)09:40
sil2100didrocks: yes, since there are many switcher tests that assume 4 workspaces and switch to them to test the switcher09:40
didrockssil2100: hum, so maybe I shouldn't have switch in advance09:41
didrocksswitched*09:41
sil2100didrocks: let me check how many tests are there with that09:41
didrockssil2100: ok, waiting for you first then :)09:41
sil2100didrocks: that's a good occasion to making the tests better09:41
sil2100didrocks: since we can simply not execute those tests that assume more than one workspace ;)09:41
didrocksyep :)09:41
didrockssil2100: I guess you'll have to change hsize and vsize09:42
didrockssil2100: any estimate? Should I just revert the change for today?09:49
sil2100didrocks: there's a lot of tests with workspaces > 1 requirement, but I'll try fixing that - give me an hour09:52
didrockssil2100: oh, if you can do that in the same run so quickly, this would be awesome and I can include this additional commit :)09:53
didrockssil2100: you do know which keys to change, right? need any help?09:53
sil2100Yes, no problem right now09:57
sil2100didrocks: anyway, you want the indicator-related tests to be fixed ASAP?10:20
sil2100Regarding one workspace?10:20
didrockssil2100: I relaunched the indicator already, just to ensure we have less failures10:20
didrockssil2100: but no, just have one MP with everything if possible10:20
luvhi, so Im working on this patch to add list of open windows to GetMenus() ... it's basically based on a for loop for ( auto w: Windows() ) { }10:21
luvin the for loop I would like to initialize the callbacks for when a menuitem is activated. What would you recommend? closures?10:22
rperiersil2100: I plan to work on bug 1019457 . (The previous one about the trash is still under discussion) What do you think ? Is it easy ?10:43
ubot5bug 1019457 in unity (Ubuntu) "The Dash closes when trying to switch to the Command lens (Alt+F2)" [Medium,Confirmed] https://launchpad.net/bugs/101945710:43
rperierhi btw10:43
didrockssil2100: if you are interested, I've done a rerun of the indicator tests10:45
didrocksit's weird that nvidia just has one failure though, as more tests are failing10:45
didrocks(on the others)10:45
didrockswhich seems to be multiple workspace related10:46
sil2100huh10:46
didrocksah, less tests run10:46
didrockson nvidia10:46
seb128didrocks, so the dconf issue is resolved?10:47
didrocksseb128: we have less failures, so I guess so, the rest seems to be related to hsize and vsize == 110:47
didrocksseb128: I'll keep you posted once we can run the unity ones10:48
didrocks(will probably be this afternoon)10:48
Trevinhodidrocks, seb128: are you referring to the impossibility to change the vsize/hsize values?ù10:49
didrocksTrevinho: hum? no, just that the tests were written with in mind that hsize,vsize was > 110:50
didrocksTrevinho: dconf had another issue tonight, but it seems to be now fixed10:50
Trevinhodidrocks: mh ok... because both me and bschaefer (and others in the unity team) had that issue10:51
sil2100didrocks: I'm almost done10:51
Trevinhodidrocks: mh, ok.. I was still upgrading last version10:51
Trevinhodidrocks: yeah, it's fixed now... :)10:53
didrocksgreat :)10:53
Trevinho:)10:54
luvhmm10:54
luvother approach I thought about would be to set up the menuitems with an id and store a map which would translate id->Window_number10:57
luvthough this doesnt seem so smooth10:57
luvwould be nice if I could send custom data to the menuitem-activated callback10:58
sil2100rperier: oh! That seems like a nice bug to start out with indeed!11:01
sil2100rperier: if you want to work on it, probably best if you 'assign' yourself to this bug - so that no duplication of work happens11:02
sil2100luv: hard to say which one is better..11:02
=== _salem is now known as salem_
=== salem_ is now known as _salem
=== MacSlow is now known as MacSlow|lunch
=== _salem is now known as salem_
MCR1andyrock, didrocks, Trevinho: Hi :) I noticed we have no default shortcut for Fullscreening/Un-Fullscreening windows... I would like to add Alt+Enter for that (like on win)... What is the procedure to get this approved ?12:34
didrocksMCR1: the procedure is to ping JohnLea ^12:35
andyrockMCR1, yeah ping John12:35
didrocksMCR1: I'm unsure that all applications have a fullscreen feature12:35
didrocksand that they are using the same shortcut for it12:35
MCR1on windows not, on Linux they have12:35
andyrockMCR1, f1112:35
didrockslike chrome has something different from totem12:35
didrocksand it seems to me this feature is per application12:36
MCR1andyrock: F11 just will work 4 some applications12:36
didrocksnot sure that compiz has anything to deal with that12:36
MCR1I want it for all of them12:36
MCR1yes it has12:36
didrocksMCR1: how compiz can command fullscreen on firefox/chromium?12:36
andyrockMCR1, no it's an app shortcut but it's the de-facto standard12:36
Trevinhodidrocks: it's not command related...12:36
Trevinhodidrocks: compiz just draw that window fullscreen undecorated12:37
TrevinhoMCR1: do you meant thtat?12:37
MCR1didrocks, andyrock: CCSM-> Extra WM Actions->Toggle Fullscreen12:37
MCR1I got that shortcut here since ages and it works for all applications12:38
MCR1those that already have F11 defined will also work12:38
MCR1then the CCSM shortcut does the same12:38
MCR1for example Chromium will enter F11 fullscreen if you hit Alt+Enter12:38
didrocksTrevinho: yeah, that's globally what I mean12:38
didrocksMCR1: are you sure it's doing the same? like really entering the fullscreen mode of the app?12:39
MCR1and it acts like you would have hit F1112:39
didrocksand not just take the window, remove decoration and put it fullscreen?12:39
didrocksinteresting12:39
MCR1didrocks: yes12:39
Trevinhodidrocks: however trying it both firefox and chrome seem to undestand that command and they behave as you'd press f1112:39
MCR1Trevinho: yes, exactly12:39
didrockswill worth testing with games and other apps12:40
Trevinhodidrocks: probably they monitor their wm hints and based on them they change UI as well (nice thing)12:40
MCR1but win users used to Alt+Enter will have that for media players for example12:40
didrocksTrevinho: possible, yeah12:40
Trevinhodidrocks: frecell works12:40
MCR1I also would like to change Ctrl+Alt+Del to not present the gnome standard (log-out) but the gnome-system-monitor, processes tab...12:41
MCR1but that is another story12:41
Trevinhodidrocks: it seems gtk apps are working as well12:41
MCR1Trevinho: As I said - I am using it since ages12:41
Trevinhoalso evince...12:42
TrevinhoMCR1: yeah, I was using that too at the compiz times, then I forgot about that :)12:42
MCR1hehe12:42
MCR1I noticed our standard-windows shortcuts have a lot of place left (see the shortcut overlay)12:43
MCR1JohnLea: ^^ Please comment on it...12:43
MCR1JohnLea: Then you can get it implemented by tomorrow ;)12:43
TrevinhoMCR1: so, it's nice for me... ask more to JohnLea (or open a bug for ayatana-design);12:44
MCR1Trevinho: TBH, I did that a long time ago, but design is not really fast to respond :(12:45
=== dandrader_ is now known as dandrader|afk
TrevinhoMCR1: eh, you know we're all busy12:45
MCR1sure.12:45
MCR1bug 106916512:46
ubot5bug 1069165 in Unity "Unity Standard Shortcuts: Make it easier for Windows users" [Undecided,Incomplete] https://launchpad.net/bugs/106916512:46
MCR1Trevinho: Btw, I am not sure which commit caused it, but Icon Edge Illumination is broken...12:50
luvwell, that's double-sided12:50
TrevinhoMCR1: mh, I've noticed that...12:50
TrevinhoMCR1: would you be so kind to give it a look? :)12:50
MCR1you mean test your fix ?12:51
luvchanging current shortcuts is not really a good idea as people are already using them - and you certainly should not annoy current users, that's more important than getting new users12:51
MCR1luv: You are right with Ctrl+Alt+Del, but for Fullscreen/Un-Fullscreen there is nothing to change12:51
MCR1as it is not there yet (this shortcut)12:52
luvwell, it might clash with current applications anyway!12:52
luvterminator, vim ...12:52
mitya57AFAIR windows now uses Ctrl+Shift+Escape for task manager, that's not bound to anything in Ubuntu12:52
MCR1mitya57: Really ? I did not know that, hehe - sounds like a funny shortcut12:53
luvthat's why i like having all shortcuts on Super+key because i can use the rest safely12:53
luvwithout nasty suprises12:54
MCR1luv: The most important thing is to not have anything hardcoded...12:54
luvso, it's not as simple as it seems .... as is always the case when designing and writing software12:54
MCR1luv: I mean hardcoded shortcuts12:55
MCR1luv: I've eliminated a few of those already12:55
MCR1but a few are still left (TODO)12:55
luvwell that's certainly nice, but still, I use alt+enter as a terminator shortcut now, I update ubuntu, press alt+enter and it will do something totally different12:55
mitya57MCR1: google says I'm right: http://dottech.org/26520/use-ctrl-shift-esc-to-quickly-bring-up-windows-task-manager/12:55
luvyou can imagine my reaction (-:12:55
MCR1luv: sure...12:56
didrocksbregma: hey, geis is daily-built, why have you done a manual upload?12:56
didrockscyphermox: FYI ^12:56
didrocksand https://code.launchpad.net/~oif-team/geis/trunk doesn't reflect anymore what you uploaded12:57
MCR1luv: But I guess it takes only a few seconds 4 you to change it in CCSM, no ?12:57
didrocksok my bad, need coffee12:58
didrocksbregma: unping, got tricked by ******* thunderbird12:58
luvMCR1: few minutes i would say12:58
MCR1(I recommend the CCSM search function -> it is excellent ;))12:58
luvwell i won't get any results for "alt+enter" though ;)12:59
MCR1try fullscreen12:59
MCR1(and you can also search for shortcuts in CCSM, btw)12:59
luvin my 12.04lts "no matches found" :-)13:00
MCR1whoot ?13:00
luvoh, i see, advanced search13:00
MCR1CCSM->Advanced Search13:00
luvthat's neat13:00
MCR1yep13:00
=== dandrader|afk is now known as dandrader
MCR1CCSM is excellent - I still do not understand why it is not in standard-ubuntu13:01
MCR1I mean I understand it was unstable13:01
MCR1but this is soon history13:01
MCR1on raring it hardly crashes nowadays...13:01
MCR1(but still some work to do...)13:02
MCR1mitya57: Thx 4 the info :)13:02
rperiersil2100: ping13:06
JohnLeaMCR1; the Ctrl Alt Del issue is a duplicate of bug #890747  , will you be able to fix this issue?13:29
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/89074713:29
MCR1Not 100% sure, but I guess so...13:29
MCR1Would need to change the gnome-compatibility CCSM options I guess - pretty sure I could do it13:30
JohnLeaMCR1; btw, if there is an important bug you need me to look at, ping me in IRC,13:30
JohnLeaMCR1; cool ;-)13:30
JohnLeaMCR1: would be good to get this fixed13:30
MCR1ok, will do - Thanks a lot. You are a bit hard to get to ;)13:30
MCR1okay - I am on it in the next days13:31
MCR1expect it to be fixed next week...13:31
JohnLeaMCR1; I have been out of the office for a month over christmas, but am back now13:31
MCR1cool13:31
JohnLeaMRC1; great, thx!13:31
MCR1good to know13:31
MCR1yw13:31
MCR1& thanks 4 the approval13:31
MCR1JohnLea: If you are here - please also approve this: bug 110402913:32
ubot5bug 1104029 in Unity "Unity Shortcut Overlay: Minor problems remain" [Undecided,New] https://launchpad.net/bugs/110402913:32
MCR1JohnLea: The last text revision of the shortcut overlay was done by me (with help of bschaefer) - but I guess I missed those last time13:33
MCR1JohnLea: I am working on fixing that right now..13:33
davidcalleJohnLea, hi, since you are looking at Unity bugs, I would need your thoughts on this one : https://bugs.launchpad.net/unity/+bug/109876913:34
ubot5Launchpad bug 1098769 in unity (Ubuntu) "Previews should not enlarge small images" [Undecided,New]13:34
JohnLeaMCR1; re. the shortcut that says "Restores or minimises the window", this is the desired behaviour, we should really fix the issue with it not minimising windows rather than change the text ;-)  Re. the spelling issue, "Minimises" is the correct spelling in the UK so this is a internationalisation issue.  Re. "current window" I think this is ok for now, both "selected window" and "focused window" would also work, and "currently selec13:48
JohnLeated window" would be the most accurate but too verbose.  I'm not sure which is better, I'll try to make sure we look at when we do the next round of user testing to try to get some data13:48
JohnLeaMCR1; any bug that deals with user facing changes should also be maked as 'also affects project'  "ayatana-design"13:49
JohnLeaMCR1; if this isn't added to bugs we don't see them13:49
MCR1JohnLea: ok13:49
JohnLeaMCR1; If you are interested in taking a look at fixing it, the window minimise issue is bug #96609913:51
ubot5bug 966099 in unity (Ubuntu) "Keyboard shortcut overlay says Ctrl+Super+Down "minimises" the current window, but it doesn't" [High,Triaged] https://launchpad.net/bugs/96609913:51
JohnLeadavidcalle; yes, previews should never upscale images, they should only down scale images.  If a image is too small, it should be rendered at 100% and centred vertically and horizontally in the preview image area13:53
MCR1JohnLea: Ok, will do that also. Probably I will find some Compiz shortcut that does minimize, but I will have to change a bit more than just a shortcut to fix this...13:53
JohnLeaMCR1; the issue with this bug is that the action of the keyboard shortcut depends on the window's state e.g. "restore the window if it is maximised" or "minimise the window if it is restored".  Compiz can't do this at the moment, but if my memory serves me correctly the last time this was discussed it was mentioned that it would not be too hard to fix (but my memory might be wrong).  I think I once discussed this with didrocks , the13:55
JohnLeaother person to ask about this is smspillaz13:55
davidcalleJohnLea, thanks for the confirmation13:57
MCR1JohnLea: Yes - that is exactly the problem - it would need quite some changes to work (not sure how to accomplish that best adhoc)...13:58
MCR1Thanks - I've contact to Sam and Daniel ;)13:58
MCR1JohnLea: Thanks 4 all your time and all the information. Highly appreciated. So thanks alot once again - I've got something to work on now ;)13:59
JohnLeadavidcalle, do you need me to ack a bug for this or contact anyone to help get this fixed?  If you haven't already, you might want to discuss this with dednick14:00
JohnLeaMCR1; no thank you ;-)  Improving quality is the most important feature we need in Ubuntu!14:01
MCR1That is my playn ;)14:01
MCR1*plan14:01
dednickJohnLea, davidcalle: I thought we discussed a maximum scaling factor for the images? Was quite some time ago though...14:02
davidcalleJohnLea, I haven't contacted anyone yet. Just wanted to make sure you knew about it first.14:02
=== MacSlow|lunch is now known as MacSlow
mterrySo those failures this morning were just due to the workspace snafu right?14:39
mterry(autopilot failures I mean)14:40
JohnLeadavidcalle; thanks, yes not scaling too small images sounds good to me15:00
=== dandrader is now known as dandrader|afk
=== dandrader|afk is now known as dandrader
rperiersil2100: oh sorry I did not see your answer ;)15:30
rperier(I did not receive the corresponding notification... strange...)15:30
sil2100rperier: did it reach now ;) ?15:35
rperieryeah :)15:35
rperierI assigned it to me15:36
rperieroh btw, if I start unity in this way:  "unity-env; unity --replace" the unity from trunk won't put craps into my existing settings ?15:37
rperier(is it really separated ?)15:37
rperier(I am talking about .config, .local and so on)15:37
sil2100hm, I'm not entirely sure anymore ;)15:39
rperiermhhh I should probably export XDG_CONFIG_HOME and XDG_DATA_HOME in unity-env to be sure...15:40
didrockssil2100: hum, it seems we still have a lot of failures…15:40
didrockssil2100: look at ati/nvidia…15:41
didrocksunity job15:41
sil2100didrocks: in indicators, or in unity autopilot?15:41
didrockssil2100: unity15:41
sil2100Build 490?15:42
didrockssil2100: 49 you mean?15:43
didrocksoupss15:43
didrocks48*15:43
didrocksps-unity-autopilot-release-testing15:43
didrockssil2100: ignore intel, it was a provisionning machine issue15:43
sil2100Ah, ok, sorry, had the wrong job loaded ;)15:44
sil2100Holy15:44
sil2100hm, ok, wait, these seem to be some autopilot issues15:44
didrockssil2100: I hope all this mess has just ONE issue :)15:45
sil2100didrocks: I think I saw once the "Class 'SwitcherController' has no attribute 'show_desktop_disabled'." issue before somewhere!15:46
sil2100btw.15:46
sil2100Ah, nevermind15:46
sil2100Funny, build 47 has only 6 failures, while suddenly 48 - blam, 183 failures ;)15:47
didrockssil2100: hum, build 47 didn't start at all15:49
didrockssee the -326 tests running15:49
didrockssil2100: nothing obvious?16:00
sil2100didrocks: uno momento more ;)?16:01
didrockssil2100: evidenza :)16:03
didrocksmterry: hangout? :)16:03
mterrydidrocks, sure16:03
didrocksit's linked to the invitation16:04
mterryoh16:06
mterryhrm16:07
sil2100didrocks: ok, I see one possible reason for some of the failures ;)16:08
mterrydidrocks, it stopped working, am trying to reconnect16:08
didrockssil2100: I'm lightning a candle16:11
sil2100didrocks: ok, so actually, we'll have to fix unity, I'll prepare a merge request - due to some code refactoring, the properties that were normally added for introspection in the switcher's Controller now are actuallty in the pimpl, while the new Controller class doesn't export them16:22
sil2100didrocks: so I'll re-add the necessary properties that we would need for the switcher16:22
didrockssil2100: ah interesting, and good catch! :)16:23
didrockssil2100: sure, keep me posted ;)16:23
bregmasil2100, I refactoring the Switcher to add the properties back in already, is there a bug number for this failure?16:25
sil2100bregma: excellent! No, no bug number yet - we just have a lot of failures due to the missing show_desktop_disabled property16:26
sil2100The lack of the rest, like monitor, detail_mode, are possibly also problematic16:27
didrockssil2100: it's not the only property lacking16:27
didrockssil2100: I click and yeah, other properties are lacking16:27
didrocks2 solutions, either reverting the whole factoring (:/) or ensuring we are exposing quickly everything16:27
sil2100bregma: is there a branch and a merge proposal already?16:27
didrocksbregma: wdyt? ^16:27
sil2100bregma: would be good if all properties from ShellController would be accessible through Controller16:28
sil2100Possibly as soon as possible ;p16:28
bregmasome of the properties were in a MP that went in late yesterday, the rest will come today16:28
sil2100bregma: excellent! \o16:28
bregmasome of those things are internal state and do not really belong as publicly introspectable16:28
=== dandrader is now known as dandrader|afk
bregmaotherwise, we're stuck with bad code because the whitebox tests require it16:29
=== ricotz_ is now known as ricotz
didrocks bregma: right, but you are stucking the releases right now :)16:30
sil2100bregma: if needed, I could try making autopilot not use some of those properties - just tell me which of the existing ones you think shouldn't be introspected16:34
bregmasil2100, I'm just going to make all the previously public properties public again, we can worry about the autopilot tests later when the time comes16:35
sil2100bregma: ok, thanks16:37
sil2100bregma: give us a sign once there's a merge request to review!16:40
bregmasil2100, re-added the introspectable properties as a part of https://code.launchpad.net/~bregma/unity/refactor-switcher-controller-1/+merge/14474316:59
bregma'cos it was already proposed and is about separating the private and public parts anyway16:59
Sid_PaytonHello, could anyone help me with a basic set of rules for creating a GUI? It was wondering what aspects a designer has to consider for whatever (desktop, phone) he is targeting. Are there some golden rules like Miller's magical seven? If this isn't the right place to ask these question please tell me where I should go to. Thanks17:02
ryeping people, https://bugs.launchpad.net/ubuntu/+source/unity/+bug/877778 - seems that there is a regression in raring with exactly the same description17:14
ubot5Launchpad bug 877778 in unity (Ubuntu) "Showdesktoped window contents invisible in window spread" [High,Fix released]17:14
ryeShould I re-open the bug or file another one?17:15
bschaeferrye, I would open a new once, since that one was fixed in 5.1017:18
ryebschaefer: ok17:18
bschaeferrye, worst case, someone goes and marks that as a dup, and reopens the other one :)17:18
rye:)17:18
fginthersil2100, did you create a bug for the unity-autopilot-release failures17:20
fginther?17:20
ryealso, it looks like alt-tabbing to show desktop broke - now it requires two alt-tabs17:23
bschaeferwell it depends...show desktop has 2 states, the first time you use it, it hides all windows. If you don't use it again, and restore the windows manually17:24
bschaeferthen it still thinks all the windows are hidden17:24
bschaeferso if you try again, it will attempt to un-hide all the windows, (which makes it look like it doesn't work)17:24
bregmawow, doesn;t that sound confusing?17:26
bschaeferyes it is...17:26
ryebschaefer: uhm... can it...mm check that there are windows to hide?17:29
ryeand hide them if there are at least one? But then it needs to track what windows were already minimized17:30
ryehm17:30
ryeor clicking one window should display all others too17:31
bschaeferyeah, but someone has to do it :)17:31
ryebut the current perception is that it is broken and requires hitting 2 times to show desktop17:31
bschaeferat best, i would think changing the icon to indicate the toggle17:31
ryebschaefer: I don't think, that would not help much, "I want to show the desktop? Huh? SHOW ME THE DESKTOP"17:33
sil2100fginther: one moment17:33
bschaeferwell, even if you were to check if all the windows were 'un-hidden' there is that change you have 1 or 2 windows still hidden creating the same confusion17:33
* rye was quite surprised to see that one friend of mine actually uses that functionality quite often. I "use" that only when I hit it by accident17:33
bschaeferso really, do you want hitting show desktop to unhide all the windows?17:33
ryebschaefer: uh-huh, tracking windows and that starts to be messy17:34
bschaeferyeah, you can remove it from the switcher in CCSM->Unity Plugin17:34
* bschaefer removes it17:34
bregmais there a separate bug for the two-click-to-show-desktop ?  Ayatana design should have some input on whether it's a functionality bug or not.17:34
bschaeferyes there is..popy made it a while ago, let me go dig that up17:35
sil2100bregma: thanks for the branch!17:35
ryebregma: https://bugs.launchpad.net/unity/+bug/966030 ?17:36
ubot5Launchpad bug 966030 in Unity ""Show desktop" in alt-tab doesn't always work" [Medium,Confirmed]17:36
bschaeferyup17:36
* bschaefer assigns ayatana17:37
ryebregma: aha, it is not seen at all when a window is fullscreen - all other windows are fading in _behind_ (well, i don't know whether they actually fade in there)17:37
ryegood to know17:37
* rye marks as affected17:37
bschaeferrye, but if you only use it by accident in the switcher you can remove it :)17:38
ryebschaefer: I will then never know whether it is fixed :)17:39
bschaeferrye, you can subscribe your self to the bug report to get emails!17:39
bschaeferlovely emails!17:39
ryebschaefer: -ETOOMANYEMAILS17:39
bschaeferhaha, yeah, there are a lot of generated emails17:40
bregmaI currently have 1642 unread emails in my Unity Bugs folder17:40
bregmaI suspect I'm never going to get caught up17:40
* bschaefer has 4212...17:42
bschaeferI should attempt to go through those more17:42
bschaeferbut thats compiz/unity/nux bugs17:42
ryeor just browse these on launchpad17:43
ryeAlso, personal "THANK YOU!" to whoever implemented the new blur. It's just so fast I actually started browsing through  unity/nux branches to see what was that :)17:44
ryewell, to Nicolas but it does not look like he's here17:45
bregmait's already pub time in the UK17:50
didrockssil2100: bregma: was on a hangout, thanks you! I think it doesn't worth running a rebuild right now as next daily release will kick in soon enough once your unity branch will get merged17:53
sil2100\o/17:57
sil2100didrocks: once you have more time, could you take a look and push (if ok) the quantal branches for bamf and u-l-f? Thanks in advance ;)17:58
didrockssil2100: will probably be tomorrow morning, but will do :)17:58
didrocksyou have a tab opened! :)17:59
=== dandrader|afk is now known as dandrader
sil2100didrocks: :)18:15
rperierbug 1019457 fixed, I proposed a merge request (I can wait until tomorrow, there's no hurry)19:19
rperier;)19:19
ubot5bug 1019457 in unity (Ubuntu) "The Dash closes when trying to switch to the Command lens (Alt+F2)" [Medium,Confirmed] https://launchpad.net/bugs/101945719:19
bschaeferrperier, make sure you link your branch to that bug19:21
rperierbschaefer: I don't remember how to do that19:29
rperierlet me check19:29
bschaeferrperier, go to your branch here: https://code.launchpad.net/~rperier/unity/exec-len19:29
bschaeferand under Link Bug Report19:29
bschaeferput the # of the bug19:29
rperierI found !19:29
rperierthanks19:30
bschaeferrperier, np, and also one thing you'll need is a test with the fix :)19:30
rperierbug linked19:32
rperierbschaefer: a unit test or an interactive test ?19:32
bregmaall you should need to do is make sure the bug number is mentioned in the changelog or commit message (preferrably using the (lp: #NNNNN) format)19:32
bschaeferrperier, well from that bug, an interactive test, but always  try a unit test first19:33
rperierthe bug number will be concatenated to the commit message, as I linked the bug to the branch. no ?19:33
bschaeferrperier, so you'll have to take a loot at the autopilot stuff we have (sudo apt-get install unity-autopilot)19:34
rperierI can write a test case into an existing unit test, sure19:34
rperierbschaefer: looking19:35
bschaeferrperier, well I don't think any unit tests are set up for opening exec mode, then going to the dash and asserting that the dash doesn't close19:35
bschaeferso in autopilot you can just do "Open the Dash, Attempt to Open exec mode, Assert the dash is still open"19:38
rperiermhhh I see19:42
bschaeferhere is what a test could look like in autopilot: http://paste.ubuntu.com/1567001/19:43
rperierI can make better, "For all available len open the dash, attempt to open exec mode, assert the dash is still open"19:43
bschaeferrperier, which should test what you want (I haven't ran it)19:43
bschaeferrperier, really you just need to test that the dash it self can go from the Dash to the Command Lens with out exiting. Make sure you test that the test fails without your fix19:44
rperieroh the test is written in python19:44
bschaeferyup, look in unity/tests/autopilot/unity/tests/test_dash.py19:44
rperierI will write the test tomorrow, I have to go now19:45
rperierthanks for your help about the test19:45
bschaefernp!19:46
rperier;)19:46
bschaeferc ya, and thanks for the fix :)19:46
rperieryw :)19:46
=== salem_ is now known as _salem
chiluknot sure if this is the right place, but I love the new unity dash icon in raring that was pushed in recently.  Kudos to whoever did it.21:17
qenghoHi all.  I have a machine that unity is unusable on.  I want to help make it better.21:47
qenghoIt's up to date with raring right now.21:47
popeyqengho: what spec is the machine?21:48
seb128qengho, define "unusable"?21:48
qenghopopey: it's custom-built, somewhat beefy.  The interesting part is that the video card is Nvidia GeForce 7900 GS.  xserver-xorg-video-nouveau=1:1.0.6-0ubuntu121:51
popey7900 isn't what I'd call "beefy"21:51
popeyits a 6 year old card21:52
qenghoWell, not that, but the rest of the machine.21:52
qenghoStill, six years old should be far far more than enough.21:53
popeythe card matters though  ☺21:53
bschaeferqengho, well what happens when you attempt to start unity?21:54
qenghoJust started afresh.  First glitch is when dragging a window, I got some harsh flashes of an opaque triangle occupying the NW-SW-NE of the whole display.21:58
qenghoThat's hard to reproduce again. Not sure what triggered it.22:00
qenghoNext I can reproduce easily. I can point a webcam off this computer (not the one in question) at the display so you can see for yourself.  GOOG Hangout, anyone?22:02
bschaeferqengho, well if that is happening you should file a bug about it, along with a video or screenshot if you22:03
bschaeferqengho, also getting specs on your machine added to the bug report would be good, using apport-bug22:03
popeyqengho: have you reproduced the bug with the nvidia binary driver instead of nouveau?22:05
qenghopopey: no, I haven't tried nvidia's driver.22:06
popeyplease do. i have seen similar issues which only occur on the noueveau driver22:06
qenghopopey: Okay, I'll try. Is that what we want to tell people to do?22:08
popeyits what I'd do if I was experiencing what you are seeing22:08
qenghoI usually just keep nouveau driver and flip to gnome-shell, which doesn't trigger these problems.22:09
popeyGNOME shell probably isnt as demanding on the 3d card22:10
qenghoThat is probably true.22:10
qenghoCan I make unity less demanding, then?22:10
popeyqengho: if you're a developer you could look at the unity and compiz code and see where we can improve performance, or look at existing bugs in that area..22:23
qenghopopey: right, thanks.22:23
qenghoLast time I tried to debug my window manager, I got stuck. Couldn't get another "f" in the GDB console because the window didn't have focus.  :)22:25
popeyi tend to do that remotely via ssh22:26
qenghopopey: what do you do? Start in session, and then attach from outside?22:26
popeyyes22:26
bschaeferqengho, well there are few things you can disable...22:27
qenghoWhat's the equivalent of "unity --reset" now?  Move away some config directory?22:28
bschaeferunity22:28
bschaefer'unity'22:28
ControlsfreekI'm new to unity development. Is there documentation somewhere the explains how LauncherOptions and nux::Property are stored/loaded?22:41
bschaeferControlsfreek, nothing besides the source code :)22:43
Controlsfreekbschaefer, ha okay.22:43
bschaeferControlsfreek, nux::Property stores <Type> and when it gets changes emits a signal22:44
Controlsfreekbschaefer, yeah im with you so far. what causes it to change though? where/when does the load happen22:44
bschaefersay you have a Property a<bool> and a is set to false.22:44
bschaeferif you change a = true, then a signal called changed is emited22:45
bschaeferwhich you can point to function to do stuff based on that variable change22:45
Controlsfreekbschaefer, I follow you- I see how that happens with LauncherOptions. I'm going a step earlier in the process... assuming when Unity starts up, it loads configuration from disk. Cant figure out when that happens22:46
Controlsfreekwhat causes that to happen22:46
bschaeferControlsfreek, what causes the variable to change?22:46
bschaeferControlsfreek, it acts like a normal variable with side effects22:47
ControlsfreekControlsfreek, when is the configuration loaded from disk22:47
bschaeferhmm when Compiz loads unity as a plugin it goes through and does all of that...I would think22:47
bschaeferthough i haven't dug through that bit of code really, Trevinho could know more about those parts22:48
Controlsfreekbschaefer, Ah yes, its that process that I'm trying to learn. I'll look at it from that angle. thanks22:48
bschaeferControlsfreek, np, it takes a while to get the stack understood (like months)22:48
bschaeferso pretty much compiz loads unity as a plugin (just like any old plugin from compiz)22:49
Controlsfreekbschaefer, Okay, so i'm not the only one who's in over their head :-)22:49
bschaeferControlsfreek, yeeah, the stack gets complicated :)22:49
Controlsfreekbschaefer, no problem, i'll keep digging. Thanks for the nudge in the right direction.22:50
bschaeferControlsfreek, good luck, if you get stuck dont' hesitate to ping me :)22:51
Controlsfreekbschaefer, I'm trying to load the source in an IDE to make things a little easier on myself as far as learning. Do you guys develop in eclipse or something? or are you all hardcore text editor people22:51
bschaeferControlsfreek, hmm I just use vim22:51
bschaeferControlsfreek, others use Sublime 2, which i've heard good things about22:52
Controlsfreekbschaefer, ok. I have it compiling from the command line, so im basically just using the IDE to browse/search the code at the moment. Was hoping it would make debugging easier.22:53
bschaeferControlsfreek, I put this in my .bashrc which makes searching very easy22:53
bschaeferg(){22:53
bschaefer  grep -Hnri --color=always "$@"22:53
bschaefer}22:53
bschaeferwhich you can go to say unity/dash/ and do g ::Show *22:54
Controlsfreekbschaefer, so basically that spits the code out with keywords highlighted and such?22:55
bschaeferControlsfreek, yup :)22:55
Controlsfreeksweet. throw that one in the bag of tricks22:55
bschaeferControlsfreek, yeah, I use grep the most, and if you ever need to do use a debugger go to a tty and type unity --advanced-debug22:55
bschaeferwhich will bring you into gdb22:56
Controlsfreekbschaefer, excellent thanks for the tips23:00
bschaeferControlsfreek, np! Have fun :)23:00
Controlsfreekbschaefer, what is the main entry point for the plugin? Is it in nux or in unity?23:03
bschaeferControlsfreek, unity, in unityshell.cpp23:03
Controlsfreekbschaefer, k. thanks.23:04
bschaeferunity/plugin/unityshell/src/unityshell.cpp is the main workhorse of unity (also a huge file that is a bit crazy....)23:04
Controlsfreekbschaefer, okay, I'll do some homework on it. Hopefully I'll be able to squash a bug or two23:05
bschaeferControlsfreek, yeah, there are lots of bugs to fix :)23:06
Controlsfreekbschaefer, was looking at this one: https://bugs.launchpad.net/unity/+bug/1099815. Seems like its just a timing issue with these options loading after the launcher is drawn23:12
ubot5Launchpad bug 1099815 in unity (Ubuntu) "Launcher starts with default size then jumps to configured one on login" [Undecided,New]23:12
bschaeferControlsfreek, well if you look at the default size in Launcher.cpp, you'll see the width is set to 6523:13
bschaeferwhich that 65 gets changed in unityshell.cpp when the compiz options get loaded23:13
Controlsfreekbschaefer, yeah i saw the 65, but didnt see where the options were loading23:14
bschaeferControlsfreek, its in unityshell where it sets the options of each controller23:15
bschaeferunityshell.cpp line 315023:16
bschaeferits in a lambda function set to trigger when size_changed emits23:16
bschaeferControlsfreek, what really needs to happen, is we need to set the launcher width before its created23:18
bschaeferwhich it looks like size_changed is a nux signal for a nux::Area23:19
Controlsfreekbschaefer, I was thinking it was rendering the launcher before the icon_size was loaded from the settings23:21
bschaeferControlsfreek, well I don't think the launcher takes that into account at start up, which is the problem :)23:22
bschaeferbut I could be mistake, I haven't dug through all of that in a bit23:23
Controlsfreekbschaefer, It does seem to eventually catch it and then re-renders at the correct size.23:24
bschaeferControlsfreek, yeah, but it really seems if the width is set correctly on start up it wouldn't set it to the wrong size :)23:24
bschaeferControlsfreek, so in Launcher.cpp::Resize, you see a width in that function, add a print statement to see what value is there when you restart unity, I bet you'll see23:27
bschaeferthe icon_size being incorrect for 1 draw23:27
=== chiluk is now known as chiluk_away

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