On 19 July 2016 at 18:47, Stefan Weil <[email protected]> wrote: > Commit e5dfc5e8e715c572aea44ac4d96c43941d4741c7 renamed README but did not > update scripts/checkpatch.pl. Fix this. > > An update was also needed for qemu.nsi (Windows). > > Reported-by: Daniel P. Berrange <[email protected]> > Signed-off-by: Stefan Weil <[email protected]> > --- > qemu.nsi | 4 ++-- > scripts/checkpatch.pl | 2 +- > 2 files changed, 3 insertions(+), 3 deletions(-) > > diff --git a/qemu.nsi b/qemu.nsi > index a20f6ef..c037017 100644 > --- a/qemu.nsi > +++ b/qemu.nsi > @@ -116,7 +116,7 @@ Section "${PRODUCT} (required)" > File "${SRCDIR}\Changelog" > File "${SRCDIR}\COPYING" > File "${SRCDIR}\COPYING.LIB" > - File "${SRCDIR}\README" > + File "${SRCDIR}\README.md" > File "${SRCDIR}\VERSION" > > File "${BINDIR}\*.bmp" > @@ -206,7 +206,7 @@ Section "Uninstall" > Delete "$INSTDIR\Changelog" > Delete "$INSTDIR\COPYING" > Delete "$INSTDIR\COPYING.LIB" > - Delete "$INSTDIR\README" > + Delete "$INSTDIR\README.md" > Delete "$INSTDIR\VERSION" > Delete "$INSTDIR\*.bmp" > Delete "$INSTDIR\*.bin" > diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl > index afa7f79..8247305 100755 > --- a/scripts/checkpatch.pl > +++ b/scripts/checkpatch.pl > @@ -286,7 +286,7 @@ sub top_of_kernel_tree { > > my @tree_check = ( > "COPYING", "MAINTAINERS", "Makefile", > - "README", "docs", "VERSION", > + "README.md", "docs", "VERSION", > "vl.c" > );
Does raise the question of whether we should be renaming the file in the first place. README is the traditional name and fits with all our other basically-plain-text document names like COPYING, MAINTAINERS, HACKING. thanks -- PMM
