summaryrefslogtreecommitdiff
path: root/lib/libalpm/package.c
diff options
context:
space:
mode:
authorAaron Griffin <aaron@archlinux.org>2007-02-01 03:00:34 +0000
committerAaron Griffin <aaron@archlinux.org>2007-02-01 03:00:34 +0000
commit21e19a7bcffac1ae2ecddbf321ad752c4b0f6692 (patch)
tree41ddccc6885447e1c56d60404789eb0dfd6b6028 /lib/libalpm/package.c
parent16e01cfe736d348b4f8276f54ebe0efa020db058 (diff)
downloadpacman-21e19a7bcffac1ae2ecddbf321ad752c4b0f6692.tar.xz
Loading package data from a file was ALWAYS generating the filelist, instead of
using the in-package one. This is now fixed, and an autogenerated one is used as a last resort. This fixes the bug where /.CHANGELOG showed up in -Ql.
Diffstat (limited to 'lib/libalpm/package.c')
-rw-r--r--lib/libalpm/package.c39
1 files changed, 22 insertions, 17 deletions
diff --git a/lib/libalpm/package.c b/lib/libalpm/package.c
index 6951bf7d..84db3719 100644
--- a/lib/libalpm/package.c
+++ b/lib/libalpm/package.c
@@ -241,11 +241,12 @@ pmpkg_t *_alpm_pkg_load(char *pkgfile)
int config = 0;
int filelist = 0;
int scriptcheck = 0;
- register struct archive *archive;
+ struct archive *archive;
struct archive_entry *entry;
pmpkg_t *info = NULL;
char *descfile = NULL;
int fd = -1;
+ alpm_list_t *all_files = NULL;
ALPM_LOG_FUNC;
@@ -272,11 +273,14 @@ pmpkg_t *_alpm_pkg_load(char *pkgfile)
* from a libarchive archive, it can be done by reading
* directly from the archive */
for(i = 0; archive_read_next_header (archive, &entry) == ARCHIVE_OK; i++) {
+ const char *entry_name = archive_entry_pathname(entry);
+
if(config && filelist && scriptcheck) {
/* we have everything we need */
break;
}
- if(strcmp(archive_entry_pathname (entry), ".PKGINFO") == 0) {
+
+ if(strcmp(entry_name, ".PKGINFO") == 0) {
/* extract this file into /tmp. it has info for us */
descfile = strdup("/tmp/alpm_XXXXXX");
fd = mkstemp(descfile);
@@ -299,10 +303,10 @@ pmpkg_t *_alpm_pkg_load(char *pkgfile)
FREE(descfile);
close(fd);
continue;
- } else if(strcmp(archive_entry_pathname (entry), ".INSTALL") == 0) {
+ } else if(strcmp(entry_name, ".INSTALL") == 0) {
info->scriptlet = 1;
scriptcheck = 1;
- } else if(strcmp(archive_entry_pathname (entry), ".FILELIST") == 0) {
+ } else if(strcmp(entry_name, ".FILELIST") == 0) {
/* Build info->files from the filelist */
FILE *fp;
char *fn;
@@ -314,7 +318,7 @@ pmpkg_t *_alpm_pkg_load(char *pkgfile)
}
fn = strdup("/tmp/alpm_XXXXXX");
fd = mkstemp(fn);
- archive_read_data_into_fd (archive,fd);
+ archive_read_data_into_fd(archive,fd);
fp = fopen(fn, "r");
while(!feof(fp)) {
if(fgets(str, PATH_MAX, fp) == NULL) {
@@ -334,28 +338,29 @@ pmpkg_t *_alpm_pkg_load(char *pkgfile)
continue;
} else {
scriptcheck = 1;
- if(!filelist) {
- /* no .FILELIST present in this package.. build the filelist the */
- /* old-fashioned way, one at a time */
- expath = strdup(archive_entry_pathname (entry));
- info->files = alpm_list_add(info->files, expath);
- }
+ /* Keep track of all files so we can generate a filelist later if missing */
+ all_files = alpm_list_add(all_files, strdup(entry_name));
}
- if(archive_read_data_skip (archive)) {
+ if(archive_read_data_skip(archive)) {
_alpm_log(PM_LOG_ERROR, _("bad package file in %s"), pkgfile);
goto error;
}
expath = NULL;
}
- archive_read_finish (archive);
+ archive_read_finish(archive);
if(!config) {
_alpm_log(PM_LOG_ERROR, _("missing package info file in %s"), pkgfile);
goto error;
- } else if(!filelist) {
- _alpm_log(PM_LOG_ERROR, _("missing package filelist in %s"), pkgfile);
- goto error;
+ }
+
+ if(!filelist) {
+ _alpm_log(PM_LOG_ERROR, _("missing package filelist in %s, generating one"), pkgfile);
+ info->files = all_files;
+ } else {
+ alpm_list_free_inner(all_files, free);
+ alpm_list_free(all_files);
}
/* internal */
@@ -376,7 +381,7 @@ pkg_invalid:
}
error:
FREEPKG(info);
- archive_read_finish (archive);
+ archive_read_finish(archive);
return(NULL);
}