Openembedded Core Discussions
 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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox