-
Notifications
You must be signed in to change notification settings - Fork 161
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
Execution cache file is per script folder leading to error when running scripts in parallel #540
Comments
Oddly, it doesn't even work when I use
|
Hi @mungojam The reason it probably won't work even with the That being said, the cache should definitely handle executing two scripts in parallell from the same folder. Looking into it👍 |
I've just spotted #507 which is related. I'm planning on submitting a PR for this one first though, using the filename in the temporary folder path if that makes sense? |
Is it possible to implement an abstraction layer of file system just like what scriptcs did? |
A workaround is to set |
Aha! Neat idea, thanks. I can then put the cache within the build workspace
too so it's easier to find and archive as needed for build diagnostics
…On Mon, 12 Apr 2021, 13:28 Michael Ganss, ***@***.***> wrote:
A workaround is to set DOTNET_SCRIPT_CACHE_LOCATION to a separate
directory for each of the scripts you are running in parallel.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#540 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAYCFS3V2BU6GFQBLDIPPPLTILRORANCNFSM4NPTG3FQ>
.
|
Fixed by #596 |
I'm using dotnet script for some scripts which I run in parallel. Both scripts are in the same folder, but unless I turn off caching, this leads to a clash on the common cache file
script.dll
:The first script to get there works fine, but the 2nd fails because it tries to use the cache at the same time.
It works if I use--no-cache
but I would rather not.The text was updated successfully, but these errors were encountered: