Skip to content

fix: feature = ["sgx", "std"] のケースで localstd が二重定義される問題を修正#597

Merged
laysakura merged 1 commit intomasterfrom
fix/localstd-feature-condition
May 22, 2021
Merged

fix: feature = ["sgx", "std"] のケースで localstd が二重定義される問題を修正#597
laysakura merged 1 commit intomasterfrom
fix/localstd-feature-condition

Conversation

@laysakura
Copy link
Copy Markdown
Contributor

@laysakura laysakura commented May 22, 2021

Issueへのリンク

(なし)

やったこと

VSCode の Remote Container で dev環境上のコンテナに接続し、rust-analyzer をかけていたところ、たまたまtitleの問題に気づきました。

image

やらないこと

考えられるfeatureの組み合わせで、CIでマトリクスビルドを書けておくべきだと思います。
今回はやらないですが、このPRが妥当と判断されたらまずはissue化します。

動作検証

rust-analyzer がエラーを吐かなくなった

参考

(なし)

@laysakura laysakura requested review from osuketh and removed request for osuketh May 22, 2021 06:03
@laysakura
Copy link
Copy Markdown
Contributor Author

(CI通る前にreviewerアサインしてしまったから解除)

@laysakura laysakura requested a review from osuketh May 22, 2021 07:08
Copy link
Copy Markdown
Member

@osuketh osuketh left a comment

Choose a reason for hiding this comment

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

本来、"sgx""std" のfeatureを両方指定する挙動は意図していないので、同時に両方指定されないような制限ができると良いかなと思いました。(具体的な方法思いついていない)
このPRは 👍 ですー

@laysakura
Copy link
Copy Markdown
Contributor Author

では

考えられるfeatureの組み合わせで、CIでマトリクスビルドを書けておくべきだと思います。
今回はやらないですが、このPRが妥当と判断されたらまずはissue化します。

このissueに

同時に両方指定されないような制限

に関しても記載しておきます 🙋

@laysakura
Copy link
Copy Markdown
Contributor Author

@osuketh ↓↓のような感じでマージできません 😭
image

@osuketh
Copy link
Copy Markdown
Member

osuketh commented May 22, 2021

すません直しました

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants