From e9a0162eea885365b371bcdb91b2e97769d4e54a Mon Sep 17 00:00:00 2001
From: Yamagi <yamagi@yamagi.org>
Date: Thu, 8 Oct 2020 11:58:14 +0200
Subject: [PATCH] -fvisibility=hidden is a compiler option, don't pass it to
 the linker.

---
 Makefile | 8 +++-----
 1 file changed, 3 insertions(+), 5 deletions(-)

diff --git a/Makefile b/Makefile
index 4d445bbb..79cee076 100755
--- a/Makefile
+++ b/Makefile
@@ -134,7 +134,9 @@ endif
 #   to get it there...
 #  -fwrapv for defined integer wrapping. MSVC6 did this
 #   and the game code requires it.
-override CFLAGS += -std=gnu99 -fno-strict-aliasing -fwrapv
+#  -fvisibility=hidden to keep symbols hidden. This is
+#   mostly best practice and not really necessary.
+override CFLAGS += -std=gnu99 -fno-strict-aliasing -fwrapv -fvisibility=hidden
 
 # -MMD to generate header dependencies. Unsupported by
 #  the Clang shipped with OS X.
@@ -294,10 +296,6 @@ else ifeq ($(YQ2_OSTYPE), Haiku)
 override LDFLAGS += -lm -lnetwork
 endif
 
-# Keep symbols hidden.
-override CFLAGS += -fvisibility=hidden
-override LDFLAGS += -fvisibility=hidden
-
 ifneq ($(YQ2_OSTYPE), Darwin)
 ifneq ($(YQ2_OSTYPE), OpenBSD)
 # For some reason the OSX & OpenBSD