1412290 - Port bug 1409533 - Require rust 1.21.0

Status: RESOLVED FIXED

(Thunderbird :: Build Config, enhancement)

Description

[Jorg K (CEST = GMT+2)]

For the update to rust 1.20.0 I landed this
https://hg.mozilla.org/comm-central/rev/5e346bc9fee657ba01309c1cd3d11ec97ef0ddb3
but now I don't know where to get the new numbers from.

Comment 1

[Jorg K (CEST = GMT+2)]

Ted and Ralph: Where do we get version, size and digest from?

Comment 2

[Jorg K (CEST = GMT+2)]

Actually, wrong changeset, that was for suite, I landed this:
https://hg.mozilla.org/comm-central/rev/7be864a6cef76ee4da1264c0430f9d02050cb97f

Comment 3

[Tom Prince [:tomprince]]

I've asked for the appropriate files to be uploaded to tooltool.

Comment 4

[Jorg K (CEST = GMT+2)]

And what about updating the C-C tooltool manifests?

Comment 5

[Tom Prince [:tomprince]]

We can't update the manifests until the files have actually been uploaded to tooltool.

Comment 6

[Jorg K (CEST = GMT+2)]

I've downloaded the linux64 package and I get:
  {
    "version": "rustc 1.21.0 (3b72af97e 2017-10-09) repack",
    "size": 164183024,
    "digest": "0f2451ac1382efe581c42c97d933aabb639bb13d9e75f0f71e241e04d0aa92fced133cff3c458c1f071d540ceac9a8c526dd1bab1d0de14bfe19fa77710941d4",
    "algorithm": "sha512",
    "filename": "rustc.tar.xz",
    "unpack": true
  }

Quite a complicated process for lin32+64, macosx64 and Win32+64 :-(

Comment 7

[Jorg K (CEST = GMT+2)]

Attachment: 1412290-rust.patch

Comment 8

[Tom Prince [:tomprince]]

Comment on attachment 8922975 [details] [diff] [review]
1412290-rust.patch

Review of attachment 8922975 [details] [diff] [review]:
-----------------------------------------------------------------

::: mail/config/tooltool-manifests/linux64/releng.manifest
@@ +20,1 @@
>      "size": 166736820,

This needs to be updated.

::: mail/config/tooltool-manifests/win64/releng.manifest
@@ +5,5 @@
>      "algorithm": "sha512",
>      "filename": "mozmake.exe"
>    },
>    {
>      "version": "rustc 1.20.0 (f3d6973f4 2017-08-27) repack",

The version should be updated here.

Comment 9

[Jorg K (CEST = GMT+2)]

Attachment: 1412290-rust.patch (v2)

I guess it's r+ with the (missing) copy/paste errors fixed ;-)

Comment 10

[Tom Prince [:tomprince]]

Comment on attachment 8922985 [details] [diff] [review]
1412290-rust.patch (v2)

Review of attachment 8922985 [details] [diff] [review]:
-----------------------------------------------------------------

::: mail/config/tooltool-manifests/linux64/releng.manifest
@@ +17,5 @@
>    },
>    {
> +    "version": "rustc 1.21.0 (3b72af97e 2017-10-09) repack",
> +    "size": 164183024,
> +    "164183024": "0f2451ac1382efe581c42c97d933aabb639bb13d9e75f0f71e241e04d0aa92fced133cff3c458c1f071d540ceac9a8c526dd1bab1d0de14bfe19fa77710941d4",

This should be digest.

Comment 11

[Jorg K (CEST = GMT+2)]

Attachment: 1412290-rust.patch

Third time lucky ;-(

Comment 12

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/comm-central/rev/98161291e20d
Port bug 1409533 - Require rust 1.21.0 - Rust details from bug 1412352. r=tomprince

Comment 13

[Ralph Giles (:rillian)]

Jorg, for future versions of this bug, you might find my script at https://github.com/rillian/rust-build/blob/master/splat_rust.py helpful. It copies manifest.tt into hardcoded tooltool manifest files.