[OE-core] [PATCH v3 1/6] toolchain-scripts: retab file

[Peter Kjellerstedt]

> -----Original Message-----
> From: openembedded-core-bounces at lists.openembedded.org
> [mailto:openembedded-core-bounces at lists.openembedded.org] On Behalf Of
> Martin Kelly
> Sent: den 5 juni 2018 01:06
> To: openembedded-core at lists.openembedded.org
> Subject: [OE-core] [PATCH v3 1/6] toolchain-scripts: retab file
> 
> Two functions is uses a mix of spaces and tabs. The rest of the file
> uses tabs, so switch to tabs uniformly.
> 
> Signed-off-by: Martin Kelly <mkelly at xevo.com>
> ---
>  meta/classes/toolchain-scripts.bbclass | 20 ++++++++++----------
>  1 file changed, 10 insertions(+), 10 deletions(-)
> 
> diff --git a/meta/classes/toolchain-scripts.bbclass b/meta/classes/toolchain-scripts.bbclass
> index 71da5e5409..a72436167c 100644
> --- a/meta/classes/toolchain-scripts.bbclass
> +++ b/meta/classes/toolchain-scripts.bbclass
> @@ -122,7 +122,7 @@ toolchain_create_post_relocate_script() {
>  	rm -f $script
>  	touch $script
> 
> -    cat >> $script <<EOF
> +	cat >> $script <<EOF
>  if [ -d "${SDKPATHNATIVE}/post-relocate-setup.d/" ]; then
>      for s in ${SDKPATHNATIVE}/post-relocate-setup.d/*.sh; do
>          \$s "\$1"

This part should be indented using tabs as well.

> @@ -166,13 +166,13 @@ toolchain_create_sdk_siteconfig () {
>  toolchain_create_sdk_siteconfig[vardepsexclude] = "TOOLCHAIN_CONFIGSITE_SYSROOTCACHE"
> 
>  python __anonymous () {
> -    import oe.classextend
> -    deps = ""
> -    for dep in (d.getVar('TOOLCHAIN_NEED_CONFIGSITE_CACHE') or "").split():
> -        deps += " %s:do_populate_sysroot" % dep
> -        for variant in (d.getVar('MULTILIB_VARIANTS') or "").split():
> -            clsextend = oe.classextend.ClassExtender(variant, d)
> -            newdep = clsextend.extend_name(dep)
> -            deps += " %s:do_populate_sysroot" % newdep
> -    d.appendVarFlag('do_configure', 'depends', deps)
> +	import oe.classextend
> +	deps = ""
> +	for dep in (d.getVar('TOOLCHAIN_NEED_CONFIGSITE_CACHE') or "").split():
> +		deps += " %s:do_populate_sysroot" % dep
> +		for variant in (d.getVar('MULTILIB_VARIANTS') or "").split():
> +			clsextend = oe.classextend.ClassExtender(variant, d)
> +			newdep = clsextend.extend_name(dep)
> +			deps += " %s:do_populate_sysroot" % newdep
> +	d.appendVarFlag('do_configure', 'depends', deps)

This is Python code. I believe the OE-Core standard is for Python code 
to be indented using four spaces (whereas shell code should be indented 
using tabs).

>  }
> --
> 2.11.0

//Peter