<br><div class="gmail_quote">On Fri, Aug 24, 2012 at 11:48 PM, McClintock Matthew-B29882 <span dir="ltr"><<a href="mailto:B29882@freescale.com" target="_blank">B29882@freescale.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="im">On Fri, Aug 24, 2012 at 11:15 AM, Richard Purdie<br>
<<a href="mailto:richard.purdie@linuxfoundation.org">richard.purdie@linuxfoundation.org</a>> wrote:<br>
</div><div class="im">> The last two distutils changes progressively broke the builds. Firstly they<br>
> moved things from the site_packages directory to being higher up the tree<br>
> which introduced package QA warnings as a side effect. Secondly, it interacts<br>
> badly with setuptools which passes in --root=${D} itself.<br>
><br>
> This patch restores the original directory layout, hence fixing the QA<br>
> warnings and also passes extra options to setuptools to deal with the<br>
> --root option it passes.<br>
<br></div></blockquote><div> </div><div>Exactly what i was working on... </div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">
</div>Hmm, I think you don't need certain bits anymore<br>
<div class="im"><br>
><br>
> Signed-off-by: Richard Purdie <<a href="mailto:richard.purdie@linuxfoundation.org">richard.purdie@linuxfoundation.org</a>><br>
> ---<br>
> diff --git a/meta/classes/distutils.bbclass b/meta/classes/distutils.bbclass<br>
> index 52a1aa8..c73b24f 100644<br>
> --- a/meta/classes/distutils.bbclass<br>
> +++ b/meta/classes/distutils.bbclass<br>
> @@ -38,7 +38,7 @@ distutils_do_install() {<br>
> STAGING_LIBDIR=${STAGING_LIBDIR} \<br>
> PYTHONPATH=${D}/${PYTHON_SITEPACKAGES_DIR} \<br>
> BUILD_SYS=${BUILD_SYS} HOST_SYS=${HOST_SYS} \<br>
> - ${STAGING_BINDIR_NATIVE}/python-native/python setup.py install --install-lib=${D}${libdir}/${PYTHON_DIR} ${DISTUTILS_INSTALL_ARGS} || \<br>
> + ${STAGING_BINDIR_NATIVE}/python-native/python setup.py install --install-lib=${D}/${PYTHON_SITEPACKAGES_DIR} ${DISTUTILS_INSTALL_ARGS} || \<br>
<br>
</div>This change should be handled by the bit below? So you could<br>
<br>
s/--install-lib=${D}/${PYTHON_SITEPACKAGES_DIR}//<br>
<div class="HOEnZb"><div class="h5"><br>
?<br>
<br>
> bbfatal "python setup.py install execution failed."<br>
><br>
> for i in `find ${D} -name "*.py"` ; do \<br>
> diff --git a/meta/classes/setuptools.bbclass b/meta/classes/setuptools.bbclass<br>
> index ced9509..ba9cf13 100644<br>
> --- a/meta/classes/setuptools.bbclass<br>
> +++ b/meta/classes/setuptools.bbclass<br>
> @@ -5,4 +5,5 @@ DEPENDS += "python-setuptools-native"<br>
> DISTUTILS_INSTALL_ARGS = "--root=${D} \<br>
> --single-version-externally-managed \<br>
> --prefix=${prefix} \<br>
> + --install-lib=${PYTHON_SITEPACKAGES_DIR} \<br>
> --install-data=${datadir}"<br></div></div></blockquote><div><br></div><div>Exactly my fix. Please merge.</div><div><br></div><div>Andrei </div></div>