There was a lot of confusion regarding these warnings, particularly for packages that create users post_install and then chown the directories. Signed-off-by: Allan McRae <allan@archlinux.org> --- lib/libalpm/add.c | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/lib/libalpm/add.c b/lib/libalpm/add.c index 82461ee..a6b83f5 100644 --- a/lib/libalpm/add.c +++ b/lib/libalpm/add.c @@ -231,6 +231,12 @@ static int extract_single_file(alpm_handle_t *handle, struct archive *archive, gid_t entrygid = archive_entry_gid(entry); /* case 6: existing dir, ignore it */ + +#if 0 + /* Disable these warnings until our user management has improved. Currently + many packages have to create users in post_install and chown the directories. + These all resulted in "false-positive" warnings. */ + if(lsbuf.st_mode != entrymode) { /* if filesystem perms are different than pkg perms, warn user */ mode_t mask = 07777; @@ -252,6 +258,7 @@ static int extract_single_file(alpm_handle_t *handle, struct archive *archive, "filesystem: %u:%u package: %u:%u\n", filename, lsbuf.st_uid, lsbuf.st_gid, entryuid, entrygid); } +#endif _alpm_log(handle, ALPM_LOG_DEBUG, "extract: skipping dir extraction of %s\n", filename); -- 2.2.2