search

YottaDB / YDB

Mirrored from https://gitlab.com/YottaDB/DB/YDB
Other
76 stars 37 forks source link

#197 ydb env set overrides set variables #238

Closed ChristopherEdwards closed 6 years ago

ChristopherEdwards commented 6 years ago

This should resolve #197

ksbhaskar commented 6 years ago

Agreed. and the ydb_ environment variable should have precedence if set over the gtm one.

On 05/08/2018 11:47 AM, Christopher Edwards wrote:

@ChristopherEdwards commented on this pull request.

In sr_unix/ydb_env_set.gtc https://github.com/YottaDB/YottaDB/pull/238#discussion_r186776030:

@@ -27,6 +27,12 @@ if [ ! -z "$ydb_chset" ] ; then export ydb_tmp_chset=$ydb_chset elif [ ! -z "$gtm_chset " ] ; then export ydb_tmp_chset=$gtm_chset fi

+# Save ydb_routines / gtmroutines if defined because +# this script will overwrite ydb_routines if defined +# to make sure there is a path to the required routine +if [ ! -z "$ydb_routines" ] ; then ydb_tmp_routines="$ydb_routines" fi +if [ ! -z "$gtmroutines" ] ; then ydb_tmp_routines="$gtmroutines" fi

I was mimicking what was done with gtm_passwd/ydb_passwd. It saves it into the same variable. I can flip the statements so that ydb_routines takes precedence.

— You are receiving this because your review was requested. Reply to this email directly, view it on GitHub https://github.com/YottaDB/YottaDB/pull/238#discussion_r186776030, or mute the thread https://github.com/notifications/unsubscribe-auth/AAiqehmgdLqZGef_09AuleFgjCqVAKX3ks5twb4egaJpZM4T2zfs.

-- YottaDB - Rock solid. Lightning fast. Secure. Pick any three.