|
8 | 8 | stdenv, |
9 | 9 | fetchurl, |
10 | 10 | nodejs, |
11 | | - nodejs_20, |
| 11 | + nodejs_20 ? null, |
12 | 12 | python3, |
13 | 13 | darwin, |
14 | 14 | nodePackages, |
|
22 | 22 | commonMeta = meta lib // { |
23 | 23 | mainProgram = name; |
24 | 24 | }; |
| 25 | + |
| 26 | + isSpago = name == "spago"; |
| 27 | + |
| 28 | + # Legacy spago versions (< 0.93.45) require Node.js 20 due to better-sqlite3 |
| 29 | + # build issues with Node 22+. Spago 0.93.45+ works with the standard nodejs. |
| 30 | + # See: https://github.com/purescript/registry-dev/blob/87c5900ff6fb7090cf2b085b7eb3f75371560522/nix/overlay.nix#L152 |
| 31 | + isLegacySpago = isSpago && builtins.compareVersions version "0.93.45" < 0; |
25 | 32 | in |
26 | 33 | # Simple tarball without dependencies - just extract and link |
27 | 34 | if isSimpleTarball then |
@@ -55,31 +62,25 @@ if isSimpleTarball then |
55 | 62 | # Package with npm dependencies - use buildNpmPackage |
56 | 63 | else |
57 | 64 | let |
58 | | - # Use Node.js 20 for spago to avoid better-sqlite3 build issues with Node 22+ |
59 | | - # Same approach as registry-dev: https://github.com/purescript/registry-dev/blob/87c5900ff6fb7090cf2b085b7eb3f75371560522/nix/overlay.nix#L152 |
60 | | - isSpago = name == "spago"; |
61 | | - selectedNodejs = if isSpago then nodejs_20 else nodejs; |
| 65 | + selectedNodejs = if isLegacySpago then nodejs_20 else nodejs; |
62 | 66 |
|
63 | 67 | packageJson = |
64 | | - if (builtins.hasAttr "lockfile" source) then |
| 68 | + if builtins.hasAttr "lockfile" source then |
65 | 69 | fetchurl source.lockfile |
66 | 70 | else |
67 | 71 | "${./. + ("/" + source.path)}"; |
68 | 72 |
|
69 | | - baseNativeBuildInputs = [ selectedNodejs ]; |
70 | | - |
71 | | - # Spago needs additional native build tools for better-sqlite3 |
| 73 | + # Spago needs native build tools for better-sqlite3 |
72 | 74 | spagoNativeBuildInputs = [ |
73 | 75 | nodePackages.node-gyp |
74 | 76 | python3 |
75 | 77 | ] |
76 | 78 | ++ lib.optionals stdenv.isDarwin [ darwin.cctools ]; |
77 | | - |
78 | | - allNativeBuildInputs = baseNativeBuildInputs ++ lib.optionals isSpago spagoNativeBuildInputs; |
79 | 79 | in |
80 | 80 | (buildNpmPackage.override { nodejs = selectedNodejs; }) { |
81 | 81 | pname = name; |
82 | 82 | inherit version; |
| 83 | + |
83 | 84 | src = fetchurl source.tarball; |
84 | 85 |
|
85 | 86 | postPatch = '' |
|
88 | 89 |
|
89 | 90 | npmDepsHash = source.depsHash; |
90 | 91 |
|
91 | | - nativeBuildInputs = allNativeBuildInputs; |
| 92 | + nativeBuildInputs = [ selectedNodejs ] ++ lib.optionals isSpago spagoNativeBuildInputs; |
92 | 93 |
|
93 | 94 | # The prepack script runs the build script, but (so far) all derivations |
94 | 95 | # are pre-built. |
95 | 96 | npmPackFlags = [ "--ignore-scripts" ]; |
96 | 97 | dontNpmBuild = true; |
97 | 98 |
|
98 | | - npmInstallFlags = [ "--loglevel=verbose" ] ++ lib.optionals isSpago [ "--omit=optional" ]; |
| 99 | + npmInstallFlags = [ |
| 100 | + "--logs-max=0" |
| 101 | + "--omit=optional" |
| 102 | + ]; |
99 | 103 |
|
100 | 104 | meta = commonMeta; |
101 | 105 | } |
0 commit comments