[03:03] elopio, ping === chihchun_afk is now known as chihchun [08:40] good morning [08:56] morning [09:20] morning [09:37] Good morning all! Happy Tuesday, and happy Australia Day! 😃 [11:02] kgunn: hey, if I can help with updating your mir snap, please do let me know [11:53] mvo: do i name the vmlinuz just like that in a snap or shall i append the -KERNELVERSION? [11:53] same for system.map etcv. [12:07] asac: sorry, I disconnected, it has to be named vmlinux and initird.img for now, but that is a bug [12:09] I've defined a git URL in "parts: pulseaudio: source:", but when I run "snapcraft pull", it does not seem to download anything. Help? (I'm using 16.04 version of snapcraft.) [12:14] mvo: vmlinux or vmlinuz? [12:15] asac: vmlinuz [12:15] asac: sorry [12:23] mvo: can you look into this one: https://people.canonical.com/~asac/kernel-snap-example_0_amd64.snap [12:23] and tell me beyond not appending the version is missing? [12:23] (ignore the odd initrd name) [12:24] asac: sure, I check after lunch [12:30] found it [12:56] asac: from a quick look it looks great, I can give it a test in a vm next. you can use symlinks for the vmlinuz, its just needed right now so that grub can find it === chihchun is now known as chihchun_afk [13:35] kyrofa: thanks for the reviews [13:36] (https://github.com/ubuntu-core/snapcraft/pull/256 has the fix you were asking for) [13:37] dholbach, thanks for the fixes :) [13:39] dholbach, for what it's worth, snappy logs . doesn't seem to work for me on 15.04. Is that a 16.04 thing? [13:39] https://github.com/ubuntu-core/snapcraft/pull/242 is updated now too [13:39] kyrofa: this was part of the 15.04 appdev manual - let me check [13:40] dholbach, yeah sanity check me there. I may be being stupid [13:45] dholbach, https://github.com/ubuntu-core/snapcraft/pull/261 is a good-looking doc, though I wonder if it actually belongs in snappy instead of snapcraft. Thoughts? [13:45] kyrofa: what works for me in rolling/edge is: sudo snappy service logs shout (for shout.sergiusens) [13:46] let me check for 15.04 [13:46] dholbach, ah indeed-- but no .servicename eh? [13:46] dholbach, that works on 15.04 as well [13:47] kyrofa: I thought about it too, but thought that if we split up our docs into "what's interesting for app developers?" and "what's general/structural information?" and "what does a device builder need to know?" ... I feel it belongs in the app context [13:47] kyrofa: yep - looks like it... ok, let me fix that [13:47] dholbach, good deal, you're the pro there [13:47] kyrofa: but if we were to move the doc I guess that'd be fine with me too [13:47] it's just how I tried to explain the situation to myself ;-) [13:48] dholbach, heh, I really just want what is most clear to developers. And you're right-- they're probably using snapcraft to make the .snap, so why not have general .snap development guides there? [13:48] dholbach, of course, once you have the syncing all done to developer.ubuntu.com, it matters less [13:48] if we want, we can still go and split it up or move it elsewhere - I'm happy to do that if we have concensus [13:49] dholbach, no I'm happy with that :) [13:49] cool [13:50] awe_, so I was wondering about the autotools plugin - two questions: [13:50] dholbach, snappy-debug has been around for a while now. Any idea when it'll actually include gdb, strace, ltrace, etc.? [13:51] jdstrand: ^ do you know if this coming soon or who might be working on it? [13:51] awe_, 1) I have build headers installed on my local development machines, but not all dev headers are (yet) in "stage-packages". [13:52] awe_, this seems to have the surprising result that ./configure detects them but gcc does not? [13:52] hmmm, that sounds odd; probably something wrong in your Makefile.in [13:52] diwic, dev headers don't need to be in stage-packages [13:53] kyrofa, where should they be, then? [13:53] kyrofa, build-packages instead? [13:53] awe_, indeed. diwic snapcraft will use your system-installed packages for building, but you need to make sure stage-packages sets it up for run-time [13:53] kyrofa, nobody's ever truly explained the diff between the two. I use build-packages for build tools ( eg. pkg-config ) and stage-packages for dev-headers [13:54] and I also build in a lxc container [13:54] that I can reset to it's initial state [13:54] so that I know my dependencies are correct [13:54] kyrofa: thanks for the reviews - I'll check if there's any other new content to add to snapcraft :-) [13:54] diwic, so put mylib-dev in build-packages, and mylib in stage-packages [13:55] kyrofa, ok [13:55] dholbach, I'm loving the docs! [13:55] diwic, build-packages install on the host (and don't go into the .snap), stage-packages go into the .snap [13:55] kyrofa, but if you put -dev in stage-packages, it automatically pulls in the lib, and then you can just not snap the dev headers [13:55] 2) whenever you add or remove anything in stage- or build-packages, you have to manually remove parts/pulseaudio/state for the changes to take effect. Is this expected? [13:56] stage-packages only go in the snap if you tell them to... [13:56] diwic, did you try using snapcraft clean? [13:56] awe_, hmm... stage-packages go into the snap unless you filter them out somehow with the stage/snap keywords [13:56] instead? [13:56] kyrofa, which I'm currently doing... [13:57] so I guess it's a style thing [13:57] awe_, I suppose :P [13:57] but your approach makes sense [13:58] awe_, *testing* hmm, that works too, but that re-clones the git repo too [13:58] I'm not sure if there's a way to clean the build w/out clobbering the source(s) [13:59] seems like you figured out a way to cheat by manually removing the file [14:00] I mean, there is code to check if there's a git repo and if so update it (rather than wipe and clone), when is that code supposed to be run? [14:01] diwic, I'm not sure... you could check the snapcraft source directly; it's all-python [14:01] awe_, already hacked it ;-) [14:01] ;D [14:02] awe_, the git server I'm working against (just as a test) didn't support --depth=1 so I had to remove that [14:02] diwic, I've never heard of that. What software is running on the server? [14:05] kyrofa, no idea - git://anonscm.debian.org/pkg-pulseaudio/pulseaudio.git is the URL I'm currently testing with [14:05] (again - might change that to something else later, this is just to get started) [14:07] dholbach: I don't, no. mvo may have more info [14:07] mvo: ^ kyrofa asked earlier: snappy-debug has been around for a while now. Any idea when it'll actually include gdb, strace, ltrace, etc.? [14:07] thanks jdstrand [14:09] awe_, 3) I was also wondering about https://github.com/ubuntu-core/snapcraft/blob/master/snapcraft/plugins/autotools.py#L85 - it ends with '--prefix=', looks weird [14:09] awe_, what is the prefix set to and how? [14:09] diwic, huh, looks like other people have hit that as well, and no one seems to know why. Perhaps you should use a snapshot .tar.gz from gitweb [14:10] diwic, https://anonscm.debian.org/gitweb/?p=pkg-pulseaudio/pulseaudio.git [14:11] diwic, I guess it depends on how self.options.configflags is initialized [14:11] I'll check when I do my next build [14:11] I'm in the middle of a change right now [14:11] sure [14:11] that said, my builds were working fine last night [14:12] diwic, --prefix= means "no prefix". You can still set the prefix in the configflags though [14:12] kyrofa, ok [14:13] diwic, autotools takes the last one [14:13] guess that explain that; thanks kyrofa [14:14] awe_, hey, any time :) [14:14] kyrofa, anyhow, so now I need to add --disable-A, --disable-B, --disable-C etc in configflags in order for it to skip that dependency (otherwise it'll find it as it is installed on the system) [14:15] are you asking how to do this? [14:15] configflags: [14:16] diwic, yeah, should be easy with configflags [14:16] awe_, I'm asking whether this is actually the desired behaviour? It seems less error prone if the part being built did *not* find system-installed headers [14:16] http://pastebin.ubuntu.com/14671891/ [14:17] awe_, and it seems even stranger that configure finds them but not gcc [14:17] well.. you are running configure right? [14:17] again, that's probably because the Makefile.in is wrong [14:17] gcc only finds what you tell it [14:18] awe_, you have to excuse me for thinking the problem is snappy rather than Makefile.in [14:18] diwic, it's new, and it's changing rapidly. It *might* be a snapcraft thing, but off the top of my head, it sounds more like a Makefile.in thing [14:19] awe_, based on the fact that snappy is very new and Makefile.in/pulseaudio is several years old, builds on many architectures and autogenerated by autotools [14:19] sure, but you'll need to dig in and see what's really happening [14:19] awe_, yeah [14:19] if it is a snapcraft bug, then please report it! ;)- [14:19] I noticed yesterday that snapcraft --help seems to be broken in 2.0 [14:20] * awe_ needs to check if that's already been filed [14:20] awe_ can you define "broken"? [14:21] it generates a stacktrace [14:21] awe_, *cough* sound broken [14:21] http://pastebin.ubuntu.com/14671916/ [14:22] Ah, wait I think I read about that on the mailing list. Due to a lack of locale setting or something [14:23] Definitely a bug that it's not handled in snapcraft, but you can work around it by setting your locale [14:23] awe_, ^^ [14:24] kyrofa, thanks. It's more of annoyance than anything else [14:24] but I'll give that a try [14:24] if/when I next need to refer to the help [14:24] been grokin' the source instead [14:25] awe_ are you running out of master? Or using 2.0.1 on xenial? [14:25] the latter [14:28] kyrofa: I bet it's the “ and ” in the help text [14:28] kyrofa, is it possible to add a multi-line description in snapcraft.yaml? This seems to cause grief whenever I try to do so [14:28] changing them to " should probably make it work [14:29] awe_ indeed, use YAML-foo! [14:29] dholbach, brilliant! I'm trying to figure out how to UNset my locale to test :P [14:29] ? [14:29] awe_, heh, hold on [14:29] awe_ https://stackoverflow.com/questions/3790454/in-yaml-how-do-i-break-a-string-over-multiple-lines [14:29] awe_ does that help? [14:30] awe_ it's just YAML, so anything YAML supports, snapcraft supports [14:30] so i just moved to snapcraft 2.0 [14:30] yes, of course it appears there are nine different ways to do it! groan [14:30] welcome to the party kgunn! [14:30] i used to be able to muck with my snap (fka apps) files [14:30] awe_ hahaha [14:30] but now i can't [14:30] * kgunn looks for disco ball [14:31] anyone know of a trick ? [14:31] kgunn, I don't quite understand [14:31] kgunn, can you define "muck"? :P [14:31] kyrofa: so i have a script a-la /snaps/app.sideload/bin/dostuff [14:31] i used to just stop my service [14:31] tweak it [14:32] but...it just refuses to become writable [14:32] ah [14:32] probably the change in file format of the snaps [14:32] kgunn, yeah squashfs [14:32] yeah figure [14:32] but is there a trick to circumvent [14:33] yea, but you need some squashfs foo, to be able to do so.. I looked at this yesterday, and decided it wasn't worth the trouble [14:33] lol [14:33] kgunn, hmm.... not that I know of I'm afraid [14:33] if you figure it out, please share the wealth [14:34] sure :) [14:34] kgunn, I see guides for layering unionfs on top of squashfs to add write capabilities, which to me means squashfs just doesn't support writing [14:35] ack [14:35] elopio, agh, sorry I'm late, got distracted [14:39] dholbach: no really good idea at this point, hopefully soon when the store lands support for per-architecture snap [14:39] kyrofa, seems on the "flow" style line wrapping methods are supported which strip any embedded new lines... [14:39] dholbach: snaps [14:39] dholbach: right now it would have to be a pretty big snap that is a bit tricky to build [14:45] fgimenez: ping. Could you ssh? [14:58] kyrofa: ^ see what mvo said? [15:00] dholbach, yeah, good info. I'm a little worried a developer might miss the "snappy-debug only includes security right now" note and wonder why he can't use strace after reading the docs [15:01] dholbach, what do you think? [15:03] mvo: is there the notion of a type: in snap.yamnl still? [15:04] e.g. should snapcraft learn about type: kernel? etc.? [15:04] asac: yes [15:05] mvo: did you take a look at the .sanp? i uploaded a new one [15:06] kyrofa: ok, let me change it a little bit [15:06] kyrofa: does snapcraft already allow me to use type: kernel ? e.g. in the yaml? [15:07] ../../bin/snapcraft clean [15:07] Issues while validating snapcraft.yaml: 'kernel' is not one of ['app'] [15:07] ok seems not [15:07] asac: should I test it? [15:07] * asac checks and adds [15:07] mvo: well, the initrd has modules only [15:07] beyond that is there anything missing compared from what you added? [15:08] asac: I think its fine, won't work right now because we don't have the generic-initrd yet, but that is something someone needs to work on soon [15:09] asac, yeah I don't think so yet [15:09] mvo: lool is experimenting with how to concat those initrd [15:09] in the bootloaders [15:10] kyrofa: ok... i will incolude that in my mammoth patchset :) [15:10] was just a scheme patch [15:10] asac: cool [15:10] asac: accoridng to apw its really just "cat initird1 initird2> initrd.img" and the kernel will DTRT [15:11] lool: will you work on getting an indep initrd into OS snap from our builders? [15:11] yeah [15:11] but we prefer to not concat on disk [15:11] but rather from bootloader [15:11] e.g. have them load both files and just boot [15:11] asac: if the boorloader supports it [15:11] thats what lool is testing now [15:11] asac: I think for grub thats fine and we can do that [15:11] e.g. grub + u-boot [15:11] think both could work [15:11] kyrofa: updated - let me know if it's cleare now :) [15:12] (I'm installing the test machine ATM) [15:12] cool [15:12] asac: but uboot seems to not support it, but that is ok because for uboot we need to extract the kernel and initird anyway so we might as well do more magic [15:12] they need to be next ot each other in memory and the right bracketing start,length passed to the kernel [15:12] uboot allows to load to raw mem position [15:12] asac: trouble if of course rollback, once we combine them on disk everything becomes harder [15:12] so trick is to just load the two files neck to neck [15:12] dholbach, ah, yeah that works :) [15:12] yeah [15:12] asac: yeah, uboot is flexible enough for that [15:12] apw: is there an easy well to tell grub / u-boot to do this, or do we need to write all the tiny scripting logic to compute the addresses and kernel args? [15:12] uboot needs to have special boot partition anyway right? [15:13] or can we read squshfs etc. there? [15:13] no we cant [15:13] asac: it has special handling [15:13] albeit there are patches to run grub-efi as a payload to u-boot now [15:13] right. so arm still needs to gert files put into special place [15:13] lool, can't say as i have ever looked at it, typically initrds are single things loaded and that sets the kernel data at the same time [15:13] we can as well concat them when doing that [15:13] but having them cleanly separate is surely better [15:13] guess also for trusted boot etc. [15:14] lool: grub allows just list of files for initrd [15:14] i found on some blog [15:14] apw: IIUC, the concatenation only works for init ramdisks, not initramfses; is there a big difference between the two we shoudl be worried about? [15:14] so just initrd list of files [15:14] asac: ah cool [15:14] uboot you have to somewaht load first, then parse the end address [15:15] and load second to that [15:15] lool, it only works for the initramfs format ones, ie the cpio.gz style ones, those can be concantenated and the kernel will unpakc them all into the kernel root ramdisk before entering it [15:16] apw: and are all kernel compression types supported, or only gz? [15:16] is it ok if we do gz? [15:16] lool, i believe its all of the initrd compression types which are enabled in your kernel config [15:18] lool: http://savannah.gnu.org/bugs/?35238 [15:18] not sure how it was fixed [15:18] but seems it got fixed [15:20] trying to setup a vagrant snappy image results in ssh timeout failure on "vagrant up" ... [15:20] apw: is it very dirty (or even breaks) to ship the full modules.dep if i only have a subset of the modules (e.g. ship the full in the initrd)? [15:22] asac, it is just a dependancy list, so i would expect that to be non-fatally "wrong" [15:22] Are there xenial snappy test images for download? I can't find any. [15:22] ok here status quo ... https://github.com/asac/snapcraft/commits/kbuild-kernel-wip for those taht want to produce not-yet-booting kernel snaps [15:22] asac, though if the device dependant parts contain nonoverlapping subsets it is not clear how you can make a correct one in the generic bit anyhow, as it does not list the non-gneeric bits [15:22] https://github.com/asac/snapcraft/blob/kbuild-kernel-wip/examples/kernel/snapcraft.yaml [15:23] asac, or indeed visa-versa [15:23] apw: hmm. well its that i managed to filter the right modules out of my local build, just dont want to write code to remake a modules.dep with just that subset [15:23] e.g. i am using it from the right tree (not the generic one) [15:23] (if i understood your comment correctly) [15:24] Showing the VM in the VirtualBox UI gives me a something apparently looping a kernel boot again and again. any hints? [15:25] utlemming: ^^ [15:25] Sweet5hark: utlemming is vagrant/vbox expert... not sure if he is around today though [15:25] hello, im busy with compiling Boost on Ubuntu 14.04 LTS but does i need to compile it before i make a snap of it? [15:26] Sweet5hark: I've seen that once before and it was the CPU. [15:26] Sweet5hark: can you capture what happens immediately before reboot? [15:29] Or can i just install boost itself on snappy? [15:30] utlemming: any hints on what to press to prevent the box from rebooting right away? [15:31] NoiZeR: you can use snapcraft to pull the binaries from archive [15:31] into your snaop [15:35] ok do you know where the binaries are? [15:36] asac see above [15:36] elopio, would you mind taking a look at https://github.com/ubuntu-core/snapcraft/pull/210 when you have a chance? I want one more pair of eyes [15:37] NoiZeR: dont know if i understand your question [15:37] do you know the ubuntu packages you want? [15:37] i would suggest to start with making a xenial chroot, and then use apt-cache search and apt-get isntall to find what you need [15:38] once you have the packages you can use stage-packages: field in snapcraft.yaml to ensure those get put into your snap [15:38] alongside with your app etc. [15:39] but i want that as framework because the C++ boost library needs to be available everywhere [15:40] asac [15:41] ps im a total noob on Snappy just started [15:43] NoiZeR: if you dont need have one runtime instance (e.g. a mediating service/agent) then dont feel bad about folks including copies in their snaps for now [15:44] make it easy to include it through having a snapcraft part etc. [15:54] elopio, travis has been failing left and right yesterday and today [15:59] elopio, hahaha, what is the standard on the copyrights? sergio told him to just make it 2016, I would have made it 2015-2016, and you said 2015, 2016 [15:59] kyrofa: you have to put every year the file was touched. And according to the fsf, if you use the - you have to documment that usage. [16:00] so I prefer to use the , [16:00] but of course, nobody pays attention to fsf and this is just a nit. [16:00] elopio, ah, excellent. I'll remember it! [16:01] two years use a , ... more years use a - [16:01] my 2c [16:02] asac: https://www.gnu.org/licenses/gpl-howto.html [16:02] elopio, for your admiration: http://pasteboard.co/15kdW9WN.jpg [16:02] sixth paragraph [16:03] kyrofa: wow. [16:04] elopio, I don't need to work out for a week after that job [16:04] here we have cozy 23 degrees C, with no clouds in the sky. [16:05] kyrofa: did you do it all the way to the car? [16:06] elopio, yeah [16:06] I'm impressed. [16:12] kyrofa: http://www.bbc.com/news/blogs-magazine-monitor-30119410 [16:12] kyrofa: I'm not saying living someplace with snow can kill you, the BBC is :-) [16:13] tedg, haha [16:25] JamesTait: hi, do you still need a meta/snap.yaml example snap? [16:26] mvo, I believe I have one (shout.seriusens), thank you. [16:26] * JamesTait can't type for toffee today. [16:28] And I've now got the documentation for the internal snappy format, so I *think* that's everything I need to get started. But I'll shout up if I find I'm missing anything. [16:28] JamesTait: cool, http://people.canonical.com/~mvo/tmp/hello-world_3.0_all.snap is also one [16:28] * JamesTait hugs mvo. [16:29] Thank you very much - having a couple of package to test with will be a great help. [16:29] JamesTait: I think for you and the metadata you need its almost identical to package.yaml, name, version are the same, vendor is no more, description comes from yaml instead of readme.md [16:29] JamesTait: your welcome, shout if you need anything :) [16:42] Well hey there sergiusens. I figured you'd be sleeping most of today [16:43] sergiusens: kyrofa: https://github.com/ubuntu-core/snapcraft/pull/257 is green now :) [16:50] asac, bloody timing, I added some review comments :-P [16:51] asac: sorry took me a while to confirm, but it works [16:51] elopio, disk space seems to be ok in the last server deployed http://paste.ubuntu.com/14672859/ [16:51] asac: I cpio -i extracted snappy initrd, then split it into two dirs (moved lib/modules to a different one) then repacked [16:52] asac: one key thing to pay attention to is that xz compression doesn't work but lzma does [16:52] kyrofa, I am super tired, but I also flew business (with a proper bed) so its not that bad [16:52] it's very similar but not identical [16:52] so just use "lzma" to compress [16:52] fgimenez: great. [16:52] asac: I added break=top to cmdline to confirm that the initrd was fully populated, and then I completed a successful boot too [16:52] sergiusens, well take it easy today :) [16:53] asac: next step is making sure our OS snap includes a generic initrd [16:53] lool: you know the magic cli arguments for xz initrd packgin? [16:53] just in case [16:53] lool: so you could load two files from bootloader? [16:53] and it was just one disk merged? [16:53] asac: find . | cpio -c -H newc [16:53] then lzma the result [16:54] (you can pipe it directly too) [16:54] probably better compression not to though [16:55] asac: I used the initrd part1.img part2.img syntax you mentioned [16:55] worked fine [16:55] didn't try with u-boot, probably need to compute offsets myself there [16:55] which wont be fun [16:55] there also seems to be some alignment constraints [16:55] apparently you can mix various types of compressions (or no compression) on the grub initrd line; not sure if this is a grub or linux thing though [16:56] hmm. guess grub might do the uncompressing then? [16:56] i would say its not good to mix :) [16:56] it might be linux [16:57] figuring that compression changes mid file? i doubt it :) [16:57] but you newver know [16:57] in fact, it's more likely linux as grub didn't bark at teh xz compression but linux failed to boot [16:57] as in it panicked [16:57] elopio, i see that the build jobs are restricted by label in the config https://github.com/ubuntu-core/snappy-jenkins/blob/master/containers/jenkins-master/config/jobs/create-cloud-image-all-snaps/config.xml#L32 [16:57] yeah [16:57] lool: so i could only use xz with funnny cli arguments [16:57] let me find them [16:57] elopio, but not in the deployed server http://10.55.32.123:8080/job/create-cloud-image-vivid/configure [16:57] asac: it's not midfile, linux gets the cpio header from the first piece [16:57] it knows how long the initrd will be [16:57] lool: xz --check=crc32 --lzma2=dict=512KiB inputfile [16:57] thats how it works [16:57] yeah [16:58] lzma is simpler to type :-) [16:58] otherwise it always panicked for me not being able to uncompress initrd [16:58] is it the same? no idea [16:58] :) [16:58] lzma is what I used [16:58] it's lzmo compression anyway [16:58] rihht, but above also works [16:58] yeah [16:58] kind of [16:58] something is different afaik :) [16:58] perhaps I should test this on rpi2 [16:59] not compression, but maybe contgainer [16:59] yeah do it [16:59] just two uinitrds and then load it with manual address offset for now [16:59] albeit if we're not doing squashfs loading there, there's no point in smarts to laod multiple initrds [16:59] we dont use uinitrd AFAIK [16:59] well, its good [16:59] just plain initrds [16:59] at least no need to add merge logic [16:59] fgimenez: that makes no sense. [16:59] and if they are on disk [16:59] there migth be stuff with secure boot [16:59] as we can validate the signatures as they were shipped [17:00] lool: dunno... uboot needs to load initrd... thought it needs the mkimage header [17:00] ifg not, then even easier i guess [17:00] maybe you can even do fatload file1 file2? [17:00] :) [17:00] elopio, maybe we are creating the job before any slave with that label is available? that shouldn't be a problem imo... [17:01] fgimenez: when that happens, the job config just prints a warning. [17:01] asac: as I said, it does not [17:01] lool: anyway, i was hoping for your magic to make the uboot script do rthe right thing :) [17:01] but you can still create it. so weird... [17:01] we dont use the mkimage headers [17:01] e.g. address calc [17:01] good [17:01] but uImage? [17:01] fatload multiple args is quite certainly incorrect as the initrds need alignment [17:02] we dont use uimage [17:02] really? wow :) [17:02] fgimenez: maybe canRoam needs to be false. [17:02] uimage is only required for flash baked u-boot payloads; if built with the right configs, u-boot will happilly start a regular initrd [17:02] guess that puts u-boot rquirement really high [17:02] like 2014 crack [17:02] just one config [17:02] elopio, the rest of things seem to be in place, once the images and packages are built we are ready to go [17:02] it's mainly something that we need to make sure the boards enable [17:02] I tried manually adding the label, and that value changes [17:03] right, which is the problem with putting sophisticated requirements on bootloaders :) [17:03] anyway, lets give it a go [17:03] fgimenez: I'll made the change manually, and propose a branch for the next deploy. [17:03] if you can figure the offset calc in uboot.scr [17:03] you are my hero :) [17:03] and we can go directly to making the plat-indep[ initrd in OS snap [17:04] I wonder whether we'd be better off with using grub-efi on top of u-boot at this point [17:05] asac: I'll try to repro on snapdragon or on rpi2 tonight, just to make sure there's no arch specific support issue there, then we can see which bootloader path we take [17:06] elopio, ok, leaving, have a great day [17:07] lool: i dont know... feels thats even more sophisticated [17:07] snapdragon has new uboot [17:07] try something with old too [17:09] asac: the thing with grub-efi is that it's a single thing to worry about enabling in u-boot, and hten we can use the same logic everywhere [19:56] elopio, kudos on the xenial mp! :) [19:56] :D [19:58] elopio, can I ask you to lobby for my MP with sergiusens ? ;-) [20:15] elopio, I'm trying to duplicate the lack of locale settings that causes snapcraft --help to barf a stacktrace on an lxc where it works fine [20:15] elopio, help? [20:20] sergiusens, you could probabyl answer that as well ^^ [20:39] kyrofa: take a look at the new .travis.yml [20:39] create a lxc following the same steps, and it won't have the locales set. [20:39] we need utf-8 to run some unicode tests, so I changed runtests.sh to set the language. [20:40] remove that, run the tests, and you'll get an error. [20:40] elopio, alright, thanks :) [20:40] kyrofa: however, my theory is that the error came from the unicode copyright symbol on some files. That's fixed already. [20:41] if my theory is correct, the unicode chars from the tests won't get imported, and you'll be able to run snapcraft in ascii. [20:41] elopio, or the quotation marks (dholbach noticed those) [20:41] hum, I didn't see them. Great we have more eyes :) [21:09] elopio, I know you've reviewed https://github.com/ubuntu-core/snapcraft/pull/209 . Do you have a sec for a question on it? [21:10] kyrofa: I just took a quick look and didn't fully understand the tests. What's your question [21:10] ? [21:10] elopio, yeah you had a good catch there [21:10] elopio, so right now, the copy plugin follows symlinks and copies in the original file, so no symlinks end up in the .snap [21:11] elopio, his change adds the ability to do the opposite, which is copy the symlinks themselves as leave them as links, even if they point outside the .snap. I think you and I agree that's not a good idea [21:11] kyrofa: that would be when this PR lands, right? [21:12] ah right, the default is true. [21:12] kyrofa: yes, I would like us to help people building correct snaps. [21:12] elopio, right. Thing is: if that PR gets fixed to prevent incorrect snaps, is there any downside to using symlinks all the time? i.e. why make it an option? [21:13] kyrofa: hum, that question means we need a good integration test for this branch. [21:13] I thought that you would have a link to a file, and you want both to end up in your snap. [21:14] but I suppose there are times when you want to keep only the real file, and get rid of the link. [21:14] elopio, ah, I was just thinking if the symlink pointed outside of the .snap, just copy the original [21:15] elopio, since sometimes the link will be absolute [21:15] kyrofa: that makes sense. If the real path is outside, copy the real, not the link. [21:16] elopio, exactly. It'll take a custom copy function and some decent tests, but then we have the best of all worlds [21:17] kyrofa: so that would be the default, and then you can override the behaviour with follow-symlinks: true | false ? [21:17] elopio, we can support that... but can you think of any reason why a developer wouldn't want symlinks in the .snap? [21:18] elopio, I can't, which begs the question: why add the option and incur the maintenance cost? [21:18] kyrofa: I assumed this would be coming from an external repo that has symlinks for some reason. [21:19] we should ask femdom for his usecase. [21:19] Well, his use-case involves _wanting_ the symlinks. I'm asking: is there ever a reason to NOT want symlinks? [21:19] kyrofa: right, I'm reading it on the bug now. [21:20] kyrofa: I agree that your proposed solution is better than his. [21:22] elopio, I'm under the impression that we didn't have symlinks in the first place to avoid the potential dangling link and just copying the original was the quickest path to that goal. I think if we eliminate that possibility symlinks will work for everyone [21:23] kyrofa: what I don't like about this is that the snap repo won't have all the files it needs. It depends on the developer machine. [21:24] but our goal is not to have reproducible builds. So I guess that's alright. [21:24] elopio, well, only potentially, and it's that way now if the repo has symlinks pointing outside of the repo [21:24] elopio, this proposal keeps everything the way it is now unless the symlink is pointing within the .snap already [21:24] yes, it'll be better than it is now. [21:25] Alright, thanks for your brain! I'll mull that over a little [21:27] elopio, also, when you have a minute: https://github.com/ubuntu-core/snapcraft/pull/264 [21:46] elopio, integration test failures suck [21:47] elopio, is there a way to get output from the build? [21:48] elopio, from run_snapcraft, specifically === popey_ is now known as popey