Fix CMake output directories (build/bin) and GitHub Actions paths

- Updated `src/c/CMakeLists.txt` to strictly enforce `CMAKE_RUNTIME_OUTPUT_DIRECTORY` to `build/bin` for all configurations.
- Updated `.github/workflows/build_and_release.yml` to package binaries from `build/bin` instead of default CMake locations.
- Ensured `enable_testing()` is called correctly and `add_test` uses `$<TARGET_FILE:run_tests>`.

Author: google-labs-jules[bot]

.github/workflows/build_and_release.yml

@@ -44,12 +44,12 @@ jobs:
         run: |
           mkdir -p dist
           if [ "${{ runner.os }}" == "Windows" ]; then
-            # Windows CMake builds output to Debug/Release subdirs
-            zip -j dist/pyraview-win-x64.zip build/Release/*.dll build/Release/*.exe
+            # Windows CMake builds output to bin due to our CMakeLists.txt fix
+            zip -j dist/pyraview-win-x64.zip build/bin/*.dll build/bin/*.exe
           elif [ "${{ runner.os }}" == "macOS" ]; then
-            zip -j dist/pyraview-mac-arm.zip build/*.dylib build/tests/run_tests
+            zip -j dist/pyraview-mac-arm.zip build/bin/*.dylib build/bin/run_tests
           else
-            zip -j dist/pyraview-linux-x64.zip build/*.so build/tests/run_tests
+            zip -j dist/pyraview-linux-x64.zip build/bin/*.so build/bin/run_tests
           fi
 
       - name: Upload Artifacts

src/c/CMakeLists.txt

@@ -3,7 +3,22 @@ project(pyraview C)
 
 find_package(OpenMP)
 
-include_directories(../../include)
+# Use absolute path for include to avoid relative path headaches
+include_directories(${CMAKE_CURRENT_SOURCE_DIR}/../../include)
+
+# --- OUTPUT DIRECTORY FIX ---
+# This forces Windows/MSVC to put everything in /bin instead of /bin/Release
+set(CMAKE_RUNTIME_OUTPUT_DIRECTORY ${CMAKE_BINARY_DIR}/bin)
+set(CMAKE_LIBRARY_OUTPUT_DIRECTORY ${CMAKE_BINARY_DIR}/bin)
+set(CMAKE_ARCHIVE_OUTPUT_DIRECTORY ${CMAKE_BINARY_DIR}/bin)
+
+foreach(config ${CMAKE_CONFIGURATION_TYPES})
+    string(TOUPPER ${config} CONFIG)
+    set(CMAKE_RUNTIME_OUTPUT_DIRECTORY_${CONFIG} ${CMAKE_BINARY_DIR}/bin)
+    set(CMAKE_LIBRARY_OUTPUT_DIRECTORY_${CONFIG} ${CMAKE_BINARY_DIR}/bin)
+    set(CMAKE_ARCHIVE_OUTPUT_DIRECTORY_${CONFIG} ${CMAKE_BINARY_DIR}/bin)
+endforeach()
+# -----------------------------
 
 add_library(pyraview SHARED pyraview.c)
 
@@ -13,18 +28,18 @@ else()
     message(WARNING "OpenMP not found. Compiling without parallel support.")
 endif()
 
-# Link standard math library (not needed on Windows)
 if(NOT WIN32)
     target_link_libraries(pyraview PRIVATE m)
 endif()
 
 # Tests
 add_executable(run_tests tests/test_main.c)
+target_link_libraries(run_tests PRIVATE pyraview)
+
 if(NOT WIN32)
-    target_link_libraries(run_tests PRIVATE pyraview m)
-else()
-    target_link_libraries(run_tests PRIVATE pyraview)
+    target_link_libraries(run_tests PRIVATE m)
 endif()
 
-# Use generator expression to handle multi-config paths (Debug/Release) correctly
+# Enable testing must be called before add_test
+enable_testing()
 add_test(NAME run_tests COMMAND $<TARGET_FILE:run_tests>)