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

V1.82 #1325

Closed
wants to merge 5 commits into from
Closed

V1.82 #1325

wants to merge 5 commits into from

Conversation

ziqiaozhou
Copy link
Collaborator

@ziqiaozhou ziqiaozhou commented Oct 29, 2024

This draft PR is only created as a draft and is not intended to be merged to v1.79.

@utaal, this is the change I mentioned about supporting v1.82. The only issue I encountered is about negative trait and conflicted trait implementations.

conflicting implementations of trait `T57_IntoIterator` for type `&mut std::boxed::Box<[_], _>`

Or

found both positive and negative implementation of trait `T55_Iterator` for type `&mut std::boxed::Box<[_], _>`.

The way I bypass it for now is to check the polarity and add negative when needed. But still, I need to explicitly rule out Box cases.

Adding with_negative_coherence feature finally solved the issue I encountered. But I have 5 failed tests in example which is expected to fail and does fail. I may need to check the test lib and see why the failure is not recognized as a correct failure.

tmp fix: conflict conflicting implementations of trait IntoIteratr in Box

wip: fix typo

wip: repalce preds_on
This is to fix
error: conflicting implementations of trait `T57_IntoIterator` for type `&mut std::boxed::Box<[_], _>`
/.rustup/toolchains/1.82.0-x86_64-unknown-linux-gnu/lib/rustlib/src/rust/library/core/src/iter/traits/collect.rs:351:1
     |
/.rustup/toolchains/1.82.0-x86_64-unknown-linux-gnu/lib/rustlib/src/rust/library/alloc/src/boxed.rs:2329:1

error: conflicting implementations of trait `T55_Iterator` for type `std::boxed::Box<[_], _>`

note: This error was found in Verus pass: ownership checking of tracked code
@ziqiaozhou ziqiaozhou closed this Oct 29, 2024
@ziqiaozhou ziqiaozhou reopened this Oct 29, 2024
@ziqiaozhou ziqiaozhou marked this pull request as draft October 29, 2024 19:51
@utaal utaal deleted the branch verus-lang:rustc-1.79.0 October 31, 2024 06:42
@utaal utaal closed this Oct 31, 2024
@utaal utaal mentioned this pull request Oct 31, 2024
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