-
-
Notifications
You must be signed in to change notification settings - Fork 5.8k
Add FHS-compliant-script #6923
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 FHS-compliant-script #6923
Conversation
Codecov Report
@@ Coverage Diff @@
## master #6923 +/- ##
==========================================
- Coverage 41.45% 41.44% -0.01%
==========================================
Files 442 442
Lines 59610 59610
==========================================
- Hits 24711 24708 -3
- Misses 31669 31672 +3
Partials 3230 3230
Continue to review full report at Codecov.
|
APP_INI="/etc/gitea/app.ini" | ||
|
||
APP_INI_SET="" | ||
for i in "$@"; 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.
nice 👍
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.
Just realised I should put $CONF_ARG bit first just in case we have commands with arguments later.
@mrsdizzie do you think we should be setting HOME too? |
I think it's probably ok not setting HOME actually - everything that defaults to using it can be set with app.ini. |
Agree, $HOME should just be whatever it is and we shouldn't mess with it |
This adds a contrib script that recommends shadowing the gitea binary on debian systems and sets environment variable defaults for these systems.
This should probably be considered a WIP or suggestion unless people think it's exactly right.