Support for gn_db env variable added#3
Conversation
…s used as default. db_admin variable deleted
doublebyte1
left a comment
There was a problem hiding this comment.
These changes are relevant, and are working as expected. Normally, by convention, UNIX environmental variables are capitalized; could you switch db_gn to DB_GN?
|
Quick question as I was about to implement the same change (and this PR is already over a year old): Why are you not using the same "default value" syntax as is already used for other variables e.g. I'd suggest something like: db_gn="${POSTGRES_DB_GN:-geonetwork}" |
|
|
|



If gn_db isn't defined when run command is executed 'geonetwork' is used as default database name. 'db_admin' variable was deleted, it isn't used by the application.