30 Sep 2022 |
Winter (she/her) | I can do the update in a few hours, if nobody else does by then. | 12:33:44 |
Winter (she/her) | also wait Sandro 🐧 what were you setting NODE_OPTIONS to? that's not in the version of the hoon you sent, so i don't think it'll reproduce this issue | 12:34:23 |
Winter (she/her) | (i don't see what you'd be setting it to) | 12:36:43 |
Dandellion | I was getting it by setting the node option in an overrideatttrs configurephase | 12:37:16 |
Winter (she/her) | what are you setting node_options to | 12:37:32 |
Winter (she/her) | (plus i don't see why svgo would complain about node_options now if it worked before, sandro -- did you only recently add the node_options setting?) | 12:40:02 |
Dandellion | https://github.com/NixOS/nixpkgs/blob/8039b055f8ef0693724a9909145f3daf5d762da5/pkgs/applications/networking/instant-messengers/element/element-web.nix#L49 | 12:41:40 |
Winter (she/her) | Dandellion: https://github.com/nodejs/node/blob/2849283c4cebbfbf523cc24303941dc36df9332f/src/node_options.cc#L924
so it is allowed in NODE_OPTIONS, which is evident by the package building.
what was your overlay? how can i reproduce this? | 12:46:43 |
Winter (she/her) | i'm confused, that option has always been allowed to be set in NODE_OPTIONS, which is evident by it working in element-web | 12:48:32 |
Winter (she/her) | so yes i'd appreciate a reproducible example :) | 12:49:12 |
Winter (she/her) | i have like, one theory that would explain it, hm | 12:50:03 |
Dandellion | It seems it was me using pkgs.fixup_yarn_lock rather than pkgs.unstable.fixup_yarn_lock | 13:01:20 |
Winter (she/her) | Is it possible for you send a reproducible example? Just want to see if my theory is correct. | 13:02:56 |
Dandellion | configurePhase = ''
runHook preConfigure
export HOME=$PWD/tmp
export NODE_OPTIONS=--openssl-legacy-provider
mkdir -p $HOME
${pkgs.fixup_yarn_lock}/bin/fixup_yarn_lock yarn.lock
yarn config --offline set yarn-offline-mirror $offlineCache
yarn install --offline --frozen-lockfile --ignore-platform --ignore-scripts --no-progress --non-interactive
patchShebangs node_modules
# Replace with my matrix-react-sdk
rm -rf node_modules/matrix-react-sdk
ln -s ${matrix-react-sdk} node_modules/matrix-react-sdk
runHook postConfigure
'';
based on summerevans stuff
| 13:05:23 |
Dandellion | with matrix-react-sdk being a callPackaged:
{ lib
, stdenv
, fetchpatch
, fetchFromGitHub
, fetchYarnDeps
, writeText
, jq
, yarn
, fixup_yarn_lock
, nodejs
}: stdenv.mkDerivation rec {
pname = "matrix-react-sdk";
version = "3.57.0";
src = fetchFromGitHub {
owner = "matrix-org";
repo = "matrix-react-sdk";
rev = "v${version}";
sha256 = "sha256-duwjxaS4dKn/GbgykqLdQAjXeBG8kt5dhGa+O9iJacg=";
};
offlineCache = fetchYarnDeps {
yarnLock = src + "/yarn.lock";
sha256 = "sha256-Y/0hyLO78e4KGMfoCRy6bF7tU6rn6LwEcDt5afWTR6U=";
};
patches = [
# All of my patches to matrix-react-sdk
(fetchpatch {
url = "https://github.com/sumnerevans/matrix-react-sdk/commit/ed00c00901c1741042162924bba5e800eee71402.patch";
sha256 = "sha256-YNXRGlTDhYdIeciVRNkERqg366OYVgUaQWFAY9aP1p4=";
})
];
nativeBuildInputs = [ yarn fixup_yarn_lock jq nodejs ];
configurePhase = ''
runHook preConfigure
export HOME=$PWD/tmp
mkdir -p $HOME
fixup_yarn_lock yarn.lock
yarn config --offline set yarn-offline-mirror $offlineCache
yarn install --offline --frozen-lockfile --ignore-platform --ignore-scripts --no-progress --non-interactive
patchShebangs node_modules
runHook postConfigure
'';
buildPhase = ''
runHook preBuild
node_modules/.bin/babel -d lib --verbose --extensions \".ts,.js,.tsx\" src
runHook postBuild
'';
installPhase = ''
runHook preInstall
cp -R . $out
runHook postInstall
'';
}
| 13:07:04 |
Sandro 🐧 | In reply to @winterqt:nixos.dev also wait Sandro 🐧 what were you setting NODE_OPTIONS to? that's not in the version of the hoon you sent, so i don't think it'll reproduce this issue The crash was in element where it is set to NODE_OPTIONS=--openssl-legacy-provider https://github.com/NixOS/nixpkgs/commit/2e3e0081e5e843599aaa8f7128d2a139bda7dbca | 13:07:10 |