Skip to content
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

astro: don't set up file watchers for direnv #1634

Closed
wants to merge 1 commit into from

Conversation

gilice
Copy link
Member

@gilice gilice commented Jan 20, 2025

astro would try to set up inotify watchers for all files in direnv, including but not exclusive to a full nixpkgs, which would spam errors like the following in the console and refuse to start:

Error: ENOSPC: System limit for number of file watchers reached, watch 'nixos-homepage/.direnv/flake-inputs/nk036dp7q03yqkikxlfsapljhgnhxkka-source/default.nix' (for each file in nixpkgs).

astro would try to set up inotify watchers for all files in direnv,
including but not exclusive to a full nixpkgs, which would spam errors
like the following in the console and refuse to start:

Error: ENOSPC: System limit for number of file watchers reached, watch 'nixos-homepage/.direnv/flake-inputs/nk036dp7q03yqkikxlfsapljhgnhxkka-source/default.nix'
(for each file in nixpkgs).

Stop this madness.
@gilice
Copy link
Member Author

gilice commented Jan 20, 2025

oops, this seems to disable inotify watchers for all files (or my system is broken).

it'd be appreciated if someone could help me figure out the correct regex.

@thilobillerbeck
Copy link
Collaborator

Issues with the development environment came up recently and we are aware and will investigate.

Stop this madness.

This line wouldn't really have been necessary. ;)

@gilice
Copy link
Member Author

gilice commented Jan 20, 2025

This line wouldn't really have been necessary. ;)

I am sorry, I didn't mean to call you or the other developers mad, nor insult anyone !!
I was referring to the situation as madness -- in this case as a synonym to "chaos". I apologize and will edit.

@gilice gilice closed this Jan 20, 2025
@thilobillerbeck
Copy link
Collaborator

oops, this seems to disable inotify watchers for all files (or my system is broken).

it'd be appreciated if someone could help me figure out the correct regex.

I guess btw. its neither your system nor inotify. :D
#1641

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants