Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Implement automatic Rust source updating. #19

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Implement automatic Rust source updating. #19

wants to merge 4 commits into from

Conversation

kevinpoitra
Copy link

@kevinpoitra kevinpoitra commented Apr 11, 2016

This isn't ready for merging yet.

Currently, it downloads the nightly source from Rust's website to the appData/tokamak/ directory, then ungzip and untars it to the /tokamak/rust-sources directory, and makes use of a progress notification.

Stuff left to do:

  • Point Racer to the correct directory for the updated files.
  • Set up configuration for the updater. We'll probably want to support three different groups of users; one who will just let the autoupdater manage everything, another who will want to set the source directory path but let the autoupdater do the rest, and lastly those who would want to manually manage their source files and turn off the auto updater.
  • Move file downloading and extracting to a temporary directory and overwrite the actual sources once we've ensured that the extraction was successful.
  • Add support for the beta and stable source files.
  • Download and store the rustc-xx-src.tar.gz.sha256 checksum for use in update checking and download validation.

@vertexclique
Copy link
Owner

👍 @kevinpoitra seems awesome!

@vertexclique
Copy link
Owner

I will add:

  • Command for updating Rust source

And with that, stable and beta channel support.
and add support for using a temporary directory for extraction.
@vertexclique
Copy link
Owner

bump? @kevinpoitra

@vertexclique
Copy link
Owner

vertexclique commented May 20, 2016

bump again? @kevinpoitra @PuppyKevin

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants