From 3a91d7f2ecf1a30543bb4d2c36450153f1dc382e Mon Sep 17 00:00:00 2001
From: A Holt <holta@users.noreply.github.com>
Date: Mon, 26 Apr 2021 14:38:03 -0400
Subject: [PATCH] 2-common/tasks/packages.yml: lint & explain

---
 roles/2-common/tasks/packages.yml | 18 +++++++++---------
 1 file changed, 9 insertions(+), 9 deletions(-)

diff --git a/roles/2-common/tasks/packages.yml b/roles/2-common/tasks/packages.yml
index 83f3db415..2224816e1 100644
--- a/roles/2-common/tasks/packages.yml
+++ b/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