[Buildroot] [PATCH v2] dieharder: new package

[Buildroot] [PATCH v2] dieharder: new package

[julien.viarddegalbert at openwide.fr]

From: Julien Viard de Galbert <julien@vdg.name>

Signed-off-by: Julien Viard de Galbert <julien@vdg.name>
---
Changes v1 -> v2
  - fixed typo "bin" instead of "big" (thanks Thomas Petazoni)
  - select gls in config (suggested by Romain Naour)
  - specified "includedir" to fix unsafe header path

Signed-off-by: Julien Viard de Galbert <julien@vdg.name>
---
 package/Config.in              |  1 +
 package/dieharder/Config.in    |  9 +++++++++
 package/dieharder/dieharder.mk | 26 ++++++++++++++++++++++++++
 3 files changed, 36 insertions(+)
 create mode 100644 package/dieharder/Config.in
 create mode 100644 package/dieharder/dieharder.mk

diff --git a/package/Config.in b/package/Config.in
index af4d2b7..cc0bd79 100644
--- a/package/Config.in
+++ b/package/Config.in
@@ -61,6 +61,7 @@ menu "Debugging, profiling and benchmark"
 	source "package/bonnie/Config.in"
 	source "package/cache-calibrator/Config.in"
 	source "package/dhrystone/Config.in"
+	source "package/dieharder/Config.in"
 	source "package/dmalloc/Config.in"
 	source "package/dropwatch/Config.in"
 	source "package/dstat/Config.in"
diff --git a/package/dieharder/Config.in b/package/dieharder/Config.in
new file mode 100644
index 0000000..9f81876
--- /dev/null
+++ b/package/dieharder/Config.in
@@ -0,0 +1,9 @@
+config BR2_PACKAGE_DIEHARDER
+	bool "dieharder"
+	select BR2_PACKAGE_GSL
+	help
+	  dieharder is a fairly involved random number/uniform deviate generator
+	  tester. It is thus suitable for use in testing both software RNG's and
+	  hardware RNG's.
+	  
+	  http://www.phy.duke.edu/~rgb/General/dieharder.php
diff --git a/package/dieharder/dieharder.mk b/package/dieharder/dieharder.mk
new file mode 100644
index 0000000..646a042
--- /dev/null
+++ b/package/dieharder/dieharder.mk
@@ -0,0 +1,26 @@
+################################################################################
+#
+# dieharder
+#
+################################################################################
+
+DIEHARDER_VERSION = 3.31.1
+DIEHARDER_SITE = http://www.phy.duke.edu/~rgb/General/dieharder/
+DIEHARDER_SOURCE = dieharder-$(DIEHARDER_VERSION).tgz
+DIEHARDER_SUBDIR = dieharder-$(DIEHARDER_VERSION)
+DIEHARDER_LICENSE = GPLv2b
+DIEHARDER_LICENSE_FILES = $(DIEHARDER_SUBDIR)/COPYING
+DIEHARDER_DEPENDENCIES = gsl
+
+# fix endiannes detection
+ifeq ($(BR2_ENDIAN),"BIG")
+BR2_AC_CV_C_ENDIAN = ac_cv_c_endian=big
+else
+BR2_AC_CV_C_ENDIAN = ac_cv_c_endian=little
+endif
+DIEHARDER_CONF_OPTS = $(BR2_AC_CV_C_ENDIAN) --includedir=$(STAGING_DIR)/usr/include
+
+# parallel build fail, disable it
+DIEHARDER_MAKE=$(MAKE1)
+
+$(eval $(autotools-package))
-- 
2.1.4

[Buildroot] [PATCH v2] dieharder: new package

[Baruch Siach]

Hi Julien,

On Sat, May 16, 2015 at 02:20:02PM +0200, julien.viarddegalbert at openwide.fr wrote:
> +# fix endiannes detection
> +ifeq ($(BR2_ENDIAN),"BIG")
> +BR2_AC_CV_C_ENDIAN = ac_cv_c_endian=big
> +else
> +BR2_AC_CV_C_ENDIAN = ac_cv_c_endian=little
> +endif
> +DIEHARDER_CONF_OPTS = $(BR2_AC_CV_C_ENDIAN) --includedir=$(STAGING_DIR)/usr/include

The make variables namespace is global. Variables with the BR2_ prefix are 
reserved for Buildroot use. You should not use the BR2_ prefix for package 
specific variables. But you don't need this helper variable here, anyway:

DIEHARDER_CONF_OPTS = --includedir=$(STAGING_DIR)/usr/include
ifeq ($(BR2_ENDIAN),"BIG")
DIEHARDER_CONF_OPTS += ac_cv_c_endian=big
else
DIEHARDER_CONF_OPTS += ac_cv_c_endian=little
endif

baruch

-- 
     http://baruch.siach.name/blog/                  ~. .~   Tk Open Systems
=}------------------------------------------------ooO--U--Ooo------------{=
   - baruch at tkos.co.il - tel: +972.2.679.5364, http://www.tkos.co.il -