[Rpm-maint] [rpm-software-management/rpm] Use runuser instead of sudo for test-suite user changes (PR #3553)

Panu Matilainen notifications at github.com
Fri Jan 31 15:15:08 UTC 2025


We don't need or want any of the sudo fanciness and privilege elevation here: we're already root inside the container and just want to run things as a different user. runuser is more than adequate for this purpose.

One such peculiarity is that apparently sudo re-expands its command, so \$PWD got evaluated to the PWD of the new shell. runuser does no such thing, but we can just as well hardcode a directory, $PWD is nothing special in this case.

Consolidate the lone runuser rpmdb export test to use runroot_user too now that it can.
You can view, comment on, or merge this pull request online at:

  https://github.com/rpm-software-management/rpm/pull/3553

-- Commit Summary --

  * Use runuser instead of sudo for test-suite user changes

-- File Changes --

    M tests/README.md (2)
    M tests/atlocal.in (2)
    M tests/rpmdb.at (4)
    M tests/rpmi.at (8)

-- Patch Links --

https://github.com/rpm-software-management/rpm/pull/3553.patch
https://github.com/rpm-software-management/rpm/pull/3553.diff

-- 
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/pull/3553
You are receiving this because you are subscribed to this thread.

Message ID: <rpm-software-management/rpm/pull/3553 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.rpm.org/pipermail/rpm-maint/attachments/20250131/5416becf/attachment.htm>


More information about the Rpm-maint mailing list