misc/build.func

@@ -129,7 +129,7 @@ ssh_check() {
   fi
 }
 
-# This function sets base settings which are common for all scripts. The settings can be overriden in default_settings or advanced_settings which are defined in each script
+# This function allows to centralize all common options so that services only need to define settings that need to be overridden (Avoid duplication of information)
 base_settings() {
   CT_TYPE="1"
   PW=""
@@ -151,7 +151,7 @@ base_settings() {
   VLAN=""
   SSH="no"
   VERB="no"
-  # Since these 2 are only defined outside of default_settings function, we add a temporary fallback. TODO: To align everything, we should add these as constant variables (e.g. OSTYPE and OSVERSION), but that would currently require updating the default_settings function for all existing scripts
+  # Since these 2 are defined outside of default_settings function, avoid overwriting them. To align everything, they should use another constant variable (say OSTYPE), but that would require updating the default_settings function for all existing scripts
   if [ -z "$var_os" ]; then
     var_os="debian"
   fi