mborzecki | morning | 06:43 |
---|---|---|
zyga | good morning | 07:22 |
Faults | Goood morning! | 07:34 |
* zyga got a power plug adapter for south africa | 07:45 | |
mup | PR snapd#5845 closed: interface: add new `{personal,system}-files` interface <Reviewed> <Created by mvo5> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/5845> | 07:51 |
pstolowski|afk | mornings | 08:05 |
=== pstolowski|afk is now known as pstolowski | ||
mborzecki | zyga: pstolowski: hey | 08:07 |
=== JanC is now known as Guest41406 | ||
=== JanC_ is now known as JanC | ||
t1mp | hello | 09:17 |
t1mp | is this the place for questions relating to snap packaging problems? | 09:17 |
zyga | t1mp: hey, for various kinds of topics related to snaps | 09:17 |
t1mp | I created a snap package, which has a python+qt app inside (using PySide2, and PyInstaller). It works on most systems, but when the host is running Wayland, it fails as follows: | 09:22 |
t1mp | https://pastebin.ubuntu.com/p/whWXk2drjW/ | 09:22 |
t1mp | greyback is already helping me with it :) | 09:22 |
t1mp | for reference, my snapcraft.yaml is here: https://pastebin.ubuntu.com/p/KN9qsVRdJ8/ | 09:27 |
Chipaca | morning all | 09:33 |
popey | Good day | 09:33 |
popey | t1mp: hey! don't you also need the 'wayland' plug? | 09:36 |
popey | I have seen some options where you use x11 and wayland plug but disable wayland in the environment which triggers it to use xwayland | 09:36 |
t1mp | popey: hello :) | 09:37 |
t1mp | I have no idea. Is there documentation about the wayland plug somewhere? | 09:37 |
t1mp | I thought all was working, but now when testing on different systems I found that the snap doesn't work on systems with wayland. | 09:38 |
mup | PR snapd#5915 closed: interfaces/network-setup-control: allow calling netplan generate/apply <Created by ogra1> <Closed by pedronis> <https://github.com/snapcore/snapd/pull/5915> | 09:38 |
popey | https://forum.snapcraft.io/t/wayland-crash-with-electron-2/5752/4?u=popey | 09:38 |
popey | t1mp: theres a similar thread, where they use DISABLE_WAYLAND=1 in the environment to force xwayland | 09:39 |
popey | (I *think* that's how it works) | 09:39 |
popey | The documentation for the wayland interface is somewhat lean https://docs.snapcraft.io/the-wayland-interface/7784 | 09:39 |
popey | https://github.com/notepadqq/notepadqq/blob/master/snap/snapcraft.yaml#L16 | 09:40 |
popey | someone else using that method | 09:40 |
t1mp | shouldn't those kind of things be included in the desktop-helpers? | 09:42 |
t1mp | I could try the DISABLE_WAYLAND env var | 09:42 |
popey | possibly | 09:42 |
t1mp | plugs are irrelevant as long as I'm using --devmode right? | 09:43 |
popey | yes | 09:43 |
popey | (I think) | 09:43 |
popey | you can use "snappy-debug.security scanlog" to learn what's failing apparmor | 09:44 |
popey | leave that running in another terminal when you run the app | 09:44 |
popey | might be enlightening | 09:44 |
zyga | brb | 09:49 |
t1mp | greyback: interesting, if I run the app (packaged with PyInstaller), but outside of the snap, it works fine. But if I set QT_QPA_PLATFORM=wayland, then it starts but without window decoration or images in the app. | 09:58 |
mup | Issue core18#93 closed: i386 support missing on core18-amd64 <Created by mmtrt> <Closed by sil2100> <https://github.com/snapcore/core18/issue/93> | 09:59 |
t1mp | The missing images looks familiar. I think I had the same on X when I missed some files to have GLSL support | 10:02 |
zyga | quick coffee to raise the pressure | 10:10 |
zyga | it's so sleepy today | 10:10 |
=== chihchun is now known as chihchun_afk | ||
=== chihchun_afk is now known as chihchun | ||
mup | PR snapd#6333 opened: daemon: intrduce /v2/connections snapd API endpoint <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/6333> | 11:06 |
t1mp | greyback, popey: https://github.com/notepadqq/notepadqq/blob/master/snap/snapcraft.yaml#L16 made the snap work for me | 12:01 |
t1mp | I don't know if it is the most elegant solution, but the package works now, which is what I need | 12:01 |
t1mp | thanks for the help :) | 12:01 |
t1mp | I still need to test it on some PCs though | 12:01 |
greyback | t1mp: ok, glad you got it working | 12:02 |
greyback | it appears wayland support still needs work in desktop-helpers | 12:02 |
popey | (bug reports welcome) :D | 12:08 |
diddledan | bug reports? https://www.youtube.com/watch?v=S-s4lEk91ng | 12:10 |
zyga | brb again | 12:11 |
popey | diddledan: I'm doing my part! | 12:14 |
pstolowski | mborzecki: hey, #6332 can land right? | 12:14 |
mup | PR #6332: spread: make Fedora 29 auto again <Simple 😃> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/6332> | 12:14 |
mborzecki | pstolowski: let me run the spread job again, sample of 2 is better than 1 ;) | 12:15 |
mup | PR snapd#6318 closed: release-tools: display self-help <Simple 😃> <Created by zyga> <Merged by stolowski> <https://github.com/snapcore/snapd/pull/6318> | 12:18 |
pstolowski | ok | 12:18 |
zyga | thanks pawel! | 12:21 |
zyga | running some spread tests, I will go for a walk with the dog | 12:21 |
zyga | it's snowing like crazy here | 12:21 |
zyga | I don't want to miss this ;0 | 12:22 |
zyga | :-) | 12:22 |
MattJ | Where is $here? I want to come! | 12:22 |
MattJ | (wait, that sounded creepy) | 12:22 |
zyga | it's all bound to melt away, it will be +2 soon | 12:23 |
zyga | but for now it's close to -1 so the view is very nice | 12:23 |
zyga | (in Warsaw) | 12:23 |
pedronis | mborzecki: hi, are you mostly blocked waiting on various reviews at this point? | 12:32 |
mborzecki | pedronis: i'm advancing respective pieces in private branches and working on a small fontconfig fix for fedora, but yeah, opening new PR is blocked until some of the current ones land | 12:34 |
=== ricab is now known as ricab|lunch | ||
pedronis | mborzecki: ok, I will get to your PRs but there are even older PRs that needs some attention too, I can point at some of the "small" cards I mentioned yesterday in the standup | 12:35 |
mup | PR snapd#6334 opened: dirs, interfaces/builtin/desktop: system fontconfig cache path is different on Fedora <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/6334> | 12:56 |
mborzecki | ^^ maybe a bit late for .37 but we could squeeze it in .37.1 | 12:57 |
mup | PR snapd#6332 closed: spread: make Fedora 29 auto again <Simple 😃> <Created by bboozzoo> <Merged by bboozzoo> <https://github.com/snapcore/snapd/pull/6332> | 13:00 |
mborzecki | off to pick up the kids | 13:03 |
=== chihchun is now known as chihchun_afk | ||
t1mp | greyback: yeah. Is any OpenGL stuff not working with Wayland now? That seems quite important ;) | 13:15 |
t1mp | or is it just Qt? Or something specific that I'm using | 13:15 |
greyback | t1mp: it has worked for me in the past. I don't know what broke for you, sorry | 13:16 |
* Chipaca ⇝ lunch | 13:16 | |
zyga | re | 13:26 |
zyga | t1mp: it probably depends | 13:26 |
t1mp | okay, thanks. :) For me it just matters that it works for my snap now ;) I'm sure the snap is optimal, but I'll see if I can work on it again in the near future. | 13:45 |
t1mp | *I'm sure my snap is NOT optimal :) | 13:46 |
pedronis | Chipaca: hi, could you review #6121 when you have a bit of time? | 13:56 |
mup | PR #6121: tests: new test for snapshots with more than 1 user <Created by sergiocazzolato> <https://github.com/snapcore/snapd/pull/6121> | 13:56 |
Chipaca | pedronis: yes | 13:57 |
Chipaca | I guess the current test that creates snapshots for root + testuser isn't enough? | 13:57 |
pedronis | Chipaca: I suppose, it's question for cachio | 13:58 |
pedronis | mborzecki: btw, did you see the request for tweaks to #6317 ? | 13:59 |
mup | PR #6317: overlord/snapstate/backend: call fontconfig helpers from the new 'current' <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/6317> | 13:59 |
mborzecki | pedronis: yup, saw that, it's in my queue | 14:02 |
pedronis | ok | 14:02 |
mup | Bug #1810948 opened: Internal Watcher Error <Snappy:New> <https://launchpad.net/bugs/1810948> | 14:31 |
mup | Bug #1810948 changed: Internal Watcher Error <Snappy:Invalid> <https://launchpad.net/bugs/1810948> | 14:43 |
cachio | zyga, https://travis-ci.org/snapcore/spread-cron/builds/476263240#L9952 | 15:08 |
zyga | looking | 15:08 |
cachio | that should be fixed with the update in the repo? | 15:08 |
zyga | so | 15:08 |
zyga | you need to update & reboot | 15:08 |
zyga | let me check that locally | 15:08 |
cachio | zyga, I did that | 15:09 |
zyga | in that case the fix is not out yet | 15:09 |
zyga | let's wait and see, I'll check the bug report in suse | 15:09 |
zyga | thanks for trying! | 15:09 |
cachio | np | 15:09 |
=== chihchun_afk is now known as chihchun | ||
* cachio lunch | 15:54 | |
pedronis | jdstrand: thanks for adding the docs about personal/system-files, shouldn't the doc mention somehow that even just installation requires the snap-declaration ? | 16:12 |
pedronis | (Not sure how we did this for other interfaces with the same requirement) | 16:12 |
mborzecki | feels like we should parse /etc/fonts/fonts.conf to figure out fontconfig locations rather than use hardcoded ones | 16:18 |
mborzecki | fedora/rhel/centos all use /usr/lib/fontconfig/cache, but amzn2 is behind with updates and uses /var/cache/fontconfig and obviously has ID_LIKE=".. fedora.." | 16:19 |
zyga | mborzecki: how hard is it to parse? | 16:27 |
mborzecki | zyga: rather easy, it's xml, encoding/xml should do just fine | 16:27 |
zyga | oh, that's good | 16:27 |
zyga | is the location easy to extract? | 16:28 |
mborzecki | zyga: yes, it's quite trivial :) | 16:29 |
mborzecki | zyga: btw. wonder if we shuld also use ~/.local/share/fonts and ~/.fonts, we're only using system locations now | 16:30 |
zyga | it depends if the code runs in user contetxt | 16:30 |
mborzecki | zyga: we setup those mounts in the desktop interface, i'd guess it does | 16:31 |
zyga | but who would run the parser? | 16:31 |
mborzecki | zyga: snapd could parse /etc/fonts/fonts.conf in destkop iface mount connected plug | 16:33 |
jdstrand | pedronis: probably, yes. we don't do that elsewhere that I can see but that is no reason not to do it here | 16:36 |
jdstrand | pedronis: I'll adjust | 16:36 |
zyga | mborzecki: can we discuss this tomorrow morning? | 16:44 |
zyga | I'm adding some tests still | 16:44 |
jdstrand | pedronis: done. fyi, I also updated snapd-control and kernel-module-control similarly. maybe that will reduce some questions | 16:44 |
pedronis | jdstrand: thanks | 16:50 |
pedronis | pstolowski: I left a comment in the hotplug-connect PR, we can talk about it tomorrow | 16:53 |
pstolowski | pedronis: ok, thanks | 16:54 |
mborzecki | zyga: sure | 16:55 |
=== pstolowski is now known as pstolowski|afk | ||
t1mp | kenvandine: hello :) | 17:45 |
t1mp | I just saw your comment on https://github.com/ubuntu/snapcraft-desktop-helpers/issues/172 | 17:45 |
kenvandine | hey t1mp | 17:46 |
t1mp | the way I understand it, plugs don't make a difference if the snap runs with --devmode (or as classic). Is that correct? | 17:46 |
kenvandine | oh | 17:46 |
kenvandine | right | 17:46 |
t1mp | hmm. I'm rebuilding my snap and (automatically, with jenkins) pushing it to the snap store, but I keep getting a 502 error | 17:46 |
t1mp | Snap Store encountered an error while processing your request: bad gateway (code 502). | 17:47 |
t1mp | The operational status of the Snap Store can be checked at https://status.snapcraft.io/ | 17:47 |
t1mp | script returned exit code 2 | 17:47 |
t1mp | I can see the revision though when I log in on snapcraft.io | 17:49 |
t1mp | ah no, I cannot. It is a different revision number | 17:50 |
t1mp | (just two digits swapped that's why I thought it is the same) | 17:50 |
kenvandine | t1mp: try added QT_QPA_PLATFORM=wayland | 17:53 |
kenvandine | i haven't trying anything qt/qml with wayland myself | 17:53 |
t1mp | kenvandine: I tried that. Still gives the same error. | 17:54 |
t1mp | I will add a comment to the issue | 17:54 |
kenvandine | greyback: ^^ do you have any ideas? | 17:54 |
kenvandine | greyback: https://github.com/ubuntu/snapcraft-desktop-helpers/issues/172#issuecomment-452388662 | 17:54 |
t1mp | greyback: was already helping me with it :) So he is aware. For me the problem is solved with the DISABLE_WAYLAND: 1. | 17:55 |
t1mp | That's enough for me, I have a working snap. But I guess it is still a workaround. | 17:55 |
kenvandine | that's not a solution though :) | 17:55 |
kenvandine | that makes it fall back to XWayland | 17:55 |
t1mp | true | 17:59 |
t1mp | but I need to release my snap this week :) | 17:59 |
kenvandine | yeah... so probably good enough :) | 18:03 |
kyrofa | Hey pedronis, you still around? | 18:06 |
greyback | kenvandine: hey, yes I looked into it. I noticed the glvnd config was missing - $SNAP/usr/share/glvdn/egl_vendor.d/50_mesa.json - (stracing the binary, it was not opening any DRI drivers). When I hacked that file into the snap, the app worked for me on my wayland. So I recommended adding "libegl0-mesa0" to the stage packages, but that didn't help t1mp | 18:08 |
kenvandine | hmm | 18:09 |
greyback | kenvandine: I didn't try further tbh | 18:09 |
kenvandine | understand | 18:09 |
=== ricab|lunch is now known as ricab | ||
popey | cjwatson: do we have an eta for core18 builds in LP via build.snapcraft.io ? | 20:45 |
cjwatson | popey: not yet | 20:47 |
diddledan | is that the eta is "not yet" or the eta on the eta is "not yet"? | 20:55 |
diddledan | kenvandine: I've done a lot of work yesterday and today on the gnome-extension POC that I created: https://github.com/snapcore/snapcraft/pull/2398 | 20:56 |
mup | PR snapcraft#2398: gnome extension <Created by diddledan> <https://github.com/snapcore/snapcraft/pull/2398> | 20:56 |
mup | PR snapd#6335 opened: NOT-REVIEW: tests: fix "No space left on device" issue on amazon-linux <Created by sergiocazzolato> <https://github.com/snapcore/snapd/pull/6335> | 21:07 |
kenvandine | diddledan: I noticed! | 21:09 |
diddledan | it finally works | 21:09 |
kenvandine | oSoMoN and I will look at it tomorrow | 21:09 |
diddledan | :-p | 21:09 |
kenvandine | Woot | 21:09 |
diddledan | I've done a test build locally with my gnome-twitch snap which successfully built the snap and when installed it runs correctly | 21:10 |
kenvandine | Excellent | 21:10 |
diddledan | including libwebkitgtk! | 21:10 |
diddledan | (I put a layout definition to fix that) | 21:11 |
kenvandine | diddledan: thanks for that | 21:11 |
diddledan | it also uses the gnome-3.xx-1x04 platforms | 21:12 |
diddledan | I couldn't publish my test build of gnome-twitch to the store, though, because it rejected the `command-chain:` block because it doesn't believe such a config exists :-) | 21:14 |
cjwatson | diddledan: yes | 22:17 |
diddledan | :-) | 22:18 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!