From 346b804fda16f26c4ca9463b6489bbea9f01837a Mon Sep 17 00:00:00 2001 From: Olivier Certner Date: Fri, 5 Feb 2021 13:11:54 +0100 Subject: [PATCH] Issue #1730 - Part 2: harfbuzz: Remove some useless define causing a build error Defining _POSIX_C_SOURCE to 199309L on FreeBSD causes compile errors, because of an obscure interaction between system wrappers and libc++ headers. The source doesn't seem to need it at all, which tests confirm. I bet that this is not needed either on other modern systems (even those based on glibc), and that the define could probably be entirely removed. Not trying it because I can't test it myself. --- gfx/harfbuzz/src/hb-blob.cc | 2 ++ 1 file changed, 2 insertions(+) diff --git a/gfx/harfbuzz/src/hb-blob.cc b/gfx/harfbuzz/src/hb-blob.cc index fb48f03cab..6577781c19 100644 --- a/gfx/harfbuzz/src/hb-blob.cc +++ b/gfx/harfbuzz/src/hb-blob.cc @@ -24,10 +24,12 @@ * Red Hat Author(s): Behdad Esfahbod */ +#ifndef __FreeBSD__ /* http://www.oracle.com/technetwork/articles/servers-storage-dev/standardheaderfiles-453865.html */ #ifndef _POSIX_C_SOURCE #define _POSIX_C_SOURCE 199309L #endif +#endif #include "hb-private.hh"