diff options
author | Dan McGee <dan@archlinux.org> | 2008-10-31 19:11:38 -0500 |
---|---|---|
committer | Dan McGee <dan@archlinux.org> | 2008-10-31 19:46:13 -0500 |
commit | 8d4e1e67546bcc04b5412e0f94a752c7215fb53d (patch) | |
tree | f106fb7c19287e40b15b0d13b918b2a3e134477d /lib | |
parent | 314b4462d2ef1264b50b6745c57072f9c7b9d14e (diff) |
Make libfetch the 'native' download library
Use libfetch naming in the code in place of libdownload names. This is in
preparation for dropping support for libdownload at some point as libfetch
can run on Linux.
Signed-off-by: Dan McGee <dan@archlinux.org>
Diffstat (limited to 'lib')
-rw-r--r-- | lib/libalpm/alpm.h | 2 | ||||
-rw-r--r-- | lib/libalpm/dload.c | 35 | ||||
-rw-r--r-- | lib/libalpm/error.c | 6 |
3 files changed, 21 insertions, 22 deletions
diff --git a/lib/libalpm/alpm.h b/lib/libalpm/alpm.h index 80a04262..eda35d30 100644 --- a/lib/libalpm/alpm.h +++ b/lib/libalpm/alpm.h @@ -526,7 +526,7 @@ enum _pmerrno_t { PM_ERR_INVALID_REGEX, /* External library errors */ PM_ERR_LIBARCHIVE, - PM_ERR_LIBDOWNLOAD, + PM_ERR_LIBFETCH, PM_ERR_EXTERNAL_DOWNLOAD }; diff --git a/lib/libalpm/dload.c b/lib/libalpm/dload.c index c07040b5..e4a8a0b0 100644 --- a/lib/libalpm/dload.c +++ b/lib/libalpm/dload.c @@ -35,14 +35,14 @@ #if defined(HAVE_LIBDOWNLOAD) #include <download.h> +#define fetchFreeURL downloadFreeURL +#define fetchLastErrCode downloadLastErrCode +#define fetchLastErrString downloadLastErrString +#define fetchParseURL downloadParseURL +#define fetchTimeout downloadTimeout +#define fetchXGet downloadXGet #elif defined(HAVE_LIBFETCH) #include <fetch.h> -#define downloadFreeURL fetchFreeURL -#define downloadLastErrCode fetchLastErrCode -#define downloadLastErrString fetchLastErrString -#define downloadParseURL fetchParseURL -#define downloadTimeout fetchTimeout -#define downloadXGet fetchXGet #endif /* libalpm */ @@ -86,7 +86,7 @@ static char *get_tempfile(const char *path, const char *filename) { static struct url *url_for_string(const char *url) { struct url *ret = NULL; - ret = downloadParseURL(url); + ret = fetchParseURL(url); if(!ret) { _alpm_log(PM_LOG_ERROR, _("url '%s' is invalid\n"), url); RET_ERR(PM_ERR_SERVER_BAD_URL, NULL); @@ -142,23 +142,22 @@ static int download_internal(const char *url, const char *localpath, dl_thisfile = 0; } - /* libdownload does not reset the error code, reset it in - * the case of previous errors */ - downloadLastErrCode = 0; + /* libfetch does not reset the error code */ + fetchLastErrCode = 0; /* 10s timeout - TODO make a config option */ - downloadTimeout = 10000; + fetchTimeout = 10000; - dlf = downloadXGet(fileurl, &ust, (handle->nopassiveftp ? "" : "p")); + dlf = fetchXGet(fileurl, &ust, (handle->nopassiveftp ? "" : "p")); - if(downloadLastErrCode != 0 || dlf == NULL) { + if(fetchLastErrCode != 0 || dlf == NULL) { const char *host = _("disk"); if(strcmp(SCHEME_FILE, fileurl->scheme) != 0) { host = fileurl->host; } - pm_errno = PM_ERR_LIBDOWNLOAD; + pm_errno = PM_ERR_LIBFETCH; _alpm_log(PM_LOG_ERROR, _("failed retrieving file '%s' from %s : %s\n"), - filename, host, downloadLastErrString); + filename, host, fetchLastErrString); ret = -1; goto cleanup; } else { @@ -204,9 +203,9 @@ static int download_internal(const char *url, const char *localpath, char buffer[PM_DLBUF_LEN]; while((nread = fread(buffer, 1, PM_DLBUF_LEN, dlf)) > 0) { if(ferror(dlf)) { - pm_errno = PM_ERR_LIBDOWNLOAD; + pm_errno = PM_ERR_LIBFETCH; _alpm_log(PM_LOG_ERROR, _("error downloading '%s': %s\n"), - filename, downloadLastErrString); + filename, fetchLastErrString); ret = -1; goto cleanup; } @@ -247,7 +246,7 @@ cleanup: if(dlf != NULL) { fclose(dlf); } - downloadFreeURL(fileurl); + fetchFreeURL(fileurl); return(ret); } #endif diff --git a/lib/libalpm/error.c b/lib/libalpm/error.c index 05caf8ec..80a4fb1a 100644 --- a/lib/libalpm/error.c +++ b/lib/libalpm/error.c @@ -32,9 +32,9 @@ #if defined(HAVE_LIBDOWNLOAD) #include <download.h> /* downloadLastErrString */ +#define fetchLastErrString downloadLastErrString #elif defined(HAVE_LIBFETCH) #include <fetch.h> /* fetchLastErrString */ -#define downloadLastErrString fetchLastErrString #endif /* libalpm */ @@ -154,9 +154,9 @@ const char SYMEXPORT *alpm_strerror(int err) * requires the archive struct, so we can't. Just use a generic * error string instead. */ return _("libarchive error"); - case PM_ERR_LIBDOWNLOAD: + case PM_ERR_LIBFETCH: #if defined(INTERNAL_DOWNLOAD) - return downloadLastErrString; + return fetchLastErrString; #else /* obviously shouldn't get here... */ return _("download library error"); |