[Yum-devel] [PATCH 1/2] add globdir: syntax to all ListOption() and UrlListOption() items in config.py

Ville Skyttä ville.skytta at iki.fi
Mon Apr 26 15:45:50 UTC 2010


On Monday 26 April 2010, Seth Vidal wrote:
> this allows the user to specify globdir:/path/somewhere.d/*.conf and the
> contents of all the files in that dir will be used to populate the config
> option as a list.

Maybe a dumb question, but why is the globdir: prefix needed?  Isn't it enough 
that if a path contains one or more globbing characters, it will be treated as 
a glob pretty much as described above?

If not, I'd suggest renaming it into simply glob: because I believe the intent 
is that /path/somewhere.d/*.conf will match _files_ matching that glob, and 
it's not completely clear what the "dir" is (somewhere.d or *.conf or ...?)


More information about the Yum-devel mailing list