From 35580419b5cb68066c9fda75085c2d944f41e241 Mon Sep 17 00:00:00 2001
From: Curtis McEnroe <june@causal.agency>
Date: Fri, 19 Apr 2019 20:00:31 -0400
Subject: Set PS1 to \$ by default

Just a simplification of presumably old behavior.
---
 bin/catsh/catsh.1 |  4 +---
 bin/catsh/var.c   | 14 ++------------
 2 files changed, 3 insertions(+), 15 deletions(-)

(limited to 'bin')

diff --git a/bin/catsh/catsh.1 b/bin/catsh/catsh.1
index 60c93bf9..59f22a3e 100644
--- a/bin/catsh/catsh.1
+++ b/bin/catsh/catsh.1
@@ -1548,9 +1548,7 @@ may include any of the formatting sequences from
 .
 .It Va PS1
 The primary prompt string, which defaults to
-.Dq Li "$ " ,
-unless you are the superuser, in which case it defaults to
-.Dq Li "# " .
+.Dq Li "\e$ " .
 .Va PS1
 may include any of the following formatting sequences,
 which are replaced by the given information:
diff --git a/bin/catsh/var.c b/bin/catsh/var.c
index f11ff8b7..435d3307 100644
--- a/bin/catsh/var.c
+++ b/bin/catsh/var.c
@@ -129,9 +129,8 @@ static const struct varinit varinit[] = {
 	  changepath },
 	{ &vps0,	VUNSET,				"PS0=",
 	  NULL },
-	/*
-	 * vps1 depends on uid
-	 */
+	{ &vps1,	0,				"PS1=\\$ ",
+	  NULL },
 	{ &vps2,	0,				"PS2=> ",
 	  NULL },
 	{ &vps4,	0,				"PS4=+ ",
@@ -194,15 +193,6 @@ initvar(void)
 		vp->flags = ip->flags | VSTRFIXED | VTEXTFIXED;
 		vp->func = ip->func;
 	}
-	/*
-	 * PS1 depends on uid
-	 */
-	if (find_var("PS1", &vpp, &vps1.name_len) == NULL) {
-		vps1.next = *vpp;
-		*vpp = &vps1;
-		vps1.text = __DECONST(char *, geteuid() ? "PS1=$ " : "PS1=# ");
-		vps1.flags = VSTRFIXED|VTEXTFIXED;
-	}
 	fmtstr(ppid, sizeof(ppid), "%d", (int)getppid());
 	setvarsafe("PPID", ppid, 0);
 	for (envp = environ ; *envp ; envp++) {
-- 
cgit 1.4.1