dirs-dev / dirs-rs

a low-level library that provides config/cache/data paths, following the respective conventions on Linux, macOS and Windows

Home Page:https://dirs.dev

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Redox

naturallymitchell opened this issue · comments

commented

Hi @mitchtbaum, what changes do you suggest?

this would seem worthwhile to work on Redox, in addition to Lin, Win, and Mac.

but actually, #12 is much better to look into first.

commented

As far as I see, Redox is already supported. Anything that is missing there?

I haven't tried it, so hard to say.

If you could add that info to the Readme, we could help push it along a bit.

commented

It's the second operating system mentioned in the README, after Linux and before Windows and macOS.

Okay. Duh.

I brushed by it and looked mainly at the Examples and Features table. The features table using a slash after Linux is hard to parse accurately+quickly, imho. What do you think about changing it to an "and"? And If the examples could also include "Redox", that'd be good. What do you think?

also the project's description line

Would be nice if there was a smaller crate than redox_users to depend to, since redox_users depends on argon2, failure, failure-derive, rand_os and a bunch of other sub-dependencies.

I know it's only included if built for redox, but still.

commented

I'd close this, dirs-sys-rs is using redox_users = {default-features = false } for a while now, which seems to avoid a lot of dependencies.

Feel free to reopen!