[RFC] syslog output from SAL_* messages

Riccardo Magliocchetti riccardo.magliocchetti at gmail.com
Tue Dec 18 12:14:19 PST 2012

Il 18/12/2012 11:52, Tor Lillqvist ha scritto:
>> - am not able to send the ENABLE_SYSLOG definition down to
>> sal/osl/unx/salinit.cxx
> Surely the tradityional name for this should be HAVE_SYSLOG_H ?
> ENABLE_FOO things are for stuff you enable or disable on the configure
> command line, not for results of checks of header file presence. (Of
> course, it might be that there are counter-examples in LO's
> configure.ac, but we shouldn't continue to follow such anti-patterns.)
> I guess you need to add to config/config_global.h.in:
> #undef HAVE_SYSLOG_H
> and change the test to a simple:
> AC_CHECK_HEADERS(syslog.h) ?
> Isn't that the simple, normal way to check for some header?

Yep, and differently from my previous approach it even works :)


More information about the LibreOffice mailing list