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 rust-toolchain override config #327

Merged
merged 1 commit into from
Mar 28, 2024
Merged

Add rust-toolchain override config #327

merged 1 commit into from
Mar 28, 2024

Conversation

sksat
Copy link
Member

@sksat sksat commented Mar 27, 2024

概要

rust-toolchain.toml をリポジトリに追加し,c2a-core リポジトリ内で使用する Rust のバージョン及び target toolchain を固定します

Issue

詳細

  • これにより,c2a-core 開発時の Rust のバージョンが固定されます
    • また,target の指定もしたため,checkout 後に rustup target add i686-unknown-linux-gnu する必要が無くなります
  • Rust CI がコケてる #326 に対する暫定的対処として,最新の stable の 1.77.0 ではなく 1.76.0 を指定しています

検証結果

Rust CI が通ればよし

影響範囲

Rust CI および c2a-core crate など Rust エコシステムを使っているもの

@sksat sksat added priority::high priorityg high tools labels Mar 27, 2024
@sksat sksat requested a review from KOBA789 March 27, 2024 10:57
@sksat sksat self-assigned this Mar 27, 2024
@sksat sksat requested a review from meltingrabbit March 28, 2024 05:37
@sksat
Copy link
Member Author

sksat commented Mar 28, 2024

@KOBA789 今これで他の CI コケてるので approve ほしいです(あと単に他の Rust 物体でも影響あるのでお知らせ)
FYI: @meltingrabbit これマージしたら別の PR rebase してほしいです

@meltingrabbit
Copy link
Member

rust-toolchain.toml ,なるほど.少々お待ち.

@sksat sksat merged commit 364ae23 into main Mar 28, 2024
37 checks passed
@sksat sksat deleted the add-rust-toolchain branch March 28, 2024 05:41
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