mtd-utils: Add mtd-utils-tests package

GIT PULL In review Martin Lund 3 participants 6 messages Apr 26, 2018 – May 1, 2018

lore

Share & tools

lore

Martin Lund

author

2018-04-26 16:21 UTC #0 lore raw

Add mtd-utils-tests package which includes the test suites mtd-tests, ubi-tests, fs-tests, etc.

These test suites are useful for verifying flash features or stress testing.

The following changes since commit da3625c52e1ab8985fba4fc3d133edf92142f182:

build-appliance-image: Update to sumo head revision (2018-04-23 17:54:32 +0100)

are available in the git repository at:

git://push.yoctoproject.org/poky-contrib malu/mtd-utils-tests

Martin Lund (1):
mtd-utils: Add mtd-utils-tests package

 meta/recipes-devtools/mtd/mtd-utils_git.bb | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

Signature · 1 line

2.14.1

Martin Lund

author

2018-04-26 09:48 UTC #1 lore raw

PATCH 1/1 mtd-utils: Add mtd-utils-tests package

Add mtd-utils-tests package which includes the test suites mtd-tests, ubi-tests, fs-tests, etc.

These test suites are useful for verifying flash features or stress testing.

Signed-off-by Martin Lund <[email protected]>

---
 meta/recipes-devtools/mtd/mtd-utils_git.bb | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

meta/recipes-devtools/mtd/mtd-utils_git.bb +5 -1

index d09d6330228..b8f2dd15c19 100644
@@ -8,6 +8,7 @@ LIC_FILES_CHKSUM = "file://COPYING;md5=0636e73ff0215e8d672dc4c32c317bb3 \
 inherit autotools pkgconfig update-alternatives
 
 DEPENDS = "zlib lzo e2fsprogs util-linux"
+RDEPENDS_mtd-utils-tests += "bash"
 
 PV = "2.0.1+${SRCPV}"

@@ -18,6 +19,8 @@ SRC_URI = "git://git.infradead.org/mtd-utils.git \
 
 S = "${WORKDIR}/git/"
 
+EXTRA_OECONF += "--enable-install-tests"
+
 # xattr support creates an additional compile-time dependency on acl because
 # the sys/acl.h header is needed. libacl is not needed and thus enabling xattr
 # regardless whether acl is enabled or disabled in the distro should be okay.

@@ -37,11 +40,12 @@ do_install () {
 	oe_runmake install DESTDIR=${D} SBINDIR=${sbindir} MANDIR=${mandir} INCLUDEDIR=${includedir}
 }
 
-PACKAGES =+ "mtd-utils-jffs2 mtd-utils-ubifs mtd-utils-misc"
+PACKAGES =+ "mtd-utils-jffs2 mtd-utils-ubifs mtd-utils-misc mtd-utils-tests"
 
 FILES_mtd-utils-jffs2 = "${sbindir}/mkfs.jffs2 ${sbindir}/jffs2dump ${sbindir}/jffs2reader ${sbindir}/sumtool"
 FILES_mtd-utils-ubifs = "${sbindir}/mkfs.ubifs ${sbindir}/ubi*"
 FILES_mtd-utils-misc = "${sbindir}/nftl* ${sbindir}/ftl* ${sbindir}/rfd* ${sbindir}/doc* ${sbindir}/serve_image ${sbindir}/recv_image"
+FILES_mtd-utils-tests = "${libexecdir}/mtd-utils/*"
 
 BBCLASSEXTEND = "native nativesdk"

Signature · 1 line

2.14.1

Khem Raj

2018-04-28 20:01 UTC #2 lore raw

Quoted 5 lines · Martin Lund

Add mtd-utils-tests package which includes the test suites mtd-tests, ubi-tests, fs-tests, etc.
These test suites are useful for verifying flash features or stress testing.

I wonder if this should be enabled when ptest is in distro features

Quoted 60 lines

Signed-off-by Martin Lund <[email protected]>
---
 meta/recipes-devtools/mtd/mtd-utils_git.bb | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)
