[00:14] <jbicha> ximion: could you explain why http://appstream.ubuntu.com/bionic/main/issues/gnome-control-center.html complains about icon-not-found
[00:14] <jbicha> but https://appstream.debian.org/sid/main/issues/gnome-control-center.html does not
[00:14] <jbicha> the icon is installed by gnome-control-center-data which is a dependency of gnome-control-center
[00:14] <jbicha> does appstream.ubuntu.com need a newer asgen version?
[00:15] <jbicha> this issue affects several other apps, Firefox is a prominent example
[00:16] <jbicha> oh, Firefox's problem might be different, it is a symlink which is a warning but it seems to work fine in Debian that way
[00:16] <jbicha> Debian & Ubuntu's Firefox packaging is different but I think that part might be close enough
[00:17] <jbicha> anyway, never mind Firefox for now
[00:17] <ximion> firefox does symlinking stuff, so that won't work
[00:18] <ximion> jbicha: the Ubuntu appstream-generator is very old, the icon finding logic has been refined quite a bit
[00:18] <jbicha> it seems to work in Debian
[00:18] <jbicha> Laney: can you update asgen some time to fix gnome-control-center? ^
[00:20] <ximion> from looking at the code, the icon should have been found in Ubuntu, but there were quite some changes in the icon finding & resizing code
[00:21] <ximion> asgen 0.6.8 had relaxed icon scaling rules, and 0.7.0 and 0.7.1 have a lot of improvements for HiDPI icons
[00:22]  * ximion puts his bets on the 0.6.8 change for making this particular case work
[00:22] <jbicha> ximion: do you want to file a Ubuntu Feature Freeze Exception for asgen 0.7.0 in bionic?
[00:22] <ximion> jbicha: for that I would also need a freeze excaption for appstream, dcontainers and stdx-allocator
[00:22] <ximion> which is quite a lot of stuff
[00:23] <ximion> I don't see those changes breaking anything, but you it's harder to justify
[00:25] <ximion> I tried to make a Snap for asgen, but that's a bit annoying, unless I figure out how to use snaps inside snaps for building :P
[00:25] <ximion> making a docker container is easy though
[00:26] <ximion> (or building it on bionic ^^)
[00:26] <jbicha> dcontainers and stdx-allocator should be easy, they're just new packages for universe
[00:26] <ximion> jup, mustache-d is also easy, no other rdeps. the major one is appstream itself, because that's a mixed bugfix and feature release
[00:28] <ximion> (having it would make the KDE people very happy though ^^)
[00:34] <ximion> (FWIW, appstream 0.12.0 and appstream 0.11.8 are all compatible with appstream-generator 0.7.1 output, there have been no incompatible changes)
[00:35] <jbicha> do you want to file the FFE bug? since you're more of the expert here
[00:36] <jbicha> what's needed for https://salsa.debian.org/gnome-team/gnome-software/commit/e52d2fa5 to work?
[00:36] <ximion> would I need to file a bug report only for appstream-generator, or one for each of its dependencies as well?
[00:37] <jbicha> don't worry about dcontainers, stdx-allocator, and mustache-d. Those are already being synced
[00:37] <jbicha> everything else can be in one bug I assume
[00:37] <ximion> in any case, I'd file it tomorrow, because I want to make the asgen 0.7.1 bugfix release tomorrow first (has some quite big bugfixes compared to 0.7.0 - and has now been tested with the whole Debian archive ^^)
[00:38] <ximion> jbicha: gir-to-d would also need to be in there, but if that is updated, appstream-generator has to be updated as well, because the older asgen in Ubuntu will not compiler with gir-to-d >= 0.14
[00:38] <ximion> *compile
[00:38]  * ximion can't type anymore
[00:39] <jbicha> ok
[00:39] <ximion> for that change you need the appstream 0.12 package
[00:40] <ximion> (it's a useful one though, avoids a lot of duplication and adds an easy way for software centers to install HiDPI support if they detect the right display)
[00:40] <ximion> and also adds an obvious way for users to prevent icon downloads :P
[00:44] <ximion> hmm, appstream 0.12 also updates the SPDX license list to 3.0, I guess I could make a good case for it
[00:51] <ximion> this just reminded me that asgen needs metadata_license expression parsing, almost exclusively for GNOME Terminal...
[00:51]  * ximion sees to add that quickly
[01:25] <Trevinho> robert_ancell: you know if there's a way to expose the appdata info from a snap?
[01:26] <Trevinho> robert_ancell: I mean, if i put it somewhere inside the snap, it could be parsed by gnome-software?
[01:26] <Trevinho> I know there's a way for importing infos so far, but not really something that is enough
[01:26] <Trevinho> nor is working fine to be fair
[01:44] <robert_ancell> Trevinho, there is not sadly. We proposed meta/appinfo.yml or similar but it was shot dwn
[01:45] <Trevinho> robert_ancell: mh, so is there any plan for something like exposing all these metadata?
[01:45] <Trevinho> rewriting them just in other format looks silly to me
[01:46] <robert_ancell> Trevinho, the appstream ID will be exposed as a "common-id" (still don't know what it's defined as) but all other metadata will be translated by snapcraft.
[01:47] <Trevinho> robert_ancell: yeah, i'm aware of that
[01:47] <Trevinho> robert_ancell: but, what I'm not understanding is then if we've the id, how can a 3rd party tool like the installer or the shell know them?
[01:47] <robert_ancell> what do you mean "know them"?
[01:47] <Trevinho> wouldn't be better to say: here's my appdata, then all the apps already speaking that language can parse it
[01:48] <Trevinho> robert_ancell: how could the installer for example know all the metadata of this app, a part the ones that are exported by snapcraft (once this is installed, I mean)
[01:48] <Trevinho> as snapcraft could not bother of knowing them all
[01:49] <Trevinho> some of these are just important at runtime or once the app is installed
[01:49] <robert_ancell> Trevinho, appdata important at runtime?
[01:51] <Trevinho> robert_ancell: like in the telegram case things like the desktop-id provided so that it could be used to detect if it is running or to run it
[01:52] <robert_ancell> Trevinho, does anything use that?
[01:53] <Trevinho> robert_ancell: well, isn't for example the shell using it as shown by jbicha for getting the alias names for desktop files?
[01:53] <Trevinho> robert_ancell: https://gitlab.gnome.org/GNOME/gnome-shell/merge_requests/41/diffs
[01:54] <ubot5-ng> GNOME bug (Merge request) 41 in gnome-shell "WIP: Use AppStream for mapping renamed apps where possible" (comments: 0) [Opened]
[01:54] <jbicha> https://github.com/3v1n0/telegram-snap/issues/5
[01:56]  * robert_ancell digs in
[02:00] <robert_ancell> Trevinho, jbicha I can't find any of those projects where they contain the old name - where is it stored?
[02:00] <jbicha> robert_ancell: https://gitlab.gnome.org/GNOME/gnome-tweaks/commit/c553730f ??
[02:01] <robert_ancell> jbicha, yeah, I was looking at gnome-documents, tali etc from the MR that Trevinho linked to, but none of the master branches seem to have provides.
[02:02] <robert_ancell> why is that not just in the .desktop file?
[02:02]  * Trevinho takes a break... 
[02:03] <jbicha> maybe they figured it was easier to update the appstream standard than the .desktop standard?
[02:04] <jbicha> appstream uses provides for other things too
[02:04] <robert_ancell> sure
[02:04] <robert_ancell> It seems overkill for something to have to open the AppStream database (which takes ages to load) just to note a .desktop file was renamed
[02:23] <ximion> robert_ancell: libappstream needs about 1-2sec for that, appstream-glib can't be that much slower
[02:24] <robert_ancell> ximion, that's pretty heavy
[02:24] <ximion> if you use libas' cache it's just 380msec ^^
[02:25] <ximion> Trevinho: we discussed AppStream in Snappy to death, the "import little bits of data" thing that is implemented in Snapcraft now was the best we could get so far
[02:25] <robert_ancell> that sounds more reasonable :)
[02:25] <robert_ancell> TO DEATH
[02:25] <ximion> well, a lot and often ^^
[02:26] <ximion> time appstreamcli get org.freedesktop.appstream.cli
[02:26] <ximion> real    0m0,279s
[02:26] <robert_ancell> ximion, so that uses a cache?
[02:26] <ximion> hmm, it's okay, I think
[02:26] <ximion> jup
[02:27] <ximion> time appstreamcli get org.freedesktop.appstream.cli --no-cache
[02:27] <ximion> real    0m0,457s
[02:27] <ximion> 0m0,420s on subsequent runs
[02:27] <robert_ancell> ximion, where is the cache stored / which API?
[02:28] <ximion> . /var/cache/app-info/gv/ for a system-wide version, otherwise each tool using libappstream can place one wherever it wants
[02:28] <robert_ancell> ah, as_pool_load_cache_file?
[02:28] <ximion> that cache is basically just a compressed GVariant :P
[02:29] <robert_ancell> sounds reasonable.
[02:29] <robert_ancell> ximion, do you just call as_pool_set_cache_flags before as_pool_load?
[02:29] <robert_ancell> (to use the system cache)
[02:29] <ximion> before load
[02:30] <ximion> hmm, appstream-util takes indeed longer for the same action, but I have to use "search" there, so not a very fair comparison
[02:30] <robert_ancell> btw, I saw there might be an as_pool_load_async coming?
[02:32] <ximion> jup, I really want that, but so far I never found time for it
[02:32] <ximion> the API is very synchronous at time
[02:33] <ximion> (the easy way around it is to just throw it into a separate thread and wait for that to become ready)
[02:34] <ximion> I never ever thought libas would be faster that asglib - before hughsie did his AsRefString stuff, it was far surpassing it in terms of memory usage though (I haven't measured how much those changes improved it though, but hughsie certainly did)
[02:34] <robert_ancell> ximion, as long as nothing inside that thread does something dangerous...
[02:37] <ximion> there is so much new API and features coming to AppStream, I wonder how Snappy would be able to keep up, even if they had some more extensive format translation going on
[02:38] <ximion> you shouldn't access an AsPool while it's loading ;-)
[02:39] <ximion> read actions might work on it, but adding components to it might result in trouble
[02:39] <robert_ancell> ximion, right, but what operations are going on inside as_pool_load?
[02:40] <ximion> nothing that would impact other threads ^^
[02:41] <robert_ancell> :_
[02:41] <robert_ancell> :)
[02:41] <ximion> disk reads, file parsing, lots of hash table lookups and insertions, etc. but no writing to disk or modifying of other connected objects
[02:41] <ximion> if there was something that would cause problems, I would know that by now, because almost all software centers use it in threads
[02:42] <ximion> (well, Elementary didn't care about the load delay for some time, but they might have changed that now)
[02:43] <ximion> I'm not exactly sure which features you need, but disabling reading of desktop files improves performance by quite a bit
[02:44] <ximion> (with metainfo files on the second place)
[02:44] <ximion> those are sometimes refreshed even if a cache is used, because the user might have installed things meanwhile
[02:44] <ximion> (see AsPoolFlags)
[06:15] <ricotz> chrisccoulson, please push your packaging branches
[07:08] <seb128> good morning destkopers
[07:14] <didrocks> good morning
[07:17] <duflu> Morning seb128, didrocks
[07:18] <didrocks> hey duflu
[07:19] <seb128> hey duflu, how are you? happy friday!
[07:19] <duflu> seb128, relatively good, you? Happy Friday
[07:21] <seb128> I'm good thanks, a bit sick but it's ok, almost the w.e and some rest :)
[07:51] <didrocks> Trevinho: when you get some time, spotted a theme regression: bug #1761684, unsure if that impacts other apps
[07:51] <ubot5`> bug 1761684 in ubuntu-themes (Ubuntu) "Regression in lists like gnome-boxes" [Undecided,New] https://launchpad.net/bugs/1761684
[08:00] <Laney> moin
[08:03] <seb128> hey early Laney! how are you?
[08:04] <seb128> poor Trevinho consider as the theme owner
[08:06] <didrocks> hey Laney
[08:06] <Laney> hey seb128 didrocks
[08:06] <Laney> I'm alright thx, seems sunny again, what about you?
[08:07] <seb128> I'm good, a bit sunny here today as well but still quite windy
[08:07] <didrocks> sunny here as well!
[08:07] <seb128> hate the wind :/
[08:07] <willcooke> bah. No sun here.
[08:07] <seb128> it's supposed to be nice this w.E though, up to 19°C
[08:08] <Laney> jbicha: I'm very aware of the changes. If you want to make it buildable in xenial + xenial-backports, that would make life easier.
[08:08] <seb128> hey willcooke
[08:11] <oSoMoN> good morning desktoppers
[08:16] <willcooke> jibel, thanks again for getting the beta out.
[08:20] <jibel> willcooke, you're welcome
[08:21] <oSoMoN> seb128, thanks for your comment on bug #1761585, I'm preparing a new upload
[08:21] <ubot5`> bug 1761585 in ibus (Ubuntu) "ibus_bus_init does an unconditional call to chmod on $HOME/.config/ibus/bus" [Low,In progress] https://launchpad.net/bugs/1761585
[08:21] <didrocks> hey oSoMoN, willcooke
[08:23] <Laney> going with the distro patch for gnome-initial-setup then?
[08:24] <Laney> 'lo oSoMoN & willcooke
[08:24] <oSoMoN> hey didrocks, hey Laney
[08:28] <tjaalton> when gnome-shell crashes and respawns, the dock seems to not come back?
[08:28] <seb128> oSoMoN, yw!
[08:28] <seb128> lut oSoMoN :)
[08:28] <didrocks> tjaalton: no, it should come back, it's part of the same process and under the same env
[08:28] <didrocks> tjaalton: if you start it manually from a tty, it doesn't have the env var though
[08:29] <seb128> Laney, for this cycle it seems like the less resistance path and closest to what we want next cycle if we want to converge on using g-i-s anyway
[08:29] <tjaalton> didrocks: ok, I filed the crasher now, bug 1761697
[08:29] <ubot5`> Error: Launchpad bug 1761697 could not be found
[08:29] <tjaalton> bah, bug 1761697
[08:30] <didrocks> let's wait for retracer/duplicate
[08:30] <Laney> seb128: we want to do that?
[08:30] <Laney> man, I should get more in the loop
[08:30] <Laney> thx for the info
[08:30] <tjaalton> looks like framebuffer compression is not happy on skylake with 4.15 :/
[08:30] <seb128> Laney, they have been mentions of looking at enabling some of the upstream pages
[08:31] <tjaalton> screen keeps blanking every 20s or so
[08:31] <seb128> I think we are at least going to look at it seriously next cycle
[08:32] <seb128> Trevinho, Laney, on the git workflow card I agree with that Laney wrote
[08:33] <oSoMoN> seb128, test packages based on your upload building in https://launchpad.net/~osomon/+archive/ubuntu/ibus-lp1761585/+packages
[08:33] <seb128> oSoMoN, great, thx
[08:39] <Laney> k
[08:49] <tjaalton> didrocks: heh, one crash later and the dock is back :)
[08:54] <didrocks> maybe a glitch :p
[09:27] <oSoMoN> seb128, ibus upload tested and ready at https://people.canonical.com/~osomon/ibus/, would you mind uploading it for me?
[09:27] <seb128> I can do that
[09:29] <seb128> oSoMoN, I'm going to include the pending queue upload in the changes in case they reject the oldest and only accept the new upload
[09:29] <seb128> oSoMoN, done
[09:34] <seb128> willcooke, have you seen the discussions/reports requesting that we switch from network-manager-pptp to -openvpn by default? I don't know much about VPNs but they state that pptp is old suboptimal standard and has been deprecated by windows and other platform as less secure, etc. The request seems like it makes sense to me but I'm not a VPNs expert, you perhaps have an opinion on that question?
[09:40] <willcooke> PPTP is a lot easier to set up, with many fewer config options, no certificates to set up and so on.  The downside is that it doesn't have (any/much?) encryption.  The up-side of that is that it (used to?) work in China with no problems where as open vpn didn't.
[09:41] <oSoMoN> seb128, thanks!
[09:41] <willcooke> You can set up a PPTP server on Ubuntu in a few mins vs long pages of reading for openvpn
[09:41] <willcooke> That said, yes, openvpn is better.
[09:41] <seb128> oSoMoN, np!
[09:42] <willcooke> It used to be that the openvpn n-m plugins were not very good. But from what I've seen in the last few weeks (I set up an OpenVPN at the sprint to test it) it's much better now.
[09:42] <willcooke> I don't think we should drop pptp yet - it's still useful to a lot of people.  But adding openvpn by default would be a good idea I think
[09:43] <willcooke> Where was the discussion?  I must have missed it
[09:44] <willcooke> you think that's something we would change for 18.04?
[09:44] <seb128> willcooke, #ubuntu-devel yesterday a bit and https://bugs.launchpad.net/ubuntu/+source/network-manager-openvpn/+bug/1752417
[09:44] <ubot5`> Ubuntu bug 1752417 in network-manager (Ubuntu) "Out of the box, Ubuntu Bionic offers only insecure VPN option" [Medium,Triaged]
[09:45] <seb128> willcooke, well as said I don't have much knowledge about VPNs, openvpn is already in main/was being used on touch so maybe it would make sense to add it while keeping pptp
[09:45] <seb128> it might be a late now for that though
[09:45] <seb128> but maybe we can look at doing it for .1 if we think it makes sense?
[09:46] <willcooke> +1
[09:46] <seb128> Laney, ^ do you have any opinion from a r-t perspective?
[09:46] <willcooke> On touch I argued to include PPTP by default
[09:46] <willcooke> :)
[09:46] <seb128> both?
[09:46] <seb128> or just pptp?
[09:46] <willcooke> both
[09:47] <willcooke> happyaron, hey!  Do you know if pptp and openvpn work well in China?  I think it used to be just PPTP?
[09:47] <Laney> I think that the security team should decide whether PPTP is a thing to recommend by installing it by default
[09:48] <Laney> and I'd be OK with including openvpn depending on whether it brings in extra stuff too
[09:48] <willcooke> seb128, I'd say it was "safe" to add openvpn now.  The UI works fine from my testing
[09:48] <Laney> it's in universe btw
[09:48] <Laney> sourcei s in main tho
[09:48] <seb128> right
[09:49] <seb128> it had an approved MIR
[09:49] <seb128> so I guess it's fine to bring the binary back in main if we want to?
[09:50] <Laney> guess so
[09:50] <seb128> nice, thx
[09:50] <willcooke> I'll ping security for their input
[09:50] <Laney> but I'd defer to the security team on what should be included
[09:50] <seb128> +1
[09:50] <seb128> willcooke, thx
[09:50] <Laney> if PPTP is as bad as people are claiming then it would be ok to make people have to go install it
[09:50] <seb128> right
[09:51] <seb128> willcooke, can you Cc me on that ping if it's via email?
[09:51] <seb128> Laney, can you skip the udisks2 s390x autopkgtest result? the flaky test on other archs has been resolved, s390x is another issue when I sent upstream but didn't figure out yet
[09:52] <seb128> that's not a regression from the new revisions, I only added the fix for the other test issue and a segfault fix from andyroc_k
[09:52] <willcooke> seb128, talking in #security now
[09:52] <willcooke> seb128, but they're mainly asleep for a few hours
[09:52] <seb128> right
[09:53] <willcooke> Wow, terrible English
[09:53] <willcooke> sorry
[09:54] <Laney> seb128: ok
[09:54] <seb128> Laney, thx!
[09:57] <seb128> bbiab, changing location
[10:59] <jbicha> Laney: hi, I made ubuntu-settings translatable (needed for the Ubuntu Welcome .desktop). I guess we should make the Amazon .desktop translatable too
[11:00] <jbicha> what do you think about moving the Amazon python script to /usr/bin/ and then we can drop the cdata stuff from meson.build ?
[11:00] <jbicha> /usr/share/ubuntu-web-launchers/amazon-launcher
[11:01] <Laney> is "Amazon" translated differently?
[11:01] <Laney> I don't really want that thing on $PATH if we can help it
[11:04] <jbicha> so I just visited amazon.cn and they have both amazon.cn as their logo and 亚马逊中
[11:04] <jbicha> ok if you don't want it in PATH, we can leave it as is
[11:06] <Laney> If you want to make the name translatable then I suggest that you make it clear in a translator comment that this is Amazon the company and not anything else which might share the word
[11:07] <seb128> if we never got a report about the fact that it can't be translated I wouldn't change it
[11:07] <Laney> when I made this thing I think that the Chinese website used the text amazon in its logo though
[11:07] <seb128> it's more likely to create issues by having some locale wrongly translating it than being useful
[11:09] <jbicha> ok, we can wait for someone to file a bug :)
[11:09] <didrocks> +1
[11:11] <Trevinho> didrocks: ok on theme...  seb128, sadly... yes :-/
[11:12]  * seb128 hugs Trevinho
[11:12] <seb128> Trevinho, good morning, happy friday! you are up early today
[11:12] <seb128> how are you?
[11:12] <Trevinho> hey seb128 happy friday too
[11:12] <Trevinho> yeah, I need to stop a bit earlier today so...
[11:13] <seb128> Trevinho, btw your new git-workflow card is sort of duplicating the exist one on https://trello.com/c/bMWcHyeJ/80-explore-ubuntu-git-workflow-and-work-with-the-server-team-to-have-some-package-maintained-that-way
[11:13] <seb128> unsure it was needed in the board
[11:15] <jbicha> seb128: so I looked a bit at our imagemagick .desktop situation yesterday. It turns out that the display .desktop is Debian-specific; it does not come from upstream at all
[11:16] <jbicha> we have a few remaining issues like bug 1740741 and bug 1740741
[11:16] <ubot5`> bug 1740741 in ubuntu-settings (Ubuntu) " * Hide imagemagick's Display app from the Ubuntu GNOME session as well" [Undecided,Confirmed] https://launchpad.net/bugs/1740741
[11:16] <jbicha> I'd like to modify our imagemagick package itself to no longer install the .desktop at all and we can then drop our NoDisplay hack from ubuntu-settings
[11:17] <jbicha> I talked very briefly with the Debian maintainer yesterday but it's hard to tell whether he even understands why people don't like the .desktop in at least GNOME
[11:17] <seb128> wfm but it's a bit late in the cycle for that no?
[11:17] <Trevinho> seb128: mh I wanted to add it to track all the items that we've to move to thaat system though
[11:18] <jbicha> seb128: it's a bugfix :) and I have realized that virtually no one actually wants the display .desktop
[11:18] <jbicha> check out the reviews in Ubuntu Software for imagemagick :(
[11:19] <seb128> jbicha, wfm as said, I just feel like it's late for changes that could surprise users/other desktops
[11:19] <seb128> but I don't think it's a big deal either
[11:19] <Laney> Trevinho: UNITY_LAUNCER_FAVORITES :(((((((((((((((((((((((((
[11:35] <willcooke> Trevinho, hey!  Do you use VPNs much while you're travelling?
[11:35] <Trevinho> willcooke: well, quite a bit, why?
[11:36] <Trevinho> willcooke: but... well, mostly company one or the one I've in my rapsberry at home
[11:36] <Trevinho> willcooke: why?
[11:36] <Trevinho> Laney: yeah, that's for removing the old names... eventually.
[11:36] <Laney> Trevinho: yeah, you typoed launcher
[11:37] <Trevinho> Laney: it's its name :-D
[11:37] <Laney> :||||||||||||
[11:37] <willcooke> Trevinho, which sort of VPN do you use?
[11:38] <willcooke> openvpn., pptp, other?
[11:38] <Trevinho> willcooke: as said, mostly those... I wanted to subscribe to torguard as there's a good deal (see on reddit)
[11:38] <Trevinho> ah, openvpn
[11:38] <willcooke> kk, thanks
[11:39] <Trevinho> my provider provided router natively only supports pptp, but it's not really nice
[11:39] <Trevinho> so I have a raspberry doing the job for me and works nicely
[11:40] <Laney> Trevinho: wait what, the schema name is com.canonical.Unity.Launcher
[11:41] <Laney> where does this launcer come from?
[11:42] <Trevinho> Laney: I was jocking :)
[11:42] <Trevinho> Laney: I've pushed the fix right now
[11:43] <Trevinho> ouch you were publishing?
[11:43] <Trevinho> :-D
[11:43] <Trevinho> damn
[11:43] <Laney> fix it for the next upload
[11:43] <Laney> no need to build for this
[11:44] <Trevinho> was already triggered :-D
[11:44] <Trevinho> well, we'll republish later then
[11:44] <Trevinho> or I can stop maybe
[11:44] <Trevinho> but bileto will copmlain
[11:44] <Trevinho> complain*
[11:44] <Laney> save the environment
[11:44] <Trevinho> if that would be enough...
[11:53] <Laney> this ubuntu-welcome thing is trying to send a report even when I picked "No, don't send any data."
[11:53] <Laney> s/data/info/
[11:55] <jbicha> my understanding is that the design was for it to send an opt-out ping so that we know how many users are opting out
[12:05] <seb128> Laney, right, it send an empty report in that case, that's something under discussion/that willcooke was going to discuss with legal as well
[12:36] <Laney> seb128: ok, I filed a bug because I think that should be explicitly decided
[12:36] <seb128> Laney, thx
[12:37] <seb128> Laney, note that it was described in the email from willcook_e to ubuntu-devel@
[12:37] <seb128> "Any user can simply opt out by unchecking the box, which triggers one simple POST stating, “diagnostics=false”.  There will be a corresponding checkbox in the Privacy panel of GNOME Settings to toggle the state of this."
[12:38] <seb128> and there were some questions/replies about it
[12:38] <Laney> I remember
[12:38] <seb128> but a bug is a good reference point as well
[12:39] <Laney> But the UI is quite misleading imo
[12:40] <Laney> also I was hoping that part would change before it made it into ubuntu :P
[12:44] <seb128> right, me to, willcooke is going to check if we really need it or not
[12:45] <seb128> also even if the UI was clear I'm unsure it would be a good option
[12:45] <seb128> tell users "we are going to send that in any case", you can't even close the wizard
[12:45] <Laney> agreed
[12:51] <seb128> jbicha, how do you quote text to reply in trello?
[12:51] <jbicha> >
[12:51] <jbicha> I just used the > character to fake quote :)
[12:51] <seb128> lol, thx
[12:54] <Laney> I'm really tired of that discussion :(
[12:58] <seb128> Laney, sry :(
[12:58] <seb128> we should probably take that out of the trello
[12:58] <seb128> rather on the community hub where people who want to argue can
[12:59] <Laney> not you seb128
[12:59] <Laney> feel free imho
[12:59] <Laney> I'd advertise it as 'please use, but details might change' for the first cycle or something
[12:59] <seb128> right
[12:59] <Laney> so if Launchpad turns out to be shit for some reason then it can be changed
[13:01] <Laney> !ohmy
[13:01] <ubot5`> Please avoid any language that may be considered offensive, including acronyms and obfuscation of such - also see https://wiki.ubuntu.com/IRC/Guidelines || The main channels are English only, for other languages, see https://wiki.ubuntu.com/IRC/ChannelList
[14:04] <oSoMoN> seb128, I've turned bug #1761585 into a xenial SRU request and attached the corresponding debdiff, can you sponsor it for me?
[14:04] <ubot5`> bug 1761585 in ibus (Ubuntu) "ibus_bus_init does an unconditional call to chmod on $HOME/.config/ibus/bus" [Low,Fix released] https://launchpad.net/bugs/1761585
[14:12] <GunnarHj> seb128: Fixed a MP for bug #1756547. Time to review/upload before w.e.?
[14:12] <ubot5`> bug 1756547 in Ubuntu Translations "LP refuses to import plural strings where e.g. msgstr[0] entries in PO file miss %d" [High,In progress] https://launchpad.net/bugs/1756547
[14:13] <seb128> jbicha, ^ maybe?
[14:13] <seb128> GunnarHj, I can add to my list but it's not likely for today at this point
[14:13] <jbicha> sure, I can give it a try
[14:14] <GunnarHj> seb128, jbicha: Ok, thanks!
[14:14] <jbicha> seb128 or whoever: could we get final wording on the "approximate location you chose (FIXME)" wording in gnome-initial-setup so that the page can be translated?
[14:14] <seb128> willcooke, ^
[14:15] <jbicha> I guess the issue is that the implementation just uses the selected timezone and there is a Design question about how to word that
[14:15] <seb128> jbicha, note that I approved the template so it can be translated now, but yeah better if the strings stop changing
[14:15] <willcooke> mpt ^ :)  Can you help with the string above?
[14:15] <willcooke> jbicha, got a link handy to show where that appears?
[14:15] <jbicha> willcooke: mpt isn't here :)
[14:15] <willcooke> oh
[14:16] <willcooke> oh yeah
[14:16] <willcooke> wow, I must have typed it without trying to tab complete.  Thats a first.
[14:16] <jbicha> willcooke: you can install https://launchpad.net/ubuntu/+source/gnome-initial-setup/3.28.0-2ubuntu1
[14:16] <jbicha> I can open a bug with a screenshot if you want
[14:16] <willcooke> jbicha, will do and I'll email mpt and cc you
[14:16] <willcooke> jbicha, nah, s'fine.  I'll do
[14:17] <jbicha> after installing gnome-initial-setup, you need to log out and log back in
[14:17] <Laney> it'd be nice if those pages got icons on them
[14:17] <Laney> those make the upstream g-i-s pages look quite smart
[14:17] <jbicha> once you've completed the wizard, you can reset it so it starts again with   rm ~/.config/gnome-initial-setup-done
[14:18] <jbicha> +1
[14:19] <didrocks> jbicha: for the report one, you need to clean that out in ~/.cache/ubuntu-report/<distro>.version
[14:19] <didrocks> (if the report was sent successfully to the server)
[14:20] <didrocks> (meaning, when we'll have a server :p)
[14:20] <seb128> oSoMoN, ibus SRU uploaded
[14:20] <oSoMoN> seb128, awesome, thanks!
[14:20] <seb128> yw!
[14:20] <seb128> l:
[14:20] <seb128> ups
[14:21] <seb128> Laney, icons like on the page content?
[14:21] <Laney> yeah
[14:21] <Laney> they have icons above the header
[14:22] <seb128> k, they probably changed compared to the screenshots I saw
[14:22] <seb128> e.g https://www.tecmint.com/wp-content/uploads/2013/12/fed-19.jpg
[14:23] <seb128> well I saw some have but it didn't seem obvious they all did to me
[14:23] <didrocks> fedora 19 ;)
[14:23] <jbicha> seb128: XDG_CURRENT_DESKTOP=GNOME /usr/lib/gnome-initial-setup/gnome-initial-setup --existing-user
[14:23] <Laney> override XDG_CURRENT_DESKTOP and have a look
[14:23] <seb128> I was concerned it would screw some of my configs :p
[14:23] <seb128> I miss guest sessions :/
[14:23] <jbicha> it won't break anything
[14:24] <seb128> like no code ever had bugs right :)
[14:24] <jbicha> 95.8% sure this time :)
[14:24] <seb128> hehe
[14:26] <seb128> oSoMoN, bah, ibus SRU rejected because the version already exist from the archive (from a newer serie I guess), I'm tweaking the number and reuploading
[14:27] <oSoMoN> seb128, oh, I didn't think of checking other series…
[14:27] <seb128> oSoMoN, yeah, neither did I :p
[14:29] <oSoMoN> seb128, that was in yakkety: https://launchpad.net/ubuntu/+source/ibus/1.5.11-1ubuntu3
[14:29] <seb128> oSoMoN, right, that was expected, the next revision is usually in the next serie :)
[14:30] <oSoMoN> but I definitely wouldn't have thought of checking yakkety, being EOL and hidden on https://launchpad.net/ubuntu/+source/ibus
[14:31] <willcooke> xnox, nice work on that dns/captive portal bug.  That must have been hard to find
[14:31] <seb128> oSoMoN, easier to check on https://launchpad.net/ubuntu/+source/ibus/+publishinghistory
[14:31] <seb128> willcooke, which one is that?
[14:31] <didrocks> we should really generalize <version>-XubuntuY.1, .2, .3 when the packages aren't synced to the new series anymore
[14:31] <willcooke> seb128, https://bugs.launchpad.net/ubuntu/+source/systemd/+bug/1727237
[14:31] <ubot5`> Ubuntu bug 1727237 in systemd (Ubuntu Artful) "systemd-resolved is not finding a domain" [High,Triaged]
[14:32]  * flocculant must tweak his xubuntu ping
[14:33] <seb128> willcooke, good to see that somebody went to the bottom of it instead of having handwaving comments stating it should be fix in the vcs or such :)
[14:33] <seb128> didrocks, yeah, that avoids those issues
[14:34] <oSoMoN> seb128, indeed, I'll make sure I check there next time I prepare a SRU
[14:34] <didrocks> seb128: I'm guilty as well I must say :)
[14:37] <willcooke> jbicha, I was looking at the designs from mpt
[14:37] <willcooke> https://docs.google.com/document/d/1IDQwpqTsB7Cmw413ApXioHeO2E9nv8bHugjxCOg9A-Q/edit
[14:38] <willcooke> I think "FIXME" is supposed to show the location, I don't think there is anything wrong with that string itself
[14:38] <jbicha> ok I see
[14:39] <jbicha> we need to remove FIXME from the string at least though
[14:39] <willcooke> yeah
[14:39] <jbicha> it can be a separate string if we need a TODO reminder
[14:40] <willcooke> also, related, Laney - we should have icons a la upstream: https://github.com/CanonicalLtd/desktop-design/issues/66
[14:40] <ubot5-ng> CanonicalLtd bug 66 in desktop-design "Visual design for first-login window" (comments: 2) [Closed] - Assigned to spencerbygraves (None)
[14:40] <ubot5`> Error: Could not gather data from Launchpad for bug #66 (https://launchpad.net/bugs/66). The error has been logged
[14:40] <willcooke> I dont think that issue should be closed though
[14:40] <Laney> are those the designs?
[14:40] <willcooke> Yes, but
[14:41] <Laney> should probably attach some screenshots of the upstream pages as they aren't going to fit in
[14:41] <willcooke> we've already pointed out that they need to move the nav to the headerbar
[14:41] <willcooke> and yeah, as you say, they wont fit
[14:44] <Laney> that first screenshot labels the app menu as a window switcher
[14:44] <willcooke> ohhhhh
[14:44] <willcooke> I've just commetned on the wrong trello card
[14:44] <willcooke> sorry didrocks
[14:45] <seb128> lol
[14:45] <willcooke> I will fix it
[14:45] <willcooke> Laney, yeah, see my comment at the bottom :)
[14:45] <Laney> ok th
[14:45] <Laney> x
[14:45]  * didrocks sobs, how dared you!
[14:46] <willcooke> fixed
[14:46] <willcooke> https://trello.com/c/OaRRF8L3/228-implement-first-run-wizard
[14:47]  * willcooke hugs didrocks 
[14:47] <didrocks> waow, your actions are even removed from the history…
[14:47] <didrocks> you rewrote the past!
[14:47]  * didrocks hugs willcooke back
[14:48] <didrocks> git-based-trello
[14:48] <willcooke> https://www.youtube.com/watch?v=I1QF_GuGOGo
[14:49] <didrocks> good fit
[14:51] <Laney> jbicha: does dh_translations work with meson?
[14:52] <seb128> Laney, good one :)
[14:52] <Laney> hmmmmmmmmm?
[14:52] <jbicha> um, I think it sort of works the way I'm doing it in ubuntu-settings
[14:52] <seb128> Laney, bug #1751820
[14:52] <ubot5`> bug 1751820 in pkgbinarymangler (Ubuntu) "Teach dh_translations how to deal with meson" [Medium,In progress] https://launchpad.net/bugs/1751820
[14:53] <seb128> meson is making that easy :/
[14:53] <seb128> there is no clear definition of the domain nor way to query it
[14:53] <seb128> unlike config.h definition in good old autotools
[14:53] <seb128> or cmake
[14:54] <Laney> tbh if you could giv ethe domain to dh_translations that would be good enough
[14:55] <seb128> yeah, that's what sort of my conclusion/what I wrote on that bug
[14:55] <Laney> indeed
[14:55] <Laney> jbicha: what kind of sort of?
[14:55] <Laney> AFAICS they just won't be stripped
[14:56] <seb128> it's a bit unfortunate that they changed the "standard" "make domain.pot" target as well
[14:56] <seb128> need to do "domain-pot-update" now in meson project
[14:56] <jbicha> Laney: https://bazaar.launchpad.net/~ubuntu-desktop/+junk/ubuntu-settings/view/head:/debian/rules
[14:56] <seb128> so we need to special case that as well
[14:56] <Laney> I know, I'm reviewing the upload in the queue
[14:57] <didrocks> annoying having to override all dh_translations call though…
[14:57] <Laney> so you get the translations in Launchpad but they won't actually be used by the desktop file...?
[14:57] <seb128> yeah, go GNOME using "better" tools :p
[14:58] <didrocks> well, the general idea isn't bad, it's just that distros weren't taken into account while designing it seems…
[14:59] <seb128> rather "translations" than "distro"
[15:00] <seb128> when I talked to Jussi he basically said that those were basically not part of meson/things dealt with by other people
[15:00] <seb128> as often I think a good part of dev are just not aware of having to deal with translations or considering them as second class problems than can be sorted out later
[15:00] <seb128> which they eventually did but it feel like it was hacked on top
[15:00] <seb128> oh well, it sort of work but could have been nicer in that regard
[15:03] <seb128> k, moving location back, brb
[15:05] <Laney> laney@nightingale> meson introspect --targets | jq -r -M '.[].name | select(endswith("-pot")) | sub("-pot"; "")'
[15:05] <Laney> ubuntu-default-launchers
[15:05] <Laney> phew
[15:05] <didrocks> \o/
[15:06] <didrocks> so, we can automate that in dh_translations
[15:07] <didrocks> hum, in some projects, meson introspect --targets gives me a stacktrace
[15:07] <jbicha> I guess I do need to add X-Ubuntu-Gettext-Domain to the .desktop's here manually though
[15:07] <Laney> that's the main missing bit
[15:08] <GunnarHj> dh_translations needs to know about the builddir too, right?
[15:09] <Laney> you need to run that command in the build dir
[15:10] <Laney> there's probably a debhelper library function to get that
[15:10] <Laney> also, hey GunnarHj
[15:11] <GunnarHj> Laney: There is, but dh_translations is a perl script, so if I understand it correctly, you need to pass the builddir to the script.
[15:11] <didrocks> Laney: even in the builddir, I needed to do meson introspect . --targets to not get a stacktrace
[15:11] <GunnarHj> Hi Laney. :)
[15:11] <didrocks> it's taking sys.argv[1:]
[15:11] <Laney> didrocks: um, ok, I just pasted what I ran so shrug I guess
[15:11] <didrocks> Laney: I bet… I'm unsure what's different here.
[15:12] <didrocks> and the --help clearly says it's optional
[15:12] <didrocks> but the stacktrace here is quite real :p
[15:12] <GunnarHj> didrocks: Are you saying that perl could grab the builddir via some sys.argv[1] variable?
[15:13] <didrocks> GunnarHj: no, I'm just talking about meson introspect, but yeah, perl dh_* can get the builddir
[15:13] <Laney> I'm sure it can somehow, the other dh_* must do that
[15:13] <didrocks> I saw a lot of self->builddir() or something similar
[15:14] <didrocks> in various debhelper releated functions
[15:14] <GunnarHj> That sounds promising. Possibly we can fix that bug completely then. :)
[15:14] <didrocks> Laney: do you mind trying it in github.com/ubuntu/gtk-communitheme ?
[15:14] <didrocks> mkdir build
[15:14] <didrocks> cd build && meson introspect --targets
[15:14] <didrocks> (without the rest, just checking if you get the stacktrace as well)
[15:16] <didrocks> (sorry mkdir build -> meson build)
[15:17] <Laney> laney@nightingale (master|✔)> meson introspect --targets | jq -r -M '.[].name | select(endswith("-pot")) | sub("-pot"; "")'                                                                           ~/temp/gtk-communitheme/build
[15:17] <Laney> laney@nightingale (master|✔)>                                                                                                                                                                         ~/temp/gtk-communitheme/build
[15:19] <Laney> there used to be a program called mesonintrospect before
[15:20] <didrocks> Laney: 0.45.1-1ubuntu2?
[15:21] <jbicha> didrocks: I get the introspect crash with ubuntu-settings too
[15:21] <didrocks> http://paste.ubuntu.com/p/DqHhp2bkBC/
[15:22] <didrocks> jbicha: and not anymore if you specify the build-dir in the arg?
[15:23] <jbicha> yes, that sounds like an upstream bug
[15:24]  * didrocks puzzled why Laney doesn't get it. Maybe it would be worth until we figure that out to always specify builddir in dh_translation calling this
[15:24] <Laney> dunno, but feel free to file it upstream I guess
[15:25]  * Laney is the lucky one
[15:25] <didrocks> LL (Lucky Laney)
[15:31]  * Laney wears a badge
[15:46] <seb128> willcooke, bug https://bugs.launchpad.net/ubuntu/+source/network-manager-openvpn/+bug/1752417 turned into a ffe for the openvpn addition
[15:46] <ubot5`> Ubuntu bug 1752417 in network-manager (Ubuntu) "[ffe] Out of the box, Ubuntu Bionic offers only insecure VPN option" [Medium,Triaged]
[15:59] <willcooke> thanks seb128
[15:59] <willcooke> internet is wonky today
[15:59]  * willcooke suspects that there will be a BT van down the street
[16:00]  * didrocks goes off. Enjoy for the week-end everyone!
[16:00] <didrocks> s/for//
[16:09] <doko> jbicha, seb128: fyi, I backported the brotli alignmnet fixes. needs a reference now in a package or a seed
[16:15] <ximion> Laney: if you also agree that that's a good idea, I would file a sync request for appstream and appstream-generator for Bionic later today, as jbicha requested
[16:15] <ximion> I think I could make a good case for it, but I would also in no way be surprised if that change gets rejected
[16:17] <jbicha> doko: thank you. brotli itself doesn't help me much without bug 1742743 though
[16:17] <ubot5`> bug 1742743 in woff2 (Ubuntu) "[MIR] woff2" [Undecided,New] https://launchpad.net/bugs/1742743
[16:19] <Laney> ximion: Depends on if you're going to break the post-update-command or not 😕
[16:20] <ximion> I actually had to think a lot about that, it was so much fun last time, so why not do it again? But then I decided that for better entertainment value it is best to not repeat an old trick and instead better come up with something new ;-)
[16:20] <ximion> so, no post-update hook will get broken ^
[16:22] <ximion> Laney: more seriously, there are actually some bugfixes in there that will be very useful. For example, did you know HiDPI icons don't really work with Bionic's current version of AS? ;-)
[16:22] <Laney> ximion: ok, feel free to file the FFe requests and explain why it's safe in there ;-)
[16:22] <ximion> also, an update to SPDX v3.0 is good in an LTS release
[16:23] <ximion> but most important question for me is actually: would this enable you to update appstream-generator on the server easier/faster?
[16:23] <Laney> a snap would be best :P
[16:23] <Laney> but without that - kind of but not massively, still need to make it buildable in xenial
[16:24] <Laney> we don't use the asgen archive package there at all
[16:25] <ximion> I played with it a bit, but making snaps on Debian is a pain: https://github.com/ximion/asgen-snap/blob/master/snapcraft.yaml and getting LDC into the Snap is even more of a pain - there is a Snap bundle for LDC, so I wonder whether I would really need to compile that thing again when creating the asgen snap
[16:26] <ximion> also, getting yarn into the snap will be... fun...
[16:27]  * ximion never had to make a snap that was that complicated
[16:29] <seb128> doko, thanks, I still don't think it was required since it was a regression in proposed and we could have stayed on the current version from bionic until the next update, but I guess at least we don't forget about it this way ... webkit2gtk is going to pull it in but it needs bug #1742743 to be acked as well first
[16:29] <ubot5`> bug 1742743 in woff2 (Ubuntu) "[MIR] woff2" [Undecided,New] https://launchpad.net/bugs/1742743
[16:32] <ximion> Laney: the new asgen will give you much better icon tarball sizes: https://appstream.debian.org/data/sid/main/
[16:32] <ximion> the reduced memory usage doesn't appear to be as strong anymore though, and I only updated the GLib bindings...
[16:32] <ximion> so I sort of blame that
[16:33] <Laney> new version of gir-to-d?
[16:34] <ximion> that and new wrapfiles is what changed
[16:34] <ximion> jup
[16:34] <ximion> I'll need to ask the author of both to please make a new release, Git master seems to have a lot of useful changes
[16:35] <ximion> but in general, memory usage is much lower, still
[16:36] <ximion> the big news is the better icon handling though, I am incredibly pleased with that now
[17:15] <Laney> https://paste.ubuntu.com/p/XVQfzKwzhQ/
[17:19] <jbicha> ?
[17:26] <GunnarHj> jbicha: Just a friendly ping about that gnome-shell MP. I'm curious about the result...
[17:32] <willcooke> gnight all.  Have a restful weekend.
[17:43] <jbicha> GunnarHj: do you need to do something to make ubuntu-settings available for translation in LP?
[17:44] <jbicha> do I need to upload the gettext domain change first?
[17:44] <jbicha> https://bazaar.launchpad.net/~ubuntu-desktop/+junk/ubuntu-settings/revision/131
[17:47] <GunnarHj> jbicha: I just approved the template in the import queue. Let's see if that's it.
[17:48] <GunnarHj> jbicha: Hasn't that package had any translatable strings before?
[17:50] <jbicha> no
[17:50] <jbicha> unless you wanted to translate "Amazon" but we decided today that wasn't very important
[17:53] <Laney> Don't upload that revision
[17:53] <Laney> I'm fixing dh-translations.
[17:55] <jbicha> ok
[17:55] <GunnarHj> jbicha: Done now. Gave it high priority, so it's on the first page of respective language's overview. Now I'll be busy for a while with figuring out a nice Swedish translation of "Welcome to Ubuntu". :)
[17:57] <Laney> there, I uploaded a diff to bug #1751820
[17:57] <ubot5`> bug 1751820 in pkgbinarymangler (Ubuntu) "Teach dh_translations how to deal with meson" [Medium,Triaged] https://launchpad.net/bugs/1751820
[17:59] <jbicha> we have a few days before the next langpack update any way :)
[18:02] <Laney> I'm looking for review if anyone's feeling up for it
[18:02] <Laney> anyways, I'll be off now
[18:02] <Laney> feel free to also upload it if you get super brave
[18:05] <seb128> Laney, look fine in principle to me but I'm about to call it a day/week as well so detailled review is for monday (unless GunnarHj or someone beats me)
[18:06] <seb128> have a nice w.e desktopers!
[18:16] <jbicha> GunnarHj: thanks, it looks like it works https://translations.launchpad.net/ubuntu/bionic/+source/ubuntu-settings/+pots/ubuntu-default-launchers
[18:17] <GunnarHj> jbicha: Indeed. Final confirmation at next langpack update.
[18:42] <andyrock> seb128: unsure you're still around but https://code.launchpad.net/~azzar1/update-notifier/xdg-autostart-path-units/+merge/342808
[20:03] <oSoMoN> good night, and have a good week-end desktoppers!