[wasm] Review temp file name generator

dotnet / runtime

.NET is a cross-platform runtime for cloud, mobile, desktop, and IoT apps.

https://docs.microsoft.com/dotnet/core/

MIT License

15.42k stars 4.75k forks source link

[wasm] Review temp file name generator #81210

Open maraf opened 1 year ago

maraf commented 1 year ago

[ ] As part WASI file system support, consider a better solution for generating temp file names. At the moment we are using posix-like implementation from emscripten. Blocked by support for temp directories: https://github.com/WebAssembly/wasi-libc/issues/229
[x] Remove workaround introduced in https://github.com/dotnet/runtime/pull/81085 after emscripten bump - https://github.com/dotnet/runtime/pull/104109

cc @pavelsavara

ghost commented 1 year ago

Tagging subscribers to 'arch-wasm': @lewing See info in area-owners.md if you want to be subscribed.

Issue Details

As part WASI file system support, consider a better solution for generating temp file names. At the moment we are using posix-like implementation from emscripten. cc @pavelsavara

Author:	maraf
Assignees:	-
Labels:	`arch-wasm`
Milestone:	8.0.0

pavelsavara commented 1 year ago

maraf commented 7 months ago

cc @radekdoulik

ilonatommy commented 4 months ago

Without adding the mentioned methods on wasi, we cannot proceed. Moving to Future