Made NBGraphics installable

CMakeLists.txt

@@ -7,6 +7,9 @@ set(CMAKE_CXX_STANDARD 17)
 set(CMAKE_CXX_STANDARD_REQUIRED ON)
 set(CMAKE_CXX_EXTENSIONS OFF)
 
+include(GNUInstallDirs)
+include(CMakePackageConfigHelpers)
+
 function(toAbsolutePath SETVAR PATHS)
     set(PATHS ${PATHS} ${ARGN})
     set(RET_VAL "")
@@ -31,6 +34,7 @@ elseif(CMAKE_BUILD_TYPE STREQUAL "Debug")
     set(NB_LOGGING ON)
     set(NB_BUILD_TESTS ON)
     set(NB_BUILD_DOCS ON)
+    set(NBENGINE_INSTALL ON)
     add_compile_definitions(_NB_BUILD_DEBUG)
 endif()
 
@@ -55,7 +59,6 @@ if(NB_BUILD_TESTS)
         FetchContent_MakeAvailable(gtest)
     endif()
     set(GTEST_COLOR ON)
-    set(GLFW_BUILD_TESTS ON CACHE BOOL "" FORCE)
 endif()
 
 if (NB_LOGGING)
@@ -70,13 +73,7 @@ elseif (NB_TARGET_LINUX)
     add_compile_definitions(_NB_TARGET_LINUX)
 endif()
 
-# External Dep paths
-set(GLFW_PATH ../glfw/)
-set(GLAD_PATH ../glad/)
-set(STBIMAGE_PATH ../stbi_image)
-get_filename_component(GLFW_PATH ${GLFW_PATH} ABSOLUTE)
-get_filename_component(GLAD_PATH ${GLAD_PATH} ABSOLUTE)
-get_filename_component(STBIMAGE_PATH ${STBIMAGE_PATH} ABSOLUTE)
+get_filename_component(NBENGINE_ROOT ./ ABSOLUTE)
 
 add_subdirectory(./engine)
 
@@ -86,3 +83,23 @@ set(NB_REBUILD_DOCS)
 if (NB_BUILD_DOCS)
     add_subdirectory(./docs)
 endif()
+
+if (NBENGINE_INSTALL)
+    include(GNUInstallDirs)
+    include(CMakePackageConfigHelpers)
+    configure_package_config_file(
+        "NBEngineConfig.cmake.in"
+        "NBEngineConfig.cmake"
+        INSTALL_DESTINATION ${CMAKE_INSTALL_PREFIX}/cmake
+        PATH_VARS CMAKE_INSTALL_LIBDIR
+    )
+    write_basic_package_version_file(
+        "${CMAKE_CURRENT_BINARY_DIR}/NBEngineConfigVersion.cmake"
+        COMPATIBILITY AnyNewerVersion 
+    )
+    install(FILES
+        "${CMAKE_CURRENT_BINARY_DIR}/NBEngineConfig.cmake"
+        "${CMAKE_CURRENT_BINARY_DIR}/NBEngineConfigVersion.cmake"
+        DESTINATION "${CMAKE_INSTALL_PREFIX}/cmake"
+    )
+endif()

NBEngineConfig.cmake.in

@@ -0,0 +1,8 @@
+@PACKAGE_INIT@
+set(NBENGINE_PACKAGES
+    NBCore
+    NBGraphics
+)
+foreach(NBENGINE_LIB ${NBENGINE_PACKAGES})
+    include("${CMAKE_CURRENT_LIST_DIR}/${NBENGINE_LIB}Config.cmake")
+endforeach()

engine/CMakeLists.txt

@@ -1,6 +1,8 @@
-include_directories(./.)
-
 add_subdirectory(./NBCore)
+
+get_target_property(NBCORE_INTERFACE_INCLUDES NBCore INTERFACE_INCLUDE_DIRECTORIES)
+include_directories(${NBCORE_INTERFACE_INCLUDES})
+
 add_subdirectory(./NBEvents)
 add_subdirectory(./NBGraphics)
 
@@ -10,7 +12,6 @@ if (NB_CORE_SOURCE OR NB_EVENTS_SOURCE OR NB_GRAPHICS_SOURCE)
     set(NB_SOURCE_FILES ${NB_SOURCE_FILES} PARENT_SCOPE)
 endif()
 
