s2e-core icon indicating copy to clipboard operation
s2e-core copied to clipboard

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

Open meltingrabbit opened this issue 4 months ago • 2 comments

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 のレビューをもらいたいです.

meltingrabbit avatar Feb 13 '24 08:02 meltingrabbit

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

200km avatar Mar 08 '24 00:03 200km

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

meltingrabbit avatar Mar 08 '24 02:03 meltingrabbit