Windows Building not correctly cacching

Slime-Dev / SlimeProject

SlimeEngine is my personal playground for learning modern Vulkan and its 1.3 features. This project serves as both a learning tool and a growing graphics framework, focusing on clean, understandable implementation of advanced Vulkan concepts. It's a work in progress, evolving as I explore and understand more about low-level graphics programming.

3 stars 0 forks source link

Windows Building not correctly cacching #37

Closed ConnorY97 closed 3 months ago

ConnorY97 commented 3 months ago

Run actions/cache@v4
  with:
    path: ~/.cpm
    key: Windows-cpm-e6f2d4b87142345682f56164fb820c9f32059b7885e7193f78980ce2fee53bdd
    restore-keys: Windows-cpm-

    enableCrossOsArchive: false
    fail-on-cache-miss: false
    lookup-only: false
    save-always: false
Cache not found for input keys: Windows-cpm-e6f2d4b87142345682f56164fb820c9f32059b7885e7193f78980ce2fee53bdd, Windows-cpm-

Need to investigate why caches are not being found.

ConnorY97 commented 3 months ago

I believe the issue is linked with this error in the Configure CMake step for windows:

2024-07-24T07:10:27.6339888Z ##[endgroup]
2024-07-24T07:10:32.6391284Z CMake Warning:
2024-07-24T07:10:32.6391923Z   Ignoring extra path from command line:
2024-07-24T07:10:32.6392313Z 
2024-07-24T07:10:32.6392470Z    ".cpm"
2024-07-24T07:10:32.6392638Z 
2024-07-24T07:10:32.6392644Z

ConnorY97 commented 3 months ago

The issue was, Windows did not know what ~/.cpm was. So we implemented different paths for Windows and Linux:

- name: Restore CPM cache
  uses: actions/cache@v4
  with:
    path: |
      ~/.cpm
      ${{ github.workspace }}/.cpm
    key: ${{ runner.os }}-cpm-${{ hashFiles('**/CMakeLists.txt', '**/*.cmake') }}
    restore-keys: |
      ${{ runner.os }}-cpm-

This have solved the issue