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: [PATCH v2 00/16] python-native: Put binaries in seperate directory
Date: Fri, 20 Jul 2012 17:16:55 +0100	[thread overview]
Message-ID: <1342801015.21788.42.camel@ted> (raw)
In-Reply-To: <1342719981-1324-1-git-send-email-morgan.little@windriver.com>

On Thu, 2012-07-19 at 13:46 -0400, Morgan Little wrote:
> I updated python-native like it was suggest by Richard here:
> http://lists.linuxtogo.org/pipermail/openembedded-core/2012-June/024644.html
> 
> It is to resolve an issue that comes up when python-native is building and 
> autoconf finds the binary but not the libraries. This fix moves where the 
> binaries are installed to remove this issue and use the system's python.
> 
> I have used bitbake world to build to make sure most things are build and 
> have added all of the recipes that I found were affected. I will be building 
> more to make sure I managed to get all of them.
> 
> -----V2 Change-----
> Updated distutils-base to inherit pythonnative
> 
> Removed added pythonnative inherit in distutils
> 
> Removed added inherit for pythonnative from python-dbus, python-pygtk,
> python-setuptools, python-gst, python-pygobject since they get the inherit from
> distutils-base
> 
> Updated perf for the change
> 
> 
> The following changes since commit 28715eff6dff3415b1d7b0be8cbb465c417e307f:
>   Richard Purdie (1):
>         boot-directdisk: Fix kernel location after STAGING_KERNEL_DIR change
> 
> are available in the git repository at:
> 
>   git@github.com:morganlittle/oe-core.git master
> 
> Morgan Little (16):
>       python-native: Put binaries in seperate directory
>       python: Update for python-native changes
>       python-scons-native: Update for python-native changes
>       python-pyrex-native: Update for python-native changes
>       mklibs-native: Update for python-native changes
>       telepathy-glib: Update for python-native changes
>       telepathy-python: Update for python-native changes
>       libxml2: Update for python-native changes
>       gnome-doc-utils: Update for python-native changes
>       gobject-introspection: Update for python-native changes
>       mesa-common: Update for python-native changes
>       libxcb: Update for python-native changes
>       xcb-proto: Update for python-native changes
>       perf: Update for python-native changes
>       kernelshark: Update for python-native changes
>       trace-cmd: Update for python-native changes

I've been testing this and making a more detailed review of the changes.
Firstly, I realised you were adding the pythonnative class to recipes
which never had DEPENDS = "python-native". That isn't right and
shouldn't be done for telepathy-glib, libxcb, kernelshark and
trace-cmd. 

gnome-doc-utils needed further attention. I've looked into it and I
think we need to put a wrapper around xmlto in the native case so that
we can use the full path to python. I'm going to submit a patch for
this.

You can see the version of this patch set I plan to merge on master-next
(dropping the above mentioned patches).

Cheers,

Richard







  parent reply	other threads:[~2012-07-20 16:28 UTC|newest]

Thread overview: 19+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-07-19 17:46 [PATCH v2 00/16] python-native: Put binaries in seperate directory Morgan Little
2012-07-19 17:46 ` [PATCH v2 01/16] " Morgan Little
2012-07-19 17:46 ` [PATCH v2 02/16] python: Update for python-native changes Morgan Little
2012-07-19 17:46 ` [PATCH v2 03/16] python-scons-native: " Morgan Little
2012-07-19 17:46 ` [PATCH v2 04/16] python-pyrex-native: " Morgan Little
2012-07-19 17:46 ` [PATCH v2 05/16] mklibs-native: " Morgan Little
2012-07-19 17:46 ` [PATCH v2 06/16] telepathy-glib: " Morgan Little
2012-07-19 17:46 ` [PATCH v2 07/16] telepathy-python: " Morgan Little
2012-07-19 17:46 ` [PATCH v2 08/16] libxml2: " Morgan Little
2012-07-19 17:46 ` [PATCH v2 09/16] gnome-doc-utils: " Morgan Little
2012-07-19 17:46 ` [PATCH v2 10/16] gobject-introspection: " Morgan Little
2012-07-19 17:46 ` [PATCH v2 11/16] mesa-common: " Morgan Little
2012-07-19 17:46 ` [PATCH v2 12/16] libxcb: " Morgan Little
2012-07-19 17:46 ` [PATCH v2 13/16] xcb-proto: " Morgan Little
2012-07-19 17:46 ` [PATCH v2 14/16] perf: " Morgan Little
2012-07-19 17:46 ` [PATCH v2 15/16] kernelshark: " Morgan Little
2012-07-19 17:46 ` [PATCH v2 16/16] trace-cmd: " Morgan Little
2012-07-20 16:16 ` Richard Purdie [this message]
2012-07-23 18:11 ` [PATCH v2 00/16] python-native: Put binaries in seperate directory Saul Wold

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=1342801015.21788.42.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.