Remove deprecated "use-std"

[Dylan-DPC-zz]

I'm submitting a(n) removal

Description

feature use-std was renamed earlier in #108 and brought back and marked deprecated in #158. Removing it now

Related Issue(s)

#108, #158

[kinggoesgaming]

I am not really comfortable with removing it quite yet. I think we should reserve this for 0.8 and aggressively advertise the removal in 0.7.0

@KodrAus opinions?

[Dylan-DPC-zz]

aggressively advertise the removal in 0.7.0

Deprecating something is aggressively advertising people to remove it.

[kinggoesgaming]

I am not strictly against this... but would love some opinion from people who use this library (especially those who provide stability guarantees like 1.0+). We might accidently definitely break the dependents of those libraries, and pretty much contribute to the dependency hell.

The way I think the way to go would be for 0.7 we have warnings in the release notes and in main crate documentation, and then in 0.8 (which is meant as a breaking change to touch the public API so that it conforms with Rust API guidelines, see #191). Since this is technically a public API wart, we should go this way.

Again I am happy if this gets merged in 0.7, but just want a smoother transition for users. We are already covering a huge surface area with 0.7.

We might add a build.rs script with the following in 0.7:

fn main() {
    println!("cargo:warning=r#"[uuid] feature gate "use_std" is deprecated and is marked for removal. Use feature gate "std" instead."#);
}

Edit: added some more info.
Edit 2: added suggestion about build script warning.
Edit 3 and 4: typos 🤦‍♂️

[Dylan-DPC-zz]

The users know that 0.7 is an breaking change, so they will expect stuff to break. Also most people use "0.6" as their dependency selector (if they use anything else such as *then there are multiple things which will already break their API), so it is not that this change will break their API overnight, and they will not just change the version from 0.6 to 0.7 without going through the list of things that have changed (which already consists of multiple API changes in the crate).

[kinggoesgaming]

Fine 🙄

[kinggoesgaming]

bors r+

[bors]

Build succeeded

• continuous-integration/travis-ci/push
• continuous-integration/appveyor/branch