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.

Sign up for GitHub

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

Jump to bottom

plugin mpdstats_ uses netcat -q which is not implemented in every version #576

Closed
azrdev opened this issue Jan 28, 2015 · 0 comments · Fixed by #759
Closed

plugin mpdstats_ uses netcat -q which is not implemented in every version #576

azrdev opened this issue Jan 28, 2015 · 0 comments · Fixed by #759
Labels
bug

Comments

@azrdev
Copy link

azrdev commented Jan 28, 2015

On my Fedora 21 install, the following netcat port is installed:

$ nc --version
Ncat: Version 6.47 ( http://nmap.org/ncat )

... which lacks support for the -q switch (wait after stdin for x seconds, then quit). mpdstats_ uses nc -q, and thus fails.
@ssm ssm added the bug label Feb 17, 2015
@sumpfralle sumpfralle mentioned this issue Oct 22, 2016
Merged
sumpfralle added a commit that referenced this issue Oct 22, 2016
@sumpfralle
Merge pull request #759 from sumpfralle/improve-mpdstats
79851ce 
* fix #576 (unsupported netcat parameter)
* use which for netcat detection
* allow unconfigured usage (print all stats)
letompouce added a commit to letompouce/munin-monitoring-contrib that referenced this issue Jan 16, 2021
@letompouce
mpdstats_: allow to pass options to nc(1).
d1ea1b3 
Different nc(1) implementations will use different flags, as highlighted
in munin-monitoring#576. This environment variable allows one to pass any option to the
nc binary.
letompouce added a commit to letompouce/munin-monitoring-contrib that referenced this issue Jan 16, 2021
@letompouce
mpdstats_: allow to pass options to nc(1).
61bf15f 
Different nc(1) implementations will use different flags, as highlighted
in munin-monitoring#576. This environment variable allows one to pass any option to the
nc binary.
sumpfralle pushed a commit that referenced this issue Jan 17, 2021
@letompouce @sumpfralle
mpdstats_: allow to pass options to nc(1).
9ad3b1f 
Different nc(1) implementations will use different flags, as highlighted
in #576. This environment variable allows one to pass any option to the
nc binary.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Improve mpdstats sumpfralle/contrib
2 participants
@ssm @azrdev