Skip to content
This repository has been archived by the owner on Dec 29, 2022. It is now read-only.

build broken: dependency crossbeam-channel v0.3.9 fails to build #1571

Closed
matthiaskrgr opened this issue Oct 19, 2019 · 5 comments · Fixed by #1576
Closed

build broken: dependency crossbeam-channel v0.3.9 fails to build #1571

matthiaskrgr opened this issue Oct 19, 2019 · 5 comments · Fixed by #1576

Comments

@matthiaskrgr
Copy link
Member

When trying to build rls from the repo ( fb9aa9f ), the dependency crossbeam-channel v0.3.9 fails to build rendering rls unbuildable.

cargo +master check

    Checking crossbeam-channel v0.3.9
error[E0432]: unresolved import `crossbeam_utils::atomic`
 --> /home/matthias/.cargo/registry/src/gh.neting.cc-1ecc6299db9ec823/crossbeam-channel-0.3.9/src/flavors/tick.rs:8:22
  |
8 | use crossbeam_utils::atomic::AtomicCell;
  |                      ^^^^^^ could not find `atomic` in `crossbeam_utils`

error: aborting due to previous error

For more information about this error, try `rustc --explain E0432`.
error: could not compile `crossbeam-channel`.

@mati865
Copy link
Contributor

mati865 commented Oct 19, 2019

It's not directly related to RLS: crossbeam-rs/crossbeam@5b5d727
Either crossbeam has to release update or RLS has to get rid of dependency enabling nightly feature of crossbeam crates (it's rustc-ap-rustc_data_structures 😞 ).

@Xanewok
Copy link
Member

Xanewok commented Oct 20, 2019

Oh no =( @mati865 I assume that means the rustc-ap-* upgrade dance all over again?

@mati865
Copy link
Contributor

mati865 commented Oct 20, 2019

@Xanewok it depends whether crossbeam-utils fixes it in major/minor or patch release but in the long term the answer is yes.

As short term solution I've opened #1576
It works because older versions wasn't using target_has_atomic: crossbeam-rs/crossbeam@96ddd4d

@Xanewok
Copy link
Member

Xanewok commented Oct 20, 2019

cc crossbeam-rs/crossbeam#435

@Xanewok
Copy link
Member

Xanewok commented Mar 27, 2020

Merged upstream in rust-lang/rust#69470, now we need to coordinate a rustc-ap-* upgrade tomorrow (which will also unbreak the current RLS toolstate)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants