On Fri, Mar 01, 2024 at 01:58:25PM +0100, Maxime Ripard wrote: > We've recently had multiple occurences of people taking patches through > the drm-misc tree for parts of the kernel that aren't under drm-misc > maintenance, and without the maintainers Ack. > > While merging patches on behalf of someone else is totally something we > can do, we need to tell the maintainer and make sure he's fine with > that. > > This patch adds some logic to detect if the the range of commits belong > indeed to the branch we are currently working on, and if not check for > the maintainer's Acked-by or Signed-off-by.
This was pushed to maintainer-tools yesterday and immediately tripped me up when trying to push a commit amending MAINTAINERS: https://lore.kernel.org/r/agjd01lyn5rev...@wunner.de That's because... git show HEAD | \ scripts/get_maintainer.pl --no-git-fallback -m --nol --norolestats ...returns nothing. I had to cheat by inserting a "return 0" in checkpatch_maintainer(). I think we either need to return 0 if the command returns nothing (i.e. no known maintainers) or we need a whitelist for certain files. Thanks, Lukas
