From 3478ffbc5a14a9dbe99ac451870cd8f2000c3350 Mon Sep 17 00:00:00 2001
From: Anmol Sethi <anmol@terrastruct.com>
Date: Tue, 6 Dec 2022 07:54:14 -0800
Subject: [PATCH] install.sh: Improvements

---
 ci/release/_install.sh             | 22 ++--------------------
 ci/release/template/scripts/lib.sh |  4 ++--
 ci/sub                             |  2 +-
 install.sh                         | 26 ++++----------------------
 4 files changed, 9 insertions(+), 45 deletions(-)

diff --git a/ci/release/_install.sh b/ci/release/_install.sh
index c791b0918..6428f432b 100755
--- a/ci/release/_install.sh
+++ b/ci/release/_install.sh
@@ -207,16 +207,12 @@ main() {
   if [ -n "${UNINSTALL-}" ]; then
     uninstall
     if [ -n "${DRY_RUN-}" ]; then
-      bigheader "***********
-Rerun without --dry-run to execute printed commands and perform install.
-***********"
+      bigheader "Rerun without --dry-run to execute printed commands and perform install."
     fi
   else
     install
     if [ -n "${DRY_RUN-}" ]; then
-      bigheader "***********
-Rerun without --dry-run to execute printed commands and perform install.
-***********"
+      bigheader "Rerun without --dry-run to execute printed commands and perform install."
     fi
   fi
 }
@@ -277,13 +273,6 @@ EOF
       log "Run man d2plugin-tala for detailed TALA docs."
     fi
   fi
-  logcat >&2 <<EOF
-
-Something not working? Please let us know:
-https://github.com/terrastruct/d2/issues
-https://github.com/terrastruct/d2/discussions
-https://discord.gg/NF6X8K4eDq
-EOF
 }
 
 install_post_brew() {
@@ -298,13 +287,6 @@ install_post_brew() {
   if [ -n "${TALA-}" ]; then
     log "Run man d2plugin-tala for detailed TALA docs."
   fi
-  logcat >&2 <<EOF
-
-Something not working? Please let us know:
-https://github.com/terrastruct/d2/issues
-https://github.com/terrastruct/d2/discussions
-https://discord.gg/NF6X8K4eDq
-EOF
 
   VERSION=$(brew info d2 | head -n1 | cut -d' ' -f4)
   VERSION=${VERSION%,}
diff --git a/ci/release/template/scripts/lib.sh b/ci/release/template/scripts/lib.sh
index bd7386b35..138f9f003 100644
--- a/ci/release/template/scripts/lib.sh
+++ b/ci/release/template/scripts/lib.sh
@@ -273,9 +273,9 @@ header() {
 
 bigheader() {
   set -- "$(echo "$*" | sed "s/^/ * /")"
-  FGCOLOR=${FGCOLOR:-5} logp "/**
+  FGCOLOR=${FGCOLOR:-3} logp "/****************************************************************
 $*
- **/"
+ ****************************************************************/"
 }
 
 # humanpath replaces all occurrences of " $HOME" with " ~"
diff --git a/ci/sub b/ci/sub
index a3961af42..718a738a1 160000
--- a/ci/sub
+++ b/ci/sub
@@ -1 +1 @@
-Subproject commit a3961af4299a6b9e642ebce305609f091eb7e88d
+Subproject commit 718a738a16a9de19ec8d5f9654a9bb94546d8898
diff --git a/install.sh b/install.sh
index a17bc2f35..71a1ba2bb 100755
--- a/install.sh
+++ b/install.sh
@@ -278,9 +278,9 @@ header() {
 
 bigheader() {
   set -- "$(echo "$*" | sed "s/^/ * /")"
-  FGCOLOR=${FGCOLOR:-5} logp "/**
+  FGCOLOR=${FGCOLOR:-3} logp "/****************************************************************
 $*
- **/"
+ ****************************************************************/"
 }
 
 # humanpath replaces all occurrences of " $HOME" with " ~"
@@ -790,16 +790,12 @@ main() {
   if [ -n "${UNINSTALL-}" ]; then
     uninstall
     if [ -n "${DRY_RUN-}" ]; then
-      bigheader "***********
-Rerun without --dry-run to execute printed commands and perform install.
-***********"
+      bigheader "Rerun without --dry-run to execute printed commands and perform install."
     fi
   else
     install
     if [ -n "${DRY_RUN-}" ]; then
-      bigheader "***********
-Rerun without --dry-run to execute printed commands and perform install.
-***********"
+      bigheader "Rerun without --dry-run to execute printed commands and perform install."
     fi
   fi
 }
@@ -860,13 +856,6 @@ EOF
       log "Run man d2plugin-tala for detailed TALA docs."
     fi
   fi
-  logcat >&2 <<EOF
-
-Something not working? Please let us know:
-https://github.com/terrastruct/d2/issues
-https://github.com/terrastruct/d2/discussions
-https://discord.gg/NF6X8K4eDq
-EOF
 }
 
 install_post_brew() {
@@ -881,13 +870,6 @@ install_post_brew() {
   if [ -n "${TALA-}" ]; then
     log "Run man d2plugin-tala for detailed TALA docs."
   fi
-  logcat >&2 <<EOF
-
-Something not working? Please let us know:
-https://github.com/terrastruct/d2/issues
-https://github.com/terrastruct/d2/discussions
-https://discord.gg/NF6X8K4eDq
-EOF
 
   VERSION=$(brew info d2 | head -n1 | cut -d' ' -f4)
   VERSION=${VERSION%,}