-
Notifications
You must be signed in to change notification settings - Fork 107
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
/tmp/.crank/wrk shared by all crank users #665
Comments
Here are some more leftovers in /tmp that will likely not have permissions for the next person that comes along: drwxrwxr-x 3 wnj wnj 4096 Nov 29 15:30 /tmp/VBCSCompiler |
Does it mean where crank uses the "temporary folder" it should use the current user's temporary folder? |
If there is a concept on linux of 'current user's temporary folder', then that makes sense. Otherwise just manufacture one with the userid in the directory name somewhere. I don't know which tool makes the /tmp/VBCSCompiler, and if crank has any control over that. the vscode-typescript directory looks OK, since '1017' is the userid of user 'wnj'. |
Maybe "current user's home folder" is better for this issue? |
When using the wrk load generator, crank mkdir /tmp/.crank and then writes an executable by get from some URL.
Unfortunately, the permissions of the last user for /tmp/.crank may preclude the current user from copying in a new version of wrk.
/tmp/.crank directory name should also encode the user's name
@sebastienros
@EgorBo
The text was updated successfully, but these errors were encountered: