| 31 Jul 2023 |
Lily Foster | Is strictDeps on? | 11:02:45 |
gdamjan | beats me | 11:02:46 |
gdamjan | In reply to @lily:lily.flowers Is strictDeps on? no? | 11:03:12 |
Lily Foster | Ah then the separation of build-time and run-time inputs isn't enforced and it's assumed both are needed for both | 11:04:00 |
Lily Foster | Don't worry about it for now then. I'll test it with strictDeps on or just test cross and might ask it to be moved when reviewing your PR (sorry I didn't get to it yesterday...) | 11:04:49 |
gdamjan | no problem | 11:06:45 |
gdamjan | I'm probably gonna be offline for the better part of the day today. but I might get back online late in the evening (CEST evening that is) | 11:07:53 |
gdamjan | In reply to @lily:lily.flowers Ah then the separation of build-time and run-time inputs isn't enforced and it's assumed both are needed for both but, how then yarn and node-gyp-build are NOT runtime dependencies? | 11:16:07 |
Lily Foster | Because they're only used for build. Nodejs is also needed for build but changing the build interpreter to change the runtine interpreter feels suspicious. But you have strictDeps off so that's probably why | 11:17:30 |
gdamjan | I wonder if node can be even removed from build deps | 11:18:19 |
Lily Foster | I think it technically can but I think that's because yarn currently hardcodes the nodejs path it uses | 11:20:09 |
gdamjan | right | 11:20:23 |
Lily Foster | But if lifecycle scripts or something expects node or npm in path that it probably can't | 11:20:39 |
gdamjan | it seems that yarn updates the path with a /tmp/yarn--1690802924119-0.32849032875764905/ like directory that has node and yarn shell scripts | 11:30:00 |
gdamjan | just adding strictDeps = true; only removes bash from the immediate dependency :) | 11:48:49 |
gdamjan | https://imgur.com/a/6jJ2AsX | 11:50:17 |
Lily Foster | Weird. I didn't read the whole derivation so I'll look later. Was on mobile and just happened to notice that and thought it was weird | 12:00:09 |
gdamjan |
mweinelt (Martin Weinelt) already merged the PR - but I'd like to see what improvements can be done to it | 12:23:42 |
gdamjan | * mweinelt (Martin Weinelt) already merged the PR - but I'd like to see what improvements can be done to it | 12:23:47 |
Lily Foster | In reply to @gdamjan:spodeli.org mweinelt (Martin Weinelt) already merged the PR - but I'd like to see what improvements can be done to it Oh dang that was fast. Yeah I'm fine to review post-merge for possible fixups. It's probably for the best a security fix got merged quickly anyway | 12:27:35 |
hexa | sorry, I missed any conversdation here apparently | 12:28:16 |
hexa | * sorry, I missed any conversation here apparently | 12:28:20 |
gdamjan | oh, wait, you're Martin :) | 12:36:32 |
gdamjan | TIL | 12:36:38 |
raitobezarius | :'DDDDDDDD | 14:01:05 |
hexa | fck | 19:21:09 |
hexa |
ERROR:CLI Error: Could not find package.json up from /nix/store/qvpjwskw6j9n4g8749snwj5b6nk9xsb1-matrix-appservice-irc-1.0.1/bin/matrix-appservice-irc
| 19:21:13 |
hexa | cc gdamjan they want the package.json | 19:32:33 |
dotlambda | I'm trying to package https://github.com/microsoft/pyright/tree/main and I need the dependencies from both package-lock.json and packages/pyright/package-lock.json. Any idea how to achieve that? | 20:13:13 |
dotlambda | Btw they use something called Lerna to manage this monorepo. | 20:18:12 |