39e6822a01c823b74a5b1b9e75c9737a1f79049e introduced this to avoid
warnings from third party code. But it broke the build with gcc6
because fftw and exiv2 return the default include directory via the
respective *_INCLUDE_DIRs on my system. There's an gcc bug report
closed as "WONTFIX" about this issue [1], which recommends not to
use -isystem with default include directories.
Details
Details
- Reviewers
rempt - Group Reviewers
Krita - Commits
- R37:4754a711af4f: Don't include fftw's and exiv2's headers with -isystem
Builds fine with gcc6.
Diff Detail
Diff Detail
- Repository
- R37 Krita
- Lint
Automatic diff as part of commit; lint not applicable. - Unit
Automatic diff as part of commit; unit tests not applicable.
Comment Actions
Works on Linux for me, but I haven't tested with mingw or clang yet. I doubt it'll make a difference, though, since it worked before the 39e6822a01c823b74a5b1b9e75c9737a1f79049e patch on those platforms.