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

BUG: Need to rename config.yaml to deej_config.yaml (false file watch detection) #112

Open
AtmanActive opened this issue Jun 19, 2024 · 0 comments

Comments

@AtmanActive
Copy link

AtmanActive commented Jun 19, 2024

deej.exe uses file watch monitor to detect changes in config.yaml, but this file watch monitor DOES NOT use a full path but only the file name. Hence, when ANY OTHER config.yaml file is touched ANYWHERE on the machine, deej.exe issues a reload.

I started noticing notifications about "Configuration reloaded!" even though I'm not touching deej folder, but I am working on some other projects that, accidentally, use the SAME NAMED file in their folders.

This can be solved either by changing the file watch monitor to include the full path to config.yaml, if possible, or simply by changing the filename from config.yaml to deej_config.yaml in the source code, thus making this filename somewhat more unique.

Thanks.

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

No branches or pull requests

1 participant