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

Pre Release (v3.9.0-beta.8): Fix get c2a-core git revision working dir #600

Merged
merged 3 commits into from
May 31, 2023

Conversation

sksat
Copy link
Collaborator

@sksat sksat commented May 31, 2023

概要

#82 で追加した CMake での git revision 出力のうち,c2a-core 部分が間違っていたので修正

Issue/PR

詳細

  • c2a-core の git revision の取得は c2a-core が checkout されているディレクトリ(ex: src/src_core)で行う必要がある
  • しかし,CMake では c2a-core の context でのソースディレクトリ(CMAKE_CURRENT_SOURCE_DIR)ではなくビルド時の context でのソースディレクトリ(CMAKE_SOURCE_DIR)になっていた
  • そのため,CMake の add_subdirectory() で c2a-core を読み込んでビルドしているような環境では,core の revision として user の revision が取り入れられてしまっていた

検証結果

  • 同じ Git repo の Example user ではなく,個別の C2A user で検証する

影響範囲

c2a-core を CMake の add_subdirectory で使っているような環境

補足

pre release 打ちたい

@sksat sksat added bug Something isn't working priority::high priorityg high tools labels May 31, 2023
@sksat sksat requested a review from meltingrabbit May 31, 2023 07:15
@sksat sksat self-assigned this May 31, 2023
@sksat
Copy link
Collaborator Author

sksat commented May 31, 2023

あー,そして S2E では C2A user をadd_subdirectory() するから S2E user の revision になってしまうのだな

@sksat sksat added the S2E label May 31, 2023
@meltingrabbit meltingrabbit changed the title Fix get c2a-core git revision working dir Pre Release (v3.9.0-beta.8): Fix get c2a-core git revision working dir May 31, 2023
@meltingrabbit
Copy link
Collaborator

バージョン上げてマージしますね

@meltingrabbit meltingrabbit merged commit aedb983 into develop May 31, 2023
@meltingrabbit meltingrabbit deleted the feature/fix-core-git-revision-output branch May 31, 2023 08:51
@meltingrabbit
Copy link
Collaborator

リリース打った: https://github.com/ut-issl/c2a-core/releases/tag/v3.9.0-beta.8

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working priority::high priorityg high S2E tools
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants