Skip to content

Stopping the logfile spam by piping STDERR to /dev/null#8081

Merged
fabaff merged 1 commit into
home-assistant:devfrom
madpilot:dev
Jun 18, 2017
Merged

Stopping the logfile spam by piping STDERR to /dev/null#8081
fabaff merged 1 commit into
home-assistant:devfrom
madpilot:dev

Conversation

@madpilot
Copy link
Copy Markdown
Contributor

When pinging computers using the Wake On LAN switch, the log files get spammed with

ping: cannot resolve computer.local: Unknown host

which makes sense, if the computer is turned off. However, this isn't really an error.

This patch pipes STDERR to /dev/null, cleaning up the log a little.

@homeassistant
Copy link
Copy Markdown
Contributor

Hi @madpilot,

It seems you haven't yet signed a CLA. Please do so here.

Once you do that we will be able to review and accept this pull request.

Thanks!

@mention-bot
Copy link
Copy Markdown

@madpilot, thanks for your PR! By analyzing the history of the files in this pull request, we identified @fabaff, @titilambert and @Chris-V to be potential reviewers.

Copy link
Copy Markdown
Member

@fabaff fabaff left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me 🐦

@fabaff fabaff merged commit ef63cfe into home-assistant:dev Jun 18, 2017
@balloob balloob mentioned this pull request Jul 1, 2017
@home-assistant home-assistant locked and limited conversation to collaborators Oct 20, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants