@pmatilai commented on this pull request.


>      if arg[1] == 'g' then
         type = 'group'
     elseif arg[1] == 'u' then
         type = 'user'
-    elseif arg[1] == 'r' or arg[1] == 'm' then
+    elseif arg[1] == 'm' and #arg >=3 then
+        type = 'usergroup'

'groupmember' might be a more obvious name for what it does. Wouldn't matter a 
bit in a script internals, but since these are visible in the package 
dependencies...

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

Message ID: <rpm-software-management/rpm/pull/2990/review/1962874...@github.com>
_______________________________________________
Rpm-maint mailing list
Rpm-maint@lists.rpm.org
http://lists.rpm.org/mailman/listinfo/rpm-maint

Reply via email to