@pmatilai commented on this pull request.


> +            goto continue
+        end
+        fields = {}
+        for w in line:gmatch("%S+") do
+            table.insert(fields, w)
+        end
+        if #fields >= 3 and fields[1] == 'm' then
+           print(string.format('user(%s)\\ngroup(%s)', fields[2], fields[3]))
+        end
+        ::continue::
+    end
+}
+
+%__sysusers_recommends() %{?_use_weak_usergroup_deps:%{__sysusers_dependencies 
%1}}
+
+%__sysusers_requires() %{!?_use_weak_usergroup_deps:%{__sysusers_dependencies 
%1}}

You'll need the %[] expression thing in these, like you had in the earlier 
version. %{?...} only tests whether the macro is defined, so this will use weak 
dependencies even if set to 0.

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

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

Reply via email to