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

[Seth Vidal]

On Mon, 26 Apr 2010, Ville Skyttä wrote:

> 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?

b/c it is perfectly reasonable for list items (like excludes) to include 
glob chars and NOT be a dir.

exclude=foo* xorg*

for example.




> 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 ...?)

I'm fine with it being a different name - sure.

but I do think we need a prefix.

-sv