[PATCH 07/12] android-wrapper: Auto accept android sdk license

[Anton Lundin]

On 06 February, 2017 - Cristian Ionescu-Idbohrn wrote:

> On Mon, 6 Feb 2017, Anton Lundin wrote:
> > 
> > Signed-off-by: Anton Lundin <glance at acc.umu.se>
> > ---
> >  packaging/android/android-build-wrapper.sh | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> > 
> > diff --git a/packaging/android/android-build-wrapper.sh b/packaging/android/android-build-wrapper.sh
> > index c8dd1e1..6242809 100644
> > --- a/packaging/android/android-build-wrapper.sh
> > +++ b/packaging/android/android-build-wrapper.sh
> > @@ -82,7 +82,7 @@ if [ ! -d $ANDROID_SDK ] ; then
> >  	mkdir $ANDROID_SDK
> >  	pushd $ANDROID_SDK
> >  	unzip -q ../$SDK_TOOLS
> > -	bash tools/android update sdk --no-ui -a -t 1,2,3,33
> > +	( sleep 5 && while true ; do sleep 1; echo y; done ) | bash tools/android update sdk --no-ui -a -t 1,2,3,33
> 
> Can it be so that the simplier:
> 
> 	yes y | bash tools/android update sdk --no-ui -a -t 1,2,3,33
> 
> is all you need?

Nope. sdk updater does read(0, dest, 1024) or something, so if yes have
pushed too many y\n into the buffer, it will fail when doing
strcmp(dest, "y\n") or something alike.

Thats why odd one here.


//Anton


-- 
Anton Lundin	+46702-161604