Feat(bash_prompt): Improve `PS1` generation

Refactor the common part (`_COMMON_PROMPT`).
Use apostrophes instead of double quotes.

diff --git a/admin/prog/bash_prompt b/admin/prog/bash_prompt
index b89d340e23b72eef93aadd2fe33bf6ea2fe38304..02141e3581ab17bf4400720b642e256a7872159d 100644 (file)
--- a/admin/prog/bash_prompt
+++ b/admin/prog/bash_prompt
@@ -103,15 +103,15 @@ prompt_git() {
 
 set_prompts() {
       OPS1=''
+      local _COMMON_PROMPT='${debian_chroot:+($debian_chroot)}\u@\h:\w'
       # display the user, host and current working directory
       # in the terminal title
       case "$TERM" in
          *rxvt*|screen*|*term*|vt100)
-            OPS1+="\[\033]0;${debian_chroot:+($debian_chroot)}\u@\h:\w\007\]"
+            OPS1+="\[\033]0;${_COMMON_PROMPT}\007\]"
             case "$TERM" in
                screen*)
-                  OPS1+="\[\033k${debian_chroot:+($debian_chroot)}"
-                  OPS1+="\u@\h:\w\033\\\\\]" # Set screen/tmux caption
+                  OPS1+="\[\033k${_COMMON_PROMPT}\033\\\\\]" # Set screen/tmux caption
                ;;
             esac
          ;;
@@ -121,17 +121,17 @@ set_prompts() {
          ;;
       esac
 
-   OPS1+="${debian_chroot:+($debian_chroot)}"
-   OPS1+="\`cgmem_which_prompt\`\u@\${HOSTNAME::5}:\`short_curdir\`"
+   OPS1+='${debian_chroot:+($debian_chroot)}'
+   OPS1+='`cgmem_which_prompt`\u@${HOSTNAME::5}:`short_curdir`'
    if test -x /usr/bin/git >/dev/null 2>&1; then
       #if [ "`type -t __git_ps1`" = function ]; then
-      #   OPS1+="\$(__git_ps1)"
+      #   OPS1+='$(__git_ps1)'
       #else
-         OPS1+="\$(prompt_git)"
+         OPS1+='$(prompt_git)'
       #fi
    fi
    # This is for .screenrc: shelltitle "\$ |$SHELL"
-   #OPS1+="\[\033k\033\\\\\]"
-   OPS1+=" \$SHLVL\\$ "
+   #OPS1+='\[\033k\033\\\]'
+   OPS1+=' $SHLVL\$ '
    export OPS1
 }