[gentoo-python] Re: [PATCH 2/3] Commonize out $(python_get_sitedir)/boost into a variable.

["Tiziano Müller" <dev-zero@gentoo.org>]

mostly bikeshedding and I am not sure whether I should support your
crusade against subshells: but ok

Am Freitag, den 14.12.2012, 23:08 +0100 schrieb Michał Górny:
> ---
>  gx86/dev-libs/boost/boost-1.52.0-r5.ebuild | 7 ++++---
>  1 file changed, 4 insertions(+), 3 deletions(-)
> 
> diff --git a/gx86/dev-libs/boost/boost-1.52.0-r5.ebuild b/gx86/dev-libs/boost/boost-1.52.0-r5.ebuild
> index 4f20fdf..87c7561 100644
> --- a/gx86/dev-libs/boost/boost-1.52.0-r5.ebuild
> +++ b/gx86/dev-libs/boost/boost-1.52.0-r5.ebuild
> @@ -205,9 +205,10 @@ src_install () {
>  			# Move mpi.so Python module to Python site-packages directory.
>  			# https://svn.boost.org/trac/boost/ticket/2838
>  			if use mpi; then
> -				dodir $(python_get_sitedir)/boost
> -				mv "${D}usr/$(get_libdir)/mpi.so" "${D}$(python_get_sitedir)/boost" || die
> -				cat << EOF > "${D}$(python_get_sitedir)/boost/__init__.py" || die
> +				local moddir=$(python_get_sitedir)/boost
> +				dodir "${moddir}"
> +				mv "${D}usr/$(get_libdir)/mpi.so" "${D}${moddir}" || die
> +				cat << EOF > "${D}${moddir}/__init__.py" || die
>  import sys
>  if sys.platform.startswith('linux'):
>  	import DLFCN