2-common/tasks/packages.yml: lint & explain

roles/2-common/tasks/packages.yml

@@ -17,12 +17,12 @@
 - name: "Install 6 deb/apt packages: avahi-daemon, exfat-fuse, exfat-utils, inetutils-syslogd, libnss-mdns, wpasupplicant (debuntu)"
   package:
     name:
-#      - avahi-discover
+      #- avahi-discover
       - avahi-daemon
-      - exfat-fuse     # Should no longer be nec with 5.4+ kernels.
-      - exfat-utils    # Likewise.
-      - inetutils-syslogd
-      - libnss-mdns
+      - exfat-fuse     # Should no longer be nec with 5.4+ kernels
+      - exfat-utils    # Likewise
+      - inetutils-syslogd    # Error logging facility
+      - libnss-mdns    # Provides name resolution via mDNS (Multicast DNS) using Zeroconf/Bonjour e.g. Avahi
       - wpasupplicant
     state: present
   when: is_debuntu
@@ -30,15 +30,15 @@
 - name: "Install 22 common packages: acpid, bridge-utils, bzip2, curl, gawk, hostapd, htop, i2c-tools, logrotate, make, mlocate, netmask, net-tools, ntfs-3g, pandoc, pastebinit, rsync, sqlite3, tar, unzip, usbutils, wget"
   package:
     name:
-      - acpid
-      - bridge-utils
+      - acpid    # Daemon for ACPI (power mgmt) events
+      - bridge-utils    # DEPRECATED according to https://wiki.linuxfoundation.org/networking/bridge
       - bzip2
       - curl
       #- etckeeper    # "nobody is really using etckeeper and it's bloating the filesystem every time apt runs" per @jvonau at https://github.com/iiab/iiab/issues/1146
       - gawk
       - hostapd
       - htop
-      - i2c-tools
+      - i2c-tools    # Low-level bus/chip/register/EEPROM tools e.g. for RTC
       - logrotate
       #- lynx    # Installed by 1-prep's roles/iiab-admin/tasks/access.yml
       - make
@@ -56,6 +56,6 @@
       - tar
       - unzip
       #- usbmount    # Moved to roles/usb_lib/tasks/install.yml
-      - usbutils
+      - usbutils     # Should this also move to roles/usb_lib/tasks/install.yml ?
       - wget
     state: present