@dmnks commented on this pull request.


> +without the parenthesis defaults to configuration. In other words,
+these two lines are exactly equivalent:
+
+```
+BuildOption: --enable-fu
+BuildOption(conf): --enable-fu
+```
+
+Passing these per-section options to the actual buildsystem of the
+package is the responsibility of the buildsystem specific macros.
+
+3) Complex packages can have things like multiple build systems, in
+which case you might want to invoke the macros manually, eg.
+
+```
+%buildsystem_autotools_build

This is probably the least favorite part of mine, in terms of verbosity, but I 
suppose it can be tweaked later. Also, it's not the common happy path anyway 
so, as a packager needing an advanced/non-standard build procedure, once you 
open the lid and peek inside, you're already on your own anyway :smile: 

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

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

Reply via email to