aliases: keep the utillinux alias until node2nix is phased out

[samueltardieu]

The util-linux alias to utillinux has been replaced by a throw in commit a9e1f4e. The check in node-env.nix must be reversed, it was previously a no-op (because of the alias) but will now lead to a failure as it will select the utillinux throw over the util-linux package.

Also, tidy things up by using the util-linux variable name instead of the older utillinux.

The code generated by node2nix checks that pkgs.utillinux exist and uses it over pkgs.util-linux. Replacing the alias by a throw, as was done in commit a9e1f4e, makes packages generated using node2nix fail.

This removes the alias removal until node2nix has been phased out, which is a work in progress started in #229475.

This PR fixes the build process of packages such as "emojione" but doesn't cause them to be rebuilt (so no "staging" merge), it prevents them from failing the build, which would happen without this change.

• Built on platform(s)
  • x86_64-linux
  • aarch64-linux
  • x86_64-darwin
  • aarch64-darwin
• For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)
  • sandbox = relaxed
  • sandbox = true
• Tested, as applicable:
  • NixOS test(s) (look inside nixos/tests)
  • and/or package tests
  • or, for functions and "core" functionality, tests in lib/tests or pkgs/test
  • made sure NixOS tests are linked to the relevant packages
• Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
• Tested basic functionality of all binary files (usually in ./result/bin/)
• 24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)
  • (Package updates) Added a release notes entry if the change is major or breaking
  • (Module updates) Added a release notes entry if the change is significant
  • (Module addition) Added a release notes entry if adding a new NixOS module
• Fits CONTRIBUTING.md.

---

Add a 👍 reaction to pull requests you find important.

[samueltardieu]

(ping @AndersonTorres)

[felbinger]

Looks good to me

[Artturin]

The whole variable should be removed

Should also be fixed upstream too @svanderburg https://github.com/svanderburg/node2nix/blob/master/nix/node-env.nix#L7

[Artturin]

This pr might also be reverted by node2nix like what happened in #319134 (comment)

https://github.com/orgs/nix-community/discussions/1523

[Artturin]

@AndersonTorres revert removing this alias and add # keep to it until its fixed upstream and a new ver is released

[samueltardieu]

@Artturin I've amended this PR to revert the alias removal until node2nix is phased out.

I have also submitted svanderburg/node2nix#336 so that node2nix can get fixed until it is phased out to prevent problems if people keep using it while the alias has been converted to a throw again in the future.

[emilazy]

We should probably just apply the patch to our node2nix so we’re not blocked on an unmaintained upstream. We already have several load‐bearing patches.

[sternenseemann]

Yeah I started on that, but we also need another patch to get rid of stdenv.isDarwin etc. in the generated output. #341407 changed that which is good, but unfortunately without adjusting the code generator.

[emilazy]

Perhaps it would be better to simply eliminate the remaining few in‐tree uses, since my experience the last time we had to test a patch was that it was very hard to even package anything modern with it at this point.

[sternenseemann]

Probably takes 15min to make a patch… much easier.

[nixos-discourse]

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:

https://discourse.nixos.org/t/cant-find-whats-incorrectly-referencing-utillinux-instead-of-util-linux/54757/3

[kjeremy]

This breaks my nixos-rebuild --switch with:

error: 'utillinux' has been renamed to/replaced by 'util-linux'

because of https://github.com/NixOS/nixpkgs/blob/master/pkgs/development/node-packages/node-env.nix#L7

[Artturin]

This breaks my nixos-rebuild --switch with:

error: 'utillinux' has been renamed to/replaced by 'util-linux'

because of master/pkgs/development/node-packages/node-env.nix#L7

This fixes that

[kjeremy]

This breaks my nixos-rebuild --switch with:

error: 'utillinux' has been renamed to/replaced by 'util-linux'

because of master/pkgs/development/node-packages/node-env.nix#L7

This fixes that

Apparently I need more sleep. I read the diff backwards

[nixos-discourse]

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:

https://discourse.nixos.org/t/cant-find-whats-incorrectly-referencing-utillinux-instead-of-util-linux/54757/8