Rhai depends on unmaintained crate "instant"

[VorpalBlade]

According to https://crates.io/crates/instant:

This crate is no longer maintained. Please consider creating a fork or using web-time instead. Or reach out if you are interested in taking over its maintenance.

rhai should probably switch?

[schungx]

Hhhhmmm.... good catch.

Not sure if web-time would be a drop-in replacement...

As instant is primarily used only in WASM and I don't usually build for WASM... Somebody needs to test it out and let me know if everything is fine if we replace instant with web-time...

[VorpalBlade]

As instant is primarily used only in WASM and I don't usually build for WASM... Somebody needs to test it out and let me know if everything is fine if we replace instant with web-time...

Same, I was just adding rhai to a new project and was auditing the recursive dependencies it pulled in and noticed this. Since I don't target wasm I don't have any reason to work on this.

[0byteme]

As instant is primarily used only in WASM and I don't usually build for WASM... Somebody needs to test it out and let me know if everything is fine if we replace instant with web-time...

If use web-time instead of instant, will it cause a breaking change?

[schungx]

As instant is primarily used only in WASM and I don't usually build for WASM... Somebody needs to test it out and let me know if everything is fine if we replace instant with web-time...

If use web-time instead of instant, will it cause a breaking change?

Not sure... I haven't used web-time before... can somebody test it out?

[TrueDoctor]

Could we make pulling in time optional? We are currently thinking about adding Rhai support to graphite: GraphiteEditor/Graphite#2460 but in our case we don't need time support at all so it would be great if we could just disable this altogether

[schungx]

Try no_time?