sqenv.sh is not idempotent

[Chris Sheedy]

Invoking sqenv.sh more than once in the same shell should be a safe operation, and not increase the size of environment variables. Setting SQ_VERBOSE=1 is an easy way to see the growth in $PATH each time sqenv.sh is invoked.

[Chris Sheedy (chris-sheedy)]

An example message that should not be given is in the developer environment, when starting with clean source code for trafodion/core :

$ . ./sqenvr.sh
$ . ./sqenvr.sh
The environment changed from a previous execution of this script.
This is not supported. To change environments, do the following:
  sqstop
  <make any changes, e.g. update Hadoop, HBase, MySQL>
  start a new shell and source in sqenv.sh
  rm $MY_SQROOT/etc/ms.env
  sqgen
  start a new shell and source in sqenv.sh
  sqstart
$

[Chris Sheedy (chris-sheedy)]

Gerrit change 811 on 2014-12-11 modified sqenvcom.sh to fix this problem.

[Chris Sheedy (chris-sheedy)]

In r1.1, this is almost fixed, the only problem is that the MANPATH variable keeps growing.
For example, sqenvr.sh was invoked several times like this:

$ . ./sqenvr.sh
$ env > env_2
$ . ./sqenvr.sh
$ env > env_3
$ . ./sqenvr.sh
$ env > env_4

Differences between the env_2, env_3 and env_4 files were only the growth of MANPATH:

$ diff env_3 env_4
< MANPATH=:<dir>/sqf/export/share/man:<dir>/sqf/export/share/man
---
> MANPATH=:<dir>/sqf/export/share/man:<dir>/sqf/export/share/man:<dir>/sqf/export/share/man
$

[Chris Sheedy (chris-sheedy)]

Fixed in change 1634 merged into master branch on 2015-05-15.