On Wed, Nov 24, 2021 at 01:56:42PM +0100, Thomas Monjalon wrote: > Options used to be sorted alphabetically. > It looks easier to read when major options are first, > then path tuning, libs options, and drivers options. > > Signed-off-by: Thomas Monjalon <tho...@monjalon.net> > --- > v2: add blank lines for readability > --- > meson_options.txt | 99 ++++++++++++++++++++++++++++++----------------- > 1 file changed, 63 insertions(+), 36 deletions(-)
While I like the idea of being able to group items, in the absense of meson support for displaying them as groups, I worry about it making it harder to find and use options. It also has some ambiguity as to what should be in what category: e.g. is max_lcores a global option or a library-specific one for EAL. I would have considered it global, but you have it here as library specific - which when I think about it is probably more correct, just not what I thought of instinctively. On the other hand, I agree that having them sorted alphabetically is not brilliant either - I think I originally chose to order them alphabetically so as to avoid arguments as to where exactly each option should go, since it makes things unambiguous. Overall, I'm fairly ambivilant about this patch, but would appreciate more input from others, and I'm happy with whatever the majority decision is. /Bruce
