Hi Dorinda,

On Sun, Jan 24, 2021 at 02:55:57PM +0100, Dorinda wrote:
> debuginfod scanner 0.181 could only support RPM and scan .debs, it can now 
> scan .ipk
> 

Typo here? The recipe version is 0.182 not 0.181.

> Signed-off-by: Dorinda Bassey <[email protected]>
> ---
>  .../elfutils/elfutils_0.182.bb                |  1 +
>  ...d-support-for-ipkg-to-debuginfod.cxx.patch | 32 +++++++++++++++++++
>  2 files changed, 33 insertions(+)
>  create mode 100644 
> meta/recipes-devtools/elfutils/files/0001-add-support-for-ipkg-to-debuginfod.cxx.patch
> 
> diff --git a/meta/recipes-devtools/elfutils/elfutils_0.182.bb 
> b/meta/recipes-devtools/elfutils/elfutils_0.182.bb
> index 7b829cc7d5..b3d8c8f8ac 100644
> --- a/meta/recipes-devtools/elfutils/elfutils_0.182.bb
> +++ b/meta/recipes-devtools/elfutils/elfutils_0.182.bb
> @@ -20,6 +20,7 @@ SRC_URI = 
> "https://sourceware.org/elfutils/ftp/${PV}/${BP}.tar.bz2 \
>             file://run-ptest \
>             file://ptest.patch \
>             
> file://0001-tests-Makefile.am-compile-test_nlist-with-standard-C.patch \
> +           file://0001-add-support-for-ipkg-to-debuginfod.cxx.patch \
>             "
>  SRC_URI_append_libc-musl = " \
>             file://0001-musl-obstack-fts.patch \
> diff --git 
> a/meta/recipes-devtools/elfutils/files/0001-add-support-for-ipkg-to-debuginfod.cxx.patch
>  
> b/meta/recipes-devtools/elfutils/files/0001-add-support-for-ipkg-to-debuginfod.cxx.patch
> new file mode 100644
> index 0000000000..4bb39c91e8
> --- /dev/null
> +++ 
> b/meta/recipes-devtools/elfutils/files/0001-add-support-for-ipkg-to-debuginfod.cxx.patch
> @@ -0,0 +1,32 @@
> +From b647d1df55c20772a9cc7ce96fcf323c500481bf Mon Sep 17 00:00:00 2001
> +From: dorindabassey <[email protected]>
> +Date: Sat, 19 Dec 2020 01:11:46 +0100
> +Subject: [PATCH] add support for ipkg to debuginfod.cxx
> +
> +added support for ipkg to the debuginfod scanner. 0.181 only supports RPM 
> and scan .debs, with this patch, debuginfod scanner would be able to scan .ipk
> +

Same here.

> +Upstream-status: Pending
> +
> +Signed-off-by: dorindabassey <[email protected]>
> +---
> + debuginfod/debuginfod.cxx | 2 ++
> + 1 file changed, 2 insertions(+)
> +
> +diff --git a/debuginfod/debuginfod.cxx b/debuginfod/debuginfod.cxx
> +index 5621030..cc78c93 100644
> +--- a/debuginfod/debuginfod.cxx
> ++++ b/debuginfod/debuginfod.cxx
> +@@ -442,11 +442,13 @@ parse_opt (int key, char *arg,
> +         {
> +           scan_archives[".deb"]="dpkg-deb --fsys-tarfile";
> +           scan_archives[".ddeb"]="dpkg-deb --fsys-tarfile";
> ++          scan_archives[".ipk"]="dpkg-deb --fsys-tarfile";
> +         }
> +       else
> +         {
> +           scan_archives[".deb"]="(bsdtar -O -x -f - data.tar.xz)<";
> +           scan_archives[".ddeb"]="(bsdtar -O -x -f - data.tar.xz)<";
> ++          scan_archives[".ipk"]="(bsdtar -O -x -f - data.tar.xz)<";
> +         }
> +       // .udeb too?
> +       break;
> -- 
> 2.17.1
> 

This looks like a trivial patch, have you sent it to elfutils mailing
list already? Didn't find any mail from you in their archives.

This way you could update the Upstream-status to Submitted and give the
link, that'll make it easier for maintainers of Yocto/this recipe to fix
a failed recipe upgrade (upstream project might decide this patch is not
enough or they want a different implementation, having the whole
discussion easily reachable is usually very helpful).

Thanks,
Quentin
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#147160): 
https://lists.openembedded.org/g/openembedded-core/message/147160
Mute This Topic: https://lists.openembedded.org/mt/80077932/21656
Group Owner: [email protected]
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[[email protected]]
-=-=-=-=-=-=-=-=-=-=-=-

Reply via email to