On 12 Nov 2024, at 18:46, Mark Asselstine via lists.openembedded.org 
<[email protected]> wrote:
>> But I'm not really sure I like globs at all in filename directives,
> 
> BTW, it could prove useful for paths as well 'include */maintainers.inc'

The current code doesn’t need layer names, so how would this actually work?  
The parser would look at the filename and if it contained globs, prefix the 
layer name onto each entry whilst iterating?

The current semantics of include are “parse the first file with this name found 
in all of the layers configured”, and the proposal for include_all simply 
changes that to be “parse every file with this name in all of the layers 
configured”.  For globbing like "include */conf/maintainers.inc” to work would 
mean ‘if the filename is a glob, for each layer construct a new path that is 
the layer name plus the path, and then match the glob’.   If the glob was 
somewhere but the beginning, would you also need to glob at the front too?

This feels a bit too much like magic to me, I prefer the simple implementation 
of include_all.

Ross
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#2075): 
https://lists.openembedded.org/g/openembedded-architecture/message/2075
Mute This Topic: https://lists.openembedded.org/mt/109425270/21656
Group Owner: [email protected]
Unsubscribe: https://lists.openembedded.org/g/openembedded-architecture/unsub 
[[email protected]]
-=-=-=-=-=-=-=-=-=-=-=-

Reply via email to