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

Exclude files from --file-watch and more debugging info #703

Closed
agrafix opened this issue Aug 1, 2015 · 4 comments
Closed

Exclude files from --file-watch and more debugging info #703

agrafix opened this issue Aug 1, 2015 · 4 comments

Comments

@agrafix
Copy link
Contributor

@agrafix agrafix commented Aug 1, 2015

I'm currently moving a project to stack, and it's been working out great so far! I like the --file-watch flag, but it seems to rebuild way to often. I think this is due to my emacs configuration which creates temporary files in the source tree. Would it be possible to read .hgignore or .gitignore files to exclude files from being watched? For debugging purposes, it would also be very useful if stack build --file-watch gave some information why it thinks a rebuild is useful, eg:

Rebuilding due to changes in files X.hs, Z.hs, ...

@snoyberg
Copy link
Contributor

@snoyberg snoyberg commented Aug 5, 2015

All of the filewatch code is in its own module at Stack.FileWatch, and should be relatively easy to modify without diving into the rest of the code. In particular, indicating which files changed should be a trivial change. Interested in trying your hand at a PR for this?

@snoyberg snoyberg added this to the 0.3.0.0 milestone Aug 5, 2015
@agrafix
Copy link
Contributor Author

@agrafix agrafix commented Aug 6, 2015

Sure, I'll look into this!

@snoyberg
Copy link
Contributor

@snoyberg snoyberg commented Sep 6, 2015

I've pushed a commit to master that significantly improves this experience. I'm going to close this issue, please ping if master doesn't do what you're looking for.

@snoyberg snoyberg closed this Sep 8, 2015
@agrafix
Copy link
Contributor Author

@agrafix agrafix commented Sep 8, 2015

Alright, great, thank you! I'm very busy this week, but I'll provide feedback next week.

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

Successfully merging a pull request may close this issue.

None yet
2 participants