[pacman-dev] [PATCH] makepkg: always check makepkg.conf for match to download protocol

Allan McRae allan at archlinux.org
Mon Aug 27 23:18:27 EDT 2012


The list of which download protocol should look in makepkg.conf for the
download agent was hard coded into makepkg.  Instead, fallback to checking
the the download agents array for any non-local or (implemented) vcs
source.

Signed-off-by: Allan McRae <allan at archlinux.org>
---
 scripts/makepkg.sh.in | 9 ++-------
 1 file changed, 2 insertions(+), 7 deletions(-)

diff --git a/scripts/makepkg.sh.in b/scripts/makepkg.sh.in
index cad1f5e..7e5aa95 100644
--- a/scripts/makepkg.sh.in
+++ b/scripts/makepkg.sh.in
@@ -284,7 +284,7 @@ get_downloadclient() {
 
 	# if we didn't find an agent, return an error
 	if [[ -z $agent ]]; then
-		error "$(gettext "There is no agent set up to handle %s URLs. Check %s.")" "$proto" "$MAKEPKG_CONF"
+		error "$(gettext "Unknown download protocol: %s")" "$proto"
 		plain "$(gettext "Aborting...")"
 		exit 1 # $E_CONFIG_ERROR
 	fi
@@ -596,9 +596,6 @@ download_sources() {
 			local)
 				download_local "$netfile"
 				;;
-			ftp|http|https|rsync|scp)
-				download_file "$netfile"
-				;;
 			git*)
 				(( GET_VCS )) && download_git "$netfile"
 				;;
@@ -609,9 +606,7 @@ download_sources() {
 				(( GET_VCS )) && download_svn "$netfile"
 				;;
 			*)
-				error "$(gettext "Unknown download protocol: %s")" "$proto"
-				plain "$(gettext "Aborting...")"
-				exit 1
+				download_file "$netfile"
 				;;
 		esac
 	done
-- 
1.7.12



More information about the pacman-dev mailing list