From: Igor Munkin <imun@tarantool.org> To: Olga Arkhangelskaia <arkholga@tarantool.org> Cc: tarantool-patches@dev.tarantool.org Subject: Re: [Tarantool-patches] [PATCH] say: fix syslog format Date: Fri, 17 Apr 2020 14:56:49 +0300 [thread overview] Message-ID: <20200417115649.GL8314@tarantool.org> (raw) In-Reply-To: <38a51d9d-eebf-3ea4-89cd-46717fd9fb75@tarantool.org> Olya, Thanks, LGTM now. Side note: I shared the recipe about diff pasting via vim[1] and hope it is helpful. On 17.04.20, Olga Arkhangelskaia wrote: > Igor, thanks a lot for your patience. > > I have fixed all nits you have mentioned. > > See diff below! > <snipped> > 12 src/lib/core/say.c | 14 ++++++++-- > 13 test/box-tap/gh-4785-syslog.test.lua | 42 > ++++++++++++++++++++++++++++ > 14 2 files changed, 54 insertions(+), 2 deletions(-) > 15 create mode 100755 test/box-tap/gh-4785-syslog.test.lua > - > 26 +`....../* > 27 +`...... * When logger type is SAY_LOGGER_BOOT it simply prints > 28 +`...... * every message to stdout intact and can't be formatted. > 29 +`...... * SAY_LOGGER_SYSLOG type uses the well-documented and > 30 +`...... * *recommended* format described in the RFC below: > 31 +`...... * https://tools.ietf.org/html/rfc3164#section-4.1 > 32 +`...... * Thereby format can't be changed for this type either. > 33 +`...... */ > 34 +`......bool unchangeable = log_default->type == SAY_LOGGER_BOOT || > 35 +`......`.......`....... log_default->type == SAY_LOGGER_SYSLOG; > 36 +`......if (unchangeable) <snipped> [1]: https://lists.tarantool.org/pipermail/tarantool-patches/2020-April/016121.html -- Best regards, IM
next prev parent reply other threads:[~2020-04-17 12:03 UTC|newest] Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top 2020-03-24 8:52 Olga Arkhangelskaia 2020-04-14 12:52 ` Igor Munkin 2020-04-15 13:25 ` Olga Arkhangelskaia 2020-04-16 21:15 ` Igor Munkin 2020-04-17 9:50 ` Olga Arkhangelskaia 2020-04-17 11:56 ` Igor Munkin [this message] 2020-04-21 16:23 ` Alexander Turenko
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=20200417115649.GL8314@tarantool.org \ --to=imun@tarantool.org \ --cc=arkholga@tarantool.org \ --cc=tarantool-patches@dev.tarantool.org \ --subject='Re: [Tarantool-patches] [PATCH] say: fix syslog format' \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: link
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox