Sorry, it probably wasn't very kind to Richard to split his fixes across two separate branches.
I also appended the gcc6 fix to the lg-pugi branch from the earlier PR so it's easier to try them together. > On Feb 4, 2016, at 11:05 AM, Larry Gritz <[email protected]> wrote: > > Try this: https://github.com/OpenImageIO/oiio/pull/1345 > <https://github.com/OpenImageIO/oiio/pull/1345> > > >> On Feb 4, 2016, at 11:00 AM, Larry Gritz <[email protected] >> <mailto:[email protected]>> wrote: >> >> It's complaining about indentation! >> >> Sheesh. >> >> Must be a GCC 6 new warning. Nothing has changed here and it's never been a >> warning before. >> >> Hang on, I'll prepare a PR that suppresses the error. >> >> >>> On Feb 4, 2016, at 10:45 AM, Richard Shaw <[email protected] >>> <mailto:[email protected]>> wrote: >>> >>> On Thu, Feb 4, 2016 at 11:49 AM, Larry Gritz <[email protected] >>> <mailto:[email protected]>> wrote: >>> Any luck with this? >>> >>> All my local builds have completed but I'm having trouble with the official >>> builds... >>> >>> [ 47%] Building CXX object >>> src/libOpenImageIO/CMakeFiles/OpenImageIO.dir/__/dpx.imageio/libdpx/Writer.cpp.o >>> cd /builddir/build/BUILD/oiio-Release-1.6.10/build/linux/src/libOpenImageIO >>> && /usr/bin/c++ -DBOOST_TEST_DYN_LINK -DEMBED_PLUGINS=1 -DNDEBUG >>> -DOpenImageIO_EXPORTS -DPTEX_EXPORTS -DUSE_BOOST_ASIO=1 >>> -DUSE_EXTERNAL_PUGIXML=1 -DUSE_FIELD3D=1 -DUSE_FREETYPE -DUSE_GIF >>> -DUSE_LIBRAW=1 -DUSE_OCIO=1 -DUSE_OPENCV -DUSE_OPENEXR_VERSION2=1 >>> -DUSE_OPENJPEG -DUSE_OPENSSL=1 -DUSE_PTEX -DUSE_WEBP=1 >>> -D__STDC_CONSTANT_MACROS -D__STDC_LIMIT_MACROS -I/usr/include/OpenEXR >>> -I/usr/include/libraw >>> -I/builddir/build/BUILD/oiio-Release-1.6.10/src/include >>> -I/builddir/build/BUILD/oiio-Release-1.6.10/build/linux/include/OpenImageIO >>> -I/builddir/build/BUILD/oiio-Release-1.6.10/build/linux/src/libOpenImageIO >>> -I/usr/include/openjpeg-1.5 -I/usr/include/freetype2 >>> -I/usr/include/freetype2/freetype2 >>> -I/usr/include/freetype2/freetype2/freetype -O2 -g -pipe -Wall >>> -Werror=format-security -Wp,-D_FORTIFY_SOURCE=2 -fexceptions >>> -fstack-protector-strong --param=ssp-buffer-size=4 -grecord-gcc-switches >>> -specs=/usr/lib/rpm/redhat/redhat-hardened-cc1 -m64 -mtune=generic -O2 -g >>> -DNDEBUG -fPIC -Wall -Werror -fno-math-errno >>> -Wno-error=unused-local-typedefs -Wno-unused-local-typedefs >>> -Wno-unused-result -o >>> CMakeFiles/OpenImageIO.dir/__/dpx.imageio/libdpx/Writer.cpp.o -c >>> /builddir/build/BUILD/oiio-Release-1.6.10/src/dpx.imageio/libdpx/Writer.cpp >>> /builddir/build/BUILD/oiio-Release-1.6.10/src/dpx.imageio/libdpx/Writer.cpp: >>> In member function 'bool dpx::Writer::WriteElement(int, void*, long int)': >>> /builddir/build/BUILD/oiio-Release-1.6.10/src/dpx.imageio/libdpx/Writer.cpp:212:2: >>> error: statement is indented as if it were guarded by... >>> [-Werror=misleading-indentation] >>> this->header.SetDataOffset(element, this->fileLoc); >>> ^~~~ >>> /builddir/build/BUILD/oiio-Release-1.6.10/src/dpx.imageio/libdpx/Writer.cpp:208:5: >>> note: ...this 'if' clause, but it is not >>> if (! this->WritePadData(0x2000)) >>> ^~ >>> /builddir/build/BUILD/oiio-Release-1.6.10/src/dpx.imageio/libdpx/Writer.cpp: >>> In member function 'bool dpx::Writer::WriteElement(int, void*, >>> dpx::DataSize)': >>> /builddir/build/BUILD/oiio-Release-1.6.10/src/dpx.imageio/libdpx/Writer.cpp:251:2: >>> error: statement is indented as if it were guarded by... >>> [-Werror=misleading-indentation] >>> if (element == 0) >>> ^~ >>> /builddir/build/BUILD/oiio-Release-1.6.10/src/dpx.imageio/libdpx/Writer.cpp:247:5: >>> note: ...this 'if' clause, but it is not >>> if (! this->WritePadData(0x2000)) >>> ^~ >>> >>> Is this a GCC 6 change or bug? >>> >>> Thanks, >>> Richard >>> _______________________________________________ >>> Oiio-dev mailing list >>> [email protected] <mailto:[email protected]> >>> http://lists.openimageio.org/listinfo.cgi/oiio-dev-openimageio.org >>> <http://lists.openimageio.org/listinfo.cgi/oiio-dev-openimageio.org> >> >> -- >> Larry Gritz >> [email protected] <mailto:[email protected]> >> >> >> _______________________________________________ >> Oiio-dev mailing list >> [email protected] <mailto:[email protected]> >> http://lists.openimageio.org/listinfo.cgi/oiio-dev-openimageio.org > > -- > Larry Gritz > [email protected] <mailto:[email protected]> > > > _______________________________________________ > Oiio-dev mailing list > [email protected] > http://lists.openimageio.org/listinfo.cgi/oiio-dev-openimageio.org -- Larry Gritz [email protected]
_______________________________________________ Oiio-dev mailing list [email protected] http://lists.openimageio.org/listinfo.cgi/oiio-dev-openimageio.org
