[Buildroot] [PATCH 1/2] gflags: new package

Thomas Petazzoni thomas.petazzoni at free-electrons.com
Tue Jun 9 07:01:19 UTC 2015


Dear Rahul Bedarkar,

Thanks for this contribution. A few comments below.

On Tue, 9 Jun 2015 11:57:29 +0530, Rahul Bedarkar wrote:

> diff --git a/package/gflags/gflags.hash b/package/gflags/gflags.hash
> new file mode 100644
> index 0000000..9c8b27a
> --- /dev/null
> +++ b/package/gflags/gflags.hash
> @@ -0,0 +1,2 @@
> +# No hash for v2.1.2, comes from the github-helper:
> +none	xxx	gflags-v2.1.2.tar.gz

I believe in this case, we generally do not provide a hash file at all.
Though maybe in the future we will want to have a hash file for *all*
packages. Yann, what is our policy on this?

> +GFLAGS_VERSION = v2.1.2
> +GFLAGS_SITE = $(call github,gflags,gflags,$(GFLAGS_VERSION))
> +GFLAGS_INSTALL_STAGING = YES
> +GFLAGS_LICENSE = BSD-3c
> +GFLAGS_LICENSE_FILES = COPYING.txt
> +
> +ifeq ($(BR2_TOOLCHAIN_HAS_THREADS),n)

This should be:

ifeq ($(BR2_TOOLCHAIN_HAS_THREADS),)

Best regards,

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com


More information about the buildroot mailing list