Skip to content

Migrate libc-test to edition 2018 #3718

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

Merged
merged 3 commits into from
May 23, 2024
Merged

Conversation

tesuji
Copy link
Contributor

@tesuji tesuji commented May 23, 2024

libc-test depens on ctest2 which has MSRV as 1.56, which already support 2018 edition.
Also remove unkown.txt from cargo:run_if. That made build script keeps running again compiled.

@rustbot
Copy link
Collaborator

rustbot commented May 23, 2024

r? @JohnTitor

rustbot has assigned @JohnTitor.
They will have a look at your PR within the next two weeks and either review your PR or reassign to another reviewer.

Use r? to explicitly pick a reviewer

tesuji added 3 commits May 23, 2024 10:55
edition: cannot update to 2021 edition because of libc msrv.
cargo of rust 1.19 cannot understand edition key in cargo.toml.
no semver vendor = 'unknown'
@tesuji tesuji changed the title Migrate libc-test to edition 2021 Migrate libc-test to edition 2018 May 23, 2024
@JohnTitor JohnTitor added this pull request to the merge queue May 23, 2024
Merged via the queue into rust-lang:libc-0.2 with commit ca70695 May 23, 2024
57 checks passed
@tesuji tesuji deleted the migrate2021 branch May 23, 2024 21:52
@tgross35 tgross35 added the main-needs-cherry-pick This PR was merged against libc-0.2 but does not yet exist in main. label Aug 13, 2024
tgross35 pushed a commit to tgross35/rust-libc that referenced this pull request Aug 29, 2024
edition: cannot update to 2021 edition because of libc msrv.
cargo of rust 1.19 cannot understand edition key in cargo.toml.

(cherry picked from commit 4057a9a)

avoid rebuilding libc-test

no semver vendor = 'unknown'

(cherry picked from commit de2c8a7)

use rel path for semver

(cherry picked from commit d6d56f1)

(apply <rust-lang#3718> to `main`)
[ resolve conflicts, squash for easier cherry-picks - Trevor ]
@tgross35 tgross35 added main-applied This PR was merged against libc-0.2 and main has since been updated to include it and removed main-needs-cherry-pick This PR was merged against libc-0.2 but does not yet exist in main. labels Aug 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
main-applied This PR was merged against libc-0.2 and main has since been updated to include it S-waiting-on-review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants