Make cpu_collect compatible with OSX and use gstdbuf

aaronlna commented 6 years ago

https://github.com/samoshkin/tmux-plugin-sysstat/issues/5

Environment

OSX Sierra 10.12.6
zsh 5.2 (x86_64-apple-darwin16.0)
coreutils: stable 8.29 (bottled)

Problems

iostat intervals are buffered-- even while using stdbuf-- until all intervals are complete. This makes the cpu_collect script not update the statusline for TMUX as a refresh happens before all intervals are completed (ie. nothing gets written to the corresponding tmp file).

Changes

Use the g prefixed gstdbuf on OSX as to not require modifying the user PATH.
Modify refresh_interval and samples_count to complete two samples of iostat per TMUX status line refresh.

chris-vest commented 6 years ago

This fixed the issue for me. Thank you!

hit9 commented 6 years ago

Will this pr be merged?

hit9 commented 6 years ago

brew install coreutils gets me gstdbuf installed.

And then CPU outputs ok.

kleinjm commented 5 years ago

@samoshkin please merge this

h0adp0re commented 2 years ago

@samoshkin please consider merging this. Currently I (and probably many others) have resorted to using aaronlna/tmux-plugin-sysstat but it'd be nice to use the main project.

samoshkin / tmux-plugin-sysstat