Skip to content

rustup-init: use no-self-update feature.#32701

Closed
reitermarkus wants to merge 3 commits intoHomebrew:masterfrom
reitermarkus:rustup-no-self-update
Closed

rustup-init: use no-self-update feature.#32701
reitermarkus wants to merge 3 commits intoHomebrew:masterfrom
reitermarkus:rustup-no-self-update

Conversation

@reitermarkus
Copy link
Copy Markdown
Member

  • Have you followed the guidelines for contributing?
  • Have you checked that there aren't other open pull requests for the same formula update/change?
  • Have you built your formula locally with brew install --build-from-source <formula>, where <formula> is the name of the formula you're submitting?
  • Does your build pass brew audit --strict <formula> (after doing brew install <formula>)?

@fxcoudert
Copy link
Copy Markdown
Member

sha256 changed without the version also changing; please create an issue upstream to rule out malicious circumstances and to find out why the file changed

😭

@reitermarkus
Copy link
Copy Markdown
Member Author

rust-lang/rustup#1513

Sorry for the confusion here, 1.14.0 isn't published or ready for release yet.

@commitay
Copy link
Copy Markdown
Contributor

commitay commented Oct 4, 2018

sha256 has changed again, hopefully for the last time.

rust-lang/rustup#1513

Ok, this is published now.

@reitermarkus reitermarkus force-pushed the rustup-no-self-update branch from 1d686f1 to 0a70c0a Compare October 4, 2018 22:22
@fxcoudert
Copy link
Copy Markdown
Member

We probably want a revision bump, since the release was updated.

@reitermarkus reitermarkus force-pushed the rustup-no-self-update branch from 0a70c0a to 24ecda1 Compare October 4, 2018 22:27
@commitay commitay closed this in b228bdf Oct 6, 2018
@reitermarkus reitermarkus deleted the rustup-no-self-update branch October 7, 2018 00:32
@lock lock bot added the outdated PR was locked due to age label Nov 6, 2018
@lock lock bot locked as resolved and limited conversation to collaborators Nov 6, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

outdated PR was locked due to age

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants