Stewart, Robert wrote:
Jeff Flinn wrote:
I've updated ticket#4724 with temp_dir_path2.patch which reflects the discussion in this thread.
POSIX now uses the first environment variable from the list TMPDIR, TMP, TEMP, TEMPDIR. Windows still uses GetTempPath.
Error handling is consistent with the rest of filesystem.
operations_test.hpp tests the existence of the path returned by temp_dir_path, and the ability to create a file, write to it, read from it, and remove it from the directory.
Did you test the various combinations of e-vars being set?
Yes, manually(not within operations_test.cpp) on mac by setting/unsetting the environment vars, not on windows though as I've not been able to make GetTempPath to fail. Not sure if mucking with the environ is acceptable to Beman's tests.
How are you testing the error handling code?
Same as above. Any thoughts on how to proceed? Thanks, Jeff