xbps_transaction_files: report all conflicting files not just the first.
This commit is contained in:
parent
0f69773273
commit
9a72aaf577
@ -518,6 +518,7 @@ collect_files(struct xbps_handle *xhp, xbps_dictionary_t d,
|
||||
unsigned int i;
|
||||
int rv = 0;
|
||||
const char *file, *sha256 = NULL;
|
||||
bool error = false;
|
||||
|
||||
if ((a = xbps_dictionary_get(d, "files"))) {
|
||||
for (i = 0; i < xbps_array_count(a); i++) {
|
||||
@ -529,10 +530,14 @@ collect_files(struct xbps_handle *xhp, xbps_dictionary_t d,
|
||||
xbps_dictionary_get_uint64(filed, "size", &size);
|
||||
rv = collect_file(xhp, file, size, pkgname, pkgver, idx, sha256,
|
||||
TYPE_FILE, update, preserve, remove);
|
||||
if (rv != 0)
|
||||
if (rv == EEXIST) {
|
||||
error = true;
|
||||
continue;
|
||||
} else if (rv != 0) {
|
||||
goto out;
|
||||
}
|
||||
}
|
||||
}
|
||||
if ((a = xbps_dictionary_get(d, "conf_files"))) {
|
||||
for (i = 0; i < xbps_array_count(a); i++) {
|
||||
filed = xbps_array_get(a, i);
|
||||
@ -548,32 +553,47 @@ collect_files(struct xbps_handle *xhp, xbps_dictionary_t d,
|
||||
#endif
|
||||
rv = collect_file(xhp, file, size, pkgname, pkgver, idx, sha256,
|
||||
TYPE_FILE, update, preserve, remove);
|
||||
if (rv != 0)
|
||||
if (rv == EEXIST) {
|
||||
error = true;
|
||||
continue;
|
||||
} else if (rv != 0) {
|
||||
goto out;
|
||||
}
|
||||
}
|
||||
}
|
||||
if ((a = xbps_dictionary_get(d, "links"))) {
|
||||
for (i = 0; i < xbps_array_count(a); i++) {
|
||||
filed = xbps_array_get(a, i);
|
||||
xbps_dictionary_get_cstring_nocopy(filed, "file", &file);
|
||||
rv = collect_file(xhp, file, 0, pkgname, pkgver, idx, NULL,
|
||||
TYPE_LINK, update, preserve, remove);
|
||||
if (rv != 0)
|
||||
if (rv == EEXIST) {
|
||||
error = true;
|
||||
continue;
|
||||
} else if (rv != 0) {
|
||||
goto out;
|
||||
}
|
||||
}
|
||||
}
|
||||
if ((a = xbps_dictionary_get(d, "dirs"))) {
|
||||
for (i = 0; i < xbps_array_count(a); i++) {
|
||||
filed = xbps_array_get(a, i);
|
||||
xbps_dictionary_get_cstring_nocopy(filed, "file", &file);
|
||||
rv = collect_file(xhp, file, 0, pkgname, pkgver, idx, NULL,
|
||||
TYPE_DIR, update, preserve, remove);
|
||||
if (rv != 0)
|
||||
if (rv == EEXIST) {
|
||||
error = true;
|
||||
continue;
|
||||
} else if (rv != 0) {
|
||||
goto out;
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
out:
|
||||
if (error)
|
||||
rv = EEXIST;
|
||||
|
||||
return rv;
|
||||
}
|
||||
|
||||
|
Loading…
Reference in New Issue
Block a user