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

Avoid hard coded /tmp directory #149

Open
jfdenise opened this issue Jan 21, 2020 · 2 comments
Open

Avoid hard coded /tmp directory #149

jfdenise opened this issue Jan 21, 2020 · 2 comments

Comments

@jfdenise
Copy link
Contributor

In the scripts that use /tmp dir, this could be replaced by: tmpdir="${TMPDIR:-/tmp}"

@yersan
Copy link
Contributor

yersan commented Jan 24, 2020

The use case sounds very strange to me, I don't see the use case yet. We can change it and replace their uses, but it will increase the "complexity" of keeping up additional PRs that include script files safe regarding not to use /tmp as the default one.

Having said that, so you still see it useful?, if so, I can create a PR with the required changes

@jfdenise
Copy link
Contributor Author

@yersan , I just created this issue not to loose feedbacks provided during a review. If your analysis show that this is not that useful and would create complexity, we certainly can not address it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants