[00:19] ochosi: new version with filtering when zoom is less than 4x, and some other improvements: http://paste.ubuntu.com/6402669/ [00:23] ali1234: wow! [00:23] very smooth [00:23] (and thanks for inverting the scroll-direction - feels *so* much better ;)) [00:25] being zoomed in does seem to eat up my cpu though (at least that's what top says) [00:25] yes it will [00:25] is that due to the smoothing? [00:25] (didn't check that before) [00:25] no, it's due to it has to poll mouse position [00:25] ah [00:25] hm [00:25] i guess there's not really a way around this [00:25] currently it is hardcoded to check it 60 times per second [00:26] probably a bit much [00:26] it's the g_timeout_add(32 ... line [00:27] 32 msec [00:27] i could make it check if the mouse coords are the same as last time and if so ignore them [00:28] to be frank, even with 10 it seems extremely smooth [00:29] heh, although i'm seeing now that that doesn't seem to help with the cpu [00:30] ali1234, I tried your patch - got a black screen with only a cursor on it. NVidia drivers. [00:30] so yeah, maybe that check would help [00:31] andrzejr: i'm also using nvidia proprietary btw [00:31] i'm on nvidia too [00:31] oh, fun :p [00:31] andrzejr: is that only when zooming, or all the time? [00:32] just after xfwm4 --replace [00:32] (fwiw, i'm not using git-master, but the ochosi/tabwin branch) [00:32] couldn't debug anything because linux console doesn't work with proprietary drivers [00:34] er.... mine does? [00:35] andrzejr: you mean VT? works fine here... [00:35] so maybe its a graphics card, or several ubuntu updates (I do get some other problems too) [00:36] try commenting lines 3009, 3011, 3013 [00:36] if you comment them all, the whole thing should become a no-op [00:36] When I switch to a console the screen becomes grayish and gets progressively brighter/lighter. [00:37] so xfwm4 works without the patch? [00:38] did you build with --enable-compositor? [00:38] xfwm4 from git master works fine [00:38] i haven't tested what happens if you don't do that, probably should [00:39] ali1234: i put the value down to "3" now, still is smooth and cpu is still burning up... [00:39] no, I didn't (autoconf did enable it though so I assumed it is OK) [00:39] ali1234: are you sure it's that which causes it? [00:39] ochosi: you should increase it, not decrease it :P [00:39] yeah, i also skipped the --enable-compositor, it should be enabled by default [00:39] ooophs [00:39] :) [00:40] hang on though, new patch [00:40] http://paste.ubuntu.com/6402727/ [00:41] building... [00:42] ali1234: a bit less cpu usage, still 100% (as in: 1 of 2 cores) though [00:44] my check doesn't work [00:44] hm, seemingly [00:45] maybe you can't use a static variable in a callback [00:45] or maybe it's just a silly mistake [00:45] the latter [00:45] y_old != x_root -> should be y_root [00:46] well it's that time of the day here already (silly-mistakes-o'clock) [00:46] much better :) [00:48] went down to 75% here [00:49] for me it went down to a negligible amount when not moving the mouse [00:50] yeah, this time it was my time for a silly mistake [00:50] * ochosi facepalms [00:50] no more cpu-eating [00:50] very nice [00:50] in theory it should now have true zero overhead if there's no change in the zoom state [00:51] yeah, that's also what top says [00:51] (i decided to zoom in on a top-window) [00:51] well, except the QueryPointer call but that's not much [00:53] nice work ali1234 ! [00:53] same problem with the new patch [00:58] andrzejr: i didn't really think the new patch would help you, sorry [00:59] try commenting lines 3009, 3011, 3013 [00:59] ali1234: zooming in on parole has funny effects :) [01:00] hmm... it does [01:01] the overlay window doesn't increase in size, so only part of the video shows [01:02] that's due to the lazy updates [01:02] or rather, forcing a full redraw fixes it [01:03] not sure how to deal with that but there must be a way [01:05] xterms do the same thing :S [01:06] and youtube [01:25] hmm... i think i know how to fix it. but i'm going to bed now... night [01:36] ali1234: night! [01:36] * ochosi goes too [09:56] morning [09:57] i found a bug in xfwm which fixing it should have fixed the clipping area issues [09:57] but it didn't :( [09:59] good morning all [10:04] ali1234: what clipping area issues? [10:04] and morning everyone [10:04] well... you know blitting? [10:04] dirty rectangles and so on? [10:05] the compositor redraws only the damaged areas in the offscreen buffer [10:05] then it copies only the damaged areas from the buffer to the display [10:05] but the XserverRegions aren't affected by the PictureTransform, so the second copy copies the wrong areas [10:06] oh [10:06] there's two problems: first is it applies the region to the display instead of the buffer, which is a bug imo [10:06] and the second is that it can't apply it to the buffer either on the second draw step, if we are zoomed in [10:06] so... it's fixed, kind of [10:07] not sure i was able to follow this all now [10:08] heh [10:08] basically XRender is a very 2D type system and it uses all the old style "redraw only the parts that changed" methods from framebuffer days [10:08] but the damage system and XRender don't really talk to each other [10:10] after fixing what i consider to be a bug, it might actually run faster anyway now [10:11] is this specifically a zoom-problem or does it affect xfwm4 in general? [10:12] (fwiw i'm startin to think you should become xfwm4's maintainer, no-one has really taken care of it for ages...) [10:12] i'm not 100% sure [10:12] depends exactly how Xrender treats the clip regions [10:12] which annoyingly is probably implementation specific [10:13] i am beginning to see why wayland [10:13] i think it's a bug, could be intentional though i think [10:14] anyway the bottom line is that we have to turn off all clipping when doing the buffer->display copy, but only if zoomed in [10:14] because the clipping rects don't follow the Xrender transform, thus the wrong areas are clipped [10:15] i'm going to push this on github because i can write comments on the source there :) [10:17] cool [10:30] https://github.com/ali1234/xfwm4/commits/zoom [10:33] you should also post that on the xfce-dev-ml [10:33] or at least the irc channel [10:33] if it gets merged, it's probably one of the most exciting features for 4.12 [10:40] ali1234: fwiw, if you wanna help me finish the ochosi/tabwin branch lemme know ;) [10:40] what does it do? [10:42] it improves the style of the tabwin (alt-tab dialog) [10:42] makes it themeable [10:42] and in the end it should enable using the mouse to select a window [10:43] nick pushed a commit for that, but it wasn't finished yet [10:43] so it's not 100% working [10:43] (i think the first click somehow doesn [10:43] 't reach the widget or something) [11:03] ali1234: this is the page where i listed some stuff i wanted to do: http://wiki.xfce.org/design/xfwm4/tabwin [11:04] (only the screenshot is really implemented, although i think nick added a text-only listmode once too) [11:31] i don't use alt-tab at all [11:31] is there some specific bug or problem? [11:32] problem 1) is that it uses weird widgets [11:33] problem 2) (following from 1) is that the theming of it is therefore not very nice [11:33] so no, it's an enhancement, not bugfix [11:41] pleia2: ping [13:11] ochosi: pong [13:24] pleia2, have fun trying to pop in the same time :) [13:24] pleia2, he wants to ask you about licenses for wallpapers [13:25] (at least) [13:25] ah [13:29] hey pleia2 [13:29] yeah, what knome said :) [13:29] cc-by is nice [13:29] i started drafting an email to the ML for wallpaper submissions for xubuntu [13:29] so far i had put down cc-by and cc-by-sa [13:29] or cc-by-sa [13:29] but one license would/could make things easier [13:29] not sure [13:30] what do you think/suggest? [13:30] consistancy would be nice [13:30] (we could end up with a package with multiple licenses, not sure how problematic that really is though) [13:30] yeah, i agree [13:30] let's see what ubuntu dose [13:30] does [13:30] good idea [13:31] they use cc-by-sa [13:31] works for me :) [13:32] https://wiki.ubuntu.com/Artwork/Documentation/Backgrounds#Constraints [13:34] pleia2: great, let's stick to that then [13:34] i'll check more of those requirements and integrate them in my call for wallpapers [13:34] would you wanna read it before i send it out? [13:35] ochosi: sure, if you'd like [13:35] knome: check the page pleia2 just pasted, i think some of those guidelines we should also include (see background guidelines) [13:35] ochosi, yup, agree [13:36] acutally most of this document makes a lot of sense [13:36] thanks for digging that up pleia2 ! [13:36] they've refined it over many cycles [13:37] yeah, they've done a few of these contests [13:37] and basically we're planning the same [13:37] cool [13:37] open submissions but a vote of the artwork-team on the "winners" [13:37] kinda an editor's pick [13:37] * pleia2 nods [14:19] I am going to try to sleep again [14:46] knome: think i might end up copying over the complete page from the ubuntu wiki, they really thought it through... [15:08] hm? [16:52] elfy, ping [17:00] slickymaster: pong if it's quick, if not then it'll need to wait - just got in [17:01] elfy: if you prefer I can ping you later on, after dinner [17:02] nope - it's ok I expect - ask and if I'm not I'll say :) [17:02] elfy, I think a correction needs to be made at http://packages.qa.ubuntu.com/qatracker/milestones/306/builds/55995/testcases/1585/results [17:03] elfy: one of the steps to be reproduced during the test is "...In dialogue type gksudo mousepad, press Launch" [17:03] elfy: gksudo and gksu, aren't presently packed with Xubuntu [17:04] slickymaster: https://bugs.launchpad.net/ubuntu-manual-tests/+filebug [17:04] elfy: either we remove/re-write that step or we have to add a note informing on the of sudo apt-get install gksu [17:04] do it there - probably make it a 'general' thing rather than specific [17:05] slickymaster: from what I can tell they're wanting people to use sudo [17:05] elfy: that's what I meant to ask you. I can fil a bug and re-write the entire testcas [17:05] testcase^^ [17:06] slickymaster: make it general rather than for one testcase - when we know exactly what we're supposed to use then we can check them all for the same issue [17:06] I'll try and get a real answer later [17:07] elfy: I'll just fil the bug for the moment and wait until you have something more specific on the issue [17:07] ok [17:07] slickymaster: I will probably work through and do the whole lot myself [17:08] elfy: well, if you feel like you need any help, just ping me [17:08] slickymaster: can you subscribe or assign me to it [17:08] slickymaster: yep - I will :) [17:08] elfy: I will [17:09] cheers [17:09] elfy: thanks [17:09] :) [17:25] elfy: here it is https://bugs.launchpad.net/ubuntu-manual-tests/+bug/1250560 [17:25] Ubuntu bug 1250560 in Ubuntu Manual Tests "Use of gksudo/gksu in testcases for Xubuntu Desktop in Trusty Daily " [Undecided,New] [17:27] elfy: surprisingly I'm not able to assign it to you, LP is not recognizing your LP ID in the Assigned to box. I keep getting "No items matched "elfy"." [17:27] not sure why that is - I just assigned it to elfy [17:28] comes up as first choice when I search for me [17:31] * elfy passes the question along the chain :p [17:31] elfy: it's strange, really, because I'm still getting the same No items matched "elfy". [17:31] odd - you putting it in the box then pressing enter? [17:31] elfy: well, any way it's done as we intended [17:32] elfy: yes, and alternatively clicking the magnifying glass [17:32] odd - but it is launchpad :) [17:33] elfy: I even went to https://launchpad.net/~elfy to confirm your LP ID [17:36] knome: Just FYI https://translations.launchpad.net/xubuntu-docs/trusty/+pots/desktop-guide/pt/+translate is as of now completely translated into Portuguese [17:36] slickymaster: though it seems that pkexec isn't supposed to be used like that [17:37] I'll try now to figure out how to build the translated package in order to do a final proofread of it [17:38] elfy: I'm under the impression that pkexec is supposed to be the intended replacement for gksudo and gksu, but I can be wrong [17:38] slickymaster: maybe you should also file a report on the debian bug tracker, bug 1250364 [17:38] bug 1185396 in gnome-system-tools (Ubuntu) "duplicate for #1250364 users-admin crashed with SIGSEGV in gst_user_profiles_get_for_user()" [High,Triaged] https://launchpad.net/bugs/1185396 [17:38] slickymaster: http://askubuntu.com/questions/313828/why-is-pkexec-preferred-over-gksudo-for-graphical-applications [17:38] slickymaster: doesn't appear so [17:39] right, brainwash. Will do it [17:39] elfy: give a moment to read through that [17:39] give me ^^ [17:41] slickymaster: I tried to understand and fix this crash, but gave up due to lack of time (users-admin is quite complicated for me and I don't feel like breaking it) [17:43] there are other issue too, like it still checks for the "admin" group instead of "sudo" one [17:43] elfy: well, now I'm confused and you are indeed right. That leaves me with the question about with the hell has gksu and gksudo been removed. Is it to just use sudo -i? [17:43] so gnome-system-tools really needs a maintainer or a working replacement [17:44] slickymaster: seems so [17:44] elfy: I'll correct the bug summary to reflect that, then [17:45] brainwash: https://launchpad.net/bugs/1185396 was present in Saucy and it's still present in Trusty, and I think that it is a important issue that should be dealt with [17:45] Ubuntu bug 1185396 in gnome-system-tools (Ubuntu) "users-admin crashed with SIGSEGV in gst_user_profiles_get_for_user()" [High,Triaged] [17:47] right, it's the same old version of g-s-t [17:47] and there is no one to maintain it [17:47] elfy: https://bugs.launchpad.net/ubuntu-manual-tests/+bug/1250560 description corrected [17:47] Ubuntu bug 1250560 in Ubuntu Manual Tests "Use of gksudo/gksu in testcases for Xubuntu Desktop in Trusty Daily " [Undecided,New] [17:48] slickymaster: maybe we could use the current gnome solution [17:50] brainwash: can't really say as I never used gnome [17:51] and thus I'm not aware of their solution [17:53] slickymaster: they did drop g-s-t [17:53] new tool to manage accounts looks similar http://i.imgur.com/B3gze7K.png [17:54] brainwash: yeah, the GUI seems very similar [17:54] so maybe a switch to the new one would be possible [17:56] brainwash: do you have any idea on who could answer that? [17:57] the xubuntu team [17:58] brainwash: maybe you should address knome or ochosi on that [17:59] a discussion would be required (do we lose functionality? does it pull more gnome dependencies? are customization needed to get it running under Xfce?) [17:59] yeah [18:00] brainwash: yeah, you're right, particularly on the dependencies factor [18:01] well, got a pick up my son at school. bbl [18:05] i propose we rize the minimum amount of needed ram to 1gb [18:05] ^ [18:06] even if 512 will work, any webbrowser needs about that much to work with 3 or more tabs [18:06] I'd agree with changing it [18:07] can you please add that to the meeting agenda and prepare the discussion in the meeting itself; and please prepare with arguments why you are proposing [18:07] alright [18:14] oh meh, they removed the groups management part from the new gnome user tool [18:15] looks like we need to keep the old user tool alive [18:21] Speaking of the next meeting, the meeting web page still shows 7 Nov as the next meeting ( https://wiki.ubuntu.com/Xubuntu/Meetings ) [18:28] it's a wiki though, so editably by anyone [22:11] good night all [22:11] hello slickymaster [22:12] hi knome [22:15] slickymaster: so the new user management tool created by the gnome dev team is part of the gnome control center and lacks the ability to manage groups [22:15] ali1234: i noticed that when e.g. typing in the zoomed state, the characters don't exactly fly to the screen (sometimes i have to move the mouse to make them visible), is that what you fixed in github this morning? [22:15] brainwash, so no dice for Xubuntu, then [22:15] brainwash: i fear most of gnome3 is so tightly integrated with each other it'll be hard for us to use it without actually using gnome... [22:16] gnome3 = goodbye modularity [22:16] yeah, we will have to fix the current users-admin [22:16] yeah with the extra burden that all that stuff would bring with it [22:18] brainwash, which reminds me that I still have to report https://launchpad.net/bugs/1185396 on the debian bug tracker [22:18] Ubuntu bug 1185396 in gnome-system-tools (Ubuntu) "users-admin crashed with SIGSEGV in gst_user_profiles_get_for_user()" [High,Triaged] [22:19] yeah [22:22] * skellat says file a bug against -docs if we have to do write-ups on command-line user management tools in case we don't find a graphical replacement [22:23] "IRC: you can contact us in the #gst channel at irc.gimp.org" [22:23] :D [22:26] not everyone can create an bzr branch or? [22:26] a bzr [22:26] brainwash, can [22:27] so I just clone the repo and create a new branch which gets uploaded to launchpad? [22:27] brainwash, you can always push to lp:~youruser [22:28] brainwash, if you want to branch, say xubuntu-docs, you can push to lp:~youruser/xubuntu-docs/my-docs-branch [22:28] ok, thanks :) [22:28] brainwash, if you want to push something that doesn't have a (project) branch in launchpad, you can push to lp:~youruser/+junk/whatever-branch [22:29] elfy, ping. https://bugs.launchpad.net/ubuntu-manual-tests/+bug/1250560 still isn't fixed. running sudo -i in Application Finder dialog won't launch anything [22:29] Ubuntu bug 1250560 in Ubuntu Manual Tests "Use of gksudo/gksu in testcases for Xubuntu Desktop in Trusty Daily " [Undecided,Fix released] [22:30] slickymaster: expected behavior, sudo won't open a terminal window or graphical password prompt [22:30] brainwash, that wasn't why he pinged elfy... [22:32] brainwash, exactly my point. that's why I'm warning elfy about that [22:32] and pkexec requires a policy for every single app or? [22:32] slickymaster, i pushed the changes to the tracker. can you confirm it is actually fix-released-and-in-the-tracker now? [22:33] knome, just a sec [22:33] wait... [22:33] i'm the one who's lost [22:33] knome, yeah, it's rollbacked to gksudo [22:33] i did that because i thought that was wanted [22:33] why did we do sudo -i in the first place? [22:34] knome, no that was the reason that lead me to raise the bug. IMO the entire test should be rewritten [22:34] aha [22:37] * slickymaster thinks that by just adding a indication alerting the user to the need of having to install the gksudo package to further proceed with the testcase. [22:37] should be enough [22:38] sounds like a semi-fishy workaround. isn't there really anything that works OOTB, and why are we dropping gksudo anyway? [22:39] knome, I think it's dropped since Saucy and tbh I also don't see a reason for that [22:40] might've been a ubuntu decision based on privacy or dropping support for certain stuff [22:40] knome: "Because it's deprecated, we use policykit now" [22:41] that's it. [22:41] knome, please refer to http://askubuntu.com/questions/313828/why-is-pkexec-preferred-over-gksudo-for-graphical-applications and http://askubuntu.com/questions/313619/resolvedcould-not-save-the-file-usr-permission-denied-13-04/313625#313625 [22:42] knome, my last link is wrong. Please don't pay no attention to it. What I meant was this one instead http://askubuntu.com/q/78352/33871 [22:44] yep, i see [22:44] TL;DR: use gksudo. ;) [22:45] * slickymaster will discuss the all thing tomorrow with elfy [22:47] but it was already missing in 13.04, right? [22:47] so the discussion seems kinda late :) [22:47] Ubuntu, yes, not Xubuntu. [22:48] ah ok [22:48] Yes, Unit193 is right [22:49] afk [23:44] ochosi: yes [23:52] ali1234: darn, then i gotta try and merge that i guess [23:54] it should be perfect now, graphically (including video players) [23:54] it might be slightly slower though