remove .service/.target/.bundle etc suffixes on service names

they don't seem to be providing value

THOUGHTS.txt

@@ -203,9 +203,11 @@ tools
 15) "ip address add" seems to magically recognise v4 vs v6 but
 is that specified or fluke?
 
-18) maybe stop suffixing all the service names with .service
+DONE 18) maybe stop suffixing all the service names with .service
+
+19) syslogd - use busybox or s6? the only thing (so far) using it is pppd.
+Maybe we could LD_PRELOAD it to use stderr instead
 
-19) syslogd - use busybox or s6?
 
 DONE 20) The option currently called defaultPackages needs a better name as
 it doesn't have the same semantics as nixos

pkgs/liminix-tools/networking/pppoe.nix

@@ -17,7 +17,7 @@ interface: {
 let
   name = "${interface.device}.pppoe";
   ip-up = writeAshScript "ip-up" {} ''
-    outputs=/run/service-state/${name}.service/
+    outputs=/run/service-state/${name}/
     mkdir -p $outputs
     (cd $outputs
     echo $1 > ifname

pkgs/liminix-tools/networking/udhcpc.nix

@@ -14,7 +14,7 @@ action=$1
 
 set_address() {
     ip address replace $ip/$mask dev $interface
-    dir=/run/service-state/${name}.service/
+    dir=/run/service-state/${name}/
     mkdir -p $dir
     for i in lease mask ip router siaddr dns serverid subnet opt53 interface ; do
         echo ''${!i} > $dir/$i

pkgs/liminix-tools/services/default.nix

@@ -16,7 +16,7 @@ let
     , notification-fd ? null
     , dependencies ? []
   } @ args: stdenvNoCC.mkDerivation {
-    name = "${name}.service";
+    inherit name;
     serviceType = "longrun";
     buildInputs = dependencies;
     dependencies = builtins.map (d: d.name) dependencies;
@@ -35,7 +35,7 @@ let
   } @ args: stdenvNoCC.mkDerivation {
     # stdenvNoCC is to avoid generating derivations with names
     # like foo.service-mips-linux-musl
-    name = "${name}.service";
+    inherit name;
     serviceType = "oneshot";
     # does this suffice to make sure dependencies are included
     # even though the built output has no references to their
@@ -62,7 +62,7 @@ let
     shell = "${busybox}/bin/sh";
     builder = ./builder.sh;
   };
-  bundle = { name, ... } @args : target (args // { name = "${name}.bundle";});
+  bundle = { name, ... } @args : target (args // { inherit name;});
 in {
   inherit target bundle oneshot longrun output;
 }

tests/smoke/run.sh

@@ -13,10 +13,10 @@ cd $dest_path;
 db=nix/store/*-s6-rc-database/compiled/
 test -d $db
 chmod -R +w $db
-# check we have closure of config.services (lo.link.service exists only
+# check we have closure of config.services (lo.link service exists only
 # as a dependency)
-test "$(s6-rc-db -c $db type lo.link.service)" = "oneshot"
-test "$(s6-rc-db -c $db type ntp.service)" = "longrun"
+test "$(s6-rc-db -c $db type lo.link)" = "oneshot"
+test "$(s6-rc-db -c $db type ntp)" = "longrun"
 echo OK
 EOF
      )