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

jbichaximion: could you explain why http://appstream.ubuntu.com/bionic/main/issues/gnome-control-center.html complains about icon-not-found00:14
jbichabut https://appstream.debian.org/sid/main/issues/gnome-control-center.html does not00:14
jbichathe icon is installed by gnome-control-center-data which is a dependency of gnome-control-center00:14
jbichadoes appstream.ubuntu.com need a newer asgen version?00:14
jbichathis issue affects several other apps, Firefox is a prominent example00:15
jbichaoh, Firefox's problem might be different, it is a symlink which is a warning but it seems to work fine in Debian that way00:16
jbichaDebian & Ubuntu's Firefox packaging is different but I think that part might be close enough00:16
jbichaanyway, never mind Firefox for now00:17
ximionfirefox does symlinking stuff, so that won't work00:17
ximionjbicha: the Ubuntu appstream-generator is very old, the icon finding logic has been refined quite a bit00:18
jbichait seems to work in Debian00:18
jbichaLaney: can you update asgen some time to fix gnome-control-center? ^00:18
ximionfrom looking at the code, the icon should have been found in Ubuntu, but there were quite some changes in the icon finding & resizing code00:20
ximionasgen 0.6.8 had relaxed icon scaling rules, and 0.7.0 and 0.7.1 have a lot of improvements for HiDPI icons00:21
* ximion puts his bets on the 0.6.8 change for making this particular case work00:22
jbichaximion: do you want to file a Ubuntu Feature Freeze Exception for asgen 0.7.0 in bionic?00:22
ximionjbicha: for that I would also need a freeze excaption for appstream, dcontainers and stdx-allocator00:22
ximionwhich is quite a lot of stuff00:22
ximionI don't see those changes breaking anything, but you it's harder to justify00:23
ximionI 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 :P00:25
ximionmaking a docker container is easy though00:25
ximion(or building it on bionic ^^)00:26
jbichadcontainers and stdx-allocator should be easy, they're just new packages for universe00:26
ximionjup, mustache-d is also easy, no other rdeps. the major one is appstream itself, because that's a mixed bugfix and feature release00:26
ximion(having it would make the KDE people very happy though ^^)00:28
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:34
jbichado you want to file the FFE bug? since you're more of the expert here00:35
jbichawhat's needed for https://salsa.debian.org/gnome-team/gnome-software/commit/e52d2fa5 to work?00:36
ximionwould I need to file a bug report only for appstream-generator, or one for each of its dependencies as well?00:36
jbichadon't worry about dcontainers, stdx-allocator, and mustache-d. Those are already being synced00:37
jbichaeverything else can be in one bug I assume00:37
ximionin 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:37
ximionjbicha: 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.1400:38
ximion*compile00:38
* ximion can't type anymore00:38
jbichaok00:39
ximionfor that change you need the appstream 0.12 package00:39
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
ximionand also adds an obvious way for users to prevent icon downloads :P00:40
ximionhmm, appstream 0.12 also updates the SPDX license list to 3.0, I guess I could make a good case for it00:44
ximionthis just reminded me that asgen needs metadata_license expression parsing, almost exclusively for GNOME Terminal...00:51
* ximion sees to add that quickly00:51
Trevinhorobert_ancell: you know if there's a way to expose the appdata info from a snap?01:25
Trevinhorobert_ancell: I mean, if i put it somewhere inside the snap, it could be parsed by gnome-software?01:26
TrevinhoI know there's a way for importing infos so far, but not really something that is enough01:26
Trevinhonor is working fine to be fair01:26
robert_ancellTrevinho, there is not sadly. We proposed meta/appinfo.yml or similar but it was shot dwn01:44
Trevinhorobert_ancell: mh, so is there any plan for something like exposing all these metadata?01:45
Trevinhorewriting them just in other format looks silly to me01:45
robert_ancellTrevinho, 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:46
Trevinhorobert_ancell: yeah, i'm aware of that01:47
Trevinhorobert_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_ancellwhat do you mean "know them"?01:47
Trevinhowouldn't be better to say: here's my appdata, then all the apps already speaking that language can parse it01:47
Trevinhorobert_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
Trevinhoas snapcraft could not bother of knowing them all01:48
Trevinhosome of these are just important at runtime or once the app is installed01:49
robert_ancellTrevinho, appdata important at runtime?01:49
Trevinhorobert_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 it01:51
robert_ancellTrevinho, does anything use that?01:52
Trevinhorobert_ancell: well, isn't for example the shell using it as shown by jbicha for getting the alias names for desktop files?01:53
Trevinhorobert_ancell: https://gitlab.gnome.org/GNOME/gnome-shell/merge_requests/41/diffs01:53
ubot5-ngGNOME bug (Merge request) 41 in gnome-shell "WIP: Use AppStream for mapping renamed apps where possible" (comments: 0) [Opened]01:54
jbichahttps://github.com/3v1n0/telegram-snap/issues/501:54
* robert_ancell digs in01:56
robert_ancellTrevinho, jbicha I can't find any of those projects where they contain the old name - where is it stored?02:00
jbicharobert_ancell: https://gitlab.gnome.org/GNOME/gnome-tweaks/commit/c553730f ??02:00
robert_ancelljbicha, 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:01
robert_ancellwhy is that not just in the .desktop file?02:02
* Trevinho takes a break... 02:02
jbichamaybe they figured it was easier to update the appstream standard than the .desktop standard?02:03
jbichaappstream uses provides for other things too02:04
robert_ancellsure02:04
robert_ancellIt seems overkill for something to have to open the AppStream database (which takes ages to load) just to note a .desktop file was renamed02:04
ximionrobert_ancell: libappstream needs about 1-2sec for that, appstream-glib can't be that much slower02:23
robert_ancellximion, that's pretty heavy02:24
ximionif you use libas' cache it's just 380msec ^^02:24
ximionTrevinho: 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 far02:25
robert_ancellthat sounds more reasonable :)02:25
robert_ancellTO DEATH02:25
ximionwell, a lot and often ^^02:25
ximiontime appstreamcli get org.freedesktop.appstream.cli02:26
ximionreal    0m0,279s02:26
robert_ancellximion, so that uses a cache?02:26
ximionhmm, it's okay, I think02:26
ximionjup02:26
ximiontime appstreamcli get org.freedesktop.appstream.cli --no-cache02:27
ximionreal    0m0,457s02:27
ximion0m0,420s on subsequent runs02:27
robert_ancellximion, where is the cache stored / which API?02:27
ximion. /var/cache/app-info/gv/ for a system-wide version, otherwise each tool using libappstream can place one wherever it wants02:28
robert_ancellah, as_pool_load_cache_file?02:28
ximionthat cache is basically just a compressed GVariant :P02:28
robert_ancellsounds reasonable.02:29
robert_ancellximion, do you just call as_pool_set_cache_flags before as_pool_load?02:29
robert_ancell(to use the system cache)02:29
ximionbefore load02:29
ximionhmm, appstream-util takes indeed longer for the same action, but I have to use "search" there, so not a very fair comparison02:30
robert_ancellbtw, I saw there might be an as_pool_load_async coming?02:30
ximionjup, I really want that, but so far I never found time for it02:32
ximionthe API is very synchronous at time02:32
ximion(the easy way around it is to just throw it into a separate thread and wait for that to become ready)02:33
ximionI 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_ancellximion, as long as nothing inside that thread does something dangerous...02:34
ximionthere 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 on02:37
ximionyou shouldn't access an AsPool while it's loading ;-)02:38
ximionread actions might work on it, but adding components to it might result in trouble02:39
robert_ancellximion, right, but what operations are going on inside as_pool_load?02:39
ximionnothing that would impact other threads ^^02:40
robert_ancell:_02:41
robert_ancell:)02:41
ximiondisk reads, file parsing, lots of hash table lookups and insertions, etc. but no writing to disk or modifying of other connected objects02:41
ximionif there was something that would cause problems, I would know that by now, because almost all software centers use it in threads02:41
ximion(well, Elementary didn't care about the load delay for some time, but they might have changed that now)02:42
ximionI'm not exactly sure which features you need, but disabling reading of desktop files improves performance by quite a bit02:43
ximion(with metainfo files on the second place)02:44
ximionthose are sometimes refreshed even if a cache is used, because the user might have installed things meanwhile02:44
ximion(see AsPoolFlags)02:44
ricotzchrisccoulson, please push your packaging branches06:15
=== JanC_ is now known as JanC
seb128good morning destkopers07:08
didrocksgood morning07:14
dufluMorning seb128, didrocks07:17
didrockshey duflu07:18
seb128hey duflu, how are you? happy friday!07:19
dufluseb128, relatively good, you? Happy Friday07:19
seb128I'm good thanks, a bit sick but it's ok, almost the w.e and some rest :)07:21
=== pstolowski|afk is now known as pstolowski
didrocksTrevinho: when you get some time, spotted a theme regression: bug #1761684, unsure if that impacts other apps07:51
ubot5`bug 1761684 in ubuntu-themes (Ubuntu) "Regression in lists like gnome-boxes" [Undecided,New] https://launchpad.net/bugs/176168407:51
Laneymoin08:00
seb128hey early Laney! how are you?08:03
seb128poor Trevinho consider as the theme owner08:04
didrockshey Laney08:06
Laneyhey seb128 didrocks08:06
LaneyI'm alright thx, seems sunny again, what about you?08:06
seb128I'm good, a bit sunny here today as well but still quite windy08:07
didrockssunny here as well!08:07
seb128hate the wind :/08:07
willcookebah. No sun here.08:07
seb128it's supposed to be nice this w.E though, up to 19°C08:07
Laneyjbicha: 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
seb128hey willcooke08:08
oSoMoNgood morning desktoppers08:11
willcookejibel, thanks again for getting the beta out.08:16
jibelwillcooke, you're welcome08:20
oSoMoNseb128, thanks for your comment on bug #1761585, I'm preparing a new upload08: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/176158508:21
didrockshey oSoMoN, willcooke08:21
Laneygoing with the distro patch for gnome-initial-setup then?08:23
Laney'lo oSoMoN & willcooke08:24
oSoMoNhey didrocks, hey Laney08:24
tjaaltonwhen gnome-shell crashes and respawns, the dock seems to not come back?08:28
seb128oSoMoN, yw!08:28
seb128lut oSoMoN :)08:28
didrockstjaalton: no, it should come back, it's part of the same process and under the same env08:28
didrockstjaalton: if you start it manually from a tty, it doesn't have the env var though08:28
seb128Laney, 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 anyway08:29
tjaaltondidrocks: ok, I filed the crasher now, bug 176169708:29
ubot5`Error: Launchpad bug 1761697 could not be found08:29
tjaaltonbah, bug 176169708:29
didrockslet's wait for retracer/duplicate08:30
Laneyseb128: we want to do that?08:30
Laneyman, I should get more in the loop08:30
Laneythx for the info08:30
tjaaltonlooks like framebuffer compression is not happy on skylake with 4.15 :/08:30
seb128Laney, they have been mentions of looking at enabling some of the upstream pages08:30
tjaaltonscreen keeps blanking every 20s or so08:31
seb128I think we are at least going to look at it seriously next cycle08:31
seb128Trevinho, Laney, on the git workflow card I agree with that Laney wrote08:32
oSoMoNseb128, test packages based on your upload building in https://launchpad.net/~osomon/+archive/ubuntu/ibus-lp1761585/+packages08:33
seb128oSoMoN, great, thx08:33
Laneyk08:39
tjaaltondidrocks: heh, one crash later and the dock is back :)08:49
didrocksmaybe a glitch :p08:54
oSoMoNseb128, ibus upload tested and ready at https://people.canonical.com/~osomon/ibus/, would you mind uploading it for me?09:27
seb128I can do that09:27
seb128oSoMoN, I'm going to include the pending queue upload in the changes in case they reject the oldest and only accept the new upload09:29
seb128oSoMoN, done09:29
seb128willcooke, 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:34
willcookePPTP 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:40
oSoMoNseb128, thanks!09:41
willcookeYou can set up a PPTP server on Ubuntu in a few mins vs long pages of reading for openvpn09:41
willcookeThat said, yes, openvpn is better.09:41
seb128oSoMoN, np!09:41
willcookeIt 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
willcookeI 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 think09:42
willcookeWhere was the discussion?  I must have missed it09:43
willcookeyou think that's something we would change for 18.04?09:44
seb128willcooke, #ubuntu-devel yesterday a bit and https://bugs.launchpad.net/ubuntu/+source/network-manager-openvpn/+bug/175241709:44
ubot5`Ubuntu bug 1752417 in network-manager (Ubuntu) "Out of the box, Ubuntu Bionic offers only insecure VPN option" [Medium,Triaged]09:44
seb128willcooke, 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 pptp09:45
seb128it might be a late now for that though09:45
seb128but maybe we can look at doing it for .1 if we think it makes sense?09:45
willcooke+109:46
seb128Laney, ^ do you have any opinion from a r-t perspective?09:46
willcookeOn touch I argued to include PPTP by default09:46
willcooke:)09:46
seb128both?09:46
seb128or just pptp?09:46
willcookeboth09:46
willcookehappyaron, hey!  Do you know if pptp and openvpn work well in China?  I think it used to be just PPTP?09:47
LaneyI think that the security team should decide whether PPTP is a thing to recommend by installing it by default09:47
Laneyand I'd be OK with including openvpn depending on whether it brings in extra stuff too09:48
willcookeseb128, I'd say it was "safe" to add openvpn now.  The UI works fine from my testing09:48
Laneyit's in universe btw09:48
Laneysourcei s in main tho09:48
seb128right09:48
seb128it had an approved MIR09:49
seb128so I guess it's fine to bring the binary back in main if we want to?09:49
Laneyguess so09:50
seb128nice, thx09:50
willcookeI'll ping security for their input09:50
Laneybut I'd defer to the security team on what should be included09:50
seb128+109:50
seb128willcooke, thx09:50
Laneyif PPTP is as bad as people are claiming then it would be ok to make people have to go install it09:50
seb128right09:50
seb128willcooke, can you Cc me on that ping if it's via email?09:51
seb128Laney, 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 yet09:51
seb128that's not a regression from the new revisions, I only added the fix for the other test issue and a segfault fix from andyroc_k09:52
willcookeseb128, talking in #security now09:52
willcookeseb128, but they're mainly asleep for a few hours09:52
seb128right09:52
willcookeWow, terrible English09:53
willcookesorry09:53
Laneyseb128: ok09:54
seb128Laney, thx!09:54
seb128bbiab, changing location09:57
=== pstolowski is now known as pstolowski|lunch
jbichaLaney: hi, I made ubuntu-settings translatable (needed for the Ubuntu Welcome .desktop). I guess we should make the Amazon .desktop translatable too10:59
jbichawhat 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-launcher11:00
Laneyis "Amazon" translated differently?11:01
LaneyI don't really want that thing on $PATH if we can help it11:01
jbichaso I just visited amazon.cn and they have both amazon.cn as their logo and 亚马逊中11:04
jbichaok if you don't want it in PATH, we can leave it as is11:04
LaneyIf 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 word11:06
seb128if we never got a report about the fact that it can't be translated I wouldn't change it11:07
Laneywhen I made this thing I think that the Chinese website used the text amazon in its logo though11:07
seb128it's more likely to create issues by having some locale wrongly translating it than being useful11:07
jbichaok, we can wait for someone to file a bug :)11:09
didrocks+111:09
Trevinhodidrocks: ok on theme...  seb128, sadly... yes :-/11:11
* seb128 hugs Trevinho11:12
seb128Trevinho, good morning, happy friday! you are up early today11:12
seb128how are you?11:12
Trevinhohey seb128 happy friday too11:12
Trevinhoyeah, I need to stop a bit earlier today so...11:12
seb128Trevinho, 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-way11:13
seb128unsure it was needed in the board11:13
jbichaseb128: 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 all11:15
jbichawe have a few remaining issues like bug 1740741 and bug 174074111: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/174074111:16
jbichaI'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-settings11:16
jbichaI 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 GNOME11:17
seb128wfm but it's a bit late in the cycle for that no?11:17
Trevinhoseb128: mh I wanted to add it to track all the items that we've to move to thaat system though11:17
jbichaseb128: it's a bugfix :) and I have realized that virtually no one actually wants the display .desktop11:18
jbichacheck out the reviews in Ubuntu Software for imagemagick :(11:18
seb128jbicha, wfm as said, I just feel like it's late for changes that could surprise users/other desktops11:19
seb128but I don't think it's a big deal either11:19
LaneyTrevinho: UNITY_LAUNCER_FAVORITES :(((((((((((((((((((((((((11:19
willcookeTrevinho, hey!  Do you use VPNs much while you're travelling?11:35
Trevinhowillcooke: well, quite a bit, why?11:35
Trevinhowillcooke: but... well, mostly company one or the one I've in my rapsberry at home11:36
Trevinhowillcooke: why?11:36
TrevinhoLaney: yeah, that's for removing the old names... eventually.11:36
LaneyTrevinho: yeah, you typoed launcher11:36
TrevinhoLaney: it's its name :-D11:37
Laney:||||||||||||11:37
willcookeTrevinho, which sort of VPN do you use?11:37
willcookeopenvpn., pptp, other?11:38
Trevinhowillcooke: as said, mostly those... I wanted to subscribe to torguard as there's a good deal (see on reddit)11:38
Trevinhoah, openvpn11:38
willcookekk, thanks11:38
Trevinhomy provider provided router natively only supports pptp, but it's not really nice11:39
Trevinhoso I have a raspberry doing the job for me and works nicely11:39
LaneyTrevinho: wait what, the schema name is com.canonical.Unity.Launcher11:40
Laneywhere does this launcer come from?11:41
TrevinhoLaney: I was jocking :)11:42
TrevinhoLaney: I've pushed the fix right now11:42
Trevinhoouch you were publishing?11:43
Trevinho:-D11:43
Trevinhodamn11:43
Laneyfix it for the next upload11:43
Laneyno need to build for this11:43
Trevinhowas already triggered :-D11:44
Trevinhowell, we'll republish later then11:44
Trevinhoor I can stop maybe11:44
Trevinhobut bileto will copmlain11:44
Trevinhocomplain*11:44
Laneysave the environment11:44
Trevinhoif that would be enough...11:44
Laneythis ubuntu-welcome thing is trying to send a report even when I picked "No, don't send any data."11:53
Laneys/data/info/11:53
jbichamy understanding is that the design was for it to send an opt-out ping so that we know how many users are opting out11:55
seb128Laney, right, it send an empty report in that case, that's something under discussion/that willcooke was going to discuss with legal as well12:05
=== pstolowski|lunch is now known as pstolowski
Laneyseb128: ok, I filed a bug because I think that should be explicitly decided12:36
seb128Laney, thx12:36
seb128Laney, 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:37
seb128and there were some questions/replies about it12:38
LaneyI remember12:38
seb128but a bug is a good reference point as well12:38
LaneyBut the UI is quite misleading imo12:39
Laneyalso I was hoping that part would change before it made it into ubuntu :P12:40
seb128right, me to, willcooke is going to check if we really need it or not12:44
seb128also even if the UI was clear I'm unsure it would be a good option12:45
seb128tell users "we are going to send that in any case", you can't even close the wizard12:45
Laneyagreed12:45
seb128jbicha, how do you quote text to reply in trello?12:51
jbicha>12:51
jbichaI just used the > character to fake quote :)12:51
seb128lol, thx12:51
LaneyI'm really tired of that discussion :(12:54
seb128Laney, sry :(12:58
seb128we should probably take that out of the trello12:58
seb128rather on the community hub where people who want to argue can12:58
Laneynot you seb12812:59
Laneyfeel free imho12:59
LaneyI'd advertise it as 'please use, but details might change' for the first cycle or something12:59
seb128right12:59
Laneyso if Launchpad turns out to be shit for some reason then it can be changed12:59
Laney!ohmy13: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/ChannelList13:01
oSoMoNseb128, 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/176158514:04
GunnarHjseb128: 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/175654714:12
seb128jbicha, ^ maybe?14:13
seb128GunnarHj, I can add to my list but it's not likely for today at this point14:13
jbichasure, I can give it a try14:13
GunnarHjseb128, jbicha: Ok, thanks!14:14
jbichaseb128 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
seb128willcooke, ^14:14
jbichaI guess the issue is that the implementation just uses the selected timezone and there is a Design question about how to word that14:15
seb128jbicha, note that I approved the template so it can be translated now, but yeah better if the strings stop changing14:15
willcookempt ^ :)  Can you help with the string above?14:15
willcookejbicha, got a link handy to show where that appears?14:15
jbichawillcooke: mpt isn't here :)14:15
willcookeoh14:15
willcookeoh yeah14:16
willcookewow, I must have typed it without trying to tab complete.  Thats a first.14:16
jbichawillcooke: you can install https://launchpad.net/ubuntu/+source/gnome-initial-setup/3.28.0-2ubuntu114:16
jbichaI can open a bug with a screenshot if you want14:16
willcookejbicha, will do and I'll email mpt and cc you14:16
willcookejbicha, nah, s'fine.  I'll do14:16
jbichaafter installing gnome-initial-setup, you need to log out and log back in14:17
Laneyit'd be nice if those pages got icons on them14:17
Laneythose make the upstream g-i-s pages look quite smart14:17
jbichaonce you've completed the wizard, you can reset it so it starts again with   rm ~/.config/gnome-initial-setup-done14:17
jbicha+114:18
didrocksjbicha: for the report one, you need to clean that out in ~/.cache/ubuntu-report/<distro>.version14:19
didrocks(if the report was sent successfully to the server)14:19
didrocks(meaning, when we'll have a server :p)14:20
seb128oSoMoN, ibus SRU uploaded14:20
oSoMoNseb128, awesome, thanks!14:20
seb128yw!14:20
seb128l:14:20
seb128ups14:20
seb128Laney, icons like on the page content?14:21
Laneyyeah14:21
Laneythey have icons above the header14:21
seb128k, they probably changed compared to the screenshots I saw14:22
seb128e.g https://www.tecmint.com/wp-content/uploads/2013/12/fed-19.jpg14:22
seb128well I saw some have but it didn't seem obvious they all did to me14:23
didrocksfedora 19 ;)14:23
jbichaseb128: XDG_CURRENT_DESKTOP=GNOME /usr/lib/gnome-initial-setup/gnome-initial-setup --existing-user14:23
Laneyoverride XDG_CURRENT_DESKTOP and have a look14:23
seb128I was concerned it would screw some of my configs :p14:23
seb128I miss guest sessions :/14:23
jbichait won't break anything14:23
seb128like no code ever had bugs right :)14:24
jbicha95.8% sure this time :)14:24
seb128hehe14:24
seb128oSoMoN, bah, ibus SRU rejected because the version already exist from the archive (from a newer serie I guess), I'm tweaking the number and reuploading14:26
oSoMoNseb128, oh, I didn't think of checking other series…14:27
seb128oSoMoN, yeah, neither did I :p14:27
oSoMoNseb128, that was in yakkety: https://launchpad.net/ubuntu/+source/ibus/1.5.11-1ubuntu314:29
seb128oSoMoN, right, that was expected, the next revision is usually in the next serie :)14:29
oSoMoNbut I definitely wouldn't have thought of checking yakkety, being EOL and hidden on https://launchpad.net/ubuntu/+source/ibus14:30
willcookexnox, nice work on that dns/captive portal bug.  That must have been hard to find14:31
seb128oSoMoN, easier to check on https://launchpad.net/ubuntu/+source/ibus/+publishinghistory14:31
seb128willcooke, which one is that?14:31
didrockswe should really generalize <version>-XubuntuY.1, .2, .3 when the packages aren't synced to the new series anymore14:31
willcookeseb128, https://bugs.launchpad.net/ubuntu/+source/systemd/+bug/172723714:31
ubot5`Ubuntu bug 1727237 in systemd (Ubuntu Artful) "systemd-resolved is not finding a domain" [High,Triaged]14:31
* flocculant must tweak his xubuntu ping14:32
seb128willcooke, 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
seb128didrocks, yeah, that avoids those issues14:33
oSoMoNseb128, indeed, I'll make sure I check there next time I prepare a SRU14:34
didrocksseb128: I'm guilty as well I must say :)14:34
willcookejbicha, I was looking at the designs from mpt14:37
willcookehttps://docs.google.com/document/d/1IDQwpqTsB7Cmw413ApXioHeO2E9nv8bHugjxCOg9A-Q/edit14:37
willcookeI think "FIXME" is supposed to show the location, I don't think there is anything wrong with that string itself14:38
jbichaok I see14:38
jbichawe need to remove FIXME from the string at least though14:39
willcookeyeah14:39
jbichait can be a separate string if we need a TODO reminder14:39
willcookealso, related, Laney - we should have icons a la upstream: https://github.com/CanonicalLtd/desktop-design/issues/6614:40
ubot5-ngCanonicalLtd 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 logged14:40
willcookeI dont think that issue should be closed though14:40
Laneyare those the designs?14:40
willcookeYes, but14:40
Laneyshould probably attach some screenshots of the upstream pages as they aren't going to fit in14:41
willcookewe've already pointed out that they need to move the nav to the headerbar14:41
willcookeand yeah, as you say, they wont fit14:41
Laneythat first screenshot labels the app menu as a window switcher14:44
willcookeohhhhh14:44
willcookeI've just commetned on the wrong trello card14:44
willcookesorry didrocks14:44
seb128lol14:45
willcookeI will fix it14:45
willcookeLaney, yeah, see my comment at the bottom :)14:45
Laneyok th14:45
Laneyx14:45
* didrocks sobs, how dared you!14:45
willcookefixed14:46
willcookehttps://trello.com/c/OaRRF8L3/228-implement-first-run-wizard14:46
* willcooke hugs didrocks 14:47
didrockswaow, your actions are even removed from the history…14:47
didrocksyou rewrote the past!14:47
* didrocks hugs willcooke back14:47
didrocksgit-based-trello14:48
willcookehttps://www.youtube.com/watch?v=I1QF_GuGOGo14:48
didrocksgood fit14:49
Laneyjbicha: does dh_translations work with meson?14:51
seb128Laney, good one :)14:52
Laneyhmmmmmmmmm?14:52
jbichaum, I think it sort of works the way I'm doing it in ubuntu-settings14:52
seb128Laney, bug #175182014:52
ubot5`bug 1751820 in pkgbinarymangler (Ubuntu) "Teach dh_translations how to deal with meson" [Medium,In progress] https://launchpad.net/bugs/175182014:52
seb128meson is making that easy :/14:53
seb128there is no clear definition of the domain nor way to query it14:53
seb128unlike config.h definition in good old autotools14:53
seb128or cmake14:53
Laneytbh if you could giv ethe domain to dh_translations that would be good enough14:54
seb128yeah, that's what sort of my conclusion/what I wrote on that bug14:55
Laneyindeed14:55
Laneyjbicha: what kind of sort of?14:55
LaneyAFAICS they just won't be stripped14:55
seb128it's a bit unfortunate that they changed the "standard" "make domain.pot" target as well14:56
seb128need to do "domain-pot-update" now in meson project14:56
jbichaLaney: https://bazaar.launchpad.net/~ubuntu-desktop/+junk/ubuntu-settings/view/head:/debian/rules14:56
seb128so we need to special case that as well14:56
LaneyI know, I'm reviewing the upload in the queue14:56
didrocksannoying having to override all dh_translations call though…14:57
Laneyso you get the translations in Launchpad but they won't actually be used by the desktop file...?14:57
seb128yeah, go GNOME using "better" tools :p14:57
didrockswell, the general idea isn't bad, it's just that distros weren't taken into account while designing it seems…14:58
seb128rather "translations" than "distro"14:59
seb128when I talked to Jussi he basically said that those were basically not part of meson/things dealt with by other people15:00
seb128as 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 later15:00
seb128which they eventually did but it feel like it was hacked on top15:00
seb128oh well, it sort of work but could have been nicer in that regard15:00
seb128k, moving location back, brb15:03
Laneylaney@nightingale> meson introspect --targets | jq -r -M '.[].name | select(endswith("-pot")) | sub("-pot"; "")'15:05
Laneyubuntu-default-launchers15:05
Laneyphew15:05
didrocks\o/15:05
didrocksso, we can automate that in dh_translations15:06
didrockshum, in some projects, meson introspect --targets gives me a stacktrace15:07
jbichaI guess I do need to add X-Ubuntu-Gettext-Domain to the .desktop's here manually though15:07
Laneythat's the main missing bit15:07
GunnarHjdh_translations needs to know about the builddir too, right?15:08
Laneyyou need to run that command in the build dir15:09
Laneythere's probably a debhelper library function to get that15:10
Laneyalso, hey GunnarHj15:10
GunnarHjLaney: 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
didrocksLaney: even in the builddir, I needed to do meson introspect . --targets to not get a stacktrace15:11
GunnarHjHi Laney. :)15:11
didrocksit's taking sys.argv[1:]15:11
Laneydidrocks: um, ok, I just pasted what I ran so shrug I guess15:11
didrocksLaney: I bet… I'm unsure what's different here.15:11
didrocksand the --help clearly says it's optional15:12
didrocksbut the stacktrace here is quite real :p15:12
GunnarHjdidrocks: Are you saying that perl could grab the builddir via some sys.argv[1] variable?15:12
didrocksGunnarHj: no, I'm just talking about meson introspect, but yeah, perl dh_* can get the builddir15:13
LaneyI'm sure it can somehow, the other dh_* must do that15:13
didrocksI saw a lot of self->builddir() or something similar15:13
didrocksin various debhelper releated functions15:14
GunnarHjThat sounds promising. Possibly we can fix that bug completely then. :)15:14
didrocksLaney: do you mind trying it in github.com/ubuntu/gtk-communitheme ?15:14
didrocksmkdir build15:14
didrockscd build && meson introspect --targets15:14
didrocks(without the rest, just checking if you get the stacktrace as well)15:14
didrocks(sorry mkdir build -> meson build)15:16
Laneylaney@nightingale (master|✔)> meson introspect --targets | jq -r -M '.[].name | select(endswith("-pot")) | sub("-pot"; "")'                                                                           ~/temp/gtk-communitheme/build15:17
Laneylaney@nightingale (master|✔)>                                                                                                                                                                         ~/temp/gtk-communitheme/build15:17
Laneythere used to be a program called mesonintrospect before15:19
didrocksLaney: 0.45.1-1ubuntu2?15:20
jbichadidrocks: I get the introspect crash with ubuntu-settings too15:21
didrockshttp://paste.ubuntu.com/p/DqHhp2bkBC/15:21
didrocksjbicha: and not anymore if you specify the build-dir in the arg?15:22
jbichayes, that sounds like an upstream bug15:23
* 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 this15:24
Laneydunno, but feel free to file it upstream I guess15:24
* Laney is the lucky one15:25
didrocksLL (Lucky Laney)15:25
* Laney wears a badge15:31
seb128willcooke, bug https://bugs.launchpad.net/ubuntu/+source/network-manager-openvpn/+bug/1752417 turned into a ffe for the openvpn addition15:46
ubot5`Ubuntu bug 1752417 in network-manager (Ubuntu) "[ffe] Out of the box, Ubuntu Bionic offers only insecure VPN option" [Medium,Triaged]15:46
=== pstolowski is now known as pstolowski|eow
willcookethanks seb12815:59
willcookeinternet is wonky today15:59
* willcooke suspects that there will be a BT van down the street15:59
* didrocks goes off. Enjoy for the week-end everyone!16:00
didrockss/for//16:00
dokojbicha, seb128: fyi, I backported the brotli alignmnet fixes. needs a reference now in a package or a seed16:09
ximionLaney: 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 requested16:15
ximionI think I could make a good case for it, but I would also in no way be surprised if that change gets rejected16:15
jbichadoko: thank you. brotli itself doesn't help me much without bug 1742743 though16:17
ubot5`bug 1742743 in woff2 (Ubuntu) "[MIR] woff2" [Undecided,New] https://launchpad.net/bugs/174274316:17
Laneyximion: Depends on if you're going to break the post-update-command or not 😕16:19
ximionI 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
ximionso, no post-update hook will get broken ^16:20
ximionLaney: 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
Laneyximion: ok, feel free to file the FFe requests and explain why it's safe in there ;-)16:22
ximionalso, an update to SPDX v3.0 is good in an LTS release16:22
ximionbut most important question for me is actually: would this enable you to update appstream-generator on the server easier/faster?16:23
Laneya snap would be best :P16:23
Laneybut without that - kind of but not massively, still need to make it buildable in xenial16:23
Laneywe don't use the asgen archive package there at all16:24
ximionI 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 snap16:25
ximionalso, getting yarn into the snap will be... fun...16:26
* ximion never had to make a snap that was that complicated16:27
seb128doko, 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 first16:29
ubot5`bug 1742743 in woff2 (Ubuntu) "[MIR] woff2" [Undecided,New] https://launchpad.net/bugs/174274316:29
ximionLaney: the new asgen will give you much better icon tarball sizes: https://appstream.debian.org/data/sid/main/16:32
ximionthe reduced memory usage doesn't appear to be as strong anymore though, and I only updated the GLib bindings...16:32
ximionso I sort of blame that16:32
Laneynew version of gir-to-d?16:33
ximionthat and new wrapfiles is what changed16:34
ximionjup16:34
ximionI'll need to ask the author of both to please make a new release, Git master seems to have a lot of useful changes16:34
ximionbut in general, memory usage is much lower, still16:35
ximionthe big news is the better icon handling though, I am incredibly pleased with that now16:36
Laneyhttps://paste.ubuntu.com/p/XVQfzKwzhQ/17:15
jbicha?17:19
GunnarHjjbicha: Just a friendly ping about that gnome-shell MP. I'm curious about the result...17:26
=== mdeslaur_ is now known as mdeslaur
willcookegnight all.  Have a restful weekend.17:32
jbichaGunnarHj: do you need to do something to make ubuntu-settings available for translation in LP?17:43
jbichado I need to upload the gettext domain change first?17:44
jbichahttps://bazaar.launchpad.net/~ubuntu-desktop/+junk/ubuntu-settings/revision/13117:44
GunnarHjjbicha: I just approved the template in the import queue. Let's see if that's it.17:47
GunnarHjjbicha: Hasn't that package had any translatable strings before?17:48
jbichano17:50
jbichaunless you wanted to translate "Amazon" but we decided today that wasn't very important17:50
LaneyDon't upload that revision17:53
LaneyI'm fixing dh-translations.17:53
jbichaok17:55
GunnarHjjbicha: 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:55
Laneythere, I uploaded a diff to bug #175182017:57
ubot5`bug 1751820 in pkgbinarymangler (Ubuntu) "Teach dh_translations how to deal with meson" [Medium,Triaged] https://launchpad.net/bugs/175182017:57
jbichawe have a few days before the next langpack update any way :)17:59
LaneyI'm looking for review if anyone's feeling up for it18:02
Laneyanyways, I'll be off now18:02
Laneyfeel free to also upload it if you get super brave18:02
seb128Laney, 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:05
seb128have a nice w.e desktopers!18:06
jbichaGunnarHj: thanks, it looks like it works https://translations.launchpad.net/ubuntu/bionic/+source/ubuntu-settings/+pots/ubuntu-default-launchers18:16
GunnarHjjbicha: Indeed. Final confirmation at next langpack update.18:17
andyrockseb128: unsure you're still around but https://code.launchpad.net/~azzar1/update-notifier/xdg-autostart-path-units/+merge/34280818:42
oSoMoNgood night, and have a good week-end desktoppers!20:03
=== maxb_ is now known as maxb

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