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

Temp files on Windows #576

Open
gyk opened this issue Jan 9, 2025 · 2 comments
Open

Temp files on Windows #576

gyk opened this issue Jan 9, 2025 · 2 comments
Labels
enhancement New feature or request

Comments

@gyk
Copy link

gyk commented Jan 9, 2025

  • Consider using std::env::temp_dir() for temp files, as it is the standard location. However, ANSI APIs may not play well with some usernames.
  • Consider relaxing the initial privileges for temp files. Currently admin privileges are required for removing some of them, so the program itself cannot clean up the resources after being ended in task manager.
@gyk gyk added the enhancement New feature or request label Jan 9, 2025
@elfenpiff
Copy link
Contributor

@gyk

However, ANSI APIs may not play well with some usernames.

This is why we initially decided against it and used C:\Temp. But you can override the default folder via the iceoryx2 config and choose whatever you want, see https://github.com/eclipse-iceoryx/iceoryx2/tree/main/config

Consider relaxing the initial privileges for temp files.

This makes sense! We will look into it.

@gyk
Copy link
Author

gyk commented Jan 9, 2025

Thanks! Didn't know it is configurable.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants