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>
Subject: Re: [PATCHv 08/10] libusb*: import native support from meta-oe
Date: Sat, 24 Mar 2012 10:02:33 +0000	[thread overview]
Message-ID: <1332583353.9740.500.camel@ted> (raw)
In-Reply-To: <313583f23e026bed3155643d1e8914313fce7478.1332541342.git.Martin.Jansa@gmail.com>

On Fri, 2012-03-23 at 23:30 +0100, Martin Jansa wrote:
> Signed-off-by: Martin Jansa <Martin.Jansa@gmail.com>
> ---
>  meta/recipes-support/libusb/libusb-compat_0.1.3.bb |    7 +++++--
>  meta/recipes-support/libusb/libusb1_1.0.8.bb       |    2 ++
>  2 files changed, 7 insertions(+), 2 deletions(-)

Looking at this, it makes no sense. Why does our build system need to be
trying to extract information from the *native* system's USB bus?

I can understand pango and cairo for font rendering in something like
icons but libusb-native?

Are you really sure we need this and have no other options as I can't
see it actually being used...

Looking at the other patches, I'm guessing they're broken without your
PACKAGECONFIG change to fix -natives. A goal of this release is to have
everything building and working that we ship. These 'simple'
BBCLASSEXTENDs are likely going to change that. 

I don't really want to spend my weekend trying to fix and then test the
PACKAGECONFIG change so I can merge these patches, particular as I've
spent most of the week including the evenings just getting things to
build. Arguably there could be other PACKAGECONFIG issue already I guess
so we probably should fix that before release anyway. With -rc1 on
Monday, that will be tricky :(.

Cheers,

Richard




  reply	other threads:[~2012-03-24 10:11 UTC|newest]

Thread overview: 39+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-03-21 21:36 [PATCH 00/13] Merge native BBCLASSEXTENDs and SDK, PACKAGECONFIG fix Martin Jansa
2012-03-21 21:36 ` [PATCH 01/13] bitbake.conf: use TUNE_PKGARCH instead of TARGET_ARCH in SDK_NAME Martin Jansa
2012-03-21 21:36 ` [PATCH 02/13] xev: move from meta-demoapps Martin Jansa
2012-03-21 21:36 ` [PATCH 03/13] classes: scons: add EXTRA_OESCONS Martin Jansa
2012-03-21 21:36 ` [PATCH 04/13] base.bbclass: extract *virtclass_map_dependencies logic from native/nativesdk bbclasses and call it from packageconfig Martin Jansa
2012-03-21 21:36 ` [PATCH 05/13] xorg: add more native BBCLASSEXTENDs for gtk+-native Martin Jansa
2012-03-21 21:36 ` [PATCH 06/13] gtk+: import native support from meta-oe Martin Jansa
2012-03-21 23:11   ` Richard Purdie
2012-03-21 23:50     ` Martin Jansa
2012-03-22  0:02       ` Richard Purdie
2012-03-22  6:51         ` Martin Jansa
2012-03-22  9:52           ` Richard Purdie
2012-03-22 10:04             ` Martin Jansa
2012-03-21 21:36 ` [PATCH 07/13] librsvg: " Martin Jansa
2012-03-21 21:36 ` [PATCH 08/13] pixman: add native support and perl-native to DEPENDS " Martin Jansa
2012-03-21 21:36 ` [PATCH 09/13] cairo: import native support " Martin Jansa
2012-03-21 21:36 ` [PATCH 10/13] tiff: " Martin Jansa
2012-03-21 21:36 ` [PATCH 11/13] libusb*: " Martin Jansa
2012-03-21 21:36 ` [PATCH 12/13] pango: import native support and --disable-introspection " Martin Jansa
2012-03-21 21:36 ` [PATCH 13/13] rootfs_ipk: replace 3 opkg-cl calls with one in get_package_filename Martin Jansa
2012-03-23 22:28 ` [PATCHv 00/10] Merge native BBCLASSEXTENDs and SDK fix Martin Jansa
2012-03-23 22:30   ` [PATCHv 01/10] bitbake.conf: use TUNE_PKGARCH instead of TARGET_ARCH in SDK_NAME Martin Jansa
2012-03-23 22:30   ` [PATCHv 02/10] xev: move from meta-demoapps Martin Jansa
2012-03-23 22:30   ` [PATCHv 03/10] classes: scons: add EXTRA_OESCONS Martin Jansa
2012-03-23 22:30   ` [PATCHv 04/10] librsvg: import native support from meta-oe Martin Jansa
2012-03-23 22:30   ` [PATCHv 05/10] pixman: add native support and perl-native to DEPENDS " Martin Jansa
2012-03-30 14:10     ` Richard Purdie
2012-03-30 14:22       ` Martin Jansa
2012-03-30 14:43         ` Richard Purdie
2012-03-23 22:30   ` [PATCHv 06/10] cairo: import native support " Martin Jansa
2012-03-23 22:30   ` [PATCHv 07/10] tiff: " Martin Jansa
2012-03-23 22:30   ` [PATCHv 08/10] libusb*: " Martin Jansa
2012-03-24 10:02     ` Richard Purdie [this message]
2012-03-24 10:56       ` Martin Jansa
2012-03-26 11:44     ` Richard Purdie
2012-03-26 12:19       ` Martin Jansa
2012-03-26 12:27       ` [PATCHv2 " Martin Jansa
2012-03-23 22:30   ` [PATCHv 09/10] pango: import native support and --disable-introspection " Martin Jansa
2012-03-23 22:30   ` [PATCHv 10/10] rootfs_ipk: replace 3 opkg-cl calls with one in get_package_filename Martin Jansa

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=1332583353.9740.500.camel@ted \
    --to=richard.purdie@linuxfoundation.org \
    --cc=openembedded-core@lists.openembedded.org \
    /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.