/srv/irclogs.ubuntu.com/2019/01/08/#snappy.txt

mborzeckimorning06:43
zygagood morning07:22
FaultsGoood morning!07:34
* zyga got a power plug adapter for south africa07:45
mupPR 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|afkmornings08:05
=== pstolowski|afk is now known as pstolowski
mborzeckizyga: pstolowski: hey08:07
=== JanC is now known as Guest41406
=== JanC_ is now known as JanC
t1mphello09:17
t1mpis this the place for questions relating to snap packaging problems?09:17
zygat1mp: hey, for various kinds of topics related to snaps09:17
t1mpI 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
t1mphttps://pastebin.ubuntu.com/p/whWXk2drjW/09:22
t1mpgreyback is already helping me with it :)09:22
t1mpfor reference, my snapcraft.yaml is here: https://pastebin.ubuntu.com/p/KN9qsVRdJ8/09:27
Chipacamorning all09:33
popeyGood day09:33
popeyt1mp: hey! don't you also need the 'wayland' plug?09:36
popeyI have seen some options where you use x11 and wayland plug but disable wayland in the environment which triggers it to use xwayland09:36
t1mppopey: hello :)09:37
t1mpI have no idea. Is there documentation about the wayland plug somewhere?09:37
t1mpI 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
mupPR 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
popeyhttps://forum.snapcraft.io/t/wayland-crash-with-electron-2/5752/4?u=popey09:38
popeyt1mp: theres a similar thread, where they use DISABLE_WAYLAND=1 in the environment to force xwayland09:39
popey(I *think* that's how it works)09:39
popeyThe documentation for the wayland interface is somewhat lean https://docs.snapcraft.io/the-wayland-interface/778409:39
popeyhttps://github.com/notepadqq/notepadqq/blob/master/snap/snapcraft.yaml#L1609:40
popeysomeone else using that method09:40
t1mpshouldn't those kind of things be included in the desktop-helpers?09:42
t1mpI could try the DISABLE_WAYLAND env var09:42
popeypossibly09:42
t1mpplugs are irrelevant as long as I'm using --devmode right?09:43
popeyyes09:43
popey(I think)09:43
popeyyou can use "snappy-debug.security scanlog" to learn what's failing apparmor09:44
popeyleave that running in another terminal when you run the app09:44
popeymight be enlightening09:44
zygabrb09:49
t1mpgreyback: 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
mupIssue core18#93 closed: i386 support missing on core18-amd64 <Created by mmtrt> <Closed by sil2100> <https://github.com/snapcore/core18/issue/93>09:59
t1mpThe missing images looks familiar. I think I had the same on X when I missed some files to have GLSL support10:02
zygaquick coffee to raise the pressure10:10
zygait's so sleepy today10:10
=== chihchun is now known as chihchun_afk
=== chihchun_afk is now known as chihchun
mupPR snapd#6333 opened: daemon: intrduce /v2/connections snapd API endpoint <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/6333>11:06
t1mpgreyback, popey: https://github.com/notepadqq/notepadqq/blob/master/snap/snapcraft.yaml#L16 made the snap work for me12:01
t1mpI don't know if it is the most elegant solution, but the package works now, which is what I need12:01
t1mpthanks for the help :)12:01
t1mpI still need to test it on some PCs though12:01
greybackt1mp: ok, glad you got it working12:02
greybackit appears wayland support still needs work in desktop-helpers12:02
popey(bug reports welcome) :D12:08
diddledanbug reports? https://www.youtube.com/watch?v=S-s4lEk91ng12:10
zygabrb again12:11
popeydiddledan: I'm doing my part!12:14
pstolowskimborzecki: hey, #6332 can land right?12:14
mupPR #6332: spread: make Fedora 29 auto again <Simple 😃> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/6332>12:14
mborzeckipstolowski: let me run the spread job again, sample of 2 is better than 1 ;)12:15
mupPR snapd#6318 closed: release-tools: display self-help <Simple 😃> <Created by zyga> <Merged by stolowski> <https://github.com/snapcore/snapd/pull/6318>12:18
pstolowskiok12:18
zygathanks pawel!12:21
zygarunning some spread tests, I will go for a walk with the dog12:21
zygait's snowing like crazy here12:21
zygaI don't want to miss this ;012:22
zyga:-)12:22
MattJWhere is $here? I want to come!12:22
MattJ(wait, that sounded creepy)12:22
zygait's all bound to melt away, it will be +2 soon12:23
zygabut for now it's close to -1 so the view is very nice12:23
zyga(in Warsaw)12:23
pedronismborzecki: hi, are you mostly blocked waiting on various reviews at this point?12:32
mborzeckipedronis: 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 land12:34
=== ricab is now known as ricab|lunch
pedronismborzecki: 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 standup12:35
mupPR 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.112:57
mupPR 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
mborzeckioff to pick up the kids13:03
=== chihchun is now known as chihchun_afk
t1mpgreyback: yeah. Is any OpenGL stuff not working with Wayland now? That seems quite important ;)13:15
t1mpor is it just Qt? Or something specific that I'm using13:15
greybackt1mp: it has worked for me in the past. I don't know what broke for you, sorry13:16
* Chipaca ⇝ lunch13:16
zygare13:26
zygat1mp: it probably depends13:26
t1mpokay, 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
pedronisChipaca: hi, could you review #6121 when you have a bit of time?13:56
mupPR #6121: tests: new test for snapshots with more than 1 user <Created by sergiocazzolato> <https://github.com/snapcore/snapd/pull/6121>13:56
Chipacapedronis: yes13:57
ChipacaI guess the current test that creates snapshots for root + testuser isn't enough?13:57
pedronisChipaca: I suppose, it's  question for cachio13:58
pedronismborzecki: btw, did you see the request for tweaks to #6317 ?13:59
mupPR #6317: overlord/snapstate/backend: call fontconfig helpers from the new 'current' <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/6317>13:59
mborzeckipedronis: yup, saw that, it's in my queue14:02
pedronisok14:02
mupBug #1810948 opened: Internal Watcher Error <Snappy:New> <https://launchpad.net/bugs/1810948>14:31
mupBug #1810948 changed: Internal Watcher Error <Snappy:Invalid> <https://launchpad.net/bugs/1810948>14:43
cachiozyga, https://travis-ci.org/snapcore/spread-cron/builds/476263240#L995215:08
zygalooking15:08
cachiothat should be fixed with the update in the repo?15:08
zygaso15:08
zygayou need to update & reboot15:08
zygalet me check that locally15:08
cachiozyga, I did that15:09
zygain that case the fix is not out yet15:09
zygalet's wait and see, I'll check the bug report in suse15:09
zygathanks for trying!15:09
cachionp15:09
=== chihchun_afk is now known as chihchun
* cachio lunch15:54
pedronisjdstrand: 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
mborzeckifeels like we should parse /etc/fonts/fonts.conf to figure out fontconfig locations rather than use hardcoded ones16:18
mborzeckifedora/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
zygamborzecki: how hard is it to parse?16:27
mborzeckizyga: rather easy, it's xml, encoding/xml should do just fine16:27
zygaoh, that's good16:27
zygais the location easy to extract?16:28
mborzeckizyga: yes, it's quite trivial :)16:29
mborzeckizyga: btw. wonder if we shuld also use ~/.local/share/fonts and ~/.fonts, we're only using system locations now16:30
zygait depends if the code runs in user contetxt16:30
mborzeckizyga: we setup those mounts in the desktop interface, i'd guess it does16:31
zygabut who would run the parser?16:31
mborzeckizyga: snapd could parse /etc/fonts/fonts.conf in destkop iface mount connected plug16:33
jdstrandpedronis: probably, yes. we don't do that elsewhere that I can see but that is no reason not to do it here16:36
jdstrandpedronis: I'll adjust16:36
zygamborzecki: can we discuss this tomorrow morning?16:44
zygaI'm adding some tests still16:44
jdstrandpedronis: done. fyi, I also updated snapd-control and kernel-module-control similarly. maybe that will reduce some questions16:44
pedronisjdstrand: thanks16:50
pedronispstolowski: I left a comment in the hotplug-connect PR, we can talk about it tomorrow16:53
pstolowskipedronis: ok, thanks16:54
mborzeckizyga: sure16:55
=== pstolowski is now known as pstolowski|afk
t1mpkenvandine: hello :)17:45
t1mpI just saw your comment on https://github.com/ubuntu/snapcraft-desktop-helpers/issues/17217:45
kenvandinehey t1mp17:46
t1mpthe way I understand it, plugs don't make a difference if the snap runs with --devmode (or as classic). Is that correct?17:46
kenvandineoh17:46
kenvandineright17:46
t1mphmm. I'm rebuilding my snap and (automatically, with jenkins) pushing it to the snap store, but I keep getting a 502 error17:46
t1mpSnap Store encountered an error while processing your request: bad gateway (code 502).17:47
t1mpThe operational status of the Snap Store can be checked at https://status.snapcraft.io/17:47
t1mpscript returned exit code 217:47
t1mpI can see the revision though when I log in on snapcraft.io17:49
t1mpah no, I cannot. It is a different revision number17:50
t1mp(just two digits swapped that's why I thought it is the same)17:50
kenvandinet1mp: try added QT_QPA_PLATFORM=wayland17:53
kenvandinei haven't  trying anything qt/qml with wayland myself17:53
t1mpkenvandine: I tried that. Still gives the same error.17:54
t1mpI will add a comment to the issue17:54
kenvandinegreyback: ^^ do you have any ideas?17:54
kenvandinegreyback: https://github.com/ubuntu/snapcraft-desktop-helpers/issues/172#issuecomment-45238866217:54
t1mpgreyback: was already helping me with it :) So he is aware. For me the problem is solved with the DISABLE_WAYLAND: 1.17:55
t1mpThat's enough for me, I have a working snap. But I guess it is still a workaround.17:55
kenvandinethat's not a solution though :)17:55
kenvandinethat makes it fall back to XWayland17:55
t1mptrue17:59
t1mpbut I need to release my snap this week :)17:59
kenvandineyeah... so probably good enough :)18:03
kyrofaHey pedronis, you still around?18:06
greybackkenvandine: 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 t1mp18:08
kenvandinehmm18:09
greybackkenvandine: I didn't try further tbh18:09
kenvandineunderstand18:09
=== ricab|lunch is now known as ricab
popeycjwatson: do we have an eta for core18 builds in LP via build.snapcraft.io ?20:45
cjwatsonpopey: not yet20:47
diddledanis that the eta is "not yet" or the eta on the eta is "not yet"?20:55
diddledankenvandine: I've done a lot of work yesterday and today on the gnome-extension POC that I created: https://github.com/snapcore/snapcraft/pull/239820:56
mupPR snapcraft#2398: gnome extension <Created by diddledan> <https://github.com/snapcore/snapcraft/pull/2398>20:56
mupPR 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
kenvandinediddledan: I noticed!21:09
diddledanit finally works21:09
kenvandineoSoMoN and I will look at it tomorrow21:09
diddledan:-p21:09
kenvandineWoot21:09
diddledanI've done a test build locally with my gnome-twitch snap which successfully built the snap and when installed it runs correctly21:10
kenvandineExcellent21:10
diddledanincluding libwebkitgtk!21:10
diddledan(I put a layout definition to fix that)21:11
kenvandinediddledan: thanks for that21:11
diddledanit also uses the gnome-3.xx-1x04 platforms21:12
diddledanI 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
cjwatsondiddledan: yes22:17
diddledan:-)22:18

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