[gnome-continuous-yocto/gnomeostree-3.28-rocko: 1877/8267] classes/populate_sdk_ext: properly handle buildtools install failure

[Emmanuele Bassi <ebassi src gnome org>]

commit 5895bb6d2c1cbc38872716ec2c03c56990cb164c
Author: Paul Eggleton <paul eggleton linux intel com>
Date:   Thu Aug 11 16:45:03 2016 +1200

    classes/populate_sdk_ext: properly handle buildtools install failure
    
    If the buildtools installation failed, we were using a subshell instead
    of a compound command and thus the subshell exited but the script
    continued on, which is really not what we want to happen. Additionally
    log the buildtools installer output to a file and cat it if it fails so
    that you can actually see what went wrong, as well as amending the
    environment setup script to print a warning as we do when the
    preparation fails.
    
    (From OE-Core rev: 8fb8adf309823660c3943df973c216621a71850d)
    
    Signed-off-by: Paul Eggleton <paul eggleton linux intel com>
    Signed-off-by: Ross Burton <ross burton intel com>
    Signed-off-by: Richard Purdie <richard purdie linuxfoundation org>

 meta/classes/populate_sdk_ext.bbclass |    6 ++++--
 1 files changed, 4 insertions(+), 2 deletions(-)
---
diff --git a/meta/classes/populate_sdk_ext.bbclass b/meta/classes/populate_sdk_ext.bbclass
index a075a09..1801356 100644
--- a/meta/classes/populate_sdk_ext.bbclass
+++ b/meta/classes/populate_sdk_ext.bbclass
@@ -490,14 +490,16 @@ SDK_PRE_INSTALL_COMMAND_task-populate-sdk-ext = "${sdk_ext_preinst}"
 sdk_ext_postinst() {
        printf "\nExtracting buildtools...\n"
        cd $target_sdk_dir
-       printf "buildtools\ny" | ./*buildtools-nativesdk-standalone* > /dev/null || ( printf 'ERROR: 
buildtools installation failed\n' ; exit 1 )
+       env_setup_script="$target_sdk_dir/environment-setup-${REAL_MULTIMACH_TARGET_SYS}"
+       printf "buildtools\ny" | ./*buildtools-nativesdk-standalone* > buildtools.log || { printf 'ERROR: 
buildtools installation failed:\n' ; cat buildtools.log ; echo "printf 'ERROR: this SDK was not fully 
installed and needs reinstalling\n'" >> $env_setup_script ; exit 1 ; }
 
        # Delete the buildtools tar file since it won't be used again
        rm ./*buildtools-nativesdk-standalone*.sh -f
+       # We don't need the log either since it succeeded
+       rm -f buildtools.log
 
        # Make sure when the user sets up the environment, they also get
        # the buildtools-tarball tools in their path.
-       env_setup_script="$target_sdk_dir/environment-setup-${REAL_MULTIMACH_TARGET_SYS}"
        echo ". $target_sdk_dir/buildtools/environment-setup*" >> $env_setup_script
 
        # Allow bitbake environment setup to be ran as part of this sdk.