[tarantool-patches] Re: [PATCH v2 2/2] say: take getaddrinfo() errors into account

Alexander Turenko alexander.turenko at tarantool.org
Tue Jul 23 17:52:49 MSK 2019


> @@ -594,6 +594,7 @@ log_syslog_init(struct log *log, const char *init_str)
>  	log->fd = log_syslog_connect(log);
>  	if (log->fd < 0) {
>  		/* syslog indent is freed in atexit(). */
> +		diag_log();

1. It is need to be properly commented: we need to log a diagnostics
here until stacked diagnostics will be implemented (#1148).

2. syslog_connect_unix() does not set a diag, diag_log() will lead to an
assertion fail.

3. I would mention this change in the commit message, because it is not
part of the problem with Mac OS you described in it.




More information about the Tarantool-patches mailing list