1261503 - Update win32 rust to target i586-pc-windows-msvc

Status: RESOLVED FIXED

(Firefox Build System :: General, defect)

Description

[Ralph Giles (:rillian)]

Upstream rust now has nightly builds of the standard library for the i586-pc-windows-msvc target. This should be a more permanent fix for the sse2 instruction issues.

Comment 1

[Ralph Giles (:rillian)]

Attachment: Update win32 tooltool rust to 1.9 nightly + i586 target

Squash of patches pushed to try as https://treeherder.mozilla.org/#/jobs?repo=try&revision=5eba08277f42&selectedJob=18928571

Comment 2

[Michael Shal [:mshal]]

Comment on attachment 8737382 [details] [diff] [review]
Update win32 tooltool rust to 1.9 nightly + i586 target

>diff --git a/browser/config/tooltool-manifests/win32/releng.manifest b/browser/config/tooltool-manifests/win32/releng.manifest
>+"filename": "rustc-i586-pc-windows-msvc_20160401.tar.bz2",

Actually, this should probably just be "rustc.tar.bz2", since tooltool uses the basename of the package to determine how to remove the previous package. In this case, the package contains a "rustc" directory, so by naming the file rustc.tar.bz2, tooltool will try to rmtree('rustc') and remove the old package. As-is, tooltool will try to rmtree('rustc-i586-pc-windows-msvc_20160401'), which doesn't exist. The contents are still unpacked into the rustc directory, overwriting anything there. This would only present a problem if you remove a file in a new package, and if its presence would cause problems somehow.

Comment 3

[Ralph Giles (:rillian)]

Attachment: Update win32 tooltool rust to 1.9 nightly + i586 target v2

> Actually, this should probably just be "rustc.tar.bz2"

Ok changed. Carrying forward r=mshal

Unfortunate though. It's really nice to have a distinguishing filename so one has some idea where the build comes from.

Comment 4

[Pulsebot]

https://hg.mozilla.org/integration/mozilla-inbound/rev/8c19cce84a1b

Comment 5

[Mike Hommey [:glandium]]

(In reply to Ralph Giles (:rillian) from comment #3)
> Created attachment 8737857 [details] [diff] [review]
> Update win32 tooltool rust to 1.9 nightly + i586 target v2
> 
> > Actually, this should probably just be "rustc.tar.bz2"
> 
> Ok changed. Carrying forward r=mshal
> 
> Unfortunate though. It's really nice to have a distinguishing filename so
> one has some idea where the build comes from.

That's why we have "version" in gcc and clang stanzas.

Comment 6

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/8c19cce84a1b