Skip to content
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

APP_DATA_PATH should not default to directory next to executable file #3079

Closed
0rzech opened this issue Dec 3, 2017 · 5 comments
Closed

Comments

@0rzech
Copy link
Contributor

0rzech commented Dec 3, 2017

As of now, the lack of GITEA_CUSTOM variable results in Gitea attempting to create its data directory next to executable file.

This is both counter intuitive and defective, because in most (I guess) cases, the executable will be placed in read-only directory. This may lead to issues like #3052, where developer uses right variable, but user still ends up with broken install.

The APP_DATA_PATH should default to ${HOME}/gitea on Linux and similar directories on other supported operating systems. These directories are writable by default and when there's no home directory, it was likely due to conscious admin decision.

@lafriks
Copy link
Member

lafriks commented Dec 3, 2017

If no environment variable is set it defaults to current working directory I think

@0rzech
Copy link
Contributor Author

0rzech commented Dec 3, 2017

Looks like it was the case until #3052, nevertheless the documentation says:

If you install Gitea from binary, after the installation process ends, you can find the custom folder next to the binary.

All in all, both relative-to-executable and PWD dependent values are rather unusual choices for this kind of software.

@lafriks
Copy link
Member

lafriks commented Dec 3, 2017

CWD is for ease of use when developing and also has been so historically.
It has been relative to executable from the start of the development but we have tried to move away from that and with 1.3.0 I hoped we have done that but probably there is still some error somewhere :(

@stale
Copy link

stale bot commented Feb 10, 2019

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs during the next 2 weeks. Thank you for your contributions.

@stale stale bot added the issue/stale label Feb 10, 2019
@stale
Copy link

stale bot commented Feb 24, 2019

This issue has been automatically closed because of inactivity. You can re-open it if needed.

@stale stale bot closed this as completed Feb 24, 2019
@go-gitea go-gitea locked and limited conversation to collaborators Nov 24, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants