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

mborzeckimorning05:55
mborzecki54 PRs, not bad at all05:57
=== mborzeck1 is now known as mborzecki
zygaGood morning07:32
mborzeckizyga: hey07:33
zygahttps://www.irccloud.com/pastebin/MMF5SCYo/07:40
mborzeckizyga: no snapd-lib?07:42
mborzeckior some path component07:42
mborzeckizyga: looks like glob wasn't expanded by the shell07:42
zygamborzecki: looks like teardown went south07:44
zygaanyway, that's just random noise today07:45
mvohey zyga  and mborzecki07:45
zygamvo: hey :)07:45
mborzeckimvo: hey07:45
zygamvo: ready to face the music?07:45
zygamborzecki: can you please review https://github.com/snapcore/snapd/pull/643107:46
mupPR #6431: snapcraft.yaml: fix snap building in launchpad <Simple 😃> <⚠ Critical> <Created by mvo5> <https://github.com/snapcore/snapd/pull/6431>07:46
zygamvo: updated https://github.com/snapcore/snapd/pull/6426 with a tiny comment tweak, I think it is ready now07:48
mupPR snapd#6431 closed: snapcraft.yaml: fix snap building in launchpad <Simple 😃> <⚠ Critical> <Created by mvo5> <Merged by bboozzoo> <https://github.com/snapcore/snapd/pull/6431>07:48
mupPR #6426: cmd/snap-confine: refactor and cleanup of seccomp loading <Created by zyga> <https://github.com/snapcore/snapd/pull/6426>07:48
zygamvo: also good news, https://tracker.debian.org/pkg/snapd will migrate tomorrow07:49
mvozyga: thank you07:53
mupPR snapd#6437 opened: snap/channel: improve channel parsing <Simple 😃> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/6437>07:56
zygamborzecki: school winter holidays started today07:58
zygamborzecki: two weeks with all the kids at home :)07:58
mborzeckizyga: nice, it starts on 11.02 here, right before the sprint, i'll probably take the week before sprint off07:58
zygabring them to malta :)07:59
zygamvo: is the plan still to review PRs more than open new ones?08:02
mvozyga: yeah, I think thats wise08:02
mupPR snapd#6428 closed: interfaces: add display-control interface <Created by jdstrand> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/6428>08:05
mupPR snapd#6435 closed: interfaces: add display-control interface - 2.37 <Created by jdstrand> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/6435>08:05
=== pstolowski|afk is now known as pstolowski
pstolowskimornings08:07
zygahello pawel, welcome back08:07
mborzeckipstolowski: hey08:09
mvohey pstolowski! welcome back08:10
mborzeckihm not that many reviews that aren't waiting for the author to address the feedback they got08:28
mborzeckii'll be looking into what's wrong with #6252 spread tests08:28
mupPR #6252: userd: handle help urls which requires prepending XDG_DATA_DIRS <Created by kenvandine> <https://github.com/snapcore/snapd/pull/6252>08:28
mvomborzecki: this one is a good one to look at, thank you08:33
mvomborzecki: I think the issue is that we need to either do more mocking or use less mocking :) I mean, we might look into using some real xdg helpers (less mocking)08:34
mvolooks like tests are unhappy right now in debian-9 (/me looks)08:36
=== mvo_ is now known as mvo
mvohm, so stretch no longer has golang-1.10 (just 1.11)08:47
* mvo scratches head08:48
zygamvo: with integration of debian packaging we cannot assume to build for older releases08:50
zygamvo: realistically we should target testing and sid08:50
zygaor really just sid perhaps08:50
zygaas soon as we fix lintian warnings we may start using standards version that is from the future from the perspective of debian-908:51
pedronispstolowski: hi, I did another pass on #6322 last week08:51
zygahey pedronis08:52
mupPR #6322: overlord/hookstate: apply pending transaction changes onto temporary configuration for snapctl get <Complex> <Created by stolowski> <https://github.com/snapcore/snapd/pull/6322>08:52
zygamvo: shall we sync with pedronis?08:52
mvozyga: yeah, when he has time this morning we should have a quick HO08:53
mupPR snapd#6438 opened: data/systemd: helper service to hold refreshes when waking up from suspend <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/6438>08:53
pedronismborzecki: did you see my small comment on #6016 ?08:55
mupPR #6016: snap/naming: move various name validation helpers to separate package <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/6016>08:55
pedronis*comments08:55
mborzeckipedronis: yes, pushing those soon08:56
mborzeckipedronis: i've pushed a PR to improve channel validation08:56
mborzeckipedronis: ah, i see you've already looked at it, thanks!08:57
pstolowskipedronis: hi! yes i saw that, thanks. btw, not sure if i was super clear when we talked last week, but 6106 is ready imo08:57
mborzeckimvo: zyga: is there a way to depend on golang-any 1.7+ and then use `go env` to discover things?09:00
zygapresumably, yes09:01
zygamvo: is the problem to add the correct go things to path?09:01
pedronismborzecki: yes, reviewed that, but it's red09:01
mborzeckipedronis: yeah, debian doesn't build atm, Dependency is not satisfiable: golang-any|golang-1.1009:02
pedronispstolowski: yes, you were clear,  just didn't get to it yet09:02
pstolowskipedronis: ah, sure, ok09:02
mvomborzecki: I need to look, not sure there is anything that references golang-1.11 in stretch09:03
mvomborzecki: let me log into a debug system09:03
mborzeckimvo: log from a job on master branch: https://api.travis-ci.org/v3/job/485270403/log.txt09:04
mvomborzecki: thanks! golang-any seems to be a confusing name, it seems to refer to gccgo ./09:04
mborzeckiw8, rly?09:04
mborzeckimvo: isn't that a golang-go or gccgo-go dependning on platform support?09:05
mvomborzecki: aha, no09:05
mvomborzecki: sorry, its just confusing on debian-9 because golang-1.11 comes via backports09:06
mvomborzecki: so golang-any has no idea about its existance09:06
pedroniszyga: mvo: I'm available to chat if needed09:09
mvomborzecki, zyga I'm looking into this right now09:14
zyga  thank you09:15
mborzeckimvo: need help?09:20
zygamborzecki: reading the suspend refresh postponing PR09:25
zygamborzecki: is there any way snapd could learn from systemd or from the system itself that the system was just resumed09:25
zygato do this internally?09:25
* zyga goes to make tea09:26
mborzeckizyga: yes, afaik dbus is the only way: https://www.freedesktop.org/wiki/Software/systemd/inhibit/09:27
mvomborzecki: should be fine, up for review now09:28
mborzeckizyga: a topic that we haven't looked into yet, is also inhibiting sleep during refresh09:28
mvomborzecki: might be overkill (the fix) though, alternatively we could simple do "apt install -y golang-1.11" and fix it again in ~1-2y when go changes09:28
mupPR snapd#6439 opened: tests: workaround missing go dependencies in debian-9 <Created by mvo5> <https://github.com/snapcore/snapd/pull/6439>09:28
mborzeckimvo: wow :P so cool you know about python apt bindings09:31
mvomborzecki: *cough* I do09:34
mborzeckimvo: failed on shellcheck09:37
mvomborzecki: meh, silly me - let me check09:38
pedronismborzecki: zyga: I'm -1  on using a service to set refresh.hold from outside09:39
mupPR snapd#6438 closed: data/systemd: helper service to hold refreshes when waking up from suspend <Created by bboozzoo> <Closed by pedronis> <https://github.com/snapcore/snapd/pull/6438>09:39
mborzeckipedronis: should have added RFC there, i think we can discuss it during the sprint09:41
pedronismborzecki: it's ok, I appreciate having prototyped this, but we also don't need more open forever rfc PRs09:42
Chipacamvo: do you know if there's a reason private-files can't be used yet?09:46
mvoChipaca: that should work if 2.37 is installed09:47
Chipacamvo: store says no09:47
mvoChipaca: its very strict though, deny auto connect and connect09:47
mvoChipaca: manual review?09:47
mvoChipaca: or no in more ways?09:47
Chipacamvo: ah, maybe09:48
pedronisChipaca: you need a snap declaration, otherwise it will not even install (unless --dangerous)09:48
Chipacaright, but I need it in the store to ask for that09:48
Chipacabut you're right, it's up09:48
Chipacathe error from snapcraft sounded too much like an outright rejection :-)09:48
Chipacapstolowski: welcome back!09:49
pstolowskihey Chipaca!09:49
Chipacapstolowski: thank you for the review09:49
pstolowskinp09:50
mupPR snapd#6414 closed: daemon: try to tidy up the icon stuff a little <Simple 😃> <Created by chipaca> <Merged by chipaca> <https://github.com/snapcore/snapd/pull/6414>09:51
pedronisChipaca: hi, should we have a HO about epochs last PR ?09:54
Chipacapedronis: I'm about to have a meet about coh*09:56
pedronisChipaca: ah, do you need me?09:57
Chipacapedronis: not necessarily, but you're welcome if you're curious09:58
zygaChipaca: coh?09:59
pedroniszyga: cohorts and coherence09:59
Chipacapedronis: it's more for me to sync with sparkiegeek and us to get a shared idea of timelines than anything else09:59
zygaahh09:59
Chipacazyga: carbon-oxygen-hydrogen compounds09:59
mvoChipaca: hm, we talked recently about snap info foo.snap not returning the arch of the snap. what was the conclusion/workaround, do you remember?10:01
Chipacamvo: will be added once info is nicer10:06
mvoChipaca: ok, no worries, I need this too for something but its easy to workaround10:07
mborzeckithere's some snap instance name validation weirdness, it's happening super late for installing from store now, so you can actually attempt to install hello-world_123__a and it fails as late as setting up mounts10:13
mupPR snapd#6440 opened: Fix typo in cmd_wait.go <Created by sparkiegeek> <https://github.com/snapcore/snapd/pull/6440>10:27
zygabrb, need to pick up something10:29
Chipacapedronis, mvo: wrt personal-files, https://forum.snapcraft.io/t/request-for-automatic-alias-and-connection-for-icdiff/9703 (weekend was fun)10:32
Chipacadon't tell sergio but that'll be my first snap to use snapcraft10:32
sergiusensChipaca: good for you to have joined the parade 🙂10:36
Chipacasergiusens: btw is there a way to build using 3.0 without bases?10:37
Chipacamvo: pedronis: OTOH, is it on purpose that core can't be used as an explicit base?10:37
sergiusensno, and the latter about core, I requested that 6 months ago 🙂10:38
sergiusensbut `base: core` is illegal for snapd10:39
pedronisChipaca: yes, core is not a base10:40
Chipacasergiusens: pedronis: I guess #6418 makes it moot10:40
mupPR #6418: many: allow core as a fallback for core16  <Created by mvo5> <https://github.com/snapcore/snapd/pull/6418>10:40
sergiusensChipaca: not entirely, as now I need to go into snapcraft and workaround the fact that when someone says `base: core16`, to say that `core` satisfies that requirement.10:42
Chipacasergiusens: BTW², snapcraft doesn't understand license yet?10:42
ChipacaI thought that was Done10:42
sergiusensit should10:42
mupPR snapd#6441 opened: overlord/snapstate: validate instance names early <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/6441>10:43
Chipacausing 3.0.1, using a base, had to put it in passthrough10:43
sergiusensthere's even integration tests enabled for that10:43
sergiusenshmm10:43
mborzeckipedronis: you might be interested in 644110:44
popeyniemeyer: Would you consider adding a robots.txt to snapdocs.labix.org? It's coming up in google search results, and I don't think that's useful.10:53
popeycurrently looking for robots.txt on the site redirects to the home page.10:56
niemeyerpopey: I'm thinking of killing it actually11:02
niemeyerpopey: The new docs site is already out11:03
popeyAwesome, even better :D11:04
* Chipaca afk for a while11:31
* Chipaca bok11:49
Chipacabak?11:49
pstolowskimvo: a question on 640411:50
mvopstolowski: thanks, I check in a sec11:50
mborzeckimvo: looks like we need to manipulate PATH in run-checks too11:51
mborzeckimvo: that's regarding #643911:51
mupPR #6439: tests: workaround missing go dependencies in debian-9 <Created by mvo5> <https://github.com/snapcore/snapd/pull/6439>11:51
zygamborzecki:  can you try fixing that?11:55
mborzeckizyga: yeah, in a couple of minutes, got some changes in snap connections i need to wrap up11:57
pedronisChipaca: yes, 6389 looked like my suggestion, it needs a 2nd review12:19
Chipacapedronis: ack tks12:28
ackkhi, I was trying snap parallel installs on snaps and noticed some issues with hook not being found. is that something that's known?12:29
mborzeckiackk: we've addressed an issue regarding hooks and parallel installs last week or so, let me see12:36
mborzeckiackk: what is the problem that you see?12:36
ackkmborzecki, snapd says it can't find the hook12:37
ackkmborzecki, snap install --edge h2static_test12:37
=== ricab is now known as ricab|lunch
ackkmborzecki, ok that's just weird because it now worked12:43
ackktried it different times and didn't seem to12:44
pedronisChipaca: can you look again at #6333 to consider the last changes12:53
mupPR #6333: daemon: introduce /v2/connections snapd API endpoint <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/6333>12:53
Chipacapedronis: in a bit13:00
mborzeckimvo: i'll look into #6439 if you don't mind13:13
mupPR #6439: tests: workaround missing go dependencies in debian-9 <Created by mvo5> <https://github.com/snapcore/snapd/pull/6439>13:13
mupPR snapcraft#2452 closed: Fix typo in comments <Created by Lin-Buo-Ren> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/2452>13:16
mupPR snapcraft#2450 closed: pluginhandler: handle removal of inconsistent files <Created by cmatsuoka> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/2450>13:19
mvomborzecki: sure13:39
mvomborzecki: its actually probably just a sed / on debina/control13:40
mvomborzecki: like "sed -i s/golang-1.10/$best_golang/ debian/control"13:40
mborzeckimvo: yeah, and also need a correct symlink to /usr/bin/go13:40
mvomborzecki: indeed13:42
mvomborzecki: hopefully just ln -s /usr/lib/$best_golang/bni/go /usr/bin/go - should be all pretty uniform13:42
mborzeckimvo: actually it's /usr/lib/go-<ver>/bin/go :P13:43
mborzeckis/lang//13:43
mvomborzecki: meh, ok13:47
mvomborzecki: thanks for looking into it13:47
pedronisChipaca: there's a couple of nitpicks by mvo for 6411 that seems worth considering13:52
Son_Gokumborzecki, so I've proposed this change to fedora-release: https://src.fedoraproject.org/rpms/fedora-release/pull-request/6713:52
Chipacayeah13:52
mborzeckiSon_Goku: looking13:52
zygaSon_Goku: thank you, that's nice :)13:53
Son_Gokuthe only thing is that I'm not sure if I should call the variant snappy, similar to Ubuntu Core13:53
Son_Gokusnap sounds... dumb13:53
Son_Gokubut snappy isn't that much better13:53
mborzeckiSon_Goku: nice13:54
mborzeckiSon_Goku: so the -snap variant would be included in fedora base then?13:54
Son_Gokuyes13:54
Son_Gokuthis will also give us the key we need to make snapd aware of Fedora Snappy variant too13:55
zygaSon_Goku: perhaps ask pedronis for naming opinions13:55
Son_Gokubecause the VARIANT_ID will allow us to identify a bootable Fedora Snappy variant13:55
Son_Gokupedronis, should the variant for the base snap os-release be "Snap" or "Snappy?13:56
zygaSon_Goku: I think that's not required, snapd doesn't decide if a base snap is bootable based on  this AFAIK13:56
Son_Goku*"Snappy"?13:56
zygabut having a variant is good IMO13:56
zygaSon_Goku: my preference for the VARIANT_ID is lowercase word13:56
mborzeckiSon_Goku: btw. does bluesilver have VARIANT_ID=flatpak?13:56
Son_Gokuzyga, there's a couple of os-release detection behaviors for handling normal systems and snappy-based systems13:56
zygaVARIANT can be anything nice13:56
Son_Gokumborzecki: silverblue is variant silverblue13:57
zygaSon_Goku: yes but not for bootable vs non-bootable bases13:57
mborzeckiSon_Goku: ah ok13:57
Son_Gokuunless someone here has an interesting marketing name for a snappy variant of Fedora, that's all I got13:57
Son_Gokuzyga: VARIANT_ID is lowercase13:57
=== ricab|lunch is now known as ricab
mborzeckimvo: if 18.04 gets go-1.11 will we continue building with 1.10?13:59
ChipacaSon_Goku: zyga: we distinguish current ubuntu core from the pre-snappy ubuntu core by calling the former "snappy ubuntu core"13:59
Chipacawe might need to check about whether the use of the brand is ok?13:59
Chipacais it a brand?13:59
* Chipaca has no idea14:00
Son_GokuI think the brand is Snapcraft?14:00
Son_Gokuwhich I've deliberately avoided using14:00
ChipacaI'm sure somebody will have Opinions14:00
* Son_Goku groans14:00
Chipacashould I ask?14:00
Son_Gokuas much as it pains me, we do need to know the answer to this14:01
Son_GokuI need a name for the variant14:01
Chipacaa'ight14:01
Son_GokuI think I'd prefer to use "snappy" rather than "snap", but I dunno14:01
mborzeckizyga: standup?14:02
zygajoining14:02
cwayneHow do they not go with fed-core-a14:09
* cwayne will see myself out14:09
zygajamesh: hell14:38
zygahello :)14:39
zygasorry, typo14:39
=== ricab is now known as ricab|brb
=== ricab|brb is now known as ricab
mupIssue core18#56 closed: Sensible version number <Created by mvo5> <https://github.com/snapcore/core18/issue/56>15:22
mupIssue core18#86 closed: Ubuntu 18.10 <Created by kravietz> <https://github.com/snapcore/core18/issue/86>15:22
mupIssue core18#89 closed: bash completion not installed if "core" snap is not installed <Created by albertodonato> <https://github.com/snapcore/core18/issue/89>15:22
mupPR # closed: core18#43, core18#63, core18#72, core18#90, core18#98, core18#11315:22
mupIssue core18#56 opened: Sensible version number <Created by mvo5> <https://github.com/snapcore/core18/issue/56>15:23
mupIssue core18#86 opened: Ubuntu 18.10 <Created by kravietz> <https://github.com/snapcore/core18/issue/86>15:23
mupIssue core18#89 opened: bash completion not installed if "core" snap is not installed <Created by albertodonato> <https://github.com/snapcore/core18/issue/89>15:23
mupPR # opened: core18#43, core18#63, core18#72, core18#90, core18#98, core18#11315:23
popeyare layouts still behind an experimental feature flag?15:30
popey(yes, it seems, as snap install refuses to install my snap which uses them)15:52
ogradid you try with 2.37 ? iirc zyga said it was a 2.37 feature15:57
zygapopey: no15:57
zygapopey: they are available now15:57
zygabut there are weird conditions on snapcraft side15:57
zygayou must use bases15:57
Chipacaalso, maybe you explicitly turned them off?15:58
Chipacawhat does 'snap get system experimental' say15:58
Chipacapopey: ^15:58
popeyah, i wasn't using bases16:06
popeyerror: snap "core" has no "experimental" configuration option16:07
sergiusenszyga: popey bases should not be required on snapcraft for a snap to work during runtime, passthrough will just allow it into `snap.yaml` and let by gones be by gones16:46
mvosergiusens: hey, the autopkgtest results for 2.37 have some failures in the  integration test for plainbox, I will give you a pastebin soon. do you know anything about those?16:49
sergiusensmvo: autopkgtests using the deb I suspect, something happened along the release line for plainbox that broke the test, we have not SRUed since, so should be safe to ignore, except, maybe you want to consult with cwayne just in case.16:50
mvosergiusens: https://paste.ubuntu.com/p/GGm3mqvGgY/ (cc cwayne )16:51
mvocwayne: is the above plainbox/checkbox failure a reason for concern? just ensureing that the 2.37 release we want to push out soon (later tonight or tomorrow morning) can also be pushed to -updates16:52
cwayne kissiel ^16:52
zygaFYI: core snap with snapd 2.37 has been promoted to stable just now17:06
popeyTime to update the wikipedia page then! :D https://en.wikipedia.org/wiki/Snappy_(package_manager)17:07
zygalet me do that17:07
popeyhuh17:07
popeysomeone broke the version part and refers to snapcraft only and not snapd17:08
zygahmm17:08
zyga3.0.1?17:08
sil2100mvo, zyga, pedronis: hey! Do we have an official model assertion in the store for pi3 arm64? Would be nice to have one if we don't17:08
zygasil2100: I don't know, would love one if we did17:08
sil2100The snap is in the store since a while (with a new one in the review queue)17:08
popeyyeah, they mistakenly changed the snapd to snapcraft versaion17:08
sil2100mvo, zyga, pedronis: if we could get the assertion in, I could enable pi3 arm64 core18 as an official image we build and get that out for the .2 point release17:09
zygaah17:09
zygathere's a confusion between snapd and snapcraft there :(17:09
popeywhole page needs a rewrite17:10
popeyI see someone re-added the critisism from the peek developer back in after it was removed17:10
pedronissil2100: we don't, we need to go through the process the get one signed17:10
pedronissil2100: that's something foundation drives17:10
pedronisnow17:10
pedronisI think17:11
* mvo afk17:11
sil2100I could drive it if someone could point me to how! Didn't do that before, can you link me some docs?17:11
pedronisor maybe mvo did it, but I don't think it should be mvo17:11
sil2100Since I think none of the current core18 assertions have been done by Foundations17:12
pedronissil2100: they weren't but should I think17:13
pedronislike you do fro cloud images17:13
pedronisthey are connected to the images17:13
pedronissil2100: let me forward you one of the relevant RTs17:14
sil2100pedronis: thank you!17:14
sil2100:)17:14
pedronissil2100: you also need a name,   ubuntu-core-18-pi3  is the armhf ?17:15
pedronishow do you plan to call the arm64 ?17:15
sil2100I need to think about it still, not sure who invented the previous naming, I'd have to look at the other standards - first thing that comes into my mind is simply ubuntu-core-18-pi3-arm64, just like we had pc-amd64 and pc-i386 for 1617:16
sil2100(now we just have -i386 and -amd64 but yeah)17:16
pedronissil2100: forwarded17:16
sil2100pedronis: thanks again o/17:17
zygasil2100: wrt docs, you should be in sync with degville17:23
=== pstolowski is now known as pstolowski|afk
xnoxis my lxd busted, or my snapd busted?17:41
xnox$ lxc list17:41
xnoxcannot read mount namespace identifier of pid 1: Permission denied17:41
popeyxnox: what if you "snap disable lxd && snap enable lxd"?17:45
ondraxnox in other words, have you tried to switch it off and back on? :D17:46
xnoxpopey, https://paste.ubuntu.com/p/N5yvH9CDyr/17:46
xnoxondra, still bad https://paste.ubuntu.com/p/N5yvH9CDyr/17:46
xnoxbut is it like lxd bad or snapd bad? =) is there anything else i can try to see if /all/ of snaps are broken, or just this one....17:47
ondraxnox I was more laughing we are at that stage already when we try to remedy things by restarting them :)17:47
xnox$ /snap/bin/gnome-calculator17:48
xnoxcannot read mount namespace identifier of pid 1: Permission denied17:48
xnoxok, i claim snapd is busted for me.17:48
xnoxof all the things, gnome-calculator should work.17:48
ondrasergiusens here is bug we talked last week https://bugs.launchpad.net/snapcraft/+bug/181363417:50
mupBug #1813634: not reliable way to detect target architecture <Snapcraft:New> <https://launchpad.net/bugs/1813634>17:50
ondrasergiusens seems like SNAPCRAFT_ARCH_TRIPLET is also busted when cross compiling using build-on: run-on definition17:51
xnoxOdd_Bloke, popey - sudo apt install --reinstall snapd => helped!17:51
sergiusensondra: could you create a bug for that last one too?17:52
ondrasergiusens and new question I have, is maven supporting core18? I have change base snap in my snapcraft.yaml and my build is now failing17:52
sergiusensondra: yes, look at the 3.0 release notes17:53
sergiusensondra: https://github.com/snapcore/snapcraft/releases/tag/3.017:53
ondrasergiusens I added both (SNAPCRAFT_ARCH_TRIPLET and project.target_arch) to same bug, it seems related17:53
ondrasergiusens unless you tell me to split it17:53
sergiusensok17:54
ondrasergiusens with snapcraft 3.0.1 I'm getting maven.py", line 263, in _create_symlinks IndexError: list index out of range17:54
ondrasergiusens and only thing I changed was base snap17:54
sergiusenscross compilation are low priority bugs, btw17:54
sergiusensondra: please log a bug17:54
sergiusensor submit to sentry17:54
ondrasergiusens I did already uploaded traceback17:55
sergiusensok17:55
ondrasergiusens same happens with edge and candidate17:55
ondrasergiusens BTW, this line is funny "if self.project.info.base not in ("core18", "core19"):"17:55
sergiusenshmm, that is a bug17:56
ondrasergiusens I know core18 made it out in 2019, but did not know we actually have core19 now :D17:56
sergiusensshould be core16... not tests in place as core16 did not really exist then17:56
ondrasergiusens same in ant, maven gradle :)17:56
sergiusensondra: I will need your snapcraft.yaml for the maven issue18:00
ondrasergiusens https://bugs.launchpad.net/snapcraft/+bug/181363618:02
mupBug #1813636: ant maven and gradle plugins referene mysterious core19 <Snapcraft:New> <https://launchpad.net/bugs/1813636>18:02
ondrasergiusens OK let me share it18:02
ondrasergiusens I will add it to bug report18:02
ondrasergiusens https://bugs.launchpad.net/snapcraft/+bug/181363718:06
mupBug #1813637: maven plugin crashes with base core18 <Snapcraft:New> <https://launchpad.net/bugs/1813637>18:06
ondrasergiusens I will clean whole snapcraft and push it to git18:06
Odd_Blokexnox: Glad to hear it.18:50
sergiusensondra: try setting `maven-openjdk-version: "11"` for the part18:57
diddledanI think the code should be slightly smarter by checking that there's at least one result from glob() before plucking the first element19:01
diddledanthe error is here: https://www.irccloud.com/pastebin/7vs1guXj/19:01
mupPR snapcraft#2453 opened: ant, maven and gradle plugins: use correct defaults for jre <Created by sergiusens> <https://github.com/snapcore/snapcraft/pull/2453>19:44
sergiusensondra: ^19:44
sergiusensdiddledan: it is a problem in itself that we failed to find what we wanted in our controlled environment though, that has been addressed ^19:46
diddledanaha19:47
mupPR snapd#6439 closed: tests: workaround missing go dependencies in debian-9 <âš  Critical> <Created by mvo5> <Merged by chipaca> <https://github.com/snapcore/snapd/pull/6439>20:31
mvocwayne, kissiel let sync about https://paste.ubuntu.com/p/GGm3mqvGgY/ in my tomorrow, my feeling is it should be fine but I feel better if one of you could double check :)20:46
kissielmvo: ack21:03
mvokissiel: thanks - so I take it you also need to investigate this first, right?21:17
mvokissiel: I gtg, lets sync tomorrow21:21
* mvo waves21:21
ondrasergiusens I will try maven-openjdk-version: "11"23:20
ondrasergiusens also openjdk-11 does not have jre subdir when I was checking code originally, if that will have impact on that code23:21
FrogCasthey your documentation is not working here: https://tutorials.ubuntu.com/tutorial/basic-snap-usage#223:26
FrogCast snapd find hello23:26
FrogCastzsh: command not found: snapd23:26
FrogCastit should be corrected to "snap"23:26
ondrasergiusens adding maven-openjdk-version: "11" did not help23:29
mupPR snapd#6411 closed: cmd/snap: wrap "summary" better <Squash-merge> <Created by chipaca> <Merged by chipaca> <https://github.com/snapcore/snapd/pull/6411>23:35
cjwatsonFrogCast: I don't see "snapd find hello" on that page - only "snap find hello".23:54
cjwatson(And that page hasn't been touched in the git repository that hosts it for over a year)23:55
FrogCasto_O23:55
FrogCastI thought I copied and pasted23:55
FrogCastcjwatson: guess my mistake. Sorry for that23:55
cjwatsonNo problem23:55
cjwatsonI was all set to propose a quick PR to fix it ;-)23:55

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