Josh Triplett wrote on Thu, Dec 08, 2016 at 23:31:45 -0800:
> + ':section:(admin base comm contrib/admin contrib/comm contrib/devel
> contrib/doc contrib/games contrib/graphics contrib/interpreters contrib/java
> contrib/javascript contrib/kde contrib/libdevel contrib/libs contrib/mail
> contrib/math contrib/misc contrib/net contrib/otherosfs contrib/perl
> contrib/python contrib/ruby contrib/rust contrib/science contrib/sound
> contrib/tex contrib/text contrib/utils contrib/web contrib/x11 devel doc
> editors electronics embedded games gnome graphics hamradio interpreters java
> javascript kde libdevel libs mail math misc net news non-free/admin
> non-free/base non-free/comm non-free/devel non-free/doc non-free/editors
> non-free/electronics non-free/games non-free/graphics non-free/hamradio
> non-free/java non-free/javascript non-free/libdevel non-free/libs
> non-free/mail non-free/math non-free/misc non-free/net non-free/news
> non-free/otherosfs non-free/python non-free/ruby non-free/rust
> non-free/science non-free/sound non-free/tex non-free/text non-free/utils
> non-free/web non-free/x11 oldlibs otherosfs perl python ruby rust science
> shells sound tex text utils web x11)'
Before your patch, there are 33 main, 26 non-free, 24 contrib sections
in this list.
I'd rather rewrite the whole thing using brace expansion:
{,contrib/,non-free/}{admin,base,...,x11}
Other than that, looks good to me.
Thanks,
Daniel