diff mbox series

[v3,2/3] binutils-cross-canadian: add gprofng package as runtime dependency

Message ID 20240912054628.1539323-2-Harish.Sadineni@windriver.com
State New
Headers show
Series [v3,1/3] binutils: Fix packaging of bfd-plugins | expand

Commit Message

Sadineni, Harish Sept. 12, 2024, 5:46 a.m. UTC
From: Harish Sadineni <Harish.Sadineni@windriver.com>

In SDK, while running "x86_64-poky-linux-gp-collect-app ./exe" fails
to create 'test.1.er' due to missing gprofng configure file in sysconfdir
and aborting with below error while generating the profiling data.
 - configuration error: can not find libgp-collector.so. run aborted

Signed-off-by: Harish Sadineni <Harish.Sadineni@windriver.com>
---
 meta/recipes-devtools/binutils/binutils-cross-canadian.inc | 1 +
 1 file changed, 1 insertion(+)

Comments

Richard Purdie Sept. 12, 2024, 6:42 a.m. UTC | #1
On Wed, 2024-09-11 at 22:46 -0700, Sadineni, Harish via lists.openembedded.org wrote:
> From: Harish Sadineni <Harish.Sadineni@windriver.com>
> 
> In SDK, while running "x86_64-poky-linux-gp-collect-app ./exe" fails
> to create 'test.1.er' due to missing gprofng configure file in sysconfdir
> and aborting with below error while generating the profiling data.
>  - configuration error: can not find libgp-collector.so. run aborted
> 
> Signed-off-by: Harish Sadineni <Harish.Sadineni@windriver.com>
> ---
>  meta/recipes-devtools/binutils/binutils-cross-canadian.inc | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/meta/recipes-devtools/binutils/binutils-cross-canadian.inc b/meta/recipes-devtools/binutils/binutils-cross-canadian.inc
> index 62ebb3f5bc..7f2699089e 100644
> --- a/meta/recipes-devtools/binutils/binutils-cross-canadian.inc
> +++ b/meta/recipes-devtools/binutils/binutils-cross-canadian.inc
> @@ -30,3 +30,4 @@ do_install () {
>  }
>  
>  BBCLASSEXTEND = ""
> +RDEPENDS:${PN} = "nativesdk-gprofng"

Shouldn't gp-collect-app be put into the gprofng package instead?

Adding that dependency effectively join the gprof package into the main
PN and makes the separation pointless.

Cheers,

Richard
diff mbox series

Patch

diff --git a/meta/recipes-devtools/binutils/binutils-cross-canadian.inc b/meta/recipes-devtools/binutils/binutils-cross-canadian.inc
index 62ebb3f5bc..7f2699089e 100644
--- a/meta/recipes-devtools/binutils/binutils-cross-canadian.inc
+++ b/meta/recipes-devtools/binutils/binutils-cross-canadian.inc
@@ -30,3 +30,4 @@  do_install () {
 }
 
 BBCLASSEXTEND = ""
+RDEPENDS:${PN} = "nativesdk-gprofng"