Skip to contentSkip to conversation

lsof_4.83.bb: Make recipe buildable again.

PATCH In review Graham Gower 3 participants 4 messages Jul 16, 2010 – Jul 18, 2010

lore
Share & tools
lore

Graham Gower

author
#0 lore raw

PATCH 1/1 lsof_4.83.bb: Make recipe buildable again.

Breakage introduced by 74e32f114272668dfa4c88805741cf55ef965989. Other recipes which call base_do_unpack directly do not share this issue.

Signed-off-by Graham Gower <[email protected]>
---
 recipes/lsof/lsof_4.83.bb |   13 +++++--------
 1 files changed, 5 insertions(+), 8 deletions(-)
recipes/lsof/lsof_4.83.bb +5 -8
index d74c574..0a5d557 100644
@@ -9,17 +9,14 @@ SRC_URI = "ftp://lsof.itap.purdue.edu/pub/tools/unix/lsof/lsof_${PV}.tar.bz2;nam
 SRC_URI[lsof483tarbz2.md5sum] = "8f731a6251b8c0143d585df0d5ca779e"
 SRC_URI[lsof483tarbz2.sha256sum] = "b89f930bbe36b970e3cd070b9860ee701d8c7285ffedf2fbcec0e5fa3cb1f544"
 
-LOCALSRC = "file://${WORKDIR}/lsof_${PV}/lsof_${PV}_src.tar"
+LOCALSRC = "${WORKDIR}/lsof_${PV}/lsof_${PV}_src.tar"
 S = "${WORKDIR}/lsof_${PV}_src"
 
-# the tar.bz2 file contains another tar, cde below unpacks it
-python do_unpack () {
-	bb.build.exec_func('base_do_unpack', d)
-	src_uri = bb.data.getVar('SRC_URI', d)
-	bb.data.setVar('SRC_URI', '${LOCALSRC}', d)
-	bb.build.exec_func('base_do_unpack', d)
-	bb.data.setVar('SRC_URI', src_uri, d)
+# The tar.bz2 file contains another tar.
+do_unpackpost() {
+	tar x --no-same-owner -f ${LOCALSRC} -C ${WORKDIR}
 }
+addtask do_unpackpost before do_patch after do_unpack
 
 export LSOF_OS = "${TARGET_OS}"
 LSOF_OS_linux-uclibc = "linux"
Signature · 1 line
1.7.1
#2 lore raw
Quoted 11 lines · Stefan Schmidt

Hello.

Quoted 4 lines · Graham Gower

Breakage introduced by 74e32f114272668dfa4c88805741cf55ef965989. Other recipes which call base_do_unpack directly do not share this issue.

Signed-off-by Graham Gower <[email protected]>
Acked-by Stefan Schmidt <[email protected]>

I just run into the same issue and this fixes it for me. Will check this in.

Does not work here. This change fixes it:

recipes/lsof/lsof_4.83.bb +2 -2
index 0a5d557..8982a2c 100644
@@ -3,7 +3,7 @@ Its name stands for LiSt Open Files, and it does just that."
 SECTION = "devel"
 LICENSE = "BSD"
 
-PR = "r0"
+PR = "r1"
 
 SRC_URI = "ftp://lsof.itap.purdue.edu/pub/tools/unix/lsof/lsof_${PV}.tar.bz2;name=lsof483tarbz2"
 SRC_URI[lsof483tarbz2.md5sum] = "8f731a6251b8c0143d585df0d5ca779e"

@@ -16,7 +16,7 @@ S = "${WORKDIR}/lsof_${PV}_src"
 do_unpackpost() {
 	tar x --no-same-owner -f ${LOCALSRC} -C ${WORKDIR}
 }
-addtask do_unpackpost before do_patch after do_unpack
+addtask unpackpost before do_patch after do_unpack
 
 export LSOF_OS = "${TARGET_OS}"
 LSOF_OS_linux-uclibc = "linux"
Signature · 3 lines
________________________________________________________________________
Stanislav Brabec
http://www.penguin.cz/~utx/zaurus
Acked-by
#3 lore raw

Hello.

Quoted 6 lines · Stanislav Brabec
Quoted 5 lines · Stefan Schmidt
Quoted 3 lines · Graham Gower

Breakage introduced by 74e32f114272668dfa4c88805741cf55ef965989. Other recipes which call base_do_unpack directly do not share this issue.

Quoted 7 lines
Quoted 5 lines
Quoted 1 line
Signed-off-by Graham Gower <[email protected]>
Acked-by Stefan Schmidt <[email protected]>

I just run into the same issue and this fixes it for me. Will check this in.

Does not work here. This change fixes it:

And you are right here. I wonder why it worked for me when I tested it. It fails now again and your patch fixes it.

Acked-by Stefan Schmidt <[email protected]>

Will you commit this yourself?

regards
Stefan Schmidt