-
 if (NB_CORE_INCLUDE OR NB_EVENTS_INCLUDE OR NB_GRAPHICS_INCLUDE)
     set(NB_INCLUDE_FILES "")
     list(APPEND NB_INCLUDE_FILES ${NB_CORE_INCLUDE} ${NB_GRAPHICS_INCLUDE} ${NB_EVENTS_INCLUDE})

engine/NBCore/CMakeLists.txt

@@ -1,3 +1,6 @@
+cmake_minimum_required(VERSION 3.10)
+project(NBCore VERSION 0.1)
+
 toAbsolutePath(NB_CORE_SOURCE 
     ./src/ErrorsImpl.cpp
     ./src/Logger.cpp
@@ -5,27 +8,64 @@ toAbsolutePath(NB_CORE_SOURCE
     ./src/StringUtils.cpp
     ./src/Utils.cpp
 )
-
 toAbsolutePath(NB_CORE_INCLUDE
-    ./ANSITerm.hpp
-    ./DataSink.hpp
-    ./Errors.hpp
-    ./ErrorsImpl.hpp
-    ./Logger.hpp
-    ./Processes.hpp
-    ./StringUtils.hpp
-    ./ThreadsafeQueue.hpp
-    ./Types.hpp
-    ./TypeTraits.hpp
-    ./Utils.hpp
+    ./include/NBCore/ANSITerm.hpp
+    ./include/NBCore/DataSink.hpp
+    ./include/NBCore/Errors.hpp
+    ./include/NBCore/ErrorsImpl.hpp
+    ./include/NBCore/Logger.hpp
+    ./include/NBCore/Processes.hpp
+    ./include/NBCore/StringUtils.hpp
+    ./include/NBCore/ThreadsafeQueue.hpp
+    ./include/NBCore/Types.hpp
+    ./include/NBCore/TypeTraits.hpp
+    ./include/NBCore/Utils.hpp
 )
 
 set(NB_CORE_SOURCE ${NB_CORE_SOURCE} PARENT_SCOPE)
 set(NB_CORE_INCLUDE ${NB_CORE_INCLUDE} PARENT_SCOPE)
 
 add_library(NBCore ${NB_CORE_SOURCE})
+add_library(NBEngine::Core ALIAS NBCore)
+target_include_directories(NBCore
+    PUBLIC "$<BUILD_INTERFACE:${PROJECT_SOURCE_DIR}/include>"
+    PUBLIC "$<INSTALL_INTERFACE:${CMAKE_INSTALL_INCLUDEDIR}>"
+)
 
-target_include_directories(NBCore PUBLIC ./.)
+if (NBENGINE_INSTALL)
+    message("Installing NBCore to ${CMAKE_INSTALL_PREFIX}")
+    include(GNUInstallDirs)
+    include(CMakePackageConfigHelpers)
+    install(
+        TARGETS NBCore
+        EXPORT NBCoreTargets
+    )
+    install(
+        DIRECTORY "${PROJECT_SOURCE_DIR}/include/NBCore"
+        DESTINATION "${CMAKE_INSTALL_INCLUDEDIR}"
+        FILES_MATCHING PATTERN "*.h" PATTERN "*.hpp"
+    )
+    install(
+        EXPORT NBCoreTargets
+        DESTINATION ${CMAKE_INSTALL_PREFIX}/cmake
+        NAMESPACE NBEngine::
+    )
+    configure_package_config_file(
+        "NBCoreConfig.cmake.in"
+        "NBCoreConfig.cmake"
+        INSTALL_DESTINATION ${CMAKE_INSTALL_PREFIX}/cmake
+        PATH_VARS CMAKE_INSTALL_LIBDIR
+    )
+    write_basic_package_version_file(
+        "${CMAKE_CURRENT_BINARY_DIR}/NBCoreConfigVersion.cmake"
+        COMPATIBILITY AnyNewerVersion 
+    )
+    install(FILES
+        "${CMAKE_CURRENT_BINARY_DIR}/NBCoreConfig.cmake"
+        "${CMAKE_CURRENT_BINARY_DIR}/NBCoreConfigVersion.cmake"
+        DESTINATION "${CMAKE_INSTALL_PREFIX}/cmake"
+    )
+endif()
 
 if (NB_BUILD_TESTS)
     add_subdirectory(./tests)

engine/NBCore/NBCoreConfig.cmake.in

@@ -0,0 +1,2 @@
+@PACKAGE_INIT@
+include("${CMAKE_CURRENT_LIST_DIR}/NBCoreTargets.cmake")

engine/NBCore/include/NBCore/DataSink.hpp

@@ -5,7 +5,7 @@
 #include <atomic>
 #include <thread>
 
-#include "ThreadSafeQueue.hpp"
+#include <NBCore/ThreadSafeQueue.hpp>
 
 namespace nb {
 

engine/NBCore/include/NBCore/Errors.hpp

@@ -2,8 +2,8 @@
 #ifndef _NB_ERROR
 #define _NB_ERROR
 
-#include "ErrorsImpl.hpp"
-#include "Logger.hpp"
+#include <NBCore/ErrorsImpl.hpp>
+#include <NBCore/Logger.hpp>
 
 namespace nb {
 

engine/NBCore/include/NBCore/ErrorsImpl.hpp

@@ -8,7 +8,7 @@
 #include <type_traits>
 #include <unordered_map>
 
-#include "StringUtils.hpp"
+#include <NBCore/StringUtils.hpp>
 
 namespace nb {
 
@@ -25,14 +25,15 @@ class ErrorBase {
 
     ErrorBase(const ErrorBase&) = default;
     ErrorBase(const std::exception&) noexcept;
+    ErrorBase(const std::string&) noexcept;
     virtual std::string what() const noexcept {
         std::string ret = msg;
         if (trace) {
             std::string trace_msg = msg;
-            ret += nb::NEWLINE + indent_strblock(
+            ret += NEWLINE + indent_strblock(
                 trace_msg,
-                nb::TABOVER,
-                nb::TABOVER+"Trace: "
+                TABOVER,
+                TABOVER+"Trace: "
             );
         }
         return ret;
@@ -52,7 +53,7 @@ class ErrorBase {
 {}
 };
 
-template <class ErrorType>
+template <class ErrorType=NoneType>
 class Error : public ErrorBase {
     private:
     void inline check_asserts() {
@@ -132,39 +133,43 @@ class Error : public ErrorBase {
     using ErrorBase::type;
 };
 
-class NBError : public Error<NBError> {
+template<>
+class Error<NoneType> : public Error<Error<NoneType>> {
     protected:
-    using Base = Error<NBError>;
+    using Base = Error<Error<NoneType>>;
     
     public:
+    using Base::what;
+    using Base::code;
+    using Base::trace;
     enum Codes : unsigned int {
         STANDARD, UNDEFINED, INDEX_ERROR
     };
-
-    NBError(unsigned int code_=1) noexcept : Base(code_) {}
-    NBError(const std::exception& err) : Base(
-        STANDARD,
-        std::string(err.what()),
-        "std::exception",
-        nullptr
-    ) {}
-
-    // fix
-    NBError(std::string msg_) noexcept : Base(
-        Codes::UNDEFINED, 
-        msg_
-    ) {}
-
     inline static const std::string type = "nb::Error";
     inline static const ErrorCodeMap ErrorMessages = {
         {STANDARD, "std::exception"},
         {UNDEFINED, "Error"},
         {INDEX_ERROR, "Indexing error"}
     };
+
+    Error(unsigned int code_=1) noexcept : Base(code_) {}
+    Error(const std::exception& err) : Base(
+        STANDARD,
+        std::string(err.what()),
+        "std::exception",
+        nullptr
+    ) {}
+
+    Error(const std::string& msg_) noexcept : Base(
+        Codes::UNDEFINED, 
+        msg_,
+        type,
+        nullptr
+    ) {}
 };
 
-// template<typename... Args>
-// Error(Args...) -> Error<NoneType>;
+template<typename... Args>
+Error(Args...) -> Error<NoneType>;
 
 } // namespace nb
 

engine/NBCore/include/NBCore/Logger.hpp

@@ -8,11 +8,11 @@
 #include <unordered_map>
 #include <vector>
 
-#include "DataSink.hpp"
-#include "ErrorsImpl.hpp"
-#include "Processes.hpp"
-#include "ThreadSafeQueue.hpp"
-#include "TypeTraits.hpp"
+#include <NBCore/DataSink.hpp>
+#include <NBCore/ErrorsImpl.hpp>
+#include <NBCore/Processes.hpp>
+#include <NBCore/ThreadSafeQueue.hpp>
+#include <NBCore/TypeTraits.hpp>
 
 namespace nb {
 
@@ -99,6 +99,14 @@ public:
         static_cast<LoggerType*>(this)->write_message(val, 0xFF, file, line);
         static_cast<LoggerType*>(this)->flush();
     }
+    void error(
+        const std::string& val,
+        std::string file="",
+        unsigned int line=0
+    ) {
+        static_cast<LoggerType*>(this)->write_message(Error(val), 0xFF, file, line);
+        static_cast<LoggerType*>(this)->flush();
+    }
 
     protected:
     std::vector<std::ostream*> _ostream;

engine/NBCore/include/NBCore/StringUtils.hpp

@@ -6,17 +6,17 @@
 #include <string>
 #include <string_view>
 
-#include "TypeTraits.hpp"
+#include <NBCore/TypeTraits.hpp>
 
 namespace nb {
 
 #ifdef _NB_TARGET_WINDOWS
-    const std::string NEWLINE = "\n";
-    const std::wstring WNEWLINE = L"\n";
+    inline static const std::string NEWLINE = "\n";
+    inline static const std::wstring WNEWLINE = L"\n";
 #endif // _NB_TARGET_WINDOWS
 #ifdef _NB_TARGET_LINUX
-    const std::string NEWLINE = "\n";
-    const std::wstring WNEWLINE = L"\n";
+    inline const std::string NEWLINE = "\n";
+    inline const std::wstring WNEWLINE = L"\n";
 #endif // _NB_TARGET_LINUX
 
 const std::string TABOVER = "   ";
@@ -34,7 +34,7 @@ void stream(const Stream& s, Args&&... args) {
 }
 
 template<typename... Args>
-void term(Args&&... args) { stream(std::cout, args..., nb::NEWLINE); }
+void term(Args&&... args) { stream(std::cout, args..., NEWLINE); }
 
 template<typename... Args>
 void wterm(Args&&... args) { stream(std::wcout, args..., nb::WNEWLINE); }

engine/NBCore/include/NBCore/Utils.hpp

@@ -5,7 +5,7 @@
 #include <mutex>
 #include <vector>
 
-#include "Errors.hpp"
+#include <NBCore/Errors.hpp>
 
 namespace nb {
 

engine/NBCore/src/ErrorsImpl.cpp

@@ -1,10 +1,10 @@
-#include "ErrorsImpl.hpp"
+#include <NBCore/ErrorsImpl.hpp>
 
 namespace nb {
 
 //ErrorBase::
 ErrorBase::ErrorBase(const std::exception& exception_) noexcept 
-: ErrorBase(NBError(exception_)) {}
+: ErrorBase(Error(exception_)) {}
 
 } // namespace nb
 

engine/NBCore/src/Logger.cpp

@@ -1,7 +1,7 @@
 #include <iostream>
 
-#include "Logger.hpp"
-#include "StringUtils.hpp"
+#include <NBCore/Logger.hpp>
+#include <NBCore/StringUtils.hpp>
 
 namespace nb {
 

engine/NBCore/src/Processes.cpp

@@ -6,7 +6,7 @@
 #include <unistd.h>
 #endif // _NB_TARGET_LINUX
 
-#include "Processes.hpp"
+#include <NBCore/Processes.hpp>
 
 namespace nb {
 

engine/NBCore/src/StringUtils.cpp

@@ -1,4 +1,4 @@
-#include "StringUtils.hpp"
+#include <NBCore/StringUtils.hpp>
 
 /* std::string nb::wstr_to_str(std::wstring in) {
     std::size_t wstrlen = in.length();

engine/NBCore/src/Utils.cpp

@@ -1,4 +1,4 @@
-#include "Utils.hpp"
+#include <NBCore/Utils.hpp>
 
 namespace nb {
 

engine/NBCore/tests/testErrors.cpp

@@ -1,10 +1,10 @@
 #define CODE_ERROR_LOCATIONS
 
 #include <gtest/gtest.h>
-
-#include "Errors.hpp"
 #include <string>
 
+#include <NBCore/Errors.hpp>
+
 using namespace nb;
 
 class TestError : public Error<TestError> {

engine/NBCore/tests/testLogger.cpp

@@ -1,7 +1,7 @@
 #include <exception>
 
-#include "Errors.hpp"
-#include "Logger.hpp"
+#include <NBCore/Errors.hpp>
+#include <NBCore/Logger.hpp>
 
 class TestError : public nb::Error<TestError> {
     using Base = Error<TestError>;

engine/NBCore/tests/testProcesses.cpp

@@ -1,7 +1,7 @@
 #define CODE_ERROR_LOCATIONS
 
 #include <gtest/gtest.h>
-#include "Processes.hpp"
+#include <NBCore/Processes.hpp>
 #include <Windows.h>
 
 TEST(ProcessesTest, GetPID) {

engine/NBCore/tests/testUtils.cpp

@@ -3,8 +3,8 @@
 #include <gtest/gtest.h>
 
 #include <string>
-#include "TypeTraits.hpp"
-#include "StringUtils.hpp"
+#include <NBCore/TypeTraits.hpp>
+#include <NBCore/StringUtils.hpp>
 
 namespace nb {
 

engine/NBEvents/src/Events.cpp

@@ -1,4 +1,4 @@
-#include "Events.hpp"
+#include <NBCore/Events.hpp>
 
 namespace NB {
 

engine/NBGraphics/CMakeLists.txt

@@ -1,14 +1,24 @@
-find_package(OpenGL)
-add_subdirectory(${GLFW_PATH} ${GLFW_PATH}/build)
+cmake_minimum_required(VERSION 3.10)
+project(NBGraphics VERSION 0.1)
 
-include_directories(
-    ${GLFW_PATH}/include
-    ${GLAD_PATH}/include
-    ${STBIMAGE_PATH}
+set(GLAD_PATH ${NBENGINE_ROOT}/../glad/)
+set(STBIMAGE_PATH ${NBENGINE_ROOT}/../stbi_image)
+get_filename_component(GLAD_PATH ${GLAD_PATH} ABSOLUTE)
+get_filename_component(STBIMAGE_PATH ${STBIMAGE_PATH} ABSOLUTE)
+
+set(CMAKE_PREFIX_PATH
+    "${CMAKE_PREFIX_PATH}"
+    "C:/Program Files (x86)/GLFW/lib/cmake/glfw3/"    
 )
+find_package(OpenGL REQUIRED)
+find_package(glfw3 REQUIRED)
 
 set(GLFW_BUILD_DOCS OFF CACHE BOOL "" FORCE)
-set(GLFW_BUILD_TESTS OFF CACHE BOOL "" FORCE)
+if (NB_BUILD_TESTS)
+    set(GLFW_BUILD_TESTS ON CACHE BOOL "" FORCE)
+else()
+    set(GLFW_BUILD_TESTS OFF CACHE BOOL "" FORCE)
+endif()
 set(GLFW_BUILD_EXAMPLES OFF CACHE BOOL "" FORCE)
 
 toAbsolutePath(NB_GRAPHICS_SOURCE
@@ -21,19 +31,18 @@ toAbsolutePath(NB_GRAPHICS_SOURCE
     ./src/VertexArray.cpp
     ./src/Window.cpp
 )
-
 toAbsolutePath(NB_GRAPHICS_INCLUDE
-    ./Buffers.hpp
-    ./Camera.hpp
-    ./Draw.hpp
-    ./FrameBuffers.hpp
-    ./GLLoad.hpp
-    ./Image.hpp
-    ./OGLObjects.hpp
-    ./ProgramPipeline.hpp
-    ./Textures.hpp
-    ./VertexArray.hpp
-    ./Window.hpp
+    ./include/NBGraphics/Buffers.hpp
+    ./include/NBGraphics/Camera.hpp
+    ./include/NBGraphics/Draw.hpp
+    ./include/NBGraphics/FrameBuffers.hpp
+    ./include/NBGraphics/GLLoad.hpp
+    ./include/NBGraphics/Image.hpp
+    ./include/NBGraphics/OGLObjects.hpp
+    ./include/NBGraphics/ProgramPipeline.hpp
+    ./include/NBGraphics/Textures.hpp
+    ./include/NBGraphics/VertexArray.hpp
+    ./include/NBGraphics/Window.hpp
 )
 
 set(NB_GRAPHICS_SOURCE ${NB_GRAPHICS_SOURCE} PARENT_SCOPE)
@@ -43,9 +52,54 @@ add_library(NBGraphics
     ${NB_GRAPHICS_SOURCE}
     ${GLAD_PATH}/src/glad.c
 )
-target_link_libraries(NBGraphics glfw)
+add_library(NBEngine::Graphics ALIAS NBGraphics)
+target_link_libraries(NBGraphics
+    PUBLIC glfw
+    PUBLIC NBCore    
+)
+get_target_property(GLFW_INTERFACE_INCLUDES glfw INTERFACE_INCLUDE_DIRECTORIES)
+#include_directories(${GLFW_INTERFACE_INCLUDES})
+target_include_directories(NBGraphics 
+    PUBLIC "$<BUILD_INTERFACE:${PROJECT_SOURCE_DIR}/include>"
+    PUBLIC "$<INSTALL_INTERFACE:${CMAKE_INSTALL_INCLUDEDIR}>"
+    PUBLIC "${STBIMAGE_PATH}"
+    PUBLIC "${GLAD_PATH}/include"
+)
 
-target_include_directories(NBGraphics PUBLIC ./.)
+if (NBENGINE_INSTALL)
+    message("Installing NBGraphics to ${CMAKE_INSTALL_PREFIX}")
+    include(GNUInstallDirs)
+    include(CMakePackageConfigHelpers)
+    install(
+        TARGETS NBGraphics
+        EXPORT NBGraphicsTargets
+    )
+    install(
+        DIRECTORY "${PROJECT_SOURCE_DIR}/include/NBGraphics"
+        DESTINATION "${CMAKE_INSTALL_INCLUDEDIR}"
+        FILES_MATCHING PATTERN "*.h" PATTERN "*.hpp"
+    )
+    install(
+        EXPORT NBGraphicsTargets
+        DESTINATION ${CMAKE_INSTALL_PREFIX}/cmake
+        NAMESPACE NBEngine::
+    )
+    configure_package_config_file(
+        "NBGraphicsConfig.cmake.in"
+        "NBGraphicsConfig.cmake"
+        INSTALL_DESTINATION ${CMAKE_INSTALL_PREFIX}/cmake
+        PATH_VARS CMAKE_INSTALL_LIBDIR
+    )
+    write_basic_package_version_file(
+        "${CMAKE_CURRENT_BINARY_DIR}/NBGraphicsConfigVersion.cmake"
+        COMPATIBILITY AnyNewerVersion 
+    )
+    install(FILES
+        "${CMAKE_CURRENT_BINARY_DIR}/NBGraphicsConfig.cmake"
+        "${CMAKE_CURRENT_BINARY_DIR}/NBGraphicsConfigVersion.cmake"
+        DESTINATION "${CMAKE_INSTALL_PREFIX}/cmake"
+    )
+endif()
 
 if (NB_BUILD_TESTS)
     add_subdirectory(./tests)

engine/NBGraphics/NBGraphicsConfig.cmake.in

@@ -0,0 +1,2 @@
+@PACKAGE_INIT@
+include("${CMAKE_CURRENT_LIST_DIR}/NBGraphicsTargets.cmake")

engine/NBGraphics/include/NBGraphics/Buffers.hpp

@@ -2,7 +2,7 @@
 #ifndef _NB_BUFFER
 #define _NB_BUFFER
 
-#include "GLLoad.hpp"
+#include <NBGraphics/GLLoad.hpp>
 
 #include <string>
 
@@ -10,7 +10,7 @@
 #include <NBCore/Logger.hpp>
 #include <NBCore/Utils.hpp>
 
-#include "OGLObjects.hpp"
+#include <NBGraphics/OGLObjects.hpp>
 
 namespace nb {
 

engine/NBGraphics/include/NBGraphics/Camera.hpp

@@ -2,7 +2,7 @@
 #ifndef _NB_CAMERA
 #define _NB_CAMERA
 
-#include "GLLoad.hpp"
+#include <NBGraphics/GLLoad.hpp>
 
 #include <glm/glm.hpp>
 #include <glm/gtc/matrix_inverse.hpp>

engine/NBGraphics/include/NBGraphics/Draw.hpp

@@ -2,16 +2,16 @@
 #ifndef _NB_DRAW
 #define _NB_DRAW
 
-#include <GLLoad.h>
+#include <NBGraphics/GLLoad.hpp>
 
 #include <exception>
 #include <map>
 #include <string>
 #include <vector>
 
-#include "Buffers.h"
-#include "Shader.h"
-#include "VAOManager.hpp"
+#include <NBGraphics/Buffers.h>
+#include <NBGraphics/Shader.h>
+#include <NBGraphics/VAOManager.hpp>
 
 #define THROW_DRAW_ERROR(msg) throw DrawError(msg, __FILE__, __LINE__);
 

engine/NBGraphics/include/NBGraphics/FrameBuffers.hpp

@@ -2,13 +2,13 @@
 #ifndef _NB_FRAMEBUFFER
 #define _NB_FRAMEBUFFER
 
-#include "GLLoad.hpp"
+#include <NBGraphics/GLLoad.hpp>
 
 #include <NBCore/Errors.hpp>
 
-#include "Image.hpp"
-#include "OGLObjects.hpp"
-#include "Textures.hpp"
+#include <NBGraphics/Image.hpp>
+#include <NBGraphics/OGLObjects.hpp>
+#include <NBGraphics/Textures.hpp>
 
 namespace nb {
 

engine/NBGraphics/include/NBGraphics/OGLObjects.hpp

@@ -2,7 +2,7 @@
 #ifndef _NB_OGL_OBJECTS
 #define _NB_OGL_OBJECTS
 
-#include "GLLoad.hpp"
+#include <NBGraphics/GLLoad.hpp>
 
 #include <NBCore/Errors.hpp>
 

engine/NBGraphics/include/NBGraphics/ProgramPipeline.hpp

@@ -2,14 +2,14 @@
 #ifndef _NB_SHADER
 #define _NB_SHADER
 
-#include "GLLoad.hpp"
+#include <NBGraphics/GLLoad.hpp>
 
 #include <string>
 
 #include <NBCore/Errors.hpp>
 #include <NBCore/Utils.hpp>
 
-#include "OGLObjects.hpp"
+#include <NBGraphics/OGLObjects.hpp>
 
 namespace nb {
 

engine/NBGraphics/include/NBGraphics/Textures.hpp

@@ -2,9 +2,9 @@
 #ifndef _NB_TEXTURES
 #define _NB_TEXTURES
 
-#include "Buffers.hpp"
-#include "Image.hpp"
-#include "OGLObjects.hpp"
+#include <NBGraphics/Buffers.hpp>
+#include <NBGraphics/Image.hpp>
+#include <NBGraphics/OGLObjects.hpp>
 
 namespace nb {
 

engine/NBGraphics/include/NBGraphics/VertexArray.hpp

@@ -2,12 +2,12 @@
 #ifndef _NB_VERTEX_ARRAY
 #define _NB_VERTEX_ARRAY
 
-#include "GLLoad.hpp"
+#include <NBGraphics/GLLoad.hpp>
 
 #include <NBCore/Errors.hpp>
 
-#include "Buffers.hpp"
-#include "OGLObjects.hpp"
+#include <NBGraphics/Buffers.hpp>
+#include <NBGraphics/OGLObjects.hpp>
 
 
 namespace nb {

engine/NBGraphics/include/NBGraphics/Window.hpp

@@ -2,11 +2,10 @@
 #ifndef _NB_WINDOW
 #define _NB_WINDOW
 
-#include "GLLoad.hpp"
+#include <NBGraphics/GLLoad.hpp>
 
 #include <array>
 #include <map>
-#include <stdexcept>
 #include <string>
 
 #include <NBCore/Errors.hpp>

engine/NBGraphics/src/Buffers.cpp

@@ -1,4 +1,4 @@
-#include "Buffers.hpp"
+#include <NBGraphics/Buffers.hpp>
 
 namespace nb {
 

engine/NBGraphics/src/Camera.cpp

@@ -1,4 +1,4 @@
-#include "Camera.hpp"
+#include <NBGraphics/Camera.hpp>
 
 namespace nb {
 

engine/NBGraphics/src/Draw.cpp

@@ -1,4 +1,4 @@
-#include "Draw.h"
+#include <NBGraphics/Draw.hpp>
 
 namespace NB{
 

engine/NBGraphics/src/FrameBuffers.cpp

@@ -1,4 +1,4 @@
-#include "FrameBuffers.hpp"
+#include <NBGraphics/FrameBuffers.hpp>
 
 namespace nb {
 

engine/NBGraphics/src/Image.cpp

@@ -1,4 +1,4 @@
-#include "Image.hpp"
+#include <NBGraphics/Image.hpp>
 
 using ImageErrorCodes = nb::ImageError::Codes;
 const std::string nb::ImageError::type = "nb::ImageError";

engine/NBGraphics/src/OGLObjects.cpp

@@ -1,4 +1,4 @@
-#include "OGLObjects.hpp"
+#include <NBGraphics/OGLObjects.hpp>
 
 namespace nb {
 

engine/NBGraphics/src/ProgramPipeline.cpp

@@ -1,4 +1,4 @@
-#include "ProgramPipeline.hpp"
+#include <NBGraphics/ProgramPipeline.hpp>
 
 namespace nb{
 

engine/NBGraphics/src/Textures.cpp

@@ -1,4 +1,4 @@
-#include "Textures.hpp"
+#include <NBGraphics/Textures.hpp>
 
 namespace nb {
 

engine/NBGraphics/src/VertexArray.cpp

@@ -1,4 +1,4 @@
-#include "VertexArray.hpp"
+#include <NBGraphics/VertexArray.hpp>
 
 namespace nb {
 
@@ -177,7 +177,7 @@ VertexDataVec VertexGroup::dropBuffers() {
 
 VertexData VertexGroup::dropBuffer(size_t idx) {
     if (idx >= _vertex_data.size()) {
-        THROW(NBError(NBError::INDEX_ERROR));
+        THROW(Error(Error<>::INDEX_ERROR));
     }
     VertexDataVec tmp;
     VertexData ret;

engine/NBGraphics/src/Window.cpp

@@ -1,4 +1,4 @@
-#include "Window.hpp"
+#include <NBGraphics/Window.hpp>
 
 namespace nb {
 

engine/NBGraphics/tests/CMakeLists.txt

@@ -8,7 +8,6 @@ if (NB_BUILD_TESTS)
         ./TestWindow.cpp
     )
     target_link_libraries(TestWindow
-        NBCore
         NBGraphics
     )
 

engine/NBGraphics/tests/TestWindow.cpp

@@ -1,12 +1,12 @@
-#include "GLLoad.hpp"
+#include <NBGraphics/GLLoad.hpp>
 #define STB_IMAGE_IMPLEMENTATION
 #include "stb_image.h"
 
-#include "Image.hpp"
-#include "ProgramPipeline.hpp"
-#include "VertexArray.hpp"
-#include "Window.hpp"
-#include "Textures.hpp"
+#include <NBGraphics/Image.hpp>
+#include <NBGraphics/ProgramPipeline.hpp>
+#include <NBGraphics/VertexArray.hpp>
+#include <NBGraphics/Window.hpp>
+#include <NBGraphics/Textures.hpp>
 
 int main() {
     nb::logger.log("Howdy!");

engine/NBGraphics/tests/testImages.cpp

@@ -5,7 +5,7 @@
 #define STB_IMAGE_IMPLEMENTATION
 #include "stb_image.h"
 
-#include "Image.hpp"
+#include <NBGraphics/Image.hpp>
 
 TEST(TestImage, SinglePixel) {
     using RGB = nb::Pixel<uint8_t, nb::Red, nb::Green, nb::Blue>;