Repository navigation
fix for /tmp/tmp access issues - #7733
Conversation
|
@ChiCheng45 Many thanks for the PR! Is this now ready for review? |
|
@gmarkall Hi yes this is ready for review now but I'm not sure why this docs are taking so long to be built, has it gotten stuck? |
Many thanks, I've added it to the queue. I didn't see a problem with the doc building - I think sometimes RTD builders wait in a queue for a little while though. |
apmasell
added
5 - Ready to merge
|
BFID |
sklam
added
Pending BuildFarm
|
Checking it in the farm because the use of |
sklam
removed
Pending BuildFarm
sklam
left a comment
sklam
left a comment
There was a problem hiding this comment.
CI passed and all review addressed
|
Would this run with |
|
@OlafvdSpek yea that should also work but I preferred Path(ntf.name).anchor since they might have the tmp dir on another drive on windows |
|
Yeah, was just asking what the effective output_dir would be. Wouldn't this create a new |
|
@OlafvdSpek no it creates a directory in whatever the temp directory is for example, C:\Users\username\AppData\Local\Temp\tmp4kygk7kx. |
7 participants
Changes are made on how the temp files and temp dirs are created by numba for testing the compilers in the function _check_external_compiler in platform.py
This is my proposed fix for the issue #7218 which can occur when multiple users are working on a shared resource such as on a HPC. Basically if one user builds numba AOT code they will create a directory named /tmp/tmp which may mean that other users will unable to build AOT due to a permission error.
fixes #7218