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

Windows: temp file names too short #33321

Closed
StefanKarpinski opened this issue Sep 18, 2019 · 3 comments
Closed

Windows: temp file names too short #33321

StefanKarpinski opened this issue Sep 18, 2019 · 3 comments
Labels
bug Indicates an unexpected problem or unintended behavior ci Continuous integration system:windows Affects only Windows

Comments

@StefanKarpinski
Copy link
Member

From https://build.julialang.org/#/builders/32/builds/4736/steps/2/logs/stdio for example:

Error in testset Test:
Error During Test at C:\cygwin\home\Administrator\buildbot\worker-tabularasa\tester_win64\build\share\julia\stdlib\v1.4\Test\test\runtests.jl:863
  Got exception outside of a @test
  SystemError: opening file "C:\\cygwin\\tmp\\jl_6E.tmp": No such file or directory

With only two characters in the random temp file name, there are going to be a lot of collisions.

@StefanKarpinski StefanKarpinski added bug Indicates an unexpected problem or unintended behavior ci Continuous integration system:windows Affects only Windows labels Sep 18, 2019
@vtjnash
Copy link
Member

vtjnash commented Sep 18, 2019

There's actually 4 characters (65k values), but seems like we should take Jeff's advice? from #9053 (comment)

@StefanKarpinski
Copy link
Member Author

How are there 4 characters? jl_6E.tmp has two variable characters as most.

@JeffBezanson
Copy link
Member

I think this is fixed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Indicates an unexpected problem or unintended behavior ci Continuous integration system:windows Affects only Windows
Projects
None yet
Development

No branches or pull requests

3 participants