meta/recipes-devtools/mtd/mtd-utils_git.bb +3 -0
index d09d6330228..b8f2dd15c19 100644
@@ -8,6 +8,7 @@ LIC_FILES_CHKSUM =
 "file://COPYING;md5=0636e73ff0215e8d672dc4c32c317bb3 \
 inherit autotools pkgconfig update-alternatives
 
 DEPENDS = "zlib lzo e2fsprogs util-linux"
+RDEPENDS_mtd-utils-tests += "bash"
 
 PV = "2.0.1+${SRCPV}"
@@ -18,6 +19,8 @@ SRC_URI = "git://git.infradead.org/mtd-utils.git \
 
 S = "${WORKDIR}/git/"
 
+EXTRA_OECONF += "--enable-install-tests"
+
 # xattr support creates an additional compile-time dependency on acl
because
 # the sys/acl.h header is needed. libacl is not needed and thus enabling
xattr
 # regardless whether acl is enabled or disabled in the distro should be
okay.
meta/recipes-devtools/mtd/mtd-utils_git.bb +1 -1
@@ -37,11 +40,12 @@ do_install () {
        oe_runmake install DESTDIR=${D} SBINDIR=${sbindir}
 MANDIR=${mandir} INCLUDEDIR=${includedir}
 }
 
