[fluo-uno] branch master updated: Recommend sourcing uno env instead of using eval (#231)

[ct...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

ctubbsii pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/fluo-uno.git


The following commit(s) were added to refs/heads/master by this push:
     new ae2bb4f  Recommend sourcing uno env instead of using eval (#231)
ae2bb4f is described below

commit ae2bb4f1cfcca752104ca3792943839d01f86533
Author: Christopher Tubbs <ct...@apache.org>
AuthorDate: Thu Sep 19 13:13:21 2019 -0700

    Recommend sourcing uno env instead of using eval (#231)
---
 README.md            | 4 ++--
 bin/impl/load-env.sh | 2 +-
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/README.md b/README.md
index fcdc2ef..741e6ad 100644
--- a/README.md
+++ b/README.md
@@ -38,7 +38,7 @@ git clone https://github.com/apache/fluo-uno.git
 cd fluo-uno
 ./bin/uno fetch accumulo            # Fetches binary tarballs of Accumulo and its dependencies
 ./bin/uno setup accumulo            # Sets up Accumulo and its dependencies (Hadoop & ZooKeeper)
-eval "$(./bin/uno env)"             # Bash-specific command that sets up current shell
+source <(./bin/uno env)             # Bash-specific command that sets up current shell
 ```
 
 Accumulo is now ready to use. Verify your installation by checking the [Accumulo Monitor](http://localhost:9995/)
@@ -99,7 +99,7 @@ You can either copy and paste this output into your shell or add the following (
 to your ~/.bashrc automatically configure every new shell.
 
 ```bash
-eval "$(/path/to/uno/bin/uno env)"
+source <(/path/to/uno/bin/uno env)
 ```
 
 With `uno` script set up, you can now use it to download, configure, and run Fluo's dependencies.
diff --git a/bin/impl/load-env.sh b/bin/impl/load-env.sh
index b5033ef..d465b5c 100755
--- a/bin/impl/load-env.sh
+++ b/bin/impl/load-env.sh
@@ -53,7 +53,7 @@ else
 fi
 
 function env_error() {
-  echo 'Make your shell env match uno.conf by running: eval "$(./bin/uno env)"'
+  echo 'Make your shell env match uno.conf by running: source <(./bin/uno env)'
   exit 1
 }