<!DOCTYPE html><html><head><meta http-equiv="Content-Type" content="text/html; charset=utf-8" /></head><body><div data-html-editor-font-wrapper="true" style="font-family: arial, sans-serif; font-size: 13px;"><div><div><div style="font-family: arial, sans-serif;font-size: 13px">On Tue, 08 Mar 2016 16:25:06 +0000 "Lennart Weller" <lhw@ring0.de> wrote:<br>> March 8 2016 1:52 PM, "Sergey Vojtovich" <svoj@mariadb.org> wrote:<br>> > I adjusted your patch a bit, it seem to work well for me. Could you please<br>> > verify if you're fine with the attached version and it works for you too?<br>><br>> Well you basically dropped all the safe guards. But in the end its a logrotate script.<br>> Most people will never have the problem in the first place or know whatwent wrong on<br>> their end when they see your version of the script. So yeah fine with me.<br><br>For the record: this patched script fails on my Debian Jessie. For whatever reason, 'my_print_defaults' returns two 'pid-file' lines, so the 'if' statement chokes, 'flush-logs' doesn't run, and cron emails me an "unexpected operator" error.  My workaround was to change the grep parameters to ensure only one 'pid-file' line is returned.<br>e.g.<br>    grep -oPm1 ...<br> </div></div></div></div></body></html>