Updates for modern nightly Rust and latest dependencies#75
Open
iceiix wants to merge 45 commits intoThinkofname:masterfrom
Open
Updates for modern nightly Rust and latest dependencies#75iceiix wants to merge 45 commits intoThinkofname:masterfrom
iceiix wants to merge 45 commits intoThinkofname:masterfrom
Conversation
Allows upgrading from nightly-2017-10-12 to nightly-2018-01-04 which is aligned with Rust version 1.23.0 https://github.com/rust-lang/rust/blob/master/RELEASES.md#version-1230-2018-01-04 See https://github.com/iceiix/steven/issues/3#issuecomment-425688203
… nightly-2018-02-15
For https://github.com/iceiix/steven/issues/5 GL_FRAMEBUFFER_INCOMPLETE_MISSING_ATTACHMENT
No errors, so not the cause of https://github.com/iceiix/steven/issues/5
For investigating https://github.com/iceiix/steven/issues/5 No errors seen
* Replace find() with get() * Update for renamed as_string->as_str and as_boolean->as_bool https://github.com/serde-rs/json/releases/tag/v0.8.0 Value::as_string() has been renamed to as_str() and Value::as_boolean() has been renamed to as_bool() to improve consistency serde-rs/json#126 * No serde_json::Value::I64/U64/F64 anymore, only Number * Update from lookup() to pointer(), using JSON pointer syntax #6 (comment) * Remove unused and removed ObjectBuilder import * Use into_iter().collect() to convert BTreeMap to serde_json::Map * Change parse_rules to accept serde_json::Map instead of BTreeMap * Remove unused serde_json macro_use * Update Cargo.lock
An old version of hyper was used before (0.8.0), in the process of updating to hyper 0.12.11, found this higher-level replacement/wrapper, reqwest 0.9.4 which is simpler to use than the latest hyper and serves the purpose of a simple HTTP client well * Begin updating to hyper 0.12.11 https://github.com/iceiix/steven/issues/4#issuecomment-425759778 * Use type variables for hyper::Client * Fix setting header syntax, Content-Type: application/json, 17->13 * Parse strings into URLs with url.parse::<hyper::Uri>().unwrap() https://github.com/hyperium/hyper/blob/b20971cb4e5f158844aec5829eea1854e5b7d4b6/examples/client.rs#L25 * Use hyper::Request::post() then client.request() since client.post() removed * wait() on the ResponseFuture to get the Result * try! to unwrap the Result * status() is now a method * Concatenate body chunks unwrap into bytes, then parse JSON from byte slice, instead of from_reader which didn't compile * Replace send() with wait() on ResponseFuture * Parse HeaderValue to u64 * Slices implement std::io::Read trait * Read into_bytes() instead of read_to_end() * Disable boxed logger for now to workaround 'expected function, found macro' * Remove unnecessary mutability, warnings * Hack to parse twice to avoid double move * Use hyper-rustls pure Rust implementation for TLS for HTTPS in hyper * Start converting to reqwest: add Protocol::Error and reqwest::Error conversion * Use reqwest, replacing hyper, in protocol * Convert resources to use reqwest instead of hyper * Convert skin download to reqwest, instead of hyper * Remove hyper * Revert unnecessary variable name change req/body to reduce diff * Revert unnecessary whitespace change to reduce diff, align indentation on . * Fix authenticating to server, wrong method and join URL * Update Cargo.lock
Major API change, the last of the outdated dependencies Closes https://github.com/iceiix/steven/issues/4 Note: would still like to replace the last usages of the OpenSSL crate https://github.com/iceiix/steven/issues/2 but it is needed for CFB8 until a replacement is available (maybe RustCrypto/stream-ciphers#4)
https://travis-ci.org/iceiix/steven/jobs/447445632 The following packages have unmet dependencies: libsdl2-dev : Depends: libegl1-mesa-dev Depends: libgles2-mesa-dev
bjorn3
reviewed
Oct 29, 2018
src/main.rs
Outdated
| @@ -180,7 +180,7 @@ fn main() { | |||
| let proxy = console::ConsoleProxy::new(con.clone()); | |||
|
|
|||
| // TODO: fix error[E0423]: expected function, found macro `log::set_boxed_logger` | |||
Author
|
If anyone is interested in contributing, I'm continuing this project here: https://github.com/iceiix/stevenarella |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Steven wouldn't build with modern versions of nightly Rust due to various languages changes and deprecations, with this pull request it now does. Tested on nightly-2018-10-24. Also updated to the latest versions of all the dependencies, replacing deprecated crates where possible, bringing Steven into the modern age (Rust 2018).