Pre Release (v3.9.0-beta.2): Split C89 compile option to BUILD_C2A_AS_C99 #527
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
既存の設定にそのまま取り込んだ時の挙動を変更しないために，C99 の方をオプションとし直した |
sksat
changed the title
|
これよく考えたら hotfix だな（つまり，main にマージするべきだな） |
|
もう 3.9.0 の beta 出しちゃってるし hotfix はいいか......（微妙に面倒になるだけでそこまでメリットが大きくない） |
meltingrabbit
approved these changes
Apr 23, 2023
There was a problem hiding this comment.
@sksat OKなんですが，リリースはどうします？
（suer で使いたいのでという理由で） beta.2 を打つのもありですが，気持ちが知りたい
and
beta.1 でちゃってるので，どちらにせよ rebase してから merge してほしい
|
気持ちとしては v3.8.1 にできるならしたかったんですが，現状の c2a-core のリリース手順・更新手順を考えると hotfix 扱いにしたところで微妙に面倒になるだけなので，今まで通り beta でいいかな，というかんじです．明確に user で使いたい理由があるので beta にせよリリースはマスト． |
|
rebase |
sksat
force-pushed
the
feature/split-c89-to-cmake-option
branch
from
7a0f641 to
5077d44
Compare
|
リリースする．OK．ちょっとまってね |
meltingrabbit
changed the title
|
@sksat バージョン上げたので，マージOK． リリースはこっちで書くのでよい？ |
|
ではマージします．リリースは @meltingrabbit の責任範囲という認識なので，そっちで書くので OK |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
概要
Pre Release (v3.9.0-beta.2): Split C89 compile option to BUILD_C2A_AS_C99
C99 のためのコンパイルオプションの指定を
BUILD_C2A_AS_C99に分割するIssue/PR
詳細
BUILD_C2A_AS_C99という CMake オプションに分割BUILD_C2A_AS_C99は c2a-core では default OFF とする（C89 な C2A user が存在するため）BUILD_C2A_AS_C99 = ONのとき，C99 としてビルドする影響範囲
C99 でビルドしており，かつ，#526 のような雑な
common.cmakeを書いていた C2A user のみBUILD_C2A_AS_C99=ONにする対応が必要（ただし，これをしていた場合 clang で意図した挙動になっていないため，このパッチを取り込む必要がある）補足