.direnv/flake-profile

@@ -1 +1 @@
-flake-profile-4-link
+flake-profile-11-link

.direnv/flake-profile-11-link

@@ -0,0 +1 @@
+/nix/store/51yb1yhqssknlmxvvscvj0cqbvs0z915-nix-shell-env

.direnv/flake-profile-4-link

@@ -1 +0,0 @@
-/nix/store/30prmd5nyydss0bcs7d578grjav6i7x3-nix-shell-env

.github/workflows/rust.yml

@@ -1,33 +1,25 @@
-# inspired by https://github.com/danth/stylix/blob/master/.github/workflows/docs.yml
-name: Rust-build
+name: Rust
+
 on:
   push:
-    branches: ["main"]
+    branches: [ "main" ]
   pull_request:
-    branches: ["main"]
+    branches: [ "main" ]
+
+env:
+  CARGO_TERM_COLOR: always
+
 jobs:
   build:
-    name: Build
-    permissions:
-      contents: read
-    runs-on: ubuntu-latest
+    runs-on: [self-hosted, ubuntu]
     steps:
-      - name: Install Nix
-        uses: DeterminateSystems/nix-installer-action@main
+      - uses: actions/checkout@v3
+      - name: nightly-rust
+        uses: actions-rs/toolchain@v1
         with:
-          github-token: ${{ secrets.GITHUB_TOKEN }}
-          extra-conf: |
-            extra-experimental-features = nix-command flakes
-      - name: Set up cache
-        uses: DeterminateSystems/magic-nix-cache-action@main
-      - name: action
-        uses: cachix/install-nix-action@v25
-        with:
-          nix_path: nixpkgs=channel:nixos-unstable
-      - name: cache
-        uses: cachix/cachix-action@v14
-        with:
-          name: reset
-          authToken: '${{ secrets.CACHIX_AUTH_TOKEN }}'
+          profile: minimal
+          toolchain: nightly
       - name: Build
-        run: nix -L build github:${{ github.repository }}/${{ github.sha }} --no-write-lock-file
+        run: cargo build --verbose
+      - name: Run clippy
+        run: cargo clippy --fix

Cargo.lock

