New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Minor changes in Readme #499
base: master
Are you sure you want to change the base?
Conversation
Signed-off-by: megha070 <varshney.megha070@gmail.com>
@bmpotter existing readme consists of postgres installation instructions for Mac OS but not Linux. Shall I include those too? |
@@ -53,6 +53,7 @@ export EXCHANGE_ROOTPW=myrootpw | |||
``` | |||
export EXCHANGE_KEY_PW=<pass-phrase> | |||
make gen-key | |||
Note : pass-phrase can be any alphanumeric string |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Are you sure special characters aren't allowed?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I need to confirm this once.
- Before exporting below environment variables manually set them in `src/test/bash/primedb.sh` | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't understand this sentence. primedb.sh
gets these variables from the environment at
exchange-api/src/test/bash/primedb.sh
Line 40 in b905464
user="${EXCHANGE_USER:-me}" |
primedb.sh
``` | ||
export EXCHANGE_USER=<my-user-in-IBM-org> | ||
export EXCHANGE_PW=<my-pw-in-IBM-org> | ||
export EXCHANGE_ROOTPW = <Exchange root password (Must be similar to what is set in exchange's config.json)> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The Preconditions section already tells them to set EXCHANGE_ROOTPW
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oh yes, I missed it.
Signed-off-by: megha070 varshney.megha070@gmail.com