-PACKAGES =+ "mtd-utils-jffs2 mtd-utils-ubifs mtd-utils-misc"
+PACKAGES =+ "mtd-utils-jffs2 mtd-utils-ubifs mtd-utils-misc
mtd-utils-tests"
FILES_mtd-utils-jffs2 = "${sbindir}/mkfs.jffs2 ${sbindir}/jffs2dump
${sbindir}/jffs2reader ${sbindir}/sumtool"
FILES_mtd-utils-ubifs = "${sbindir}/mkfs.ubifs ${sbindir}/ubi*" FILES_mtd-utils-misc = "${sbindir}/nftl* ${sbindir}/ftl* ${sbindir}/rfd*
${sbindir}/doc* ${sbindir}/serve_image ${sbindir}/recv_image" +FILES_mtd-utils-tests = "${libexecdir}/mtd-utils/*"
BBCLASSEXTEND = "native nativesdk"
--
2.14.1
--
_______________________________________________
poky mailing list
[email protected]
https://lists.yoctoproject.org/listinfo/poky

Burton, Ross

2018-04-28 22:28 UTC #3 lore raw

No, as they're not a test suite, just tools to test with.

Ross

Quoted 79 lines · Khem Raj

Quoted 6 lines · Martin Lund
Add mtd-utils-tests package which includes the test suites mtd-tests, ubi-tests, fs-tests, etc.
These test suites are useful for verifying flash features or stress testing.
I wonder if this should be enabled when ptest is in distro features
Quoted 52 lines
Signed-off-by Martin Lund <[email protected]>
---
 meta/recipes-devtools/mtd/mtd-utils_git.bb | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)
meta/recipes-devtools/mtd/mtd-utils_git.bb +3 -0
index d09d6330228..b8f2dd15c19 100644
@@ -8,6 +8,7 @@ LIC_FILES_CHKSUM =
 "file://COPYING;md5=0636e73ff0215e8d672dc4c32c317bb3 \
 inherit autotools pkgconfig update-alternatives
 
 DEPENDS = "zlib lzo e2fsprogs util-linux"
+RDEPENDS_mtd-utils-tests += "bash"
 
 PV = "2.0.1+${SRCPV}"
@@ -18,6 +19,8 @@ SRC_URI = "git://git.infradead.org/mtd-utils.git \
 
 S = "${WORKDIR}/git/"
 
+EXTRA_OECONF += "--enable-install-tests"
+
 # xattr support creates an additional compile-time dependency on acl
because
 # the sys/acl.h header is needed. libacl is not needed and thus enabling
xattr
 # regardless whether acl is enabled or disabled in the distro should be
okay.
meta/recipes-devtools/mtd/mtd-utils_git.bb +1 -1
@@ -37,11 +40,12 @@ do_install () {
        oe_runmake install DESTDIR=${D} SBINDIR=${sbindir}
 MANDIR=${mandir} INCLUDEDIR=${includedir}
 }
 
-PACKAGES =+ "mtd-utils-jffs2 mtd-utils-ubifs mtd-utils-misc"
+PACKAGES =+ "mtd-utils-jffs2 mtd-utils-ubifs mtd-utils-misc
mtd-utils-tests"
FILES_mtd-utils-jffs2 = "${sbindir}/mkfs.jffs2 ${sbindir}/jffs2dump
${sbindir}/jffs2reader ${sbindir}/sumtool"
FILES_mtd-utils-ubifs = "${sbindir}/mkfs.ubifs ${sbindir}/ubi*" FILES_mtd-utils-misc = "${sbindir}/nftl* ${sbindir}/ftl* ${sbindir}/rfd*
${sbindir}/doc* ${sbindir}/serve_image ${sbindir}/recv_image" +FILES_mtd-utils-tests = "${libexecdir}/mtd-utils/*"
BBCLASSEXTEND = "native nativesdk"
Quoted 8 lines
--
2.14.1
--
_______________________________________________
poky mailing list
[email protected]
https://lists.yoctoproject.org/listinfo/poky
--
_______________________________________________
poky mailing list
[email protected]
https://lists.yoctoproject.org/listinfo/poky

Martin Lund

author

2018-05-01 13:00 UTC #4 lore raw

I hope you guys will accept the patch - the tools are quite useful for those of us creating and testing BSPs with NAND/NOR features. It would be nice to have it available to build directly from poky.

________________________________
From: Burton, Ross <[email protected]>
Sent: Sunday, April 29, 2018 12:28:12 AM
To: Khem Raj
Cc: Martin Lund; Poky Project
Subject: Re: [poky] [PATCH 1/1] mtd-utils: Add mtd-utils-tests package

No, as they're not a test suite, just tools to test with.

Ross

Quoted 79 lines · Khem Raj

Quoted 6 lines · Martin Lund
Add mtd-utils-tests package which includes the test suites mtd-tests, ubi-tests, fs-tests, etc.
These test suites are useful for verifying flash features or stress testing.
I wonder if this should be enabled when ptest is in distro features
Quoted 52 lines
Signed-off-by Martin Lund <[email protected]>
---
 meta/recipes-devtools/mtd/mtd-utils_git.bb | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)
meta/recipes-devtools/mtd/mtd-utils_git.bb +3 -0
index d09d6330228..b8f2dd15c19 100644
@@ -8,6 +8,7 @@ LIC_FILES_CHKSUM =
 "file://COPYING;md5=0636e73ff0215e8d672dc4c32c317bb3 \
 inherit autotools pkgconfig update-alternatives
 
 DEPENDS = "zlib lzo e2fsprogs util-linux"
+RDEPENDS_mtd-utils-tests += "bash"
 
 PV = "2.0.1+${SRCPV}"
@@ -18,6 +19,8 @@ SRC_URI = "git://git.infradead.org/mtd-utils.git \
 
 S = "${WORKDIR}/git/"
 
+EXTRA_OECONF += "--enable-install-tests"
+
 # xattr support creates an additional compile-time dependency on acl
because
 # the sys/acl.h header is needed. libacl is not needed and thus enabling
xattr
 # regardless whether acl is enabled or disabled in the distro should be
okay.
meta/recipes-devtools/mtd/mtd-utils_git.bb +1 -1
@@ -37,11 +40,12 @@ do_install () {
        oe_runmake install DESTDIR=${D} SBINDIR=${sbindir}
 MANDIR=${mandir} INCLUDEDIR=${includedir}
 }
 
-PACKAGES =+ "mtd-utils-jffs2 mtd-utils-ubifs mtd-utils-misc"
+PACKAGES =+ "mtd-utils-jffs2 mtd-utils-ubifs mtd-utils-misc
mtd-utils-tests"
FILES_mtd-utils-jffs2 = "${sbindir}/mkfs.jffs2 ${sbindir}/jffs2dump
${sbindir}/jffs2reader ${sbindir}/sumtool"
FILES_mtd-utils-ubifs = "${sbindir}/mkfs.ubifs ${sbindir}/ubi*" FILES_mtd-utils-misc = "${sbindir}/nftl* ${sbindir}/ftl* ${sbindir}/rfd*
${sbindir}/doc* ${sbindir}/serve_image ${sbindir}/recv_image" +FILES_mtd-utils-tests = "${libexecdir}/mtd-utils/*"
BBCLASSEXTEND = "native nativesdk"
Quoted 8 lines
--
2.14.1
--
_______________________________________________
poky mailing list
[email protected]
https://lists.yoctoproject.org/listinfo/poky
--
_______________________________________________
poky mailing list
[email protected]
https://lists.yoctoproject.org/listinfo/poky

Burton, Ross

2018-05-01 13:05 UTC #5 lore raw

It's queued in mut so will be in master soon enough.

Quoted 102 lines · Martin Lund

I hope you guys will accept the patch - the tools are quite useful for those of us creating and testing BSPs with NAND/NOR features. It would be nice to have it available to build directly from poky.
________________________________
From: Burton, Ross <[email protected]>
Sent: Sunday, April 29, 2018 12:28:12 AM
To: Khem Raj
Cc: Martin Lund; Poky Project
Subject: Re: [poky] [PATCH 1/1] mtd-utils: Add mtd-utils-tests package
No, as they're not a test suite, just tools to test with.
Ross
Quoted 66 lines · Khem Raj
Quoted 6 lines · Martin Lund
Add mtd-utils-tests package which includes the test suites mtd-tests, ubi-tests, fs-tests, etc.
These test suites are useful for verifying flash features or stress testing.
I wonder if this should be enabled when ptest is in distro features
Quoted 55 lines
Signed-off-by Martin Lund <[email protected]>
---
 meta/recipes-devtools/mtd/mtd-utils_git.bb | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)
meta/recipes-devtools/mtd/mtd-utils_git.bb +3 -0
index d09d6330228..b8f2dd15c19 100644
@@ -8,6 +8,7 @@ LIC_FILES_CHKSUM =
 "file://COPYING;md5=0636e73ff0215e8d672dc4c32c317bb3 \
 inherit autotools pkgconfig update-alternatives
 
 DEPENDS = "zlib lzo e2fsprogs util-linux"
+RDEPENDS_mtd-utils-tests += "bash"
 
 PV = "2.0.1+${SRCPV}"
@@ -18,6 +19,8 @@ SRC_URI = "git://git.infradead.org/mtd-utils.git \
 
 S = "${WORKDIR}/git/"
 
+EXTRA_OECONF += "--enable-install-tests"
+
 # xattr support creates an additional compile-time dependency on acl
because
 # the sys/acl.h header is needed. libacl is not needed and thus enabling
xattr
 # regardless whether acl is enabled or disabled in the distro should be
okay.
meta/recipes-devtools/mtd/mtd-utils_git.bb +1 -1
@@ -37,11 +40,12 @@ do_install () {
        oe_runmake install DESTDIR=${D} SBINDIR=${sbindir}
 MANDIR=${mandir} INCLUDEDIR=${includedir}
 }
 
-PACKAGES =+ "mtd-utils-jffs2 mtd-utils-ubifs mtd-utils-misc"
+PACKAGES =+ "mtd-utils-jffs2 mtd-utils-ubifs mtd-utils-misc
mtd-utils-tests"
FILES_mtd-utils-jffs2 = "${sbindir}/mkfs.jffs2 ${sbindir}/jffs2dump
${sbindir}/jffs2reader ${sbindir}/sumtool"
FILES_mtd-utils-ubifs = "${sbindir}/mkfs.ubifs ${sbindir}/ubi*" FILES_mtd-utils-misc = "${sbindir}/nftl* ${sbindir}/ftl* ${sbindir}/rfd*
${sbindir}/doc* ${sbindir}/serve_image ${sbindir}/recv_image" +FILES_mtd-utils-tests = "${libexecdir}/mtd-utils/*"
BBCLASSEXTEND = "native nativesdk"
--
2.14.1
Quoted 13 lines
Quoted 5 lines
--
_______________________________________________
poky mailing list
[email protected]
https://lists.yoctoproject.org/listinfo/poky
--
_______________________________________________
poky mailing list
[email protected]
https://lists.yoctoproject.org/listinfo/poky
--
_______________________________________________
poky mailing list
[email protected]
https://lists.yoctoproject.org/listinfo/poky