/srv/irclogs.ubuntu.com/2018/04/13/#ubuntu-desktop.txt

Trevinhojbicha: against which branch I should base?00:00
jbichadebian/master00:00
Trevinhomaster?00:00
Trevinhook00:00
Trevinhohaving salsa there helps a lot, I already use it with bzr anyway00:00
Trevinhobut still cause me to do a 2nd pass00:00
Trevinhowow, for g-s there's not even a 3.28 releas or what?00:03
Trevinhofor sure there's not the branch00:03
Trevinhohmhmmh00:08
Trevinhohow is it that https://gitlab.gnome.org/GNOME/gnome-shell/commits/master has not https://salsa.debian.org/gnome-team/gnome-shell/commit/6b2ebee26c7053f4c45cab05b208770c96845479 ?00:08
jbichathat's a merge commit00:10
jbichaLan_ey briefly mentioned the upstream/latest branch in his gvfs demo00:11
jbichagit-buildpackage creates a merge commit for every upstream tarball imported00:12
jbichaand there is another merge commit to get the upstream/ branch into the debian/master branch00:13
jbichathis is the upstream 3.28.0 release https://gitlab.gnome.org/GNOME/gnome-shell/tags/3.28.000:14
jameshrobert_ancell: looking.01:22
jameshrobert_ancell: I gave the new gnome-software branch a spin on a clean xenial VM, and it worked as intended for the libreoffice case04:08
robert_ancelljamesh, ok, good! so the tester didn't get it right? Perhaps they didn't restart G-S04:09
jameshI'll restart the VM and give it another shot just to make sure04:10
jameshit still launches the main libreoffice command04:16
jameshnot restarting gnome-software seems like a definite possibility04:16
robert_ancelljamesh, thanks for testing that05:04
seb128good morning desktopers05:09
jibelgood morning05:23
jibelsalut seb12805:24
dufluMorning seb128, jibel05:35
jibelHi duflu06:16
seb128lut jibel, duflu, how are you?06:17
dufluseb128, going well. You?06:17
seb128I'm doing ok, the cold is a bit better and it's friday!06:18
jibelseb128, I'm good, thanks06:19
didrocksgood morning06:25
dufluHey didrocks06:54
=== pstolowski|afk is now known as pstolowski
didrockshey duflu07:16
didrocksjamesh: didn't know about build-snaps, thanks for pointing it out!07:19
jameshdidrocks: it doesn't look like it has hit the documentation on the website yet.  I heard about it at the Snapcraft Summit07:19
didrocksyou, insider! :-)07:20
didrocksI'll clean up that logic in gtk-common-themes and as well the crafted CI in communithemes07:21
didrocksthanks ;)07:21
didrocksjamesh: oh, as it's not documented, do you know if build-snaps enable you to select the channel?07:29
didrocksjamesh: remember that we build from the edge channel07:29
jameshlets see07:30
* didrocks looks into snapcraft source07:30
jameshdidrocks: looks like communitheme/edge should do it07:31
jameshlooking at install_snaps() in snapcraft/internal/repo/snaps.py07:31
didrocksI was in /usr/lib/python3/dist-packages/snapcraft/internal/errors.py:        '`build-snaps: [patchelf/latest/edge]` to the failing part in your '07:31
didrockswhich seems to confirm :)07:31
didrocksjamesh: I'll give it a try, thanks for the suggestion again!07:32
didrocks(for reference for those interested, it's communitheme/latest/edge)07:36
seb128hey jamesh didrocks07:44
didrockshey seb128!07:45
didrocksjamesh: pushed those changes with a testbuild + content inspection07:46
jameshhi seb12807:46
jameshdidrocks: looking07:46
oSoMoNgood morning desktoppers07:57
didrockshey oSoMoN!07:58
seb128lut oSoMoN, en forme ?08:00
oSoMoNsalut didrocks, seb12808:00
oSoMoNla forme, et vous?08:00
oSoMoNseb128, ça va mieux?08:01
didrocksoSoMoN: ça va :)08:01
seb128moi un peu, ça commence à passer,  c'est au tour du petit d'être bien pris et avec la fièvre en plus :/08:01
Laneyhey08:02
Laneyhappy friday (I hope it is friday)08:02
oSoMoNseb128, bon courage, c’est pas rigolo quand c’est les petits qui sont malades :/08:03
oSoMoNLaney, I hope so too!08:03
seb128oSoMoN, 'ci!08:03
seb128happy friday Laney! how are you? ready for the w.e ;)08:03
seb128well, still some work in between but you know what I mean!08:04
Laneyhey seb12808:04
LaneyI caught your cold over IRC I think08:04
seb128:-(08:04
seb128get better!08:04
didrocksmorning Laney08:04
Laneyhey didrocks08:04
Laneyhow are you both?08:04
didrocksLaney: you infer that seb128 should stop talking to not contaminate us? :p08:05
seb128I'm slightly better, hopefully back on shape tomorow08:05
didrocksrather good, finally cold is almost over for me as well08:05
Laneyhigh fives08:06
willcookeo/08:08
didrocksfriday willcooke!08:09
willcookewoo08:09
willcookefeels like it's been a long week08:09
oSoMoNhey willcooke08:10
oSoMoNhow was the homemade pizza?08:10
seb128hey willcooke08:10
willcookeoSoMoN, actually, quite good08:12
willcookeOne sec08:12
willcookeMy internet is bad again today, trying to share via Telegram but it's just sat there retrying08:14
willcookeGetting new internets on Monday, so I wonder if they are doing some work on my line08:14
willcookeAnyway, the important this, pizza..08:14
seb128right, blame it on the internet when you are using an Apple device :p08:15
didrocksThe real one correct? American pizza ofc?08:15
didrocks(f r i d a y)08:15
willcookeI made the sauce earlier in the week for pasta, and then left it to reduce on the hob all day yesterday.  Has a little sharp, but worked really well on the pizza.  Made the dough at lunchtime, let it rise, knocked it back and put in the fridge wrapped in film.  Where it continuted to expand and pushed everything off the shelf08:15
willcookePut the oven on as hot as it would go and put a stone thing in there, and yeah, 10 mins or so and it was really tasty08:16
didrocksbe careful when you put it in the fridge, we left some home made pizza pasta a week once, and didn't notice it was really expanding like crazy08:16
willcookeyeah!  I thought the cold would stop it, but no08:16
didrocksonly freeze does :)08:17
didrocksfreezer*08:17
didrocksshouldn't have used the team "freezer" with seb128's cold :p08:17
willcookeI had loads of dough left, so I've frozen that.  Wonder if it will work? Will I still be able to eat it once it's defrosted08:17
didrockswillcooke: it does, this is what we generally do (and keep it frozen for a couple of months, if not more)08:18
willcookecool!08:18
willcookethat will make it easier next time08:18
didrocksyeah, we do like 4-6 parts of dough, and then, it's easy to do a quick pizza in the week-end :)08:18
willcooke\o/08:18
dufluoh, morning oSoMoN, Laney, willcooke08:19
willcookeahoy duflu08:19
oSoMoNhey duflu08:20
didrockshum, snapcraft --help doesn't list all options… weird for things that are supposed to be autogenerated08:43
willcookeRemoving the ubuntu-wallpapers package stops the desktop from starting09:50
willcookehm, maybe that's not what caused it09:51
willcookebah, I've broken this machine now.  Will just reinstall, and see if I can break it again09:51
willcookeLaney, typo in this commit message?  https://bugs.launchpad.net/ubuntu/+source/ubuntu-wallpapers/+bug/1762722/comments/3  (says 17.10) or am I missing somthing?10:00
ubot5`Ubuntu bug 1762722 in ubuntu-wallpapers (Ubuntu) "Ubuntu 18.04 LTS Free Culture Showcase community wallpapers" [Undecided,Fix released]10:00
Laneyyes10:03
Laneyit doesn't do anything though10:03
Laneyhttps://bazaar.launchpad.net/~ubuntu-art-pkg/ubuntu-wallpapers/ubuntu/revision/24310:05
willcookethx Laney10:06
Mirvtypo in "typp", unless intended :)10:07
Laneyit was a little joke :(10:08
MirvI thought it might be that deep10:08
* Laney feels scrutinised10:08
Laneythanks guys :P10:08
* willcooke is newslettering. Going through the notes from the meeting for news fodder10:12
oSoMoNwillcooke, libreoffice 6.0.3 is in bionic, that's new since the meeting10:20
willcookethanks oSoMoN10:26
Laneyseb128: did you make any progress on that orca thing?11:13
seb128Laney, I didn't get the "drop priviledge" things working, I'm pondering writing a simple 10 lines python script that does the watch/start (in fact I had it since I made a standalone version to test) and install that, it just means another process using memory which is suboptimal12:04
seb128Laney, the callback is never called, even if it has been registered after a drop of privileges12:05
Laneyseb128: it's weird, I just looked a bit and if you make it set the value right after connecting the signal the callback is called12:31
GunnarHjLaney: Do we want it to be possible to set any value via the --domain option?12:35
Laneyany value?12:35
GunnarHjLaney: Yeah, it's supposed to be the correct domain, people make mistakes.12:36
LaneyI suppose warn?12:37
LaneyI think we only have good detection for meson12:37
GunnarHjLaney: In case of meson, is it possible that the correct domain is not detected via introspect? In case of other packages, where dh_translations fails to get the domain by parsing, I can't see what a warning would be based on.12:39
seb128the meson introspection just rely on conventions no?12:41
seb128technically you can call the target as you want, they is no way to query a "type"? like those are not flagged as "translations update"?12:41
Laneyit's not really right to call it a convention12:42
Laneymore an implementation detail, but it's probably part of the interface now so unlikely to change12:42
Laneythe thing is generated from the domain you give to the i18n call12:42
GunnarHjIf you set a domain which introspect hasn't identified, I take it that the ninja call for building POT will fail anyway.12:42
seb128right, but you can't tell that because the target exists then it is a translations one12:43
LaneyYou can tell /if/ the target the user supplied exists12:43
seb128right12:44
LaneyIf we're going to call ninja <target> and we can tell it doesn't exist12:44
Laneycould error with a more friendly message12:44
seb128makes sense12:44
LaneyGunnarHj: you're talking about upgrading the warning in your patch to an error?12:45
GunnarHjNo, I'm not ( even if it would be tempting ;) ). I suppose we can basically have it as is (checking if the domain exists) for Meson, and still make it possible to use --domain for other build systems without any check.12:47
LaneyOK I'm not sure what you're asking then, sorry12:48
LaneyI don't see how we can check for non-meson things unless there's a similar interface for (say) CMake12:48
LaneyIf we can tell it's going to error out in a second anyway then it's potentially a good idea to do it earlier with a nicer message12:48
GunnarHjLaney: Maybe I jumped at conclusions. So then my understanding is: Make it possible to use --domain everywhere, and warn / ignore when possible (i.e. for Meson packages). Is that what we should do?12:51
LaneyGunnarHj: Right, except I'm saying (and I thought this was what you were asking for) that erroring for a bad --domain where you can tell it's bad is not on the face of it an awful idea.12:52
LaneyI don't have a super firm opinion on that though.12:52
GunnarHjLaney: I'd love to trigger a fatal error in those cases and make package maintainers take more responsibility for translations. :)12:53
LaneyIf it's only for explicit --domain then it shouldn't break anything that already exists12:54
GunnarHjLaney: Not sure what you meant by your latest comment,.12:55
GunnarHjLaney: Or maybe I do. You argue for triggering a fatal error.12:56
GunnarHjLaney: And that makes sense. If we add a new option, and people make mistakes when making use of it, why wouldn't we tell the loudly if possible?12:59
LaneyThat's what I'm saying13:00
GunnarHjLaney: Ok. Then I just wait for a final decision before starting to work with a new patch. :)13:01
LaneyGunnarHj: On this point? I'd say feel free if you agree it's right.13:03
GunnarHjLaney: I do. Just a detail about the syntax in a dh_* program for doing that. would it be 'error "..."' instead of 'warning "..."' or should we say exit 1 explicitly?13:05
LaneyI'm not sure if error exits on its own - try it :P13:06
GunnarHjOk.13:07
seb128Laney, you are debugging the ubiquity/signal issue more?13:08
seb128or do you have any idea why the callback stop working later? how did you test it?13:09
Laneyseb128: I just added a call to change a value in the schema right after you connect to the signal, and it gets called then13:09
Laneyit's probably some uid thing13:10
Laneythere's a message in the debug log about not starting a message bus when setuid13:10
Laney(/var/log/installer/debug)13:11
Laneygot to go have lunch now, back in a bit13:11
seb128Laney, enjoy! and yeah, if you do it from the code the process "context" might be the same as ubiquity when it might not be from the gsd process13:12
didrocksenjoy Laney ;)13:18
kenvandinedang... i just missed Laney  :)13:18
kenvandinei'm working on creating preinstalled disk images for ubuntu-desktop with livecd-rootfs13:20
kenvandinei'm assuming i should be using live-build to create them?13:20
kenvandinei've added a SUBPROJECT of desktop-preinstalled to build IMAGEFORMAT=ext413:21
kenvandineand i'm getting an ext4 image that i don't think has a bootloader13:21
kenvandinejust want to make sure i'm actually using the right tool before i go to far :)13:21
kenvandineseb128, didrocks ^^ either of you know?13:21
seb128kenvandine, it has been a while I didn't touch to those projects and I don't remember the details, sorry13:22
kenvandinei guess didrocks did that recently for minimal though :)13:22
ogra_typically live-build only builds rootfses (that might have changed in the times of could though, but for a decade it only did that)13:23
ogra_making bootable images out of that used to be a task of debian-cd13:23
kenvandineogra_, yeah... i couldn't find anything else documented though13:23
seb128kenvandine, the minimal work is not an iso, it's just an ubiquity change13:23
kenvandineoh, there is some stuff in livecd-rootfs that adds packages to an exclude list, maybe that's for other flavors or something13:24
kenvandineogra_, does debian-cd make bootable disk images?  not just isos?13:24
seb128kenvandine, like desktop-next at the time was added with that change https://bazaar.launchpad.net/~ubuntu-core-dev/livecd-rootfs/trunk/revision/88713:24
jbichakenvandine: building isos is a huge pain. It's strong incentive for Ubuntu flavors to become official so devs don't have to do it any more13:24
seb128k, need to step out for a bit13:25
kenvandinejbicha, yeah, i don't want an iso13:25
seb128bbl13:25
kenvandinei want a disk image13:25
kenvandinebye seb128!13:25
ogra_kenvandine, it used to ... i havent touched it in ages though ... the whole setup might have changed (UbuntuCore FTW :) )13:25
kenvandineogra_, cool!13:25
didrocksyeah, I didn't build it for quite a while, I used to unsquashfs, then modify the chroot, then resquash13:30
didrocksit was the easiest ;)13:30
didrockskenvandine: btw, here we go: https://github.com/ubuntu/communitheme-snap-helpers/pull/4#issuecomment-38113505313:30
ubot5-ngubuntu bug (Pull request) 4 in communitheme-snap-helpers "Enable releasing as well gtk-common-themes with latest communitheme" (comments: 4) [Open]13:30
didrockssee the addition of "You may want to download as well gtk-common-themes snap from edge/communitheme-snap-helpers-pr4 channel to test your snaps with those changes.13:30
didrocks"13:30
didrocksso, basically, every communitheme (on any of the 5 projects) commit in master will now push communitheme/edge and gtk-common-themes/edge13:31
kenvandinedidrocks, cool, thx13:31
didrocksand any PR will release communitheme/edge/pr-name and gtk-common-themes/edge/pr-name13:31
didrocks(and any further push to PR will do that as well ;))13:31
didrockstokens updated in all project, time to merge!13:31
kenvandineoh... maybe i should be using ubuntu-image instead of live-build directly :)13:36
* kenvandine peels the onion13:36
didrockskenvandine: do you want to test some changes or do something official?13:37
didrockskenvandine: if it's just for testing something, I would really say unsquashfs/resquash it13:38
kenvandinefor image creation?13:38
didrocksyep13:38
kenvandineno, this is for microsoft's hyper-v gallery13:38
kenvandinegoing to be official13:38
didrocksyeah, so you may want something a little bit more official then ;)13:39
didrocks                 edge                                0.1        313:39
didrocks\o/13:39
didrocks(for gtk-common-themes)13:39
didrockskenvandine: hope that makes sense: https://forum.snapcraft.io/t/snap-application-and-snap-themes/4946/9?u=didrocks13:52
=== maclin1 is now known as maclin
Laneyseb128: don't understand what gsd has to do with it at this point, the callback is in ubiquity too.14:19
Laneykenvandine: did you get my email?14:19
Laneysounds like you reinvented a bit what I sent you in that?14:19
kenvandineLaney, no... i didn't14:19
LaneyDate: Wed, 4 Apr 2018 18:42:01 +010014:20
LaneyFrom: Iain Lane <iain.lane@canonical.com>14:20
LaneyTo: ken.vandine@canonical.com14:20
LaneySubject: livefs-preinstalled notes14:20
kenvandineweird... i have no emails from you on that day14:21
LaneyApr  4 18:42:03 cripps postfix/smtp[24104]: D9B0C206B8: to=<ken.vandine@canonical.com>, relay=mx.canonical.com[91.189.95.10]:25, delay=0.08, delays=0.01/0.02/0.04/0.02, dsn=2.0.0, status=sent (250 2.0.0 Ok: queued as E94AC1A2968)14:22
kenvandinedang... can you resend it?14:22
Laneysure14:22
kenvandinemaybe i accidentally deleted it or something14:22
didrocksor… Laney was *filtered* out!!!14:23
kenvandinelol14:23
kenvandinei have other mails from him, on different days14:23
didrocksprobably spam then! :p14:23
didrocksdoko: any news on the ubuntu-report MIR now that the different comments were addressed?14:24
kenvandineLaney, thx14:26
kenvandineLaney, that's my exact diff :)14:27
Laneygood sign14:28
kenvandineLaney, ok, i did that and i built it with live-build and got a .ext4 file14:28
kenvandinebut it's not bootable, so was thinking it didn't install a bootloader or something14:28
kenvandinethe contents of the file look sane, so i think it's pretty close14:29
kenvandinenow i see there is ubuntu-image that seems to work some nice magic14:29
LaneyI think you can use qemu-img on those14:29
kenvandinedid that14:29
Laneybut it was too big so I didn't actually download the artifact that LP made14:29
kenvandineubuntu-image wraps live-build and then works some magic, i think14:30
Laneynever used it14:30
ricotzhey desktopers14:30
kenvandinebut now live-build isn't working... held packages14:30
kenvandineThe following packages have unmet dependencies:14:30
kenvandine rsyslog : Depends: libfastjson4 (>= 0.99.7) but it is not installable14:30
kenvandineso i guess bionic might be a little broken today?14:30
jbichaseb128: I don't know if you were subscribed to this MR but it was just merged14:32
jbichahttps://gitlab.gnome.org/GNOME/gnome-control-center/merge_requests/2214:32
ubot5-ngGNOME bug (Merge request) 22 in gnome-control-center "thunderbolt: new panel for device management" (comments: 29) [Merged]14:33
Laneylooks ok here14:33
jbichakenvandine: it sounds like it's building from bionic-proposed? rsyslog is stuck in -proposed because of bug 174632714:34
ubot5`bug 1746327 in libfastjson (Ubuntu) "[MIR] libfastjson" [High,New] https://launchpad.net/bugs/174632714:34
ricotzis there a chance for appstream 0.12.0-3 and appstream-generator 0.7.1-3 being synced?14:35
Laneyfirst one is done14:35
Laneywhy do you care about the generator?14:35
ricotzfor 3rd party reasons, I guess they should go together14:36
ricotzah didn't see the appstream update yet, looks like the amd64 package publications is stuck14:38
kenvandinejbicha, should be14:38
kenvandineCOMMAND FAILED: ['sudo', 'PROJECT=ubuntu', 'SUITE=bionic', 'ARCH=amd64', 'SUBPROJECT=desktop-preinstalled', 'PROPOSED=False', 'IMAGEFORMAT=none', 'lb', 'build']14:38
kenvandineshouldn't rather :)14:38
didrockskenvandine: do you mind pointing robert to bug #1763736, I guess it worths a look to see what is different between 18.04 and 16.0414:38
ubot5`bug 1763736 in gnome-software (Ubuntu) "communitheme snap not present in gnome software on bionic, but it is on 16.04" [Undecided,New] https://launchpad.net/bugs/176373614:38
ricotzLaney, it needs to be new'ed ;)14:38
jbichakenvandine: nope, you're not tricking me into building Ubuntu images ;)14:38
didrocks(I tagged it so that we remember)14:39
didrocksand maybe it's not the only case triggering this14:39
kenvandinedidrocks, sure14:39
didrocksthx!14:39
seb128jbicha, yeah, I'm talking regularly to gicmo as said, but you seem pretty interested by that panel/feature, do you want to take over? there is no need to have several persons doing the same tracking/upstream relation work there15:11
jbichaseb128: do you have thunderbolt?15:11
seb128jbicha, not at the moment no, do you?15:12
jbichamy computer might support it but I don't have any peripherals so "no"15:12
seb128jbicha, I just know gicmo (upstream) from long time and I've good work relations with him15:12
jbichaI'm happy if you keep handling the thunderbolt work. I was just letting you know it was merged :)15:13
seb128thx15:14
seb128I'm going to look to get that in .115:14
jbichaok15:14
seb128it feel like it's late to try to get a ffe/in .0 now15:14
seb128like we wouldn't get translations and such and it didn't get any testing yet15:14
seb128neither upstream nor in other distro15:15
jbichathat makes sense15:15
seb128Laney, my understand was that gsd-media-keys is what catch the keybinding and toggle the gsettings key to enable the screen-reader15:15
seb128now I'm unsure how to test15:15
seb128since we don't have the indicator atm I've been testing using the keybinding15:16
seb128the gsettings key get correctly toggle in response so that side of things seem to work15:16
Laneyseb128: oh, well I was using gsettings from a terminal to set the key on or off15:16
seb128ah15:16
Laneyand watching if the callback gets called15:16
Laneywhich it is not15:16
seb128using what user?15:16
Laneyubuntu one15:16
seb128k, so either something doesn't work in the drop priviledge world or it's not listening on the same bus or something15:17
seb128do you debug that?15:17
Trevinhomorning15:17
seb128hey Trevinho15:17
didrockshey Trevinho15:17
Laneynot right now, looking at some appstream-generator problem with ximio_n in another channel atm15:18
LaneyI did print the uid and euid and they were right though15:18
seb128k15:18
seb128day is a bit cahotic here so I'm probably not going to debug much before monday, especially I'm unsure how to go about debugging that15:19
seb128andyrock, good to see they fixed that gtk segfault :)15:21
seb128andyrock, and please stop putting cards in done when the fix is upstream and not in Ubuntu yet :p15:22
andyrockops sorry about that15:23
seb128no worry :)15:24
seb128Trevinho, btw "gnome-shell migration list" got recent tweaks for gnome-tweaks and evolution name changes, you might need to catch up with unity (unsure if you did already in your first version)15:25
jbichaseb128: our compatibility .desktop files have enough bugs I'd really like to drop them now if we could get away with it :|15:33
seb128jbicha, what sort of bugs?15:35
seb128too late now for bionic though15:35
seb128I had a card for it and tried to push for that at the start of the cycle15:35
seb128but d_idrocks though it was important and L_aney that we could handle things better iirc, anyway we never got consensus15:36
seb128at this point we have migrations for GNOME and Unity so we can probably drop them next cycle15:36
jbichathe compatibility .desktop is set the wrong direction15:36
jbichathe new file has NoDisplay=True when it should be the old one that does15:37
seb128is that a specific one?15:37
jbichait's our script that's broken15:37
seb128or did we do that for each? and if so do you remember why and what issue does it create?15:37
jbichayes15:37
jbichathe issue is that we're not really converting people to the new .desktop15:37
seb128(I need to step out for a bit but I'm going to read backlog/comment again once I'm back)15:38
seb128isn't gnome-shell doing that for us?15:38
jbichamaybe, but it's still wrong for other desktops15:38
seb128https://sources.debian.org/src/gnome-shell/3.28.0-1/js/ui/appFavorites.js/?hl=42#L1015:38
jbichaand other desktops was the reason we were still keeping the old .desktop's, right?15:38
seb128well nothing we do to the .desktop is going to migrate configs for users15:38
seb128well, main reason was unity15:39
jbicha2. right-click an app (like Text Editor) in the Activities Overview and click Show Details. It doesn't go to the right place (although GNOME Software is admittedly smarter there than it was in zesty or artful)15:39
seb128but now that has a conversion in bionic15:39
seb128so we can drop in bionic+115:39
seb128the desktop we care about are GNOME and Unity15:39
jbicha"Open with Other Application" shows duplicate apps15:39
seb128other desktops are welcome to fix things15:40
seb128but we are not going to block on them15:40
seb128right15:40
jbichamy #2 is pointing out that we're not actually doing the conversion in bionic anyway…15:40
seb128I patched g-c-c for a duplicate issue15:40
seb128we should probably include a workaround patch in gtk for bionic15:40
seb128until we drop them next cycle15:40
jbichathe NoDisplay is telling desktops besides GNOME and Unity to use the old names15:40
seb128that topic is complicated, I don't remember the details of the previous discussions15:41
jbichaI mean I understand if our final answer is "not for bionic" since things are really late. I'm just pointing out that there are unnecessary bugs here because of the compatibility .desktop files15:41
LaneyIf you have a configuration which specifies the old name15:41
seb128but I was in favor of dropping them and d_idrocks (and maybe L_aney) agreed against iirc15:41
LaneyAnd it gets NoDisplay in a new version15:41
jbichaand it's incomplete, we don't have compatibility .desktops for most of the universe renames anyway15:41
LaneyThen you have a bad time.15:41
LaneyThat's why it is the way round it is.15:41
seb128right15:42
LaneyI was in favour of dropping them.15:42
LaneyI proposed a solution where you made a centralised mapping of names somewhere on the system15:43
seb128so it was d_idrocks who argue that it's small polish like respecting user config on upgrades that make a difference15:43
Laneyand desktop environments could write their own way to migrate, consuming that list15:43
* didrocks argues it's not "small" :)15:43
LaneyI don't exactly remember why we didn't do anything like that15:43
seb128lack of resources to allocate to that work I guess15:43
didrocksbut yeah, we should handle a one shot migration, maybe with session-migration15:43
didrocksbefore the DE starts15:43
didrocks(and so unity drops unknown desktop files)15:43
seb128anyway it's too late now for bionic15:43
LaneyThere was this unity problem15:44
LaneyWe had a card to get that fixed, and I thought it got assigned15:44
jbichathis is what was done in Unity this week: https://bazaar.launchpad.net/~unity-team/unity/trunk/view/head:/tools/migration-scripts/07_unity_migrate_gnome_favorites15:44
Laneybut maybe not15:44
seb128jbicha, please report the issues or give me the bug numbers, we can at least patch the duplicates out of "open with" in gtk15:44
didrocksLaney: yeah, but IIRC, it was a lot of work to fix it15:44
didrocksif we do it all in session-migration, which forces us to have another list though, it's done before Unity starts15:44
didrocksand so, we should be good even without the Unity fix15:44
Laneyok, but it got assigned at the start of this cycle15:44
seb128Laney, unity is fixed in bionic15:44
seb128so we can drop them next cycle15:44
didrocksoh, it's fixed?15:44
Laneyit is?15:44
LaneyI thought the point was to fix it in xenial15:45
didrockslast time I heard about it was "not going to do it, too complex"é15:45
seb128well, unity does the same as gnome-shell in bionic now15:45
didrockssweet15:45
seb128so if we keep the old files we end up with them rewritten to the new one15:45
jbichaseb128: my old bugs were bug 1606901 and bug 162374815:45
ubot5`bug 1606901 in nautilus (Ubuntu Xenial) "Clicking "Show details" in AO for Nautilus produces "Could not find 'nautilus.desktop'" error from gnome-software" [Low,Triaged] https://launchpad.net/bugs/160690115:45
ubot5`bug 1623748 in gnome-pkg-tools (Ubuntu) "Use non-compatibility .desktop files by default" [Undecided,Confirmed] https://launchpad.net/bugs/162374815:45
Laneyok, well the topic got away from me then15:45
seb128which means unity users on bionic are going to have the new names in use15:45
seb128so we can drop the compat in +115:45
didrockssounds like a plan15:45
Laneythe problem was that as soon as you upgraded and a desktop file went away15:46
Laneyit was purged from your launcher15:46
Laneyso you could not migrate15:46
didrocksyep, that was the issue15:46
didrocksbut with a migration list, that's fine15:46
Laneyno it's not, not if it's removed from the launcher15:46
Laneythen you don't know what to rewrite15:46
didrocksif unity fixes it, I think they are doing it when they load the gsettings list15:47
didrocksso before matching/checking file existing state15:47
didrocksTrevinho/andyrock would know15:48
Trevinhoseb128: yeah, I noticed that (migration list), I have in my list15:53
TrevinhoYeah, the migration happens in a script, so it runs just after the session starts15:53
Trevinhodidrocks: ^15:53
didrocksTrevinho: ah, so using session-migration?15:54
jbichaseb128: I filed LP: #176376315:59
ubot5`Launchpad bug 1763763 in nautilus (Ubuntu) "Duplicate apps in Open With Other Application" [Undecided,New] https://launchpad.net/bugs/176376315:59
Trevinhodidrocks: yes, your kid :)16:24
didrocks:p16:24
* didrocks waves good evening and good week-end16:24
seb128jbicha, thx16:26
seb128Laney, right, so in bionic unity migrates the .desktop to the new name so the bionic configs are never going to include the compat names so in +1 we are fine to drop them, no?16:26
Laneyseb128: Upgrade from Xenial to Bionic while you are logged in. If that bug is still present, as the upgrade progresses your favourites get removed.16:29
LaneySo it doesn't matter if they are migrated when you next log in, since the entries are already gone.16:30
seb128Laney, right, which is why we don't drop the old names in bionic but in +116:30
LaneyIn any event Unity isn't supported then16:30
LaneySo we could jsut do nothing at all and this problem resolves itself.16:30
seb128the only issue would be people upgrading from xenial to bionic +1 through bionic without login in16:30
seb128right16:31
seb128I don't understand why didrocks cares so much about that :/16:31
seb128we should have dropped those compat files in bionic16:31
LaneyNo, we'd have had to fix that bug then and apparently we didn't want to16:31
seb128Unity to GNOME is enough of a change, that's not a potential tiny config change due to launcher renames that would have been the end of the world16:31
Laneymaybe, we could have made that decision16:32
seb128well, as you said Unity is no supported16:32
LaneyIt's supported in Xenial.16:32
LaneyFor C there is no supported upgrade path from Unity.16:32
seb128true16:32
seb128anyway we are where we are now16:32
seb128too late to remove them16:32
seb128let's drop them next cycle16:33
seb128jbicha, the "show in software" thing is really minor and not likely to happen since g-s migrates the config16:34
seb128and we can fix the dup entry one16:34
jbichaseb128: the show in Software bug still exists but it's a lot more minor now because GNOME Software works better than it used to16:34
seb128I don't understand why it exist16:35
seb128isn't g-s migrating the launcher items?16:35
seb128so it should query with the correct .desktop name no?16:35
jbichaI guess it doesn't. The results look like  gnome-software --details=gedit.desktop  instead of  gnome-software --details=org.gnome.gedit.desktop16:37
=== sergiusens_ is now known as sergiusens
Laneythe bionic solution does rely on us having made those compat desktop files, and I don't think we've been particularly comprehensive in doing that16:38
seb128you mean?16:39
jbichabut gnome-shell does save org.gnome.gedit.desktop to favorites correctly16:39
Laneyhave we created those compat files for every package which renamed its desktop file?16:39
seb128Laney, https://sources.debian.org/src/gnome-shell/3.28.0-1/js/ui/appFavorites.js/?hl=42#L9 is the bionic solution, at least for our main session16:39
Laneytalking about the unity migration16:40
seb128https://bazaar.launchpad.net/~unity-team/unity/trunk/view/head:/tools/migration-scripts/07_unity_migrate_gnome_favorites is the same16:40
seb128no?16:40
Laneyyes16:40
Laneybut if the old desktop file stops existing16:40
Laneythis doesn't work16:40
seb128ah right16:40
seb128indeed16:40
seb128we only covered some of the most likely to be important/in user configs16:40
seb128like gedit, nautilus16:41
jbichaI never added compatibility .desktop's because we all know I didn't like them so universe apps in particular don't have them16:41
Laneyoh well16:42
Laneythis is a description of the situation, doubt anything is going to change16:43
Laneyunless we get this prioritised for upgrades before .116:43
jbicha(oh and it created a lot of extra diff with Debian, but that's not much of a problem if we're dropping them in Chaotic Chameleon)16:44
seb128k, enough for this week, more orca debugging on monday17:10
seb128have a good w.e desktopers!17:10
TrevinhoI need to add a migration script to u-s-d which is not using dh though, how can I do that?17:28
TrevinhoLaney: maybe? :)17:28
Laneywhat does dh have to do with migration scripts anyway?17:30
Laneyjust add a line in the .install file listing it17:30
Laneyand maybe depend on session-migration too17:30
TrevinhoLaney: for using dh-migraitons I mean17:31
Trevinhoas that will parse a .migration file and do all the things17:31
Laneyjust do what I said, or maybe you could call dh_migrations from the build17:32
TrevinhoLaney: yeah, that was I wanted to do17:32
Trevinhobut at what level?17:32
Trevinhoinstall?17:32
Laneythis is going to be more effort than doing it manually17:32
Laneybinary-post-install/package:: maybe17:33
willcookenight all17:39
TrevinhoLaney: ok, i see if it goes well otherwise I'm going with install17:39
TrevinhoLaney: yeah, worked quickly, thanks17:43
jbichaTrevinho: I see that fmuellener has tagged mutter 3.28.118:09
jbichahe does this entertaining thing where he tags releases several hours before he actually publishes the tarballs18:10
Trevinhojbicha: mh I see18:15
Trevinhojbicha: so feel free to update salsa so I will work on latest18:15
jbichaI need to wait for the tarball to import it into salsa18:16
Trevinhok18:17
jbichathe most plausible reason to wait hours is if you're doing testing of the tarballs, but if you're doing testing, shouldn't you wait to push the tag?18:20
TrevinhoLaney: still here?18:47
Trevinhowell, in case... https://code.launchpad.net/~3v1n0/unity-settings-daemon/use-and-migrate-unity-monitors-xml/+merge/343230 if not, have nice weekend EU, i'm leaving for lunch.18:53
Trevinhojbicha: maybe you can review that? ^19:34
jbichaTrevinho: I'm hoping someone else could take a look at that one since it's a bit outside my expertise, sorry19:49
oSoMoNkenvandine, https://forum.snapcraft.io/t/call-for-testing-chromium-65-0-3325-146/4390/20 looks like the issue you were seeing 2 days ago I think19:55
oSoMoNI'm looking into it19:55
oSoMoNadding libsecret to the snap to see if that makes things better19:55
Trevinhojbicha: ok, ok.. i'll wait next week19:56
kenvandineoSoMoN, hmm19:56
kenvandineoSoMoN, i don't even remember now :)19:56
oSoMoNkenvandine, the chromium snap not starting any longer, I can reliably reproduce the issue after logging in to a google account19:58
kenvandinei don't remember what i did now though19:58
kenvandinemaybe started over?19:58
oSoMoNso if I add libsecret to the snap and do "snap connect chromium:password-manager-service" then it starts20:00
oSoMoNgot a workaround for that issue that seems to work reliably, this will require some more testing on Monday21:22
oSoMoNtime for some rest21:22
oSoMoNhave a good week-end everyone!21:22
GunnarHjHi jbicha, how would I do to see the result of that apport hook?23:30
jbichasorry, I should have mentioned that in the bug. Just run   ubuntu-bug yelp23:35
GunnarHjAh, thanks. Yeah, I think the proper place to point the users is https://www.ubuntu.com/support/community-support .23:43
GunnarHjjbicha: Btw, I tried ubuntu-bug ubuntu-docs first, and was told that ubuntu-docs is not an official Ubuntu package. (Not that I would like to see apport data on ubuntu-docs bugs, but still...)23:46
jbichaubutnu-bug ubuntu-docs works here23:47
jbichado you have a different version installed?23:48
jbichait doesn't have an apport hook so it would just open a LP bug23:48
GunnarHjI have. :) I'm on artful but have the bionic version of ubuntu-docs... Thanks.23:48
GunnarHjjbicha: As regards your latest dh_translations suggestion, wouldn't that be an enhancement compared to what the program has done up to now?23:50
jbichathe --domain stuff is an enhancement too23:50
jbicharunning the help-*-pot target just brings us up to where we are with autotools I think23:51
GunnarHjIt is, but the purpose is to fix regressions due to Meson. Not saying it would be a bad idea, just that dh_translations always has figured out *the* domain and built one POT.23:52
jbichawell something is building the help .pot for gnome-mahjongg23:53
GunnarHjI'm going to look at the packages you mention and comment on the bug report. But that will happen tomorrow.23:54
jbichaI think it's dh_translations that builds the help .pot for gnome-mahjongg because I just built on Debian and it wasn't built there23:57
jbichahave a good night :)23:57
GunnarHjYou too (whenever the night comes for you).23:58

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