> OK, just hard coding one file attribute doesn't seem like a good idea. I 
> added a macro that allows you to register an arbitrary number of local file 
> attributes and generators.

Nice, now we can argue if the `__local_file_attrs` is the right name or rather 
e.g. `__package_local_file_attrs` 🙈🤣 Sorry, couldn't resist.

Nevertheless, thx for contributing. I see your point. OTOH, if I read your 
commit correctly, my original proposal had the advantage, where one could count 
with convention over configuration. IOW with your changes one additional line 
is needed to make this work.

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

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

Reply via email to