Skip to content

csmock: install SRPM as mockbuild user#206

Merged
sfowl merged 1 commit intomainfrom
mockbuild-install-srpm
Aug 4, 2025
Merged

csmock: install SRPM as mockbuild user#206
sfowl merged 1 commit intomainfrom
mockbuild-install-srpm

Conversation

@sfowl
Copy link
Contributor

@sfowl sfowl commented Jul 30, 2025

During a csmock run, there are two uses of rpm -Uvh <srpm> which install the sources into the build dir. The first runs as mockbuild user, whilst the second ran as root. This second run installed files owned by root, which could prevent some file operations performed during the rpm build from succeeding.

PSSECAUT-1201

During a csmock run, there are two uses of `rpm -Uvh <srpm>` which
install the sources into the build dir. The first runs as mockbuild
user, whilst the second ran as root. This second run installed files
owned by root, which could prevent some file operations performed
during the rpm build from succeeding.
@sfowl sfowl requested a review from kdudka July 30, 2025 06:12
Copy link
Member

@kdudka kdudka left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good. Thanks!

@sfowl sfowl merged commit 2a76525 into main Aug 4, 2025
42 checks passed
@kdudka kdudka deleted the mockbuild-install-srpm branch August 5, 2025 08:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants