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

> On Mon, 6 Feb 2017, Anton Lundin wrote:
> > 
> > Signed-off-by: Anton Lundin <[email protected]>
> > ---
> >  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
_______________________________________________
subsurface mailing list
[email protected]
http://lists.subsurface-divelog.org/cgi-bin/mailman/listinfo/subsurface

Reply via email to