Trying to fix windows

.github/workflows/all-builds.yml

@@ -10,14 +10,15 @@ jobs:
     strategy:
       fail-fast: false
       matrix:
-        os: [ubuntu-latest, windows-latest, macos-latest]
+#        os: [ubuntu-latest, windows-latest, macos-latest]
+        os: [windows-latest]
         include:
-          - os: ubuntu-latest
+#          - os: ubuntu-latest
-            cmake: -DCMAKE_PREFIX_PATH="/home/runner/work/qtk/Qt/6.5.0/gcc_64/"
+#            cmake: -DCMAKE_PREFIX_PATH="/home/runner/work/qtk/Qt/6.5.0/gcc_64/"
           - os: windows-latest
             cmake: -DCMAKE_PREFIX_PATH="D:/a/qtk/qtk/Qt/6.5.0/mingw81_64/"
-          - os: macos-latest
+#          - os: macos-latest
-            cmake: -DCMAKE_PREFIX_PATH="/home/runner/work/qtk/Qt/6.5.0/gcc_64/"
+#            cmake: -DCMAKE_PREFIX_PATH="/home/runner/work/qtk/Qt/6.5.0/gcc_64/"
 
     runs-on: ${{ matrix.os }}
     steps:
@@ -35,15 +36,23 @@ jobs:
           args: install pkgconfiglite --checksum e87b5ea3c9142256af60f2d5b917aa63b571e6a0 --checksum-type sha1
 
       - name: Build Qtk
+        if: matrix.os == 'windows-latest'
         shell: bash
         run: |
-          cmake -B build/ ${{ matrix.cmake }} -DQTK_UPDATE_SUBMODULES=ON
+          cmake -B build/ ${{ matrix.cmake }} -DQTK_UPDATE_SUBMODULES=ON -DQTK_DEBUG=OFF
-          cmake --build build/ -- -j $(nproc)
+          cmake --build build/ --config Release
+
+      - name: Build Qtk
+        if: matrix.os != 'windows-latest'
+        shell: bash
+        run: |
+          cmake -B build/ ${{ matrix.cmake }} -DQTK_UPDATE_SUBMODULES=ON -DQTK_DEBUG=OFF
+          cmake --build build/ --config Release -- -j $(nproc)
 
       - name: Install Qtk
         shell: bash
         run: |
-          cmake --install build/
+          cmake --install build/ --config Release
 
       - uses: actions/upload-artifact@v3
         if: always()
