I also rolled a small fix for the buggy edge case that I mentioned @infinisi found with his script.

The package seems to be marked as broken however, so I dunno.

Hmm, ihaskell does not look like fixing an URL. documentation-highlighter is a rebuild because it copies its dir including default.nix. Not sure about the other rebuilds

@7c6f434c ihaskell's derivation's original behavior is buggy, and that behavior was exposed by an automatic tool written by @infinisil to take care of this problem. The URL detected is PATH:+ and this is because the \${} does not prevent nix from attempting to evaluate PATH:+':' as a nix expression. Of course, this is unintended. The \ was intended to prevent this from being evaluated. But \ doesn't work inside of '' quoted strings. So this change is to fix that bug.

Can you update the git commit subject to match the updated PR description? There are some guidelines for commit msg formatting in https://github.com/NixOS/nixpkgs/blob/master/.github/CONTRIBUTING.md, but our convention for PRs touching a bunch of packages is to start it with treewide:

https://github.com/NixOS/nixpkgs/pulls?q=is%3Apr+in%3Atitle+treewide+

Result of nixpkgs-review pr 84130 1

I would be happy to. I am sorry, I was unaware of the treewide name.

Fixed the trailing whitespace