setup: etc/profile PS1 it's shell specific and should not be exported! con...

glen glen at pld-linux.org
Wed Jun 12 01:09:42 CEST 2013


Author: glen                         Date: Tue Jun 11 23:09:42 2013 GMT
Module: setup                         Tag: HEAD
---- Log message:
PS1 it's shell specific and should not be exported!
consider scenario: PS1 in mksh having bultin 'print':
  PS1=$(print "$PWD")
now, executing bash, which does not have 'print' builtin, and no no bashrc
present. bash would get invalid PS1 with mksh builtin 'print' and each prompt
giving error about command not found.

---- Files affected:
setup/etc:
   profile (1.58 -> 1.59) 

---- Diffs:

================================================================
Index: setup/etc/profile
diff -u setup/etc/profile:1.58 setup/etc/profile:1.59
--- setup/etc/profile:1.58	Fri Mar 30 21:43:52 2012
+++ setup/etc/profile	Wed Jun 12 01:09:37 2013
@@ -110,7 +110,7 @@
 esac
 
 unset SH
-export PATH USER LOGNAME HOSTNAME HISTFILE HISTSIZE PS1 PS2
+export PATH USER LOGNAME HOSTNAME HISTFILE HISTSIZE PS2
 
 # Put all of the variables as files in /etc/env.d/
 # example:
================================================================

---- CVS-web:
    http://cvs.pld-linux.org/setup/etc/profile?r1=1.58&r2=1.59



More information about the pld-cvs-commit mailing list