=== tomwardill_ is now known as tomwardill [09:11] sigh, whoopsied and `git clean`ed my wheels directory [09:12] * tomwardill waits for everything to rebuild [09:27] pappacena: I updated dogfood and reran your push job. https://pastebin.canonical.com/p/8tW3jn8fBz/ - looks good now I think? [09:53] "Found node-sass binary" [09:53] proceeds to rebuild it [09:53] gaaaah [09:56] I remember that taking some work to get right on SCA but don't quite remember the details [09:59] I think I've just run into the same problem as I did there [09:59] it needs to be an absolute path and only works in the config file [09:59] might have to copy it to somewhere like /tmp/ and then reference it [10:53] cjwatson: it looks good now! :D [10:57] https://hub.docker.com/repository/docker/pappacena/foo-image [10:58] `docker run -it --rm pappacena/foo-image:edge bash` seems to be fine too :) [11:00] Excellent. Shall we mark green and deploy then? [11:58] pappacena: ^- above question was to you BTW :) [12:20] Ah, sorry. I thought I have marked. Just marked it now [12:34] pappacena: OK - can you work with Tom to sort out a deployment then? I updated the docs this morning so you should only have a single place to look [12:34] Yes, I'm taking a look at the document right now. Maybe we can do it together after the standup, tomwardill ? [12:34] sure! [14:36] oookay, closer [14:36] now just to work out why my Make isn't working [14:38] cjwatson: any idea why `NODE_ABI = $(call lazy_eval,NODE_ABI,nodejs -p process.versions.modules)` is returning empty if I put it in a LP Makefile? [14:40] tomwardill: What's lazy_eval defined as? [14:40] aha! [14:40] yeah.... [14:40] I thought that might be it :-) [14:40] defining it [14:40] that's a thing [14:40] * tomwardill might try that [14:41] oh, look, a value! [14:41] <_< [14:41] >_> [14:42] :D [14:44] Can anyone review https://code.launchpad.net/~cjwatson/launchpad/+git/launchpad/+merge/384556 for me? [15:04] node-sass installation: https://code.launchpad.net/~twom/launchpad/+git/launchpad/+merge/384644 and dependencies: https://code.launchpad.net/~twom/launchpad/+git/launchpad/+merge/384614 [15:07] Looks very familiar, I suspect the lazy_eval stuff was originally me :) [15:07] it was :) [15:07] all of that preamble was yours [15:07] I forgot that you made it actually none-hardcoded after I just plonked a bunch of version numbers in there [15:08] Dependencies URL should be https://code.launchpad.net/~twom/lp-source-dependencies/+git/lp-source-dependencies/+merge/384643 [15:08] right? [15:08] main difference is that yarn uses the environment variable, where npm uses a cli switch [15:08] cjwatson: oh yes, copy paste fail [15:09] What a lot of dependencies, but that's node for you I guess [15:09] r=me [15:09] I haven't attempted to prune them or antyhing, it's just the result of `yarn add node-sass` === mup_ is now known as mup [16:41] from the SCSS docs: "Prefer @use over @import as it is deprecated" [16:41] what they don't say: "only the dart-sass implementation actually has a working @use" [16:41] rargh [16:41] https://github.com/facebook/create-react-app/issues/8316 [16:42] * tomwardill just uses @import [16:43] oh hey, it works now! [16:58] 17k of CSS, lovely [17:01] yet the old one is 86k [17:02] hmm, what have I missed [17:03] 130k now [17:03] that's... more believable [17:04] 92k in compressed mode, that's probably close enough [17:04] * tomwardill -> EOD [17:27] tomwardill: diff -w I guess [17:27] Well, -bu maybe