[x265] [PATCH] cmake: restore order of options changed by commit d00f2f5 (important for GCC < 6)
Pradeep Ramachandran
pradeep at multicorewareinc.com
Tue May 9 09:15:17 CEST 2017
On Sat, May 6, 2017 at 3:33 AM, Mateusz Brzostek <mateusz at msystem.waw.pl>
wrote:
> For GCC 5.4, 4.9 and 4.8 option '-std=gnu++11' must be more in front of
> rest of options.
> Sorry that I didn't check this earlier.
>
> 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.
>
> Please review.
>
> # HG changeset patch
> # User Ma0 <mateuszb at poczta.onet.pl>
> # Date 1494020993 -7200
> # Fri May 05 23:49:53 2017 +0200
> # Node ID 773050491507885eb5658be01fe8ee85deec4d54
> # Parent fd01abfc7898283922b34f7fcd9192ddf9c7818a
> cmake: restore order of options changed by commit d00f2f5 (important for
> GCC < 6)
>
Thanks - this is useful. Have pulled this into the stable branch and merged
with default.
>
> diff -r fd01abfc7898 -r 773050491507 source/CMakeLists.txt
> --- a/source/CMakeLists.txt Tue May 02 14:09:48 2017 -0700
> +++ b/source/CMakeLists.txt Fri May 05 23:49:53 2017 +0200
> @@ -183,10 +183,17 @@
> add_definitions(-qinline=level=10 -qpath=IL:/data/video_files/
> latest.tpo/)
> endif()
>
> +# this option is to enable the inclusion of dynamic HDR10 library to the
> libx265 compilation
> +option(ENABLE_DYNAMIC_HDR10 "Enable dynamic HDR10 compilation" OFF)
>
> if(GCC)
> add_definitions(-Wall -Wextra -Wshadow)
> add_definitions(-D__STDC_LIMIT_MACROS=1)
> + if(ENABLE_DYNAMIC_HDR10)
> + add_definitions(-std=gnu++11)
> + else()
> + add_definitions(-std=gnu++98)
> + endif()
> if(ENABLE_PIC)
> add_definitions(-fPIC)
> endif(ENABLE_PIC)
> @@ -362,20 +369,12 @@
> else(HIGH_BIT_DEPTH)
> add_definitions(-DHIGH_BIT_DEPTH=0 -DX265_DEPTH=8)
> endif(HIGH_BIT_DEPTH)
> -# this option is to enable the inclusion of dynamic HDR10 library to the
> libx265 compilation
> -option(ENABLE_DYNAMIC_HDR10 "Enable dynamic HDR10 compilation" OFF)
> +
> if (ENABLE_DYNAMIC_HDR10)
> add_subdirectory(dynamicHDR10)
> include_directories(dynamicHDR10)
> add_definitions(-DENABLE_DYNAMIC_HDR10)
> endif(ENABLE_DYNAMIC_HDR10)
> -if(GCC)
> - if(ENABLE_DYNAMIC_HDR10)
> - add_definitions(-std=gnu++11)
> - else()
> - add_definitions(-std=gnu++98)
> - endif()
> -endif(GCC)
>
> # this option can only be used when linking multiple libx265 libraries
> # together, and some alternate API access method is implemented.
>
>
> _______________________________________________
> x265-devel mailing list
> x265-devel at videolan.org
> https://mailman.videolan.org/listinfo/x265-devel
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mailman.videolan.org/pipermail/x265-devel/attachments/20170509/ecb2ed42/attachment-0001.html>
More information about the x265-devel
mailing list