From: Saul Wold <sgw@linux.intel.com>
To: Patches and discussions about the oe-core layer
<openembedded-core@lists.openembedded.org>
Cc: Darren Hart <dvhart@linux.intel.com>,
Antonio Ospite <ospite@studenti.unina.it>,
Chris Larson <clarson@kergoth.com>
Subject: Re: [PATCH 34/52] logging: add bb* logging mechanisms for bash recipe functions
Date: Thu, 28 Apr 2011 21:34:44 -0700 [thread overview]
Message-ID: <4DBA3FE4.8040501@linux.intel.com> (raw)
In-Reply-To: <4DBA3EE3.6020006@linux.intel.com>
On 04/28/2011 09:30 PM, Darren Hart wrote:
>
>
> On 04/28/2011 02:05 AM, Richard Purdie wrote:
>> Hi Darren,
>>
>> On Wed, 2011-04-27 at 00:29 -0700, Saul Wold wrote:
>>> The following logging mechanisms are to be used in bash functions of recipes.
>>> They are intended to map one to one in intention and output format with the
>>> python recipe logging functions of a similar naming convention: bb.plain(),
>>> bb.note(), etc.
>>>
>>> For the time being, all of these print only to the task logs. Future
>>> enhancements may integrate these calls with the bitbake logging infrastructure,
>>> allowing for printing to the console as appropriate. The interface and intention
>>> statements reflect that future goal. Once it is in place, no changes will be
>>> necessary to recipes using these logging mechanisms.
>>
>> This looks good but could you do a search and replace of any existing
>> users of the oe* functions and then remove them from base.bbclass
>> please? At the very least they should be calling the bb* equivalents.
>> I'd like to clean up this kind of thing as we go.
>
> Sure thing. Is a follow-up patch acceptable or do you want me to
> resubmit these in one series?
>
Since we have pulled this set, a follow-up patch would be most excellent!
Thanks
Sau!
> --
> Darren
>
>>
>> Cheers,
>>
>> Richard
>>
>>
>>
>> _______________________________________________
>> Openembedded-core mailing list
>> Openembedded-core@lists.openembedded.org
>> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core
>
next prev parent reply other threads:[~2011-04-29 4:37 UTC|newest]
Thread overview: 84+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-04-27 7:29 [PATCH 00/52] Updated Consolidated pull Saul Wold
2011-04-27 7:29 ` [PATCH 01/52] sat-solver: fix arch=all packages Saul Wold
2011-04-27 8:24 ` Antonio Ospite
2011-04-27 18:24 ` Saul Wold
2011-04-27 7:29 ` [PATCH 02/52] quilt: fix test for target build Saul Wold
2011-04-27 7:29 ` [PATCH 03/52] gcc: Add recipes for 4.6.0 Saul Wold
2011-04-27 8:17 ` Antonio Ospite
2011-04-27 18:23 ` Saul Wold
2011-04-27 18:42 ` Khem Raj
2011-04-28 9:14 ` Richard Purdie
2011-04-28 8:53 ` Richard Purdie
2011-04-27 7:29 ` [PATCH 04/52] perl: upgrade from 5.12.2 to 5.12.3 Saul Wold
2011-04-27 7:29 ` [PATCH 06/52] web: update svn to 131 to fix build issue Saul Wold
2011-04-27 7:29 ` [PATCH 05/52] gdb: set --without-ust for EXTRA_OEFLAGS Saul Wold
2011-04-27 7:29 ` [PATCH 07/52] tar-1.17 (GPLv2) bugfix Saul Wold
2011-04-27 7:29 ` [PATCH 08/52] cpio-2.8 (GPLv2) bug fix Saul Wold
2011-04-27 7:29 ` [PATCH 09/52] rxvt-unicode: upgrade to version 9.10 Saul Wold
2011-04-27 7:29 ` [PATCH 10/52] valgrind: upgrade to version 3.6.1 Saul Wold
2011-04-27 7:29 ` [PATCH 11/52] bluez-hcidump: upgrade to version 2.0 Saul Wold
2011-04-27 7:29 ` [PATCH 12/52] gupnp-av: upgrade to version 0.8.0 Saul Wold
2011-04-27 7:29 ` [PATCH 13/52] libsndfile1: upgrade to version 1.0.24 Saul Wold
2011-04-27 7:29 ` [PATCH 14/52] glib-2.0: upgrade to version 2.28.6 Saul Wold
2011-04-27 7:29 ` [PATCH 15/52] libfm: Remove the flag of G_DISABLE_DEPRECATED Saul Wold
2011-04-27 7:29 ` [PATCH 16/52] polkit: upgrade to version 0.101 Saul Wold
2011-04-27 7:29 ` [PATCH 17/52] gst-fluendo-mpegdemux: upgrade to version 0.10.67 Saul Wold
2011-04-27 7:29 ` [PATCH 18/52] libsamplerate0: upgrade to version 0.1.7 Saul Wold
2011-04-27 7:29 ` [PATCH 19/52] alsa-utils: upgrade to version 1.0.24.2 Saul Wold
2011-04-27 7:29 ` [PATCH 21/52] distro_tracking_fields: cwautomacros, valgrind, etc Saul Wold
2011-04-27 8:20 ` Antonio Ospite
2011-04-27 7:29 ` [PATCH 20/52] cwautomacros: upgrade to version 20110201 Saul Wold
2011-04-27 7:29 ` [PATCH 22/52] sstate.bbclass: Reduce SSTATE_MANIFESTS assignment to be weak Saul Wold
2011-04-27 7:29 ` [PATCH 23/52] libiconv: update from 1.9.2 -> 1.11.1 Saul Wold
2011-04-27 7:29 ` [PATCH 24/52] elfutils_0.148.bb: Fix compilation issues on uclibc Saul Wold
2011-04-27 7:29 ` [PATCH 25/52] perl_5.12.2.bb: Undefine features not found in uclibc Saul Wold
2011-04-27 7:29 ` [PATCH 26/52] xorg-lib: inherit gettext class instead of adding gettext to DEPENDS directly Saul Wold
2011-04-27 7:29 ` [PATCH 27/52] attr: Fix compilation on uclibc Saul Wold
2011-04-27 7:29 ` [PATCH 28/52] linux-tools.inc: Dummify do_compile_perf and do_install_perf for uclibc Saul Wold
2011-04-27 14:34 ` Darren Hart
2011-04-27 14:37 ` Darren Hart
2011-04-27 16:10 ` Khem Raj
2011-04-27 7:29 ` [PATCH 29/52] python: add missing cytpes modules Saul Wold
2011-04-27 7:29 ` [PATCH 30/52] cdrtools: recipe and patch cleanup Saul Wold
2011-04-27 7:29 ` [PATCH 31/52] distro_tracking_fields.inc: add cdrtools recipe Saul Wold
2011-04-27 7:29 ` [PATCH 32/52] logging: fix oedebug loglevel test Saul Wold
2011-04-27 7:29 ` [PATCH 33/52] logging: remove unused BBDEBUG from local.conf.sample Saul Wold
2011-04-27 7:29 ` [PATCH 34/52] logging: add bb* logging mechanisms for bash recipe functions Saul Wold
2011-04-28 9:05 ` Richard Purdie
2011-04-29 4:30 ` Darren Hart
2011-04-29 4:34 ` Saul Wold [this message]
2011-04-27 7:29 ` [PATCH 35/52] libsdl: add SRC_URI Checksums Saul Wold
2011-04-27 7:29 ` [PATCH 36/52] gettext.bbclass: Use _append instead of =+ Saul Wold
2011-04-28 9:01 ` Richard Purdie
2011-04-29 16:11 ` Khem Raj
2011-05-03 12:14 ` Richard Purdie
2011-05-03 18:04 ` Khem Raj
2011-05-03 22:39 ` Richard Purdie
2011-05-05 1:07 ` Khem Raj
2011-05-05 4:19 ` Khem Raj
2011-05-05 5:06 ` Saul Wold
2011-05-05 6:38 ` Khem Raj
2011-05-05 9:41 ` Richard Purdie
2011-05-05 16:58 ` Khem Raj
2011-04-27 7:29 ` [PATCH 37/52] insane.bbclass: Move code to add function to tasks toward the end Saul Wold
2011-04-28 8:46 ` Richard Purdie
2011-04-27 7:29 ` [PATCH 38/52] insane.bbclass: Checking for NLS too when checking gettext dependency Saul Wold
2011-04-28 9:02 ` Richard Purdie
2011-04-27 7:30 ` [PATCH 39/52] autotools.bbclass: Copy gettext files only if --disable-nls is not set Saul Wold
2011-04-27 7:30 ` [PATCH 40/52] gettext: Divide packaging into gettext-runtime and rest Saul Wold
2011-04-27 7:30 ` [PATCH 41/52] gettext_0.17.bb: PROVIDE virtual/gettext and RCONFLICT with proxy-libintl Saul Wold
2011-04-27 7:30 ` [PATCH 42/52] poky-default.inc: Weakly assign gettext to provide virtual/gettext Saul Wold
2011-04-27 7:30 ` [PATCH 43/52] glib-2.0: Inherit gettext should provide right libintl so remove from DEPENDS Saul Wold
2011-04-27 7:30 ` [PATCH 45/52] perl_5.12.2.bb: Undefine features not found in uclibc Saul Wold
2011-04-27 7:30 ` [PATCH 44/52] util-linux.inc: remove virtual/libintl from DEPENDS Saul Wold
2011-04-28 9:03 ` Richard Purdie
2011-04-27 7:30 ` [PATCH 46/52] alsa-utils_1.0.23.bb: Fix compilation for uclibc targets Saul Wold
2011-04-27 7:30 ` [PATCH 47/52] clutter-box2d: add DEPENDS for clutter-1.6 Saul Wold
2011-04-27 7:30 ` [PATCH 48/52] table: move old clutter demo to meta-demoapps Saul Wold
2011-04-27 7:30 ` [PATCH 49/52] tidy: " Saul Wold
2011-04-27 7:30 ` [PATCH 50/52] alsa-tools: Add checksums Saul Wold
2011-04-27 7:30 ` [PATCH 51/52] world-broken: add older clutters since we don't want clutter confusion Saul Wold
2011-04-27 7:30 ` [PATCH 52/52] qemu: Fix non-sdl/opengl build Saul Wold
2011-04-27 8:17 ` [PATCH 00/52] Updated Consolidated pull Antonio Ospite
2011-04-27 17:38 ` Saul Wold
2011-04-28 9:12 ` Richard Purdie
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=4DBA3FE4.8040501@linux.intel.com \
--to=sgw@linux.intel.com \
--cc=clarson@kergoth.com \
--cc=dvhart@linux.intel.com \
--cc=openembedded-core@lists.openembedded.org \
--cc=ospite@studenti.unina.it \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox