=== mmrazik is now known as mmrazik|otp
tsdgeosTrevinho: mhr3: https://code.launchpad.net/~aacid/unity/make_it_compile/+merge/128642 ?07:52
mhr3ehm, how would the mergers push something that doesn't compile?07:56
mhr3didrocks, ^^?07:59
TrevinhoI'm asking myself the same...07:59
sil2100tsdgeos: duflu already submitted an MR for that ;)08:01
tsdgeossil2100: oh, didn't see it08:01
* tsdgeos cancels his08:02
tsdgeosoh wait08:02
tsdgeosTrevinho approved it08:02
Trevinhotsdgeos: I've rejected08:02
dufluSee? This is the right timezone08:02
tsdgeosso i win :D08:02
Trevinhoit now08:02
Trevinhoduflu: approved yours08:02
tsdgeoshope the merger doesn't get confused now08:02
tsdgeoslast time someoner rejected a merge that had been just approved it was merged anyway :D08:03
sil2100Just in case I linked tsdgeos branch to duflus bug as well08:03
Trevinhotsdgeos: so, remove your branch so it won't pull08:03
=== mmrazik|otp is now known as mmrazik
didrocksmhr3: that's the ping I did to mmrazik yesterday, telling that since they -j4, it seems the merger doesnt' wait for the jenkins result08:18
didrocksmhr3: watch your logs! :)08:18
mhr3didrocks, oh, yea i do remember that, didn't realize it actually caused wrong merge08:19
tsdgeosTrevinho: mhr3: approve https://code.launchpad.net/~aacid/unity/do_not_reuse_menus_on_order_change/+merge/128243 again now that the compile fix is in?08:50
Trevinhotsdgeos: done08:51
popeyTrevinho, can you open a guest session and then click the amazon link in the launcher? do you get a second icon appear, i.e. not matched to the amazon icon already there?08:55
Trevinhopopey: yes, I noticed that few days ago installing Q in a friend's PC...08:56
Trevinhopopey: but I think the problem is that the web-app gives unity a different-desktop than the one that is sticky08:56
Trevinhopopey: in fact removing the sticked one and using that, it works08:56
Trevinho(i.e. sticking the one opened)08:56
* popey files bug08:58
seb128Trevinho, what is the name for each? is that a bug in the launcher default config then?09:01
popeyyou end up with /usr/share/applications/ubuntu-amazon-default.desktop and ~/.local/share/applications/wwwamazoncoukwwwamazoncouk.desktop09:01
Trevinhoseb128: yes, I think that09:02
Trevinhoalso in the migration script maybe (need to check)... Another patch I sent to the migration script is still approved but not merged tough...09:02
Trevinhois that using the merger?09:03
seb128Trevinho, not sure, is that upstream or in the packaging? the packaging doesn't use the merger09:03
Trevinhoseb128: upstream09:05
seb128so I guess the merger should handle it...09:05
popeybug 106426009:13
ubot5Launchpad bug 1064260 in libunity-webapps (Ubuntu) "Launching Amazon webapp results in two Amazon icons" [Undecided,New] https://launchpad.net/bugs/106426009:13
popeyTrevinho / om26er could you confirm pls09:13
Trevinhopopey: done09:14
om26eryeah I was just looking at bug 1061973 which may have the same cause09:15
ubot5Launchpad bug 1061973 in unity (Ubuntu) "double listing of menu choices in Applications" [Undecided,Confirmed] https://launchpad.net/bugs/106197309:15
ricotzTrevinho, hi, is something holding back https://code.launchpad.net/~ricotz/bamf/annotation-fixes/+merge/128544 ?09:15
Trevinhoricotz: hi... just forgot to approve, sorry09:16
Trevinhoricotz: done now09:16
ricotzTrevinho, btw, make distcheck it pretty happy here now09:16
Trevinhoricotz: coooool09:16
ricotzTrevinho, i was trying to be as less invasive as possible, but the makefiles still really need a clean up09:18
Trevinhoricotz: be invasive as you need in the daemon... don't break too much the lib though :)09:19
ricotzTrevinho, this is better be done in two merges, the distcheck fix and a "no-change" style fix09:21
ricotzTrevinho, i am curious is the marshaller really needed while having g_cclosure_marshal_generic since 2.3009:22
=== mmrazik is now known as mmrazik|lunch
Trevinhoricotz: probably it can ben removed, but I didn't look at that yet09:33
ricotzTrevinho, https://code.launchpad.net/~ricotz/bamf/fix-distcheck/+merge/12866109:37
tsdgeosTrevinho: https://code.launchpad.net/~aacid/unity/do_not_reuse_menus_on_order_change_for_6/+merge/128663 ready, i'm working on thr 5.0 one now09:50
Trevinhotsdgeos: approved10:03
tsdgeos5.0 will take a while10:04
tsdgeosi just destroyed my precise VM10:04
tsdgeosso i'm reinstalling now10:04
=== mmrazik|lunch is now known as mmrazik
=== MacSlow is now known as MacSlow|lunch
=== mmrazik is now known as mmrazik|afk
tsdgeosTrevinho: and the 5.0 one https://code.launchpad.net/~aacid/unity/do_not_reuse_menus_on_order_change_for_5/+merge/12869211:55
tsdgeosand now foood11:55
didrocksMirv: do you have a minute for a compiz test?12:03
=== MacSlow|lunch is now known as MacSlow
Mirvdidrocks: should have12:11
didrocksMirv: so can you try merging https://code.launchpad.net/~vanvugt/ubuntu/quantal/compiz/fix-1039155/+merge/128684, and getting /usr/lib/nux/unity_support_test returning 1?12:11
didrocksMirv: this should fall you back with llvmpipe12:12
didrocks(just a dummy script returning 1 and replacing /usr/lib/nux/unity_support_test should work)12:12
didrocksMirv: thanks! :)12:25
=== mmrazik|afk is now known as mmrazik
=== _salem is now known as salem_
Mirvdidrocks: confirming, if 1 is returned low graphics + llvmpipe compiz starts, if 0 (ie. normal unity_support_test in my case) then normal function12:45
didrocksMirv: excellent! thanks a lot for testing :)12:46
didrocksMirv: I think we just wait for duflu to fix the env var and we can shelve that in SRU012:47
Mirvdidrocks: sounds good12:47
didrocksMirv: actually, I'll try to get it on the CD, one sec :)12:50
=== MacSlow is now known as MacSlow|errand
=== MacSlow|errand is now known as MacSlow
=== dandrader is now known as dandrader|afk
=== dandrader|afk is now known as dandrader
=== dandrader is now known as dandrader|afk
ricotzTrevinho, did you had a  chance yet to test the distcheck branch?15:16
=== dandrader|afk is now known as dandrader
thamIs there some way to make the Unity Launcher display apps that are in the current workspace only?16:34
thamIt gets cluttered after a while16:35
davidcalletham, no, the option is only available for alt+tab.16:39
thamdavidcalle: oh, too bad. This was not so with classic gnome :/ THanks anyway16:42
thamthis may be stupid, but is it possible to use unity except the launcher and use the classic gnome's taskbar?16:42
davidcalletham, yw. But I'm pretty sure there is a bug open for this, you should add yourself to it to follow its status.16:43
thamI make use of all four workspaces and Unity launcher is really cluttered16:43
thamdavidcalle, Hmm, I'll find it and follow16:43
davidcalletham, I *think* the folding effect is going to be worked on a bit next cycle, which could partially solve your issue.16:44
thamdavidcalle: good to hear.16:47
=== dandrader is now known as dandrader|lunch
MCR1bschaefer: Hi :) Do you have a minute ?17:44
bschaeferMCR1, hello, and yes I do17:44
MCR1I need your approval once again :)17:44
MCR1bschaefer: https://code.launchpad.net/~mc-return/unity/unity.merge-fix1006429-fix1006434-fix1063171/+merge/12874517:45
bschaeferMCR1, sweet, which one. I see 2 MPs by you17:45
* bschaefer looks17:45
MCR1bschaefer: The problem is that Unity has its own code regarding the "Show Desktop" function, but there are also 2 additional plug-ins providing the same feature17:46
bschaeferhmm which one is better? Im guessing the unity stuff needs to be there17:47
MCR1bschaefer: Ofc those create problems once activated in CCSM.17:47
MCR1The best one is the "Show Desktop" plug-in, but for several reasons smspillaz had to integrate a 3rd show-desktop functionality and code to Unity17:48
MCR1so once Unity is active those 2 CCSM plug-ins should be disabled17:48
MCR1but nothing is preventing the user from enabling them in CCSM17:48
MCR1which then leads to several different bugs17:49
MCR1(linked to the merge request)17:49
bschaeferso this xml feature thing you added will warn when the user tries?17:49
bschaeferto set it17:49
MCR1So now there will be a warning displayed in CCSM (like for other plug-ins) - yes17:50
bschaeferawesome, Ill test it out17:50
MCR1It will provide the ability to choose for the user17:50
bschaeferooo so it will say there is a conflict with activating this plug in blah blah blah17:51
MCR1So either "Show Desktop/Fade to Desktop" or "Unity" and its built-in showdesktop functionality, but both will not be allowed...17:51
MCR1like you cannot enable "Show Desktop" and "Fade to Desktop" at the same time17:51
bschaefercool, I guess im missing those plugin ...17:52
* bschaefer goes to install them17:52
MCR1maybe those are in some extra package, but they are in trunk and active17:52
MCR1and working (but only without Unity) and with Unity they cause troubles only...17:53
bschaeferhmm I just got a crash and no warning when activating it...17:53
* bschaefer double checks he didn't do anything stupid17:53
bschaeferMCR1, duh, I have to remove the file so it will re gen the code17:54
MCR1ah, maybe - it should not crash17:54
bschaeferit shouldn't, but well see if it crashes for me again, if it does then umm ill get a backtrace and see whats up17:54
* bschaefer has to re build unity....17:56
MCR1Actually the best version (most visually attractive) is the one from the "Show Desktop" plug-in, because there you can make the windows slide to the edges of the screen to reveal the desktop, but Unity got its own fading to desktop code - IIRC it was because of the integration to the switcher, but smspillaz knows more... ;)17:57
MCR1so there is a reason why we had to integrate the showdesktop functionality to Unity itself and the Compiz plug-ins cannot be used in the case Unity runs...18:00
MCR1bschaefer: I'm sorry if I am causing troubles :-D18:02
bschaeferoo no, im just recompiling cause I forgot to remove all this stuff from a compiz install18:02
bschaeferand was missing some env var18:02
* bschaefer has a broken desktop atm 18:03
bschaeferim recompiling, ill be back in a few minutes18:03
MCR1sure, no stress18:03
bschaeferthere better not be any!18:03
bschaeferMCR1, finally, and awesome that change looks good18:25
bschaeferMCR1, but i suppose im worried it Show Desktop should be completely incompatible with the unity pluging...18:25
bschaeferMCR1, I also think that bug needs to be reviewed/discussed as a bunch of people see to want an opinion on it18:28
MCR1bschaefer: The only problem I can see here is that the user might be able to turn off Unity if he does not read carefully18:29
MCR1but that problem cannot be eliminated (and should not be)18:29
MCR1for users using CCSM18:30
MCR1but the user should be prevented from enabling a second showdesktop feature, because this just causes problems18:30
bschaeferMCR1, yes, but at the same time when I see Daniel mark it for an opinion i would like to hear more of a discussion with smspillaz and him18:30
MCR1and the user is not even informed why (it took me some time to figure out)18:31
bschaeferyes, but those problems could be fixed as opposed to axing the plugin18:31
MCR1The plug-in is not axed and the discussion with duflu was before I found a correct solution18:32
bschaeferwell the show desktop plugin will no work with the unity plugin with this fix18:32
MCR1With my new solution the plug-ins still are available and act like they should and others do already18:32
MCR1It did not work before either, see my bug reports18:32
MCR1I linked them to the branch merge request18:33
* bschaefer looks18:33
MCR1Try to enable Show Desktop and set it to slide the windows to the edges of the screen18:33
MCR1then activate it18:34
MCR1with Unity it does not work18:34
MCR1without Unity or with Unity-2d it will be functional18:34
bschaeferwell I can't now because it says Don't enable Show Desktop or Disable Unity18:34
MCR1yeah, my fix ofc prevents that now18:35
bschaeferMCR1, I would prefer to hear what Daniel says or smspillaz about it18:35
bschaeferMCR1, Ill talk to one of them later18:35
MCR1bschaefer: Yeah sure, but I hope you read my reports and simply try to reproduce those problems I encountered18:36
bschaeferMCR1, and that bug you are talking about to slide the window to the edge of the screen could possibly be fixed18:36
bschaefervs having a dialog that says choose one plug in or the other18:36
MCR1bschaefer: No it cannot, because Unity has its own code18:36
bschaeferwhich is why I would like to talk to one of them :)18:36
MCR1it is in UnityShowDesktopHandler - wait I'll tell you exactly18:37
MCR1Unity/plugins/UnityShowdesktopHandler.cpp + .h18:38
bschaeferMCR1, hmm yeah. Well it should be an easy answer for one of them then :)18:38
MCR1So neither "Fade to Desktop" nor "Show Desktop" work if unityshell runs - that is why they are not even included in the main package18:39
bschaeferMCR1, yes, I just want to double check with one of them before a merge :), I do agree with your branch though18:39
bschaeferbecause Im wondering why they wouldn't have done it them selfs18:40
bschaeferwhich makes me think I might be overlooking something18:40
MCR1if the warning and dialogue are displayed it works like it should, but ofc I agree on smspillaz or duflu also looking at it :)18:41
MCR1Thanks 4 your time.18:41
bschaeferMCR1, np! you'll see some sort of review by tomorrow :) and thank you!18:41
MCR1I just want to help eliminating frustration when using CCSM to configure Compiz/Unity to ones liking18:43
bschaeferI agree, but just to be safe I would prefer sam or duflu to take a quick look since we are pretty close to 12.10 :)18:45
=== dandrader|lunch is now known as dandrader
=== dandrader is now known as dandrader|afk
=== salem_ is now known as _salem
=== dandrader|afk is now known as dandrader

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