<div dir="ltr">Hi,<div><br></div><div>If that's the only reason for this change, I would keep the previous version and let the user decide where to append GRASS_ADDON_PATH in ~/.grass7/bashrc. </div><div><br></div><div>
1. It's breaking "export NEW_VAR=$HOME/grassdata" because $HOME doesn't get expanded by bash.<br></div><div><br></div><div>2. It's inconsistent to append ("keep") GRASS_ADDON_PATH and ignore other environment variables in ~/.grass7/bashrc. </div>
<div><br></div><div>3. We should "not" simply ignore any environment variables in ~/.grass7/bashrc even if they already exist outside a GRASS session.</div><div><br></div><div>4. grass.py is appending GRASS_ADDON_PATH in ~/.grass7/bashrc only "after" existing GRASS_ADDON_PATH. What if they want to append their path "before" the one in ~/.grass7/bashrc?<br>
</div><div><br></div><div>As to my original concern, even <span style="font-size:13px;font-family:arial,sans-serif">r61216 doesn't allow overwriting PS1 initially created by grass.py. As a side effect of startswith('export'), I can work around it by appending whitespaces before export PS1=, but I would say that's a bug, not a feature.</span><br>
</div><div><span style="font-size:13px;font-family:arial,sans-serif"><br></span></div><div><span style="font-size:13px;font-family:arial,sans-serif">Anyway, my point is we can address the original issue by adding the following line in ~/.grass7/bashrc:</span></div>
<div><span style="font-size:13px;font-family:arial,sans-serif"><br></span></div><div><font face="arial, sans-serif">export GRASS_ADDON_PATH="$GRASS_ADDON_PATH:~/grass-addon/bin"</font></div><div><font face="arial, sans-serif"><br>
</font></div><div><font face="arial, sans-serif">This is more flexible and doesn't impose "appending" of GRASS_ADDON_PATH and ignoring other variables in ~/.grass7/bashrc. In fact, like I said above in 1, the new version "breaks" this feature:</font></div>
<div><font face="arial, sans-serif"><br></font></div><div><font face="arial, sans-serif">export NEW_VAR=$HOME/grassdata</font></div><div><font face="arial, sans-serif"><br></font></div><div><font face="arial, sans-serif">doesn't expand to /home/user/grassdata.</font></div>
<div><font face="arial, sans-serif"><br></font></div><div><font face="arial, sans-serif">IMHO, it should be the user's responsibility and rights to overwrite or not overwrite anything available outside a GRASS session and that should happen in ~/.grass7/bashrc.</font></div>
<div><span style="font-size:13px;font-family:arial,sans-serif"><br></span></div><div><span style="font-size:13px;font-family:arial,sans-serif">Best,</span></div><div><font face="arial, sans-serif">Huidae</font></div><div>
<span style="font-size:13px;font-family:arial,sans-serif"><br></span></div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, Jul 9, 2014 at 7:44 AM, Martin Landa <span dir="ltr"><<a href="mailto:landa.martin@gmail.com" target="_blank">landa.martin@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi,<br>
<br>
2014-07-04 21:02 GMT+02:00 Huidae Cho <<a href="mailto:grass4u@gmail.com">grass4u@gmail.com</a>>:<br>
<div class="">> Please explain why we need this change. It breaks custom PS1 and aliases in<br>
> $HOME/.grass7/bashrc.<br>
<br>
</div>sorry for late answer, the original reason was:<br>
<br>
1) you have defined GRASS_ADDON_PATH in $HOME/.grass7/bashrc<br>
2) you launch GRASS with $ GRASS_ADDON_PATH=~/bin/md-iso grass71<br>
<br>
GRASS_ADDON_PATH points just to the path from `$HOME/.grass7/bashrc`.<br>
The patch enables to merge paths from $HOME/.grass7/bashrc together<br>
with `~/bin/md-iso`. I was not aware of aliases, you are absolutely<br>
right, I will fix it ASAP. Martin<br>
<span class="HOEnZb"><font color="#888888"><br>
<br>
--<br>
Martin Landa * <a href="http://geo.fsv.cvut.cz/gwiki/Landa" target="_blank">http://geo.fsv.cvut.cz/gwiki/Landa</a><br>
</font></span></blockquote></div><br></div>