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

Reply via email to