[Rpm-maint] [rpm-software-management/rpm] Automatically create users and groups from sysusers.d(5) files (PR #2432)

[Florian Festi]

@ffesti commented on this pull request.



> @@ -645,6 +663,20 @@ rpmScript rpmScriptFromTriggerTag(Header h, rpmTagVal triggerTag,
     return script;
 }
 
+rpmScript rpmScriptFromArgv(Header h, ARGV_t argv, rpmscriptFlags flags, int chroot)

It's kinda weird to have a generic name like `rpmScriptFromArgv` but still hard code  the `scriptTag` to `RPMTAG_SYSUSERS`. May be the scriptTag should be a parameter.

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

Message ID: <rpm-software-management/rpm/pull/2432/review/1360882871 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.rpm.org/pipermail/rpm-maint/attachments/20230328/fbbec9c8/attachment-0001.html>