search

ut-issl / s2e-core

Spacecraft Simulation Environment Core codes
MIT License
46 stars 18 forks source link

Support c2a-core v4.4.0 (C2A command sender) #610

Open meltingrabbit opened 8 months ago

meltingrabbit commented 8 months ago

Related issues

NA

Description

After c2a-core v4.4.0, the common packet source code directory will be changed (https://github.com/arkedge/c2a-core/pull/310). The C2A command sender (https://github.com/ut-issl/s2e-core/pull/485) is an experimental feature that temporarily includes header file of C2A util function directly, but https://github.com/arkedge/c2a-core/pull/310 is a breaking change to this feature.

Therefore, add a conditional branch to change the include path depending on the version of c2a-core.

Test results

NA

Impact

Supplementary information

NA

備考

まず @sksat のレビューで議論後， @ut-issl/aocs のレビューをもらいたいです．

200km commented 7 months ago

@meltingrabbit #619 がマージされましたが、こちらはどうしますか？ isslのレビューはあとでと言われているので、レビューしていませんが必要ならレビューはすぐにでもできます。

meltingrabbit commented 7 months ago

@200km ありがとうございます． ひとまず c2a-core 側の CI が通らない問題は https://github.com/arkedge/c2a-core/pull/317 で解消予定なので，この PR は c2a-core v4.4.0 が出そうになったときにまた動かそうと思います． それまでは放置していただいて構いません．