@@ -52,10 +61,18 @@ jobs:
           path: install/*
 
       - name: Package Qtk
+        if: matrix.os == 'windows-latest'
         shell: bash
         run: |
           cd build/
-          cmake --build . --target package -- -j $(nproc)
+          cmake --build . --target package --config Release
+
+      - name: Package Qtk
+        if: matrix.os != 'windows-latest'
+        shell: bash
+        run: |
+          cd build/
+          cmake --build . --target package --config Release -- -j $(nproc)
 
 #      - name: Package Qtk Sources
 #        if: matrix.os != 'macos-latest'
@@ -104,9 +121,18 @@ jobs:
           sudo apt install libassimp-dev
 
       - name: Build Qtk
+        if: matrix.os == 'windows-latest'
         shell: bash
         run: |
-          cmake -B build/ ${{ matrix.cmake }} && cmake --build build/ -- -j $(nproc)
+          cmake -B build/ ${{ matrix.cmake }} -DQTK_DEBUG=OFF
+          cmake --build build/ --config Release
+
+      - name: Build Qtk
+        if: matrix.os != 'windows-latest'
+        shell: bash
+        run: |
+          cmake -B build/ ${{ matrix.cmake }} -DQTK_DEBUG=OFF
+          cmake --build build/ --config Release -- -j $(nproc)
 
   # TODO: Enable with tag only when done testing
 #  Release-Qtk:

CMakeLists.txt

@@ -22,12 +22,11 @@ set(CMAKE_AUTORCC ON)
 set(CMAKE_CXX_STANDARD 17)
 set(CMAKE_CXX_STANDARD_REQUIRED ON)
 set(CMAKE_EXPORT_COMPILE_COMMANDS ON)
-#set(CMAKE_MACOSX_BUNDLE ON)
+set(CMAKE_MACOSX_BUNDLE ON)
 set(CMAKE_RUNTIME_OUTPUT_DIRECTORY ${CMAKE_BINARY_DIR}/bin)
 
-if(MSVC)
+if(WIN32)
   set(CMAKE_COMPILE_WARNING_AS_ERROR OFF)
-  add_compile_options(-warnnotaserror)
   add_compile_options(/wd4131 /wd4127)
 endif()
 
@@ -58,6 +57,12 @@ option(
     OFF
 )
 
+if(NOT QTK_DEBUG)
+  set(CMAKE_BUILD_TYPE Release)
+else()
+  set(CMAKE_BUILD_TYPE Debug)
+endif()
+
 set(QT_DIR "$ENV{HOME}/Qt/" CACHE PATH "Path to Qt6")
 
 # Qt Designer will look in different locations if WIN / Unix.
@@ -116,10 +121,11 @@ if(NOT Qt6_FOUND)
 endif()
 
 # Find Assimp
-set(ASSIMP_BUILD_TESTS OFF)
+set(ASSIMP_BUILD_TESTS OFF CACHE STRING "Do not build assimp tests." FORCE)
 if(QTK_UPDATE_SUBMODULES)
   message(STATUS "[Qtk] Updating submodules...")
-  set(ASSIMP_INSTALL OFF)
+  set(ASSIMP_INSTALL OFF CACHE STRING "Use assimp as a submodule." FORCE)
+  set(ASSIMP_WARNINGS_AS_ERRORS OFF CACHE STRING "No warnings as errors." FORCE)
   submodule_update("${CMAKE_CURRENT_SOURCE_DIR}/extern/assimp/assimp/")
   add_subdirectory("${CMAKE_CURRENT_SOURCE_DIR}/extern/assimp/assimp/")
 else()

example-app/CMakeLists.txt

@@ -47,12 +47,14 @@ if(NOT TARGET Qtk::qtk_library)
 endif()
 
 # Print all QTK variables
-get_cmake_property(VAR_NAMES VARIABLES)
+if (NOT Qtk_IS_TOP_LEVEL)
-list(FILTER VAR_NAMES INCLUDE REGEX "^Q[tT][kK]_.*$")
+  get_cmake_property(VAR_NAMES VARIABLES)
-list(SORT VAR_NAMES)
+  list(FILTER VAR_NAMES INCLUDE REGEX "^Q[tT][kK]_.*$")
-foreach(VAR_NAME ${VAR_NAMES})
+  list(SORT VAR_NAMES)
-  message(STATUS "[Qtk] ${VAR_NAME}=${${VAR_NAME}}")
+  foreach(VAR_NAME ${VAR_NAMES})
-endforeach()
+    message(STATUS "[Qtk] ${VAR_NAME}=${${VAR_NAME}}")
+  endforeach()
+endif()
 
 find_package(Qt6 COMPONENTS Core Widgets OpenGLWidgets REQUIRED)
 

src/CMakeLists.txt

@@ -46,6 +46,7 @@ set(CPACK_PACKAGE_DIRECTORY packages/)
 set(CPACK_PACKAGE_CONTACT "shaunreed.com")
 set(CPACK_PACKAGE_ICON "${CMAKE_SOURCE_DIR}/resources/icon.png")
 set(CPACK_THREADS 0)
+set(CPACK_PACKAGE_INSTALL_DIRECTORY "Qtk")
 
 # Debian
 set(CPACK_DEBIAN_PACKAGE_HOMEPAGE ${CPACK_PACKAGE_HOMEPAGE_URL})

src/app/CMakeLists.txt

@@ -121,7 +121,7 @@ qt_generate_deploy_app_script(
     FILENAME_VARIABLE QTK_DEPLOY_SCRIPT
     NO_UNSUPPORTED_PLATFORM_ERROR
 )
-install(SCRIPT ${QTK_DEPLOY_SCRIPT})
+#install(SCRIPT ${QTK_DEPLOY_SCRIPT})
 
 if(WIN32)
   get_target_property(_qt6_qmake_location Qt6::qmake IMPORTED_LOCATION)

src/app/qtkwidget.cpp

@@ -42,9 +42,7 @@ QtkWidget::QtkWidget(QWidget * parent, const QString & name, Scene * scene) :
   // Set the size of the depth bufer for glEnable(GL_DEPTH_TEST)
   format.setDepthBufferSize(16);
   // If QTK_DEBUG is set, enable debug context
-#ifdef QTK_DEBUG
   format.setOption(QSurfaceFormat::DebugContext);
-#endif
   setFormat(format);
   setFocusPolicy(Qt::ClickFocus);
 }
@@ -73,7 +71,6 @@ void QtkWidget::initializeGL() {
   connect(this, SIGNAL(frameSwapped()), this, SLOT(update()));
 
   // Initialize OpenGL debug context
-#ifdef QTK_DEBUG
   mDebugLogger = new QOpenGLDebugLogger(this);
   if(mDebugLogger->initialize()) {
     qDebug() << "GL_DEBUG Debug Logger" << mDebugLogger << "\n";
@@ -82,7 +79,6 @@ void QtkWidget::initializeGL() {
         SLOT(messageLogged(QOpenGLDebugMessage)));
     mDebugLogger->startLogging();
   }
-#endif  // QTK_DEBUG
 
   printContextInformation();
 

src/app/qtkwidget.h

@@ -156,7 +156,6 @@ namespace Qtk {
        */
       void update();
 
-#ifdef QTK_DEBUG
       /**
        * Called when the `messageLogged` signal is caught.
        * See definition of initializeGL()
@@ -164,7 +163,6 @@ namespace Qtk {
        * @param msg The message logged.
        */
       void messageLogged(const QOpenGLDebugMessage & msg);
-#endif
 
     private:
       /*************************************************************************
@@ -181,20 +179,16 @@ namespace Qtk {
        */
       static void updateCameraInput();
 
-#ifdef QTK_DEBUG
       /**
        * Prints OpenGL context information at start of debug session.
        */
       void printContextInformation();
-#endif
 
       /*************************************************************************
        * Private Members
        ************************************************************************/
 
-#ifdef QTK_DEBUG
       QOpenGLDebugLogger * mDebugLogger;
-#endif
       Qtk::Scene * mScene;
       Qtk::DebugConsole * mConsole;
       bool mConsoleActive = false;

src/app/toolbox.h

@@ -18,7 +18,7 @@ namespace Ui {
 }
 
 namespace Qtk {
-  class QDESIGNER_WIDGET_EXPORT ToolBox : public QDockWidget {
+  class ToolBox : public QDockWidget {
       Q_OBJECT
 
     public:

src/app/treeview.h

@@ -22,7 +22,7 @@ namespace Ui {
 }
 
 namespace Qtk {
-  class QDESIGNER_WIDGET_EXPORT TreeView : public QDockWidget {
+  class TreeView : public QDockWidget {
       Q_OBJECT
 
     public: