Fix cargo-xwin build by using `CARGO_CFG_TARGET_OS` env var instead of `#[cfg(target_os = "x")`

[dzervas]

To my surprise something is wrong with the #[cfg(target_os = "whatever")]. My journey started when I tried to build my project that uses the frida crate using cargo-xwin - I wanted to cross-compile it from a linux host to target x86_64-pc-windows-msvc. When I did that the linker nagged that it couldn't find pthread.lib and resolv.lib - which are not windows libraries. They shouldn't be required. The problem was that for some reason the target_os value was not set according to the target but rather according to the host

I THINK it's due to the build.rs being targeted to the host (as it's the build script).

Some useful links:

• https://github.com/rust-cross/cargo-xwin/issues/35
• https://github.com/Byron/trash-rs/issues/39#issue-981022414
• https://kazlauskas.me/entries/writing-proper-buildrs-scripts

If this gets merged can we get a new patch release please?

[meme]

Likewise, but I wonder what happened to the checks here... can you maybe push another commit to re-run the CI?

[dzervas]

Likewise, but I wonder what happened to the checks here... can you maybe push another commit to re-run the CI?

Problem is it 404s in some packages. Me committing somethings won't change anything, as I see that you tried to re-run the jobs

Maybe something's wrong?

[s1341]

I've fixed CI in https://github.com/frida/frida-rust/pull/97. Once it's merged, you can rebase and we can merge this.

[meme]

If you rebase CI should be passing. Thanks.

[dzervas]

just fixed the formatting

[s1341]

@meme can you please cut a version with this change?

[meme]

Done!