cleanup

flake.lock

@@ -20,11 +20,11 @@
     },
     "nixpkgs": {
       "locked": {
-        "lastModified": 1743964447,
+        "lastModified": 1748190013,
-        "narHash": "sha256-nEo1t3Q0F+0jQ36HJfbJtiRU4OI+/0jX/iITURKe3EE=",
+        "narHash": "sha256-R5HJFflOfsP5FBtk+zE8FpL8uqE7n62jqOsADvVshhE=",
         "owner": "NixOS",
         "repo": "nixpkgs",
-        "rev": "063dece00c5a77e4a0ea24e5e5a5bd75232806f8",
+        "rev": "62b852f6c6742134ade1abdd2a21685fd617a291",
         "type": "github"
       },
       "original": {

flake.nix

@@ -15,12 +15,7 @@
     flake-parts.lib.mkFlake {inherit inputs;} {
       systems = ["x86_64-linux" "aarch64-linux"];
 
-      flake.nixosModules = let
+      flake.nixosModules.default = import ./module.nix self;
-        musicomp = import ./nixos/musicomp self;
-      in {
-        inherit musicomp;
-        default = musicomp;
-      };
 
       perSystem = {
         pkgs,
@@ -35,14 +30,7 @@
           ];
         };
 
-        packages = let
+        packages.default = pkgs.callPackage ./package.nix {};
-          packages = lib.packagesFromDirectoryRecursive {
-            inherit (pkgs) callPackage;
-            directory = ./packages;
-          };
-        in packages // {
-          default = packages.musicomp;
-        };
       };
     };
 }

module.nix

@@ -5,6 +5,7 @@ self: {
   config,
   ...
 }: let
+  cfg = config.services.musicomp;
   inherit (lib) types;
   inherit (utils.systemdUtils.unitOptions) unitOption;
 in {
@@ -73,43 +74,48 @@ in {
   };
 
   config = {
-    systemd.services =
+    systemd = lib.mkMerge (
-      lib.mapAttrs'
+      lib.mapAttrsToList (
-      (
+        jobName: job:
-        name: job:
+        let
-          lib.nameValuePair "musicomp-jobs-${name}" {
+          systemdName = "musicomp-job-${jobName}";
-            wantedBy = ["multi-user.target"];
+          description = "musicomp job ${jobName}";
-            restartIfChanged = false;
+        in
+        {
+          timers.${systemdName} = {
+            wantedBy = [ "timers.target" ];
+            inherit description;
+            inherit (job) timerConfig;
+          };
+
+          services.${systemdName} = {
+            inherit description;
+
+            serviceConfig = {
+              Type = "oneshot";
+              User = "root";
+              Group = "root";
+            };
 
             script = ''
                 ${lib.optionalString job.inhibitsSleep ''
-                ${lib.getExe' pkgs.systemd "systemd-inhibit"} \
+                  ${lib.getExe' config.systemd.package "systemd-inhibit"} \
                     --mode block \
-                  --who musicomp \
+                    --who ${lib.escapeShellArg systemdName} \
                     --what sleep \
-                  --why ${lib.escapeShellArg "Scheduled musicomp ${name}"}
+                    --why ${lib.escapeShellArg "Scheduled musicomp job ${jobName}"}
                 ''}
 
                 ${lib.getExe job.package} \
-                ${lib.optionalString (job.workers > 0) "--workers ${job.workers}"} \
+                  ${lib.optionalString (job.workers > 0) "--workers ${lib.escapeShellArg job.workers}"} \
                   --verbose \
-                -- ${job.music} ${job.comp}
+                  -- ${lib.escapeShellArg job.music} ${lib.escapeShellArg job.comp}
               '';
 
             postStart = job.post;
-
+          };
-            serviceConfig.Type = "oneshot";
+        }
-          }
+      ) cfg.jobs
-      )
+    );
-      config.services.musicomp.jobs;
-
-    systemd.timers =
-      lib.mapAttrs'
-      (name: job:
-        lib.nameValuePair "musicomp-jobs-${name}" {
-          wantedBy = ["timers.target"];
-          inherit (job) timerConfig;
-        })
-      (lib.filterAttrs (_: job: job.timerConfig != null) config.services.musicomp.jobs);
   };
 }

package.nix

@@ -7,7 +7,7 @@ python3Packages.buildPythonApplication {
   pname = "musicomp";
   version = "0.1.0";
 
-  src = ../../.;
+  src = ./.;
 
   pyproject = true;
 

src/musicomp/cli.py

@@ -90,6 +90,9 @@ def main():
     src_dir = args.music[0]  # pyright: ignore[reportUnknownMemberType]
     dst_dir = args.comp[0]  # pyright: ignore[reportUnknownMemberType]
 
+    if args.verbose >= 1 or args.interactive:
+        print("Planning...", file=sys.stderr)
+
     plan = list(
         Todo.plan(
             src_dir,