From: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH next v2] cmake: bump version to 3.7.0
Date: Wed, 16 Nov 2016 00:02:32 +0100 [thread overview]
Message-ID: <20161116000232.51cfc4fa@free-electrons.com> (raw)
In-Reply-To: <20161114131541.54948-1-Vincent.Riera@imgtec.com>
Samuel,
Since you are our CMake guy, could you please review/test this?
More specifically, I'm interested in understanding if there are some
potential compatibility issues with our packages that use a CMake-based
build system.
Thanks!
Thomas
On Mon, 14 Nov 2016 13:15:41 +0000, Vicente Olivert Riera wrote:
> libuv is now a required dependency.
>
> Signed-off-by: Vicente Olivert Riera <Vincent.Riera@imgtec.com>
> ---
> Changes v1 -> v2:
> - Do not say that !BR2_STATIC_LIBS dependency is there only because libuv
> ---
> package/cmake/Config.in | 9 +++++++--
> package/cmake/cmake.hash | 4 ++--
> package/cmake/cmake.mk | 6 +++---
> 3 files changed, 12 insertions(+), 7 deletions(-)
>
> diff --git a/package/cmake/Config.in b/package/cmake/Config.in
> index 6f04a4a..716d80e 100644
> --- a/package/cmake/Config.in
> +++ b/package/cmake/Config.in
> @@ -20,11 +20,14 @@ config BR2_PACKAGE_CMAKE_CTEST
> select BR2_PACKAGE_EXPAT
> select BR2_PACKAGE_BZIP2
> select BR2_PACKAGE_XZ
> + select BR2_PACKAGE_LIBUV
> depends on BR2_PACKAGE_CMAKE_ARCH_SUPPORTS
> depends on BR2_TOOLCHAIN_GCC_AT_LEAST_4_7 # from jsoncpp
> depends on BR2_USE_WCHAR # libarchive
> depends on BR2_INSTALL_LIBSTDCPP
> depends on !BR2_STATIC_LIBS
> + depends on BR2_TOOLCHAIN_HAS_THREADS # libuv
> + depends on BR2_USE_MMU # libuv
> help
> CTest is a testing tool distributed as a part of CMake. It
> can be used to automate updating (using CVS for example),
> @@ -34,7 +37,9 @@ config BR2_PACKAGE_CMAKE_CTEST
>
> http://www.cmake.org/
>
> -comment "ctest needs a toolchain w/ C++, wchar, dynamic library, gcc >= 4.7"
> +comment "ctest needs a toolchain w/ C++, wchar, dynamic library, gcc >= 4.7, threads"
> depends on BR2_PACKAGE_CMAKE_ARCH_SUPPORTS
> + depends on BR2_USE_MMU
> depends on !BR2_INSTALL_LIBSTDCPP || !BR2_USE_WCHAR || \
> - BR2_STATIC_LIBS || !BR2_TOOLCHAIN_GCC_AT_LEAST_4_7
> + BR2_STATIC_LIBS || !BR2_TOOLCHAIN_GCC_AT_LEAST_4_7 || \
> + !BR2_TOOLCHAIN_HAS_THREADS
> diff --git a/package/cmake/cmake.hash b/package/cmake/cmake.hash
> index f846dc9..687e5a4 100644
> --- a/package/cmake/cmake.hash
> +++ b/package/cmake/cmake.hash
> @@ -1,2 +1,2 @@
> -# From http://www.cmake.org/files/v3.6/cmake-3.6.3-SHA-256.txt
> -sha256 7d73ee4fae572eb2d7cd3feb48971aea903bb30a20ea5ae8b4da826d8ccad5fe cmake-3.6.3.tar.gz
> +# From http://www.cmake.org/files/v3.7/cmake-3.7.0-SHA-256.txt
> +sha256 ed63e05c41aeb6c036e503114ab15847f29c312f9f21f5f1a7060a4b4ec2fb31 cmake-3.7.0.tar.gz
> diff --git a/package/cmake/cmake.mk b/package/cmake/cmake.mk
> index fadb09e..522d7fe 100644
> --- a/package/cmake/cmake.mk
> +++ b/package/cmake/cmake.mk
> @@ -4,8 +4,8 @@
> #
> ################################################################################
>
> -CMAKE_VERSION_MAJOR = 3.6
> -CMAKE_VERSION = $(CMAKE_VERSION_MAJOR).3
> +CMAKE_VERSION_MAJOR = 3.7
> +CMAKE_VERSION = $(CMAKE_VERSION_MAJOR).0
> CMAKE_SITE = https://cmake.org/files/v$(CMAKE_VERSION_MAJOR)
> CMAKE_LICENSE = BSD-3c
> CMAKE_LICENSE_FILES = Copyright.txt
> @@ -22,7 +22,7 @@ CMAKE_LICENSE_FILES = Copyright.txt
> # the system-wide libraries instead of rebuilding and statically
> # linking with the ones bundled into the CMake sources.
>
> -CMAKE_DEPENDENCIES = zlib jsoncpp libcurl libarchive expat bzip2 xz
> +CMAKE_DEPENDENCIES = zlib jsoncpp libcurl libarchive expat bzip2 xz libuv
>
> CMAKE_CONF_OPTS = \
> -DKWSYS_LFS_WORKS=TRUE \
--
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com
next prev parent reply other threads:[~2016-11-15 23:02 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-11-14 13:15 [Buildroot] [PATCH next v2] cmake: bump version to 3.7.0 Vicente Olivert Riera
2016-11-15 23:02 ` Thomas Petazzoni [this message]
2016-11-20 12:05 ` Samuel Martin
2016-11-25 22:50 ` Thomas Petazzoni
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20161116000232.51cfc4fa@free-electrons.com \
--to=thomas.petazzoni@free-electrons.com \
--cc=buildroot@busybox.net \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.