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.

Sign up for GitHub

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

Jump to bottom

mkstemp unsupported on Windows #342

Closed
eyalroz opened this issue May 27, 2022 · 0 comments
Closed

mkstemp unsupported on Windows #342

eyalroz opened this issue May 27, 2022 · 0 comments
Assignees
@eyalroz
Labels
ms-windows Related to the Microsoft Windows operating system resolved-on-development

Comments

@eyalroz
Copy link
Owner

eyalroz commented May 27, 2022

the modified simpleDrvRuntimePTX CUDA sample currently uses mkstemp() to safely generate a temporary file - but that's only supported on POSIX systems (or BSD 4.3 and later) - not on Windows.

Let's use something like tmpnam() by default - since that is part of the standard library, even if warty.
@eyalroz eyalroz added the ms-windows Related to the Microsoft Windows operating system label May 27, 2022
@eyalroz eyalroz self-assigned this May 27, 2022
eyalroz added a commit that referenced this issue May 27, 2022
@eyalroz
Fixes #342: Using tmpnam() instead of mkstemp() when the latter i…
e695574 
…s not known to be available.
eyalroz added a commit that referenced this issue May 28, 2022
@eyalroz
Fixes #342: Using tmpnam() instead of mkstemp() when the latter i…
c98c7ea 
…s not known to be available.
eyalroz added a commit that referenced this issue May 29, 2022
@eyalroz
Fixes #342: Using tmpnam() instead of mkstemp() when the latter i…
65e3bd5 
…s not known to be available.
eyalroz added a commit that referenced this issue Jun 2, 2022
@eyalroz
Fixes #342: Using tmpnam() instead of mkstemp() when the latter i…
ef66469 
…s not known to be available.
eyalroz added a commit that referenced this issue Jun 20, 2022
@eyalroz
Fixes #342: Using tmpnam() instead of mkstemp() when the latter i…
b428297 
…s not known to be available.
eyalroz added a commit that referenced this issue Jun 20, 2022
@eyalroz
Fixes #342: Using tmpnam() instead of mkstemp() when the latter i…
7a6473c 
…s not known to be available.
eyalroz added a commit that referenced this issue Jun 20, 2022
@eyalroz
Fixes #342: Using tmpnam() instead of mkstemp() when the latter i…
a90b9cd 
…s not known to be available.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@eyalroz eyalroz

Labels
ms-windows Related to the Microsoft Windows operating system resolved-on-development
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@eyalroz