@@ -926,9 +926,9 @@ dependencies = [
 
 [[package]]
 name = "re_set-lib"
-version = "5.2.5"
+version = "5.2.4"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "69ec9fc4d9e919feb030e5382de1aa49447404a57ecc19361bf7710f1cf9063e"
+checksum = "f1f1aa52c35d115b4efc5de2d41d0288f20705dd7f3ca6a49a68f2bab9c31a8e"
 dependencies = [
  "dbus",
  "dbus-crossroads",
@@ -952,7 +952,7 @@ dependencies = [
 
 [[package]]
 name = "reset"
-version = "2.0.0"
+version = "1.2.7"
 dependencies = [
  "dbus",
  "fork",
@@ -968,9 +968,9 @@ dependencies = [
 
 [[package]]
 name = "reset_daemon"
-version = "2.2.0"
+version = "2.1.3"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "f72dd51da41c2fc3ac37ae55fbdeb2670d32cad39cc85cc95a9ea25fc02b6086"
+checksum = "aa3666f8d12729c489f0d5d60fa8f03299da80859d72507c9e4ae6d56e61fe79"
 dependencies = [
  "crossbeam",
  "dbus",

Cargo.toml

@@ -1,6 +1,6 @@
 [package]
 name = "reset"
-version = "2.0.0"
+version = "1.2.7"
 edition = "2021"
 description = "A wip universal Linux settings application."
 repository = "https://github.com/Xetibo/ReSet"
@@ -11,8 +11,8 @@ name = "ReSet"
 path = "src/main.rs"
 
 [dependencies]
-reset_daemon = "2.2.0"
-re_set-lib = "5.2.5"
+reset_daemon = "2.1.3"
+re_set-lib = "5.2.4"
 adw = { version = "0.6.0", package = "libadwaita", features = ["v1_4"] }
 dbus = "0.9.7"
 gtk = { version = "0.8.1", package = "gtk4", features = ["v4_12"] }

PKGBUILD

@@ -1,7 +1,7 @@
 # Maintainer: Fabio Lenherr <dashie@dashie.org>
 
 pkgname=ReSet
-pkgver=2.0.0
+pkgver=1.2.7
 pkgrel=0
 arch=('x86_64')
 pkgdir="/usr/bin/${pkgname}"

README.md

@@ -34,22 +34,7 @@ A window manager/compositor agnostic settings application for Linux written in r
 ## Plugins
 
 ReSet features a plugin system by loading dynamic libraries for both the daemon and the ReSet graphical user interface.
-A list of official plugins, installation guides and their documentation can be found at [ReSet-Plugins](https://github.com/Xetibo/ReSet-Plugins).
-
-### Installation
-
-Plugins are loaded either from `/usr/lib/reset` or `~/.config/reset/plugins`. In order to install the plugin, either install a distribution specific package that places the library into the specified system folder, or place the library in the plugins folder in your config directory.
-
-Note, after installation, please move to confirmation.
-
-### Confirmation
-
-In order for your plugins to load, you have to define them in `.config/reset/ReSet.toml`.
-This is done to avoid loading of arbitrary plugins that might be placed within this folder by accident.
-
-```toml
-plugins = ["libreset_monitors.so", "libreset_keyboard_plugin.so"]
-```
+A list of official plugins and their documentation can be found at [ReSet-Plugins](https://github.com/Xetibo/ReSet-Plugins).
 
 ## Packaging
 
@@ -61,7 +46,7 @@ We are currently not published on flatpak due to issues with permissions.
 This is being worked on...
 
 Installation:
-Download the flatpak package (reset.flatpak) from the release and install with the terminal.
+Download the flatpak package from the release and install with the terminal.
 
 ```
 flatpak install --user reset.flatpak
@@ -74,7 +59,7 @@ flatpak install --user reset.flatpak
 <!-- ```paru -S ReSet``` -->
 
 Manually:
-Download the package (ReSet-version-x86_64.pkg.tar.zst) from the releases tab and install it with pacman.
+Download the package from the releases tab and install it with pacman.
 
 ```
 sudo pacman -U /path/to/reset
@@ -82,7 +67,7 @@ sudo pacman -U /path/to/reset
 
 ### Debian Package(Ubuntu 23.04 dependencies)
 
-Download the package (ReSet.deb) from the releases tab and install it with apt.
+Download the package from the releases tab and install it with apt.
 
 ```
 sudo apt install ./path/to/reset

debian/DEBIAN/control

@@ -1,9 +1,9 @@
 Package: ReSet
-Version: 2.0.0
+Version: 1.2.7
 Maintainer: DashieTM
 Architecture: all
 Description: A wip universal Linux settings application.
 Homepage: https://github.com/Xetibo/ReSet
 Build-Depends: rust
-Depends: libadwaita-1-0, libgtk-4-1, dbus
-Recommends: pipewire-pulse, network-manager, bluez
+Depends: libadwaita, gtk, dbus
+Recommends: pipewire-pulse, networkmanager, bluez

flake.lock

@@ -7,11 +7,11 @@
         ]
       },
       "locked": {
-        "lastModified": 1738453229,
-        "narHash": "sha256-7H9XgNiGLKN1G1CgRh0vUL4AheZSYzPm+zmZ7vxbJdo=",
+        "lastModified": 1715865404,
+        "narHash": "sha256-/GJvTdTpuDjNn84j82cU6bXztE0MSkdnTWClUCRub78=",
         "owner": "hercules-ci",
         "repo": "flake-parts",
-        "rev": "32ea77a06711b758da0ad9bd6a844c5740a87abd",
+        "rev": "8dc45382d5206bd292f9c2768b8058a8fd8311d9",
         "type": "github"
       },
       "original": {
@@ -20,13 +20,31 @@
         "type": "github"
       }
     },
+    "flake-utils": {
+      "inputs": {
+        "systems": "systems"
+      },
+      "locked": {
+        "lastModified": 1705309234,
+        "narHash": "sha256-uNRRNRKmJyCRC/8y1RqBkqWBLM034y4qN7EprSdmgyA=",
+        "owner": "numtide",
+        "repo": "flake-utils",
+        "rev": "1ef2e671c3b0c19053962c07dbda38332dcebf26",
+        "type": "github"
+      },
+      "original": {
+        "owner": "numtide",
+        "repo": "flake-utils",
+        "type": "github"
+      }
+    },
     "nixpkgs": {
       "locked": {
-        "lastModified": 1739446958,
-        "narHash": "sha256-+/bYK3DbPxMIvSL4zArkMX0LQvS7rzBKXnDXLfKyRVc=",
+        "lastModified": 1716137900,
+        "narHash": "sha256-sowPU+tLQv8GlqtVtsXioTKeaQvlMz/pefcdwg8MvfM=",
         "owner": "NixOS",
         "repo": "nixpkgs",
-        "rev": "2ff53fe64443980e139eaa286017f53f88336dd0",
+        "rev": "6c0b7a92c30122196a761b440ac0d46d3d9954f1",
         "type": "github"
       },
       "original": {
@@ -38,11 +56,11 @@
     },
     "nixpkgs_2": {
       "locked": {
-        "lastModified": 1736320768,
-        "narHash": "sha256-nIYdTAiKIGnFNugbomgBJR+Xv5F1ZQU+HfaBqJKroC0=",
+        "lastModified": 1706487304,
+        "narHash": "sha256-LE8lVX28MV2jWJsidW13D2qrHU/RUUONendL2Q/WlJg=",
         "owner": "NixOS",
         "repo": "nixpkgs",
-        "rev": "4bc9c909d9ac828a039f288cf872d16d38185db8",
+        "rev": "90f456026d284c22b3e3497be980b2e47d0b28ac",
         "type": "github"
       },
       "original": {
@@ -61,14 +79,15 @@
     },
     "rust-overlay": {
       "inputs": {
+        "flake-utils": "flake-utils",
         "nixpkgs": "nixpkgs_2"
       },
       "locked": {
-        "lastModified": 1739586408,
-        "narHash": "sha256-UN9hRKRE1eLU8C0cioTZubaCZQTA8NDc8/4vCpS5pS0=",
+        "lastModified": 1716171463,
+        "narHash": "sha256-lc7wOh5BjYUoxdhcPkeUY8BmuL2qtRaHlW1403RW48E=",
         "owner": "oxalica",
         "repo": "rust-overlay",
-        "rev": "3dbc0ce1c0690b83cfb9a9a51fbe90c3bc8f9916",
+        "rev": "04d61d14803854fd8453ec43c5c53a471e5407a8",
         "type": "github"
       },
       "original": {
@@ -76,6 +95,21 @@
         "repo": "rust-overlay",
         "type": "github"
       }
+    },
+    "systems": {
+      "locked": {
+        "lastModified": 1681028828,
+        "narHash": "sha256-Vy1rq5AaRuLzOxct8nz4T6wlgyUR7zLU309k9mBC768=",
+        "owner": "nix-systems",
+        "repo": "default",
+        "rev": "da67096a3b9bf56a91d16901293e51ba5b49a27e",
+        "type": "github"
+      },
+      "original": {
+        "owner": "nix-systems",
+        "repo": "default",
+        "type": "github"
+      }
     }
   },
   "root": "root",

nix/default.nix

@@ -1,69 +1,69 @@
-{
-  rustPlatform,
-  rust-bin,
-  pulseaudio,
-  dbus,
-  gdk-pixbuf,
-  adwaita-icon-theme,
-  pkg-config,
-  wrapGAppsHook4,
-  gtk4,
-  libadwaita,
-  python312Packages,
-  flatpak,
-  flatpak-builder,
-  lib,
-  lockFile,
-  ...
-}: let
+{ rustPlatform
+, rust-bin
+, pulseaudio
+, dbus
+, gdk-pixbuf
+, gnome
+, pkg-config
+, wrapGAppsHook4
+, gtk4
+, libadwaita
+, python312Packages
+, flatpak
+, flatpak-builder
+, lib
+, lockFile
+, ...
+}:
+let
   cargoToml = builtins.fromTOML (builtins.readFile ../Cargo.toml);
 in
-  rustPlatform.buildRustPackage rec {
-    pname = "ReSet";
-    version = cargoToml.package.version;
+rustPlatform.buildRustPackage rec {
+  pname = "ReSet";
+  version = cargoToml.package.version;
 
-    src = ../.;
+  src = ../.;
 
-    buildInputs = [
-      gtk4
-      libadwaita
-      pulseaudio
-      dbus
-      gdk-pixbuf
-      adwaita-icon-theme
-      python312Packages.aiohttp
-      python312Packages.toml
-      flatpak
-      flatpak-builder
-    ];
+  buildInputs = [
+    gtk4
+    libadwaita
+    pulseaudio
+    dbus
+    gdk-pixbuf
+    gnome.adwaita-icon-theme
+    python312Packages.aiohttp
+    python312Packages.toml
+    flatpak
+    flatpak-builder
+  ];
 
-    cargoLock = {
-      inherit lockFile;
-    };
+  cargoLock = {
+    inherit lockFile;
+  };
 
-    nativeBuildInputs = [
-      pkg-config
-      wrapGAppsHook4
-      rust-bin.nightly."2024-05-10".default
-    ];
+  nativeBuildInputs = [
+    pkg-config
+    wrapGAppsHook4
+    rust-bin.nightly."2024-05-10".default
+  ];
 
-    copyLibs = true;
+  copyLibs = true;
 
-    postInstall = ''
-      install -D --mode=444 $src/${pname}.desktop $out/share/applications/${pname}.desktop
-      install -D --mode=444 $src/src/resources/icons/${pname}.svg $out/share/pixmaps/${pname}.svg
-    '';
+  postInstall = ''
+    	install -D --mode=444 $src/${pname}.desktop $out/share/applications/${pname}.desktop
+    	install -D --mode=444 $src/src/resources/icons/${pname}.svg $out/share/pixmaps/${pname}.svg
+  '';
 
-    # test is broken in nix for some reason
-    doInstallCheck = false;
-    doCheck = false;
+  # test is broken in nix for some reason
+  doInstallCheck = false;
+  doCheck = false;
 
-    meta = with lib; {
-      description = "A wip universal Linux settings application.";
-      homepage = "https://github.com/Xetibo/ReSet";
-      changelog = "https://github.com/Xetibo/ReSet/releases/tag/${version}";
-      license = licenses.gpl3;
-      maintainers = with maintainers; [DashieTM];
-      mainProgram = "ReSet";
-    };
-  }
+  meta = with lib; {
+    description = "A wip universal Linux settings application.";
+    homepage = "https://github.com/Xetibo/ReSet";
+    changelog = "https://github.com/Xetibo/ReSet/releases/tag/${version}";
+    license = licenses.gpl3;
+    maintainers = with maintainers; [ DashieTM ];
+    mainProgram = "ReSet";
+  };
+}

src/components/audio/audio_box_utils.rs

@@ -334,7 +334,7 @@ pub fn start_audio_box_listener<
     get_default_name_function: &'static DBusFunction,
     dummy_name: &'static str,
 ) -> Connection {
-    // FUTURE TODO: make the failed logs generically sound -> deynamic output for both
+    // TODO: make the failed logs generically sound -> deynamic output for both
     let object_added =
         ObjectAdded::match_rule(Some(&BASE.into()), Some(&Path::from(DBUS_PATH))).static_clone();
     let object_changed =

src/components/audio/audio_entry.rs

@@ -117,7 +117,7 @@ pub fn new_entry<
     reset_box: Arc<AudioBox>,
 ) -> Arc<AudioEntry> {
     let obj: Arc<AudioEntry> = Arc::new(Object::builder().build());
-    // FUTURE TODO: use event callback for progress bar -> this is the "im speaking" indicator
+    // TODO use event callback for progress bar -> this is the "im speaking" indicator
     {
         let imp = obj.entry_imp();
         let slider_obj_ref = obj.clone();

src/components/audio/audio_functions.rs

@@ -91,7 +91,7 @@ pub fn new_stream_entry<
     stream: StreamObject,
 ) -> Arc<AudioStream> {
     let obj: Arc<AudioStream> = Arc::new(Object::builder().build());
-    // FUTURE TODO: use event callback for progress bar -> this is the "im speaking" indicator
+    // TODO use event callback for progress bar -> this is the "im speaking" indicator
     let output_box_mute_ref = audio_box.clone();
     let output_box_volume_ref = audio_box.clone();
     let output_box_sink_ref = audio_box.clone();

src/components/base/error_impl.rs

@@ -54,7 +54,7 @@ pub fn show_error<T: ReSetErrorImpl + Send + Sync + 'static>(
     parent: Arc<T>,
     message: &'static str,
 ) {
-    // FUTURE TODO: Add error to log
+    // TODO: Add error to log
     glib::spawn_future(async move {
         glib::idle_add_once(move || {
             let error = parent.error();

src/components/bluetooth/bluetooth_box.rs

@@ -57,7 +57,7 @@ impl BluetoothBox {
     }
 }
 
-// FUTURE TODO:
+// TODO
 // handle bonded -> this means saved but not connected
 // handle rssi below x -> don't show device
 
@@ -185,6 +185,7 @@ pub fn populate_connected_bluetooth_devices(
     listeners: Arc<Listeners>,
     bluetooth_box: Arc<BluetoothBox>,
 ) {
+    // TODO handle saved devices -> they also exist
     gio::spawn_blocking(move || {
         let ref_box = bluetooth_box.clone();
         let adapters = get_bluetooth_adapters(ref_box.clone());

src/components/bluetooth/bluetooth_entry.rs

@@ -70,7 +70,7 @@ impl BluetoothEntry {
             });
         let gesture = GestureClick::new();
         // paired is not what we think
-        // FUTURE TODO: implement paired
+        // TODO implement paired
         gesture.connect_released(move |_, _, _, _| {
             let imp = entry_ref.imp();
             let borrow = imp.bluetooth_device.borrow();

src/components/bluetooth/bluetooth_event_handlers.rs

@@ -69,7 +69,7 @@ pub fn device_removed_handler(
                 if list_entry.imp().bluetooth_device.borrow().connected {
                     imp.reset_bluetooth_connected_devices.remove(&*list_entry);
                 } else {
-                    // FUTURE TODO: is there a better way for this?
+                    // TODO: is there a better way for this?
                     imp.reset_bluetooth_available_devices.remove(&*list_entry);
                     imp.reset_bluetooth_saved_devices.remove(&*list_entry);
                 }

src/components/utils.rs

@@ -10,9 +10,6 @@ use glib::prelude::Cast;
 use glib::Object;
 use gtk::prelude::{GObjectPropertyExpressionExt, ListBoxRowExt, ListItemExt, WidgetExt};
 use gtk::{Align, SignalListItemFactory, StringObject};
-use re_set_lib::ERROR;
-#[cfg(debug_assertions)]
-use re_set_lib::{utils::macros::ErrorLevel, write_log_to_file};
 
 pub const DBUS_PATH: &str = "/org/Xetibo/ReSet/Daemon";
 pub const WIRELESS: &str = "org.Xetibo.ReSet.Network";
@@ -96,12 +93,5 @@ pub fn get_capabilities() -> Vec<String> {
     let conn = Connection::new_session().unwrap();
     let proxy = conn.with_proxy(BASE, DBUS_PATH, Duration::from_millis(10000));
     let res: Result<(Vec<String>,), Error> = proxy.method_call(BASE, "GetCapabilities", ());
-    if res.is_err() {
-        ERROR!(
-            "Could not call capabilities from daemon",
-            ErrorLevel::Critical
-        );
-        return Vec::new();
-    }
     res.unwrap().0
 }

src/components/wifi/saved_wifi_entry.rs

@@ -55,7 +55,7 @@ impl SavedWifiEntry {
         let entry_ref = entry.clone();
         delete_button.connect_clicked(clone!(@weak wifi_box => move |_| {
             delete_connection(entry_ref.imp().reset_connection_path.take());
-            // FUTURE TODO: handle error
+            // TODO handle error
             wifi_box.reset_stored_wifi_list.remove(&*entry_ref);
 
         }));

src/components/wifi/wifi_box.rs

@@ -209,7 +209,7 @@ pub fn show_stored_connections(wifi_box: Arc<WifiBox>) {
             glib::idle_add_once(move || {
                 let self_imp = wifibox_ref.imp();
                 for connection in connections {
-                    // FUTURE TODO: include button for settings
+                    // TODO include button for settings
                     let name =
                         &String::from_utf8(connection.1).unwrap_or_else(|_| String::from(""));
                     let entry = SavedWifiEntry::new(name, connection.0, self_imp);

src/components/wifi/wifi_entry.rs

@@ -48,7 +48,7 @@ impl WifiEntry {
                 .build(),
         );
 
-        // FUTURE TODO: handle encryption
+        // TODO handle encryption thing
         let wifi_strength = Image::builder()
             .icon_name(match strength {
                 WifiStrength::Excellent => "network-wireless-signal-excellent-symbolic",
@@ -177,7 +177,7 @@ pub fn click_stored_network(entry: Arc<WifiEntry>) {
             });
         });
     });
-    // FUTURE TODO: crate spinner animation and block UI
+    // TODO crate spinner animation and block UI
 }
 
 pub fn click_new_network(entry: Arc<WifiEntry>) {
@@ -228,7 +228,7 @@ pub fn click_new_network(entry: Arc<WifiEntry>) {
                     });
                 });
             });
-            // FUTURE TODO: crate spinner animation and block UI
+            // TODO crate spinner animation and block UI
         };
 
     let entry_imp = entry.imp();

src/components/wifi/wifi_event_handlers.rs

@@ -72,7 +72,7 @@ pub fn access_point_changed_handler(wifi_box: Arc<WifiBox>, ir: AccessPointChang
             let name = name_opt.as_str();
             entry_imp.wifi_strength.set(strength);
             entry.set_title(name);
-            // FUTURE TODO: handle encryption thing
+            // TODO handle encryption thing
             entry_imp
                 .reset_wifi_strength
                 .borrow()

src/components/window/reset_window.rs

@@ -207,6 +207,7 @@ impl ReSetWindow {
             .connect_row_activated(clone!(@ weak self_imp => move |_, _| {
                 self_imp.reset_search_entry.set_text("");
             }));
+        // TODO: refactor this
         let mut i = 0;
         for info in sidebar_list {
             if info.parent.is_none() && i != 0 {

src/components/window/sidebar_entry_impl.rs

@@ -11,7 +11,7 @@ use crate::components::window::sidebar_entry;
 
 #[derive(Default)]
 pub enum Categories {
-    // FUTURE TODO: are these ever used ?
+    // TODO: are these ever used ?
     // Connectivity,
     // Audio,
     // Peripherals,

src/main.rs

@@ -1,5 +1,5 @@
 use std::hint::{self};
-use std::sync::atomic::{AtomicBool, Ordering};
+use std::sync::atomic::AtomicBool;
 use std::sync::Arc;
 use std::thread;
 use std::time::Duration;
@@ -81,7 +81,5 @@ async fn daemon_check(ready: Arc<AtomicBool>) {
     let res = handle.join();
     if res.unwrap().is_err() {
         run_daemon(Some(ready)).await;
-    } else {
-        ready.store(true, Ordering::SeqCst);
     }
 }

test_plugin/src/lib.rs

@@ -8,7 +8,7 @@ use gtk::prelude::{BoxExt, ButtonExt};
 use re_set_lib::utils::plugin::{PluginCapabilities, PluginImplementation, PluginTestFunc, SidebarInfo};
 
 pub const BASE: &str = "org.Xetibo.ReSet.Daemon";
-pub const DBUS_PATH: &str = "/org/Xetibo/ReSet/Plugins/test";
+pub const DBUS_PATH: &str = "/org/Xebito/ReSet/Plugins/test";
 pub const INTERFACE: &str = "org.Xetibo.ReSet.TestPlugin";
 
 #[no_mangle]