svn commit: r433881 - in head/sysutils: . lttng-tools lttng-tools/files lttng-ust lttng-ust/files

[Jan Beich]

Mahdi Mokhtari <mmokhi at FreeBSD.org> writes:

> + 	char tmp_path[PATH_MAX];
> ++#if defined(__FreeBSD__)
> ++	const size_t HOST_NAME_MAX = sysconf(_SC_HOST_NAME_MAX);
> ++#endif
> + 	char hostname[HOST_NAME_MAX];
[...]
> + struct ltt_session {
> + 	char name[NAME_MAX];
> ++#if defined(__FreeBSD__)
> ++	char hostname[256]; /* Local hostname. */
> ++#else
> + 	char hostname[HOST_NAME_MAX]; /* Local hostname. */
> ++#endif

Why do you mix constant and sysconf() usage?

> + #if (defined(__FreeBSD__) || defined(__CYGWIN__))
> + typedef long long off64_t;
> + #endif

FreeBSD 11.0 or later define off64_t as an alias for off_t aka int64_t
which is "long" (not "long long") on amd64. Is the conflicting
definition intentional?