Re: [PATCH] scripts/tags.sh: ignore symlink'ed source files

From: Neil Horman
Date: Wed May 21 2014 - 11:28:22 EST


On Wed, May 21, 2014 at 03:32:17PM +0200, Yann Droneaud wrote:
> Since commit 22d651dcef536c75f75537290bf3da5038e68b6b ('selftests/powerpc:
> Import Anton's memcpy / copy_tofrom_user tests'), some source files in the
> tree appear as symlink.
> Until commit 8c38a5328af8080bc69a25b3e4e144b03eeea95e ('scripts/tags.sh:
> ignore code of user space tools'), those symlinks made cscope report some
> warnings:
>
> $ make ALLSOURCE_ARCHS=all O=./obj-cscope/ cscope
> GEN cscope
> cscope: cannot find
> file .../tools/testing/selftests/powerpc/copyloops/copyuser_power7.S
> cscope: cannot find
> file .../tools/testing/selftests/powerpc/copyloops/memcpy_64.S
> cscope: cannot find
> file .../tools/testing/selftests/powerpc/copyloops/memcpy_power7.S
> cscope: cannot find
> file .../tools/testing/selftests/powerpc/copyloops/copyuser_64.S
>
> In order to prevent the same kind of warnings to be triggered by future
> addition of symlinks, the best option is to ignore all symlinks when
> building the file list to be processed by cscope (and other tools
> supported by scripts/tags.sh).
>
> Ignoring symlinks won't hide source files from cscope (and others) as the
> target of these symlinks already appear somewhere else in the tree, and,
> as such, should be processed by cscope (or others).
>
> Note that, cscope, when used with -R option to make it find the files to
> process by itself, already skip symlinks: it's not expected that cscope
> access source files through symlink.
>
> On top of commit 8c38a5328af8080bc69a25b3e4e144b03eeea95e ('scripts/tags.sh:
> ignore code of user space tools'), scripts/tags.sh output from
> "make cscope tags TAGS" is the same with and without this patch: it doesn't
> seems to introduce any regression (on Fedora 20).
>
> Link: http://lkml.kernel.org/r/1396530975.4361.28.camel@xxxxxxxxxxxxxxxxxxxxx
> Link: http://mid.gmane.org/534312F8.5090609@xxxxxxxxxxx
> Cc: Michael Ellerman <mpe@xxxxxxxxxxxxxx>
> Cc: Anton Blanchard <anton@xxxxxxxxx>
> Cc: Benjamin Herrenschmidt <benh@xxxxxxxxxxxxxxxxxxx>
> Cc: Hans-Bernhard Bröker <broeker@xxxxxxxxxxxxxxxxxxxxx>,
> Cc: Hans-Bernhard Broeker <broeker@xxxxxxxxxxxxxxxxxxxxx>,
> Cc: Neil Horman <nhorman@xxxxxxxxxxxxxxxxxxxxx>
> Cc: Neil Horman <nhorman@xxxxxxxxxxxxx>
> Cc: Gerhard Sittig <gsi@xxxxxxx>
> Cc: Konstantin Khlebnikov <koct9i@xxxxxxxxx>
> Signed-off-by: Yann Droneaud <ydroneaud@xxxxxxxxxx>
>
Acked-by: Neil Horman <nhorman@xxxxxxxxxxxxx>

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/