1219983 - Update tooltool rustc to 1.4

Status: RESOLVED FIXED

(Firefox Build System :: General, defect)

Description

[Ralph Giles (:rillian)]

Now that rustc 1.4.0 is stable, we should bump the version available in tooltool for integration builds.

Comment 1

[Ralph Giles (:rillian)]

Attachment: Bump macosx64 tooltool manifest for rustc 1.4.0.

Build on MacOS X 10.10.5 (XCode 7.0.1) from https://static.rust-lang.org/dist/rustc-1.4.0-src.tar.gz with

MACOSX_DEPLOYMENT_TARGET=10.7
./configure --disable-elf-tls

Comment 2

[Ralph Giles (:rillian)]

Comment on attachment 8680955 [details] [diff] [review]
Bump macosx64 tooltool manifest for rustc 1.4.0.

Green on try. https://treeherder.mozilla.org/#/jobs?repo=try&revision=f6f1c59e9498

16:42:05     INFO -  checking rustc version... 1.4.0-dev (v1.4.0)
16:42:05     INFO -  checking rustc compatibility with MacOS X... 10.6 is ok with this rustc

Comment 3

[(not currently active) Ted Mielczarek]

Do you think we'll ever be able to use the stock rust release binaries?

Comment 4

[Ralph Giles (:rillian)]

So far the rust maintainers have been unsupportive.

Note we can use upstream release builds on Linux and Windows; it's just MacOS X 10.6 support that needs a custom build. And android where there are no upstream release binaries. I'm hoping I can automate the mac build with taskcluster, which will remove some of the pain.

Alex, can you update us on MacOS X 10.6 tls support plans? I recall you had a follow-up idea which would allow you to drop --disable-elf-tls.

Comment 5

[Alex Crichton [:acrichto]]

Unfortunately I haven't had the chance to flesh out a design to drop --disable-elf-tls, although I'd believe that it's still possible to do so! (as detailed in https://github.com/rust-lang/rust/issues/26581).

One point of note, however, is that we do actually have release binaries for Android now! We've got some more work to do to make them easily available, but all the pieces are in place to produce them and ensure they keep working.

How urgent is getting 10.6 binary support upstream? Is it bad enough that we should look into doing so sooner rather than later?

Comment 6

[(not currently active) Ted Mielczarek]

(In reply to Ralph Giles (:rillian) from comment #4)
> Note we can use upstream release builds on Linux and Windows; it's just
> MacOS X 10.6 support that needs a custom build. And android where there are
> no upstream release binaries. I'm hoping I can automate the mac build with
> taskcluster, which will remove some of the pain.

If you can cross-compile it from Linux then it should be straightforward to do in Taskcluster right now. If you need to build on a Mac then that's not something you can do just yet.

(In reply to Alex Crichton [:acrichto] from comment #5)
> How urgent is getting 10.6 binary support upstream? Is it bad enough that we
> should look into doing so sooner rather than later?

It's not a dealbreaker--it's not like we use upstream binaries for our other toolchains. It just seems silly to have to do a custom build of a toolchain that we're building in-house. :)

Comment 7

[Ralph Giles (:rillian)]

(In reply to Alex Crichton [:acrichto] from comment #5)

> One point of note, however, is that we do actually have release binaries for
> Android now!

How wonderful. They're not linked from https://www.rust-lang.org/downloads.html. I found https://static.rust-lang.org/dist/rust-std-nightly-arm-linux-androideabi.tar.gz (cross build of the rust standard library) but haven't figured out how to hook it up. Does cargo doesn't seem to know either.

Comment 8

[Alex Crichton [:acrichto]]

Right yeah, that's what I meant by "we've got some more work to do to make them easily available" :)

The gist of it is that the `rust-std-*` tarball has a `rustlib` directory containing the target triple (in this case arm-linux-androideabi), and if you put that in the normal rustc rustlib directory then rustc should be able to find the libs and cross compile to Andorid.

Comment 9

[Ralph Giles (:rillian)]

Aha. I opened bug 1220307 to follow up on the cross support questions.

Comment 10

[Pulsebot]

https://hg.mozilla.org/integration/mozilla-inbound/rev/4b3c08886ba9

Comment 11

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/4b3c08886ba9

Comment 12

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/releases/mozilla-b2g44_v2_5/rev/4b3c08886ba9