Re: [PATCH 0/1] MR7717: Initial patch for converting Wine to Rust

[Jinoh Kang (＠iamahuman)]

Jinoh Kang (@iamahuman) commented about tools/wine/wine.rs:
> +use libloading::{Library, Symbol};
> +use std::{env, ffi::CString, fs::canonicalize, process::exit};
> -
>
> +#[cfg(target_os = "linux")]
> +fn get_bindir(argv0: &str) -> Option<String> {
> - if let Ok(real_path) = canonicalize("/proc/self/exe") {
> -
>    return Some(real_path.parent()?.to_str()?.to_string());
> - }
> - canonicalize(argv0)
> -
>    .ok()?
> -
>    .parent()?
> -
>    .to_str()
> -
>    .map(|s| s.to_string())
>
> +}
> -
>
> +#[cfg(target_os = "windows")]

I see that you're finally taking up the long overdue work to make Wine run on...Windows.

A giant step to the sommelierkind.

That said, split the windows support into its own commit.