From: Prarit Bhargava on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1779#note_945877674
>> which variables directly affected the SPEC file and which were used for other reasons in the Makefile. > >But is that relevant for people that do anything with kernel-ark or only useful to make the Makefiles more readable? If it's just the latter I'd say it's more confusing then helpful. > I'd argue it's to make the Makefile more readable and easier for us to debug. But I would disagree with the confusing statement. The variables that users can use are defined in redhat/Makefile.variables (see rest of my answer below) >This got me thinking: the document added by this patch afaics doesn't even tell me what I can do with the variable names I just learned from that text. Or am I missing something? > redhat/docs/makefile-changes.rst already exists and explains the layout. I think that is sufficient but since documentation can ALWAYS be improved patches are definitely welcomed :) >I assume I can pass them to make (say `make dist-srpm BUILD=123` ?), but shouldn't the text explain this? Maybe adding something like "Variables in this file can be overridden on the command line." to the Makefile.variables section with your example? _______________________________________________ kernel mailing list -- [email protected] To unsubscribe send an email to [email protected] Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedoraproject.org/archives/list/[email protected] Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure
