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

Address issues with mingw and win32 wide filenames #523

Merged
merged 4 commits into from
Aug 12, 2019

Conversation

kdt3rd
Copy link
Contributor

@kdt3rd kdt3rd commented Aug 12, 2019

This cleans up the internals of ImfStdIO.cpp to use wide filenames under win32 so as to handle when compiling with mingw, where older versions do not correctly implement the MSVC extension to take a wide filename in to the filename. This does that, and then adds a similar mechanism to the locations in the various test programs that were creating i/o fstreams.

Further, it fixes issues accessing the temp folder for the IlmImfTest and UtilTest programs under win32, and cleans up files.

The default "make test" now passes when cross compiling for windows using mingw from linux, running under wine.

kdt3rd added 4 commits August 12, 2019 23:46
Signed-off-by: Kimball Thurston <kdt3rd@gmail.com>
Signed-off-by: Kimball Thurston <kdt3rd@gmail.com>
… util tests

Signed-off-by: Kimball Thurston <kdt3rd@gmail.com>
…anually creating std::fstreams

Signed-off-by: Kimball Thurston <kdt3rd@gmail.com>
Copy link
Contributor

@lgritz lgritz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@kdt3rd kdt3rd merged commit e0ac10e into AcademySoftwareFoundation:master Aug 12, 2019
@kdt3rd kdt3rd deleted the fix_mingw branch August 12, 2019 19:44
@cary-ilm cary-ilm added this to the v2.4.0 milestone Aug 29, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants