Skip to content
Snippets Groups Projects

cmake: Use TheLartians' Ccache script

Merged Eidolon requested to merge cmake-thelartians-ccache into next
1 unresolved thread

Improves ccache support for Xcode

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • SteelT approved this merge request

    approved this merge request

    • Contributor

      Wasn't able to get this to work. Configuring was fine, build failed immediately (on Windows w/ Ninja generator)

      $ cmake --build . --clean-first
      [1/1] Cleaning all built files...
      Cleaning... 821 files.
      [0/820] Building C object thirdparty/CMakeFiles/.../_deps/openmpt-src/include/minimp3/minimp3.c.obj
      CreateProcess failed. Command attempted:
      "C:\Users\tehre\Documents\GitHub\SRB2-build\_deps\ccache.cmake-build\launch-c C:\msys64\mingw32\bin\
      cc.exe -DLIBOPENMPT_BUILD -IC:/Users/tehre/Documents/GitHub/SRB2-build/_deps/openmpt-src/common -IC:
      /Users/tehre/Documents/GitHub/SRB2-build/_deps/openmpt-src/src -IC:/Users/tehre/Documents/GitHub/SRB
      2-build/_deps/openmpt-src/include -IC:/Users/tehre/Documents/GitHub/SRB2-build/_deps/openmpt-src -IC
      :/Users/tehre/Documents/GitHub/SRB2-build/thirdparty  -MD -MT thirdparty/CMakeFiles/openmpt.dir/__/_
      deps/openmpt-src/include/minimp3/minimp3.c.obj -MF thirdparty\CMakeFiles\openmpt.dir\__\_deps\openmp
      t-src\include\minimp3\minimp3.c.obj.d -o thirdparty/CMakeFiles/openmpt.dir/__/_deps/openmpt-src/incl
      ude/minimp3/minimp3.c.obj -c C:/Users/tehre/Documents/GitHub/SRB2-build/_deps/openmpt-src/include/mi
      nimp3/minimp3.c"
      ninja: fatal: CreateProcess: %1 is not a valid Win32 application.```
    • I think Windows is unsupported for this script at the moment. Was it working before? I could vendor it with a tweak to fix it.

    • Contributor

      I started using CCACHE in Makefiles a lot after jartha recommended it to me, I can't imagine not using it anymore :p

    • Please register or sign in to reply
  • Eidolon added 1 commit

    added 1 commit

    • 036d5cce - cmake: Use TheLartians' Ccache script on non-Win32

    Compare with previous version

  • Contributor

    Latest version of the commit fixes my issue. :thumbsup:

  • Eidolon mentioned in commit 2e9260c8

    mentioned in commit 2e9260c8

  • merged

Please register or sign in to reply
Loading