libpcre, ncurses: Fix problems expanding the IMAGE_INSTALL package groups

Submitted by Jason Wessel on April 23, 2013, 4:08 p.m.

Details

Message ID 1366733299-14804-1-git-send-email-jason.wessel@windriver.com
State New
Headers show

Commit Message

Jason Wessel April 23, 2013, 4:08 p.m.
The ncurses and libpcre packages were generating the following errors
as a result of not specifing the PACKAGES_DYNAMIC correctly.  These
errors only appear when using the IMAGE_INSTALL list that has been
expanded by the hob or from the pkgdata.

ERROR: Nothing RPROVIDES 'ncurses-libtinfo'
ERROR: Nothing RPROVIDES 'libpcreposix'

Signed-off-by: Jason Wessel <jason.wessel@windriver.com>
Signed-off-by: Mark Hatle <mark.hatle@windriver.com>
---
 meta/recipes-core/ncurses/ncurses.inc        |    2 ++
 meta/recipes-support/libpcre/libpcre_8.32.bb |    2 ++
 2 files changed, 4 insertions(+), 0 deletions(-)

Patch hide | download patch | download mbox

diff --git a/meta/recipes-core/ncurses/ncurses.inc b/meta/recipes-core/ncurses/ncurses.inc
index bcb9c94..78ff153 100644
--- a/meta/recipes-core/ncurses/ncurses.inc
+++ b/meta/recipes-core/ncurses/ncurses.inc
@@ -42,6 +42,8 @@  EXCONFIG_ARGS_virtclass-nativesdk = " \
 		--with-chtype='long' \
 		--with-mmask-t='long'"
 
+PACKAGES_DYNAMIC = "^${PN}-.*"
+
 # Fall back to the host termcap / terminfo for -nativesdk and -native
 # The reality is a work around for strange problems with things like
 # "bitbake -c menuconfig busybox" where it cannot find the terminfo
diff --git a/meta/recipes-support/libpcre/libpcre_8.32.bb b/meta/recipes-support/libpcre/libpcre_8.32.bb
index 4d17741..71cc119 100644
--- a/meta/recipes-support/libpcre/libpcre_8.32.bb
+++ b/meta/recipes-support/libpcre/libpcre_8.32.bb
@@ -26,6 +26,8 @@  inherit autotools binconfig
 
 PARALLEL_MAKE = ""
 
+PACKAGES_DYNAMIC = "^${PN}.*"
+
 EXTRA_OECONF = "\
     --enable-newline-is-lf \
     --enable-rebuild-chartables \

Comments

Richard Purdie April 23, 2013, 5 p.m.
On Tue, 2013-04-23 at 11:08 -0500, Jason Wessel wrote:
> The ncurses and libpcre packages were generating the following errors
> as a result of not specifing the PACKAGES_DYNAMIC correctly.  These
> errors only appear when using the IMAGE_INSTALL list that has been
> expanded by the hob or from the pkgdata.
> 
> ERROR: Nothing RPROVIDES 'ncurses-libtinfo'
> ERROR: Nothing RPROVIDES 'libpcreposix'
> 
> Signed-off-by: Jason Wessel <jason.wessel@windriver.com>
> Signed-off-by: Mark Hatle <mark.hatle@windriver.com>
> ---
>  meta/recipes-core/ncurses/ncurses.inc        |    2 ++
>  meta/recipes-support/libpcre/libpcre_8.32.bb |    2 ++
>  2 files changed, 4 insertions(+), 0 deletions(-)
>
> diff --git a/meta/recipes-core/ncurses/ncurses.inc b/meta/recipes-core/ncurses/ncurses.inc
> index bcb9c94..78ff153 100644
> --- a/meta/recipes-core/ncurses/ncurses.inc
> +++ b/meta/recipes-core/ncurses/ncurses.inc
> @@ -42,6 +42,8 @@ EXCONFIG_ARGS_virtclass-nativesdk = " \
>  		--with-chtype='long' \
>  		--with-mmask-t='long'"
>  
> +PACKAGES_DYNAMIC = "^${PN}-.*"
> +
>  # Fall back to the host termcap / terminfo for -nativesdk and -native
>  # The reality is a work around for strange problems with things like
>  # "bitbake -c menuconfig busybox" where it cannot find the terminfo

