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

Remove unused -rdynamic link option #607

Merged
merged 1 commit into from
Jun 13, 2023

Conversation

sksat
Copy link
Collaborator

@sksat sksat commented Jun 12, 2023

概要

不要な -rdynamic リンクオプションを削除

詳細

  • -rdynamic は実行ファイルないし共用ライブラリにのみ効果のあるリンクオプション
  • そのため,単体ではビルドできない c2a-core で指定しても意味が無い
  • PUBLIC 指定されているので add_subdirectory() 先にも波及するが,c2a-core の責務を逸脱している
  • 一方で,-rdynamic をサポートしないようなターゲットでは,このオプションがこのような形で存在することによってビルドに失敗するケースがある

検証結果

CI が通ればよし

影響範囲

  • CMake でのビルド環境

補足

  • リリースしたい
    • まだ breaking なものが 3.9.0 以降マージされていないため,3.9.1 としたい

@sksat sksat added bug Something isn't working priority::high priorityg high tools labels Jun 12, 2023
@sksat sksat self-assigned this Jun 12, 2023
@sksat sksat requested a review from meltingrabbit June 12, 2023 07:02
@sksat sksat merged commit afeea61 into develop Jun 13, 2023
@sksat sksat deleted the feature/remove-unused-link-option branch June 13, 2023 08:02
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 tools
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants