From c54578e25441dba7caf00cfc54202dd03d87f42e Mon Sep 17 00:00:00 2001
From: NiLuJe <ninuje@gmail.com>
Date: Sat, 22 Feb 2020 09:11:29 +0100
Subject: [PATCH] [doc] macOS build info tweaks (#5891)

* Update macOS PATH shenanigans.

c.f., https://github.com/koreader/koreader-base/pull/1052

* Prefer coreutils, it now ships everything we need.

(Or it always did? I apparently don't actually have coreutils
installed on my end).

* Also recommend putting Homebrew first in PATH

Mainly because it turns out that's how I tested this the last time ;).
Also, it  just makes sense, and might be part of Homebrew's
recommendations anyway.
---
 doc/Building.md | 14 ++++++++++++--
 1 file changed, 12 insertions(+), 2 deletions(-)

diff --git a/doc/Building.md b/doc/Building.md
index d2a36eaa8..b762e8f84 100644
--- a/doc/Building.md
+++ b/doc/Building.md
@@ -39,8 +39,18 @@ Install the prerequisites using [Homebrew](https://brew.sh/):
 
 ```
 brew install nasm ragel binutils coreutils libtool autoconf automake cmake makedepend \
-sdl2 lua@5.1 luarocks gettext pkg-config wget md5sha1sum
-echo 'export PATH="/usr/local/opt/gettext/bin:$PATH"' >> "$HOME"/.bash_profile
+sdl2 lua@5.1 luarocks gettext pkg-config wget
+```
+
+You will also have to ensure Homebrew's gettext is in your path, e.g., via
+```
+export PATH="/usr/local/opt/gettext/bin:${PATH}"
+```
+See also `brew info gettext` for details on how to make that permanent in your shell.
+
+In the same vein, if that's not already the case, you probably also want to make sure Homebrew's stuff takes precedence:
+```
+export PATH="/usr/local/bin:/usr/local/sbin:${PATH/:\/usr\/local\/bin/}"
 ```
 
 *Note:* With current XCode versions, you *will* need to set a minimum deployment version higher than `10.04`. Otherwise, you'll hit various linking errors related to missing unwinding libraries/symbols.