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

VCDU を AOS transfer frame に rename する #193

Merged
merged 9 commits into from
Nov 9, 2023

Conversation

meltingrabbit
Copy link
Member

@meltingrabbit meltingrabbit commented Nov 8, 2023

概要

rename し,さらにコード規約に合わせた.
ロジックは変えてない(はず)

Issue

詳細

MOBC user 側の修正方法

  1. vcdu.c/h を消し,aos_transfer_frame.c/h をコピーする.CMakeLists.txt も修正する
  2. VCDUAOSTF に置換する(TLM DB の csv なども)
  3. vcduaostf に置換する(TLM DB の csv なども)
  4. vcdu.haos_transfer_frame.h に置換する(include の修正)
  5. コンパイルが通らないところを直す.想定されるものは以下
    • VCDU 構造体 が AosTransferFrame 構造体に変わったので,変数定義の型名が変わっているはず

検証結果

CIがとおればOK

@meltingrabbit meltingrabbit added the priority::high priorityg high label Nov 8, 2023
@meltingrabbit meltingrabbit changed the title WIP: VCDU を AOS transfer frame に rename する VCDU を AOS transfer frame に rename する Nov 8, 2023
@meltingrabbit meltingrabbit self-assigned this Nov 8, 2023
Copy link
Collaborator

@ToshiAki64 ToshiAki64 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

いいと思います。
ただ、rename系の変更になるので、shスクリプトが用意されるのかが、気になり。

@meltingrabbit
Copy link
Member Author

user example (mobc) なので sh はなしですかね.
これらのファイルは近いうちにcoreに取り込むので,そもそもuserでは不要(user側では対応するファイルを消すだけ)になりそうです

cf: #196

@sksat
Copy link
Member

sksat commented Nov 9, 2023

migration script 作っていいと思いますよ(というか作れるなら作った方がいい)

@meltingrabbit
Copy link
Member Author

かわりに丁寧なchangelogかいた

@meltingrabbit meltingrabbit merged commit 66d5d71 into develop Nov 9, 2023
35 checks passed
@meltingrabbit meltingrabbit deleted the feature/rename_vcdu branch November 9, 2023 09:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
priority::high priorityg high
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants