-
Notifications
You must be signed in to change notification settings - Fork 743
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
add the OPENGROK_DEFAULT_PROJECTS env var #2081
Conversation
976085f
to
f8627e2
Compare
OpenGrok
Outdated
@@ -311,6 +314,13 @@ DefaultInstanceConfiguration() | |||
*) ENABLE_PROJECTS="-P" ;; | |||
esac | |||
|
|||
if [ -n "${ENABLE_PROJECTS}" ]; then | |||
DEFAULT_PROJECTS="" | |||
for proj in $( IFS=","; echo ${OPENGROK_DEFAULT_PROJECTS} ); do |
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.
non blocking but wrong indentation
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.
fixed
The only point for this is if this is really useful because (as far as I remember - this might have changed) this default projects are used only when we don't have any cookies which is:
Neither of this is very often. |
Pull Request Test Coverage Report for Build 2392
💛 - Coveralls |
f8627e2
to
ddad4dc
Compare
True, this only works when there are no cookies. I guess it is a way how to guide first time users a bit. |
added some documentation to https://github.com/oracle/opengrok/wiki/Webapp-configuration#default-projects |
Yes it is a bit more user friendly 😀 |
passthru for the -p option. I don't like that the shell script is a filter to
Indexer
however until it is rewritten such changes need to happen.