node-packages-generated.nix is more than 4,000 lines of generated code; that is much worse than only including the package-lock.json!

Only possible (in my knowledge) are:

• keep the old package-json.lock
• generate node packages for Iosevka only
• generate node packages globally (in pkgs/development/node-packages)

What do you think is the best option ? I think the third one might be the best one but adding this much packages to node-packages might be an issue.

EDIT: It looks like it's harder to implement too. (we need to find which dependencies need to be copied, apart from the one specified in the package.json)

I think the third one might be the best one but adding this much packages to node-packages might be an issue.

I tried to do this locally, it wasn't hard to add it to the global nodePackages set, but I can't figure out how to override the generated package (to build the font) without breaking the build. I'm in favor of keeping the generated file until someone complains, because at least it works.

Thanks!

Thanks too !

As far as I understand, there should only be one generated node file in nixpkgs. This is too big and too many duplicates between that generated node file and the one in nodeModules.

Yes, this seems to be one of the problem with node packages. It's too difficult to use one global generated node-packages.nix as you can't import all dependencies (you have to do it manually)

You could take the vector it pkgs/data/fonts/iosevka/node-packages.json and turn it into a package.json, then you can symlink the node_modules directory

ln -s ${nodePackages."yourPackageName"}/lib/node_modules/yourPackageName/node_modules ./node_modules

Oh I see. I didn't know that. Maybe you could specify it for the new PR: #69219 ? It aims to clean the package

@hlolli The issue with doing it that way is Iosevka uses dev dependencies to build, so node2nix would have to be run in dev mode on the global nodeModules file. I commented on svanderburg/node2nix#149 about this, but I'm not sure if there's a better way right now.

I think adding devDependencies is the wrong way to go. Because then we would also adding hot-reloaders and stuff unrelated to the build (and runtime) of the application. I Think rather that the dependencies should be extanded, so that we as package maintainers add the packages manually which we need to build. This I did with lumoBuildDeps using hand-made package.json. It's already taking 15 minutes to generate the node-packages, but I don't see a better way to combine small size and fast generation time.