Update rustup link. Bug: 1662960

[cholcombe973]

Rustup has changed over to a new location .

Thanks for helping us make a better Snapcraft!

Checklist:

• Have you signed the contributor licence agreement? Yes
  LP: 1662960
• Have you read our contribution guide? Yes

[sergiusens]

thanks for this but mind fixing the tests? If you do it today it will get into 2.27

[cholcombe973]

Yeah I'm working on them now.

…

On Wed, Feb 8, 2017 at 10:54 AM, Sergio Schvezov ***@***.***> wrote: thanks for this but mind fixing the tests? If you do it today it will get into 2.27 — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#1120 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AC6qE0AlSZlNlAMUxY5FVX33mq6V84Dnks5rag98gaJpZM4L7Fms> .

[cholcombe973]

@sergiusens I think this should fix it up

[sergiusens]

i am seeing things like
Expected: [call(['/tmp/tmp_345efam/parts/test-part/rust/sh.rustup.rs', '--prefix=/tmp/tmp_345efam/parts/test-part/rust', '--disable-sudo', '--save', '--channel=nightly']), call(['/tmp/tmp_345efam/parts/test-part/rust/bin/cargo', 'fetch', '--manifest-path', '/tmp/tmp_345efam/parts/test-part/src/Cargo.toml'])]
Actual: [call(['/tmp/tmp_345efam/parts/test-part/rust/rustup.sh', '--prefix=/tmp/tmp_345efam/parts/test-part/rust', '--disable-sudo', '--save', '--channel=nightly']),
call(['/tmp/tmp_345efam/parts/test-part/rust/bin/cargo', 'fetch', '--manifest-path', '/tmp/tmp_345efam/parts/test-part/src/Cargo.toml'])]

[elopio]

wow, this one supports aarch64!
Do you know where is the git repository for this script?

[cholcombe973]

@elopio I do. It's here: https://github.com/rust-lang-nursery/rustup.rs :)

[sergiusens]

@elopio @cholcombe973 reading the script it still doesn't support 64 bit kernels on 32 bit user spaces

[cholcombe973]

Ah crap I broke a bunch of things. This new rustup script doesn't act like the previous one. Let me work this out.

[cholcombe973]

I think i need to abandon this change. The new rustup is significantly different from the old one and it no longer supports an install prefix.

[sergiusens]

should we close this or re-evaluate. I suppose we can propose fixes upstream. They reviewed my stuff pretty quick last time.

[cholcombe973]

Yeah I think we should close this for now. I'll try to work with upstream to get things fixed before proposing a new PR

…

On Thu, Feb 9, 2017 at 4:18 AM, Sergio Schvezov ***@***.***> wrote: should we close this or re-evaluate. I suppose we can propose fixes upstream. They reviewed my stuff pretty quick last time. — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#1120 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AC6qE-ZAXE1PYqu1XDs8MRFiz4sCUbzzks5rawQDgaJpZM4L7Fms> .

[kyrofa]

Alright, closing for now.