mborzecki | morning | 06:45 |
---|---|---|
mvo | hey mborzecki | 06:48 |
mborzecki | mvo: did you get my messages yesterday? | 06:49 |
mvo | mborzecki: yes, interessting results | 06:54 |
mborzecki | mvo: even though just a few interfaces are connected, we compile bpf for every declared plug | 06:55 |
mvo | mborzecki: yeah, thta seems very wasteful | 06:56 |
mborzecki | mvo: w8, hm i got it wrong, it's compiling bpf for each app, and that snap has plenty of apps | 06:57 |
mborzecki | mvo: still, it's weird, looking at the log, we connect opengl, but still compile new profiles for apps that don't declare opengl | 07:00 |
mvo | mborzecki: oka | 07:05 |
=== pstolowski|afk is now known as pstolowski | ||
pstolowski | mornings | 08:03 |
mborzecki | pstolowski: hey | 08:04 |
pstolowski | mvo: thanks for the review of hotplug pr! | 08:12 |
mvo | pstolowski: my pleasure | 08:12 |
Chipaca | moin all | 08:24 |
mvo | hey Chipaca - good morning | 08:39 |
pstolowski | mvo: addressed your comments re #6465 ; regarding open points (the Specification stuff), we agreed with pedronis that it will be addressed in a followup, so once you're happy about the resolution of your suggestions this PR can land | 08:50 |
mup | PR #6465: overlord/ifacestate: hotplug-add-slot handler <Hotplug 🔌> <Created by stolowski> <https://github.com/snapcore/snapd/pull/6465> | 08:50 |
mvo | pstolowski: cool, looking | 08:50 |
mvo | pstolowski: I added feedback to 6322 as well (not sure if you saw it) | 09:19 |
pedronis | Chipaca: hi, could double check my last changes to #6469 | 09:19 |
mup | PR #6469: image,cmd/snap,tests: introduce support for modern prepare-image --snap <snap>[=<channel>] <Created by pedronis> <https://github.com/snapcore/snapd/pull/6469> | 09:19 |
pstolowski | mvo: yes i saw it, will get to it later today, thank you | 09:20 |
mvo | pstolowski: thanks, no rush | 09:21 |
mup | PR snapd#6465 closed: overlord/ifacestate: hotplug-add-slot handler <Hotplug 🔌> <Created by stolowski> <Merged by stolowski> <https://github.com/snapcore/snapd/pull/6465> | 09:40 |
pstolowski | yay \o/ | 09:40 |
pedronis | great! | 09:40 |
* pstolowski reading my master theis about selinux from 2008. real journey to the past ;) | 09:41 | |
mvo | pstolowski: nice | 09:42 |
leinardi | kenvandine: hi Ken, not sure if my message was posted here yesterday because I had connection issues. Any news about building GTK 3.24 for GWE? https://forum.snapcraft.io/t/support-for-gtk-3-24/9782/6 | 09:43 |
mup | PR snapd#6484 opened: debian: ensure leftover usr.lib.snapd.snap-confine is gone <Created by mvo5> <https://github.com/snapcore/snapd/pull/6484> | 09:49 |
pstolowski | mvo: review of #6478 would be great | 09:50 |
mup | PR #6478: overlord/ifacestate: tweak logic for generating unique slot names <Hotplug 🔌> <Created by stolowski> <https://github.com/snapcore/snapd/pull/6478> | 09:50 |
mvo | pstolowski: have you merged master? | 09:50 |
mvo | xnox: I updated lp 1814141 - if you could double check the PR that would be great | 09:51 |
pstolowski | mvo yes | 09:51 |
mvo | ta | 09:51 |
pedronis | Chipaca: have you applied all feedback to #6034 or is it still wip ? | 09:54 |
mup | PR #6034: many: save media info when installing, show it when listing <Squash-merge> <Created by chipaca> <https://github.com/snapcore/snapd/pull/6034> | 09:54 |
pstolowski | mvo: ty! | 09:57 |
Chipaca | pedronis: i think it's ready | 10:02 |
Chipaca | pedronis: "Specify extra snaps from store or local or optionally control the channel to track for a snap" | 10:02 |
Chipaca | pedronis: hmm | 10:02 |
pedronis | Chipaca: it was there before | 10:03 |
Chipaca | pedronis: i missed it before :-) | 10:03 |
pedronis | I mean, I didn't change it in this last commit | 10:03 |
Chipaca | yeah | 10:03 |
pedronis | I moved a bit out | 10:03 |
pedronis | but not changed it | 10:03 |
Chipaca | pedronis: thank you for the value-name change, makes things nicer | 10:04 |
pedronis | yea | 10:04 |
pedronis | Chipaca: about your other PR, I asked because loadStore seems still there ? | 10:04 |
Chipaca | pedronis: we can i18n that if you want, fwiw (would take abit of coding similar to other things) | 10:04 |
Chipaca | i thought i'd changed that | 10:05 |
* Chipaca checks | 10:05 | |
Chipaca | dang | 10:05 |
pedronis | Chipaca: anyway at least that help sentence for --snap should use the singular now | 10:05 |
pedronis | I think | 10:05 |
Chipaca | pedronis: sorry, will fix in a bit | 10:05 |
pedronis | Chipaca: np | 10:06 |
Chipaca | pedronis: maybe, Include this snap (optionally controling the channel it tracks); may be from store or local | 10:07 |
Chipaca | pedronis: but, I'd also say something about repeating it | 10:08 |
Chipaca | because the help doesn't | 10:08 |
Chipaca | I wish it did :-/ | 10:08 |
Chipaca | the help sucks | 10:08 |
Chipaca | prepare-image-OPTIONS what | 10:08 |
pedronis | Chipaca: but you can control also the channel of a snap you are not including | 10:08 |
Chipaca | :-/ | 10:08 |
pedronis | that is also already included | 10:08 |
Chipaca | yeah | 10:09 |
Chipaca | Have this snap track the given channel (default: latest/stable); if not present, include it from store or local file | 10:11 |
Chipaca | if not already included*? | 10:11 |
mborzecki | re | 10:12 |
pedronis | Chipaca: pushed something | 10:14 |
pedronis | a bit different | 10:14 |
Chipaca | pedronis: +1 | 10:15 |
pedronis | Chipaca: yes, the prepare-image-OPTIONS is not beautiful | 10:17 |
pedronis | that's go-flags though, no? | 10:17 |
Chipaca | yes | 10:17 |
Chipaca | and, you'd think we could just set Usage to override that | 10:17 |
Chipaca | but https://github.com/jessevdk/go-flags/issues/264 | 10:18 |
pedronis | heh (me has stopped making optimistic/navie assumption about that package long ago) | 10:18 |
Chipaca | yeah, no navy assumptions | 10:20 |
pedronis | naive :P | 10:21 |
Chipaca | no u | 10:23 |
mup | PR snapd#6406 closed: tests: enable debian sid as part of the main suite on travis <Created by sergiocazzolato> <Closed by mvo5> <https://github.com/snapcore/snapd/pull/6406> | 10:28 |
mup | PR snapd#6413 closed: packaging: import debian salsa packaging work, add sbuild test and use in spead <Created by mvo5> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/6413> | 10:28 |
mvo | down to 39! | 10:28 |
pedronis | Chipaca: the description (future commit message) needs updating as well | 10:32 |
Chipaca | yep | 10:32 |
Chipaca | it's going to get squashed, but yes | 10:32 |
pedronis | Chipaca: also my two comments about the undo logic seems unanswered | 10:34 |
Chipaca | pedronis: i answered them in my head, does that count? | 10:35 |
* Chipaca looks at the pr again | 10:35 | |
pedronis | Chipaca: .5 cookie | 10:35 |
* Chipaca could have second breakfast | 10:35 | |
Chipaca | pedronis: actually, which two? | 10:36 |
pedronis | Chipaca: https://github.com/snapcore/snapd/pull/6034#discussion_r254207769 and https://github.com/snapcore/snapd/pull/6034#discussion_r254207844 | 10:36 |
mup | PR #6034: many: save media info when installing, show it when listing <Squash-merge> <Created by chipaca> <https://github.com/snapcore/snapd/pull/6034> | 10:36 |
Chipaca | pedronis: ah, phew | 10:37 |
Chipaca | pedronis: answered | 10:37 |
Chipaca | that wasn't undo, that was cleanup of do | 10:38 |
Chipaca | ;) | 10:38 |
mvo | mwhudson: the debian packaging of salsa is now part of our tree in packaging/debian-sid and we use that for automatic testing - juts fyi - so on the next merge you can hopefully just "git mv package/debian-sid debian" and then we are in sync and we can just cherry-pick/merge your changes back into our tree (and vice-versa) | 10:38 |
mup | PR snapd#6478 closed: overlord/ifacestate: tweak logic for generating unique slot names <Hotplug 🔌> <Created by stolowski> <Merged by stolowski> <https://github.com/snapcore/snapd/pull/6478> | 10:39 |
pedronis | Chipaca: ah, this: TestDoLinkSnapFailureCleansUpAux | 10:40 |
Chipaca | pedronis: ye | 10:40 |
Chipaca | it's a bit chummy with the implementation in how it triggers the failure | 10:41 |
Chipaca | but i reckon it's good enough | 10:41 |
pstolowski | naming tweaks landed as well, and #5962 updated with master and ready for reviews. i'll prep followup with the simplification of spec | 10:43 |
mup | PR #5962: ifacestate/hotplug: hotplug handlers <Complex> <Hotplug 🔌> <Created by stolowski> <https://github.com/snapcore/snapd/pull/5962> | 10:43 |
pedronis | Chipaca: +1 with 1 nitpick, it needs a 2nd review tough because I think it changed too much since zygmunt looked at it | 10:43 |
pedronis | (I dismissed already his review) | 10:44 |
Chipaca | pedronis: agreed | 10:44 |
Chipaca | pedronis: there's an obvious followup up to this with categories | 10:45 |
pedronis | Chipaca: btw, do we really need the store bit in "store/aux" , other things under cache also come from the store already, no? | 10:46 |
Chipaca | pedronis: good point | 10:46 |
pedronis | we could s/aux/aux-info/ if it's clearer, not sure | 10:47 |
mup | PR snapd#6485 opened: interfaces/seccomp: regenerate changed profiles only <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/6485> | 10:59 |
mborzecki | mvo: ^^ | 10:59 |
mborzecki | mvo: there are some numbers in the PR description | 10:59 |
Wimpress | Morning mvo. I have a font related query regarding snapd. | 11:10 |
Wimpress | It doesn't appear to be possible to user installed or user local fonts in classic snaps. | 11:10 |
mborzecki | Wimpress: only in classic snaps? | 11:11 |
Wimpress | For examples, if I install fonts in ~/.fonts, ~/.local/share/fonts or /usr/local/share fonts I am unable to use them in the code-insiders snap, which uses classic confinement. | 11:11 |
Wimpress | mborzecki: Right now, I'm only interested in a classic snaps. | 11:12 |
Chipaca | Wimpress: have you tried using them in a classic snap that doesn't use the desktopy parts? | 11:12 |
Wimpress | Chipaca: There are no desktop helpers in the code-insiders snap. | 11:12 |
Chipaca | ah | 11:13 |
mvo | mborzecki: nice, looking | 11:13 |
mvo | Wimpress: in classic confined snaps, that is slightly unexpected | 11:13 |
Wimpress | Glad you say that, because I thought so too. | 11:14 |
Wimpress | There is an upstream bug raised against VSCode. I am investigating. | 11:14 |
mvo | Wimpress: silly question, what do I need to do to reproduce? | 11:15 |
mvo | Wimpress: I have vscode, how to instlal a local font? | 11:15 |
* mvo feels like a real n00b | 11:15 | |
Wimpress | Install a monospaced font to a user location, such as ~/.fonts or ~/.local/share/fonts and then be able to set that font in the VSCode font preferences. | 11:16 |
Chipaca | mvo: git clone https://go.googlesource.com/image | 11:16 |
Wimpress | `Ctrl+,` to open preferences. | 11:17 |
Chipaca | mvo: and copy the resulting image/font/gofont/Go-Mono*.ttf to ~/.fonts | 11:17 |
Chipaca | :-) | 11:17 |
mborzecki | Wimpress: trying that now | 11:17 |
kenvandine | leinardi: we're still working on that. Once we figure out the gtk build failure in the build snap i'll give you a patch that adds it to your snap | 11:17 |
kenvandine | leinardi: oSoMoN is looking into that now | 11:18 |
mborzecki | Wimpress: do you know if any other classic snaps are affected too? eg. atom maybe? | 11:18 |
Wimpress | I have checked any others, but I can. | 11:18 |
Wimpress | *haven't | 11:20 |
pstolowski | http://paste.ubuntu.com/p/SCRGVsgCnq/mvo: got external:ubuntu-core-18-arm-32:tests/core18/compat failure on pi3; will re-run manually ; have you seen it failing before? full log: | 11:23 |
Wimpress | The font being referenced in the upstream bug is this - https://github.com/tonsky/FiraCode/releases/download/1.206/FiraCode_1.206.zip | 11:23 |
pstolowski | mvo: http://paste.ubuntu.com/p/SCRGVsgCnq/ | 11:23 |
mvo | pstolowski: yes, I think its the same as yesterday, i.e. there is a core installed in the state but not on disk because prepare/restore do crazy stuff | 11:24 |
Wimpress | I've also tried rebuiling font cahces using `fc-cache -f -v` and `/snap/core/current/bin/fc-cache-v6 -f -v` and `/snap/core/current/bin/fc-cache-v7 -f -v` | 11:25 |
mvo | Wimpress: this is slightly mysterious, so I ran: "snap run --shell vscode" and in there gnome-font-viewer and I see the go fonts there but I don't have them in vscode | 11:26 |
Wimpress | mvo, Install `code-insider` instead. It is published by Microsoft. | 11:26 |
Wimpress | But yes, I am seeing the same issue. | 11:27 |
Chipaca | code-insiders* | 11:27 |
Wimpress | Although I using `mate-font-viewer` ;-) | 11:27 |
Wimpress | Yes, sorry. | 11:28 |
mvo | Wimpress: haha | 11:28 |
mvo | Wimpress: no worries | 11:28 |
Wimpress | Trying the atom snap. | 11:28 |
mvo | Wimpress: trying code-insiders now, *maybe* its related to the framework they use? | 11:28 |
mvo | Wimpress: I will try to run it outside of snap, just from local dir next | 11:28 |
Wimpress | Does fc-cache-v7 and fc-cache-v6 run in the root context when a snap is installed. | 11:29 |
Chipaca | yeah, fc-list works fine inside the snap | 11:29 |
mvo | Wimpress: fwiw, they really should consider using ubuntu mono as their defualt font | 11:30 |
mvo | Wimpress: yes | 11:30 |
Wimpress | Chipaca: You ran that from within the Code terminal, right? | 11:30 |
mvo | Wimpress: it runs as root but that should not matter, its just a cache | 11:30 |
Chipaca | um | 11:31 |
Chipaca | it picks up the font just fine | 11:31 |
Wimpress | Running `fc-list | grep Fira` from within the Code terminal does show me the user installed fonts from ~/.fonts | 11:31 |
mvo | Wimpress: also running vscode without confinement (just ./bin/electron-launch ...) gives me the same result so it looks like its not our fault(tm) :) | 11:32 |
Wimpress | Chipaca: So, if you change the Editor: Font Family in Code, you can see any open files rendered using that font? | 11:32 |
Chipaca | Wimpress: https://i.imgur.com/N333CRv.png | 11:32 |
Wimpress | Chipaca: Please can you confim the Fira Code font also works for you? | 11:34 |
Wimpress | I'll try the Go font you suggested here. | 11:34 |
Chipaca | Wimpress: I don't have Fira | 11:34 |
Chipaca | isn't that the one with all the funky ligatures? | 11:34 |
tomwardill | I use the vscode snap with Fira, if that helps | 11:36 |
Chipaca | Wimpress: does Fira Code work on 16.04? | 11:36 |
Wimpress | tomwardill: Where did you install the Fira font? | 11:38 |
tomwardill | sec, checking | 11:38 |
Wimpress | Chipaca: I can't get the Go font to render. | 11:38 |
Chipaca | Wimpress: what are you putting in settings? | 11:38 |
Wimpress | ...in the code-insiders snap. | 11:38 |
Wimpress | `'Go', 'Droid Sans Mono', 'monospace', 'Droid Sans Fallback'` | 11:38 |
Chipaca | Wimpress: the font is called Go Mono | 11:38 |
tomwardill | Wimpress: I have it install via the apt package | 11:38 |
Wimpress | tomwardill: OK, thanks. system wide fonts are working fine. | 11:39 |
Wimpress | The issue here user installed fonts in ~/.fonts for example. | 11:39 |
tomwardill | ah, sorry, missed that context in scrollback | 11:40 |
Wimpress | np :-) | 11:40 |
Chipaca | Wimpress: https://imgur.com/a/Rz26nPz | 11:41 |
Chipaca | Wimpress: fira code works | 11:41 |
Chipaca | using the fonts-firacode deb package from bionic | 11:42 |
Chipaca | Wimpress: I'm on 16.04 | 11:42 |
Chipaca | ah | 11:42 |
Chipaca | oh | 11:42 |
mvo | Chipaca: aha, using the deb | 11:42 |
Wimpress | I am on 18.10 | 11:42 |
Chipaca | removing the deb, moving the fonts to local dir, 1 sec | 11:42 |
* mvo also tested on 18.10 | 11:42 | |
Wimpress | Ah, you have the deb. | 11:42 |
mvo | Chipaca: aha, ok | 11:42 |
Chipaca | mvo: Wimpress: removed the deb, downloaded the .ttf and put them in .fonts, it works | 11:46 |
Wimpress | Works on 16.04? | 11:46 |
Chipaca | yes | 11:46 |
zyga | hey | 11:46 |
Chipaca | Wimpress: what do you have in your fonts dir? | 11:46 |
Chipaca | Wimpress: and, what are you putting in settings | 11:46 |
* zyga pops in briefly for the last day of his "holiday" break | 11:46 | |
Chipaca | zyga: shoo go coo at a baby | 11:46 |
zyga | mvo: I will send the patches that you requested in a moment | 11:46 |
zyga | Chipaca: the baby is still at the hospital, not coming home yet, still sick :( | 11:47 |
Chipaca | zyga: :'( | 11:47 |
zyga | Chipaca: not serious but must stay for a few more days | 11:47 |
Wimpress | https://www.irccloud.com/pastebin/xysZGLIn/ | 11:47 |
zyga | how are fires this week? | 11:47 |
Wimpress | Those are the fonts I have in ~/.fonts | 11:47 |
mvo | zyga: ta | 11:47 |
zyga | mwhudson: thank you for the debian upload, I was going to make one but I'm super happy to see it's done :) | 11:47 |
mvo | zyga: good luck | 11:47 |
Chipaca | Wimpress: yep, got those also | 11:47 |
mvo | zyga: packaging is merged | 11:48 |
mvo | :) | 11:48 |
Chipaca | Wimpress: and in your settings? | 11:48 |
Wimpress | Chipaca: I will test on a 16.04 VM | 11:48 |
Wimpress | `'Fira Code', 'Droid Sans Mono', 'monospace', 'Droid Sans Fallback'` | 11:49 |
Chipaca | Wimpress: and if you put just: Fira Code | 11:49 |
Chipaca | Wimpress: no quotes, no list, no nuthin' | 11:49 |
Chipaca | (that's what i have) | 11:49 |
Wimpress | It is not Fira Code rendering the test I do that. | 11:50 |
Wimpress | Looks like Courier. | 11:50 |
Chipaca | yeah it's very obvious when it's not finding it | 11:50 |
Chipaca | it uses a proportional font, even, at least here | 11:50 |
Chipaca | (but i probably don't have courier new) | 11:50 |
Chipaca | Wimpress: I'll try on 18.04 here and report back | 11:51 |
Wimpress | Chipaca: thanks. I'll give 16.04 a go. | 11:51 |
Chipaca | Wimpress: works on 18.04 as well | 11:55 |
Chipaca | Wimpress: with both Go Mono and Fira Code | 11:56 |
Chipaca | Wimpress: ligatures and all | 11:56 |
Wimpress | mvo, noted. I'll feed that back. | 11:58 |
Wimpress | regarding using ubuntu mono as a default. | 11:59 |
mvo | Wimpress: ta | 12:01 |
mvo | Wimpress, Chipaca interessting - works for me on 18.04 as well | 12:02 |
Wimpress | Chipaca: popey confirms Fira Code in working on 18.04 with the code-insiders snap. | 12:02 |
mvo | I tested with the go fonts and vscode snap | 12:02 |
diddledan | broken in 18.10 for me | 12:02 |
popey | https://usercontent.irccloud-cdn.com/file/GGM2tbAk/Bootiful%20font%20on%2018.04 | 12:02 |
mvo | yeah, same here, no fun on 18.10 | 12:02 |
diddledan | I figured I had configured it wrong all this time >.< | 12:03 |
Wimpress | Thanks for the confirmations. | 12:03 |
popey | i just booted an 18.10 vm... | 12:04 |
diddledan | fc-list from within the code inbuilt shell (ctrl+`) doesn't show the font I've configured | 12:04 |
popey | but i appear to have picked the wrong vm | 12:04 |
popey | https://usercontent.irccloud-cdn.com/file/n1NdD344/does%20this%20support%20snaps%20%3B)%20 | 12:04 |
diddledan | from within the inbuilt vscode shell (ctrl+`) https://www.irccloud.com/pastebin/PNe8FQgQ/ | 12:05 |
diddledan | oh, I lie, it is there.. just intermixed with all the others even though it is in my locala fonts dir | 12:06 |
diddledan | fura code regular | 12:06 |
diddledan | could it be, in my case, that I'm using an otf font? | 12:07 |
Wimpress | diddledan: I only have "installed" the ttf version oof Fira Code in ~/.fonts | 12:09 |
diddledan | aah ok | 12:09 |
diddledan | I'm using the repacked fira code by nerd fonts | 12:10 |
diddledan | added DejaVu Sans Mono as a fallback when it can't find fura code regular and it looks sooo much better :-p | 12:12 |
diddledan | I think it uses a bitmap font when none of the ones in the list are found | 12:12 |
diddledan | ugly jaggies | 12:12 |
popey | Ok, font looks bad on 18.10 | 12:15 |
popey | https://usercontent.irccloud-cdn.com/file/CS7igM08/Screenshot%20from%202019-02-08%2012-15-14.png | 12:15 |
popey | choosing ubuntu mono works fine | 12:16 |
diddledan | yours is decidedly more pretty https://usercontent.irccloud-cdn.com/file/IunzHJTb/image.png | 12:17 |
diddledan | I miss hidpi when I'm on this desktop :-( | 12:19 |
ogra | just move the monitors 2m away | 12:26 |
Chipaca | Wimpress: so | 12:31 |
Chipaca | Wimpress: I can confirm it's not finding the font in 18.10 | 12:31 |
Chipaca | Wimpress: but | 12:31 |
Chipaca | Wimpress: putting the full path to the font works | 12:31 |
Wimpress | Whhhhaaaaat?! | 12:32 |
Chipaca | Wimpress: dude try it :-) | 12:32 |
Wimpress | Full path to a ttf? | 12:32 |
Chipaca | yes | 12:32 |
Chipaca | Wimpress: what was another snap that didn't find local fonts in 18.10? | 12:33 |
Wimpress | I can't confirm your workaround. | 12:34 |
Chipaca | Wimpress: why? | 12:34 |
Wimpress | `/home/martin/.fonts/FiraCode-Regular.ttf, 'Droid Sans Mono', 'monospace', monospace, 'Droid Sans Fallback'` | 12:34 |
Wimpress | Does not render using Fira. | 12:34 |
Chipaca | Wimpress: just the path, nothing more | 12:34 |
Chipaca | I have no idea what the rules are for quoting and enumerating in that text entry | 12:35 |
popey | "nothing more"? that *is* the path | 12:35 |
Chipaca | booting the cd up again just in case | 12:35 |
Wimpress | WHat is your font settting? | 12:35 |
Chipaca | popey: all the things after the comma, remove them | 12:35 |
popey | those are the defaults that vscode supports | 12:36 |
Chipaca | doing this in a vm all over again is very slow | 12:39 |
Wimpress | Atom also has the same issue | 12:39 |
Chipaca | booting with kvm, putting just /home/ubuntu/.fonts/FiraCode-Medium.ttf | 12:42 |
Chipaca | it picks it up | 12:42 |
Chipaca | doesn't get ligatures | 12:42 |
Chipaca | in code | 12:43 |
Chipaca | I can try atom next if you want? | 12:43 |
popey | specifying the full path to the font on 18.10 does not work here | 12:47 |
popey | it defaults back to a variable width font | 12:47 |
Wimpress | Just tested with pycharm-community snap. Also classic but implemented using Java, not Electron. | 12:48 |
Wimpress | I can set the editor font to Fira Code on 18.10 just fine. | 12:48 |
* pstolowski lunch | 13:18 | |
=== ricab is now known as ricab|bbl | ||
kravietz | Hello, maybe a silly question - how do I run snapcraft 3 on a VM? It requires multipass but multipass won't run on a VM and all of my build servers run in cloud... | 13:38 |
mup | PR snapd#6469 closed: image,cmd/snap,tests: introduce support for modern prepare-image --snap <snap>[=<channel>] <Created by pedronis> <Merged by pedronis> <https://github.com/snapcore/snapd/pull/6469> | 13:48 |
pedronis | sil2100: ^ https://forum.snapcraft.io/t/evolving-prepare-image-syntax/5988/4 what was mentioned on Tue | 13:50 |
=== Son_Goku is now known as Conan_Kudo | ||
=== Conan_Kudo is now known as Son_Goku | ||
sil2100 | pedronis: excellent | 13:53 |
sil2100 | I'll pick that up for ubuntu-image, possibly next week | 13:54 |
pedronis | thank you | 13:54 |
mup | Issue # closed: core18#56, core18#86, core18#89, core18#117 | 14:03 |
mup | PR # closed: core18#43, core18#63, core18#72, core18#90, core18#98 | 14:03 |
mup | Issue # opened: core18#56, core18#86, core18#89, core18#117 | 14:04 |
mup | PR # opened: core18#43, core18#63, core18#72, core18#90, core18#98 | 14:04 |
dot-tobias | Does someone have experience with network-manager / nmcli calls from a core18 base'd snap? I'm really lost as to what I might be doing wrong … https://forum.snapcraft.io/t/apparmor-denials-for-dbus-objectmanager-network-manager/9885 | 14:28 |
ondra | sergiusens is there easy way to tell snapcraft which multipass instance to use for build? | 14:37 |
sergiusens | Wimpress: I followed your instructions for the font with code-insiders on disco and it wasn't picked up | 14:37 |
sergiusens | ondra: what do you mean? | 14:38 |
ondra | sergiusens correct me if I'm wrong, but snapcraft will try to run clean build by creating new multipass instance? | 14:39 |
ondra | sergiusens or is it more instance per-project? | 14:39 |
ondra | sergiusens so is there option instead to tell it to use existing instance I already have? | 14:39 |
sergiusens | ondra: it is, per project | 14:40 |
ondra | sergiusens ah cool, and can I tell it to use specific one? | 14:40 |
sergiusens | ondra: it is simple today, by name, "snapcraft-<snap-name>", but if you do manual things, it could get icky with some things done using cloud-init | 14:40 |
ondra | sergiusens ah OK, so better to map dir and build inside instance directly | 14:41 |
sergiusens | when we have support of some form of "project" from multipass, we will lock it down more | 14:41 |
sergiusens | yes | 14:41 |
ondra | sergiusens cool | 14:41 |
leinardi | kenvandine: cool, thanks a lot | 15:08 |
kenvandine | leinardi: np | 15:08 |
roadmr | jdstrand: hi :) | 15:09 |
mup | PR snapd#6484 closed: debian: ensure leftover usr.lib.snapd.snap-confine is gone <Created by mvo5> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/6484> | 15:18 |
pedronis | mvo: did a first pass on the 1st Remodel PR | 15:24 |
mup | PR snapd#6486 opened: interfaces/hotplug: renamed RequestedSlotSpec to ProposedSlot, removed Specification <Hotplug 🔌> <Created by stolowski> <https://github.com/snapcore/snapd/pull/6486> | 15:28 |
mvo | pedronis: thank you | 15:29 |
pedronis | mvo: what's the status of the discussion about where to do things in snapd vs snap-confine, in https://github.com/snapcore/snapd/pull/6418? | 15:30 |
mup | PR #6418: many: allow core as a fallback for core16 <Created by mvo5> <https://github.com/snapcore/snapd/pull/6418> | 15:30 |
mvo | pedronis: impas it seems, I would prefer snap-confine.c as we already to something similar there - unless I miss something? | 15:31 |
pedronis | mvo: do we know where in snapd would we do this? | 15:31 |
mvo | pedronis: in cmd_run.go I think | 15:31 |
pedronis | or is that snap-exec or snap run? | 15:31 |
mvo | pedronis: snap run | 15:31 |
mvo | pedronis: I can explore this and push a passtebin with the required work | 15:32 |
pedronis | mvo: just remember my point, we don't want base: core16 not to pivot | 15:32 |
pedronis | it will hide bugs that will hunt us later | 15:32 |
mvo | pedronis: aha, I see | 15:33 |
pedronis | so from that point view no base, base: core16 are not quite the seem | 15:33 |
pedronis | it feels right | 15:33 |
pedronis | but caveats | 15:33 |
mvo | pedronis: yeah, if we do it in snap run this will be less clear | 15:33 |
pedronis | mvo: yes, we cannot turn base: core16 into no base | 15:34 |
pedronis | and be done | 15:34 |
mvo | pedronis: right | 15:34 |
mup | PR snapd#6487 opened: interfaces: add new intel-management-interface interface <Created by mvo5> <https://github.com/snapcore/snapd/pull/6487> | 15:48 |
pstolowski | mvo: i'm going to run "console-conf core16 fresh install pi3", ok? | 16:41 |
=== ricab|bbl is now known as ricab | ||
mvo | pstolowski: yes, thank you! | 16:44 |
pstolowski | mvo: for console-conf, i need to first configure device manually, then run the tests (and presumably they re-run the setup)? | 17:13 |
mvo | pstolowski: actually I don't know, I think its explained in the README.md, at least I did run this successfully once | 17:15 |
mvo | pstolowski: if you don't figure it out I can look after dinner | 17:15 |
pstolowski | mvo: hmm, it tries to ssh to the given IP, which won't work until pi3 is configured (with my email address). i'm running "DEVICE_USER=stolowski DEVICE_IP=192.168.1.56 scripts/run_external_device.sh dev_cconf_pi3" | 17:23 |
=== phoenix_firebrd is now known as murthy | ||
pstolowski | mvo: i'll get back to this on monday, have a nice weekend! | 17:27 |
=== pstolowski is now known as pstolowski|afk | ||
pedronis | have a great weekend! | 17:35 |
=== murthy is now known as phoenix_firebrd_ | ||
=== phoenix_firebrd_ is now known as murthy | ||
mvo | pstolowski|afk: yeah, sounds reasonable - have a great weekend too | 18:45 |
=== phoenix_firebrd is now known as murthy | ||
mup | PR snapcraft#2465 opened: tests: make before/after items an array in schema test <Created by kyrofa> <https://github.com/snapcore/snapcraft/pull/2465> | 22:33 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!