/srv/irclogs.ubuntu.com/2016/03/10/#ubuntu-desktop.txt

robert_ancellGunnarHj, I'm not sure about other desktops, but when I looked into fixing the XDG_CURRENT_DESKTOP usage it seemed that very few (if any) of them make sense anyway.00:42
robert_ancellSo if we fix those first we'll be able to see what remains.00:42
robert_ancellI think the answer is if it is being used by other desktops it still makes sense to show all the options00:42
=== thumper-dogwalk is now known as thumper
GunnarHjrobert_ancell: Of course, as long as we aren't sure that it won't be used by other desktops, it makes sense to keep the options. I suppose that it's a little late in the cycle to figure it out. I'll try this weekend to make an MP which fixes the tests so they handle cases like "GNOME-Flashback:Unity" properly. And also make sure that u-c-c does not override an existing XDG_CURRENT_DESKTOP value, but only sets "Unity" as a fallba00:55
GunnarHjck if XDG_CURRENT_DESKTOP is empty for some reason.00:55
robert_ancellGunnarHj, I think it's been set because some of the panels have OnlyShowIn=Unity. Which seems to me like they should just be fixed to either have a list of desktops or u-c-c should show them regardless of what desktop it is in.00:56
robert_ancellI'm fixing GTK+ at the moment00:57
GunnarHjrobert_ancell: If I understand it correctly, the panels will be shown in the case of OnlyShowIn=Unity as long as Unity is one of the elements in XDG_CURRENT_DESKTOP's colon separated list. So in the example I mentioned (GNOME-Flashback:Unity) the panels should be shown. (But I haven't tested yet.) In that case, it's the remaining XDG_CURRENT_DESKTOP tests within u-c-c which need to be modified (split on ":" etc.).01:04
=== JoseeAntonioR is now known as jose
robert_ancellattente, ping?03:00
robert_ancellattente, oh, now I get it. No problem.03:01
=== tsimonq2alt is now known as tsimonq2-
=== tsimonq2- is now known as tsimonq2
hikikohi05:53
=== tjaalton_ is now known as tjaalton
=== mfisch is now known as Guest38484
didrocksgood morning06:46
=== Guest77642 is now known as fredp
=== fredp is now known as Guest77696
andyrockgood morning08:19
darkxstGunnarHj, late reply to your XDG_CURRENT_DESKTOP comment, the desktop files are not an issue, yes OnlyShowIn=Unity works fine with GNOME-Flashback:Unity08:47
pittiseb128: bonjour ! comment รงa va?08:51
seb128salut pitti, รงa va bien et toi ?08:51
pittiseb128: je vais mieux, merci08:52
seb128super !08:52
pittiseb128: gnome-desktop got removed in Debian; we still have one rdepends, banshee08:52
seb128how did Debian fix that?08:52
pittihttps://tracker.debian.org/news/71255608:53
pittiseb128: that's a newer upstream version, though08:54
seb128right, i was going to say08:54
pittiseb128: (doing process-removals today, and it's depressing..)08:54
pittiseb128: so, do we want to merge and follow along? or keep that old stuff?08:54
seb128(why? less to maintain is good no?)08:54
pittiseb128: well, once you can *actually* remove packages, yes :) but I'm running into countless cases of "one rdepends left in Ubuntu which hasn't been merged/updated in ages"08:55
pittiI did a few, but this is a ratsnest :)08:55
seb128I've no idea about banshee, I never really used it and didn't look at it for years08:56
pittiright, i. e. do we still actually want to keep it}08:56
pitti?08:56
seb128if the new version is only in experimental there might be a reason08:56
seb128I assume we do08:56
pittiseb128: ah no, unstable has https://tracker.debian.org/news/71278208:56
seb128having things in universe doesn't cost us much, we can as well keep it08:57
pittiwhich also dropped the deps08:57
seb128nice08:57
pittiwell, it's still growing tech debt08:57
seb128let's just do that then08:57
pittiVersion: 2.9.0+really2.6.2-3ubuntu108:57
seb128it's not like universe was close from not have all sort of old unmaintained things08:57
pittiso at some point we wanted to go back, but I don't have the details any more08:57
seb128having*08:57
seb128or had ever been08:57
pittiat some point we had ubuntuone integration and all that, perhaps that's all obsolete08:57
seb128yeah08:58
pittiDowngrade to 2.6.1 -- 2.9.x is unstable and 3.0 won't arrive in time for08:58
pitti    release08:58
pitti(09 Feb 2014)08:58
seb128if somebody wants to update it/clean patches they can go for it08:59
pittiI suppose that's still true :)08:59
pittiwell, that's the thing -- nobody does it08:59
seb128I wouldn't spend efforts on universe things08:59
seb128right08:59
pitti(and no, I've already touched too many KDE packges to pile up even more TIL)08:59
seb128like for some other 10k packages in universe08:59
seb128I would just ignore it08:59
willcookemorning all08:59
seb128hey willcooke08:59
willcookeTheMuso, be with you in 508:59
=== maclin1 is now known as maclin
pittiseb128: this all started with me cleaning up some wx2.8 stuff for removing gst 0.10; shouldn't have done this :)09:00
TheMusowillcooke: No worries.09:00
* TheMuso waves to everybody else. :)09:00
* pitti waves back to TheMuso09:00
seb128pitti, good man, trying to fight the universe ;-)09:01
Laneymorning!09:01
pittihey Laney09:01
seb128hey TheMuso09:01
seb128hey Laney09:01
pittiseb128: do I look green and really musculous yet?09:01
darkxstHey Laney seb128 willcooke09:02
seb128we need a sprint or UDS so I can tell you :p09:02
seb128hey09:02
Laneywhat's up?09:02
pittihttp://vignette1.wikia.nocookie.net/vsbattles/images/8/88/AoU_Hulk_0004.png/revision/latest?cb=20150608160344 โ† me running process-removals09:03
seb128pitti, going to clean up libgnome gnome-vfs etc from the archive next, right? ;-)09:04
* seb128 hides09:04
pittiseb128: the cleaning isn't the problem :)09:04
darkxstenjoying the wet weather for a change ;)09:04
darkxstgood time to do a pilot shift when you can't physically leave the house ;)09:05
seb128indeed!09:08
pittichecking for Mono 2.0 GAC for Mono.Cairo.dll... not found09:08
pitticonfigure: error: missing required Mono 2.0 assembly: Mono.Cairo.dll09:08
pitti^ awesome, banshee is FTBFS09:08
pittiso at some point doko will haunt you anyway :)09:09
seb128pitti, how did you manage to get a vivid upload to xenial?09:09
seb128"xchat-indicator (0.3.11-0ubuntu7) vivid; urgency=medium"09:09
seb128"  * Add hexchat-indicator."09:09
seb128that's confusing09:09
pittiwut?09:09
seb128on xenial-changes09:09
pittiah sorry, part of cleaning up xchat, just noise09:10
willcookeTrevinho, hikiko - Shadows vs CSD fixes....10:10
willcookeSo hikiko is close to landing a fix for shadows in Compiz which will fix it for all apps not just Gtk10:10
willcookehikiko, you think it will be finished today?10:10
willcookeand how much work is there still to do on E Zoom?10:11
hikikoI hope so willcooke10:11
hikikoon ezoom I am blocked at an issue10:11
hikikotransformation stack10:11
hikikoI use compiz transformations with nux but there's something that doesn't work10:11
hikikoif I fix tha10:11
hikikothat10:11
hikikothen it's like 2 days work10:12
hikikothe trick there is to pass the compiz ezoom transformation to all the nux components10:12
hikikobut for some reason some of them misbehave10:12
Trevinhohikiko: as I said, if you can figure out that ezoom thing, it would be cool... As it helps in a11y. As for gtk apps I think we're all set now. Not the best fix, but it's good enough and we can then try to fix it better later (and in case SRU to xenial), but the one we've now to me is fine.10:12
hikikoand I am looking for that "reason" :.10:12
hikiko:/10:12
hikikowell, the thing is that the proper fix is almost done and will work with all transparent windows10:13
hikikoif qt decides to have transparency10:13
hikikothat will work too10:13
hikikoyour hack is cool but works only with gtk10:13
hikikoso, depends on how much we want to support everything10:14
hikikoor just gtk10:14
Trevinhoyeah, but it's all we need for now... as transparent windows are in general something that should care about their shadows by design.10:14
TrevinhoI've seen a branch on compiz side, which is only gtk related, but we can't do that without changing the whole compiz geo system (which is something we can't do now)10:15
Trevinhoinstead if you can extract alpha from windows and get its shape it would be cool, but still we can't apply that to all windows... Since we can't be sure that a window is already providing a shadow (and thus double-shadowing it). I think to cairo-dock for example10:15
hikikoyou mean that we don't need a general solution that supports all the windowxs?10:16
hikikowhat do you mean?10:16
hikikowe have 1- rectangular windows (supported)10:16
hikiko2- shaped windows (supported)10:16
hikiko3- windows that are not shaped - not rectangular and have alpha (under construction)10:17
hikikohow it comes and we double shadow them?10:17
TrevinhoYeah, 3 is fine, but as I said if you try apps like cair-dock, or many other alpha windows around, they already provide their own shadow. Since being alpha is a contract with compositor that says: "I'm caring about my own shadows". So, for gtk ones or anyone supporting us, we can say "Ok, I'm alpha, but I still want your shadows". we unfortunately can't10:18
Trevinhoapply that to any window, without risking regressions10:18
TrevinhoAnyway, if you can get 3 on is nice, but I think we can now delay that compared to ezoom.10:19
hikiko1 day? :p10:19
TrevinhoIf you have some code I can look at, I might help anwyay10:19
Trevinhook10:19
willcookeok, hard dead line is midday UTC tomorrow.  If it's not done, then work stops there and we move to E Zoom.  How's that sound hikiko, Trevinho?10:21
hikikofine for me10:21
Trevinhok10:21
willcookethe clock is ticking . . . . . .10:25
willcooke;)10:25
willcookegood luck!10:25
muktupavelshikiko, https://code.launchpad.net/~albertsmuktupavels/compiz/add-gtk-frame-extents-to-net-supported/+merge/25730310:39
Trevinhomuktupavels: we can't have that without proper geometry rework... And it's too tricky to get it done with few lines unforunately10:49
Trevinhoanyway....10:50
hikikolol https://code.launchpad.net/~hikiko/compiz/compiz.black-dots-fix10:50
* Trevinho drives back to Florence10:50
hikikook he did it first I approve it in a sec10:50
Trevinhohikiko: no, please...10:50
hikikowhy not?10:50
hikikoit doesn't seem to damage anything else?10:51
Trevinhohikiko: enabling that, changes the way gtk exposes their area. And it creates a bigger input area which breaks things for us10:51
Trevinhohttps://code.launchpad.net/~albertsmuktupavels/compiz/add-gtk-frame-extents-to-net-supported/+merge/257303/comments/68081210:51
muktupavelsTrevinho, I posted link because hikiko branch had same fix and it had some comments...10:51
Trevinhoyeah, hikiko see the comments10:52
hikikosec10:52
Trevinhoso without fixing the compiz geometry we can't land that.10:52
Trevinhoand removing these borders from compiz geo seems to be harder than expected. I spent more than a week on that, but it's problematic10:53
hikikoI know what you mean10:53
hikikothat the pixmap10:53
hikikohas some space for decorations around10:54
hikikowell we can just clip the image to remove it isn't it?10:54
Trevinhohikiko: yeah, but I failed in doing that10:57
Trevinhoanyway, for removing the black dotsit's not a problem patching gtk and using a rgba surface in unity, so, we can live without that change10:59
* Trevinho has to go10:59
muktupavelsTrevinho, hikiko: does not gtk+ add bigger input area only when window-frame has margin != 0 and/or box-shadow != none. If Ambiance does not add that css then should not be problem.10:59
hikikoI was going to clip the pixmap on unity anyway because I need that for the shadows10:59
hikiko+1 muktupavels10:59
hikikoI think so, I'll test that11:00
=== Guest77696 is now known as fredp
=== fredp is now known as Guest86230
=== Guest86230 is now known as fredp
=== fredp is now known as Guest12079
bregmawillcooke, did you get that ask from the ubuntu-docs team about desktop change notes for 16.04?12:48
willcookebregma, I didn't, but strangely enough I sent an email out about 10 mins ago to start gathering release notes12:50
willcookedavmor2, cyphermox - do you know - are we supposed to inhibit the screensave during a dist-upgrade?12:51
willcookedavmor2, I forgot to disable the screensaver, and not I'm stuck at the prompt again.  Sigh.  Start again.12:52
willcooke*now12:52
davmor2willcooke: there was mention of it I think in one of the logs12:52
willcookebregma, oh, maybe I misunderstood the question?12:53
bregmawillcooke, sound like you're on it, just crossing the task off my list12:54
willcookebregma, oki cool.  Very strange timing though, like I thought of it minutes before you mentioned it.  Maybe be have a physic link?12:55
willcookewhat's that?  Take a hammer from the shed?12:55
davmor2willcooke: 2016-03-10 11:46:23,912 DEBUG Removing 'xscreensaver' (ubuntu-desktop PostUpgradeRemove rule) I'd say that was a pretty brutal way to ensure that the screensaver didn't kick in :)12:56
willcookenah, that wont stop it running though12:57
willcookethere is a "nice" way to inhibit it12:58
willcookebut perhaps we shouldnt have to12:58
=== francisco is now known as Guest63468
=== alan_g is now known as alan_g|lunch
=== hikiko is now known as hikiko|ln
=== alan_g|lunch is now known as alan_g
cyphermoxwillcooke: screensaver or not should not make a difference in the upgrade, things shouldn't crash as a result of the upgrade.14:44
willcookecyphermox, oki14:44
willcooketrying again with screensaver etc turned off14:45
willcookejust to see if I can find the exact moment it dies14:45
willcookeah14:48
willcookeI wonder if this is why we're not getting GUI prompts davmor214:49
willcookeSomething about debconf needs a screen at least 13 lines high, blah blah, falling back to readline14:50
willcookeI'm running the install on a vbox14:50
willcookewith a poor resolution14:50
willcookethat might explain that issue anyway.14:50
=== hikiko|ln is now known as hikiko
cyphermoxwillcooke: readline should be "sufficient" and the prompts would then show in the attached terminal window14:57
willcookecyphermox, yeah, thats what happens14:57
cyphermoxI mean, I see that the install on davmor's machine i stuck waiting for input for ssl, but I don't think that's the issue -- the no X is more of a problem14:57
cyphermoxthat said, I'm logged on SSH on dave's machine, and I'll try to debug compiz live maybe?14:58
cyphermoxdid we have a crash dump for it yet?14:58
willcookedont think so14:59
willcookeokay15:07
willcookethe point at which the upgrader UI crashes is about here...15:08
willcookePreparing to unpack .../xfonts-base_1%3a1.0.4+nmu1_all.deb ...15:08
willcookeUnpacking xfonts-base (1:1.0.4+nmu1) over (1:1.0.3) ...15:08
willcookePreparing to unpack .../xfonts-scalable_1%3a1.0.3-1.1_all.deb ...15:08
willcookeUnpacking xfonts-scalable (1:1.0.3-1.1) over (1:1.0.3-1) ...15:08
willcookePreparing to unpack .../xinit_1.3.4-3_amd64.deb ...15:08
willcookeUnpacking xinit (1.3.4-3) over (1.3.2-1) ...15:08
willcookePreparing to unpack .../xinput_1.6.2-1_amd64.deb ...15:08
willcookeand then the upgrade seems to stop here:15:09
willcookeSetting up python-debtagshw (2.0.1ubuntu6) ...15:09
willcookeSetting up ureadahead (0.100.0-19) ...15:09
willcookeSetting up perl-modules-5.22 (5.22.1-8) ...15:09
willcookeand it kicked me out of one of my SSH sessions15:09
cyphermoxok15:10
cyphermoxthat was using the graphical release-upgrader?15:11
willcookethat was using "update-manager -d"15:11
cyphermox(ie. not do-release-upgrade -d in a terminal)15:11
cyphermoxok15:11
cyphermoxI probably wouldn't make a difference, but just in case15:11
willcookeso I had top running in another ssh session15:11
willcookeand it kicked me out of that session at about  15:05:5815:12
willcookeI still have one ssh session open some hout15:12
willcookehow15:12
willcookeand in syslog15:12
willcookeMar 10 15:05:22 test14to16 ModemManager[28639]: <info>  Couldn't find support for device at '/sys/devices/pci0000:00/0000:00:03.0': not supported by any plugin15:12
willcookeMar 10 15:05:57 test14to16 dbus[394]: [system] Reloaded configuration15:12
willcookeMar 10 15:06:00 test14to16 dbus[394]: message repeated 2 times: [ [system] Reloaded configuration]15:12
willcookeMar 10 15:06:00 test14to16 dbus[394]: [system] Activating service name='org.freedesktop.UDisks2' (using servicehelper)15:12
willcookeMar 10 15:06:00 test14to16 udisksd[29495]: udisks daemon version 2.1.7 starting15:12
willcookeMar 10 15:06:00 test14to16 dbus[394]: [system] Activating service name='org.freedesktop.PolicyKit1' (using servicehelper)15:12
willcookeMar 10 15:06:00 test14to16 polkitd[29500]: started daemon version 0.105 using authority implementation `local' version `0.105'15:12
willcookeoh, and I had disabled the lock and the screensaver before running this upgrade15:14
willcookelast thing in dpkg.log:15:15
willcooke2016-03-10 15:06:26 status installed libgtkmm-3.0-1v5:amd64 3.18.0-115:15
willcookeummmm15:18
willcookeI wonder15:18
willcookeapt.log:15:18
willcookeMarkDelete python3.4 [ amd64 ] < 3.4.3-1ubuntu1~14.04.3 > ( python ) FU=015:18
cyphermoxwillcooke: gtkmm is not typically something that would break everything15:19
willcookeI wonder if the installer has removed python from underneath the installer?15:19
cyphermoxthough a GLib / Gtk upgrade at the wrong time might cause crashes15:19
cyphermoxwillcooke: I thought all of X crashed?15:20
willcookeoh, right yeah15:20
willcookeyeah my X session is dead now15:20
willcookenothing in /var/crash15:21
willcookexorg log:15:21
willcooke[  6323.283] (EE) FBDEV(0): FBIOBLANK: Invalid argument15:21
willcooke[  6323.297] (EE) FBDEV(0): FBIOBLANK: Invalid argument15:21
willcookerestarted lightdm15:24
willcookeinit: Error while reading from descriptor: Bad file descriptor15:25
willcookeinstead of a greeter15:25
GunnarHjdarkxst: Thanks for confirming about multidesktop XDG_CURRENT_DESKTOP values and .desktop files.15:34
davmor2willcooke, cyphermox: I think it just hates us :'(15:53
willcooke:((15:54
davmor2oh Laney looks like you might actually be able to detach the hdmi lead from the xps now without kernel panicking the system \o/ I don't want to try it again incase it was pure fluke :D15:55
=== JoseeAntonioR is now known as jose
=== JanC_ is now known as JanC
Laneydavmor2: :-o16:48
attenteLaney: hey, is there some way to ask someone (aptdaemon/packagekit) over dbus to do an apt update? i tried org.debian.apt.RefreshCache but that didn't populate /var/lib/app-info/17:10
Laneyattente: I'm not sure, I would ask juliank in #ubuntu-devel - he's more likely to know17:16
attenteok, thanks17:17
Laneymaybe that function does it in some different way that doesn't run the Post-Update-Success hook17:17
attenteLaney: it's hard to tell what that function does at all. it seems to do nothing afaict17:18
Laneywhat does (e.g.) software-properties use?17:19
attenteLaney: ok, desrt just helped me out here. apparently i have to create a transaction and call GetUpdates on it17:31
Laneycool17:33
Laneysorry I don't know those APIs very well & I'm in the middle of my quest to kill juju17:33
desrtLaney: we kinda want to expand your original bug17:42
desrtand make it so that we do a refresh on startup any time we detect that the last time 'apt update' was run is more than 1 day ago (or never)17:42
desrtit turns out that the timestamp on /var/cache/apt is a pretty good place to start -- but it's a bit complicated if this directory was created during install and nobody actually ran 'apt update' yet17:43
Laneydesrt: as long as you fix the bug then you can expand whatever you want17:43
Laneythe rest is a matter for robert_ancell17:43
desrtit would also be ideal if we could plumb this through packagekit...17:43
desrtwell, fixing it is easy enough now that we found the right set of APIs17:44
Laneysehr gut17:44
desrtbut "first install and user didn't apt update yet" is not the only place where apps may be missing17:44
desrtcould be nobody ran "apt update" in months and apps got added in the meantime17:44
Laneyupdate-manager is updating for you17:45
desrtjust not right after install :p17:45
Laneyright17:45
desrtcan we fix that? :D17:45
Laneynot for the general 'sudo apt install gnome-software' case17:46
desrti ate you17:46
desrtactually, i'm kinda hungry17:47
LaneyI think that ximion said that g-s already does have this timed refresh thing17:47
Laneyjust not the always-do-it-if-empty case17:47
desrtya... i think it stores the timestamp int he user session tho17:47
desrtwe could do the same here....... just seems ugly to have 100 copies of timestamps stored per user for the same thing (which is really system-wide)17:47
Laneyfor apt it is...17:48
desrtlike, even separate copies of it stored for the same user, even17:48
desrttrue.17:48
* desrt snaps her fingers17:48
Laneyanyways!17:48
desrtya.  a bit of a sidetrack here17:48
LaneyAs long as updaters have it working17:48
desrtseems that the easiest thing to do now is to fix the bug as filed, using the dbus api17:48
Laneyany extra coolness is cool too17:48
ximionLaney: that timed refresh doesn't happen though if you patch out the g-s daemon mode17:49
desrtit's cool to be cool!17:49
desrtkaj mojoseco mojosas17:49
Laneyximion: ok I didn't track what they did on the client side to be honest17:49
ximionLaney: AFAIr removed that feature ^^17:49
ximion(but better check for that, don't take my word for it)17:50
attentewe disabled daemon mode except in the case where it's explicitly run with --gapplication-service17:51
desrtis it DBusActivatable?17:51
attenteyes17:51
desrtso then it's always run with --gapplication-service...17:52
willcookeg'nigth all18:08
willcookenight18:08
Laneyme too, see you18:12
seb128night Laney & co18:15
desrtgood night europeans19:46
robert_ancelltkamppeter, I'm looking at the unity-settings-daemon code and the print notification plugin seems completely disabled. Does that make sense? I'm going to drop all the code for it if that's the case.20:58
TrevinhoMh, with recent changes I did, I'm about to think that all the apps we patched to get headerbar would just look better as native... ๐Ÿ™Š21:06
Trevinhostarting from https://code.launchpad.net/~3v1n0/ubuntu-themes/toolbar-mode-headerbar-gradient/+merge/288701 (and related gtk patch)21:06
robert_ancellTrevinho, yeah, they look nice now21:07
Trevinhorobert_ancell: about that.... can you give a look at the gtk branch? ๐Ÿ˜ƒ21:07
robert_ancellTrevinho, which branch?21:08
Trevinhorobert_ancell: https://code.launchpad.net/+branch/~3v1n0/gtk/unity-maximized-headerbar-buttons-hide21:08
robert_ancellTrevinho, you have an unused local variable "unity_environment" in the patch21:09
TrevinhoI do use it...21:09
robert_ancellTrevinho, you use priv->unity_environment21:09
robert_ancelloh, duh21:10
robert_ancellreading the patch wrong21:10
Trevinho:)21:10
robert_ancellI thought that was at the top of the function21:10
robert_ancellReviewing patches really feels wrong21:10
robert_ancellIt's like we have a better system for all this... version control?21:10
TrevinhoActually I should probably change it so that it only hides the title if != from toplevel window title21:10
robert_ancell;)21:10
TrevinhoMaybe somneone could set an headerbar title that is different...21:11
robert_ancellTrevinho, yeah, I was thinking that might even be likely. Do HeaderBar apps even set the traditional title?21:12
robert_ancellAlso, the placement of the title is probably significant (i.e. being centered)21:13
robert_ancellOr does Unity handle that21:13
Trevinhorobert_ancell: placement shouldn't be a problem,...21:50
Trevinhorobert_ancell: I've updated the patch to care about window title, though.21:51
tkamppeterrobert_ancell, I did not really have a look at the print notifications on the desktop. CUPS produces notifications on D-Bus which I have already made use of in cups-browsed (making use of some code snippets of larsu's printer-indicator).22:20
tkamppeterrobert_ancell, I think it is a good idea to have some kind of notifications, so that users can access their jobs and know which printers are currently free.22:21
robert_ancelltkamppeter, I think g-s-d had some notification system that we disabled, so it's no longer required in u-s-d22:21
tkamppeterrobert_ancell, could be either the printer indicator showing a printer at the top right and having a menu to reach the print queues or like in mpt's printing design a printer in the launcher on the left, doing more or less the same thing.22:23
tkamppeterrobert_ancell, if both g-s-d and u-s-d are running, then only one needs to handle printing notification (to avoid duplicates), so if both contain the code, it can be dropped in one.22:24
tkamppeterrobert_ancell, and g-s-d is probably the better place as it is running in both Unity and GNOME desktops.22:24
robert_ancelltkamppeter, u-s-d checks if it is running in Unity and disables the code. I just wanted to check you didn't expect u-s-d to provide this functionality22:25
tkamppeterrobert_ancell, I did not code anything of the print notification functionality for the desktop, we only should make sure not to duplicate functionality to ease maintaining the code and make sure that all users get notifications, both of GNOME and of Unity.22:27

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