Shift to serde from rustc-serialize: rustc-serialize deprecated

[dtolnay]

It has been deprecated: announcement.

[pavanky]

@dtolnay care to send in a PR ?

[dtolnay]

Nope, just canvassing to make sure you've planned for it in case it affects your API. 🙂

[pavanky]

@dtolnay Thanks for the heads up.

[9prady9]

Thanks @dtolnay . The replacement crate is serde. Renaming the issue to reflect the shift to serde.

[pavanky]

@9prady9 what is this used for ? Do we really need it?

[9prady9]

It is build-script dependency, not used by crates that are used by users. It is only needed by someone who is developing arrayfire-rust.

[9prady9]

It is used to parse the json config file.

[9prady9]

I have been thinking about it for a while now - we can remove this dependency if the rust wrapper doesn't try to build arrayfire locally - separate from installed binaries.

[9prady9]

@pavanky and @jramapuram What do you guys think about my last comment of removing option to build ArrayFire locally ?

[jramapuram]

Should be fair enough that AF is expected to be installed. I liked having the af as a submodule because it keeps the tight coupling as well. If we can be a little more explicit about this that would be good.

[9prady9]

I have no objections to either way of doing it. We just have to update the current config and build file to reflect latest changes in build procedure.