chore: release 6.1.14

CHANGELOG.md

@@ -2,6 +2,12 @@
 
 ## [Unreleased]
 
+## [v6.1.14]
+
+### Changed
+- Fix formatting
+- Attempt to fix tests
+
 ## [v6.1.13]
 
 ### Changed

Cargo.lock

@@ -155,7 +155,7 @@ checksum = "175571dd1d178ced59193a6fc02dde1b972eb0bc56c892cde9beeceac5bf0f6b"
 
 [[package]]
 name = "asusctl"
-version = "6.1.13"
+version = "6.1.14"
 dependencies = [
  "dmi_id",
  "env_logger",
@@ -174,7 +174,7 @@ dependencies = [
 
 [[package]]
 name = "asusd"
-version = "6.1.13"
+version = "6.1.14"
 dependencies = [
  "cargo-husky",
  "concat-idents",
@@ -201,7 +201,7 @@ dependencies = [
 
 [[package]]
 name = "asusd-user"
-version = "6.1.13"
+version = "6.1.14"
 dependencies = [
  "config-traits",
  "dirs",
@@ -892,7 +892,7 @@ dependencies = [
 
 [[package]]
 name = "config-traits"
-version = "6.1.13"
+version = "6.1.14"
 dependencies = [
  "log",
  "ron",
@@ -1232,7 +1232,7 @@ dependencies = [
 
 [[package]]
 name = "dmi_id"
-version = "6.1.13"
+version = "6.1.14"
 dependencies = [
  "log",
  "udev 0.8.0",
@@ -4371,7 +4371,7 @@ dependencies = [
 
 [[package]]
 name = "rog-control-center"
-version = "6.1.13"
+version = "6.1.14"
 dependencies = [
  "asusd",
  "concat-idents",
@@ -4402,7 +4402,7 @@ dependencies = [
 
 [[package]]
 name = "rog_anime"
-version = "6.1.13"
+version = "6.1.14"
 dependencies = [
  "dmi_id",
  "gif 0.12.0",
@@ -4416,7 +4416,7 @@ dependencies = [
 
 [[package]]
 name = "rog_aura"
-version = "6.1.13"
+version = "6.1.14"
 dependencies = [
  "dmi_id",
  "log",
@@ -4427,7 +4427,7 @@ dependencies = [
 
 [[package]]
 name = "rog_dbus"
-version = "6.1.13"
+version = "6.1.14"
 dependencies = [
  "asusd",
  "rog_anime",
@@ -4441,7 +4441,7 @@ dependencies = [
 
 [[package]]
 name = "rog_platform"
-version = "6.1.13"
+version = "6.1.14"
 dependencies = [
  "concat-idents",
  "inotify",
@@ -4454,7 +4454,7 @@ dependencies = [
 
 [[package]]
 name = "rog_profiles"
-version = "6.1.13"
+version = "6.1.14"
 dependencies = [
  "log",
  "rog_platform",
@@ -4465,7 +4465,7 @@ dependencies = [
 
 [[package]]
 name = "rog_scsi"
-version = "6.1.13"
+version = "6.1.14"
 dependencies = [
  "ron",
  "serde",
@@ -4475,7 +4475,7 @@ dependencies = [
 
 [[package]]
 name = "rog_simulators"
-version = "6.1.13"
+version = "6.1.14"
 dependencies = [
  "log",
  "rog_anime",
@@ -4485,7 +4485,7 @@ dependencies = [
 
 [[package]]
 name = "rog_slash"
-version = "6.1.13"
+version = "6.1.14"
 dependencies = [
  "dmi_id",
  "serde",

Cargo.toml

@@ -1,5 +1,5 @@
 [workspace.package]
-version = "6.1.13"
+version = "6.1.14"
 rust-version = "1.82"
 license = "MPL-2.0"
 readme = "README.md"

asusctl/examples/anime-gif.rs

@@ -21,11 +21,14 @@ fn main() {
     let brightness = args[2].parse::<f32>().unwrap();
     let anime_type = get_anime_type();
     let mut seq = Sequences::new(anime_type);
-    seq.insert(0, &ActionLoader::AsusAnimation {
+    seq.insert(
-        file: path.into(),
+        0,
-        time: rog_anime::AnimTime::Infinite,
+        &ActionLoader::AsusAnimation {
-        brightness,
+            file: path.into(),
-    })
+            time: rog_anime::AnimTime::Infinite,
+            brightness,
+        },
+    )
     .unwrap();
 
     loop {

asusd/src/asus_armoury.rs

@@ -160,7 +160,11 @@ impl crate::Reloadable for AsusArmouryAttribute {
                         self.attr.base_path_exists();
                         e
                     })?;
-                info!("Restored armoury setting {} to {:?}", self.attr.name(), saved_value);
+                info!(
+                    "Restored armoury setting {} to {:?}",
+                    self.attr.name(),
+                    saved_value
+                );
             }
         }
 
@@ -412,21 +416,19 @@ pub async fn start_attributes_zbus(
         match zbus::object_server::SignalEmitter::new(conn, path) {
             Ok(sig) => {
                 if let Err(e) = attr.watch_and_notify(sig).await {
-                    error!(
+                    error!("Failed to start watcher for '{}': {e:?}", attr.attr.name());
-                        "Failed to start watcher for '{}': {e:?}",
-                        attr.attr.name()
-                    );
                 }
             }
             Err(e) => {
-                error!("Failed to create SignalEmitter for '{}': {e:?}", attr.attr.name());
+                error!(
+                    "Failed to create SignalEmitter for '{}': {e:?}",
+                    attr.attr.name()
+                );
             }
         }
 
         if let Err(e) = attr.move_to_zbus(conn).await {
-            error!(
+            error!("Failed to register attribute '{attr_name}' on zbus: {e:?}");
-                "Failed to register attribute '{attr_name}' on zbus: {e:?}"
-            );
         }
     }
     Ok(())

asusd/src/aura_laptop/config.rs

@@ -82,8 +82,9 @@ impl AuraConfig {
             config
                 .builtins
                 .insert(*n, AuraEffect::default_with_mode(*n));
-
+        }
-            if !config.support_data.basic_zones.is_empty() {
+        if !config.support_data.basic_zones.is_empty() {
+            for n in &config.support_data.basic_modes {
                 let mut default = vec![];
                 for (i, tmp) in config.support_data.basic_zones.iter().enumerate() {
                     default.push(AuraEffect {
@@ -118,14 +119,14 @@ impl AuraConfig {
             self.multizone_on = false;
         } else {
             if let Some(multi) = self.multizone.as_mut() {
-                if let Some(fx) = multi.get_mut(effect.mode()) {
+                if let Some(fx_vec) = multi.get_mut(effect.mode()) {
-                    for fx in fx.iter_mut() {
+                    for fx in fx_vec.iter_mut() {
                         if fx.zone == effect.zone {
                             *fx = effect;
                             return;
                         }
                     }
-                    fx.push(effect);
+                    fx_vec.push(effect);
                 } else {
                     multi.insert(*effect.mode(), vec![effect]);
                 }
@@ -294,26 +295,38 @@ mod tests {
         let res = config.multizone.unwrap();
         let sta = res.get(&AuraModeNum::Static).unwrap();
         assert_eq!(sta.len(), 4);
-        assert_eq!(sta[0].colour1, Colour {
+        assert_eq!(
-            r: 0xff,
+            sta[0].colour1,
-            g: 0x00,
+            Colour {
-            b: 0xff
+                r: 0xff,
-        });
+                g: 0x00,
-        assert_eq!(sta[1].colour1, Colour {
+                b: 0xff
-            r: 0x00,
+            }
-            g: 0xff,
+        );
-            b: 0xff
+        assert_eq!(
-        });
+            sta[1].colour1,
-        assert_eq!(sta[2].colour1, Colour {
+            Colour {
-            r: 0xff,
+                r: 0x00,
-            g: 0xff,
+                g: 0xff,
-            b: 0x00
+                b: 0xff
-        });
+            }
-        assert_eq!(sta[3].colour1, Colour {
+        );
-            r: 0x00,
+        assert_eq!(
-            g: 0xff,
+            sta[2].colour1,
-            b: 0x00
+            Colour {
-        });
+                r: 0xff,
+                g: 0xff,
+                b: 0x00
+            }
+        );
+        assert_eq!(
+            sta[3].colour1,
+            Colour {
+                r: 0x00,
+                g: 0xff,
+                b: 0x00
+            }
+        );
     }
 
     #[test]
@@ -371,22 +384,28 @@ mod tests {
 
         assert_eq!(config.brightness, LedBrightness::Med);
         assert_eq!(config.builtins.len(), 5);
-        assert_eq!(config.builtins.first_entry().unwrap().get(), &AuraEffect {
+        assert_eq!(
-            mode: AuraModeNum::Static,
+            config.builtins.first_entry().unwrap().get(),
-            zone: AuraZone::None,
+            &AuraEffect {
-            colour1: Colour { r: 166, g: 0, b: 0 },
+                mode: AuraModeNum::Static,
-            colour2: Colour { r: 0, g: 0, b: 0 },
+                zone: AuraZone::None,
-            speed: Speed::Med,
+                colour1: Colour { r: 166, g: 0, b: 0 },
-            direction: Direction::Right
+                colour2: Colour { r: 0, g: 0, b: 0 },
-        });
+                speed: Speed::Med,
+                direction: Direction::Right
+            }
+        );
         assert_eq!(config.enabled.states.len(), 1);
-        assert_eq!(config.enabled.states[0], AuraPowerState {
+        assert_eq!(
-            zone: PowerZones::KeyboardAndLightbar,
+            config.enabled.states[0],
-            boot: true,
+            AuraPowerState {
-            awake: true,
+                zone: PowerZones::KeyboardAndLightbar,
-            sleep: true,
+                boot: true,
-            shutdown: true
+                awake: true,
-        });
+                sleep: true,
+                shutdown: true
+            }
+        );
     }
 
     #[test]
@@ -396,21 +415,27 @@ mod tests {
 
         assert_eq!(config.brightness, LedBrightness::Med);
         assert_eq!(config.builtins.len(), 12);
-        assert_eq!(config.builtins.first_entry().unwrap().get(), &AuraEffect {
+        assert_eq!(
-            mode: AuraModeNum::Static,
+            config.builtins.first_entry().unwrap().get(),
-            zone: AuraZone::None,
+            &AuraEffect {
-            colour1: Colour { r: 166, g: 0, b: 0 },
+                mode: AuraModeNum::Static,
-            colour2: Colour { r: 0, g: 0, b: 0 },
+                zone: AuraZone::None,
-            speed: Speed::Med,
+                colour1: Colour { r: 166, g: 0, b: 0 },
-            direction: Direction::Right
+                colour2: Colour { r: 0, g: 0, b: 0 },
-        });
+                speed: Speed::Med,
+                direction: Direction::Right
+            }
+        );
         assert_eq!(config.enabled.states.len(), 4);
-        assert_eq!(config.enabled.states[0], AuraPowerState {
+        assert_eq!(
-            zone: PowerZones::Keyboard,
+            config.enabled.states[0],
-            boot: true,
+            AuraPowerState {
-            awake: true,
+                zone: PowerZones::Keyboard,
-            sleep: true,
+                boot: true,
-            shutdown: true
+                awake: true,
-        });
+                sleep: true,
+                shutdown: true
+            }
+        );
     }
 }

distro-packaging/asusctl.spec

@@ -20,7 +20,7 @@
 %global debug_package %{nil}
 %endif
 
-%define version 6.1.13
+%define version 6.1.14
 %define specrelease %{?dist}
 %define pkg_release 9%{specrelease}
 

rog-aura/src/effects/mod.rs

@@ -207,12 +207,14 @@ mod tests {
     fn single_key_next_state_then_create() {
         let layout = KeyLayout::default_layout();
         let mut seq = AdvancedEffects::new(false);
-        seq.effects
+        seq.effects.push(Effect::Static(Static::new(
-            .push(Effect::Static(Static::new(LedCode::F, Colour {
+            LedCode::F,
+            Colour {
                 r: 255,
                 g: 127,
                 b: 0,
-            })));
+            },
+        )));
 
         seq.next_state(&layout);
         let packets = seq.create_packets();

rog-aura/src/keyboard/layouts.rs

@@ -335,93 +335,117 @@ impl KeyLayout {
                 KeyShape::new_led(1.0, 1.0, 0.1, 0.1, 0.1, 0.1),
             )]),
             key_rows: vec![
-                KeyRow::new(0.1, 0.1, vec![
+                KeyRow::new(
-                    (LedCode::Esc, "regular".to_owned()),
+                    0.1,
-                    (LedCode::F1, "regular".to_owned()),
+                    0.1,
-                    (LedCode::F2, "regular".to_owned()),
+                    vec![
-                    (LedCode::F3, "regular".to_owned()),
+                        (LedCode::Esc, "regular".to_owned()),
-                    (LedCode::F4, "regular".to_owned()),
+                        (LedCode::F1, "regular".to_owned()),
-                    // not sure which key to put here
+                        (LedCode::F2, "regular".to_owned()),
-                    (LedCode::F5, "regular".to_owned()),
+                        (LedCode::F3, "regular".to_owned()),
-                    (LedCode::F6, "regular".to_owned()),
+                        (LedCode::F4, "regular".to_owned()),
-                    (LedCode::F7, "regular".to_owned()),
+                        // not sure which key to put here
-                    (LedCode::F8, "regular".to_owned()),
+                        (LedCode::F5, "regular".to_owned()),
-                    (LedCode::F9, "regular".to_owned()),
+                        (LedCode::F6, "regular".to_owned()),
-                    (LedCode::F10, "regular".to_owned()),
+                        (LedCode::F7, "regular".to_owned()),
-                    (LedCode::F11, "regular".to_owned()),
+                        (LedCode::F8, "regular".to_owned()),
-                    (LedCode::F12, "regular".to_owned()),
+                        (LedCode::F9, "regular".to_owned()),
-                ]),
+                        (LedCode::F10, "regular".to_owned()),
-                KeyRow::new(0.1, 0.1, vec![
+                        (LedCode::F11, "regular".to_owned()),
-                    (LedCode::Tilde, "regular".to_owned()),
+                        (LedCode::F12, "regular".to_owned()),
-                    (LedCode::N1, "regular".to_owned()),
+                    ],
-                    (LedCode::N2, "regular".to_owned()),
+                ),
-                    (LedCode::N3, "regular".to_owned()),
+                KeyRow::new(
-                    (LedCode::N4, "regular".to_owned()),
+                    0.1,
-                    (LedCode::N5, "regular".to_owned()),
+                    0.1,
-                    (LedCode::N6, "regular".to_owned()),
+                    vec![
-                    (LedCode::N7, "regular".to_owned()),
+                        (LedCode::Tilde, "regular".to_owned()),
-                    (LedCode::N8, "regular".to_owned()),
+                        (LedCode::N1, "regular".to_owned()),
-                    (LedCode::N9, "regular".to_owned()),
+                        (LedCode::N2, "regular".to_owned()),
-                    (LedCode::N0, "regular".to_owned()),
+                        (LedCode::N3, "regular".to_owned()),
-                    (LedCode::Hyphen, "regular".to_owned()),
+                        (LedCode::N4, "regular".to_owned()),
-                    (LedCode::Equals, "regular".to_owned()),
+                        (LedCode::N5, "regular".to_owned()),
-                    (LedCode::Backspace, "regular".to_owned()),
+                        (LedCode::N6, "regular".to_owned()),
-                ]),
+                        (LedCode::N7, "regular".to_owned()),
-                KeyRow::new(0.1, 0.1, vec![
+                        (LedCode::N8, "regular".to_owned()),
-                    (LedCode::Tab, "regular".to_owned()),
+                        (LedCode::N9, "regular".to_owned()),
-                    (LedCode::Q, "regular".to_owned()),
+                        (LedCode::N0, "regular".to_owned()),
-                    (LedCode::W, "regular".to_owned()),
+                        (LedCode::Hyphen, "regular".to_owned()),
-                    (LedCode::E, "regular".to_owned()),
+                        (LedCode::Equals, "regular".to_owned()),
-                    (LedCode::R, "regular".to_owned()),
+                        (LedCode::Backspace, "regular".to_owned()),
-                    (LedCode::T, "regular".to_owned()),
+                    ],
-                    (LedCode::Y, "regular".to_owned()),
+                ),
-                    (LedCode::U, "regular".to_owned()),
+                KeyRow::new(
-                    (LedCode::I, "regular".to_owned()),
+                    0.1,
-                    (LedCode::O, "regular".to_owned()),
+                    0.1,
-                    (LedCode::P, "regular".to_owned()),
+                    vec![
-                    (LedCode::LBracket, "regular".to_owned()),
+                        (LedCode::Tab, "regular".to_owned()),
-                    (LedCode::RBracket, "regular".to_owned()),
+                        (LedCode::Q, "regular".to_owned()),
-                    (LedCode::BackSlash, "regular".to_owned()),
+                        (LedCode::W, "regular".to_owned()),
-                ]),
+                        (LedCode::E, "regular".to_owned()),
-                KeyRow::new(0.1, 0.1, vec![
+                        (LedCode::R, "regular".to_owned()),
-                    (LedCode::Caps, "regular".to_owned()),
+                        (LedCode::T, "regular".to_owned()),
-                    (LedCode::A, "regular".to_owned()),
+                        (LedCode::Y, "regular".to_owned()),
-                    (LedCode::S, "regular".to_owned()),
+                        (LedCode::U, "regular".to_owned()),
-                    (LedCode::D, "regular".to_owned()),
+                        (LedCode::I, "regular".to_owned()),
-                    (LedCode::F, "regular".to_owned()),
+                        (LedCode::O, "regular".to_owned()),
-                    (LedCode::G, "regular".to_owned()),
+                        (LedCode::P, "regular".to_owned()),
-                    (LedCode::H, "regular".to_owned()),
+                        (LedCode::LBracket, "regular".to_owned()),
-                    (LedCode::J, "regular".to_owned()),
+                        (LedCode::RBracket, "regular".to_owned()),
-                    (LedCode::K, "regular".to_owned()),
+                        (LedCode::BackSlash, "regular".to_owned()),
-                    (LedCode::L, "regular".to_owned()),
+                    ],
-                    (LedCode::SemiColon, "regular".to_owned()),
+                ),
-                    (LedCode::Quote, "regular".to_owned()),
+                KeyRow::new(
-                    (LedCode::Return, "regular".to_owned()),
+                    0.1,
-                ]),
+                    0.1,
-                KeyRow::new(0.1, 0.1, vec![
+                    vec![
-                    (LedCode::LShift, "regular".to_owned()),
+                        (LedCode::Caps, "regular".to_owned()),
-                    (LedCode::Z, "regular".to_owned()),
+                        (LedCode::A, "regular".to_owned()),
-                    (LedCode::X, "regular".to_owned()),
+                        (LedCode::S, "regular".to_owned()),
-                    (LedCode::C, "regular".to_owned()),
+                        (LedCode::D, "regular".to_owned()),
-                    (LedCode::V, "regular".to_owned()),
+                        (LedCode::F, "regular".to_owned()),
-                    (LedCode::B, "regular".to_owned()),
+                        (LedCode::G, "regular".to_owned()),
-                    (LedCode::N, "regular".to_owned()),
+                        (LedCode::H, "regular".to_owned()),
-                    (LedCode::M, "regular".to_owned()),
+                        (LedCode::J, "regular".to_owned()),
-                    (LedCode::Comma, "regular".to_owned()),
+                        (LedCode::K, "regular".to_owned()),
-                    (LedCode::Period, "regular".to_owned()),
+                        (LedCode::L, "regular".to_owned()),
-                    (LedCode::FwdSlash, "regular".to_owned()),
+                        (LedCode::SemiColon, "regular".to_owned()),
-                    (LedCode::Rshift, "regular".to_owned()),
+                        (LedCode::Quote, "regular".to_owned()),
-                ]),
+                        (LedCode::Return, "regular".to_owned()),
-                KeyRow::new(0.1, 0.1, vec![
+                    ],
-                    (LedCode::LCtrl, "regular".to_owned()),
+                ),
-                    (LedCode::LFn, "regular".to_owned()),
+                KeyRow::new(
-                    (LedCode::Meta, "regular".to_owned()),
+                    0.1,
-                    (LedCode::LAlt, "regular".to_owned()),
+                    0.1,
-                    (LedCode::Spacebar, "regular".to_owned()),
+                    vec![
-                    (LedCode::RAlt, "regular".to_owned()),
+                        (LedCode::LShift, "regular".to_owned()),
-                    (LedCode::PrtSc, "regular".to_owned()),
+                        (LedCode::Z, "regular".to_owned()),
-                    (LedCode::RCtrl, "regular".to_owned()),
+                        (LedCode::X, "regular".to_owned()),
-                ]),
+                        (LedCode::C, "regular".to_owned()),
+                        (LedCode::V, "regular".to_owned()),
+                        (LedCode::B, "regular".to_owned()),
+                        (LedCode::N, "regular".to_owned()),
+                        (LedCode::M, "regular".to_owned()),
+                        (LedCode::Comma, "regular".to_owned()),
+                        (LedCode::Period, "regular".to_owned()),
+                        (LedCode::FwdSlash, "regular".to_owned()),
+                        (LedCode::Rshift, "regular".to_owned()),
+                    ],
+                ),
+                KeyRow::new(
+                    0.1,
+                    0.1,
+                    vec![
+                        (LedCode::LCtrl, "regular".to_owned()),
+                        (LedCode::LFn, "regular".to_owned()),
+                        (LedCode::Meta, "regular".to_owned()),
+                        (LedCode::LAlt, "regular".to_owned()),
+                        (LedCode::Spacebar, "regular".to_owned()),
+                        (LedCode::RAlt, "regular".to_owned()),
+                        (LedCode::PrtSc, "regular".to_owned()),
+                        (LedCode::RCtrl, "regular".to_owned()),
+                    ],
+                ),
             ],
         }
     }

rog-platform/src/cpu.rs

@@ -267,18 +267,24 @@ mod tests {
     fn check_cpu() {
         let cpu = CPUControl::new().unwrap();
         assert_eq!(cpu.get_governor().unwrap(), CPUGovernor::Powersave);
-        assert_eq!(cpu.get_available_governors().unwrap(), vec![
+        assert_eq!(
-            CPUGovernor::Performance,
+            cpu.get_available_governors().unwrap(),
-            CPUGovernor::Powersave
+            vec![
-        ]);
+                CPUGovernor::Performance,
+                CPUGovernor::Powersave
+            ]
+        );
 
         assert_eq!(cpu.get_epp().unwrap(), CPUEPP::BalancePower);
-        assert_eq!(cpu.get_available_epp().unwrap(), vec![
+        assert_eq!(
-            CPUEPP::Default,
+            cpu.get_available_epp().unwrap(),
-            CPUEPP::Performance,
+            vec![
-            CPUEPP::BalancePerformance,
+                CPUEPP::Default,
-            CPUEPP::BalancePower,
+                CPUEPP::Performance,
-            CPUEPP::Power,
+                CPUEPP::BalancePerformance,
-        ]);
+                CPUEPP::BalancePower,
+                CPUEPP::Power,
+            ]
+        );
     }
 }