systemd mount defaults

bundles/systemd-mount/metadata.py

@@ -15,16 +15,14 @@ defaults = {
 def units(metadata):
     units = {}
     services = {}
-    
+
     for mountpoint, conf in metadata.get('systemd-mount').items():
         formatted_name = mountpoint[1:].replace('-', '\\x2d').replace('/', '-') + '.mount'
-        
+
         units[formatted_name] = {
             'Unit': {
                 'Description': f"Mount {conf['source']} -> {mountpoint}",
                 'DefaultDependencies': 'no',
-                'Conflicts': 'umount.target',
-                'Before': 'umount.target',
             },
             'Mount': {
                 'What': conf['source'],
@@ -32,16 +30,11 @@ def units(metadata):
                 'Type': 'fuse.bindfs',
                 'Options': f"nonempty",
             },
-            'Install': {
-                'WantedBy': {
-                    'local-fs.target',
-                },
-            },
         }
-        
+
         if conf.get('user'):
             units[formatted_name]['Mount']['Options'] += f",force-user={conf.get('user')}"
-        
+
         services[formatted_name] = {}
 
     return {

bundles/systemd/README.md

@@ -3,7 +3,7 @@
 ## show unit paths
 
 ```
-systemctl --no-pager --property=UnitPath show | tr ' ' '\n'
+systemctl --no-pager --property=UnitPath show --value | tr ' ' '\n'
 ```
 
 ## metadata

bundles/systemd/metadata.py

@@ -25,19 +25,14 @@ def units(metadata):
 
         type = name.split('.')[-1]
 
-        if not config.get('Install/WantedBy'):
+        if type == 'service':
-            if type == 'service':
+            units.setdefault(name, {}).setdefault('Install', {}).setdefault('WantedBy', {'multi-user.target'})
-                units[name] = {
+        elif type == 'timer':
-                    'Install': {
+            units.setdefault(name, {}).setdefault('Install', {}).setdefault('WantedBy', {'timers.target'})
-                        'WantedBy': {'multi-user.target'},
+        elif type == 'mount':
-                    }
+            units.setdefault(name, {}).setdefault('Install', {}).setdefault('WantedBy', {'local-fs.target'})
-                }
+            units.setdefault(name, {}).setdefault('Unit', {}).setdefault('Conflicts', {'umount.target'})
-            elif type == 'timer':
+            units.setdefault(name, {}).setdefault('Unit', {}).setdefault('Before', {'umount.target'})
-                units[name] = {
-                    'Install': {
-                        'WantedBy': {'timers.target'},
-                    }
-                }
 
     return {
         'systemd': {