Browse Source

show dltool (curl/wget) errors when make package=foo clean package ADK_VERBOSE=1 is used

Waldemar Brodkorb 7 years ago
parent
commit
10e3a2834c
3 changed files with 3 additions and 13 deletions
  1. 1 1
      mk/fetch.mk
  2. 2 0
      rules.mk
  3. 0 12
      scripts/prereq.sh

+ 1 - 1
mk/fetch.mk

@@ -89,7 +89,7 @@ $(1):
 		for site in $${PKG_SITES} $${MASTER_SITE_BACKUP}; do \
 			: echo "$${FETCHCMD} $$$$site$$$$filename"; \
 			rm -f "$$$$filename"; \
-			if $${FETCHCMD} $$$$filename $$$$site$$$$filename; then \
+			if $${FETCHCMD} $$$$filename $$$$site$$$$filename $(DL_TRACE); then \
 				: check the size here; \
 				[[ ! -e $$$$filename ]] || exit 0; \
 			fi; \

+ 2 - 0
rules.mk

@@ -11,6 +11,7 @@ TRACE:=			:
 CMD_TRACE:=		:
 PKG_TRACE:=		:
 MAKE_TRACE:=
+DL_TRACE:=
 EXTRA_MAKEFLAGS:=
 SET_DASHX:=		set -x
 else
@@ -21,6 +22,7 @@ CMD_TRACE:=		echo -n
 PKG_TRACE:=		echo "------> "
 EXTRA_MAKEFLAGS:=	-s
 MAKE_TRACE:=		>/dev/null 2>&1 || { echo "Build failed. Please re-run make with v to see what's going on"; false; }
+DL_TRACE:=		>/dev/null 2>&1
 SET_DASHX:=		:
 endif
 

+ 0 - 12
scripts/prereq.sh

@@ -42,10 +42,6 @@ if [ -e foo ]; then
 fi
 rm -f FOO
 
-if [ "$target" = "v" ]; then
-  dlverbose=1
-fi
-
 # do we have a download tool?
 tools="curl wget"
 for tool in $tools; do
@@ -54,18 +50,10 @@ for tool in $tools; do
     printf "found\n"
     case $tool in
       curl)
-        if [ $dlverbose -eq 1 ]; then
           FETCHCMD="$(which $tool) -L -k -f -\# -o "
-        else
-          FETCHCMD="$(which $tool) -L -k -f -s -o "
-        fi
         ;;
       wget)
-        if [ $dlverbose -eq 1 ]; then
           FETCHCMD="$(which $tool) --no-check-certificate -O "
-        else
-          FETCHCMD="$(which $tool) --no-check-certificate -q -O "
-        fi
         ;;
     esac
     break