| Andy80 | hi guys :) | 00:15 |
|---|---|---|
| === dragonfyre13 is now known as dragonfyre_away | ||
| didrocks | good morning | 05:46 |
| nhaines | didrocks: good morning! | 05:49 |
| didrocks | hey nhaines | 05:52 |
| === API is now known as Guest32644 | ||
| MacSlow | hey folks | 07:29 |
| === kim0|holiday is now known as kim0 | ||
| didrocks | kamstrup: hey, do you have a second for playing with some cjk foo? | 08:25 |
| didrocks | in xapian (apt-xapian index) | 08:25 |
| kamstrup | didrocks: give me 15 mins | 08:26 |
| === Guest32644 is now known as apinheiro | ||
| htorque | njpatel: it's likely not one of those indicators: https://launchpadlibrarian.net/77655516/log.png | 09:30 |
| htorque | the jump at the beginning was caused by opening some programs iirc | 09:30 |
| njpatel | htorque, ah, excellent | 09:30 |
| njpatel | htorque, I'll try and take a look today | 09:30 |
| htorque | but i see that the appmenu is missing, so that's something left to test | 09:30 |
| htorque | njpatel: however, i don't quite understand your directions: "probably from metacity, without an exsiting global menu" - should i run metacity --replace and run the command? with valgrind? or just look at mem consumption? | 09:32 |
| njpatel | htorque, you can initially just look at memory consumption of the indicator-loader with appmenu, to confirm/deny if it's leaking when you start an app | 09:33 |
| njpatel | htorque, valgrind will give you a deeper analysis if it is, I guess | 09:33 |
| htorque | but i should replace compiz & co with metacity, right? | 09:33 |
| njpatel | htorque, the reason for running it in metacity is that you can't have two appmenus running, so you can't run in unity | 09:34 |
| njpatel | htorque, right, exactly | 09:34 |
| njpatel | just metacity --replace from a terminal will do | 09:34 |
| htorque | so now i don't have a panel at all - is it still useful then? | 09:35 |
| njpatel | yep | 09:35 |
| njpatel | htorque, with indicator-loader, you're just testing a single indicator at a time, which is good to isolate the issue | 09:36 |
| njpatel | if it doesnt' leak, most likely something is wrong in the service itself, or maybe another indicator | 09:36 |
| didrocks | kamstrup: I didn't touch that ;) | 10:11 |
| kamstrup | didrocks: in a nutshell what I mean: python -c "import os; os.environ['FOOBAR'] = 1; print os.environ['FOOBAR'] == 1" | 10:11 |
| apinheiro | njpatel, ok thanks, although he is not here ... | 10:11 |
| apinheiro | just in case | 10:11 |
| kamstrup | throws an error because you can only pass strings into environ[] | 10:11 |
| njpatel | apinheiro, yeah, he's in US so her ein a few hours | 10:11 |
| apinheiro | njpatel, these messages from panel-service: | 10:11 |
| apinheiro | (unity-panel-service:5672): LIBDBUSMENU-GTK-WARNING **: Child is realized, but doesn't have a GTK Widget! | 10:11 |
| apinheiro | (unity-panel-service:5672): LIBDBUSMENU-GTK-WARNING **: Got new menuitem signal, which means they want something | 10:11 |
| apinheiro | (unity-panel-service:5672): LIBDBUSMENU-GTK-WARNING **: that I simply don't have | 10:11 |
| apinheiro | do you know if those are normal messages | 10:12 |
| njpatel | no idea | 10:12 |
| njpatel | I actually have the same question for him | 10:12 |
| kamstrup | didrocks: and if you change 1 to '1' you get False: python -c "import os; os.environ['FOOBAR'] = '1'; print os.environ['FOOBAR'] == 1" | 10:12 |
| didrocks | kamstrup: oh, I clearly know that, I'm not the one making the patch once again :-) | 10:12 |
| apinheiro | njpatel, ok, is because I'm working on that panel-service crash with a11y enabled, and that "he doesn't have a GTK widget" sounds something to look at | 10:14 |
| kamstrup | didrocks: the reason I bang on about this is that afaics this means the db is reindexed on *every* run | 10:14 |
| kamstrup | didrocks: is the cjk patch applied to xapian by default now in Oneiric? | 10:15 |
| didrocks | kamstrup: yeah, and the testsuite doesn't pass anyway, so I think nobody tested it | 10:15 |
| didrocks | kamstrup: yes, it is :) | 10:15 |
| kamstrup | didrocks: how can that possibly go wrong?!?!?!11 ;-) | 10:15 |
| didrocks | kamstrup: touching data! never be afraid :-) | 10:16 |
| njpatel | apinheiro, heh, yeah :) | 10:16 |
| kamstrup | didrocks: i'll let you touch my data | 10:16 |
| didrocks | kamstrup: that's so kind! I let you spy my computer with zg already, so, in some kind of mutual touch ;) | 10:17 |
| kamstrup | didrocks: kinky | 10:17 |
| htorque | njpatel: the amount of leakage is definitely influenced by the menu (eg. with 180 simple menu items in the menubar, u-p-s mem consumption raises by 15mb, with one it's just a couple of kb) | 10:31 |
| njpatel | htorque, right, the reason why I thought it must be appmenu is because the panel service only extracts info about the toplevel menu items (it has no idea about the actual menus), and hence I felt that appmenu is leaking something in constructing the actual menu tree | 10:32 |
| htorque | good, because putting more items in a menu didn't change anything | 10:32 |
| kamstrup | didrocks: attached a fixed patch to https://bugs.launchpad.net/ubuntu/+source/apt-xapian-index/+bug/838274 | 10:50 |
| ubot5 | Ubuntu bug 838274 in apt-xapian-index (Ubuntu) "The indexer should use the new CJK tokenizer" [High,Fix committed] | 10:50 |
| didrocks | kamstrup: did yo utest it, you should get a missing "import os" | 10:51 |
| didrocks | in update-apt-xapian-index | 10:51 |
| didrocks | kamstrup: tell me if the testsuite still run after that :) | 10:52 |
| kamstrup | didrocks: ah, wait... | 10:52 |
| kamstrup | didrocks: needs some more fixing I can see :-) | 10:57 |
| didrocks | kamstrup: indeed :) | 10:57 |
| kamstrup | didrocks: ok v3 of the patch is running with all tests - I even added 3 new test cases so the diff has grown somewhat. But this should be upstreamable now i think | 11:51 |
| didrocks | kamstrup: excellent news, do you think it's that part which is used by software-center or we should force a reindexing as well before reopening the database as well? | 11:52 |
| kamstrup | didrocks: i don't have the faintest clue what S-C is doing :-) | 11:53 |
| kamstrup | didrocks: S-C has it's own index in addition to the axi one | 11:54 |
| kamstrup | so S-C prolly needs patching in a similar spirit | 11:54 |
| didrocks | kamstrup: ah, because I was thinking that it was using the axi one | 11:54 |
| didrocks | so we could have activate this env var unconditionnaly | 11:54 |
| kamstrup | didrocks: it is using 2 indexes | 11:54 |
| kamstrup | i think we should, but still, the code needs to work regardless as other distros might not set it | 11:55 |
| kamstrup | (mostly thinking about the zg bits here though) | 11:55 |
| didrocks | kamstrup: indeed | 11:55 |
| kamstrup | didrocks: is upstream watching these bugs, or should we ping someone? | 11:56 |
| didrocks | kamstrup: I think we should ping them, but I won't block on that for now | 11:57 |
| kamstrup | didrocks: i'll move on to the zg-fts bits cjk enablement now, is mvo on top of the S-C issue or should I take that later? | 11:59 |
| didrocks | kamstrup: mvo is on vacation, not sure if you have the time to look at it, otherwise tremolux will (but he doesn't really have a clue as well) | 11:59 |
| kamstrup | hehe :-) | 12:00 |
| kamstrup | i'll see if I get around to it | 12:00 |
| === MacSlow is now known as MacSlow|lunch | ||
| om26er | JohnLea, Hi could you confirm if bug 838759 is a design decision or a bug? | 12:01 |
| ubot5 | Launchpad bug 838759 in unity "window title should not appear until cursor leaves the menubar" [Undecided,New] https://launchpad.net/bugs/838759 | 12:01 |
| didrocks | kamstrup: great ;) | 12:02 |
| Trevinho | om26er: I know it's strange but it seems something the design want... No menu unitil you move again! :o | 12:02 |
| om26er | oh twitter people are saying its by design :/ andyrock Trevinho | 12:02 |
| Trevinho | yes.. | 12:03 |
| om26er | Trevinho, :( | 12:03 |
| andyrock | om26er, have you said my pastebin? | 12:03 |
| andyrock | it should be easy to fix but the comment let me think that it's by design | 12:03 |
| om26er | andyrock, the window button problem stands, doesnt it? | 12:04 |
| om26er | on second click on the menus window buttons dont show | 12:04 |
| om26er | unless cursor is moved | 12:04 |
| Trevinho | om26er: are you still confirming the bug of the VP switch (move to another VP and select the launcher button of the focused window in another vp => Nothing happens after the click) ? | 12:05 |
| Trevinho | If you do please report, I can confirm :P | 12:05 |
| Trevinho | (and work on it) | 12:05 |
| om26er | Trevinho, i confirmed it yesterday, i am reporting it now :) | 12:06 |
| Trevinho | thanks :) | 12:06 |
| Trevinho | since DBO can't reproduce it... | 12:06 |
| andyrock | om26er, which window button problem? | 12:06 |
| Trevinho | om26er: check these issues using my branch please, I guess I've solved them... | 12:07 |
| om26er | andyrock, see in the video, when i click on the menu again window controls dont appear the faded title does | 12:07 |
| Trevinho | om26er: Ah, That's true | 12:07 |
| Trevinho | I'll fix that. | 12:07 |
| om26er | Trevinho, i'll report that and assign you as well ;) | 12:08 |
| Trevinho | ok | 12:08 |
| om26er | launcher stuck hard this time, it wont hide :/ | 12:08 |
| Trevinho | These panel issues are really annoying | 12:08 |
| andyrock | om26er, wanted behavior too? | 12:08 |
| Trevinho | but I guess we're in the right way... | 12:08 |
| andyrock | i mean the window buttons appear only when the menu appears... | 12:09 |
| Trevinho | andyrock: it could be... But it's better to have a reply by JohnLea (or design, in general) about these things. | 12:09 |
| andyrock | so if the menu is not shown why we should show the window buttons? | 12:09 |
| andyrock | Trevinho, mpt too (I guess) | 12:10 |
| Trevinho | of course... | 12:10 |
| Trevinho | no, andyrock... The problem is on the second click | 12:11 |
| Trevinho | On the second click only the menus are shown | 12:11 |
| Trevinho | and this is not so consistent | 12:11 |
| andyrock | ah i got it... | 12:11 |
| Trevinho | also if that is enough (as you reclick without moving, we guess you want to see the menu) | 12:11 |
| andyrock | Trevinho, btw i guess that design info are needed | 12:12 |
| andyrock | *is | 12:13 |
| JohnLea | Trevinho; andyrock; just about to grab lunch, will get back to you in 30min | 12:36 |
| andyrock | JohnLea, np | 12:36 |
| JohnLea | om26er, Trevinho; re. bug #838759, good catch! Yes the current behaviour wrong and needs fixing. I've confirmed the bug | 13:03 |
| ubot5 | Launchpad bug 838759 in unity "window title should not appear until cursor leaves the menubar" [Undecided,New] https://launchpad.net/bugs/838759 | 13:03 |
| JohnLea | om26er, Trevinho; any idea how people got the idea that bug 838759 behaviour was by design? !! :/ | 13:07 |
| ubot5 | Launchpad bug 838759 in unity "window title should not appear until cursor leaves the menubar" [Undecided,New] https://launchpad.net/bugs/838759 | 13:07 |
| Trevinho | JohnLea: this is the code that implement that http://pastebin.ubuntu.com/679655/ | 13:16 |
| Trevinho | I also found a bug somewhere mentioning it | 13:16 |
| JohnLea | Trevinho; cool, I've confirmed bug https://bugs.launchpad.net/unity/+bug/838759 if you could fix it that would be great ;-) | 13:18 |
| ubot5 | Ubuntu bug 838759 in unity "window title should not appear until cursor leaves the menubar" [Undecided,New] | 13:18 |
| Trevinho | ok JohnLea I'll work on it | 13:21 |
| Trevinho | JohnLea: and what about the other thing we discussed before? (as you can see in https://launchpadlibrarian.net/78696225/5ii.ogg on second click no window buttons are shown) | 13:24 |
| JohnLea | Trevinho; yes that's a bug, would that not be fixed with the solution om26er outlined in bug #838759? | 13:34 |
| ubot5 | Launchpad bug 838759 in unity "window title should not appear until cursor leaves the menubar" [Undecided,New] https://launchpad.net/bugs/838759 | 13:34 |
| Trevinho | I guess it will JohnLea. I've to check btw | 13:35 |
| Trevinho | JohnLea: however about the #838759 I'm pretty sure that I've read about that behaviour in an unity bug (maybe related to ayatana design too).... | 13:36 |
| Trevinho | Otherwise I would have fixed for a long time :D | 13:36 |
| JohnLea | Trevinho; it is definitely a bug ;-) If any other items like this come up please ping me and I'll try to help clarify | 13:38 |
| Trevinho | No problem... However I'm happy it's a bug :P I didn't like that behavior! | 13:48 |
| Andy80 | hi guys :) | 13:51 |
| Andy80 | Kaleo: I'll have to upgrade my vm to Oneiric before being able to work to that bug ;) doing right now | 13:52 |
| Kaleo | Andy80: good luck ;) | 13:55 |
| === MacSlow|lunch is now known as MacSlow | ||
| apinheiro | tedg, hi, you here? | 14:05 |
| tedg | apinheiro, Yes, but I'm testing something where my machine might shutdown (or I get a good backtrace)... give me a second. | 14:06 |
| === smstudy is now known as smspillaz | ||
| Andy80 | apinheiro: the one who work at Igalia? | 14:18 |
| apinheiro | Andy80, yes | 14:19 |
| apinheiro | I usually use API as my IRC nickname, but someone already registered it | 14:20 |
| Andy80 | apinheiro: hi :) don't know if you remember me.... I did a summer internship 2 years ago in your company :) que tal? | 14:20 |
| apinheiro | tedg, btw, my previous question, Im trying to fix that crash on panel-service when a11y is on, and I have these messages: | 14:21 |
| apinheiro | <apinheiro> (unity-panel-service:5672): LIBDBUSMENU-GTK-WARNING **: Child is realized, but doesn't have a GTK Widget! | 14:21 |
| apinheiro | <apinheiro> (unity-panel-service:5672): LIBDBUSMENU-GTK-WARNING **: Got new menuitem signal, which means they want something | 14:21 |
| apinheiro | <apinheiro> (unity-panel-service:5672): LIBDBUSMENU-GTK-WARNING **: that I simply don't have | 14:21 |
| apinheiro | it is normal? | 14:21 |
| apinheiro | Andy80, hi, yes I remember you ;), fine thanks | 14:21 |
| tedg | apinheiro, Uhm, I don't think that those should be happening. Do they happen without a11y? | 14:22 |
| apinheiro | well, I asked that in the morning | 14:24 |
| apinheiro | and | 14:24 |
| * apinheiro looking backlog | 14:24 | |
| Andy80 | andyrock: I'm upgrading my VM to Oneiric. I will need your help with that Nautilus-dbus stuff later if you can | 14:24 |
| andyrock | of course | 14:25 |
| apinheiro | <njpatel> apinheiro, yeah, he's in US so her ein a few hours | 14:25 |
| apinheiro | <apinheiro> njpatel, these messages from panel-service: | 14:25 |
| apinheiro | <apinheiro> (unity-panel-service:5672): LIBDBUSMENU-GTK-WARNING **: Child is realized, but doesn't have a GTK Widget! | 14:25 |
| apinheiro | <apinheiro> (unity-panel-service:5672): LIBDBUSMENU-GTK-WARNING **: Got new menuitem signal, which means they want something | 14:25 |
| apinheiro | <apinheiro> (unity-panel-service:5672): LIBDBUSMENU-GTK-WARNING **: that I simply don't have | 14:25 |
| apinheiro | <apinheiro> do you know if those are normal messages | 14:25 |
| apinheiro | <njpatel> no idea | 14:25 |
| apinheiro | <njpatel> I actually have the same question for him | 14:25 |
| apinheiro | tedg, ^ | 14:25 |
| apinheiro | tedg, I also see messages like: | 14:25 |
| apinheiro | "IndicatorObject class does not have an accessible description." | 14:25 |
| andyrock | Andy80, btw Kaleo told me that in QT there is already a dbus helper class... | 14:26 |
| apinheiro | and finally, I have a question about indicators, although not sure if they are related with dbusmenu | 14:26 |
| apinheiro | on indicator.h: | 14:26 |
| apinheiro | @get_menu: Gets the image for this object. Should be set | 14:26 |
| apinheiro | to #NULL if @get_entries is set. Should NOT ref the | 14:26 |
| apinheiro | object. | 14:26 |
| tedg | apinheiro, The not having an accessible description is just to make us do the work, though it shouldn't harm anything. | 14:26 |
| andyrock | so i think you should use it instead of UnityCore::DBusWrapper | 14:27 |
| Andy80 | andyrock: I know that class, but I don't need help for using the QDbus class, I need to know which parameters to give it, which strings to use ecc.... | 14:27 |
| apinheiro | tedg, ok, anyway I'm just setting "" in this case | 14:27 |
| apinheiro | but about that doc | 14:27 |
| apinheiro | do you know what would happen if you ref that menu? | 14:27 |
| andyrock | Andy80, no problem then :) | 14:27 |
| tedg | apinheiro, Memory leak | 14:27 |
| Andy80 | andyrock: thanks :) | 14:27 |
| apinheiro | tedg, why, shouldn't the indicator be freed on the last unref? | 14:28 |
| tedg | apinheiro, But, in general, it should be "does not" instead of "should not" | 14:28 |
| andyrock | Andy80, yw... | 14:28 |
| tedg | apinheiro, The indicator should, but it's saying that by calling "get_menu" you are not incrementing the ref count. | 14:28 |
| apinheiro | tedg, ah ok | 14:28 |
| apinheiro | but you could increment the ref count | 14:28 |
| apinheiro | right? | 14:28 |
| apinheiro | I mean to ensure that the object will be alive while you are working with it | 14:29 |
| tedg | apinheiro, Yes, and you should if you're working with it. Just some of the callers just pass it to GTK which does it's own increment, so I didn't want to have one stuck in the middle. | 14:29 |
| tedg | apinheiro, This way users can treat it more like a floating count. | 14:30 |
| apinheiro | tedg, anyway don't worry, just asking, on the a11y side there are a weak reaf, so if the object is freed, the atk object should be notified | 14:30 |
| apinheiro | tedg, so going back to my "weird messages questions" | 14:30 |
| apinheiro | do you know if I'm doing something wrong? | 14:30 |
| apinheiro | I mean if there are something missing in my system? | 14:30 |
| tedg | apinheiro, No, it's basically saying that it thinks the menu item should have a GTK Menu Item associated with it, but it doesn't. | 14:31 |
| tedg | apinheiro, It could be a sign of some sort of race condition. | 14:31 |
| tedg | apinheiro, I'd be curious if the a11y slows down the construction of the GTK+ widgets in some way that we're not taking into account. | 14:31 |
| apinheiro | working on a bug and found a race condition ... this can be tricky | 14:32 |
| apinheiro | hmm | 14:32 |
| apinheiro | tedg, well, in that case I will disable a11y and check it | 14:32 |
| apinheiro | but anyway, njpatel said that he also wanted to ask | 14:32 |
| tedg | apinheiro, Yeah, that'd be best, just to see. | 14:32 |
| apinheiro | so I suppose that this also happens without a11y enabled | 14:32 |
| tedg | apinheiro, Yeah, njpatel is like that ;-) | 14:32 |
| njpatel | I like to ask things | 14:33 |
| htorque | tedg: njpatel asked me to run indicator-loader to check memory consumption of an indicator, but that seemed to fail and i think i got the very same warnings | 14:33 |
| htorque | could this be connected? | 14:33 |
| * tedg keeps njpatel's secret that he's really 5 years old. | 14:33 | |
| njpatel | htorque, UBUNTU_MENU_PROXY="" before using indicator-loader apparently! | 14:33 |
| njpatel | tedg, but WHY? | 14:33 |
| htorque | tried that | 14:33 |
| tedg | htorque, Getting the same messages or no indicator at all? | 14:33 |
| htorque | the indicator-loader just showed "File" (appmenu loaded) | 14:34 |
| apinheiro | njpatel, we were talking about this: | 14:34 |
| apinheiro | <njpatel> apinheiro, yeah, he's in US so her ein a few hours | 14:34 |
| apinheiro | <apinheiro> njpatel, these messages from panel-service: | 14:34 |
| apinheiro | <apinheiro> (unity-panel-service:5672): LIBDBUSMENU-GTK-WARNING **: Child is realized, but doesn't have a GTK Widget! | 14:34 |
| apinheiro | <apinheiro> (unity-panel-service:5672): LIBDBUSMENU-GTK-WARNING **: Got new menuitem signal, which means they want something | 14:34 |
| apinheiro | <apinheiro> (unity-panel-service:5672): LIBDBUSMENU-GTK-WARNING **: that I simply don't have | 14:34 |
| apinheiro | <apinheiro> do you know if those are normal messages | 14:34 |
| tedg | htorque, Did you set UBUNTU_MENUPROXY= or the wrong env like njpatel said ;-) | 14:34 |
| apinheiro | <njpatel> no idea | 14:34 |
| htorque | yeah, thanks apinheiro | 14:34 |
| apinheiro | <njpatel> I actually have the same question for him | 14:34 |
| apinheiro | and your "I actually have the same question for him" | 14:34 |
| htorque | tedg: the right one ;) | 14:35 |
| tedg | htorque, Hmm, that works for me... | 14:35 |
| tedg | htorque, Which indicator? | 14:35 |
| htorque | i posted the output of indicator-loader here but that did get logged and i'm on a different machine now :( | 14:35 |
| htorque | appmenu | 14:35 |
| apinheiro | btw, when I get the panel-service working without a crash | 14:36 |
| apinheiro | I can open the menus with the mouse pointer | 14:36 |
| apinheiro | but I'm not able to navigate or give the focus with F10 | 14:36 |
| apinheiro | is this only happening to me? | 14:37 |
| apinheiro | njpatel, htorque ? ^ | 14:37 |
| htorque | F10 doesn't do anything here | 14:37 |
| apinheiro | ?¿ | 14:38 |
| apinheiro | F10 is supposed to give the focus to the panel | 14:38 |
| apinheiro | have this changed? | 14:38 |
| apinheiro | is not related to those memory comsuption issues, but just asking as is related with the panel | 14:39 |
| htorque | when opened with the mouse i can navigate within the toplevel menu, but i cannot switch left and right | 14:39 |
| apinheiro | somewhat offtopic I know | 14:39 |
| apinheiro | htorque, urgh | 14:39 |
| apinheiro | that confirms that panel key nav is broken :/ | 14:39 |
| tedg | htorque, Hmm, I'm not sure that appmenu works in the loader in general, since it's based on which window has focus. When the loader has focus, it tries to show menus from it. So you can't really click on another window and interact with its menus. | 14:39 |
| tedg | It unfortunately makes appmenu ridiculously hard to debug. | 14:40 |
| htorque | tedg it was just to confirm whether it's causing bug 835646 | 14:40 |
| ubot5 | Launchpad bug 835646 in unity (Ubuntu) "unity-panel-service memory leak when starting various applications" [High,Confirmed] https://launchpad.net/bugs/835646 | 14:40 |
| tedg | We should probably add a delayed mode for using the loader. We could use the loading hints to determine whether it should be on or not. | 14:40 |
| tedg | njpatel, Did you ever include a mode in unity-panel-service where you could specify which indicators it would load? | 14:41 |
| htorque | apinheiro: i think it's not broken per se, but rather that the mouse pointer keeps the selection on the toplevel menu it's pointing to - with F10 working, this wouldn't be a problem i guess | 14:42 |
| njpatel | tedg, I did, but it seems like someone removed it :( | 14:43 |
| apinheiro | njpatel, do you know something about this F10 issue? I mean, is F10 now assigned to a different short-cut? | 14:44 |
| njpatel | apinheiro, nope, need to investigate it | 14:44 |
| apinheiro | njpatel, ok, I will keep focus on the crash then | 14:45 |
| * apinheiro reminds that he need to check if those messages are also present without a11y enabled | 14:45 | |
| === davidcalle_ is now known as davidcalle | ||
| htorque_ | tedg: that's the output i got from indicator-loader when testing it earlier: http://paste.ubuntu.com/679567/ | 14:56 |
| htorque_ | will try again | 14:57 |
| tedg | htorque_, You've got a GTK2/3 conflict there. You need to use loader3 with the GTK3 indicators. | 14:58 |
| htorque_ | oh dear | 14:59 |
| htorque_ | is that in libindicator-tools? | 14:59 |
| tedg | htorque_, I think so. | 15:00 |
| htorque_ | libindicator3-tools... | 15:00 |
| tedg | Oh, yes | 15:00 |
| htorque_ | meh | 15:00 |
| apinheiro | tedg, forget my question, it seems that those warning only appear if I launch by hand the panel service from a terminal that it is not in the same session that unity | 15:03 |
| apinheiro | executing this "in a normal way" I don't get those messages | 15:04 |
| apinheiro | sorry for the noies | 15:04 |
| apinheiro | noise | 15:04 |
| tedg | Hmm, that's interesting... I wonder why that is. | 15:04 |
| htorque_ | \o/ loader3 works | 15:05 |
| apinheiro | tedg, in my environment I have a "working session" and the session where I made the "unity --replace", so I move from one to the other using ctrl+alt+F7/F8 | 15:06 |
| apinheiro | I was running the panel-service from a terminal on my working session, | 15:06 |
| apinheiro | (with a export DISPLAY) | 15:06 |
| apinheiro | and in this case I got those messages | 15:06 |
| htorque_ | njpatel: starting gimp adds lots of ram to the loader process ;-) | 15:06 |
| apinheiro | again, sorry for the noise | 15:06 |
| htorque_ | njpatel: should i now run valgrind on that and attach the output to the bug report? | 15:08 |
| tedg | htorque_, Yeah, that'd be good. | 15:08 |
| htorque_ | will do :) | 15:08 |
| tedg | htorque_, Firefox is also good if you have lots of bookmarks. | 15:08 |
| htorque_ | unfortunately no ff user | 15:09 |
| htorque_ | kile and gimp work fine | 15:09 |
| htorque_ | tedg: there are a couple of "???" entries in the log about libfontconfig/libexpat/libpango but i couldn't find debug symbols for those packages. is that a problem? also i forgot to install dbgsym for loader3 :-/ | 15:30 |
| tedg | htorque_, naw, we're not going to get those solved today :-/ | 15:30 |
| htorque_ | ok, it's uploaded: bug 835646 | 15:34 |
| ubot5 | Launchpad bug 835646 in unity (Ubuntu) "unity-panel-service memory leak when starting various applications" [High,Confirmed] https://launchpad.net/bugs/835646 | 15:34 |
| htorque_ | if you need anything else, let me know | 15:34 |
| jml | tedg: hi | 15:37 |
| tedg | jml, Howdy | 15:37 |
| tedg | htorque_, Thanks! | 15:37 |
| htorque_ | tedg: yw :) | 15:38 |
| tedg | Unfortunately that doesn't seem to have any smoking guns in it. | 15:39 |
| * tedg quickly concludes that there's no way it's his fault, it must be njpatel's bug ;-) | 15:40 | |
| jml | tedg: I just tried the valgrind thing you suggested | 15:41 |
| jml | tedg: but... umm... where does the output go? | 15:41 |
| jml | bug 740382 | 15:42 |
| ubot5 | Launchpad bug 740382 in indicator-session (Ubuntu) "gtk-logout-helper crashed with SIGABRT in __libc_start_main()" [Low,Confirmed] https://launchpad.net/bugs/740382 | 15:42 |
| tedg | jml, You'll need to redirect it to a file. | 15:42 |
| tedg | jml, So I guess I should have included that in my command line :-) | 15:42 |
| tedg | htorque_, Did you just start GIMP, or start it and shut it down before closing the loader? | 15:43 |
| jml | tedg: np. I'll do that now. How will I know if I've triggered the condition? | 15:44 |
| tedg | jml, Well, I'm guessing that it doesn't shutdown, no? | 15:44 |
| jml | tedg: ok :) | 15:44 |
| tedg | jml, I'm guessing that somewhere in that log there has to be an invalid read/write. | 15:45 |
| jml | tedg: stdout & stderr, or just one of them? | 15:46 |
| tedg | jml, Oh, grab them all. /me is feeling generous with LP's disk space. | 15:46 |
| htorque_ | tedg: i started it, waited for the loader to show the menu, then closed it. started it again, a.s.o. | 15:48 |
| tedg | htorque_, Okay, makes sense. Just making sure. | 15:48 |
| tedg | As much as I love blaming njpatel, I try to not do it arbitrarily, well, not all the time. | 15:48 |
| njpatel | htorque, it's tedg's fault and he knows it | 15:49 |
| njpatel | I don't leak. Nor does my code. | 15:50 |
| htorque_ | i only know that it's not my fault. opening applications seems like expected behavior. :P | 15:50 |
| Trevinho | Some times ago I also gave a look to this problem | 15:52 |
| andyrock | JohnLea, Finally I found the time to fix this https://bugs.launchpad.net/unity/+bug/832631 | 15:52 |
| ubot5 | Ubuntu bug 832631 in unity (Ubuntu) "Dnd hover timeout in spread view does not work as expected" [Undecided,Confirmed] | 15:52 |
| andyrock | i've add this behavior | 15:52 |
| Trevinho | and it seems that unity-panel-service itself is working well | 15:52 |
| Trevinho | according to valgrind | 15:52 |
| Trevinho | it seems something more related to the indicators | 15:52 |
| Trevinho | and libappmenu seems the one | 15:53 |
| tedg | Trevinho, Heresy I say! | 15:53 |
| * tedg sends the Spanish inquisition! | 15:53 | |
| Trevinho | tedg: I know it's not your fault... :) | 15:53 |
| andyrock | the timeout is reset if the mouse hover another window or if the mouse move more than 3px | 15:53 |
| Trevinho | but, the fact is that also running the appmenu-indicator in natty, the leak persists | 15:54 |
| tedg | Trevinho, Did you try running the test app indicators under valgrind and see if anything came up? | 15:54 |
| Trevinho | so, maybe it's not directly connected to that indicator, but to something going wrong in the framework | 15:54 |
| Trevinho | I don't, but there's a valgrind log attached to the mentioned bug | 15:54 |
| Trevinho | https://launchpadlibrarian.net/78705404/valgrind.log | 15:54 |
| andyrock | there is a 6x6 rect and the mouse pointer is the center, when the mouse pointer go outside of this rect the timeout is reset | 15:55 |
| andyrock | is it ok? | 15:55 |
| tedg | Trevinho, That is appmenu not application though... | 15:55 |
| tedg | Really both of them are pretty dynamic. If there's a memory leak, it'd have to be one of those. | 15:55 |
| jml | tedg: attached | 15:57 |
| jml | tedg: my computer doesn't have working g-s-d now. | 15:57 |
| Trevinho | ah, sorry tedg I'll check | 15:57 |
| htorque_ | applications definitely has one: bug 829961 | 15:58 |
| ubot5 | Launchpad bug 829961 in Unity Foundations "indicator-applications leaks memory" [High,Triaged] https://launchpad.net/bugs/829961 | 15:58 |
| jml | tedg: so, uhh, I'll do launch that manually | 15:58 |
| htorque_ | but that's not affected by starting applications | 15:58 |
| JohnLea | andyrock; great news! Do you know if it will be landing in today's or next week's release? | 16:14 |
| andyrock | it's not a unity bug but a scale plugin one | 16:15 |
| andyrock | https://code.launchpad.net/~andyrock/compiz-core/fix-832631 | 16:15 |
| smspillaz | andyrock: poke | 16:33 |
| smspillaz | andyrock: what's the reason for resetting scale on mouse motion ? | 16:33 |
| smspillaz | (or a buffer of mouse motion) | 16:34 |
| smspillaz | seems like it only makes sense to do it per window | 16:34 |
| smspillaz | err | 16:35 |
| smspillaz | the scale dnd timeout | 16:35 |
| andyrock | smspillaz, well if you move the mouse a lot without changing window the timeout don't reset | 16:35 |
| andyrock | *doesn't | 16:35 |
| andyrock | JohnLea, ^^^ | 16:35 |
| smspillaz | andyrock: ah, so you want it so that it it resets when you move the mouse around inside the client window a little bit, but not when you move it a lot | 16:36 |
| andyrock | smspillaz, yes | 16:37 |
| smspillaz | andyrock: in that case it might make more sense to accumulate motion events | 16:37 |
| andyrock | feel free to remove the rect... :) | 16:37 |
| smspillaz | andyrock: like, watch MotionNotify, and if we've moved more than X pixels then reset the timeout | 16:38 |
| smspillaz | (timer.stop (), timer.start ()) | 16:38 |
| andyrock | during the xndd we don't get the motionnotify events | 16:38 |
| smspillaz | ah, crap, that's right | 16:38 |
| smspillaz | ehh | 16:38 |
| andyrock | i think because the mouse is grabbed by the source | 16:38 |
| smspillaz | well hang on | 16:39 |
| andyrock | but the source should do it for us | 16:39 |
| om26er | Trevinho, bug 838923 wont that be compiz? | 16:39 |
| ubot5 | Launchpad bug 838923 in unity (Ubuntu) "Start dragging a maximized application title causes it to re-decorate" [Undecided,In progress] https://launchpad.net/bugs/838923 | 16:39 |
| smspillaz | you should get xdndPosition? | 16:39 |
| smspillaz | om26er: the redecoration is done by unity | 16:39 |
| andyrock | yes.... | 16:39 |
| smspillaz | andyrock: so you can use that, right ? | 16:39 |
| om26er | there are a few other bugs reported already about it though. | 16:39 |
| andyrock | case ClientMessage: | 16:39 |
| andyrock | if (event->xclient.message_type == Atoms::xdndPosition) | 16:39 |
| andyrock | { | 16:39 |
| andyrock | smspillaz, ^^^ | 16:40 |
| om26er | smspillaz, oh ok.. | 16:40 |
| smspillaz | andyrock: what's wrong with using xdndPosition ? | 16:40 |
| andyrock | smspillaz, we alredy use xdndPosition :) | 16:40 |
| andyrock | and get x,y form the message event | 16:41 |
| smspillaz | andyrock: right. so instead of using the rect, what you could do is accumulate the pointer movement | 16:41 |
| Trevinho | om26er: I gess that it's just an unity policy | 16:41 |
| smspillaz | (eg, save the last x and y co-ords, and apply that to a global dmovement) | 16:41 |
| Trevinho | I can control the decoration, but I don't know how to manage it | 16:41 |
| smspillaz | and then if dmovement > threshold reset the timer | 16:41 |
| andyrock | ok... i got it | 16:42 |
| andyrock | can i ask you why? CompRect is slow? | 16:42 |
| smspillaz | andyrock: I'm thinking of cases where the user moves their cursor in one direction the entire time | 16:42 |
| andyrock | smspillaz, i don't understand... if the mouse move outside of the rect the rect update itself. | 16:45 |
| andyrock | but i will do it... np | 16:45 |
| smspillaz | andyrock: right, although this way is slightly more consistent | 16:46 |
| andyrock | what do you think is a good threshold? | 16:46 |
| smspillaz | might be worth making it an option | 16:46 |
| andyrock | ok ok :) | 16:46 |
| smspillaz | you know how that works? | 16:46 |
| andyrock | of course... | 16:46 |
| smspillaz | awesome | 16:46 |
| andyrock | i added some option to unity code | 16:47 |
| smspillaz | :) | 16:47 |
| andyrock | i think it's the same | 16:47 |
| smspillaz | yep, same | 16:47 |
| andyrock | we have an option for the timeout so it make sense | 16:47 |
| smspillaz | also, compiz coding style has spaces between function calls and brackets | 16:47 |
| smspillaz | and spaces between operators and members | 16:47 |
| smspillaz | oh, I think that it might be better to have a member in the ScaleScreen class "mDistance" | 16:48 |
| smspillaz | oh! and I almost forgot, pointerX and pointerY are updated on xdndPosition iirc | 16:48 |
| smspillaz | so you don't need to save their position :) | 16:48 |
| smspillaz | you can just use pointerX - lastPointerX | 16:48 |
| andyrock | so there is no need to use this stuff | 16:58 |
| andyrock | int x = event->xclient.data.l[2] >> 16; | 16:58 |
| andyrock | int y = event->xclient.data.l[2] & 0xffff; | 16:58 |
| andyrock | smspillaz, ^^^ | 16:58 |
| smspillaz | andyrock: right, although you'll need to know what the last pointer position was | 17:02 |
| andyrock | smspillaz, using lastPointerX and pointerX and Pythagorean Theorem works well :) | 17:30 |
| smspillaz | andyrock: sweet :) | 17:30 |
| smspillaz | andyrock: update your merge proposal and I'll merge that in | 17:30 |
| smspillaz | good work | 17:30 |
| andyrock | smspillaz, do you prefer a private member or i can use optionGetDndDistance every time? | 17:45 |
| andyrock | all the other option are not saved in a private member | 17:45 |
| om26er | smspillaz, is this branch https://code.launchpad.net/~smspillaz/unity/unity.fix_838062/+merge/73703 supposed to fix those issues reported in minimize? | 17:46 |
| om26er | like windows wont raise etc | 17:47 |
| smspillaz | om26er: it will fix input still being around for those windows like chromium with CSD | 17:50 |
| smspillaz | andyrock: yeah, using the option directly is fine | 17:50 |
| om26er | smspillaz, that would close a bunch of lp bugs, I'll link them as I find :0 | 17:50 |
| om26er | *:) | 17:51 |
| smspillaz | om26er: :) | 17:51 |
| smspillaz | some applications do some really weird stuff | 17:51 |
| andyrock | smspillaz, can you give me the astyle command to format properly the code | 17:57 |
| andyrock | you know compiz style is weird (tab + space i really don't understand why :) ) | 17:57 |
| om26er | Trevinho, will your panel fix branch also fix bug 835989 ? | 18:01 |
| ubot5 | Launchpad bug 835989 in unity (Ubuntu) "Launcher - does not hide if revealed from top 24px of the left edge of the screen" [Medium,Confirmed] https://launchpad.net/bugs/835989 | 18:02 |
| andyrock | om26er, about dnd dash bug | 18:07 |
| andyrock | the cause of the problem is similar but it's due to different parts of code | 18:09 |
| andyrock | so fixing one doesn't fix the other | 18:09 |
| andyrock | one... but if we found a solution we can use it both in the dash problem and collection window one | 18:10 |
| Trevinho | om26er: I've to check but I guess no | 18:10 |
| Trevinho | om26er: checked. No | 18:11 |
| om26er | andyrock, aha alright | 18:11 |
| Trevinho | Maybe I'll look to that aftherwards | 18:12 |
| om26er | Trevinho, that'l be cool | 18:12 |
| Andy80 | Kaleo: I've noticed this behaviour on Unity-2d running on Oneiric: even if you maximize a window, you still see two application window title. The one in the top unity bar and the other one below. I try to make a screenshot... | 18:18 |
| * om26er think Andy80 is using compiz | 18:18 | |
| Andy80 | om26er: eh? | 18:19 |
| === daker_ is now known as daker | ||
| smspillaz | andyrock: I don't have it on me | 18:19 |
| om26er | Andy80, the problem happens when you are using compiz with unity-2d | 18:20 |
| om26er | Andy80, bug 705182 | 18:20 |
| ubot5 | Launchpad bug 705182 in unity-2d "[window management] With Compiz window decorations are shown both in the panel and in the window itself" [Medium,Confirmed] https://launchpad.net/bugs/705182 | 18:20 |
| Andy80 | om26er: I don't know if I'm using compiz or not... I just upgraded my VM from Natty to Oneiric | 18:20 |
| Andy80 | I didn't choose to use compiz by my own | 18:21 |
| om26er | Andy80, you can try metacity --replace and afterwards see if the issue happens or not | 18:21 |
| om26er | but there must be a better way to know which window manager is running though i dont know how :/ | 18:22 |
| Andy80 | om26er: tried.... I still have the same bug | 18:22 |
| Andy80 | om26er: how do I disable Compiz? | 18:22 |
| om26er | Andy80, not sure how, metacity --replace disables compiz | 18:23 |
| om26er | gotta run, its EID :D | 18:23 |
| === daker is now known as daker_ | ||
| andyrock | smspillaz, what do you mean for "I don't have it on me"? | 19:00 |
| smspillaz | andyrock: like, I don't have any astyle command that you can use | 19:01 |
| smspillaz | andyrock: but I don't really like autoindent | 19:01 |
| smspillaz | causes problems | 19:01 |
| smspillaz | andyrock: the general style is | 19:01 |
| smspillaz | 8 wide tabs | 19:01 |
| smspillaz | spaces between functions and brackets () | 19:01 |
| smspillaz | camelCaseVariables | 19:01 |
| smspillaz | and the tabbing goes something like | 19:01 |
| smspillaz | 1 indent: | 19:02 |
| smspillaz | 4 spaces | 19:02 |
| smspillaz | 2 indents | 19:02 |
| smspillaz | 1 tab | 19:02 |
| smspillaz | 3 indents: | 19:02 |
| smspillaz | 1 tab, 4 spaces | 19:02 |
| smspillaz | 4 indents | 19:02 |
| smspillaz | 2 tabs | 19:02 |
| smspillaz | etc | 19:02 |
| smspillaz | so think of it as "indents" and "half indents" | 19:02 |
| smspillaz | where an indent is 1 tab and a half indent is 4 spaces | 19:02 |
| andyrock | ok.... it's a bit weird :) | 19:02 |
| smspillaz | (it's X11 style) | 19:02 |
| smspillaz | yes it is | 19:02 |
| smspillaz | however, changing it is a nightmare | 19:03 |
| L42y | Hi, I'm building unity in Arch Linux. When I run unity with command: "unity --replace -v --log=~/unity.log", it ouput "unity-panel-service: no process found" "compiz (core) - Debug: Could not stat() file /usr/lib/compiz/libcore.so : No such file or directory" "compiz (core) - Warn: Value type is not yet set" and unity dosen't show up | 19:03 |
| smspillaz | because if you change it you'll break bzr blame and git blame and we can't do that since a lot of the code is not well documented | 19:03 |
| smspillaz | L42y: is the unity plugin being loaded? | 19:03 |
| smspillaz | look for "Initializing unityshell options ... done" | 19:04 |
| === zyga is now known as zyga-afk | ||
| L42y | smspillaz: I enable it in ccsm | 19:04 |
| smspillaz | L42y: right, but is it actually being loaded | 19:04 |
| andyrock | my unity creshed.... damn | 19:05 |
| L42y | smspillaz: When I run ccsm, I see "Initializing unityshell options...done" | 19:06 |
| L42y | Is this means unity loaded? | 19:07 |
| smspillaz | yes | 19:07 |
| smspillaz | and you can't see anything ? | 19:07 |
| L42y | smspillaz: Yeah, I run the command in kde. | 19:09 |
| smspillaz | haven't really got any ideas then, sorry man :/ | 19:15 |
| L42y | smspillaz: Thank you for your help. | 19:18 |
| === daker_ is now known as daker | ||
| === daker is now known as daker_ | ||
| smspillaz | andyrock: mergee | 19:53 |
| smspillaz | andyrock: merged | 19:53 |
| andyrock | smspillaz, thx :) | 19:53 |
| === yofel_ is now known as yofel | ||
| thumper | morning | 21:25 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!
