Minor: fix missing import in doc example

[petertodd]

Also, how do I get the doc-tests to actually test this example?

[Kixunil]

ACK 7b7085f

[Kixunil]

I suspect that rust after the triple backticks is interfering with running the tests. IIRC I've seen something like that elsewhere.

[tcharding]

cargo test --doc --all-features seems to run that code. I threw a panic in there to check it was run.

[petertodd]

cargo test --doc --all-features seems to run that code. I threw a panic in there to check it was run.

Thanks! That does seem to run the test. Though it actually passes without my change; I guess the test must import rand some other way.

[tcharding]

Oh rand is a dependency so its available without importing. Sorry I didn't look closer. Our CI would have to be broken if this code didn't work already.

[Kixunil]

Regarding the wasm failure, how come it broke now and not before? Looks like we're missing a pin somewhere but also the code needs to be fixed (possibly also in upstream).

[apoelstra]

@Kixunil wasm has been broken for months and months.

Lol @ "missing a pin". Good luck pinning anything in wasm. See my nearly-two-year-old "you can't pin anything" issue rustwasm/wasm-pack#1316

[apoelstra]

ACK 7b7085f; successfully ran local tests

[Kixunil]

@apoelstra clearly, this is some sort of compiler thing (though might be the cc crate). Anyway, that issue is actually very concerning to me for other reasons, so I'm going to check if I can fix it.

[Kixunil]

Oh, it looks like build should work, it's just install that doesn't.