[04:30] <zyga> Good morning
[04:31] <zyga> I did an experiment and the glob issue cannot explain the bug reported by our customer
[04:31] <zyga> I will focus on debugging that next
[04:38] <zyga> also the 2.32 branches cannot land because spread has issues on linode
[04:38] <zyga> many machine images don't pick up the ssh connection
[04:38] <zyga> must be Friday :)
[05:07] <mborzecki> morning
[05:10] <zyga> hey
[05:10] <zyga> mborzecki: there must be a 2nd bug
[05:10] <zyga> mborzecki: what I saw and fixed cannot explain the issue reported by the customer
[05:10] <zyga> mborzecki: I'd love to get to the bottom of 2.32 test failures
[05:10] <zyga> mborzecki: it seems we cannot reach our booted images anymore.
[05:11] <zyga> mborzecki: this is all on linode
[05:11] <zyga> mborzecki: for logs see #5110
[05:11] <mup> PR #5110: interfaces/apparmor: fix incorrect apparmor profile glob (2.32) <Critical> <Created by zyga> <https://github.com/snapcore/snapd/pull/5110>
[05:17] <mborzecki> globs everywhere
[05:20] <mborzecki> had to double check what SecurityTagGlob looks like
[05:24] <mborzecki> zyga: travis build failure is the same as in #5073, my guess is that the images do not work, iirc those were -32 ones
[05:24] <mup> PR #5073: set up journal streams in user session application autostart (2.32) <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/5073>
[05:24] <mborzecki> if we do .6 it woul dbe nice to get 5073 in as well
[06:03] <zyga> mborzecki: I asked mvo about what else to include but IIRC he wanted critical things only
[06:03] <mborzecki> ok
[06:04] <zyga> I wonder why the -32 bit images broke on us
[06:45] <mborzecki> hm 249 test files modified, if i split that into groups of 20, we'll get 12 PRs
[06:46] <zyga> Sounds good
[06:47] <zyga> Will the match list conflict?
[06:51] <mborzecki> i'm afraid so, but if we land at least one of these PRs a day, we'll be done in 3 weeks :P
[07:03] <pstolowski> morning
[07:05] <mborzecki> pstolowski: hey
[07:06] <zyga> hey pawel
[07:06] <zyga> mborzecki: maybe a .d directory for match could do it
[07:06] <zyga> mborzecki: but maybe it is better to just review
[07:06] <zyga> I'm a bit tired this morning, not sure why
[07:07] <mborzecki> zyga: we have to review it either way ;)
[07:12] <mborzecki> need to drop my dogs off at the groomer, bb in ~30 minutes
[07:20] <zyga> I'm sleepy
[07:30] <zyga> ok, coffee is ready
[07:30] <zyga> let's fight
[07:30] <zyga> mborzecki, pstolowski: can you guys please have a 2nd look on #4545
[07:30] <mup> PR #4545: interfaces/x11: allow X11 slot implementations <Created by gerboland> <https://github.com/snapcore/snapd/pull/4545>
[07:31] <zyga> it's green and I'd like to merge it but a fresh set of eyes would help
[07:33] <mborzecki> re
[07:37] <mborzecki> pstolowski: can you take a look at https://github.com/snapcore/snapd/pull/5103 ?
[07:37] <mup> PR #5103: tests: shellcheck spread tasks <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/5103>
[07:37] <mborzecki> once we land it i'll open a PR with the first barch of fixes
[07:41] <pstolowski> zyga: will do
[07:41] <zyga> thank you
[07:41] <pstolowski> pedronis: hey, do you have a moment for quick HO? i'd like to run an idea re hooks by you
[07:46] <pedronis> pstolowski: finishing breakfast,  can in a little bit
[07:47] <pstolowski> sure
[08:07] <pstolowski> zyga: is the bug re incorrect glob responsible for the issue you discussed yesterday and were trying to reproduce?
[08:07] <zyga> pstolowski: yes
[08:07] <zyga> pstolowski: but I think there's another bug too
[08:07] <pstolowski> zyga: yay!
[08:07] <zyga> but I'm still zombified and think slowly
[08:08] <zyga> I wrote a test that this bug fix from yesterday doesn't affect the case saw by our customer
[08:08] <zyga> I have a device to test now
[08:10] <pedronis> pstolowski: I'm available now
[08:11] <pstolowski> pedronis: ok, standup HO
[08:28] <zyga> actually
[08:28] <zyga> I wonder now
[08:28] <zyga> what is coming up
[08:30] <mborzecki> zyga: hm?
[08:30] <zyga> c-release
[08:32] <zyga> cagey camel
[08:32] <zyga> cheery chiru
[08:34] <Chipaca> crispy civet
[08:34] <Chipaca> croissant cat
[08:34] <Chipaca> cuddly cockroach
[08:35] <Chipaca> morning all
[08:35] <mup> PR snapd#5089 closed: tests: adding google-sru backend replacing linode-sur <Simple> <Created by sergiocazzolato> <Merged by bboozzoo> <https://github.com/snapcore/snapd/pull/5089>
[08:35] <zyga> good morning :)
[08:35] <mborzecki> cuddly cockroach
[08:35] <zyga> haha
[08:35] <mborzecki> instant +1 from me
[08:36] <kalikiana> that's a bad idea. email filters will turn that into cuddly roach
[08:36] <mborzecki> crispy courgette
[08:36] <Chipaca> http://goodyfeed.com/wp-content/uploads/2017/06/cockroach3.jpg
[08:36] <kalikiana> ha. that's the way to deal with filters :-D
[08:36] <mborzecki> and make an official cooking recipe with it
[08:38] <Chipaca> we should make a nsfwbuntu that was just ubuntu with a nsfw dev name
[08:38] <mborzecki> heh, reminds me of fedora with their 'beefy miracle' thing
[08:39] <Chipaca> ssh! don't say the b word
[08:40] <Chipaca> it sets neal off
[08:40] <Chipaca> mborzecki: what's your thoughts on the drop privs thing?
[08:41] <mborzecki> go for the mutex
[08:41] <Chipaca> mborzecki: option a) say "can't fix before we move to 1.10+"
[08:41] <Chipaca> mborzecki: option b) put a mutex, with a comment that says "this can still end up with the wrong uid trying to do stuff but we couldn't reproduce  it so it's probably very unlikely"
[08:42]  * Chipaca felt less and less confident about (b) as he wrote the comment for it
[08:42] <Chipaca> in fact, let me try something
[08:58] <Chipaca> mborzecki: https://pastebin.ubuntu.com/p/Mn8nF8VV62/
[09:01] <Chipaca> zyga: cosmopolitan cockapoo
[09:02] <mborzecki> Chipaca: seems to work now
[09:04] <Chipaca> mborzecki: in which sense of the word 'work'?
[09:05] <Chipaca> mborzecki: lots of dots, no asterisks?
[09:05] <mborzecki> Chipaca: yes
[09:05] <Chipaca> mborzecki: go < 1.10?
[09:05]  * Chipaca hugs mwhudson for snapping go
[09:07] <mborzecki> pfff, got a couple of terminal-width lines of dots, switched back now and asterisks :/ it did take longer though
[09:07] <Chipaca> mborzecki: seems to vary, but yes
[09:07] <Chipaca> mborzecki: not surprised that a race is racy though :-)
[09:07] <mborzecki> not failed after 3 dots
[09:08] <mborzecki> s/not/now/
[09:08] <Chipaca> i'm very intentionally tripping the bug, there
[09:08] <Chipaca> so in practice it'll be a lot less likely
[09:08] <mborzecki> even if we get someone to packport the patches to 1.9, the previous releases will stay unpatched
[09:09] <Chipaca> but with so many people using it, less likely is still quite likely
[09:09] <Chipaca> mborzecki: mwhudson has backported some cherries for us :-) but that won't help potato linux
[09:10] <Chipaca> an alternative that should work is to use cgo, clone a thread there, do the syscall there (we _don't_ want the libc wrapper)
[09:11] <Chipaca> but, ew
[09:13] <Chipaca> also the semantics of calling from cgo back to go are not nice
[09:22] <Chipaca> niemeyer: would appreciate your thoughts on https://github.com/snapcore/snapd/pull/4983#issuecomment-384915539
[09:22] <mup> PR #4983: osutil/sys, client: add sys.RunAsUidGid, use it for auth.json <Created by chipaca> <https://github.com/snapcore/snapd/pull/4983>
[09:23] <Chipaca> niemeyer: (not on the PR, on the comment)
[09:26] <mborzecki> Chipaca: do you recall if it's also the case in older releasese that if a goroutine exits in a locked thread, the thread will be terminated?
[09:35] <mborzecki> zyga: conflicts in #5081
[09:35] <mup> PR #5081: interfaces/apparmor: add chopTree <Created by zyga> <https://github.com/snapcore/snapd/pull/5081>
[09:35] <zyga> mmm
[09:36] <mborzecki> Chipaca: ca you take another look at #5103 ?
[09:36] <mup> PR #5103: tests: shellcheck spread tasks <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/5103>
[09:36] <zyga> fixed
[09:39] <Chipaca> mborzecki: sorry was afk getting coffee (and maybe cake)
[09:39] <Chipaca> mborzecki: it's very much _not_ terminated
[09:39] <mborzecki> Chipaca: nice combo
[09:39] <Chipaca> especially this cake :-)
[09:39] <Chipaca> my mum's chocolate fudge cake
[09:40] <uebera||> Hi. What is the equivalent of "apt-file list <package-name>" for displaying the contents of a snap package (e.g., "nextcloud") without having to install it first? (I'm using Ubuntu Xenial.)
[09:40] <Chipaca> uebera||: there isn't one
[09:40] <Chipaca> uebera||: what are you trying to do?
[09:41] <Chipaca> uebera||: (outside of some corner cases, AFAIK the files in a snap are not that interesting)
[09:42] <uebera||> I want to see what's inside the package "nextcloud" w/o installing it.
[09:42] <Chipaca> uebera||: I meant one step back from that
[09:43] <Chipaca> uebera||: I mean: what do you need this information for?
[09:43] <uebera||> I don't understand. This is the only step. I do not intend to install the package using snap, I just want to see its contents in order to see what is included as a dependency (b/c I doubt that this is in line with what the documentation lists).
[09:45] <Chipaca> uebera||: there isn't a way to do that without downloading the snap
[09:45] <mborzecki> uebera||: snap download && unsquasfs -l probably makes most sense then
[09:45] <Chipaca> or snap install and find :)
[09:46] <Chipaca> it's a 200MB download, so I can understand not wanting to do it
[09:47] <Chipaca> uebera||: is it really just this snap,  or is it a general problem?
[09:48] <uebera||> Well, this is the first snap I want to inspect (the third one I've ever come across), but I'd consider it a general problem, yes. Unless there are public catalogues like for apt...
[09:50] <Chipaca> uebera||: ok, as I said, there isn't a way (today) of doing this; we don't index that data
[09:50] <pedronis> snaps are also much more fast moving than debs, so the information can be stale quickly
[09:50] <Chipaca> heck yeah :-)
[09:50] <uebera||> Well, if the above "snap download && unsquasfs -l" works, that's good enough for a start atm.
[09:51] <Chipaca> why on earth is mysqld 200MB+
[09:51] <Chipaca> uebera||: also -lls fwiw
[09:51] <uebera||> However, having to start up a throw-away container just to inspect the contents of a snap in a safe environment is not convenient.
[09:51] <Chipaca> uebera||: why would you need a container for download+unsquash?
[09:52] <uebera||> Not for download, but for installation.
[09:52] <mup> PR snapcraft#2107 closed: Release changelog for 2.42 <Created by sergiusens> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/2107>
[09:52] <uebera||> I.e., the "download/unsquash" method should be cited in a FAQ if it isn't already (I've missed it until now).
[09:53] <Chipaca> uebera||: right, for installation i could understand it, as it's mounting the squash and you'd have to trust us that it's ok to do so
[09:53] <Chipaca> uebera||: well, download is documented, as snaps are documented as being squashfs
[09:53] <Chipaca> uebera||: the rest is just joining the dots
[09:54] <Chipaca> s/as/and/
[09:55] <didrocks> zyga: so, due to my classic snap segfaulting as I can only build it on bionic due to golang version, I heard there was a way to declare which core snap you are using, I don't find the reference for it in the doc, any hint?
[09:56] <zyga> didrocks: "base: core18"
[09:56] <zyga> but I don't know if snapcraft supports this yet
[09:56] <zyga> it's very early
[09:56] <zyga> didrocks: if you build your snap on bionic you must use a container
[09:56] <mborzecki> uebera||: hm alternative would be to try https://uappexplorer.com/snap/ubuntu/nextcloud and download from there
[09:56] <zyga> didrocks: and for classic snap you must use a xenial-abi-compatible build of go
[09:56] <didrocks> zyga: the issue is that snapcraft doesn't allow you to select the go version if I'm correct
[09:57] <zyga> didrocks: perhaps, kalikiana would know
[09:57] <didrocks> sounds like bug #1616985 didn't really move
[09:57] <mup> Bug #1616985: the go plugin doesn't use go build-snaps <isv> <plugin> <Snapcraft:In Progress by kalikiana> <https://launchpad.net/bugs/1616985>
[09:57] <didrocks> so, basically, I'm between FTBFS and segfault :p
[09:59] <Chipaca> didrocks: even with core18?
[09:59] <didrocks> Chipaca: well, I'm unsure how to inject that in my snapcraft.yaml definition
[10:00] <Chipaca> try "base: core18"
[10:00] <didrocks> same level than grade: I guess?
[10:00] <Chipaca> right next to "name: doesrocks"
[10:00] <Chipaca> :-p
[10:00] <zyga> yes
[10:00] <didrocks> ;)
[10:01]  * didrocks tries, thanks!
[10:01] <Chipaca> didrocks: if it shouts at you let us know
[10:01] <uebera||> mborzecki: Thanks, this is an alternative w/o using snap (though "snap download" worked like a charm)
[10:02] <kalikiana> didrocks: That's not perfectly accurate. The go plugin insists on installing the deb, but it doesn't prevent you from using a remote part with your favorite version
[10:02] <didrocks> Chipaca: at least, it downloads core18 when installing, good starting point :)
[10:02] <Chipaca> uebera||: 'snap download' also gives you the assertions which can be helpful
[10:02] <Chipaca> didrocks: huzzah
[10:03] <didrocks> and it works! thanks Chipaca, zyga
[10:03] <mborzecki> uebera||: a word or warning, iirc the search by architecture is broken in uappexplorer
[10:03] <didrocks> kalikiana: yeah, that's what I read, but I prefer to avoid relying on something else like another snap or a ppa with a newer build
[10:04] <kalikiana> didrocks: I can't parse that sentence. You want a specific go version that isn't in the archive, no?
[10:05] <didrocks> kalikiana: exactly! It seems the only way right now is to have a part downloading go for you (from another snap or using adding a ppa as a build-dep)
[10:07] <kalikiana> didrocks: My suggestion above was a remote part. Not a PPA. As in, "after: [go]   go:     source-tag: go1.7.5"
[10:08] <didrocks> kalikiana: ah ok, nice that there is a remote part! I'll see between that or using core18…
[10:08] <didrocks> thanks :)
[10:08] <Chipaca> didrocks: i'd consider core18 'developer preview' quality, fwiw
[10:08] <Chipaca> didrocks: tested for some very narrow verticals right now afaik
[10:09] <Chipaca> so if you can get it to work without, you'll be happier
[10:09] <didrocks> Chipaca: well, as it's for a classic snap, I think I only need an ABI compatible linker with my build
[10:09] <didrocks> and as it's Go without Cgo, it's statically compiled (apart from accessing the network config)
[10:09] <Chipaca> didrocks: in particular I'm entirely unsure how well core18 will play with classic, today
[10:10] <Chipaca> didrocks: and 'base: core18' means installing your snap will download core18
[10:10] <Chipaca> didrocks: so... let us know how it goes :-D
[10:10] <didrocks> Chipaca: which we are going to do anyway soon AFAIK for 3.28 GNOME snaps
[10:10] <didrocks> yeah, I'm happy to be the guinee pig here ;)
[10:10] <didrocks> guinea*
[10:10] <Chipaca> didrocks: I'm happy for you to be, as well
[10:10] <Chipaca> as long as you know you are
[10:11] <didrocks> heh, I know at least there is a fallback plan
[10:11] <didrocks> I'll keep you posted in case I spot anything wrong
[10:16] <Chipaca> didrocks: to be clear if it's going to fail anywhere it'll be on 16.04 (or 14.04)
[10:16] <didrocks> yeah, I'll try testing there
[10:16] <Chipaca> didrocks: if it's a desktop thing, don't try 14.04 :-)
[10:18]  * didrocks will happily refrain thus!
[10:44] <mborzecki> had to restart, master travis job again, and again it failed because not output was received
[10:51] <Son_Goku> does anyone know why I default to the edge channel in Ubuntu Software?
[10:51] <Son_Goku> snaps I install are apparently coming from edge by default
[10:51] <Son_Goku> which is something I totally did not expect
[10:52] <Son_Goku> or at least, it looks like they are
[10:52] <Son_Goku> it says "stable" channel but gives me the version in the edge channel...
[10:55] <mup> PR snapcraft#2109 opened: sources: clean up IncompatibleOptionsError <Created by kalikiana> <https://github.com/snapcore/snapcraft/pull/2109>
[10:57] <zyga> Son_Goku: that looks like a bug for sure
[10:57] <zyga> it should be the opposite
[10:57] <Son_Goku> zyga: it's very obvious with the blender snap
[10:57] <ogra_> Son_Goku, it knows who you are !
[10:57] <Son_Goku> meep
[10:57] <ogra_> :)
[10:57] <zyga> and after installing snap info says you are tracking edge?
[11:09] <Son_Goku> zyga, nope
[11:09] <Son_Goku> it switches back to stable
[11:09] <Son_Goku> it's very strange and confusing
[11:09] <zyga> yes
[11:09] <zyga> but is the snap you got from edge?
[11:09] <zyga> as in the revision matches what is in edge and not what it is in stable
[11:10] <Son_Goku> no it matches stable
[11:10] <zyga> hmm?
[11:10] <Son_Goku> it looks like something's confused in ubuntu software
[11:10] <zyga> so what is the bug
[11:10] <Son_Goku> even when tracking stable, it's actually rendering info from edge
[11:10] <Son_Goku> in ubuntu software
[11:11] <Son_Goku> also, niemeyer, you really should fix the metadata for your blender snap
[11:11] <Son_Goku> it's completely wrong :)
[11:12] <Chipaca> Son_Goku: showing info from edge?
[11:12] <Chipaca> Son_Goku: tell me moar
[11:12] <Son_Goku> Chipaca, ubuntu software renders info from edge, even though it defaults to tracking stable
[11:12] <zyga> https://www.youtube.com/watch?v=DOYRLdisrTE
[11:12] <Chipaca> Son_Goku: in what case?
[11:12] <zyga> (wrong video though ;)
[11:12] <Son_Goku> snap not installed
[11:12] <Chipaca> Son_Goku: i mean: what do you see as 'from edge'?
[11:12] <Son_Goku> Ubuntu Software on 18.04, just open it up, and search for blender
[11:13] <Son_Goku> well, the version
[11:13] <sparkiegeek> yeah, reproducible here - gnome software it showing the version from non-stable (hard to see if it's edge, beta or candidate)
[11:13] <sparkiegeek> but it's not in stable
[11:14] <Son_Goku> also, Fedora 28 is GOLD
[11:14] <Chipaca> sparkiegeek: Son_Goku: any example that isn't also a deb?
[11:15] <Chipaca> Son_Goku: congrats!
[11:15] <Son_Goku> Chipaca, not sure, I noticed it because the blender snap is missing so much data it shows up as separate from the deb
[11:15] <Son_Goku> the vlc snap also shows the same issue
[11:15] <sparkiegeek> Chipaca: just trying to check the snapd API, do you have the curl invocation to hand?
[11:16] <Son_Goku> though it renders the beta channel's number
[11:16] <sparkiegeek> Chipaca: reproducible for gitlptools (my snap)
[11:16] <Chipaca> sparkiegeek: snap install http && http snapd:///v2/find name==thesnapname
[11:17] <Chipaca> not seeing this in 16.04, fwiw
[11:17] <Chipaca> gitlptools showing 388db20 as is proper
[11:17] <sparkiegeek> Chipaca: http: error: InvalidSchema: No connection adapters were found for 'snapd://v2/find'
[11:17] <Chipaca> eh
[11:17] <Chipaca> sparkiegeek: forgot the extra /
[11:17] <Chipaca> snapd:///v2/...
[11:18] <Chipaca> wait, no i didn't
[11:18] <Chipaca> you did :-D
[11:18] <sparkiegeek> $ http snapd:///v2/find name==gitlptools
[11:18] <sparkiegeek>  
[11:18] <sparkiegeek> http: error: InvalidSchema: No connection adapters were found for 'snapd:///v2/find'
[11:19] <Chipaca> sparkiegeek: which http
[11:19] <sparkiegeek> ah ha!
[11:19] <Chipaca> ¯\_(ツ)_/¯
[11:19] <sparkiegeek> Chipaca: thanks, that was it
[11:20] <zyga> so what is broken?
[11:21] <Chipaca> completely in the dark guess? it's taking the first entry from the channel map instead of latest/stable
[11:21] <Chipaca> (or just the one outside the channel map)
[11:21] <sparkiegeek> Chipaca, zyga: it looks like version number outside of the channel map is reporting the wrong one
[11:21] <sparkiegeek> right, the latter
[11:21] <Chipaca> ah, ok
[11:21] <Chipaca> then, yes, that.
[11:21] <Chipaca> bug in store, already getting fixed
[11:22] <zyga> thanks!
[11:22] <Son_Goku> :)
[11:22] <Chipaca> the snap client just happens to walk around the issue
[11:23] <sparkiegeek> the snap that's installed is tracking stable though, and then gnome-software is reporting the installed version (as tested with blender)
[11:23] <sparkiegeek> so I can't exactly reproduce what Son_Goku sees
[11:24] <Chipaca> sparkiegeek: i thought it wasn't the installed version
[11:24] <Son_Goku> oh, after install, it shows the right thing
[11:24] <Chipaca> right
[11:24] <Son_Goku> it was _before_ install it didn't
[11:24] <Son_Goku> so it was just confusing
[11:24] <sparkiegeek> Son_Goku: agreed
[11:24] <Chipaca> easiest and quickest and most future-safe thing would be to tell snapd-glib (or was it glib-snapd) to grab the info from the channel map
[11:24] <Chipaca> although...
[11:24] <Chipaca> i should sit down with 'em and make it better
[11:25] <Chipaca> they're having to jump through hoops to get their stuff done, and it's nasty
[11:25] <Chipaca> i'll do that next week
[11:25] <Son_Goku> Chipaca, snapd-glib
[11:25] <sparkiegeek> Chipaca: right, but then they'd have to encode the channel map priorities - e.g. no guarantee of there being latest/stable
[11:25] <Chipaca> Facu: remember the 'ha ha lol the store picks a revision at random to show the info if you say channel=""' thing we talked about? gnome-software in 18.04 hits it :-)
[11:26] <Son_Goku> that explains why I get the beta version rendered for vlc
[11:26] <Son_Goku> rather than edge, if it's "random"
[11:26] <Chipaca> Son_Goku: well, it's not random, it's most-recently-pushed afaik
[11:26] <Chipaca> it just seems random
[11:27] <Son_Goku> now, if the store was open source... :P
[11:27] <thresh> ugh, snap store shows vlc edge to install by default?
[11:28] <Chipaca> thresh: no(ish)
[11:28] <Chipaca> thresh: it installs stable; the version number is wrong
[11:28] <Chipaca> thresh: in 18.04
[11:28] <thresh> that's still bad
[11:28] <thresh> don't want the user to have an impression they'll install vlc 4.0
[11:29] <Chipaca> right
[11:29] <Chipaca> we'll sort it
[11:29] <Son_Goku> someday :P
[11:30] <Chipaca> thresh: to be clear, it's gnome-software (and presumably other stores using snapd-glib but i haven't checked) in 18.04 that expose the bug
[11:31] <thresh> good to know, fingers crossed on a swift fix :)
[11:31] <Chipaca> we can argue about whether it's the store, snapd, or snapd-glib that have the bug -- but it's fixable in any of those places :-)
[11:31] <thresh> also congrats on a release for everyone involved!
[11:32] <Chipaca> thresh: i think everyone directly involved is off today :-)
[11:34] <Facu> Chipaca, ugh
[11:34] <Chipaca> Facu: :-)
[11:34] <Chipaca> Facu: also, good morning
[11:35] <Chipaca> Facu: sorry to greet you with that news
[11:38] <Son_Goku> holy shit
[11:38] <Son_Goku> VLC looks ugly
[11:40] <Chipaca> Son_Goku: sshh! you'll make thresh cry :-(
[11:40] <thresh> nah, I already cry daily
[11:40] <Chipaca> Son_Goku: try saying the same thing but with kinder words
[11:40]  * Chipaca hugs thresh 
[11:40] <thresh> Son_Goku, what DE are you under?
[11:40] <Son_Goku> GNOME
[11:40] <Son_Goku> Fedora 28
[11:40] <thresh> Son_Goku, can you please try beta?
[11:40] <Son_Goku> sure, one sec
[11:42]  * Son_Goku sighs
[11:42] <Son_Goku> I can't do it from g-s
[11:42] <Son_Goku> down the CLI we go!
[11:42] <Son_Goku> oh we've gotten fancy now
[11:42] <Son_Goku> overlaying progress bars
[11:42] <Chipaca> Son_Goku: how long has it been since you used the CLI :-)
[11:43] <Son_Goku> more months than it probably should be
[11:43] <thresh> I'd release the beta to stable any time, but the UI (e.g. open file dialog) is completely broken on KDE
[11:44] <Son_Goku> thresh, not better
[11:44] <thresh> hmm, okay
[11:45] <thresh> can you drop the output of env | grep XDG_ to pastebin somewhere?  or PM me?
[11:45] <thresh> and a screenshot of VLC / menus
[11:45] <Chipaca> thresh: also maybe 'snap run --shell vlc' and do the same in there
[11:47] <Son_Goku> thresh: http://kinginuyasha.enanocms.org/downloads/screenshot-vlc-snap-about-20180427.png
[11:48] <thresh> Son_Goku, what about the icons / general UI fonts?
[11:49] <thresh> in the menu bar etc
[11:51] <Son_Goku> thresh: http://kinginuyasha.enanocms.org/downloads/screenshot-vlc-snap-with-env-20180427.png
[11:51] <Son_Goku> it's pretty bad
[11:51] <thresh> ok, so it's an issue with the fonts
[11:51] <Chipaca> Son_Goku: thresh: aren't themes and fonts working?
[11:52] <thresh> the UI actually does not look like windows 95
[11:52] <Chipaca> i thought they were
[11:52] <thresh> Chipaca, I'm pretty sure I need to add something to my snap to enable it?
[11:52] <Son_Goku> I thought they were too
[11:52] <Son_Goku> but apparently no?
[11:52] <sparkiegeek> Son_Goku: can you file a bug about the version info being wrong? i.e. ubuntu-bug gnome-software-plugin-snap
[11:53] <Son_Goku> sparkiegeek, I don't have the Ubuntu 18.04 VM anymore :)
[11:53] <Son_Goku> I only spun it up to check out g-s there ;)
[11:53] <Son_Goku> because I heard there was funny business in g-s in Ubuntu
[11:53] <Son_Goku> like the ability to select channels and permissions
[11:53] <Son_Goku> neither of which I can do from g-s in Fedora
[11:53] <Chipaca> zyga: do you know about fonts?
[11:56] <zyga> Chipaca: is this a tricky question where I say yes and then go and dig through unholy C code that renders vectors?
[11:56] <zyga> popey: https://twitter.com/zygoon/status/989835561639841797 :-)
[11:57] <zyga> popey: it still needs some upstreaming on my part
[11:57] <zyga> and packaging love
[11:57] <zyga> and more
[11:57] <Chipaca> zyga: I don't want you dissing metafont now
[11:57] <zyga> but :-)
[11:57] <popey> that skype snap isnt confined
[11:57] <zyga> popey: yes but it's not because of opensuse
[11:57] <popey> neither is slack
[11:57] <popey> ok
[11:57] <zyga> popey: look at the terminal
[11:57] <zyga> but yeah, I see your point
[11:57] <Chipaca> zyga: the work to let snaps access fonts from the local system, was that landed?
[11:57] <popey> Sweet!
[11:58] <zyga> Chipaca: yes
[11:58] <zyga> but it's working partially by accident
[11:58] <Chipaca> Son_Goku: FWIW i seem to recall that after agreeing that font metadata nearly never changes, it changed
[11:58] <Chipaca> just too late to break 18.04, but in time to break fedora 28, if i remember well?
[11:59] <Son_Goku> probably
[11:59] <Son_Goku> it will probably break openSUSE soon too
[11:59] <zyga> Chipaca: yes
[11:59] <zyga> 2.13
[11:59] <Son_Goku> yep
[11:59] <Son_Goku> fontconfig-2.13.0-3.fc28
[11:59] <zyga> (13 , see it's _bound_ to happen ;)
[12:00] <thresh> what do I need to do to enable it?
[12:00] <Chipaca> zyga: how were we going to fix that?
[12:01] <Chipaca> zyga: and what thresh is asking is whether the snap needs to do anything to access the 'outside' fonts, and if so what :-)
[12:01] <zyga> Chipaca: let me think, AFAIK the issue is the old fontconfing doesn't grok the new syntax so we must not share /etc/fonts (or something like that) and probably must synthesise per-core variant somewhere in /var/lib/snapd so that we can bind mount it into /etc
[12:02]  * Son_Goku whispers fedora core snap to zyga 
[12:02] <zyga> Son_Goku: I played with that, need more weekends to finish it
[12:02] <zyga> Chipaca: which sucks because /etc/fonts may not be there
[12:02] <zyga> may be a symlink
[12:02] <zyga> or whatever
[12:02] <zyga> I suspect we want a writable mimic with filtering
[12:02] <zyga> over /etc
[12:02] <zyga> so that we can assure /etc/fonts is something we control
[12:05] <zyga>  Chipaca does that answer your question?
[12:07] <Chipaca> zyga: it answers my first question -- thresh's question about how to use it still stands
[12:07] <Chipaca> zyga: (with the caveat that it won't work on anything using fontconfig 2.13 because they hate us)
[12:07] <thresh> I'll be more than happy to drop fonts from bloated vlc snap indeed :-)
[12:09] <zyga> Chipaca: we need to write a tool (Alex expressed interest in cooperation there) that would take a fontconfig "config" file and convert it to a version appropriate for a given fontconfig major/minor release
[12:10] <Chipaca> zyga: how. does. a. snap. author. use. this.
[12:10] <zyga> no no
[12:10] <Son_Goku> because fonts are evil
[12:10] <zyga> it's for us
[12:10] <zyga> snap authors don't need to do anything
[12:10] <Chipaca> zyga: duuuude
[12:10] <Son_Goku> Chipaca, it's intended to be automatic
[12:10] <Chipaca> zyga: I create a snap
[12:10] <pedronis> magic and pixie dust
[12:10] <Chipaca> zyga: how do i tell it to use the system's fonts?
[12:11] <Son_Goku> automagic :D
[12:11] <Son_Goku> it Just Works(TM)
[12:11] <zyga> if you use fonconfig and you use one of the desktop interfaces it is indeed automatic
[12:11] <zyga> but that's what we _want_
[12:11] <zyga> what is reality is more complex
[12:11] <Chipaca> so vlc could drop its bundled fonts and it'd still work (except on fedora 28)?
[12:11] <zyga> you can see the fonts and they are mounted where people expect them
[12:12] <zyga> but from one system to another you many not be configured to use them
[12:12] <zyga> because "configuration" is really not configuration, it is mandatory side-channel to tell fontconfig how to use a particular specific font correctly
[12:12] <zyga> it's not "fonts are here", it's down to font substitution and other magic that makes the font actually useful
[12:13] <pedronis> so we need to know what config version the snaps expect?
[12:13] <mborzecki> w8, why are vlc fonts ok here then?
[12:13] <Chipaca> mborzecki: because vlc bundles its fonts maybe?
[12:14] <Chipaca> mborzecki: here == arch? you're on 2.13+?
[12:14] <zyga> pedronis: it seems we need to know the expected format of fontconfig configuration per base, yes
[12:14] <mborzecki> Chipaca: 2.13 to be exact
[12:14] <Chipaca> mborzecki: or maybe you just like the cool 'everything is courier new' look
[12:14] <zyga> mborzecki: not sure, perhaps vlc reconfigures the place for fonts because of one of the desktop helpers
[12:14] <pedronis> zyga: well, unless the  snap bundles the relevant libraries? in which case is per snap
[12:14] <zyga> (aka magic boxes we don't grok in this team)
[12:14] <mborzecki> I see a bunch of warnings https://paste.ubuntu.com/p/RwdnCCTqZ6/ but that's that
[12:14] <thresh> I'm on fontconfig 2.13, on debian testing, and the fonts are fine for me
[12:14] <zyga> pedronis: yes but even then we can offer the older format fine
[12:15] <zyga> pedronis: I think it could be something we convey on the desktop interface
[12:15] <zyga> but it must be consistent inside the whole snap
[12:15] <Chipaca> thresh: mborzecki: so, the problem with 2.13 is that it supports config stuff that <2.13 does not
[12:15] <Chipaca> thresh: mborzecki: as long as you're not using that new stuff it still works
[12:16]  * zyga gets back to debugging
[12:16] <Chipaca> the problem AFAIR was that a brand new install does use that new config stuff
[12:16] <mborzecki> https://i.imgur.com/tEIrLip.jpg
[12:16] <zyga> I really don't know more about fonts than you guys :)
[12:16] <Chipaca> I don't know the details, i'm just repeating what i heard in the standup :-)
[12:16] <mborzecki> hmm
[12:17] <mborzecki> maybe there's some decent fallback mechanism
[12:17] <mborzecki> like picking the fallback fonts by name/type
[12:18] <mborzecki> Son_Goku just happens to not have the right fonts installed or sth
[12:18] <Son_Goku> this is literally the default font set for Fedora Workstation 28
[12:18] <thresh> woah, I don't get any of those warnings
[12:18] <mborzecki> the monospace one? :)
[12:18] <Son_Goku> well, not that
[12:18] <Son_Goku> that's the shit-tier font
[12:18] <Son_Goku> it renders incredibly badly too
[12:19] <Son_Goku> some parts aren't even readable
[12:19] <thresh> and that's how it looks for me: https://dashboard.snapcraft.io/site_media/appmedia/2018/04/2018-04-26-161203_1126x899_scrot_yBFL55L.png
[12:19] <mborzecki> yeah, i can see that the font i have is also slightly different (smaller?), it surely ignores the dpi/scaling settings
[12:19] <thresh> I do use scaling though for QT apps
[12:20] <thresh> QT_SCREEN_SCALE_FACTORS=1.5 and QT_AUTO_SCREEN_SCALE_FACTOR=0
[12:37]  * zyga sighs
[13:13] <ogra_> hrm
[13:16] <ogra_> tyhicks, jdstrand, is the store tool that notifies me about outdated packages with USN your baby ? it semds emailös without any timestamps at all it seems (making the mails end up in a special folder in my setup)
[13:18] <nsg> I'm going crazy, all text in my snap is just squares. I'm not sure how to debug this. Works on Ubuntu, just squares on Debian and Solus. Any idea how to debug this?
[13:18] <vidal72[m]> what is the status of this https://forum.snapcraft.io/t/firefox-please-create-the-track-esr/5006
[13:18] <ogra_> nsg, include fonts (or even better use a desktop-helper)
[13:20] <nsg> ogra_: I'm using the helper, and I think I have all the fonts I need. The application has also the option to change fonts ... any selected font is just squares.
[13:23] <nsg> The interesting thing is, on Solus I only see 3 fonts: https://imgur.com/a/lEbReMW
[13:24] <nsg> fc-list outside the snap lists more or less the same fonts like in my Ubuntu install. I'm using desktop-gtk2, unity7 and so on.
[13:24] <nsg> A friend has reported the same problem on Debian, so I do not think it's Solus related, more non-Ubuntu related :)
[13:25] <nsg> I'm just not sure what to do next ... have tried to get this thing fixed on my spare time for 2-3 weeks now.
[13:25] <diddledan> nsg: have you plugged `desktop` and `desktop-legacy`?
[13:27] <nsg> hm... not at the snap in the solus install... I will give that a try. These are new to me :)
[13:27] <nsg> I will try to _add_ them and see what happens
[13:27] <diddledan> they're automatically connected once the snap defines the interfaces
[13:29] <zyga> jdstrand: hey, can you tell me why we chose to use /var/lib/snapd/apparmor/profiles over /etc/apparmor.d
[13:29] <zyga> jdstrand: I'm working on opensuse down streaming and one issue I'm facing is that our profiles don't reload on boot
[13:30] <zyga> jdstrand: I patched snapd locally to see what happens when we put our profiles in /etc/apparmor.d and ... it works okay
[13:31] <jdstrand> zyga: because /etc/apparmor.d is for system profiles and profiles the admin uses. /var/lib/snaps/apparmor/profiles are managed by snapd
[13:32] <zyga> yes I know but that is not upstream
[13:32] <zyga> I'm trying to figure out what to do
[13:32] <zyga> teach upstream apparmor about /var/lib/snapd/apparmor/profiles
[13:32] <zyga> patch opensuse specifically
[13:32] <zyga> or patch snapd on opensuse
[13:32] <jdstrand> zyga: please join #apparmor. opensuse is redoing all their profile loading
[13:33] <jdstrand> zyga: on OFTC
[13:33] <zyga> one sec, I don't have that set up
[13:35] <jdstrand> zyga: jjohansen from upstream apparmor has also been working on making all this easier to configure. today everyone uses their own apparmor init script. we want to have an 'upstream way' of doing it where you configure additional locations in various ways. this is good for precompiled caches, system profiles, admin profiles, service-managed profiles (snapd, lxd, libvirt, etc), ...
[13:39] <Son_Goku> Solus completely rewrote how their aa profiles are managed too
[13:39] <Son_Goku> they didn't like how Ubuntu or openSUSE did it
[13:41] <Chipaca> Son_Goku: https://forum.snapcraft.io/t/5163
[13:42] <vidal72[m]> zyga: # /usr/lib/systemd/system/apparmor.service.d/snapd.conf
[13:42] <vidal72[m]> zyga: [Service]
[13:42] <vidal72[m]> zyga: ExecStart=-/usr/bin/apparmor_parser --write-cache --cache-loc=/var/cache/apparmor -r /var/lib/snapd/apparmor/profiles/
[13:42] <zyga> vidal72[m]: pastebin please
[13:42] <zyga> vidal72[m]: does this unit run when you update apparmor-profiles?
[13:42] <zyga> I know it's possible to load them ourselves but there are benefits when they are loaded by the main unit
[13:43] <vidal72[m]> https://paste.ubuntu.com/p/z5bfPrbrbZ/
[13:44] <vidal72[m]> zyga: they will be loaded by main apparmo unit
[13:44] <Son_Goku> Chipaca: yay
[13:44] <zyga> vidal72[m]: hmm, this probably won't be enough
[13:45] <vidal72[m]> why?
[13:45] <zyga> vidal72[m]: because services will start before snapd
[13:46] <zyga> and they will fail because their confinement profile won't be present yet
[13:46] <zyga> ah
[13:46] <zyga> sorry, I misread what you wrote
[13:46] <zyga> chaining this to apparmor will work
[13:47] <vidal72[m]> it worked for me
[13:48] <zyga> thank you for the suggestion, another option on the table
[13:50] <Chipaca> raining cats and dogs and i need to do the school run
[13:50] <pedronis> Chipaca: btw according to the dictionary both those pronunciations of ephemeral exists
[13:51] <Chipaca> pedronis: i'll pronounce it "eph'g'meral" just to be sure
[13:51] <Chipaca> that's a double glottal stop with a g thrown in for good measure
[13:51] <pedronis> Chipaca: iphimeral or iphemeral (or something like that)
[13:51] <vidal72[m]> zyga: this is what upstream suggested to me after I asked about snapd support
[13:51]  * Chipaca -> school run
[13:52] <Chipaca> pedronis: pretty sure people in the uk make decisions about the sort of person you are based on which one you use though
[13:52] <Chipaca> so i should learn which one i want to be :-)
[13:52] <zyga> vidal72[m]: which upstream?
[13:52] <Chipaca> or use both and confuse people
[13:52]  * Chipaca -> really off
[13:52] <jamesh> zyga: hi.  I've addressed some of the review comments on https://github.com/snapcore/snapd/pull/3963 and replied to the others.  CI failed with a timeout though, but I think it is probably fine.
[13:52] <mup> PR #3963: cmd/snap-confine: add support for per-user mounts <Created by jhenstridge> <https://github.com/snapcore/snapd/pull/3963>
[13:52] <ogra_> jdstrand, seen my ping above ? if it isnt your tool, whom do i need to poke so the mails get a proper send-date in them ?
[13:53] <zyga> jamesh: thank you, I think at this stage we are waiting for Niemeyer's review
[13:53] <vidal72[m]> zyga: apparmor
[13:53] <jamesh> zyga: he reviewed it earlier today
[13:54] <zyga> oh, sorry I didn't know
[13:55] <zyga> I was fire-fighting somewhere else
[13:56] <zyga>  jamesh I restarted the build now
[13:56] <jamesh> zyga: thanks
[13:56] <zyga> I will merge it as soon as it goes green
[14:00] <zyga> ok I need a break
[14:00] <zyga> for food/dog/walk
[14:02] <roadmr> that makes it ambiguous whether you'll eat the dog and then go for a walk or other possible combinations :P
[14:02] <zyga> that makes it more interesting
[14:03] <jdstrand> ogra_: that is me (tyhicks, ignore this)
[14:03] <zyga> maybe the dog is typing
[14:03] <jdstrand> ogra_: well, it is the security team, but I've done the initial implementation, so me :)
[14:03] <jdstrand> ogra_: what is one of the affected snaps?
[14:03] <ogra_> jdstrand, packageproxy and jtiledownloader
[14:04] <jdstrand> roadmr: fyi, https://forum.snapcraft.io/t/notifications-for-out-of-date-stage-packages/5161
[14:04] <roadmr> jdstrand: awesome! yes, Celso just got one!
[14:05] <jdstrand> I imagine a lot of people did :)
[14:05] <roadmr> I didn't, which is surprising because my snaps are all crap and haven't been rebuilt in ages
[14:05] <ratliff> if you didn't get one and you are a snap publisher, give yourself a gold star and a pat on the back :)
[14:05] <jdstrand> ogra_: I see a 'Date: Fri, 27 Apr 2018 14:08:20 +0100 (04/27/2018 08:08:20 AM)'
[14:05] <mup> PR #2018: snapstate: pass errors from ListRefresh in updateInfo <Created by mvo5> <Merged by niemeyer> <https://github.com/snapcore/snapd/pull/2018>
[14:05] <roadmr> maybe that's why - they're so old they likely don't have manifest_raw :D
[14:06] <jdstrand> roadmr: or build you snap with SNAPCRAFT_BUILD_INFO=1 ;)
[14:06] <jdstrand> roadmr: *and* publish it to a channel :)
[14:06] <jdstrand> ogra_: what is it that you want?
[14:06] <roadmr> jdstrand: it is published (gnuchess). I guess I could fire a rebuild
[14:07] <roadmr> jdstrand: oh so snapcraft doesn't manifest_raw by default? I thought it did... hmm. Will keep in mind!
[14:07] <diddledan> did the forum just die?
[14:07] <jdstrand> roadmr: I did the same thing this morning. had 3 snaps built without it so I rebuilt them
[14:07] <roadmr> diddledan: wfm \o/
[14:07] <jdstrand> roadmr: LP does manifest raw by default. snapcraft does not
[14:08] <roadmr> jdstrand: gotcha, that must be the source of my confusion
[14:08] <jdstrand> roadmr: but yeah, gnuchess is ancient by snapcraft standards, so it wouldn't have it no matter where you built it :)
[14:08] <roadmr> I'll try to rebuild it
[14:11] <jdstrand> ogra_: I'm confused by your request since there is a 'Date: ...' in the email. please clarify
[14:13] <ogra_> jdstrand, https://imgur.com/a/fgZ6am
[14:13] <ogra_> i want the question marks gone :)
[14:14] <ogra_> i typicall yonly get these from spambot mails
[14:14] <jdstrand> ogra_: 404
[14:14] <ogra_> oops
[14:15] <ogra_> jdstrand, https://imgur.com/a/fgZ6amU
[14:16] <jdstrand> ogra_: that is weird. I'm using the same email client as you and don't see that
[14:18] <sergiusens> jdstrand: nice email I just got from usn :-)
[14:18] <jdstrand> ogra_: I see 'Content-Type: text/plain; charset="us-ascii"' in it but 'Content-Type: text/plain; charset="utf-8"' in other store-generated emails
[14:19] <jdstrand> sergiusens: thanks! and thanks for the manifest.yaml. not sure if you saw, but fyi https://github.com/snapcore/snapcraft/issues/2093
[14:20] <sergiusens> jdstrand: I did, you did not read the template ;-) (we will add it regardless) :-)
[14:20] <jdstrand> sergiusens: whoops!
[14:21] <jdstrand> ogra_: let me play with making that utf-8, then send you a test email
[14:22] <ogra_> jdstrand, thanks
[14:33] <kyrofa> Ooo jdstrand I'm seeing some interesting emails about the contents of my snaps
[14:35] <roadmr> \o/
[14:35] <jdstrand> kyrofa: yep :)
[14:39] <cachio> mborzecki, hey, I noticed the arch linux images are not being discarded from google
[14:40] <cachio> mborzecki, any idea why it could be happening?
[14:42] <cachio> mborzecki, the image has not changed from long time
[14:42] <cachio> and when I log into it, I don't see nothing weird
[14:44] <ogra_> jdstrand, nope, same
[14:44] <jdstrand> hmm
[14:44] <ogra_> (teh 0a mail you just triggered)
[14:44]  * jdstrand nods
[14:44] <ogra_> *0ad
[14:46] <ogra_> jdstrand, if it is really only me then feel free to ignore. i can handle it on my side ... perhaps my mailserver or spamassassin mess it up here
[14:48] <jdstrand> ogra_: I just can't see any differences
[14:48] <jdstrand> ogra_: emails to ubuntu-devel look ok?
[14:48] <ogra_> yeah
[14:48] <jdstrand> let me look at one of those
[14:48] <ogra_> in general all other m,ails look fine
[14:49] <ogra_> i occasionally get spam with broken date but thats about it
[14:56] <jdstrand> ogra_: makes no sense. I looked in ubuntu-devel and other text/plain emails with charset of utf-8 and us-ascii and everything looks ok
[14:57] <ogra_> yeah, lets blame my side unless you hear anything from others then
[14:58]  * roadmr blames it on ogra_
[14:58] <ogra_> :D
[14:58] <jdstrand> ogra_: I wonder if it is evolution and the format of the date column... again, the date format is the same as other emails so don't know why that would happen
[15:01] <ogra_> Content-Type: text/plain; charset="utf-8"
[15:01] <ogra_> should that perhaps be charset=UTF-8 instead ?
[15:03] <ogra_> hmm, no
[15:04] <Chipaca> ogra_: no?
[15:05] <Chipaca> i don't see anything about quoting it in Rfc 2046
[15:05] <ogra_> i was more concerned about the non-capitalized vaersion there ... not about the quoting
[15:05] <ogra_> *version
[15:06] <ogra_> but digging throuhg mail sources i see both variants and both seem to work fine
[15:06] <Chipaca> ah there it is, quotes are fine
[15:09] <mborzecki> cachio: no clue, do you know if these were stated automatically due to a travis job or manually?
[15:09] <mborzecki> cachio: is there even a difference, maybe there's some magic logic in gce that collects unused vms
[15:28] <cachio> mborzecki, started by travis
[15:54] <nsg> diddledan: "desktop" did the trick! Thank you, it works now on my Solus test VM.
[16:04] <pedronis> Chipaca: when do we show the long help of commands?
[16:05] <pedronis> Chipaca: I see that refresh and install ones is out-of-date but is also not shown
[16:06] <Chipaca> pedronis: 'snap help --man | man -l -'
[16:06] <Chipaca> pedronis: might help
[16:06] <pedronis> Chipaca: but I don't see it in "man snap" either
[16:07] <Chipaca> pedronis: also master is not 2.23.5
[16:07] <Chipaca> pedronis: that fixed text is new
[16:08] <Chipaca> pedronis: go build -o /tmp/snp ./cmd/snap && SNAP_REEXEC=0 SNAPD_DEBUG=1 /tmp/snp install --help  :-)
[16:08] <pedronis> I'm on master and not seeing it either
[16:08] <Chipaca> pedronis: reexec
[16:09] <pedronis> Chipaca: so it's recent?
[16:10] <Chipaca> pedronis: not really
[16:10] <pedronis> well, newer than 2.32.5 ?
[16:10] <Chipaca> pedronis: 2018-03-14
[16:10] <Chipaca> over a month old
[16:10] <Chipaca> but, yes, not in 2.32.5
[16:11] <pedronis> ok, it needs reviewing given that it doesn't match reality anymore
[16:11] <pedronis> I was planning to fix the help but then saw that there's much more text, that I wasn't seeing
[16:11] <pedronis> bit confused was I
[16:13] <pedronis> maybe I can try anyway quickly
[16:20] <mup> PR snapd#5111 opened: cmd/snap: update install/refresh help vs --revision <Created by pedronis> <https://github.com/snapcore/snapd/pull/5111>
[16:20] <pedronis> Chipaca: https://github.com/snapcore/snapd/pull/5111
[16:20] <mup> PR #5111: cmd/snap: update install/refresh help vs --revision <Created by pedronis> <https://github.com/snapcore/snapd/pull/5111>
[16:22] <Chipaca> pedronis: you can also refresh to a local revision
[16:24] <pedronis> I killed too much text?
[16:24] <Chipaca> reading
[16:24] <pedronis> it was not saying that before either
[16:24] <Chipaca> ah, no, it didn't mention this before either
[16:24] <pedronis> we can fix it though
[16:25] <Chipaca> wording is tricky :-)
[16:25] <pedronis> just trying to understand if I killed something that was there
[16:25] <Chipaca> no, it's fine
[16:25] <Chipaca> you even fixed the revision descr on refresh which i'd missed
[16:29] <pedronis> ", unless is one of the previous locally kept revisions,"  ?
[16:31] <pedronis> Chipaca:   https://pastebin.ubuntu.com/p/w3sTVMDQXv/  ?
[16:31] <Chipaca> pedronis: hmm, that's rather awkward
[16:31] <Chipaca> pedronis: leave it as is for now
[16:32] <Chipaca> pedronis: we can iterate when my head is more into it
[16:32] <Chipaca> sorry
[16:32] <pedronis> that's fine
[16:32] <pedronis> at least there's a PR
[16:32] <pedronis> so we don't forget
[16:32] <pedronis> and don't lie either
[16:34]  * pedronis -> dinner
[16:35] <Chipaca> ok, ttfn
[17:10] <cachio_> zyga, https://github.com/snapcore/spread-images/pull/10
[17:10] <mup> PR spread-images#10: New task to add debian-sid as a gce image <Created by sergiocazzolato> <https://github.com/snapcore/spread-images/pull/10>
[17:10] <cachio_> could you please take a look
[17:10] <cachio_> it is ready again
[17:10] <cachio_> thanks
[17:13] <cachio_> zyga, we have again working the builds in travis
[17:13] <zyga> Thank you
[17:25] <mup> PR snapcraft#2110 opened: tests: conditionally add python3-distutils <Created by sergiusens> <https://github.com/snapcore/snapcraft/pull/2110>
[17:50] <pstolowski> niemeyer: #4358 is ready for your re-review
[17:50] <mup> PR #4358: interfaces: interface hooks implementation <Created by stolowski> <https://github.com/snapcore/snapd/pull/4358>
[17:50] <niemeyer> pstolowski: Thanks!
[17:50] <niemeyer> pstolowski: Not quite sure if we should merge it now, though, given that you'll be away most of next week.. we risk having issues in edge that won't be addressed before you're back
[17:51] <pstolowski> niemeyer: valid concern
[17:52] <pstolowski> niemeyer: perhaps jdstrand would like to have a look still.. not sure
[17:52] <niemeyer> pstolowski: Not sure if that's required.. have we changed anything since his last review?
[17:53] <pedronis> he didn't review it at all yet
[17:53] <pedronis> I think
[17:54] <pedronis> mmh, github is giving me a unicorn
[17:54] <pedronis> no, indeed, no review from him yet
[17:55] <pstolowski> niemeyer: yes, as pedronis says, Jamie didn't look at it yet
[17:55] <pstolowski> yes i've unicorns a lot today too
[17:56] <niemeyer> pstolowski: Would be worth pinging jdstrand for a review then
[17:56] <niemeyer> pstolowski: He'll have all of next week to have a look
[17:56] <pstolowski> allright
[17:56] <pedronis> he is at the sprint tough
[17:57] <pstolowski> jdstrand: in case you'll be reviewing #4358, the most critical part is small changes around policy checks
[17:57] <mup> PR #4358: interfaces: interface hooks implementation <Created by stolowski> <https://github.com/snapcore/snapd/pull/4358>
[18:00] <pstolowski> ok, i'm signing off, have safe trips!
[18:17] <jdstrand> pstolowski|eow: ack and noted
[19:32] <zyga> jdstrand: are you in Europe already?
[19:33] <magicaltrout> congrats to all you snappy folk for the new found 18.04 love! Well deserved.
[19:34] <zyga> woot, snaps progressed quite a lot since 15.04 days :)
[19:41] <thresh> they were available since 15.04?!
[19:42] <thresh> you should really spend some $$$ on marketing ...
[19:42] <jdstrand> zyga: leave tomorrow
[20:52] <om26er> zyga: anything specific you noticed ?
[21:21] <mcphail> Hmm. building on build.snapcraft.io: the amd64 build fails due to a compilation error, the i386 build completes successfully and then the arm build fails due to a different compilation error. No idea how to debug that one :(