Skip to content

Commit 4ada9fd

Browse files
committed
clean up CMakeLists.txt and Dockerfile
1 parent f76edbd commit 4ada9fd

File tree

2 files changed

+13
-9
lines changed

2 files changed

+13
-9
lines changed

Dockerfile

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,7 @@
1-
FROM ubuntu:noble
1+
#in noble, apt-get installs OpenEXR v3
2+
FROM ubuntu:noble
3+
#in jammy, apt-get installs OpenEXR v2
4+
#FROM ubuntu:jammy
25

36
RUN apt-get update
47

@@ -13,7 +16,8 @@ RUN apt-get -y install valgrind
1316

1417
# install optional openjph dependencies
1518
RUN apt-get -y install libtiff-dev
16-
#RUN apt-get -y install libilmbase-dev
19+
#libilmbase-dev is needed or OpenEXR v2 but not OpenEXR v3
20+
#RUN apt-get -y install libilmbase-dev
1721
RUN apt-get -y install libopenexr-dev
1822

1923

src/apps/CMakeLists.txt

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -9,13 +9,13 @@ if( OJPH_ENABLE_TIFF_SUPPORT )
99
set(USE_TIFF TRUE CACHE BOOL "Add TIFF support")
1010
include_directories( ${TIFF_INCLUDE_DIR} )
1111
add_definitions(-DOJPH_ENABLE_TIFF_SUPPORT)
12-
elseif(MSVC)
12+
elseif(TIFF_FOUND)
1313
message(STATUS "TIFF support has been enabled but no path to the TIFF library "
1414
"has been specified; please configure with -DCMAKE_PREFIX_PATH=<TIFF library directory>, "
1515
"or disable TIFF support using -DOJPH_ENABLE_TIFF_SUPPORT=OFF.")
1616
endif( TIFF_FOUND )
1717

18-
endif()
18+
endif(OJPH_ENABLE_TIFF_SUPPORT)
1919
############################################################
2020

2121
# Add OPENEXR library
@@ -57,7 +57,7 @@ if( OJPH_ENABLE_OPENEXR_SUPPORT )
5757
message(STATUS "Found OpenEXR ${OpenEXR_VERSION}")
5858
set(USE_OPENEXR TRUE CACHE BOOL "Add OpenEXR support")
5959
add_definitions(-DOJPH_ENABLE_OPENEXR_SUPPORT)
60-
else()
60+
else(OpenEXR_FOUND)
6161
find_package(IlmBase)
6262
if(IlmBase_FOUND)
6363
message(STATUS "Found IlmBase ${IlmBase_VERSION}")
@@ -67,13 +67,13 @@ if( OJPH_ENABLE_OPENEXR_SUPPORT )
6767
message(STATUS "Found OpenEXR ${OpenEXR_VERSION}")
6868
set(USE_OPENEXR TRUE CACHE BOOL "Add OpenEXR support")
6969
add_definitions(-DOJPH_ENABLE_OPENEXR_SUPPORT)
70-
endif()
70+
endif(OpenEXR_FOUND)
7171

72-
endif()
72+
endif(IlmBase_FOUND)
7373

74-
endif()
74+
endif(OpenEXR_FOUND)
7575

76-
endif()
76+
endif(OJPH_ENABLE_OPENEXR_SUPPORT)
7777
############################################################
7878

7979
## Build executables

0 commit comments

Comments
 (0)