-
Notifications
You must be signed in to change notification settings - Fork 2
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
[code-generator] 自動生成コードの @note に,生成元DBのコミットハッシュと生成パラメータを出力する #240
Conversation
@ToshiAki64 @hender14 |
* path_to_db: ../examples/subobc/tlm-cmd-db/ | ||
* max_tlm_num: 256 | ||
* driver_path: aocs/ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
path_to_db
やdriver_path
って、各自で別のpathになる可能性があり、各衛星ブランチで共通のドライバをdevelopからマージする際にドライバを生成した人の環境によって、変わってしまうのではないかと思ったのですが?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
たしかにそうですね.
消す場合,sub obcの場合は,どのリポジトリか?の情報が残せないのがつらいですね.少し考えます
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
リポジトリ名と,
自身のDBの場合は,DBの全csvのダイジェスト,
sub obcについてはそのコミットハッシュを残すようにしました.
examples/mobc/src/src_user/component_driver/aocs/aobc_command_definitions.h
Outdated
Show resolved
Hide resolved
examples/subobc/src/src_user/tlm_cmd/block_command_definitions.h
Outdated
Show resolved
Hide resolved
6ba4ef7
to
9105700
Compare
@hender14 @ToshiAki64 レビューお願いします |
概要
自動生成コードの @note に,生成元DBのコミットハッシュと生成パラメータを出力する.
これによって,生成されたコードから,元のDBや生成パラメータがわかるようになる
自身のdbから生成されたコードには,そのdbに含まれるすべてのcsvのmd5を結合したものをさらにmd5にしたもの,
sub obcのdbについては,そのコミットハッシュを記録している.
Issue
NA
検証結果
CIが通ればOK
影響範囲