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

Add export src interface (c2a-core crate) #568

Merged
merged 3 commits into from
May 10, 2023

Conversation

sksat
Copy link
Collaborator

@sksat sksat commented May 10, 2023

概要

  • c2a-core crate に export-src feature を追加
  • export-src feature のときのみ,この crate のソースコードがあるディレクトリを露出させるためのユーティリティ関数を追加

Issue/PR

影響範囲

  • c2a-core crate のみ
  • export-src feature の時は no_std ではなくなる

補足

マージ後リリースしてほしい

@sksat sksat added enhancement New feature or request priority::high priorityg high tools labels May 10, 2023
@sksat sksat self-assigned this May 10, 2023
@sksat sksat requested a review from meltingrabbit May 10, 2023 17:06
@meltingrabbit
Copy link
Collaborator

meltingrabbit commented May 10, 2023

ごめん

この crate のソースコードがあるディレクトリを露出させる

がわかってないので,ちょっと教えて欲しい.(何に露出させる?)

@sksat
Copy link
Collaborator Author

sksat commented May 10, 2023

crate として外から使う時.つまり,Cargo で c2a-core crate を引っ張ってきた時に,c2a-core crate 自体のディレクトリ(ビルドのために生えるやつ)を c2a-core crate のユーザが取得できるようにする(実装が CARGO_MANIFEST_DIR になるのはややハックなのであんまり気にしないでください).

@meltingrabbit
Copy link
Collaborator

なるほどね.
今後したいことがなんとなくわかった.

Copy link
Collaborator

@meltingrabbit meltingrabbit left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

リリースだけど,この直後 #561 がリリースされるので,このPRではリリースうたなくても良い?

@sksat
Copy link
Collaborator Author

sksat commented May 10, 2023

そこはメンテナの気分次第かなという気持ち(直後にリリースするからええやろというのは運用でカバーの類なので)

@meltingrabbit
Copy link
Collaborator

OK.まあじゃあそうさせてもらって,これはこのままマージで

@sksat
Copy link
Collaborator Author

sksat commented May 10, 2023

OK

@sksat sksat merged commit 5a96e61 into develop May 10, 2023
@sksat sksat deleted the feature/add-export-src-interface branch May 10, 2023 17:25
@sksat sksat mentioned this pull request May 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request priority::high priorityg high tools
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants