Skip to contentSkip to conversation

package/cunit: new package

PATCH New Matt Weber 3 participants 5 messages Nov 1, 2018 – Nov 29, 2018

lore
Share & tools
lore

Matt Weber

author
#0 lore raw

PATCH 1/2 package/cunit: new package

---
 DEVELOPERS               |  1 +
 package/Config.in        |  1 +
 package/cunit/Config.in  |  6 ++++++
 package/cunit/cunit.hash |  3 +++
 package/cunit/cunit.mk   | 17 +++++++++++++++++
 5 files changed, 28 insertions(+)
 create mode 100644 package/cunit/Config.in
 create mode 100644 package/cunit/cunit.hash
 create mode 100644 package/cunit/cunit.mk
DEVELOPERS +1 -0
index a3d97eb..d91a093 100644
@@ -1348,6 +1348,7 @@ F:	package/checkpolicy/
 F:	package/checksec/
 F:	package/cgroupfs-mount/
 F:	package/crda/
+F:	package/cunit/
 F:	package/devmem2/
 F:	package/dnsmasq/
 F:	package/dosfstools/
package/Config.in +1 -0
index 7e8224c..c27253c 100644
@@ -141,6 +141,7 @@ menu "Development tools"
 	source "package/check/Config.in"
 	source "package/cmake/Config.in"
 	source "package/cppunit/Config.in"
+	source "package/cunit/Config.in"
 	source "package/cvs/Config.in"
 	source "package/cxxtest/Config.in"
 	source "package/diffutils/Config.in"
package/cunit/Config.in new file +6 -0
new file mode 100644
index 0000000..8bcfa2a
@@ -0,0 +1,6 @@
+config BR2_PACKAGE_CUNIT
+	bool "cunit"
+	help
+	  An automated testing framework in 'C'.
+
+	  http://cunit.sourceforge.net/
package/cunit/cunit.hash new file +3 -0
new file mode 100644
index 0000000..bbd43a7
@@ -0,0 +1,3 @@
+# Locally calculated:
+sha256 f5b29137f845bb08b77ec60584fdb728b4e58f1023e6f249a464efa49a40f214  CUnit-2.1-3.tar.bz2
+sha256 5d9d73d41a57dd2f34487ef3978a2c13cdb97294baeeb81fcd274796399eb15f  COPYING
package/cunit/cunit.mk new file +17 -0
new file mode 100644
index 0000000..df300fd
@@ -0,0 +1,17 @@
+################################################################################
+#
+# cunit
+#
+################################################################################
+
+CUNIT_VERSION = 2.1-3
+CUNIT_SITE = http://downloads.sourceforge.net/project/cunit/CUnit/$(CUNIT_VERSION)
+CUNIT_SOURCE = CUnit-$(CUNIT_VERSION).tar.bz2
+CUNIT_INSTALL_STAGING = YES
+CUNIT_LICENSE = LGPL-2.0
+CUNIT_LICENSE_FILES = COPYING
+
+# The source archive does not have the autoconf/automake material generated.
+CUNIT_AUTORECONF = YES
+
+$(eval $(autotools-package))
Signature · 1 line
1.9.1
author
#1 lore raw

PATCH 2/2 package/libdrm: correct cunit test dependency

Previously the option to install tests would result in the test cases that don't have a cunit dependency, to build and be installed.

This patch adds the cunit dependency so all test cases are built and installed to target when that is selected via kconfig.

---
 package/libdrm/Config.in | 1 +
 package/libdrm/libdrm.mk | 3 +++
 2 files changed, 4 insertions(+)
package/libdrm/Config.in +1 -0
index 581f921..9dd1fb7 100644
@@ -99,6 +99,7 @@ config BR2_PACKAGE_LIBDRM_VC4
 
 config BR2_PACKAGE_LIBDRM_INSTALL_TESTS
 	bool "Install test programs"
+	select BR2_PACKAGE_CUNIT
 	help
 	  This option allows to install the libdrm test programs.
 
package/libdrm/libdrm.mk +3 -0
index 18ef9cd..e927dd1 100644
@@ -115,6 +115,9 @@ endif
 
 ifeq ($(BR2_PACKAGE_LIBDRM_INSTALL_TESTS),y)
 LIBDRM_CONF_OPTS += --enable-install-test-programs
+# By default most of the test cases with automatically
+# be disabled if cunit is not present
+LIBDRM_DEPENDENCIES += cunit
 endif
 
 $(eval $(autotools-package))
Signature · 1 line
1.9.1
#2 lore raw

PATCH 2/2 package/libdrm: correct cunit test dependency

Hello,

Quoted 5 lines · Matt Weber

Previously the option to install tests would result in the test cases that don't have a cunit dependency, to build and be installed.

This patch adds the cunit dependency so all test cases are built and installed to target when that is selected via kconfig.

Looking at the libdrm build system, only the tests specific to the amdgpu backend/driver need cunit. So when you say "By default most of the test cases will automatically be disabled if cunit is not present" doesn't seem really accurate. So I've changed this patch to make cunit an optional dependency: if it's there, we use it, otherwise, fine, we have some tests, but not the amdgpu tests.

Did I miss something?

Anyway, I've applied the patch that adds it as an optional dependency to the next branch.

Best regards,

Thomas

Signature · 3 lines
Thomas Petazzoni, CTO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com
#3 lore raw

PATCH 2/2 package/libdrm: correct cunit test dependency

Thomas,

Quoted 10 lines · Thomas Petazzoni

Hello,

Quoted 5 lines · Matt Weber

Previously the option to install tests would result in the test cases that don't have a cunit dependency, to build and be installed.

This patch adds the cunit dependency so all test cases are built and installed to target when that is selected via kconfig.

Quoted 8 lines

Looking at the libdrm build system, only the tests specific to the amdgpu backend/driver need cunit. So when you say "By default most of the test cases will automatically be disabled if cunit is not present" doesn't seem really accurate. So I've changed this patch to make cunit an optional dependency: if it's there, we use it, otherwise, fine, we have some tests, but not the amdgpu tests.

Did I miss something?

No, that should be ok.

#4 lore raw

PATCH 1/2 package/cunit: new package

In reply to Matt Weber

Hello,

package/cunit/cunit.mk line 11

Quoted 5 lines · Matt Weber
package/cunit/cunit.mk new file +5 -0
+CUNIT_VERSION = 2.1-3
+CUNIT_SITE = http://downloads.sourceforge.net/project/cunit/CUnit/$(CUNIT_VERSION)
+CUNIT_SOURCE = CUnit-$(CUNIT_VERSION).tar.bz2
+CUNIT_INSTALL_STAGING = YES
+CUNIT_LICENSE = LGPL-2.0

Actually the license is LGPL-2.0+.

I've applied to next after fixing this. Thanks!

Thomas

Signature · 3 lines
Thomas Petazzoni, CTO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com