Skip to content

Commit

Permalink
Don't warn about missing user/group on skipped files
Browse files Browse the repository at this point in the history
There's no reason to complain about missing user/group for entities
we don't create at all. It's cosmetical only, but "regressed" in the
4.17 fsm robustness rewrite.

Reported in https://issues.redhat.com/browse/RHEL-18037
  • Loading branch information
pmatilai authored and ffesti committed Dec 13, 2023
1 parent 386ccfc commit 6c66abd
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion lib/fsm.c
Original file line number Diff line number Diff line change
Expand Up @@ -903,7 +903,7 @@ int rpmPackageFilesInstall(rpmts ts, rpmte te, rpmfiles files,
fp->fpath = fsmFsPath(fi, fp->suffix);

/* Remap file perms, owner, and group. */
rc = rpmfiStat(fi, 1, &fp->sb);
rc = rpmfiStat(fi, (fp->skip == 0), &fp->sb);

/* Hardlinks are tricky and handled elsewhere for install */
fp->setmeta = (fp->skip == 0) &&
Expand Down

0 comments on commit 6c66abd

Please sign in to comment.