Closed sdaoden closed 2 years ago
Hm. Second github pull request ever. Here rotate is on top of notify, but.. well. Sorry for that, dunno how to split it via github, but of course one needs notify to not have merge conflicts anyhow.
LGTM, except for the test code, see the first PR for test changes required. Nice work!
Thanks! I have updated the test accordingly.
Rebased and merged manually to master.
On top of
https://github.com/troglobit/sysklogd/pull/45
(merge conflicts otherwise) here is the forced-rotation patch again. Here too i have split out the test, which is untouched for 24 months.