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

> On Mon, 6 Feb 2017, Anton Lundin wrote:
> > 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.
> 
> I guess you've tested `yes', that's why you know ;)

Yep =)

-- 
Anton Lundin    +46702-161604
_______________________________________________
subsurface mailing list
[email protected]
http://lists.subsurface-divelog.org/cgi-bin/mailman/listinfo/subsurface

Reply via email to