From 05a98b6be06d68c4a6086a0adb0a9b058833b41a Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Dani=C3=ABl=20de=20Kok?= <me@danieldk.eu>
Date: Sat, 13 Oct 2018 19:19:09 +0200
Subject: [PATCH] mbsync: change service unit type to 'oneshot'

The ExecStartPost command is currently started when the mbsync is
invoked succesfully. However, we typically want to run something like
'mu index' or 'notmuch new' after mbsync completes.  This changes the
unit type to oneshot, so that the ExecStartPost command is run after
mbsync finishes succesfully.
---
 modules/services/mbsync.nix | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/modules/services/mbsync.nix b/modules/services/mbsync.nix
index a14e25237..5c994cc6b 100644
--- a/modules/services/mbsync.nix
+++ b/modules/services/mbsync.nix
@@ -87,7 +87,7 @@ in
       };
 
       Service = {
-        Type = "simple";
+        Type = "oneshot";
         ExecStart = "${cfg.package}/bin/mbsync ${concatStringsSep " " mbsyncOptions}";
       } // (optionalAttrs (cfg.postExec != null) { ExecStartPost = cfg.postExec; })
         // (optionalAttrs (cfg.preExec  != null) { ExecStartPre  = cfg.preExec;  });