Skip to content

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
@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.

3 participants