Closed Bug 1806156 Opened 2 years ago Closed 2 years ago

`testing/webdriver` emits `rustc` error with Rust 1.66

Categories

(Testing :: geckodriver, defect)

Firefox 110
defect

Tracking

(firefox110 fixed)

RESOLVED FIXED
110 Branch
Tracking Status
firefox110 --- fixed

People

(Reporter: ErichDonGubler, Assigned: ErichDonGubler)

References

Details

Attachments

(1 file)

On current autoland (revision 564efd83794f), when Rust 1.66 is used as the toolchain backing a ./mach build invocation, an error is raised for warp::generic::Tuple not being fulfilling by current trait bounds in server::create_warp_routes:

# With Rust 1.66 set as the current Rust toolchain, i.e., via `rustup default stable`:
$ ./mach build

0:42.89    Compiling webdriver v0.47.0 (C:\Users\Erich\workspace\mozilla-source\mozilla-unified\testing\webdriver)
 0:43.39 error: opaque type `impl warp::Filter + Clone + warp::filter::FilterBase<Extract = impl Reply, Error = Rejection>` does not satisfy its associated type bounds
 0:43.39    --> testing\webdriver\src\server.rs:257:18
 0:43.39     |
 0:43.39 257 | ) -> impl Filter<Extract = impl warp::Reply, Error = Rejection> + Clone {
 0:43.39     |                  ^^^^^^^^^^^^^^^^^^^^^^^^^^
 0:43.39     |
 0:43.39    ::: C:\Users\Erich\workspace\mozilla-source\mozilla-unified\third_party\rust\warp\src\filter\mod.rs:40:19
 0:43.39     |
 0:43.40 40  |     type Extract: Tuple; // + Send;
 0:43.40     |                   ----- this associated type bound is unsatisfied for `impl Reply`
 0:43.40     |
 0:43.40     = note: `-D opaque-hidden-inferred-bound` implied by `-D warnings`
 0:43.40 help: add this bound
 0:43.40     |
 0:43.40 257 | ) -> impl Filter<Extract = impl warp::Reply + warp::generic::Tuple, Error = Rejection> + Clone {
 0:43.41     |                                             ++++++++++++++++++++++
 0:43.46 error: could not compile `webdriver` due to previous error

This error matches a new issue already in warp upstream, which has been resolved after a user workaround was found (changing impl Reply to be (impl Reply,). This blocks an eventual upgrade to Rust 1.66. Because of this, it can disrupt any Rust users may already be using.

Assignee: nobody → egubler
Status: NEW → ASSIGNED
Pushed by egubler@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/86a2b795327e Fix `webdriver::server::build_warp_routes` compile failures with Rust 1.66 r=webdriver-reviewers,jgraham
Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → 110 Branch
Summary: `testing/webdriver` emits `rustc` warning with Rust 1.66 → `testing/webdriver` emits `rustc` error with Rust 1.66
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: