fix: cosmetic comment fixes

This rephrases a comment in the check() as requested on another PR
and generalizes the install() section comments on par with other recent
module changes.

diff --git a/modules.d/01systemd-coredump/module-setup.sh b/modules.d/01systemd-coredump/module-setup.sh
index 8b85e93cfdb0d8865102fccabcb5da6f732ddd7c..adb69777712672355acb193ddf2a7be91eaed4d7 100755 (executable)
--- a/modules.d/01systemd-coredump/module-setup.sh
+++ b/modules.d/01systemd-coredump/module-setup.sh
@@ -5,8 +5,7 @@
 # Prerequisite check(s) for module.
 check() {
 
-    # If the binary(s) requirements are not fulfilled
-    # return 1 to not include the binary.
+    # If the binary(s) requirements are not fulfilled the module can't be installed
     require_binaries coredumpctl || return 1
     require_binaries $systemdutildir/systemd-coredump || return 1
 
@@ -25,16 +24,10 @@ depends() {
 
 }
 
-# Install the required file(s) for the module in the initramfs.
+# Install the required file(s) and directories for the module in the initramfs.
 install() {
 
-    # Install the required directories.
     inst_dir   /var/lib/systemd/coredump
-    # Install the required file(s.
-    # Install the kernel configuration parameters for coredump.
-    # Install vendor configuration files.
-    # Install the systemd type service unit for coredump.
-    # Install the binary executable(s) for sysusers.
     inst_multiple -o \
         $sysctld/50-coredump.conf \
         $systemdutildir/coredump.conf \

diff --git a/modules.d/01systemd-repart/module-setup.sh b/modules.d/01systemd-repart/module-setup.sh
index 6f370bd468b0baeda937fd9b634e384320b192b9..0c6b6b50f04341ebf49f27e89501c03b80844ddc 100755 (executable)
--- a/modules.d/01systemd-repart/module-setup.sh
+++ b/modules.d/01systemd-repart/module-setup.sh
@@ -5,8 +5,7 @@
 # Prerequisite check(s) for module.
 check() {
 
-    # If the binary(s) requirements are not fulfilled
-    # return 1 to not include the binary.
+    # If the binary(s) requirements are not fulfilled the module can't be installed
     require_binaries systemd-repart || return 1
 
     # Return 255 to only include the module, if another module requires it.
@@ -27,9 +26,6 @@ depends() {
 # Install the required file(s) for the module in the initramfs.
 install() {
 
-    # Install vendor repartition configurations
-    # Install the systemd type service unit for systemd repart.
-    # Install the binary executable(s) for systemd repart
     inst_multiple -o \
         $libdir/repart.d/*.conf \
         $systemdsystemunitdir/systemd-repart.service \

diff --git a/modules.d/06dbus-broker/module-setup.sh b/modules.d/06dbus-broker/module-setup.sh
index d18deb41311afdfdc2222f06c01016f25e29ea7a..efed8079672043e1b345a2e994869df35102bd96 100755 (executable)
--- a/modules.d/06dbus-broker/module-setup.sh
+++ b/modules.d/06dbus-broker/module-setup.sh
@@ -5,8 +5,7 @@
 # Prerequisite check(s) for module.
 check() {
 
-    # If the binary(s) requirements are not fulfilled
-    # return 1 to not include the binary.
+    # If the binary(s) requirements are not fulfilled the module can't be installed
     require_binaries busctl || return 1
     require_binaries dbus-broker || return 1
     require_binaries dbus-broker-launch || return 1
@@ -25,7 +24,7 @@ depends() {
 
 }
 
-# Install the required file(s) for the module in the initramfs.
+# Install the required file(s) and directories for the module in the initramfs.
 install() {
 
     # Create dbus related directories.
@@ -42,16 +41,6 @@ install() {
     inst_dir $dbussystemconfdir
     inst_dir $dbussystemservicesconfdir
 
-    # Install the dbus user session configuration file.
-    # Install the dbus system configuration file.
-    # The systemd module should be providing this and
-    # depend on the dbus module. Added here until it does.
-    # Install the dbus users and groups configuration file.
-    # Install the dbus-broker systemd journal message catalogs files.
-    # Install the systemd type service unit for dbus-broker.
-    # Install the systemd type socket unit for dbus.
-    # Install the dbus target.
-    # Install the binary executable(s) for dbus-broker.
     inst_multiple -o \
         $dbus/session.conf \
         $dbus/system.conf \

diff --git a/modules.d/06dbus-daemon/module-setup.sh b/modules.d/06dbus-daemon/module-setup.sh
index 273dd3d7c38e5419b5758c40a0a958fc2e8f70a7..fe0d64c45d53b6c4c3bd75884dfa6ce093759b37 100755 (executable)
--- a/modules.d/06dbus-daemon/module-setup.sh
+++ b/modules.d/06dbus-daemon/module-setup.sh
@@ -5,8 +5,7 @@
 # Prerequisite check(s) for module.
 check() {
 
-    # If the binary(s) requirements are not fulfilled
-    # return 1 to not include the binary.
+    # If the binary(s) requirements are not fulfilled the module can't be installed
     require_binaries busctl || return 1
     require_binaries dbus-daemon || return 1
     require_binaries dbus-send || return 1
@@ -30,7 +29,7 @@ depends() {
     return 0
 }
 
-# Install the required file(s) for the module in the initramfs.
+# Install the required file(s) and directories for the module in the initramfs.
 install() {
 
     # Create dbus related directories.
@@ -47,13 +46,6 @@ install() {
     inst_dir $dbussystemconfdir
     inst_dir $dbussystemservicesconfdir
 
-    # Install the dbus system configuration file.
-    # The systemd module should be providing this and
-    # depend on the dbus module. Added here until it does.
-    # Install the systemd type service unit for dbus.
-    # Install the systemd type socket unit for dbus.
-    # Install the dbus target.
-    # Install the binary executable(s) for dbus.
     inst_multiple -o \
         $dbus/system.conf \
         $dbussystem/org.freedesktop.systemd1.conf \