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

Fix relative c2a-core header include #37

Merged
merged 1 commit into from
May 11, 2023

Conversation

sksat
Copy link
Collaborator

@sksat sksat commented Apr 19, 2023

概要

相対パスで c2a-core のヘッダを include しているのをやめる

Issue

詳細

現在の C2A での標準的な include の仕方に則り,#include "../../src_core/hoge.h" をやめ,#include <src_core/hoge.h> のようにする

以下とともにマージする

検証結果

影響範囲

  • 非互換なのでリリースを打つ

@sksat sksat self-assigned this Apr 19, 2023
@sksat sksat requested a review from meltingrabbit April 19, 2023 17:39
@sksat sksat added priority::medium priority medium tools labels May 10, 2023
@meltingrabbit
Copy link
Collaborator

@sksat v1.7.0 がすでに出てるので,rebase して欲しいです

@sksat
Copy link
Collaborator Author

sksat commented May 10, 2023

OK.rebase します.

@sksat
Copy link
Collaborator Author

sksat commented May 10, 2023

あれ

@sksat
Copy link
Collaborator Author

sksat commented May 10, 2023

あーーー,後から outside collabolator になった & origin 間違えた.やり直し.

@meltingrabbit meltingrabbit merged commit bc90d33 into ut-issl:main May 11, 2023
@sksat sksat deleted the feature/fix-relative-core-include branch May 11, 2023 16:03
@meltingrabbit
Copy link
Collaborator

リリース打ちました https://github.com/ut-issl/c2a-tlm-cmd-code-generator/releases/tag/v1.8.0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants