Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use Ninja #521

Open
wants to merge 3 commits into
base: develop
Choose a base branch
from
Open

Use Ninja #521

wants to merge 3 commits into from

Conversation

sksat
Copy link
Collaborator

@sksat sksat commented Mar 16, 2023

概要

SSIA

Issue

  • 関連する issue

詳細

詳しく

検証結果

test へのリンクや,検証結果へのリンク

影響範囲

XX系の動作がガラッと変わる,とか.

補足

何かあれば

注意

  • 関連する Projects が存在する場合,それの紐付けを行うこと
  • Assignees を設定すること
  • 可能ならば Reviewers を設定すること
  • 可能ならば priority ラベルを付けること

@sksat sksat added enhancement New feature or request tools labels Mar 16, 2023
@sksat sksat self-assigned this Mar 16, 2023
@sksat
Copy link
Collaborator Author

sksat commented Mar 16, 2023

エーーー

CMake Error at CMakeLists.txt:5 (project):
  Generator

    Ninja

  does not support platform specification, but platform

    Win32

  was specified.

@sksat
Copy link
Collaborator Author

sksat commented Mar 16, 2023

あーーー,いや,これ CMakeSettings.json で generator 指定するやつか(VS CMake はビックリドッキリで困る)

@sksat sksat changed the title Use Ninja on build CI Use Ninja Mar 16, 2023
@sksat sksat force-pushed the feature/use-ninja-on-build-ci branch from 66bdd20 to 581a6b8 Compare March 16, 2023 10:31
@sksat
Copy link
Collaborator Author

sksat commented Mar 16, 2023

なんかむしろ遅くなっていて謎だな

@sksat
Copy link
Collaborator Author

sksat commented Mar 16, 2023

気持ち速い感はありつつ,standard GitHub hosted runner が 2core なのもあってそんなに差無いなあ......どちらかというとキャッシュすべきか

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request tools
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant