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

Fix building issues on OpenSUSE #294

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

JieRen98
Copy link

the config_arch script is for determining the architecture of the system. However, it generates a temp dir at /tmp using mktemp in this line and then compile and run an executable. However, a lot of Linux distributions, e.g., OpenSUSE, mount the /tmp directory with the noexec flag. So it is not allowed to run any executables in /tmp.

This PR generates the temp dir at the current dir rather than in /tmp.

@pakmarkthub
Copy link
Collaborator

Thank you, @JieRen98. However, this does not solve the fundamental issue. On some clusters, the current directory is immutable inside compute nodes (i.e., it is read only). Introducing a variable to Makefile to set the temp directory location might be better.

@JieRen98
Copy link
Author

Thank you, @JieRen98. However, this does not solve the fundamental issue. On some clusters, the current directory is immutable inside compute nodes (i.e., it is read only). Introducing a variable to Makefile to set the temp directory location might be better.

Thanks for your prompt reply. I added several candidates (/tmp, /dev/shm, and pwd). The function checks the existence, writable property, and noexec flag.

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

Successfully merging this pull request may close these issues.

2 participants