package.bbclass: convert unpackaged file message from 'info' to 'warn' so that it shows up on the console

Submitted by Koen Kooi on May 6, 2011, 2:48 p.m.

Details

Message ID 1304693308-5625-1-git-send-email-koen@dominion.thruhere.net
State New, archived
Headers show

Commit Message

Koen Kooi May 6, 2011, 2:48 p.m.
Signed-off-by: Koen Kooi <koen@dominion.thruhere.net>
---
 meta/classes/package.bbclass |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

Patch hide | download patch | download mbox

diff --git a/meta/classes/package.bbclass b/meta/classes/package.bbclass
index e6b3df7..082f233 100644
--- a/meta/classes/package.bbclass
+++ b/meta/classes/package.bbclass
@@ -676,9 +676,9 @@  python populate_packages () {
 				unshipped.append(path)
 
 	if unshipped != []:
-		bb.note("the following files were installed but not shipped in any package:")
+		bb.warn("the following files were installed but not shipped in any package:")
 		for f in unshipped:
-			bb.note("  " + f)
+			bb.warn("  " + f)
 
 	bb.build.exec_func("package_name_hook", d)
 

Comments

Mark Hatle May 6, 2011, 2:57 p.m.
Acked-by: Mark Hatle <mark.hatle@windriver.com>

On 5/6/11 9:48 AM, Koen Kooi wrote:
> Signed-off-by: Koen Kooi <koen@dominion.thruhere.net>
> ---
>  meta/classes/package.bbclass |    4 ++--
>  1 files changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/meta/classes/package.bbclass b/meta/classes/package.bbclass
> index e6b3df7..082f233 100644
> --- a/meta/classes/package.bbclass
> +++ b/meta/classes/package.bbclass
> @@ -676,9 +676,9 @@ python populate_packages () {
>  				unshipped.append(path)
>  
>  	if unshipped != []:
> -		bb.note("the following files were installed but not shipped in any package:")
> +		bb.warn("the following files were installed but not shipped in any package:")
>  		for f in unshipped:
> -			bb.note("  " + f)
> +			bb.warn("  " + f)
>  
>  	bb.build.exec_func("package_name_hook", d)
>
Tom Rini May 6, 2011, 3:19 p.m.
On 05/06/2011 07:48 AM, Koen Kooi wrote:
> Signed-off-by: Koen Kooi <koen@dominion.thruhere.net>
> ---
>  meta/classes/package.bbclass |    4 ++--
>  1 files changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/meta/classes/package.bbclass b/meta/classes/package.bbclass
> index e6b3df7..082f233 100644
> --- a/meta/classes/package.bbclass
> +++ b/meta/classes/package.bbclass
> @@ -676,9 +676,9 @@ python populate_packages () {
>  				unshipped.append(path)
>  
>  	if unshipped != []:
> -		bb.note("the following files were installed but not shipped in any package:")
> +		bb.warn("the following files were installed but not shipped in any package:")
>  		for f in unshipped:
> -			bb.note("  " + f)
> +			bb.warn("  " + f)
>  
>  	bb.build.exec_func("package_name_hook", d)
>  

Acked-by: Tom Rini <tom_rini@mentor.com>
Darren Hart May 7, 2011, 4:50 p.m.
According to my discussion with RP, bb.note() is supposed to appear on
the console. If it doesn't, that is a bug in bitbake, not the recipe
logging. I used this information to document the intended use of the
similarly named bbnote() in logging.bbclass.

--
Darren

On Fri, 2011-05-06 at 16:48 +0200, Koen Kooi wrote:
> Signed-off-by: Koen Kooi <koen@dominion.thruhere.net>
> ---
>  meta/classes/package.bbclass |    4 ++--
>  1 files changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/meta/classes/package.bbclass b/meta/classes/package.bbclass
> index e6b3df7..082f233 100644
> --- a/meta/classes/package.bbclass
> +++ b/meta/classes/package.bbclass
> @@ -676,9 +676,9 @@ python populate_packages () {
>  				unshipped.append(path)
>  
>  	if unshipped != []:
> -		bb.note("the following files were installed but not shipped in any package:")
> +		bb.warn("the following files were installed but not shipped in any package:")
>  		for f in unshipped:
> -			bb.note("  " + f)
> +			bb.warn("  " + f)
>  
>  	bb.build.exec_func("package_name_hook", d)
>
Khem Raj May 8, 2011, 6:23 a.m.
On 5/7/2011 9:50 AM, Darren Hart wrote:
> According to my discussion with RP, bb.note() is supposed to appear on
> the console. If it doesn't, that is a bug in bitbake, not the recipe

it does if we increase verbosity but it also dumps a lot of other 
information along. Default is to print errors and warnings which seems a 
sane default to me. May be there is another level which dumps notes on
top and then as you increase the level you get more info dumped would be
a good to have

> logging. I used this information to document the intended use of the
> similarly named bbnote() in logging.bbclass.
>
> --
> Darren
>
> On Fri, 2011-05-06 at 16:48 +0200, Koen Kooi wrote:
>> Signed-off-by: Koen Kooi<koen@dominion.thruhere.net>
>> ---
>>   meta/classes/package.bbclass |    4 ++--
>>   1 files changed, 2 insertions(+), 2 deletions(-)
>>
>> diff --git a/meta/classes/package.bbclass b/meta/classes/package.bbclass
>> index e6b3df7..082f233 100644
>> --- a/meta/classes/package.bbclass
>> +++ b/meta/classes/package.bbclass
>> @@ -676,9 +676,9 @@ python populate_packages () {
>>   				unshipped.append(path)
>>
>>   	if unshipped != []:
>> -		bb.note("the following files were installed but not shipped in any package:")
>> +		bb.warn("the following files were installed but not shipped in any package:")
>>   		for f in unshipped:
>> -			bb.note("  " + f)
>> +			bb.warn("  " + f)
>>
>>   	bb.build.exec_func("package_name_hook", d)
>>
>
Koen Kooi May 9, 2011, 7:11 a.m.
Op 8 mei 2011, om 08:23 heeft Khem Raj het volgende geschreven:

> On 5/7/2011 9:50 AM, Darren Hart wrote:
>> According to my discussion with RP, bb.note() is supposed to appear on
>> the console. If it doesn't, that is a bug in bitbake, not the recipe
> 
> it does if we increase verbosity but it also dumps a lot of other information along. Default is to print errors and warnings which seems a sane default to me. May be there is another level which dumps notes on
> top and then as you increase the level you get more info dumped would be
> a good to have

I still think that all unpackaged files are a bug, so it's more than a note anyway.

regards,

Koen

> 
>> logging. I used this information to document the intended use of the
>> similarly named bbnote() in logging.bbclass.
>> 
>> --
>> Darren
>> 
>> On Fri, 2011-05-06 at 16:48 +0200, Koen Kooi wrote:
>>> Signed-off-by: Koen Kooi<koen@dominion.thruhere.net>
>>> ---
>>>  meta/classes/package.bbclass |    4 ++--
>>>  1 files changed, 2 insertions(+), 2 deletions(-)
>>> 
>>> diff --git a/meta/classes/package.bbclass b/meta/classes/package.bbclass
>>> index e6b3df7..082f233 100644
>>> --- a/meta/classes/package.bbclass
>>> +++ b/meta/classes/package.bbclass
>>> @@ -676,9 +676,9 @@ python populate_packages () {
>>>  				unshipped.append(path)
>>> 
>>>  	if unshipped != []:
>>> -		bb.note("the following files were installed but not shipped in any package:")
>>> +		bb.warn("the following files were installed but not shipped in any package:")
>>>  		for f in unshipped:
>>> -			bb.note("  " + f)
>>> +			bb.warn("  " + f)
>>> 
>>>  	bb.build.exec_func("package_name_hook", d)
>>> 
>> 
> 
> 
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core
Graeme Gregory May 9, 2011, 7:16 a.m.
On 09/05/2011 08:11, Koen Kooi wrote:
> Op 8 mei 2011, om 08:23 heeft Khem Raj het volgende geschreven:
>
>> On 5/7/2011 9:50 AM, Darren Hart wrote:
>>> According to my discussion with RP, bb.note() is supposed to appear on
>>> the console. If it doesn't, that is a bug in bitbake, not the recipe
>> it does if we increase verbosity but it also dumps a lot of other information along. Default is to print errors and warnings which seems a sane default to me. May be there is another level which dumps notes on
>> top and then as you increase the level you get more info dumped would be
>> a good to have
> I still think that all unpackaged files are a bug, so it's more than a note anyway.
>
rpmbuild would agree with you there, so its not without precedence.

G
Richard Purdie May 9, 2011, 9:48 a.m.
On Sat, 2011-05-07 at 09:50 -0700, Darren Hart wrote:
> According to my discussion with RP, bb.note() is supposed to appear on
> the console. If it doesn't, that is a bug in bitbake, not the recipe
> logging. I used this information to document the intended use of the
> similarly named bbnote() in logging.bbclass.

What we said was that bb.note from "core" context appears on the
console, bb.note from task context does not as it makes the default
console too verbose.

I'm in agreement that unpackaged files should be a warning.

Cheers,

Richard
Saul Wold May 9, 2011, 4:06 p.m.
On 05/06/2011 07:48 AM, Koen Kooi wrote:
> Signed-off-by: Koen Kooi<koen@dominion.thruhere.net>
> ---
>   meta/classes/package.bbclass |    4 ++--
>   1 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/meta/classes/package.bbclass b/meta/classes/package.bbclass
> index e6b3df7..082f233 100644
> --- a/meta/classes/package.bbclass
> +++ b/meta/classes/package.bbclass
> @@ -676,9 +676,9 @@ python populate_packages () {
>   				unshipped.append(path)
>
>   	if unshipped != []:
> -		bb.note("the following files were installed but not shipped in any package:")
> +		bb.warn("the following files were installed but not shipped in any package:")
>   		for f in unshipped:
> -			bb.note("  " + f)
> +			bb.warn("  " + f)
>
>   	bb.build.exec_func("package_name_hook", d)
>

Pulled into oe-Core

Thanks
	Sau!