diff mbox

ltp: update to new release 20130904

Message ID 1378586028-6613-1-git-send-email-anders.roxell@linaro.org
State Accepted
Commit 5d6ef973fe101bb3547a14452ded79536d9d2582
Headers show

Commit Message

Anders Roxell Sept. 7, 2013, 8:33 p.m. UTC
Signed-off-by: Anders Roxell <anders.roxell@linaro.org>
---
 .../ltp/{ltp_20130503.bb => ltp_20130904.bb}                | 13 ++++++++-----
 1 file changed, 8 insertions(+), 5 deletions(-)
 rename meta/recipes-extended/ltp/{ltp_20130503.bb => ltp_20130904.bb} (93%)

Comments

Saul Wold Sept. 7, 2013, 8:46 p.m. UTC | #1
On 09/07/2013 01:33 PM, Anders Roxell wrote:
> Signed-off-by: Anders Roxell <anders.roxell@linaro.org>
> ---
>   .../ltp/{ltp_20130503.bb => ltp_20130904.bb}                | 13 ++++++++-----
>   1 file changed, 8 insertions(+), 5 deletions(-)
>   rename meta/recipes-extended/ltp/{ltp_20130503.bb => ltp_20130904.bb} (93%)
>
> diff --git a/meta/recipes-extended/ltp/ltp_20130503.bb b/meta/recipes-extended/ltp/ltp_20130904.bb
> similarity index 93%
> rename from meta/recipes-extended/ltp/ltp_20130503.bb
> rename to meta/recipes-extended/ltp/ltp_20130904.bb
> index 387826d..4f2b9e7 100644
> --- a/meta/recipes-extended/ltp/ltp_20130503.bb
> +++ b/meta/recipes-extended/ltp/ltp_20130904.bb
> @@ -21,17 +21,20 @@ LIC_FILES_CHKSUM = "file://COPYING;md5=0636e73ff0215e8d672dc4c32c317bb3 \
>
>   DEPENDS = "attr libaio libcap acl openssl"
>
> -SRC_URI = "${SOURCEFORGE_MIRROR}/ltp/ltp-full-${PV}.bz2"
> +SRC_URI = "git://github.com/linux-test-project/ltp.git"
>
> -SRC_URI[md5sum] = "d448d9e9731d7ef45352fc74633cc97f"
> -SRC_URI[sha256sum] = "afdb1479e73d4da7f0d4d5d3fe1570bc5fc96e3317d4a5c10c59c046d3dfa4a0"
> +SRC_URI[md5sum] = "c8a5f6f397d2c021a0d21ad28992aa7d"
> +SRC_URI[sha256sum] = "4c13e00c1bed8f1b85daf934cd7b54a08b7b7e31874085ab9277ee563af87139"
> +
> +SRCREV = "${PV}"

Is the PV really a git hash?  That's a branch or tag which will also 
require and extra lookup.  It needs to be a git hash.

Also as noted in the last email the SRC_URI checksums are not needed as 
you are using git now.

Sau!

