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

Support c2a-core v4 C++ build option #61

Merged
merged 1 commit into from
Sep 7, 2023
Merged

Conversation

sksat
Copy link
Member

@sksat sksat commented Sep 7, 2023

  • 本 workflow のビルド手順が根本的に変わるようなことがあった場合は c2a-core の major update を行うようにしていきたい
    • これはつまり同時に複数の c2a-core major version をサポートする必要が発生しうるということでもある
    • 実際,現状でも v3 系と v4 系を同時にサポートする必要がある
  • c2a-core v4 の一環として,CMake option を整理している: Pre Release (v4.0.0-beta.2): Refactor cmake option c2a-core#86
  • ここで,本 workflow が用いている BUILD_AS_CXX option が C2A_BUILD_AS_CXX に変わる
  • c2a-core major version での処理の切り替えの方法にはいくつか手段があり,それは c2a-core の major version の切り替え方法を決める #60 で検討する
  • それはそれとして,今回の CMake option の変更については影響があるのは単なる rename である上に,存在しなくなった option を指定してもビルドはできる(CMake の warning は出る)ため,一旦同時に新旧両方の option を指定することで簡易的に v3, v4 の両サポートを行う

@sksat sksat added the enhancement New feature or request label Sep 7, 2023
@sksat sksat self-assigned this Sep 7, 2023
@sksat sksat merged commit e806cd9 into main Sep 7, 2023
@sksat sksat deleted the support-c2a-core-v4-cxx-option branch September 7, 2023 14:35
@sksat
Copy link
Member Author

sksat commented Oct 12, 2023

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

Successfully merging this pull request may close these issues.

1 participant