Revert "Merge pull request #4922 from nrdxp/default-submodules" by edolstra · Pull Request #5284 · NixOS/nix (original) (raw)
I don't really understand the complaints personally, and they can be easily worked around with an alternate fetcher. I don't know why you'd want to pull all of llvm as a submodule in a flake input anyway 😕
But alas, we are using flakes in production and this broke our workflow so we are probably going to maintain a patch and build our own Nix on our Hydra until a better solution emerges unfortunately. I originally opened the PR so we wouldn't have to do this.
I'd really love to study up on C++ and solve this myself, but Nix is an especially intimidating codebase and my backlog just keeps growing unfortunately 🤦
I guess the most annoying thing about it is that one guys usecase seems to trump like 15 people in favor of this 😒