From fa117686c873b162eca85e324dd988b292541cac Mon Sep 17 00:00:00 2001
From: Gabriel Pettier <gabriel.pettier@gmail.com>
Date: Sun, 15 Sep 2019 20:13:04 +0200
Subject: [PATCH] fix: show output of sdk update if auto_accept_license is
 false

---
 buildozer/targets/android.py | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/buildozer/targets/android.py b/buildozer/targets/android.py
index 7e677db..8438573 100644
--- a/buildozer/targets/android.py
+++ b/buildozer/targets/android.py
@@ -515,6 +515,7 @@ class TargetAndroid(Target):
         auto_accept_license = self.buildozer.config.getbooldefault(
             'app', 'android.accept_sdk_license', False)
 
+        kwargs = {}
         if auto_accept_license:
             # `SIGPIPE` is not being reported somehow, but `EPIPE` is.
             # This leads to a stderr "Broken pipe" message which is harmless,
@@ -523,7 +524,10 @@ class TargetAndroid(Target):
             command = '{} | {} --licenses'.format(
                 yes_command, self.sdkmanager_path)
             self.buildozer.cmd(command, cwd=self.android_sdk_dir)
-        self._sdkmanager(*sdkmanager_commands)
+        else:
+            kwargs['show_output'] = True
+
+        self._sdkmanager(*sdkmanager_commands, **kwargs)
 
     def _read_version_subdir(self, *args):
         versions = []