Hi Otavio, Thanks for taking a look at my patch.
On 02/02/14 19:23, Otavio Salvador wrote: > I like to idea and I second this pull request, the only thing I would > like to see added is the possibility of grabing the maintainers for > boards from the machine .conf file. This is the way we're using at > meta-fsl-arm and meta-fsl-arm-extra. What do you think? To be honest I don't have much experience with perl, this script is in perl simply because that's the way it comes to us from the Linux kernel (although I was able to understand enough of it to be able to make the changes required so it would fit into this project). I'm quite sure, with StackOverflow by my side, I'd be more than able to make these changes :-) However... the MAINTAINERS syntax[1] does allow for "F", "N", "X", and "K" clauses, to wit: F: Files and directories with wildcard patterns. A trailing slash includes all files and subdirectory files. F: drivers/net/ all files in and below drivers/net F: drivers/net/* all files in drivers/net, but not below F: */net/* all files in "any top level directory"/net One pattern per line. Multiple F: lines acceptable. N: Files and directories with regex patterns. N: [^a-z]tegra all files whose path contains the word tegra One pattern per line. Multiple N: lines acceptable. scripts/get_maintainer.pl has different behavior for files that match F: pattern and matches of N: patterns. By default, get_maintainer will not look at git log history when an F: pattern match occurs. When an N: match occurs, git log history is used to also notify the people that have git commit signatures. X: Files and directories that are NOT maintained, same rules as F: Files exclusions are tested before file matches. Can be useful for excluding a specific subdirectory, for instance: F: net/ X: net/ipv6/ matches all files in and below net excluding net/ipv6/ K: Keyword perl extended regex pattern to match content in a patch or file. For instance: K: of_get_profile matches patches or files that contain "of_get_profile" K: \b(printk|pr_(info|err))\b matches patches or files that contain one or more of the words printk, pr_info or pr_err One regex pattern per line. Multiple K: lines acceptable. As you can see, use of these clauses would cover your use case. What I was hoping for is that this script would be added to the project, then I could go about creating MAINTAINERS files for the various layers which would incorporate the information to which you referred above in a more standard, cross-project way. If you feel strongly about keeping the maintainers information in these files and this is the only thing holding up the script's inclusion I'd be happy to make the necessary modifications. But personally I'd prefer to use the existing MAINTAINERS syntax instead. Would moving that information into a standard top-level MAINTAINERS file suffice? Is there a reason it needs to be in those files (perhaps some other tool is processing them that couldn't be switched to process a MAINTAINERS file instead)? Best regards, Trevor [1] https://www.kernel.org/doc/linux/MAINTAINERS _______________________________________________ Openembedded-core mailing list Openembedded-core@lists.openembedded.org http://lists.openembedded.org/mailman/listinfo/openembedded-core