<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Sat, May 6, 2017 at 3:33 AM, Mateusz Brzostek <span dir="ltr"><<a href="mailto:mateusz@msystem.waw.pl" target="_blank">mateusz@msystem.waw.pl</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">For GCC 5.4, 4.9 and 4.8 option '-std=gnu++11' must be more in front of rest of options.<br>
Sorry that I didn't check this earlier.<br>
<br>
I propose to move 'option(ENABLE_DYNAMIC_HDR10 ...' before we check if ENABLE_DYNAMIC_HDR10 is ON -- now it is not important, but in the future it might be ON by default, so we will be ready.<br>
<br>
Please review.<br>
<br>
# HG changeset patch<br>
# User Ma0 <<a href="mailto:mateuszb@poczta.onet.pl">mateuszb@poczta.onet.pl</a>><br>
# Date 1494020993 -7200<br>
#      Fri May 05 23:49:53 2017 +0200<br>
# Node ID 773050491507885eb5658be01fe8ee<wbr>85deec4d54<br>
# Parent  fd01abfc7898283922b34f7fcd9192<wbr>ddf9c7818a<br>
cmake: restore order of options changed by commit d00f2f5 (important for GCC < 6)<br></blockquote><div><br></div><div>Thanks - this is useful. Have pulled this into the stable branch and merged with default.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
diff -r fd01abfc7898 -r 773050491507 source/CMakeLists.txt<br>
--- a/source/CMakeLists.txt    Tue May 02 14:09:48 2017 -0700<br>
+++ b/source/CMakeLists.txt    Fri May 05 23:49:53 2017 +0200<br>
@@ -183,10 +183,17 @@<br>
     add_definitions(-qinline=<wbr>level=10 -qpath=IL:/data/video_files/<wbr>latest.tpo/)<br>
 endif()<br>
<br>
+# this option is to enable the inclusion of dynamic HDR10 library to the libx265 compilation<br>
+option(ENABLE_DYNAMIC_HDR10 "Enable dynamic HDR10 compilation" OFF)<br>
<br>
 if(GCC)<br>
     add_definitions(-Wall -Wextra -Wshadow)<br>
     add_definitions(-D__STDC_<wbr>LIMIT_MACROS=1)<br>
+    if(ENABLE_DYNAMIC_HDR10)<br>
+        add_definitions(-std=gnu++11)<br>
+    else()<br>
+        add_definitions(-std=gnu++98)<br>
+    endif()<br>
     if(ENABLE_PIC)<br>
          add_definitions(-fPIC)<br>
     endif(ENABLE_PIC)<br>
@@ -362,20 +369,12 @@<br>
 else(HIGH_BIT_DEPTH)<br>
     add_definitions(-DHIGH_BIT_<wbr>DEPTH=0 -DX265_DEPTH=8)<br>
 endif(HIGH_BIT_DEPTH)<br>
-# this option is to enable the inclusion of dynamic HDR10 library to the libx265 compilation<br>
-option(ENABLE_DYNAMIC_HDR10 "Enable dynamic HDR10 compilation" OFF)<br>
+<br>
 if (ENABLE_DYNAMIC_HDR10)<br>
     add_subdirectory(dynamicHDR10)<br>
     include_directories(<wbr>dynamicHDR10)<br>
     add_definitions(-DENABLE_<wbr>DYNAMIC_HDR10)<br>
 endif(ENABLE_DYNAMIC_HDR10)<br>
-if(GCC)<br>
-    if(ENABLE_DYNAMIC_HDR10)<br>
-        add_definitions(-std=gnu++11)<br>
-    else()<br>
-        add_definitions(-std=gnu++98)<br>
-    endif()<br>
-endif(GCC)<br>
<br>
 # this option can only be used when linking multiple libx265 libraries<br>
 # together, and some alternate API access method is implemented.<br>
<br>
<br>______________________________<wbr>_________________<br>
x265-devel mailing list<br>
<a href="mailto:x265-devel@videolan.org">x265-devel@videolan.org</a><br>
<a href="https://mailman.videolan.org/listinfo/x265-devel" rel="noreferrer" target="_blank">https://mailman.videolan.org/<wbr>listinfo/x265-devel</a><br>
<br></blockquote></div><br></div></div>