Add fs::try_exists

[kevinkassimo]

Motivation

std has added Path::exists and (unstable) fs::try_exists. Adding try_exists to tokio::fs based on prior discussions in #3375 (which has not been updated for a while)

Solution

Add fs::try_exists function.
Fixes: #3373

[Darksonn]

Let's wait for std to stabilize try_exists. I'd rather not stabilize this, then see std change the signature of theirs so we don't match.

[isikkema]

try_exists has now been stabilized in std rust-lang/rust#97912.
Perhaps this should be revisited?

[Darksonn]

Sure! I see that it hasn't been released yet, but I would be happy to include our own version of it when that happens.

[just-chillin]

Unless I'm misunderstanding looks like std::path::try_exists has been stabilized? rust-lang/rust@77316a4

Would you folks find it helpful if I contributed a version using Path::try_exists?

[Darksonn]

Well, the documentation for fs::try_exists still says "This is a nightly-only experimental API". Once that changes, I would be happy with adding it to Tokio.

[kevinkassimo]

Didn't realize there are more updates on this PR after 2 years. I can still update this PR to make it work with the latest Tokio version once the API is stablized.

[Darksonn]

Ah, I see. It's been stabilized as Path::try_exists. That's an interesting situation because we don't have a Path type in Tokio. In that case, I am ok with adding an tokio::fs::try_exists as a replacement, since we cannot mirror what std is doing.

[kevinkassimo]

It seems that minrust is installing Rust 1.49.0 which is too hold for the feature? (at 1.63.0, already >= 6 months old) Can I update it in the ci.yml file?

[brody4hire]

As an outside contributor, I would personally consider that to be a breaking change and vote for something that will work with Rust 1.49, if possible. From a quick search I found this: https://stackoverflow.com/questions/32384594/how-to-check-whether-a-path-exists

But some may not agree with me.

See https://doc.rust-lang.org/cargo/reference/semver.html#env-new-rust

[kevinkassimo]

https://stackoverflow.com/questions/32384594/how-to-check-whether-a-path-exists

The was actually originally used in the implementation (before my recent sync). I can consider reverting it back if deemed better to use fs::metadata

[Darksonn]

Ah, yes, we need to use an implementation that will work on 1.49. Sorry if that was unclear. My main concern was matching the API in std, but it turns out that they have chosen an API we cannot match, so I am ok with moving forward with an fs::try_exists instead.

[kevinkassimo]

Updated implementation to use the exact fs::try_exists impl from std

[brody4hire]

Updated implementation to use the exact fs::try_exists impl from std

Another outsider comment I would kinda favor adding a comment with this info.

---

P.S. +1 on the explanatory comment added thanks!

[just-chillin]

Looks great! Thanks so much!

[Darksonn]

Thanks.

[just-chillin]

🙌🏻