Refactor example user IfWrapper CMake #532
Conversation
| SilsMockup/ccsds_sils.c | ||
| SilsMockup/i2c_sils.c | ||
| SilsMockup/uart_sils.c | ||
| SilsMockup/wdt_sils.c | ||
| ) | ||
| else() | ||
| #target_sources(${PROJECT_NAME} PUBLIC | ||
| list(APPEND C2A_SRCS | ||
| set(C2A_IF_IMPL_COMMON_SRCS |
There was a problem hiding this comment.
オプションによって追加する側（ C2A_IF_COM_WINGS_SRCS など）に対して，追加される側って，なんかもっといい表現無いかな．
basis でもないしなぁ
There was a problem hiding this comment.
feature とか？でも表現以前に現在の実装が同じディレクトリにベチャッと撒かれてるだけだから，現時点であんまりカッコイイ表現をすべきでもないと思う（過剰な期待を抱いてしまう）
| #target_sources(${PROJECT_NAME} PUBLIC | ||
| list(APPEND C2A_SRCS | ||
| Sils/uart_sils_sci_if.cpp | ||
| if(USE_SCI_COM_WINGS) # TODO: これ USE_SCI_COM_UART では？ |
There was a problem hiding this comment.
文脈次第かな．
MOBC - 2nd OBC を com0com でつなげてSILSするときはそうなんだけど，
2nd OBC を直接WINGSに指すときはこれでいい．（そのOBCに対して，ペリフェラルを操作するためのUARTではなく，upstream 側の UART なんだけど，命名がな．．．）
There was a problem hiding this comment.
ちなみに，ここなおすなら Examples\2nd_obc_user\CMakeLists.txt の L11 も直さないとだめ
There was a problem hiding this comment.
これは TODO comment に対する話？
There was a problem hiding this comment.
L11 は指摘と現状の書き方の方が嘘で，これが正しい
There was a problem hiding this comment.
if 文のネストが深くなっていることに注意（GitHub の表示だとたしかにちょっとわかりにくいけど）
|
マージし損ねてた |
概要
list(APPEND)を使わないようにしたIssue
詳細
詳しく
検証結果
test へのリンクや，検証結果へのリンク
影響範囲
XX系の動作がガラッと変わる，とか．
補足
何かあれば
注意
priorityラベルを付けること