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

Recently added ASTC support causing MinGW build issues #16

Closed
Crunkle opened this issue Jul 30, 2018 · 1 comment
Closed

Recently added ASTC support causing MinGW build issues #16

Crunkle opened this issue Jul 30, 2018 · 1 comment

Comments

@Crunkle
Copy link

Crunkle commented Jul 30, 2018

Whilst attempting to build bgfx using mingw-gcc-release64 and, as a result, the associated bimg_encode dependency, an error occurs due to the recent commit 03ad392 adding new third-party sources.

Example error

In file included from ../../../../bimg/3rdparty/astc/astc_codec_internals.h:22:0,
from ../../../../bimg/3rdparty/astc/astc_averages_and_directions.cpp:17:
../../../../bimg/3rdparty/astc/mathlib.h:30:7: warning: "_MSC_VER" is not defined [-Wundef]
#if (!_MSC_VER) && (__cplusplus < 201103L)
^~~~~~~~
In file included from /usr/lib/gcc/x86_64-w64-mingw32/6.3-win32/include/c++/cmath:45:0,
from /usr/lib/gcc/x86_64-w64-mingw32/6.3-win32/include/c++/math.h:36,
from ../../../../bimg/3rdparty/astc/astc_averages_and_directions.cpp:19:
/usr/share/mingw-w64/include/math.h: In function ‘int __isnanl(long double)’:
/usr/share/mingw-w64/include/math.h:573:65: error: lvalue required as left operand of assignment
xx = (int) (ld.ldt->lh.low | (ld.ldt->lh.high & 0x7fffffffu)); /* explicit */
^

Unsure whether this is a bug in MinGW or improper use of the headers - it is preventing the build either way and the only non-intrusive solution is to rollback to an older version.

@cloudwu
Copy link
Contributor

cloudwu commented Aug 1, 2018

I fixed this issue, see #17

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

No branches or pull requests

3 participants