fix cmake

.github/workflows/release.yml

@@ -7,22 +7,20 @@ name: Build and Release
 on:
   push:
     tags:
-      - '*'    # Trigger on any tag push
+      - '*' # Any tag
 
 jobs:
-  build_and_release:
-    strategy:
-      matrix:
-        os: [ubuntu-latest, macos-latest, windows-latest]
-    runs-on: ${{ matrix.os }}
-
+  build-linux:
+    runs-on: ubuntu-latest
     steps:
-      - name: Checkout code
-        uses: actions/checkout@v3
+      - uses: actions/checkout@v3
         with:
           submodules: recursive
 
-      - name: Setup Python (needed for Conan)
+      - name: Install build tools
+        run: sudo apt-get update && sudo apt-get install -y flex bison
+
+      - name: Setup Python
         uses: actions/setup-python@v4
         with:
           python-version: '3.x'
@@ -31,70 +29,137 @@ jobs:
         run: |
           python -m pip install --upgrade pip
           pip install conan
-        shell: ${{ matrix.os == 'windows-latest' && 'pwsh' || 'bash' }}
+          conan profile detect
 
-      - name: Configure Conan
-        run: conan profile detect
-        shell: ${{ matrix.os == 'windows-latest' && 'pwsh' || 'bash' }}
-
-      - name: Install dependencies and build with Conan
+      - name: Build
         run: |
           conan install . --build=missing
           conan build .
-        shell: ${{ matrix.os == 'windows-latest' && 'pwsh' || 'bash' }}
 
-      - name: Determine if prerelease
-        id: prerelease_check
+      - name: Package
         run: |
           TAG=${GITHUB_REF##refs/tags/}
-          if [[ "$TAG" == prerelease-* ]]; then
-            echo "true" > prerelease.txt
-          else
-            echo "false" > prerelease.txt
-          fi
-          echo "prerelease=$(cat prerelease.txt)" >> $GITHUB_OUTPUT
-        shell: ${{ matrix.os == 'windows-latest' && 'pwsh' || 'bash' }}
+          OS=$(uname -s | tr '[:upper:]' '[:lower:]')
+          ARCH=$(uname -m)
+          FOLDER="chloride-$TAG-$OS-$ARCH"
+          TAR="$FOLDER.tar.gz"
+          mv build/bin "$FOLDER"
+          cp LICENSE.txt "$FOLDER"
+          tar -czf "$TAR" "$FOLDER"
+          echo "TAR_NAME=$TAR" >> $GITHUB_ENV
 
-      - name: Archive build folder
+      - name: Upload artifact
+        uses: actions/upload-artifact@v4
+        with:
+          name: linux-artifact
+          path: ${{ env.TAR_NAME }}
+
+  build-macos:
+    runs-on: macos-latest
+    steps:
+      - uses: actions/checkout@v3
+        with:
+          submodules: recursive
+
+      - name: Install build tools
+        run: brew install flex bison
+
+      - name: Setup Python
+        uses: actions/setup-python@v4
+        with:
+          python-version: '3.x'
+
+      - name: Install Conan
+        run: |
+          python -m pip install --upgrade pip
+          pip install conan
+          conan profile detect
+
+      - name: Build
+        run: |
+          conan install . --build=missing
+          conan build .
+
+      - name: Package
         run: |
           TAG=${GITHUB_REF##refs/tags/}
-          if [[ "$RUNNER_OS" == "Linux" ]]; then
-            OS=linux
+          OS=$(uname -s | tr '[:upper:]' '[:lower:]')
           ARCH=$(uname -m)
-            FOLDER_NAME="chloride-$TAG-$OS-$ARCH"
-            TAR_NAME="$FOLDER_NAME.tar.gz"
-            mv build/bin "$FOLDER_NAME"
-            cp LICENSE "$FOLDER_NAME"
-            tar -czf "$TAR_NAME" "$FOLDER_NAME"
-          elif [[ "$RUNNER_OS" == "macOS" ]]; then
-            OS=macos
-            ARCH=$(uname -m)
-            FOLDER_NAME="chloride-$TAG-$OS-$ARCH"
-            TAR_NAME="$FOLDER_NAME.tar.gz"
-            mv build/bin "$FOLDER_NAME"
-            cp LICENSE "$FOLDER_NAME"
-            tar -czf "$TAR_NAME" "$FOLDER_NAME"
-          else
-            # Windows
+          FOLDER="chloride-$TAG-$OS-$ARCH"
+          TAR="$FOLDER.tar.gz"
+          mv build/bin "$FOLDER"
+          cp LICENSE.txt "$FOLDER"
+          tar -czf "$TAR" "$FOLDER"
+          echo "TAR_NAME=$TAR" >> $GITHUB_ENV
+
+      - name: Upload artifact
+        uses: actions/upload-artifact@v4
+        with:
+          name: macos-artifact
+          path: ${{ env.TAR_NAME }}
+
+  build-windows:
+    runs-on: windows-latest
+    steps:
+      - uses: actions/checkout@v3
+        with:
+          submodules: recursive
+
+      - name: Install build tools
+        run: choco install winflexbison -y
+        shell: pwsh
+
+      - name: Setup Python
+        uses: actions/setup-python@v4
+        with:
+          python-version: '3.x'
+
+      - name: Install Conan
+        run: |
+          python -m pip install --upgrade pip
+          pip install conan
+          conan profile detect
+        shell: pwsh
+
+      - name: Build with MinGW
+        run: |
+          conan install .
+          conan build .
+        shell: pwsh
+
+      - name: Package
+        run: |
           $TAG = $env:GITHUB_REF -replace 'refs/tags/', ''
           $ARCH = if ([Environment]::Is64BitOperatingSystem) { 'x64' } else { 'x86' }
-            $FOLDER_NAME = "chloride-$TAG-windows-$ARCH"
-            $TAR_NAME = "$FOLDER_NAME.zip"
-            Rename-Item build\bin $FOLDER_NAME
-            Copy-Item LICENSE $FOLDER_NAME
-            Compress-Archive -Path $FOLDER_NAME -DestinationPath $TAR_NAME
-          fi
-          echo "TAR_NAME=$TAR_NAME" >> $GITHUB_ENV
-        shell: ${{ matrix.os == 'windows-latest' && 'pwsh' || 'bash' }}
+          $FOLDER = "chloride-$TAG-windows-$ARCH"
+          $ZIP = "$FOLDER.zip"
+          Rename-Item build\bin $FOLDER
+          Copy-Item LICENSE.txt $FOLDER
+          Compress-Archive -Path $FOLDER -DestinationPath $ZIP
+          echo "TAR_NAME=$ZIP" >> $env:GITHUB_ENV
+        shell: pwsh
+
+      - name: Upload artifact
+        uses: actions/upload-artifact@v4
+        with:
+          name: windows-artifact
+          path: ${{ env.TAR_NAME }}
+
+  release:
+    runs-on: ubuntu-latest
+    needs: [build-linux, build-macos, build-windows]
+    steps:
+      - name: Download artifacts
+        uses: actions/download-artifact@v4
+        with:
+          path: ./artifacts
 
       - name: Create GitHub Release
-        id: create_release
         uses: ncipollo/release-action@v1
         with:
           tag: ${{ github.ref_name }}
           name: Release ${{ github.ref_name }}
-          body: |
-            Automated release based on tag ${{ github.ref_name }}
+          body: Automated release based on tag ${{ github.ref_name }}
           draft: false
-          prerelease: ${{ steps.prerelease_check.outputs.prerelease }}
-          artifacts: ${{ env.TAR_NAME }}
+          prerelease: ${{ startsWith(github.ref_name, 'prerelease-') }}
+          artifacts: "./artifacts/*"

CMakeLists.txt

@@ -51,8 +51,20 @@ set_target_properties(argon PROPERTIES
 find_package(BDWgc REQUIRED)
 find_package(gmp REQUIRED)
 
-target_compile_options(argon PRIVATE -O3 -Wall -Wextra -Wno-unused-function -s)
-target_link_options(argon PRIVATE -static)
+target_compile_options(argon PRIVATE -O3 -Wall -s)
+if(MSVC)
+    # Disable warning C4061 (enum in switch not handled)
+    target_compile_options(argon PRIVATE /wd4061)
+
+    # Disable Spectre mitigation warning C5045
+    target_compile_options(argon PRIVATE /wd5045)
+
+    # Optionally, remove "treat warnings as errors" if enabled
+    # target_compile_options(argon PRIVATE /WX-)  # uncomment if you previously used /WX
+endif()
+if(NOT APPLE)
+    target_link_options(argon PRIVATE -static)
+endif()
 
 target_link_libraries(argon PRIVATE
     BDWgc::BDWgc

conanfile.py

@@ -5,6 +5,7 @@
 from conan import ConanFile
 from conan.tools.cmake import CMake, CMakeToolchain, CMakeDeps, cmake_layout
 from shutil import which
+import os
 
 class ArgonConan(ConanFile):
     name = "argon"
@@ -29,20 +30,19 @@ class ArgonConan(ConanFile):
         self.folders.generators = "build"
 
     def generate(self):
+        os.environ["CONAN_NON_INTERACTIVE"] = "1"
         tc = CMakeToolchain(self)
 
-        # Try to find flex in system PATH first
-        flex_path = which("flex")
-
-        # If not found, fallback to flex from Conan build requirements
+        if os.name == "nt":  # Windows
+            flex_path = which("win_flex") or which("win_flex.exe")
             if not flex_path:
-            flex_dep = self.dependencies.build.get("flex", None)
-            if not flex_dep:
-                raise Exception("Flex not found in system PATH and not declared as build requirement")
-            flex_path = join(flex_dep.package_folder, "bin", "flex")
-
-        tc.variables["FLEX_EXECUTABLE"] = flex_path
+                raise Exception("win_flex not found in PATH. Install winflexbison via choco.")
+        else:
+            flex_path = which("flex")
+            if not flex_path:
+                raise Exception("Flex not found in system PATH. Please install flex on Linux/macOS.")
 
+        tc.variables["FLEX_EXECUTABLE"] = flex_path.replace("\\", "\\\\")
         tc.generate()
 
     def build(self):