Fix MacOS build (#34)

* Testing xquartz * Update build.yml * Trying to fix macos build For #33 * utils/paths.hpp: include <cstdint> * Update gen-components.tsx * Update ui-field.types.hpp & ui-field.types.cpp to remove uint * Fixes to build.yml before merging & timeout --------- Co-authored-by: KaruroChori <[email protected]>
KaruroChori · Dec 4, 2024 · 1d50780 · 1d50780
1 parent 0ff3686
commit 1d50780
Show file tree

Hide file tree

Showing 5 changed files with 25 additions and 8 deletions.
diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml
@@ -30,6 +30,7 @@ jobs:
       matrix:
         os: [ubuntu-24.04]
     runs-on: ${{ matrix.os }}
+    timeout-minutes: 8
     env:
       CC: clang-19
       CXX: clang++-19
@@ -70,9 +71,15 @@ jobs:
         bun run codegen
         bun run meson-setup.clang-release
         meson compile -C build/ vs:executable
-
+    - name: Archive production artifacts
+      uses: actions/upload-artifact@v4
+      with:
+        name: logs
+        path: |
+          build/meson-logs/**/*
   macos:
     runs-on: macos-14
+    timeout-minutes: 8
     env:
       LDFLAGS: '-L/opt/homebrew/opt/llvm/lib'
       CPPFLAGS: '-I/opt/homebrew/opt/llvm/include'
@@ -89,6 +96,7 @@ jobs:
           libpng \
           meson \
           ninja
+        brew install --cask xquartz
         cd /opt/homebrew/opt/llvm/bin
         #ln -s clang++ clang++-19
         PATH=/opt/homebrew/opt/llvm/bin:$PATH
@@ -100,5 +108,12 @@ jobs:
         bun install
         bun run codegen
         meson setup --reconfigure build/ --buildtype=release --native-file toolchains/flatpak.ini
+        # Unclear fix to be investigated
+        rm subprojects/libtcc/VERSION
         meson compile -C build/ vs:executable
-
+    - name: Archive production artifacts
+      uses: actions/upload-artifact@v4
+      with:
+        name: logs
+        path: |
+          build/meson-logs/**/*
diff --git a/include/ui-field.types.hpp b/include/ui-field.types.hpp
@@ -30,12 +30,12 @@ namespace field_types{
 
   // Helpers to compute expressions into values that Fl_Widgets can use.
 
-  bool h_px(uint T, size_t *dst, const char *expr,
+  bool h_px(unsigned int T, size_t *dst, const char *expr,
                    const ui_base *env);
   bool h_colour(uint32_t *dst, const char *expr,
                        const ui_base *env);
 
   bool h_flag(bool *dst, const char *expr,
                        const ui_base *env);
 }
-}
+}
diff --git a/include/utils/paths.hpp b/include/utils/paths.hpp
@@ -8,6 +8,7 @@
  * 
  */
 
+#include <cstdint>
 #include <cstring>
 #include <string>
 #include <utils/policies.hpp>
@@ -185,4 +186,4 @@ struct resolve_path{
 #undef tnk
 #undef prefix
 
-}
+}
diff --git a/scripts/codegen/gen-components.tsx b/scripts/codegen/gen-components.tsx
@@ -64,6 +64,7 @@ class ${cppname} : public ${data.codegen.extends}{
 `;
 
     let class_impl = `
+#include <cstdint>
 #include <components/autogen/${data.ns}/${data.name}.hpp>
 
 namespace vs{
@@ -209,4 +210,4 @@ await Bun.write('./commons/schemas/json-component.schema.json', JSON.stringify(w
 
 if (process.argv[2] != 'quick') {
     await Bun.write('./src/components/autogen/meson.build', `autogen_components = [${cpp_files.map(x => `'${x}'`)}]`)
-}
+}
diff --git a/src/ui-field.types.cpp b/src/ui-field.types.cpp
@@ -149,7 +149,7 @@ namespace field_types{
 
 
   //TODO: I need to check if number conversions failed or not too.
-  bool h_px(uint T, size_t* dst, const char* expr, const ui_base* env){
+  bool h_px(unsigned int T, size_t* dst, const char* expr, const ui_base* env){
     if (T==1){
       dst[0]=std::stoi(expr);
       return true;
@@ -182,4 +182,4 @@ namespace field_types{
 
 }
 
-}
+}