/srv/irclogs.ubuntu.com/2014/12/15/#ubuntu-desktop.txt

pittiGood morning05:41
mlankhorstmorning!07:16
didrocksgood morning07:17
larsumorning!07:19
ochosimorning folks07:20
didrockshey larsu, ochosi07:20
ochosilarsu: quick question, where is your gtk3.14 staging PPA? (so i can start to prep the xubuntu vivid themes)07:20
larsuochosi: in the desktop team ppa07:22
didrockspitti: hey, welcome back! I hope you had a safe flight back :)07:22
didrockspitti: I saw that you were cooking this week-end07:22
didrocksand pushing new systemd :)07:22
didrockspitti: I wanted to know if there was any reason you didn't include bug #1400682 patch?07:23
ochosilarsu: oh indeed. note to self: don't start asking around *before* having had coffee...07:23
ubot5bug 1400682 in systemd (Ubuntu) "Add xdiagnose fallback when display-manager fails to start" [Undecided,New] https://launchpad.net/bugs/140068207:23
larsuochosi: haha, no worries ;)07:24
pittibonjour didrocks, ça va !07:55
pittididrocks: yes, flight was uneventful, and I could sleep a bit too07:55
pittididrocks: the main reason was, I did the merge/update mostly on the plane, and I didn't have that patch downloaded  yet :/07:55
pittididrocks: but there is a test regresssion and a timedatectl crash on i386 anyway, so I'll need to upload a new version today anyway07:55
pittididrocks: I have that bug open now :)07:55
didrockspitti: perfect, thanks!07:59
didrockspitti: nice that you could sleep, how long was it btw?07:59
didrocksI guess something like 9-10 hours?07:59
pittididrocks: 11 hours08:00
pitti18:55 to 5:30 on the way back, 22:55 to 10:50 on the way there (1 h time zone difference)08:00
didrocksyeah, quite long, at least, no jetlag :)08:01
ochosilarsu: quick question, is file-roller on your list of apps to remove CSD?08:11
mlankhorstI should not be allowed to have smartphones :P08:14
didrocksmlankhorst: breaking one?08:16
mlankhorstyeah I cracked the screen on the aquaris08:16
didrocksit's still a build machine at least08:17
mlankhorsttrue but I was using it to test.. if I can figure out how to get the screen permanently unlocked and bright it would be fine08:18
didrocksmlankhorst: you can maybe get it repaired?08:19
didrockslike I don't think it's a particular screen compared to android versions08:19
larsuochosi: I thought seb already patched that?!08:20
larsuochosi: it doesn't have csd on latest vivid, or are you talking about a new version?08:21
mlankhorstdidrocks: is it possible to unlock the screen without using the touchscreen? I already have the phone setup08:21
mlankhorstadb shell etc works08:21
ochosilarsu: this is vivid with the ubuntu-desktop PPA (gtk3.14) and file-roller: http://i.imgur.com/n36Mjdz.png08:21
didrocksmlankhorst: not that I know of08:22
ochosilarsu: i'll switch mirrors, maybe mine is lagging behind...08:22
didrocksmlankhorst: I would really try to go to a repair shop, the model from what I know is similar to the android one08:22
didrocksso they should have that screen08:22
larsuochosi: which desktop? I have a normal title bar on unity08:23
ochosilarsu: xubuntu08:23
mlankhorstdidrocks: it's identical, but yeah I guess I can take a look08:23
ochosilarsu: did you only add a check for unity?08:23
ochosilarsu: my guess is that both mythbuntu and ubuntu-studio (maybe even u-mate) would appreciate "!=gnome" instead of "==unity"08:24
ochosiand lubuntu probably08:24
larsuochosi: it checks for GNOME: http://bazaar.launchpad.net/~ubuntu-desktop/file-roller/ubuntu/view/head:/debian/patches/bz_unity_headerbar.patch08:24
ochosiweird08:25
didrockslarsu: we really need a framework to tweak that on runtime, by capability I guess08:25
didrockslike "support-overlay-scrollbar"08:25
didrocks"support-appmenu"08:25
didrocks"disable-foo"08:25
ochosilarsu: not sure you noticed, but it has a normal menu plus a headerbar, so something is fishy there08:25
didrocks(yeah, I know, I introduced this per-session name based in 2009/2010, but I regret now :p)08:26
ochosilarsu: better visible here: http://i.imgur.com/waneudI.png08:26
larsudidrocks: ya... we talked about having another xsetting, like we do for global menu and buttons in dialog headers08:27
didrockslarsu: yeah, that would make sense08:27
larsudidrocks: consensus so far has been "oh no, not another xsetting"08:27
didrocksas there is no phone, tablet or whatsoever08:27
didrocksdone properly, there is "touch screen", "keyboard", "screen size", "battery"08:27
larsuochosi: probably because gtk-shell-shows-menubar is 008:27
didrocksand changing the app/framework behavior on that08:27
didrockslarsu: did you try to compare that to media queries?08:28
larsudidrocks: compare what?08:28
larsuinteresting idea, though...08:28
didrockslike, media queries are really trying to do that and base on capability rather than form factor/session08:28
larsuright08:28
didrockslarsu: just taking web media queries as an example as part of that discussion08:28
didrocksas an argument :)08:28
didrockslarsu: not the most important thing right now, but this discussion just makes me thinking that :)08:29
larsudidrocks: makes a lot of sense to me. We'd still need something like xsettings to provide the values, though08:30
=== ara is now known as Guest73317
didrockslarsu: yeah, that's clearly one of the way to achieve this08:30
didrocksjust talking about the general concept, before we introduce an unity8-desktop for instance :)08:30
didrocksfinding a grub script typo -> *not* fun08:31
larsuthe unity8 guys have a plan08:31
larsuI don't know what that is, though08:31
didrockswe should follow the same one for desktop for sure (not implementation-wise, but for coherence)08:31
didrocksanyway, sorry for side-tracking :)08:32
didrockswaiting for grub2 to pass tests (again :p)08:32
ochosilarsu: right, so should i report a bug about this?08:33
larsudidrocks: haha, no worries :)08:34
larsuochosi: I'm not sure what the issue is?08:34
larsuyou don't have a global menu in xubuntu, do you?08:35
larsuso having that xsetting set to 0 is correct, and you get a local menu08:35
ochosilarsu: ehm, the issue is that file-roller has CSD..?08:35
ochosithe menu part is fine08:35
ochosiit was just confusing for me that there could be a mix08:36
larsuI don't understand how that is confusing. If you tell gtk that you want a header bar and a menu inside the window, that's what you get08:36
larsuwhat's in your XDG_CURRENT_DESKTOP?08:36
ochosilarsu: it's XFCE08:40
ochosisry, g2g, bbl08:43
larsuochosi: ah, it is a bug in the patch08:43
larsuI can reproduce08:43
larsuochosi: which version do you have installed? The patch I linked to is not yet in vivid08:46
larsuhttp://bazaar.launchpad.net/~ubuntu-desktop/file-roller/ubuntu/revision/15008:46
larsuLaney: is file-roller file-roller blocked on anything?08:46
willcookemorning08:58
didrockshey willcooke, feeling better?09:01
Laneyheya09:02
didrocksmorning Laney, good week-end?09:02
larsumorning Laney, willcooke09:03
willcookedidrocks, not really.  Paracetamol  helps09:04
didrocksurgh :/09:04
didrockswillcooke: take it easy, soon holidays!09:05
willcookeheh #09:05
Laneyhey didrocks et larsu09:05
willcookelooking forward to it09:05
Laneynot bad thanks, did some xmas shopping ;-)09:05
Laneyyou guys?09:06
Laneylarsu: file-roller> not sure offhand, did robert-ancell do that one maybe?09:06
didrocksplayed some nice board games and finished watching breaking bad :)09:06
larsuLaney: don't know, thought you might09:06
Laneystill haven't played that game I got in DC :(09:06
Laneylarsu: I'll take a look09:07
larsuthanks09:08
larsuthe one we have now checks for Unity to disable csd09:09
larsuthe new one checks for !GNOME09:09
Laneythey did that upstream?09:09
larsuno, robert_ancell did (and you committed it apparently)09:09
larsuhttp://bazaar.launchpad.net/~ubuntu-desktop/file-roller/ubuntu/revision/15009:10
Laneyi see09:10
Laney"(blocked on gtk 3.14)"09:11
darkxstLaney, about everything is blocked on gtk 3.14 now (for us anyway!)09:11
didrockspitti: I hope I'm using git-dpm well and patching the grub patch to generate other installed init system FYI09:12
Laneydarkxst: ya, lots for us too, might be that we can deal with the remaining issues post upload09:20
LaneyI need to test ubiquity though09:20
ricotzLaney, hi :), just a small request, could you define the mir minimum-requirement in the gtk+3.0 packaging?09:31
Laneywhat is it?09:32
pittididrocks: sure -- if it looks ok and dpkg-source is happy, it's good :)09:37
ricotzLaney, idk, at least the trusty version is not sufficient09:38
LaneyI could make it >= what-we-have-now09:38
Laneyyou probably just want to disable that for the PPA09:38
ricotzi know and i will, still would be nice to keep track on such things09:40
ricotzalso there is a 3.14.6 ;)09:42
Laneyyep09:44
Laneylarsu: I see you got an ok for your icon patch ;-)09:57
larsuLaney: I dumbed it down considerably with only 2 lines left09:58
Laneyya09:58
Laneyseems this is acceptable09:59
larsuspeaking of which - virtmanager has large buttons because one of the toolbar icons is 48x4809:59
* larsu figured that out after inspecting css for 20 minutes09:59
Laneywhat about the borders?09:59
larsuborders have always been on those, no?10:01
larsuI can remove them if you prefer10:01
Laneydon't think so10:02
Laneyone sec, lemme boot utopic10:02
Laneylarsu: http://people.canonical.com/~laney/weird-things/virt-manager.png trusty10:06
Laneyhttp://people.canonical.com/~laney/weird-things/virt-manager-new.png that's with 3.14.510:07
larsuI stand corrected. Thanks.10:08
desrtLaney: you never told me how great schroot is10:10
Laneydesrt: dude, welcome to the enlightened10:10
desrti jhbuild on fedora on my ubuntu linode10:11
desrtyo dawg and all that10:11
larsuschroot?10:11
larsudesrt: good ... err ... morning?10:11
desrthi :)10:11
larsuyou're up early10:11
desrtlarsu: it's a system for maintaining images of other distros that you can chroot into10:12
desrtit has some extremely lovely features10:12
larsucoool10:12
desrtlike the way i'm using it now, it unpacks a fresh .tar each time10:12
desrtso i can explode it however i want10:12
desrtand when i logout, it deletes it10:12
desrtit also has a lot of really great features like automatically bindmounting /sys /proc /dev and (insanely usefully) /home10:12
darkxstlarsu, jhbuild is a major hack, schroot would be better, but it helps finding ubuntu issues10:13
desrti've used it from time to time when Laney wants me to fix a bug on some weird arch or another but i only really "learned" it this weekend10:13
desrti should done that years ago10:13
desrt*shoulda10:13
didrocksdesrt: you can even mount the overlay as tmpfs to avoid the unpacking :)10:13
desrtdidrocks: linode has no aufs support in their kernels :(10:13
didrocksargh, so switch to btrfs and use nspawn coming with systemd 219 ;)10:14
desrtmeh10:14
desrtcontainers are too heavy10:14
desrti also really really really like how desrt on the outside is desrt on the inside10:14
desrtit's totally perfect10:14
larsudarkxst: this seems to be for a different use case than jhbuild...10:14
didrocksdesrt: ahah, nicely said :)10:14
larsu(which I'm actualy quite fond of these days)10:14
* desrt did a pile of jhbuild hacking over the weekend10:15
darkxstlarsu, yes GNOME upstream has no sbuild/schroot stugg10:15
desrti added support for finding out about _all_ of the things that are needed to build the lower bits of gnome10:15
desrtincluding the stuff that we couldn't previously add checks for (libxml2-python, docbook-xsl)10:15
desrtand now i have a new mode that dumps the full list in machine readable form like pkgconfig:egl,c_include:jpeglib.h,...10:16
darkxstdesrt, like making system modules satisfy jhbuild?10:16
desrtand another script that runs that list against a Contents.gz from apt-file and comes up with various solutions10:17
desrtdarkxst: ya...10:17
willcookedesrt, you're up early10:17
Laneyor late ...10:17
desrtearly :)10:17
desrtLaney: maybe you could help me with some of the issues i hit10:17
desrti asked in #debian but it was pretty slow over the weekend10:17
Laney#debian-devel might be better for that kind of q10:18
desrtfirst issue is that i want to find a way to effectively say: apt install a b 'c | d' e f 'g | h'10:18
desrtwhere c/d are alternatives for the same functionality and g/h the same for another10:18
desrtlike jpeg vs. jpeg-turbo or something10:18
desrtcurrently my script takes all of the possibile combinations of the alternatives, does apt dry-runs on them all and calculates a 'cost' based on the number of reported Inst and Remv lines (with removals costing 1000 times more than adds)10:19
desrtthen it picks the lowest cost10:20
LaneyI'd build a .deb with that in Depends: and let apt figure it out10:20
desrtunfortunately that involves doing apt dry-runs of ~1000 operations over 192 possible combinations... which takes 5-10 minutes :(10:20
desrtLaney: ya.... i was hoping for a better way10:20
desrtmore direct10:20
Laneythat's what sbuild does when it resolves build-deps, for example10:21
desrtalso: Depends: lines are a bit "opinionated", right?10:21
Laneyor there's a script called mk-build-deps10:21
desrtlike, the thing on the left is the one that is taken unless the others are already installed, no?10:21
Laneyya10:21
Laneywell, kind of, if it's not installable then it'll try the other one10:21
desrtdoes it do that even if the 'cost' is high?10:21
desrtlike for example say i had libfooa and libfoob mutually conflicting10:22
desrtthey have corresponding libfooa-dev and libfoob-c10:22
desrt*-dev, sorry10:22
desrtthen package 'c' depends on libfooa-dev | libfoob-dev10:22
desrtand my system had libfoob installed10:22
desrtwill apt uninstall libfoob in order to get me libfooa-dev just because i didn't have libfoob-dev?10:22
desrtbecause in that case i'd sort of hope for the more reasonable "oh... he's already got libfoob... so probably he just wants the -dev of that"... which is what my 'minimum cost' thing was tilting for10:24
desrtyou know.. mk-build-deps is not bad...... and it would additionally give me an artifact that i could install in order to force all of the depends of my jhbuild to stay installed10:25
LaneyI think apt is smart enough to prefer not to remove packages if it can10:26
Laneybut you get into mvo territory at this point10:26
desrti already sent him an email :)10:26
Laneyor you could do some experimentation10:27
desrti'll see if i can play with mk-build-deps10:27
desrtya...10:27
desrtsee what it gets me10:27
Laney'equivs' is a tool for building packages which don't do anything other than have metadata10:27
Laneyso you can test using that10:27
desrtgreat advice.  thanks.10:27
Laneyor even use it to generate your fake package for dep solving10:27
desrti had another issue, but it's less likely you know about it (or even that it has a solution at all)10:27
desrti want a command that i can call that will answer the question "which package do i have to install in order to get a given uri into the system xml catalog"10:28
desrtie: i want to know what will make 'xmlcatalog /etc/xml/catalog http://docbook.sourceforge.net/release/xsl/current/' return a positive result10:29
desrt(my current thinking towards a best solution here is to hardcode the package name 'docbook-xsl'... i don't plan on the xml depends of jhbuild undergoing a dramatic expansion any time soon... but a generic solution would be nice...)10:31
Laneysorry, had a guy around checking the energy efficiency of the house... gone now...10:39
desrtvery nice of him10:40
LaneyI think that particular operation is done by packages invoking update-xmlcatalog in their mantainer scripts which are usually generated by dh_installxmlcatalog during the build10:40
Laneysoooo no I don't know of a way you could determine that :)10:40
desrtya.. i'm aware of that much, but it's sort of the opposite of what i want10:40
desrti'd need somewhere a database of all the people who ever do that10:40
desrtin general, we need to start getting better information about this sort of stuff into apt...10:41
* desrt wants to apt install pkgconfig:gl or apt install c_include:jpeglib.h or apt install python2:libxml210:41
desrtor (xml:http://whatever)10:42
desrtLaney: is there a better way to install-with-deps than using apt-get install -f?10:45
desrtthe proposed solution in my contrived case above is removing the package with the alternative depends10:46
Laneymaybe gdebi, but I'm not sure what resolver it uses10:46
Laneyor make a local repository using dpkg-scanpackages and put that into sources.list10:46
LaneyISTR some talk about apt getting "apt install .deb"10:47
desrtLaney: long overdue, that one....10:48
LaneyDonKult @ #debian-devel might know more - he's an apt guy10:49
desrtLaney: the idea of adding a local repository is interesting -- i don't even actually need the .deb files in that case10:51
desrtcan just ask apt "what would you do" based on the Packages file10:51
desrtin any case, the result of the experiment is that apt does 'the wrong thing'10:51
desrt(ie: it wants to remove libfoob)10:55
Laneyubiquity seems ok with new gtk11:06
Laneyclosing in on the upload11:11
xnox\o/ Laney11:29
Laneyxnox: remembered to check it this time ;-)11:31
Laneyps. nice christmas tree11:31
xnox=))))11:31
Laneyshared mine with you on g+11:33
xnoxLaney: haven't been on g+ for a while.11:37
xnox.... lennart invited me to systemd hack-fest.11:37
Laneyhaha11:37
Laneyit seemed like a reasonable option from the android share menu11:37
xnoxpitti: are you going for lennarts/systemd fosdem hackfest?11:37
pittixnox: yes, I will11:38
xnoxdidrocks: as well and marco d'Itri.11:39
xnoxpitti: i'll ponder about it.11:41
Laneylarsu: do you need to get someone to review https://code.launchpad.net/~larsu/ubuntu-themes/gtk-314 ?11:59
larsuLaney: yes, but I've just started working on it again12:09
larsuseb128 tried it last week and didn't notice any issues12:09
Laneyyes I've been trying it too12:09
Laneynot hugely qualified to review though12:09
larsuthat's fine, I'm not hugely qualified to write it ;)12:10
Laneyif we're going to upload gtk then this needs to go in too12:10
larsuyep12:10
* larsu is fixing toolbar buttons right now12:11
Laneyok, please put up a MP when that's done :)12:11
larsu"done". lol.12:12
LaneyI believe in you!12:12
larsuhaha - there are little issues all over the place12:12
larsuand changing something in the theme _always_ breaks something else12:12
larsuwe need some way to properly test theming12:13
Laneyit doesn't have to be 100% to get it in vivid12:13
larsuof course12:13
larsulet me finish the toolbar/flat button stuff and then I'll MR12:13
Laneydid you notice that the session selector in unity-greeter got a border too?12:14
Laney:)12:14
larsuI did now :)12:15
Laneyput it on the pad12:15
Laneynot a priority12:15
larsuyep12:15
larsuthanks12:15
=== pstolowski is now known as pstolowski|lunch
pittididrocks: FYI, I sent a fix for the timedatectl regression upstream and fixed test_profile in Debian git; I'm now trying to reproduce the NSpawn.test_boot failure (doesn't happen locally)12:22
pittididrocks: so uploading a new version will still take a bit, but I'll see to getting it done by tomorrow12:23
pittiI'll be off for the afternoon12:23
desrtpitti: take care12:25
pittihey desrt, how are yoU?12:25
desrtbusy :)12:25
didrockspitti: thanks for keeping me posted, now that I sent the grub thing, I'm about to finish some desktop developer stuff. Then, moving on on more systemd work12:25
didrockspitti: still want to discuss with you about fsck before pushing more :)12:25
didrocksbut let's do that tomorrow12:25
desrtpitti: you going to the systemd hackfest in brussels?12:26
pittidesrt: (OTP) yes, I will12:30
pittidesrt: will you be on fosdem, too?12:30
desrtyes12:30
desrthopefully will get some kdbus stuff nicely tied up there12:31
willcookedidrocks, cheapo ebay laptop battery update.... wont charge above 60% now.12:41
didrockswillcooke: yeah, so the cheap was really "cheap" :/12:42
willcookequelle surprise12:43
didrockswell, see the positive side: it didn't explode when you plugged it12:43
willcooke:D12:43
didrocks;)12:43
larsuyet12:43
willcookeLOL!12:43
didrocksroh12:44
pittididrocks: I responded to bug 140068212:51
ubot5bug 1400682 in systemd (Ubuntu) "Add xdiagnose fallback when display-manager fails to start" [Undecided,New] https://launchpad.net/bugs/140068212:51
xnoxLaney: i don't see any pictures shared with me on G+... did you use the right account instead of a defunct @canonical.com one?12:53
Laneydon't know, I typed your name in the app ^o)12:53
didrockspitti: agreed, doing this change and closing the bug report then12:55
Laneyxnox: https://plus.google.com/u/0/109160032876474505377/posts that one12:55
xnoxLaney: right - so I see if it I go to your page, but I didn't get a notification about it =/12:56
xnoxLaney: yours is lovely and tall and nice toys - just enough.12:57
xnoxLaney: mine is much shorter than that and more fluffy12:57
Laneyit expanded after we got it home12:57
Laneywanted something skinnier12:57
Laneyi like the fluffiness ^_^12:57
didrockspitti: I guess on the bug report, you wanted to tell "graphical.target.d" or really "display-manager.service.d" (and so, using RequiredBy=)13:02
ochosilarsu: ah right. the version of file-roller i have is 3.12.2-0ubuntu1 (so standard vivid)13:21
ricotzLaney, was it intended to drop "--include-image-data" from update-icon-cache?13:32
didrockspitti: done, tested and uploaded in https://launchpad.net/ubuntu/+source/xdiagnose/3.7.113:32
* didrocks goes for some cycling13:33
=== pstolowski|lunch is now known as pstolowski
pittididrocks: I actually meant display-manager.service.d/, so that we can keep to Wants=13:37
pittididrocks: oh wow, thanks!13:37
didrockspitti: the issue is that we would have graphical.target still active13:38
didrocksI'm unsure we would want that13:38
pittididrocks: you didn't ref/close the bug?13:38
didrocksright now, it's overriding well as a Requires13:38
pittioh, invalid13:38
didrockspitti: yeah, it's not exactly the same approach/component, so didn't retarget it13:38
pittididrocks: anyway, great to see that this .d/ thing works!13:39
didrocksyeah ;)13:39
pittididrocks: now, 218 isn't in vivid yet, but I'll work on that as fast as possible :)13:39
didrockspitti: we can just override display-manager.service.d if you prefer in the future, I'm just not feeling confortable letting graphical.target as active when the graphical server didn't really start13:39
didrockspitti: actually, it's working on 217 as well13:39
didrocks218 just introduce the edit command13:40
pittididrocks: ok, that seems fine13:40
pittididrocks: oh right13:40
didrocksthis override mechanism are how generators are working in /run/system/ actually :)13:40
didrocksI just didn't map into my brain, but yeah, way nicer13:40
didrocksthanks for the suggestion :)13:40
* pitti hgus didrocks13:42
pittiok, I'm off for some hours to do some christmas shopping13:42
* didrocks hugs pitti back13:43
didrocksenjoy! (and good luck)13:43
* didrocks really off to some exercise now13:44
ochosilarsu: another question, if i may, is patching CSD in gnome-calculator also on your list?14:11
dgadomskihello everyone14:13
dgadomskiTrevinho: hey, do you remember the focus storing change (fix for bug #1125442)?14:14
ubot5bug 1125442 in Compiz 0.9.11 "Always Visible and On Top Windows Steal Focus on Workspace Switch" [Low,In progress] https://launchpad.net/bugs/112544214:14
dgadomskiTrevinho: there was a regression caused by the change (bug #1393020), do you have any idea how to differentiate viewport moving caused "manually" from those caused by clicking on the launcher (like in the bug description) on the compiz level?14:16
ubot5bug 1393020 in compiz (Ubuntu) "[regression] "Remember Focus" does not play well with unity launcher so shouldn't be default enabled or better yet the rev should be reverted" [High,In progress] https://launchpad.net/bugs/139302014:16
Trevinhodgadomski: hi14:16
Trevinhodgadomski: mh, no I don't remember a way now... but I could investigate14:17
Trevinhodgadomski: however, Brandon Schaefer (now offline, it will be back in about 2hrs) said me he prepared a fix for that changwe14:18
Trevinhodgadomski: so, if you wait him to join yo could have a chat with him about that14:18
larsuochosi: again, works on unity :)14:19
larsuochosi: http://bazaar.launchpad.net/~ubuntu-branches/ubuntu/vivid/gnome-calculator/vivid/view/head:/debian/patches/git_no_headerbars_in_unity.patch14:19
larsuochosi: probably we should update this to !GNOME as well14:19
ochosiyeah: if (!is_desktop ("Unity"))14:19
dgadomskiTrevinho: great, I will talk to him, thanks!14:19
ochosilarsu: yeah, would be great in order for things to be consistent, at least across desktops14:20
Laneyricotz: that's only in the gtk2 one, right? and I think it's going to be dropped from there too14:20
Laneyso... yes14:20
draven33need some help with compiz in 14.04  i activatewd dual loghin compoz and metacity an can't align the 3d windows on the cube they form outside the cube14:54
ricotzLaney, i meant the gtk3 3.14.5 packaging15:14
Laneywhere do you think it got dropped?15:14
ricotzLaney, iirc I added it in the early gtk3 packaging for gnome3 ppa, and I thought it is needed still15:16
ricotzignore me if it isn't needed anymore and i will drop it as well15:18
MirvI wonder if any core-dev would be happy to ack compiz https://ci-train.ubuntu.com/job/ubuntu-landing-012-2-publish/46/artifact/packaging_changes_compiz_1%3A0.9.12.0+15.04.20141210.2-0ubuntu1.diff ? ...or if I'll ask for better changelog. all of that is related to the line "added support for multi-arch installations"15:50
Mirvmostly the cleanup to remove all debian/tmp/ from *.install could have been mentioned as well15:51
bschaeferdgadomski, hello!16:54
dgadomskibschaefer: hey16:55
bschaeferdid you have some questions about the workspace needing reverting?16:55
dgadomskiI wanted to fix bug #1125442, but Trevinho told me that you were working on some fixes already16:56
ubot5bug 1125442 in Compiz 0.9.11 "Always Visible and On Top Windows Steal Focus on Workspace Switch" [Low,In progress] https://launchpad.net/bugs/112544216:56
bschaeferdgadomski, well i wasn't actually work on a fix atm16:56
bschaeferi fixed a bug your prev merge caused but that was a simple fix. The main other issue would be to not re-focus a cached window when clicking on a launcher icon16:57
bschaeferTrevinho, do you remember how the launcher icon actually focus an icon on a differnet workspace?16:58
bschaeferwe call a workspace switch in unity or compiz?16:58
dgadomskibschaefer, oh, sorry, I pasted wrong link, this was the bug I was thinking about: bug #139302016:58
ubot5bug 1393020 in compiz (Ubuntu) "[regression] "Remember Focus" does not play well with unity launcher so shouldn't be default enabled or better yet the rev should be reverted" [High,In progress] https://launchpad.net/bugs/139302016:58
bschaeferdgadomski, right, i assigned my self that bug just because i reverted your merge16:59
bschaefer(not really a fix though)16:59
Trevinhobschaefer: all unity side should bein PluginAdapter::FocusWindowGroup16:59
* bschaefer looks16:59
Trevinhobschaefer: so, there's no direct request to switch to another vp... it just happens when you ask compiz to focus a window17:00
bschaeferTrevinho, i see, so we can patch that bit of code in compiz i suppose?17:00
bschaeferthat should be in ... screen/window.cpp?17:00
Trevinhobschaefer: I suppose17:00
Trevinhobschaefer: probably on activate window17:01
bschaeferTrevinho, well ideally we need to think of a better way to cache said window17:01
bschaeferdgadomski, but ideally, we need to come up with a way to restore focus on that cached window17:06
bschaeferif and only if its on that workspace and we are not focusing a new window on that workspace17:06
dgadomskibschaefer: I was debugging this and I saw that moveViewport (where the window is cached) is called in both cases and I was not able to determine how to differentiate between "manual" viewport changing versus the launcher-initiated17:07
bschaeferright, that was a worry of mine as well... though we might have to figure out a better way to cache the window17:08
bschaeferdgadomski, theres a place somewhere in compiz17:08
bschaeferthat focuses a window17:08
bschaeferthats what the launcher uses, and in there it switches the viewport17:08
bschaeferso we can detect when the launcher is being used, or rather we can detect when a window is being focus (hopefully?)17:09
* bschaefer looks at compiz17:09
bschaeferTrevinho, dgadomski looks like the wall plugin causes the window to be moved17:21
bschaeferyou can test that out by disabling the plugin then attempting to click on a launcher icon... strange17:21
dgadomskibschaefer: what do you mean by "causes the window to be moved"?17:23
bschaeferdgadomski, err not window to be moved but workspace/viewport to be moved17:25
dgadomskibschaefer: yeah, and if you change the wall to e.g. cube you are still affected by this issue, so this needs to be fixed below the plugin layer17:26
bschaeferdgadomski, right, but we need to ensure the plugins work as well17:26
bschaeferdgadomski, the issue being that the unity launcher is using the wall plugin (or rather the wall plugin is auto moving workspaces to the "wanted" focused window)17:27
* bschaefer might not be making much sense17:27
dgadomskibschaefer: I see your point, the unity launcher also works in the same way with cube, so probably cube also moves viewports while focusing a window outside current viewport17:28
bschaeferdgadomski, right, so if those plugins are disabled17:29
bschaeferall is well17:29
bschaeferbut we have the wall enabled, meaning it'll attempt to focus a window17:29
bschaeferon its viewport17:29
bschaeferso we need to add a case in that plugin to be handled17:29
bschaeferdgadomski, then i think we can get the fix re-merged :)17:29
bschaefercompiz is ... like a N headed monster where N is the number of plugins enabled...17:29
dgadomskibschaefer: Trevinho mentioned earlier that there may be a way to recognize the reason behind the moveViewport inside compiz17:31
larsubschaefer: lol, more like N = 2^n_plugins17:31
bschaeferlarsu, haha... true17:31
willcooke+RAND(10)17:31
bschaeferTrevinho, ^17:31
bschaefern^n17:32
Trevinhodgadomski: ehm? :P17:32
bschaeferi like to imagine the best way this could turn out... We need to have ideally on each workspace a last focused window17:33
dgadomski"<Trevinho> dgadomski: mh, no I don't remember a way now... but I could investigate"17:33
bschaeferwhen we have landed on said workspace we need a way to check "Did we just change focus to get here?" vs "We just got here, change focus to last focused window"17:33
bschaeferthose two cases are the tricky part, but what "should" happen17:34
dgadomskiTrevinho: so if you don't remember a way not it means that there probably was a *WAY* :)17:34
bschaeferis as soon as the launcher focuses the window, the last focused window for that workspace should be updated17:34
bschaeferwhich would resolve our issues17:34
Trevinhodgadomski: oh, yeah... there is... :)17:34
bschaeferTrevinho, haha17:34
bschaeferdgadomski, so we need to look at making sure we update the last focused cached window per workspace17:35
bschaeferif we can do that, then when the launcher focuses a window on any workspace, that workspace updates its last focused window and we are set :)17:35
Trevinhobschaefer: it's very likely that each workspace plugin (like wall or cube) implements that switch17:35
bschaeferTrevinho, i greped17:35
bschaeferTrevinho, but it doesn't matter, we can get around this under the hood17:35
bschaeferTrevinho, all we have to check for is FOCUS OF NEW WINDOW17:36
bschaeferwhen a new window is focused update the last focused window on the workspace its on17:36
Trevinhobschaefer: I see..17:36
bschaeferso when we move to that workspace it will always get focus17:36
bschaeferthat sound reasonable?17:36
bschaeferas...other wise im just imagining a mess of code to handle all strange edge cases17:36
Trevinhobschaefer: yes, that seems correct17:39
bschaeferso instead of updating the last focused window each time we move workspace17:39
bschaeferthe details could be hammered out17:40
dgadomskibschaefer: are you thinking about caching focused window e.g. in CompWindow::activate?17:42
bschaeferdgadomski, not 100% sure where but that might be a good place17:42
bschaeferdgadomski, we just need a way to cache a window per workspace17:43
bschaeferand when a new window is focused to update that cache array17:43
=== rickspencer3_ is now known as rickspencer3
* didrocks waves good evening and good night18:05
=== alan_g is now known as alan_g|EOD
dgadomskibschaefer: caching in CompWindow::activate does not work as expected, I will continue debugging tomorrow morning18:40
bschaeferdgadomski, sounds good, i think the screen will have to know about it18:41
bschaefervs the window18:41
bschaefergood luck!18:41
dgadomskibschaefer: thanks! take care18:42
bschaeferyou to!18:44

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