This makes sense since its doing dynamic package splitting.

> diff --git a/meta/recipes-support/libpcre/libpcre_8.32.bb b/meta/recipes-support/libpcre/libpcre_8.32.bb
> index 4d17741..71cc119 100644
> --- a/meta/recipes-support/libpcre/libpcre_8.32.bb
> +++ b/meta/recipes-support/libpcre/libpcre_8.32.bb
> @@ -26,6 +26,8 @@ inherit autotools binconfig
>  
>  PARALLEL_MAKE = ""
>  
> +PACKAGES_DYNAMIC = "^${PN}.*"
> +
>  EXTRA_OECONF = "\
>      --enable-newline-is-lf \
>      --enable-rebuild-chartables \

However this does not. libpcreposix is clearly listed in PACKAGES. Was
this based on an older recipe and can someone check this is needed on
master?

Cheers,

Richard
Jason Wessel April 23, 2013, 5:26 p.m.
On 04/23/2013 12:00 PM, Richard Purdie wrote:
> On Tue, 2013-04-23 at 11:08 -0500, Jason Wessel wrote:
>> The ncurses and libpcre packages were generating the following errors
>> as a result of not specifing the PACKAGES_DYNAMIC correctly.  These
>> errors only appear when using the IMAGE_INSTALL list that has been
>> expanded by the hob or from the pkgdata.
>>
>> ERROR: Nothing RPROVIDES 'ncurses-libtinfo'
>> ERROR: Nothing RPROVIDES 'libpcreposix'
>>
>> Signed-off-by: Jason Wessel <jason.wessel@windriver.com>
>> Signed-off-by: Mark Hatle <mark.hatle@windriver.com>
>> ---
>>  meta/recipes-core/ncurses/ncurses.inc        |    2 ++
>>  meta/recipes-support/libpcre/libpcre_8.32.bb |    2 ++
>>  2 files changed, 4 insertions(+), 0 deletions(-)
>>
>> diff --git a/meta/recipes-core/ncurses/ncurses.inc b/meta/recipes-core/ncurses/ncurses.inc
>> index bcb9c94..78ff153 100644
>> --- a/meta/recipes-core/ncurses/ncurses.inc
>> +++ b/meta/recipes-core/ncurses/ncurses.inc
>> @@ -42,6 +42,8 @@ EXCONFIG_ARGS_virtclass-nativesdk = " \
>>  		--with-chtype='long' \
>>  		--with-mmask-t='long'"
>>  
>> +PACKAGES_DYNAMIC = "^${PN}-.*"
>> +
>>  # Fall back to the host termcap / terminfo for -nativesdk and -native
>>  # The reality is a work around for strange problems with things like
>>  # "bitbake -c menuconfig busybox" where it cannot find the terminfo
> This makes sense since its doing dynamic package splitting.
>
>> diff --git a/meta/recipes-support/libpcre/libpcre_8.32.bb b/meta/recipes-support/libpcre/libpcre_8.32.bb
>> index 4d17741..71cc119 100644
>> --- a/meta/recipes-support/libpcre/libpcre_8.32.bb
>> +++ b/meta/recipes-support/libpcre/libpcre_8.32.bb
>> @@ -26,6 +26,8 @@ inherit autotools binconfig
>>  
>>  PARALLEL_MAKE = ""
>>  
>> +PACKAGES_DYNAMIC = "^${PN}.*"
>> +
>>  EXTRA_OECONF = "\
>>      --enable-newline-is-lf \
>>      --enable-rebuild-chartables \
> However this does not. libpcreposix is clearly listed in PACKAGES. Was
> this based on an older recipe and can someone check this is needed on
> master?


I'll double check on this and submit a new patch.   This was originally against the libpcre 8.21 from YP 1.2.

Jason.