Merge pull request #61 from KTStephano/pr52

Refactoring dependency build

Author: KTStephano

README.md

@@ -97,6 +97,10 @@ First set up the repo
 
 Build 3rd party dependencies -> should only need to do this once per clone
 
+    python3 ./dependency_build.py --assimp
+
+-> LINUX NOTE: If you get an error while building assimp, use your local package manager to install assimp instead. Then re-run the above step as:
+
     python3 ./dependency_build.py
 
 Now build the StratusGFX source
@@ -108,14 +112,11 @@ Now build the StratusGFX source
 
 ### Debian Linux
 
-    sudo apt install libassimp5
-    sudo apt install libassimp-dev
     cmake -Bbuild -S. -DCMAKE_BUILD_TYPE=RelWithDebInfo -DBUILD_TESTS=OFF
     cmake --build build/ -j 8
 
 ### Arch Linux
 
-    sudo pacman -S assimp
     cmake -Bbuild -S. -DCMAKE_BUILD_TYPE=RelWithDebInfo -DBUILD_TESTS=OFF
     cmake --build build/ -j 8
 

assimp

@@ -3,20 +3,27 @@
 import os
 import threading
 from distutils.dir_util import copy_tree
+import argparse
+
+parser = argparse.ArgumentParser(prog='Stratus Engine Dependency Build')
+parser.add_argument('-a', '--assimp',
+                    action='store_true', 
+                    default=False)
+args = parser.parse_args()
 
 os.environ["CMAKE_BUILD_PARALLEL_LEVEL"] = str(os.cpu_count())
 
 windows = ""
-linux = False
+build_assimp = args.assimp
 if os.name == "nt":
     print("Windows")
     windows = "--config Release"
 else:
     print("Linux")
-    linux = True
-    #linux = "--compile-no-warning-as-error"
 
-configure = "cmake {} -Bbuild -S. -DBUILD_SHARED_LIBS=OFF -DBUILD_TESTING=OFF -DBUILD_TESTS=OFF -DCMAKE_BUILD_TYPE=Release".format(linux)
+print("Building assimp:", build_assimp)
+
+configure = "cmake -Bbuild -S. -DBUILD_SHARED_LIBS=OFF -DBUILD_TESTING=OFF -DBUILD_TESTS=OFF -DCMAKE_BUILD_TYPE=Release -DASSIMP_BUILD_TESTS=OFF"
 build = "cmake --build build/ -j 8 {}".format(windows)
 install = "cmake --install build/ --prefix ../ThirdParty {}".format(windows)
 configure_build_install = configure + " && " + build + " && " + install
@@ -35,7 +42,7 @@
 t.start()
 
 # Assimp
-if not linux:
+if build_assimp:
     cmd = "cd assimp && " + configure_build_install
     t = threading.Thread(target=lambda: os.system(cmd), args=())
     t.start()