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

Silence "INFO: Could not find files for the given pattern(s)" message #298

Merged
merged 2 commits into from Apr 1, 2013

Conversation

Projects
None yet
2 participants
@ddfreyne
Member

ddfreyne commented Apr 1, 2013

This gets rid of the "INFO: Could not find files for the given pattern(s)" message that is spewed out by the watch command on Windows.

Fixes #82.

@ghost ghost assigned bobthecow Apr 1, 2013

@ddfreyne

This comment has been minimized.

Show comment
Hide comment
@ddfreyne

ddfreyne Apr 1, 2013

Member

@bobthecow Review, merge and close #82 afterwards.

Member

ddfreyne commented Apr 1, 2013

@bobthecow Review, merge and close #82 afterwards.

@bobthecow

This comment has been minimized.

Show comment
Hide comment
@bobthecow

bobthecow Apr 1, 2013

Member

It does the trick, but wow, that's a bit ugly :)

Is there no stderr redirection for Windows?

Member

bobthecow commented Apr 1, 2013

It does the trick, but wow, that's a bit ugly :)

Is there no stderr redirection for Windows?

@bobthecow

This comment has been minimized.

Show comment
Hide comment
@bobthecow

bobthecow Apr 1, 2013

Member

docs say:

whatever 2> nul
Member

bobthecow commented Apr 1, 2013

docs say:

whatever 2> nul
@ddfreyne

This comment has been minimized.

Show comment
Hide comment
@ddfreyne

ddfreyne Apr 1, 2013

Member

Does that also work on Windows? What docs?

IO.popen3 could be a good alternative. I’ll look into it.

Member

ddfreyne commented Apr 1, 2013

Does that also work on Windows? What docs?

IO.popen3 could be a good alternative. I’ll look into it.

@bobthecow

This comment has been minimized.

Show comment
Hide comment
@bobthecow

bobthecow Apr 1, 2013

Member

http://support.microsoft.com/kb/110930

That's the Windows way to do it. *nix uses /dev/null instead of nul.

Member

bobthecow commented Apr 1, 2013

http://support.microsoft.com/kb/110930

That's the Windows way to do it. *nix uses /dev/null instead of nul.

@ddfreyne ddfreyne referenced this pull request Apr 1, 2013

Closed

windows watch #82

@ddfreyne

This comment has been minimized.

Show comment
Hide comment
@ddfreyne

ddfreyne Apr 1, 2013

Member

@bobthecow Review and merge.

Member

ddfreyne commented Apr 1, 2013

@bobthecow Review and merge.

@ddfreyne

This comment has been minimized.

Show comment
Hide comment
@ddfreyne

ddfreyne Apr 1, 2013

Member

Actually you already said OK. :)

Member

ddfreyne commented Apr 1, 2013

Actually you already said OK. :)

@ddfreyne ddfreyne merged commit 440b6de into nanoc:release-3.6.x Apr 1, 2013

@ddfreyne ddfreyne deleted the ddfreyne:bug-quiet-watch-on-windows branch Apr 10, 2013

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment