From efb704185fa0ee7436250fe6892bb5b41e55d359 Mon Sep 17 00:00:00 2001
From: themancalledjakob <jrkb@jrkb.land>
Date: Sat, 16 Mar 2024 13:19:54 +0100
Subject: [PATCH] cleanup and comments

---
 scripts/formulas/msdf-atlas-gen.sh | 37 +++---------------------------
 1 file changed, 3 insertions(+), 34 deletions(-)

diff --git a/scripts/formulas/msdf-atlas-gen.sh b/scripts/formulas/msdf-atlas-gen.sh
index 23594c5..c439ab7 100644
--- a/scripts/formulas/msdf-atlas-gen.sh
+++ b/scripts/formulas/msdf-atlas-gen.sh
@@ -28,40 +28,6 @@
 # $ make
 # $ sudo make install
 #
-# emscripten notes:
-# additionally to the depencies below
-# you may have to build freetype from source
-# and with that also bzip2
-# $
-# $ # tinyxml2:
-# $ cd your/favorite/persistent/build/directory
-# $ git clone https://github.com/leethomason/tinyxml2.git
-# $ cd tinyxml2
-# $ mkdir build && cd build
-# $ emcmake cmake ..
-# $ emmake make
-# $ emmake make install
-# $
-# $ # zlib:
-# $ cd your/favorite/persistent/build/directory
-# $ wget https://www.zlib.net/zlib-1.2.13.tar.gz
-# $ tar -xf zlib-1.2.13.tar.gz
-# $ cd zlib-1.2.13.tar.gz
-# $ mkdir build && cd build
-# $ emcmake cmake ..
-# $ emmake make
-# $ emmake make install
-# $
-# $ # libpng:
-# $ cd your/favorite/persistent/build/directory
-# $ wget https://sourceforge.net/projects/libpng/files/libpng16/1.6.39/libpng-1.6.39.tar.xz/download -O libpng-1.6.39.tar.xz
-# $ tar -xf libpng-1.6.39.tar.xz
-# $ cd libpng-1.6.39
-# $ mkdir build && cd build
-# $ emcmake cmake ..
-# $ emmake make
-# $ emmake make install
-
 
 # array of build types supported by this formula
 # you can delete this to implicitly support *all* types
@@ -134,6 +100,9 @@ function prepare() {
 
     PREV_DIR="$(pwd)"
 
+    # NOTE: currently, these are being build and installed in the emscripten sdk directory
+    # however, it would be better to put the sources in the ofxMsdfgen/libs directory
+    # and build them from there
     if [ "$TYPE" == "emscripten" ] ; then
         echo "preparing $TYPE thirdparty dependencies"
         if decision "download, build and install all thirdparty dependencies" $PROBABLY_YES; then