Github user lfrancke commented on a diff in the pull request:
https://github.com/apache/nifi/pull/2985#discussion_r216626331
--- Diff:
nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-resources/src/main/resources/bin/nifi-env.sh
---
@@ -16,16 +16,38 @@
# limitations under the License.
#
+# By default this file will unconditionally override whatever environment
variables you have set
+# and set them to defaults defined here.
+# If you want to define your own versions outside of this script please
set the environment variable
+# NIFI_OVERRIDE_NIFIENV to "true". That will then use whatever variables
you used outside of
+# this script.
+
# The java implementation to use.
#export JAVA_HOME=/usr/java/jdk1.8.0/
-export NIFI_HOME=$(cd "${SCRIPT_DIR}" && cd .. && pwd)
+setOrDefault() {
+ declare envvar="$1" default="$2"
+
+ local res="$envvar"
+ if [ -z "$envvar" ] || [ "$NIFI_OVERRIDE_NIFIENV" != "true" ]
--- End diff --
Sorry @pepov ! No worries at all. I'm on vacation and won't be back for
another week. I'll get to it then. Your comments make sense. I just haven't
gotten around to look at them in detail.
---
