changelog

CHANGELOG.md

@@ -30,6 +30,8 @@ That said, these are more guidelines rather than hardset rules, though the proje
 ### Changes
 
 - [#1559](https://github.com/ClementTsang/bottom/pull/1559): Rename `--enable_gpu` to `--disable_gpu`, and make GPU features enabled by default.
+- [#1570](https://github.com/ClementTsang/bottom/pull/1570): Consider `$XDG_CONFIG_HOME` on macOS when looking for a default config path in a
+  backwards-compatible fashion.
 
 ## [0.10.2] - 2024-08-05
 

src/options.rs

@@ -1219,7 +1219,10 @@ mod test {
         use std::path::PathBuf;
 
         // Case three: no previous config, no XDG var.
-        std::env::set_var("XDG_CONFIG_HOME", "");
+        // SAFETY: this is the only test that does this
+        unsafe {
+            std::env::remove_var("XDG_CONFIG_HOME");
+        }
 
         let case_1 = dirs::config_dir()
             .map(|mut path| {
@@ -1239,7 +1242,10 @@ mod test {
         assert_eq!(get_config_path(None), Some(case_2.clone()));
 
         // Case one: old non-XDG exists already
-        std::env::set_var("XDG_CONFIG_HOME", "");
+        // SAFETY: this is the only test that does this
+        unsafe {
+            std::env::remove_var("XDG_CONFIG_HOME");
+        }
         let case_3 = case_2;
         assert_eq!(get_config_path(None), Some(case_3));
     }