Add cfg illlumos in addition to solaris#49
Conversation
|
@ogham is it possible to get this merged in so we no longer need to have a patched |
|
Ping here for my metadata agent work @ogham |
|
This now block the installation of cargo-make |
|
Hi @ogham, could we get this pull requested integrated and a new version released? Various downstream crates depend on it, and we'd really really like to get it working on illumos systems. Thanks! |
|
Since this project is unmaintained, I forked to https://github.com/rustadopt/uzers-rs where I for example merged this PR and maintain it from now on. It's also available in v0.11.2 on https://crates.io/crates/uzers |
|
@gierens Thank you I'll have a go at bringing it to projects that I get build errors from. |
I initially noticed this was broken when trying to install
fd-findon OmniOS viacargo install fd-find.illumos is its own distinct rust target now. Building with rust-users on a platform such as OmniOS or SmartOS that uses the illumos target via rustup is currently broken. Adding the proper cfg parameters fixes this. Note I chose to ignore the lint warning as it's not related to this change.
Test output: