[PATCH 03/12] android-wrapper: Add wget as prerequisite

Anton Lundin glance at acc.umu.se
Mon Feb 6 08:22:45 PST 2017


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 c1afcfb..03e4d5c 100644
> > --- a/packaging/android/android-build-wrapper.sh
> > +++ b/packaging/android/android-build-wrapper.sh
> > @@ -35,7 +35,7 @@ fi
> >  
> >  # make sure we have the required commands installed
> >  MISSING=""
> > -for i in git cmake autoconf libtool java ant; do
> > +for i in git cmake autoconf libtool java ant wget; do
> >  	which $i > /dev/null 2>&1 ||
> 
> Much better to use `command' (shell builtin and available in all POSIX 
> complient shells), instead of `which' (fork and possible not available 
> on all distributions):
> 
> 	command -v $i >/dev/null $$
> 
> >  		if [ "$i" = "libtool" ] ; then
> 		     ^  ^   ^       ^
> No quotes needed.
> 

I blame Dirk for those =)

Feel free to send patches!


//Anton


-- 
Anton Lundin	+46702-161604


More information about the subsurface mailing list