mirror of
https://github.com/moparisthebest/pacman
synced 2025-03-01 01:41:52 -05:00
get_filename : use the FILENAME db field only.
Reference : FS#9547. The get_filename function first tries to get the filename field from the database, and if it doesn't find it, it tries to guess it based on the name, version and arch. This field was introduced in 3.0, but there are still many old entries in the official databases without it. So the databases need to be regenerated first before this patch can be applied. There is a second problem with the delta code, which needs the filename for locally installed packages too, but this field is not present in the local db. So the delta code needs to be fixed first. Signed-off-by: Chantry Xavier <shiningxc@gmail.com> Signed-off-by: Dan McGee <dan@archlinux.org>
This commit is contained in:
parent
8fdf08ef78
commit
481c3edc89
@ -152,19 +152,6 @@ const char SYMEXPORT *alpm_pkg_get_filename(pmpkg_t *pkg)
|
||||
_alpm_db_read(pkg->origin_data.db, pkg, INFRQ_DESC);
|
||||
}
|
||||
|
||||
if(pkg->filename == NULL || strlen(pkg->filename) == 0) {
|
||||
/* construct the file name, it's not in the desc file */
|
||||
char buffer[PATH_MAX];
|
||||
if(pkg->arch && strlen(pkg->arch) > 0) {
|
||||
snprintf(buffer, PATH_MAX, "%s-%s-%s" PKGEXT,
|
||||
pkg->name, pkg->version, pkg->arch);
|
||||
} else {
|
||||
snprintf(buffer, PATH_MAX, "%s-%s" PKGEXT,
|
||||
pkg->name, pkg->version);
|
||||
}
|
||||
STRDUP(pkg->filename, buffer, RET_ERR(PM_ERR_MEMORY, NULL));
|
||||
}
|
||||
|
||||
return pkg->filename;
|
||||
}
|
||||
|
||||
|
@ -819,6 +819,7 @@ int _alpm_sync_commit(pmtrans_t *trans, pmdb_t *db_local, alpm_list_t **data)
|
||||
const char *fname = NULL;
|
||||
|
||||
fname = alpm_pkg_get_filename(spkg);
|
||||
ASSERT(fname != NULL, RET_ERR(PM_ERR_PKG_INVALID_NAME, -1));
|
||||
if(trans->flags & PM_TRANS_FLAG_PRINTURIS) {
|
||||
EVENT(trans, PM_TRANS_EVT_PRINTURI, (char *)alpm_db_get_url(current),
|
||||
(char *)fname);
|
||||
@ -985,6 +986,9 @@ int _alpm_sync_commit(pmtrans_t *trans, pmdb_t *db_local, alpm_list_t **data)
|
||||
char *fpath;
|
||||
|
||||
fname = alpm_pkg_get_filename(spkg);
|
||||
if(fname == NULL) {
|
||||
goto error;
|
||||
}
|
||||
/* Loop through the cache dirs until we find a matching file */
|
||||
fpath = _alpm_filecache_find(fname);
|
||||
|
||||
|
Loading…
x
Reference in New Issue
Block a user