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

Using $TMPDIR as default temp working folder? #41

Open
tseemann opened this issue Jun 24, 2024 · 2 comments
Open

Using $TMPDIR as default temp working folder? #41

tseemann opened this issue Jun 24, 2024 · 2 comments
Labels
enhancement New feature or request

Comments

@tseemann
Copy link

tseemann commented Jun 24, 2024

The current directory is set as the default working directory.

Would you consider using the Unix standard $TMPDIR veriable as the default instead?

THis is honoured by the tmpdir() API in most languages and set on HPC systems to /tmp or /scratch to avoid killing NFS home mounts etc.

@rob-p
Copy link
Contributor

rob-p commented Jun 24, 2024

Hi @tseemann,

Thanks for the suggestion! @jamshed; I think we should do this. The only complication is the following. Currently, we have an explicit command line parameter to set an alternative temporary directory. I'd like to keep this to avoid breaking backward compatibility. I propose that we alter the behavior to use the path provided on the command line explicitly, if provided, and otherwise we pull from the TMPDIR variable / whatever std::filesystem::temp_directory_path (>= C++17) returns.

@jamshed jamshed added the enhancement New feature or request label Jun 25, 2024
@jamshed
Copy link
Member

jamshed commented Jun 25, 2024

This commit addresses this issue: 6f8acbd. It's in the develop branch. We should make a release soon with the other pending stuff in develop.

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

No branches or pull requests

3 participants