Vulkan icon indicating copy to clipboard operation
Vulkan copied to clipboard

New validate_all.py script not portable to linux or macOS

Open SRSaunders opened this issue 6 months ago • 1 comments

The new batch script is great, but currently works only on Windows.

Would it be possible to add support for linux & macOS? Perhaps something like:

...
import platform

if os.path.exists("validation_output.txt"):
  os.remove("validation_output.txt")
if platform.system() == 'Linux' or platform.system() == 'Darwin':
    EXE_WILDCARD = "./*"
else:
    EXE_WILDCARD = "*.exe"
for sample in glob.glob(EXE_WILDCARD):
...

SRSaunders avatar Jun 20 '25 14:06 SRSaunders

Maybe at some point. Right now it's mostly meant for (my) personal) use.

SaschaWillems avatar Jun 20 '25 14:06 SaschaWillems

The script should work with non-windows platforms. I successfully tested on Linux.

Feel free to let me know if it needs further adjustments.

SaschaWillems avatar Jul 03 '25 06:07 SaschaWillems

Thanks @SaschaWillems for adding this.

SRSaunders avatar Jul 03 '25 14:07 SRSaunders