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

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

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

meltingrabbit
Copy link
Contributor

@meltingrabbit meltingrabbit commented Feb 13, 2024

Related issues

NA

Description

After c2a-core v4.4.0, the common packet source code directory will be changed (arkedge/c2a-core#310).
The C2A command sender (#485) is an experimental feature that temporarily includes header file of C2A util function directly, but arkedge/c2a-core#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 のレビューをもらいたいです.

@meltingrabbit meltingrabbit added the C2A Something related with C2A flight software label Feb 13, 2024
@meltingrabbit meltingrabbit self-assigned this Feb 13, 2024
@meltingrabbit meltingrabbit requested review from sksat and a team as code owners February 13, 2024 08:32
@meltingrabbit meltingrabbit requested review from 200km, seki-hiro, suzuki-toshihir0, t-hosonuma and conjikidow and removed request for a team February 13, 2024 08:32
@200km 200km added priority::medium priority medium library library minor update add functionality in a backwards compatible manner labels Feb 13, 2024
@200km
Copy link
Member

200km commented Mar 8, 2024

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

@meltingrabbit
Copy link
Contributor Author

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C2A Something related with C2A flight software library library minor update add functionality in a backwards compatible manner priority::medium priority medium
Projects
Status: 👀 Waiting Review
Development

Successfully merging this pull request may close these issues.

2 participants