lkml.org 
[lkml]   [2012]   [Dec]   [17]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH] scripts: add checkmaintainers.py
Em 17-12-2012 15:00, Borislav Petkov escreveu:
> On Mon, Dec 17, 2012 at 07:35:44AM -0800, Joe Perches wrote:
>> Perhaps Cesar can use his script as a starting point to find those
>> pattern invalidating commits or maybe add the capability (or a
>> --strict check) to checkpatch.
>
> Or that, I don't have a strict preference.
>
> So, yeah, I can see how checkpatch saying: "you've just renamed a
> file and thusly invalidated a pattern in MAINTAINERS. Pls, consider
> correcting the pattern" could make sense. And I would even add it to
> default functionality since the MAINTAINERS patterns are something we
> want to always have up-to-date, IMO.

Good idea, but a standalone MAINTAINERS checker is still useful, to
check for things checkpatch did not catch, either because it was not
run, or because it did not have enough information.

For instance, consider the case of a patch removing the last file in a
directory going in via one branch, while a patch adding a file pattern
for that same directory goes in via a separate branch. There is no way
checkpatch can catch that.

As for finding the pattern invalidating commits, it is usually not that
hard, a simple "git log -- <pattern>" or similar usually does the trick,
except in the cases where the pattern addition itself was wrong. The
hard part can be interpreting it and the surrounding commits to find out
the committer's intention, how it should affect the MAINTAINERS entry,
and who should get a Cc: of the fix.

--
Cesar Eduardo Barros
cesarb@cesarb.net
cesar.barros@gmail.com


\
 
 \ /
  Last update: 2012-12-17 23:41    [W:0.069 / U:1.044 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site