From 5fae2503bf7d81b89f68f4c494f4c0e4c9eb5f46 Mon Sep 17 00:00:00 2001 From: Sergey Sharybin Date: Tue, 12 Jul 2016 17:38:26 +0200 Subject: Revert "OpenJPEG: update to 2.1 from 1.5" This reverts commit f12204196fb1ee985ab9745cf9c70877601145f9. Campbell, sorry. have to revert this for the time being. We've missed some very important bits, such as: - FFmpeg is usually linked against OpenJPEG - OIIO needs OpenJPEG as well. For FFmpeg issues we can either disable OpenJPEG there (since we don't really use it), or bump FFmpeg to version 3.1.1 which can use either of OpenJPEG 1.5 or 2.1. For OIIO we do need OpenJPEG support (otherwise Cycles will not be able to use j2k/j2c textures) and currently there is NO solution to make OIIO working with OpenJPEG 2.1. According to Matthias Fauconneau (aka mfv) Larry is working on the patch to get OIIO work with OpenJPEG 2.1, but it'll take some time still. I've tried to look into support of some sort of build system flag and do ifdefs, but it all becomes quite nasty, especially with bundled OpenJPEG bumped to 2.1. Surely such an update is something we'll have to apply to but at this exact moment it causes quite some pain for all developers. Suggest to wait for until OIIO supports OpenJPEG 2.1 and then go with the updates for real. --- build_files/cmake/Modules/FindOpenJPEG.cmake | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) (limited to 'build_files') diff --git a/build_files/cmake/Modules/FindOpenJPEG.cmake b/build_files/cmake/Modules/FindOpenJPEG.cmake index 1f825914a3c..d765103638b 100644 --- a/build_files/cmake/Modules/FindOpenJPEG.cmake +++ b/build_files/cmake/Modules/FindOpenJPEG.cmake @@ -41,14 +41,13 @@ FIND_PATH(OPENJPEG_INCLUDE_DIR HINTS ${_openjpeg_SEARCH_DIRS} PATH_SUFFIXES - openjpeg-2.1 - include/openjpeg-2.1 include + include/openjpeg-1.5 ) FIND_LIBRARY(OPENJPEG_LIBRARY NAMES - openjp2 + openjpeg HINTS ${_openjpeg_SEARCH_DIRS} PATH_SUFFIXES -- cgit v1.2.3