1321696 - Require Rust 1.13

Status: RESOLVED FIXED

(Firefox Build System :: General, defect)

Description

[Ralph Giles (:rillian)]

Bug 1320425 landed code which uses the '?' operator, stabilized in Rust 1.13.0. We should update the configure check to ensure this, rather than failing late in the build.

Comment 1

[Ralph Giles (:rillian)]

Attachment: Bug 1321696 - Require rust 1.13 with --enable-rust.

Bug 1320425 using the '?' operator stabilized in rust 1.13.0.
Update the minimum supported version to reflect this.

Review commit: https://reviewboard.mozilla.org/r/97106/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/97106/

Comment 2

[Ralph Giles (:rillian)]

Nathan, you've argued we should hold off bumping the minimum
for another three weeks until 1.14 is stable: we plan to bump
the minimum then too, because we need cargo's Workspace feature,
and doing so twice in the same month is a burden on developers.

I don't find this pursuasive. Anyone who installed rust in
response to my dev-platform post last week about `./mach boostrap`
and will have 1.13. Anyone who installs it when the build
starts requiring rust will have 1.13 or 1.14 depending on when
bug 1283898 lands. The only devs who will see the double bump
are those who've already experimented with rust in the past
either with an --enable-rust in mozconfig or in projects outside
gecko. Either way they will have previous experience installing
rust, and it's reasonable to consider early adoption as opting-in
to a little more effort.

It's a better argument for not having sent my announcement last
week, but I wasn't aware at the time you were planning to bump
only when 1.14 came out. :/

Comment 3

[Mike Hommey [:glandium]]

In the future, things that require new compiler features should land *after* a requirement bump.

Comment 4

[(not currently active) Ted Mielczarek]

(In reply to Ralph Giles (:rillian) needinfo me from comment #2)
> Nathan, you've argued we should hold off bumping the minimum
> for another three weeks until 1.14 is stable: we plan to bump
> the minimum then too, because we need cargo's Workspace feature,
> and doing so twice in the same month is a burden on developers.

cargo workspaces are supported in cargo 0.13, which shipped with Rust 1.12, IIRC.

Comment 5

[Ralph Giles (:rillian)]

Ah, I must have misunderstood the IRC discussion. Is there a reason to bump the minimum to 1.14 then?

Comment 6

[Ralph Giles (:rillian)]

(In reply to Mike Hommey [:glandium] from comment #3)
> In the future, things that require new compiler features should land *after*
> a requirement bump.

I think the only way to ensure this is with automation. I opened bug 1321847 about that.

Comment 7

[Nathan Froyd [:froydnj]]

(In reply to Ralph Giles (:rillian) needinfo me from comment #5)
> Ah, I must have misunderstood the IRC discussion. Is there a reason to bump
> the minimum to 1.14 then?

Bumping the minimum to 1.14 would ensure that people who tried to compile Fennec would have a working compiler.  I don't know of other compelling reasons (wanted: in-progress release notes for the Beta Rust channel).

Comment 8

[Ralph Giles (:rillian)]

(In reply to Nathan Froyd [:froydnj] from comment #7)

> Bumping the minimum to 1.14 would ensure that people who tried to compile
> Fennec would have a working compiler.

Ah, right. I should write all this stuff down in one place somewhere.

Comment 9

[Chris Manchester (limited bugmail, email directly)]

Comment on attachment 8816338 [details]
Bug 1321696 - Require rust 1.13 with --enable-rust.

https://reviewboard.mozilla.org/r/97106/#review97578

As far as I can tell the argument from comment 3 is fine and still applies if the reason we want 1.14 when it comes out is for Android, not workspaces.

Comment 10

[Chris Manchester (limited bugmail, email directly)]

(In reply to Chris Manchester (:chmanchester) from comment #9)
> Comment on attachment 8816338 [details]
> Bug 1321696 - Require rust 1.13 with --enable-rust.
> 
> https://reviewboard.mozilla.org/r/97106/#review97578
> 
> As far as I can tell the argument from comment 3 is fine and still applies
> if the reason we want 1.14 when it comes out is for Android, not workspaces.

Actually that's comment 2.

Comment 11

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/02aa6ad54a6e
Require rust 1.13 with --enable-rust. r=chmanchester

Comment 12

[Phil Ringnalda (:philor)]

https://hg.mozilla.org/mozilla-central/rev/02aa6ad54a6e