On Thu, Jan 21, 2021 at 5:16 PM Nathan Chancellor <natechancellor@xxxxxxxxx> wrote: > > On Thu, Jan 21, 2021 at 05:01:15PM +0100, Lukas Bulwahn wrote: > > Commit 6c8ad4427f6e ("kbuild: check the minimum compiler version in > > Kconfig") removed ./scripts/clang-version.sh and moved its content to > > ./scripts/cc-version.sh. > > > > Since then, ./scripts/get_maintainer.pl --self-test=patterns complains: > > > > warning: no file matches F: scripts/clang-version.sh > > > > The CLANG/LLVM BUILD SUPPORT section in MAINTAINERS intends to track > > changes in ./scripts/clang-version.sh; as the file is removed, track > > changes in ./scripts/cc-version.sh instead now. > > > > Signed-off-by: Lukas Bulwahn <lukas.bulwahn@xxxxxxxxx> > > --- > > applies cleanly on next-20210121 > > > > Masahiro-san, please pick this quick fix-up patch. > > Masahiro cannot pick this up because the patch to add clang-version.sh > to MAINTAINERS is in mmotm. > > I think the better solution is for Andrew to drop the current version of > > maintainers-add-a-couple-more-files-to-the-clang-llvm-section.patch > > and pick up the second one I sent, which allows us to deal with this: > > https://lore.kernel.org/lkml/20210114171629.592007-1-natechancellor@xxxxxxxxx/ > > I am not sure it is right for us to maintain cc-version.sh but I am open > to it if Masahiro agrees. > Okay, I did not see in linux-next that both changes are coming into linux-next through to different trees. Nathan, I guess if you send a follow-up patch to Andrew that is the best way to handle it, or we wait until both trees land in mainline, and then just provide a quick fix like this afterwards. So, Masahiro-san, no need to pick this patch here. Lukas