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

Add C99 build option to example users #348

Merged
merged 1 commit into from
May 17, 2024

Conversation

sksat
Copy link
Member

@sksat sksat commented May 17, 2024

概要

C99 のビルドオプションを example user に明示的に追加する

Issue

詳細

  • 現状でも、C2A user 全体は(CMake でのビルドであれば)C2A_BUILD_AS_C99 オプションで C99 ビルドを有効化できる(この実態は common.cmake にあるため)
  • しかしこれは暗黙的なものになってしまっており、C2A user で C99 ビルドを有効化する際に @sksat が毎回同様の設定を追加して回っている
  • C2A_BUILD_AS_C99 を example user にオプションとして追加することで存在を明示する
  • この PR ではあくまで存在の明示のみ行い、C99 ビルドの推奨化(= C89 ビルド非推奨の明示)は別途行う
    • これをまだ行わない理由は C2A user 的には全く無い
    • c2a-core の C89 の限定的なサポートを適切に継続するための準備を別途行うため
      • 具体的には、(c2a-core の単体ビルドはできないので)c2a-core の C89 でのビルド健全性チェックのため、C89・C99 両方での example user の Build-CI を整備する必要がある

検証結果

  • example user で cmake -B build -DC2A_BUILD_AS_C99=ON; cmake --build build して C99 でビルドされるようになる
  • example user で cmake -B build; cmake --build build して、まだ C89 でビルドされる

影響範囲

example user のビルド

@sksat sksat added the tools label May 17, 2024
@sksat sksat requested a review from ToshiAki64 May 17, 2024 06:00
@sksat sksat self-assigned this May 17, 2024
@sksat sksat added this to the v4.5.0 milestone May 17, 2024
@sksat sksat added the priority::medium priority medium label May 17, 2024
@sksat sksat force-pushed the feature/add-c99-build-option-to-examples branch from 43c4b42 to 53bbabc Compare May 17, 2024 06:08
@sksat sksat merged commit 88f009a into main May 17, 2024
72 checks passed
@sksat sksat deleted the feature/add-c99-build-option-to-examples branch May 17, 2024 06:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants