I'm a bit late to the :tada: but as far as the overall solution, I'm giving the
tag-based variant (`BuildSystem:`) implemented here a :+1: . I'm also giving a
:+1: to the shorter Build* namespace (the Auto prefix really had to go :smile:).
It looks to me like the cleanest and most flexible variant of all those
mentioned
[previously](https://github.com/rpm-software-management/rpm/pull/2620#issuecomment-1794302784),
too. In particular, the possibility to interleave these with the other tags
(such as `BuildRequires:`) makes it pretty neat as you can cluster those
together if they're part of a conditional (as mentioned in the linked comment).
I personally hate whenever I need to look up the correct syntax for line breaks
in a config format, this alleviates that problem since you just declare
multiple of these tags. Yes, it's a bit noisy but somehow reminds me of CMake
(as you also mentioned previously) and I guess I've developed the Stockholm
syndrome already because I kinda like (or rather, don't mind) it too much
:laughing:
As for the technical side of things, no opinion yet, I'm going to look at that
next.
--
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/pull/2774#issuecomment-1883049845
You are receiving this because you are subscribed to this thread.
Message ID: <rpm-software-management/rpm/pull/2774/c1883049...@github.com>
_______________________________________________
Rpm-maint mailing list
Rpm-maint@lists.rpm.org
http://lists.rpm.org/mailman/listinfo/rpm-maint
