[GRASS-dev] [GRASS-SVN] r65307 - grass/trunk/lib/init
grass4u at gmail.com
Tue Jul 14 07:17:27 PDT 2015
Is there any good reason why grass.py imports environment variables in
load_env() and writes non-/^export/ lines (r65585 does not strip out
whitespaces) into MAPSET/.bashrc in bash_startup() instead of simply
sourcing or fully writing .grass7/bashrc? I see a couple of problems with
this two step approach.
1. /^export/ lines in .grass.bashrc don't work anymore with r65585. This
file used to be "fully" written into MAPSET/.bashrc.
2. Conditional constructs like the following are not supported in
case $TERM in
export PS1=xterm # no space before export
because load_env() would overwrite PS1 and bash_startup() won't write out
the /^export PS1/ lines. It will work if there are whitespaces before
export, but I would say that was not intended by you...
On Tue, Jul 14, 2015 at 8:51 AM, Martin Landa <landa.martin at gmail.com>
> 2015-07-08 18:31 GMT+02:00 Huidae Cho <grass4u at gmail.com>:
> > I think the following change has to be reverted because it breaks aliases
> > and custom prompts defined in ~/.grass7/bashrc. Currently, only
> > lines are parsed from this file in load_env().
> sorry for delay, I took liberty to partly re-introduce r65307 in
> r65585. It should work with aliases now.
> Martin Landa
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the grass-dev