feat: icon for windows

Author: claration

Cargo.lock

@@ -24,3 +24,4 @@ keyring = { version = "3.6.3", default-features = false, features = ["windows-na
 
 [target.'cfg(target_os = "windows")'.build-dependencies]
 embed-manifest = "1.4"
+winres = "0.1"

apps/plumeimpactor/Cargo.toml

@@ -1,12 +1,9 @@
 #[cfg(windows)]
 fn main() {
     println!("cargo:rerun-if-changed=build.rs");
-    let target = std::env::var("TARGET").unwrap_or_default();
 
-    if target.contains("windows") {
-        let pkg_name = std::env::var("CARGO_PKG_NAME").unwrap();
-        embed_windows_manifest(&pkg_name);
-    }
+    let pkg_name = std::env::var("CARGO_PKG_NAME").unwrap();
+    embed_windows_manifest(&pkg_name);
 }
 
 #[cfg(not(windows))]
@@ -17,15 +14,20 @@ fn embed_windows_manifest(name: &str) {
     use embed_manifest::manifest::{ActiveCodePage, Setting, SupportedOS::*};
     use embed_manifest::{embed_manifest, new_manifest};
 
-    let manifest = new_manifest(name)
-        .supported_os(Windows7..=Windows10)
-        .active_code_page(ActiveCodePage::Utf8)
-        .heap_type(embed_manifest::manifest::HeapType::SegmentHeap)
-        .dpi_awareness(embed_manifest::manifest::DpiAwareness::PerMonitorV2)
-        .long_path_aware(Setting::Enabled);
+    {
+        let manifest = new_manifest(name)
+            .supported_os(Windows7..=Windows10)
+            .active_code_page(ActiveCodePage::Utf8)
+            .heap_type(embed_manifest::manifest::HeapType::SegmentHeap)
+            .dpi_awareness(embed_manifest::manifest::DpiAwareness::PerMonitorV2)
+            .long_path_aware(Setting::Enabled);
+
+        embed_manifest(manifest).unwrap();
+    }
 
-    if let Err(e) = embed_manifest(manifest) {
-        println!("cargo:warning=Failed to embed manifest: {e}");
-        println!("cargo:warning=The application will still work but may lack optimal Windows theming");
+    {
+        let mut res = winres::WindowsResource::new();
+        res.set_icon("../../package/windows/icon.ico");
+        res.compile().unwrap();
     }
 }