All of lore.kernel.org
 help / color / mirror / Atom feed
From: Richard Purdie <richard.purdie@linuxfoundation.org>
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 10:05:52 +0100	[thread overview]
Message-ID: <1303981552.21461.43.camel@rex> (raw)
In-Reply-To: <a34718a02b76e37c59ff67d9e182bd7b64c9f125.1303889119.git.sgw@linux.intel.com>

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.

Cheers,

Richard





  reply	other threads:[~2011-04-28  9:09 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 05/52] gdb: set --without-ust for EXTRA_OEFLAGS 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 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 20/52] cwautomacros: upgrade to version 20110201 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 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 [this message]
2011-04-29  4:30     ` Darren Hart
2011-04-29  4:34       ` Saul Wold
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 44/52] util-linux.inc: remove virtual/libintl " Saul Wold
2011-04-28  9:03   ` Richard Purdie
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 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=1303981552.21461.43.camel@rex \
    --to=richard.purdie@linuxfoundation.org \
    --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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.