diff options
| author | Chantry Xavier <shiningxc@gmail.com> | 2007-07-02 02:12:36 +0200 |
|---|---|---|
| committer | Dan McGee <dan@archlinux.org> | 2007-07-01 22:31:40 -0400 |
| commit | f55f7f100f650ff040f8721cf761a060af1c05fe (patch) | |
| tree | acfd01ac99594f8fa49e59782bce0c5e8598701a | |
| parent | cd3a63681338b7510846837dcc169d276cf75458 (diff) | |
| download | pacman-f55f7f100f650ff040f8721cf761a060af1c05fe.tar.xz | |
libalpm/add.c,util.c : log ARCHIVE_WARN warning to debug output.
Signed-off-by: Chantry Xavier <shiningxc@gmail.com>
| -rw-r--r-- | lib/libalpm/add.c | 12 | ||||
| -rw-r--r-- | lib/libalpm/util.c | 17 |
2 files changed, 23 insertions, 6 deletions
diff --git a/lib/libalpm/add.c b/lib/libalpm/add.c index 1ebf455a..c9bcea5e 100644 --- a/lib/libalpm/add.c +++ b/lib/libalpm/add.c @@ -545,7 +545,11 @@ int _alpm_add_commit(pmtrans_t *trans, pmdb_t *db) archive_entry_set_pathname(entry, tempfile); int ret = archive_read_extract(archive, entry, archive_flags); - if(ret != ARCHIVE_OK && ret != ARCHIVE_WARN) { + if(ret == ARCHIVE_WARN) { + /* operation succeeded but a non-critical error was encountered */ + _alpm_log(PM_LOG_DEBUG, _("warning extracting %s (%s)"), + entryname, archive_error_string(archive)); + } else if(ret != ARCHIVE_OK) { _alpm_log(PM_LOG_ERROR, _("could not extract %s (%s)"), entryname, archive_error_string(archive)); alpm_logaction(_("could not extract %s (%s)"), @@ -701,7 +705,11 @@ int _alpm_add_commit(pmtrans_t *trans, pmdb_t *db) archive_entry_set_pathname(entry, filename); int ret = archive_read_extract(archive, entry, archive_flags); - if(ret != ARCHIVE_OK && ret != ARCHIVE_WARN) { + if(ret == ARCHIVE_WARN) { + /* operation succeeded but a non-critical error was encountered */ + _alpm_log(PM_LOG_DEBUG, _("warning extracting %s (%s)"), + entryname, archive_error_string(archive)); + } else if(ret != ARCHIVE_OK) { _alpm_log(PM_LOG_ERROR, _("could not extract %s (%s)"), entryname, archive_error_string(archive)); alpm_logaction(_("could not extract %s (%s)"), diff --git a/lib/libalpm/util.c b/lib/libalpm/util.c index 66702691..17872429 100644 --- a/lib/libalpm/util.c +++ b/lib/libalpm/util.c @@ -327,15 +327,24 @@ int _alpm_unpack(const char *archive, const char *prefix, const char *fn) } while(archive_read_next_header(_archive, &entry) == ARCHIVE_OK) { - if (fn && strcmp(fn, archive_entry_pathname(entry))) { + const char *entryname; /* the name of the file in the archive */ + entryname = archive_entry_pathname(entry); + + if (fn && strcmp(fn, entryname)) { if (archive_read_data_skip(_archive) != ARCHIVE_OK) return(1); continue; } - snprintf(expath, PATH_MAX, "%s/%s", prefix, archive_entry_pathname(entry)); + snprintf(expath, PATH_MAX, "%s/%s", prefix, entryname); archive_entry_set_pathname(entry, expath); - if(archive_read_extract(_archive, entry, archive_flags) != ARCHIVE_OK) { - _alpm_log(PM_LOG_ERROR, _("could not extract %s: %s\n"), archive_entry_pathname(entry), archive_error_string(_archive)); + int ret = archive_read_extract(_archive, entry, archive_flags); + if(ret == ARCHIVE_WARN) { + /* operation succeeded but a non-critical error was encountered */ + _alpm_log(PM_LOG_DEBUG, _("warning extracting %s (%s)\n"), + entryname, archive_error_string(_archive)); + } else if(ret != ARCHIVE_OK) { + _alpm_log(PM_LOG_ERROR, _("could not extract %s (%s)\n"), + entryname, archive_error_string(_archive)); return(1); } |