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

Drop c2a-core crate links config #199

Merged
merged 1 commit into from
Nov 13, 2023
Merged

Drop c2a-core crate links config #199

merged 1 commit into from
Nov 13, 2023

Conversation

sksat
Copy link
Member

@sksat sksat commented Nov 9, 2023

This reverts commit db74c28, reversing changes made to 7497a8e.

概要

ut-issl/c2a-core#580 を revert する

Issue / PR

詳細

  • https://doc.rust-lang.org/cargo/reference/build-scripts.html#the-links-manifest-key
  • package.links*-sys crate のためのもの
    • c2a-core は単体ではビルド・リンクできないライブラリなので,このオプションは目的外使用
    • そのため,package.links で指定した識別子が複数(バージョン差異も含む)の crate で被ることを想定していない(エラーになる)
      • これは(一つのビルドコンテキスト ≒ C2A user で) c2a-core crate を使う crate が複数ある時に問題となる
      • つまり,指定する c2a-core crate のバージョンを丁寧に揃える必要がある
      • これは真に同じバージョンである必要があるので,別の branch の実装をちょっと使いたい,みたいな時にも困る
      • また, sils-runtime: require c2a-core v4 #133 など(c2a-core の major version で維持する API のみに依存させたい)でも困る
  • このオプションを使わないと使えない c2a-core crate -> user crate build script の変数伝搬機能を使うために Export src dir via cargo ut-issl/c2a-core#580 で導入した
    • この目的は c2a-core crate のビルド時のソースファイルディレクトリを取得するため
    • これによるデメリットが大きくなりすぎた & 他の方法でもこの情報は取得可能であるため,revert する

検証結果

CI が通ればよし

影響範囲

c2a-core crate のソースファイルディレクトリを Cargo 経由で公開する機能(breaking change ではあるが,元々まだ example user でも使っていない experimental な機能)

@sksat sksat added priority::high priorityg high tools labels Nov 9, 2023
@sksat sksat self-assigned this Nov 9, 2023
…ia-cargo"

This reverts commit db74c28, reversing
changes made to 7497a8e.
@sksat sksat added this to the v4.1.0 milestone Nov 9, 2023
@sksat sksat marked this pull request as ready for review November 9, 2023 10:39
@sksat sksat merged commit 8a7c36e into develop Nov 13, 2023
35 checks passed
@sksat sksat deleted the feature/drop-crate-links branch November 13, 2023 06:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants