mneumann / rust-msgpack

[DEPRECATED] msgpack.org implementation for Rust language. Please use https://github.com/3Hren/msgpack-rust
121 stars 31 forks source link

Update to stable rustc and current byteorder. #56

Open zbynekwinkler opened 8 years ago

zbynekwinkler commented 8 years ago

Removes #![feature(slice_splits)]. Replaces byteorder::{Error,Result} with std::io::{Error,Result}. Fixes all warnings (hopefully not breaking things). Updates github link in readme.

Fixes #55.

ShadowNinja commented 8 years ago

Bump The package is broken without this.

Also, the docs should be updated, the serialize crate has apparently been replaced with rustc_serialize:

error: use of unstable library feature 'rustc_private': deprecated in favor of rustc-serialize on crates.io (see issue #27812)
extern crate serialize;
^~~~~~~~~~~~~~~~~~~~~~~
kallisti5 commented 8 years ago

Any love for this? I'd like to play with msgpack in rust but the only implementation is broken. The package from cargo is broken. (1.11.0)

bkfox commented 7 years ago

up! I think a lot of people need this one.

Enet4 commented 7 years ago

Has the author shown any intent to maintain this project? The repository could seriously use an active maintainer, with the number of people interested in using and contributing to the crate.

avongluck-r1soft commented 7 years ago

I don't have the bandwidth to take on another project... however maybe a shared repo between a bunch of people to keep up with it? Original author seems in the wind.

lbolla commented 7 years ago

I am in favour of forking this repo, fixing it and uploading it to crates.io to replace the broken one. Also, updating http://msgpack.org/ to point to the new fixed version.

Enet4 commented 7 years ago

While I understand it's not very "correct" to advertise other solutions, it seems best for us to use and contribute to the RMP project instead. This one is actively maintainted and provides Serde integration. The official msgpack.org website does seem to point to RMP on their website already.