> +S = "${WORKDIR}/git"
>
>   export prefix = "/opt/ltp"
>   export exec_prefix = "/opt/ltp"
>
>   inherit autotools
>
> -RDEPENDS_${PN} = "perl"
> +RDEPENDS_${PN} = "perl e2fsprogs-mke2fs"
>
>   FILES_${PN}-dbg += "/opt/ltp/runtest/.debug"
>   FILES_${PN}-dbg += "/opt/ltp/testcases/bin/.debug"
> @@ -62,7 +65,7 @@ do_extract_tarball() {
>   do_install(){
>   	install -d ${D}/opt/ltp/
>   	oe_runmake DESTDIR=${D} SKIP_IDCHECK=1 install
> -	
> +
>   	# Copy POSIX test suite into ${D}/opt/ltp/testcases by manual
>   	cp -r testcases/open_posix_testsuite ${D}/opt/ltp/testcases
>
>
Anders Roxell Sept. 9, 2013, 10:14 a.m. UTC | #2
On 2013-09-07 13:46, Saul Wold wrote:
> On 09/07/2013 01:33 PM, Anders Roxell wrote:
> >Signed-off-by: Anders Roxell <anders.roxell@linaro.org>
> >---
> >  .../ltp/{ltp_20130503.bb => ltp_20130904.bb}                | 13 ++++++++-----
> >  1 file changed, 8 insertions(+), 5 deletions(-)
> >  rename meta/recipes-extended/ltp/{ltp_20130503.bb => ltp_20130904.bb} (93%)
> >
> >diff --git a/meta/recipes-extended/ltp/ltp_20130503.bb b/meta/recipes-extended/ltp/ltp_20130904.bb
> >similarity index 93%
> >rename from meta/recipes-extended/ltp/ltp_20130503.bb
> >rename to meta/recipes-extended/ltp/ltp_20130904.bb
> >index 387826d..4f2b9e7 100644
> >--- a/meta/recipes-extended/ltp/ltp_20130503.bb
> >+++ b/meta/recipes-extended/ltp/ltp_20130904.bb
> >@@ -21,17 +21,20 @@ LIC_FILES_CHKSUM = "file://COPYING;md5=0636e73ff0215e8d672dc4c32c317bb3 \
> >
> >  DEPENDS = "attr libaio libcap acl openssl"
> >
> >-SRC_URI = "${SOURCEFORGE_MIRROR}/ltp/ltp-full-${PV}.bz2"
> >+SRC_URI = "git://github.com/linux-test-project/ltp.git"
> >
> >-SRC_URI[md5sum] = "d448d9e9731d7ef45352fc74633cc97f"
> >-SRC_URI[sha256sum] = "afdb1479e73d4da7f0d4d5d3fe1570bc5fc96e3317d4a5c10c59c046d3dfa4a0"
> >+SRC_URI[md5sum] = "c8a5f6f397d2c021a0d21ad28992aa7d"
> >+SRC_URI[sha256sum] = "4c13e00c1bed8f1b85daf934cd7b54a08b7b7e31874085ab9277ee563af87139"
> >+
> >+SRCREV = "${PV}"
> 
> Is the PV really a git hash?  That's a branch or tag which will also
> require and extra lookup.  It needs to be a git hash.
Thanks, didn't know.

> 
> Also as noted in the last email the SRC_URI checksums are not needed
> as you are using git now.
My mistake sorry.

Sending a new patch soon.

Anders
> 
> Sau!
> 
> >+S = "${WORKDIR}/git"
> >
> >  export prefix = "/opt/ltp"
> >  export exec_prefix = "/opt/ltp"
> >
> >  inherit autotools
> >
> >-RDEPENDS_${PN} = "perl"
> >+RDEPENDS_${PN} = "perl e2fsprogs-mke2fs"
> >
> >  FILES_${PN}-dbg += "/opt/ltp/runtest/.debug"
> >  FILES_${PN}-dbg += "/opt/ltp/testcases/bin/.debug"
> >@@ -62,7 +65,7 @@ do_extract_tarball() {
> >  do_install(){
> >  	install -d ${D}/opt/ltp/
> >  	oe_runmake DESTDIR=${D} SKIP_IDCHECK=1 install
> >-	
> >+
> >  	# Copy POSIX test suite into ${D}/opt/ltp/testcases by manual
> >  	cp -r testcases/open_posix_testsuite ${D}/opt/ltp/testcases
> >
> >
diff mbox

Patch

diff --git a/meta/recipes-extended/ltp/ltp_20130503.bb b/meta/recipes-extended/ltp/ltp_20130904.bb
similarity index 93%
rename from meta/recipes-extended/ltp/ltp_20130503.bb
rename to meta/recipes-extended/ltp/ltp_20130904.bb
index 387826d..4f2b9e7 100644
--- a/meta/recipes-extended/ltp/ltp_20130503.bb
+++ b/meta/recipes-extended/ltp/ltp_20130904.bb
@@ -21,17 +21,20 @@  LIC_FILES_CHKSUM = "file://COPYING;md5=0636e73ff0215e8d672dc4c32c317bb3 \
 
 DEPENDS = "attr libaio libcap acl openssl"
 
-SRC_URI = "${SOURCEFORGE_MIRROR}/ltp/ltp-full-${PV}.bz2"
+SRC_URI = "git://github.com/linux-test-project/ltp.git"
 
-SRC_URI[md5sum] = "d448d9e9731d7ef45352fc74633cc97f"
-SRC_URI[sha256sum] = "afdb1479e73d4da7f0d4d5d3fe1570bc5fc96e3317d4a5c10c59c046d3dfa4a0"
+SRC_URI[md5sum] = "c8a5f6f397d2c021a0d21ad28992aa7d"
+SRC_URI[sha256sum] = "4c13e00c1bed8f1b85daf934cd7b54a08b7b7e31874085ab9277ee563af87139"
+
+SRCREV = "${PV}"
+S = "${WORKDIR}/git"
 
 export prefix = "/opt/ltp"
 export exec_prefix = "/opt/ltp"
 
 inherit autotools
 
-RDEPENDS_${PN} = "perl"
+RDEPENDS_${PN} = "perl e2fsprogs-mke2fs"
 
 FILES_${PN}-dbg += "/opt/ltp/runtest/.debug"
 FILES_${PN}-dbg += "/opt/ltp/testcases/bin/.debug"
@@ -62,7 +65,7 @@  do_extract_tarball() {
 do_install(){
 	install -d ${D}/opt/ltp/
 	oe_runmake DESTDIR=${D} SKIP_IDCHECK=1 install
-	
+
 	# Copy POSIX test suite into ${D}/opt/ltp/testcases by manual
 	cp -r testcases/open_posix_testsuite ${D}/opt/ltp/testcases