-
Notifications
You must be signed in to change notification settings - Fork 13
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Warn users of the infinite recursion bug in home-manager. #25
Comments
hey @nikp123 . Thanks for the heads up. I did not stumbled upon this bug yet. I'll have a link to the issue your linked and try to reproduce. Do you maybe have a snippet with a buggy |
IIRC, it's a home-manager thing. Apparently importing home-manager modules is scope specific due to a bug. IMO it's better to inform, just in case. To reproduce the issue, you'll have to:
|
Here's the original issue in case you haven't found it: nix-community/nixvim#83 |
Add a known issues section. Inform of infinite recursion import bug reported in #25.
hey @nikp123, thanks for this. I was able to reproduce. As far as I can tell, nesting hm imports results in circular dependencies that breaks with an infinite recursion. I added a warning to the README, hope this helps. Let's keep this issue open as a way to further document workarounds / fixes. |
* README: inform of hm import bug. Add a known issues section. Inform of infinite recursion import bug reported in #25. * README: fix wording
Basically when importing an home-manager module outside of where you imported home-manager itself, you'll experience something like akin to this:
GermanBread/declarative-flatpak#9
I know that some people will run into this exact same issue, so it's better to warn them instead of leaving them with a confusing message.
The text was updated successfully, but these errors were encountered: