Merge pull request #69 from buttfa/feature_library

Little changes about Makefile

Author: austinvhuang

Makefile

@@ -21,49 +21,35 @@ pch:
 # TODO(avh): change extension based on platform
 # Get the current OS name
 OS = $(shell uname | tr -d '\n')
+# Set the specific variables for each platform
 LIB_PATH ?= /usr/lib
 HEADER_PATH ?= /usr/include
-# Set the specific variables for each platform
 ifeq ($(OS), Linux)
-    OS_TYPE ?= Linux
-	
-	GPU_CPP_LIB_NAME ?= libgpucpp.so
-	DAWN_LIB_NAME ?= libdawn.so
+OS_TYPE ?= Linux
+GPU_CPP_LIB_NAME ?= libgpucpp.so
+DAWN_LIB_NAME ?= libdawn.so
 else ifeq ($(OS), Darwin)
-    OS_TYPE ?= macOS
-
-	GPU_CPP_LIB_NAME ?= libgpucpp.dylib
-	DAWN_LIB_NAME ?= libdawn.dylib
+OS_TYPE ?= macOS
+GPU_CPP_LIB_NAME ?= libgpucpp.dylib
+DAWN_LIB_NAME ?= libdawn.dylib
 else
-    OS_TYPE ?= unknown
+OS_TYPE ?= unknown
 endif
 
 lib: check-clang dawnlib
-ifneq ($(OS_TYPE), unknown)
 	mkdir -p build && $(CXX) -std=c++17 $(INCLUDES) -L$(LIBDIR) -ldawn -ldl -shared -fPIC gpu.cpp -o build/$(GPU_CPP_LIB_NAME)
 	python3 build.py
 	cp third_party/lib/$(DAWN_LIB_NAME) build/
-else
-	@echo "Unsupported operating system"
-endif
 
 install:
-ifneq ($(OS_TYPE), unknown)
 	cp build/$(GPU_CPP_LIB_NAME) $(LIB_PATH)
 	cp build/$(DAWN_LIB_NAME) $(LIB_PATH)
 	cp build/gpu.hpp $(HEADER_PATH)
-else
-	@echo "Unsupported operating system"
-endif
 
 uninstall:
-ifneq ($(OS_TYPE), unknown)
 	rm $(LIB_PATH)/$(GPU_CPP_LIB_NAME)
 	rm $(LIB_PATH)/$(DAWN_LIB_NAME)
 	rm $(HEADER_PATH)/gpu.hpp
-else
-	@echo "Unsupported operating system"
-endif
 
 examples/hello_world/build/hello_world: check-clang dawnlib examples/hello_world/run.cpp check-linux-vulkan
 	$(LIBSPEC) && cd examples/hello_world && make build/hello_world && ./build/hello_world
@@ -139,15 +125,24 @@ clean-all:
 # Checks
 ################################################################################
 
+# Check all
+check-all: check-os check-clang check-cmake check-python
+
+# check the os
+check-os:
+ifeq ($(OS_TYPE), unknown)
+$(error Unsupported operating system)
+endif
+
 # check for the existence of clang++ and cmake
 check-clang:
-	@command -v clang++ >/dev/null 2>&1 || { echo >&2 "Please install clang++ with 'sudo apt-get install clang' or 'brew install llvm'"; exit 1; }
+	@command -v clang++ >/dev/null 2>&1 || { echo -e >&2 "Clang++ is not installed. Please install clang++ to continue.\nOn Debian / Ubuntu: 'sudo apt-get install clang' or 'brew install llvm'\nOn Centos: 'sudo yum install clang'"; exit 1; }
 
 check-cmake:
-	@command -v cmake >/dev/null 2>&1 || { echo >&2 "Please install cmake with 'sudo apt-get install cmake' or 'brew install cmake'"; exit 1; }
+	@command -v cmake >/dev/null 2>&1 || { echo -e >&2 "Cmake is not installed. Please install cmake to continue.\nOn Debian / Ubuntu: 'sudo apt-get install cmake' or 'brew install cmake'\nOn Centos: 'sudo yum install cmake'"; exit 1; }
 
 check-python:
-	@command -v python3 >/dev/null 2>&1 || { echo >&2 "Python needs to be installed and in your path."; exit 1; } 
+	@command -v python3 >/dev/null 2>&1 || { echo -e >&2 "Python is not installed. Please install python to continue.\nOn Debian / Ubuntu: 'sudo apt-get install python'\nOn Centos: 'sudo yum install python'"; exit 1; } 
 
 check-linux-vulkan:
 	@echo "Checking system type and Vulkan availability..."
@@ -156,7 +151,7 @@ check-linux-vulkan:
 	        echo "Vulkan is installed."; \
 	        vulkaninfo; \
 	    else \
-		echo "Vulkan is not installed. Please install Vulkan drivers to continue. On Debian / Ubuntu: sudo apt install libvulkan1 mesa-vulkan-drivers vulkan-tools"; \
+		echo -e "Vulkan is not installed. Please install Vulkan drivers to continue.\nOn Debian / Ubuntu: 'sudo apt install libvulkan1 mesa-vulkan-drivers vulkan-tools'.\nOn Centos: 'sudo yum install vulkan vulkan-tools.'"; \
 	        exit 1; \
 	    fi \
 	else \