diff options
| author | Mauro Carvalho Chehab <mchehab+samsung@kernel.org> | 2018-05-09 09:18:49 -0400 |
|---|---|---|
| committer | Jonathan Corbet <corbet@lwn.net> | 2018-05-10 17:42:43 -0400 |
| commit | d26560950b6ba6454c11cd978d3e6bb4d38430e8 (patch) | |
| tree | 42c6462dd3710e16f07b6f3a05c62cbc1a8e2832 /scripts/documentation-file-ref-check | |
| parent | 02a43659e15893a6611cfc10dc7aae1746eb0cdc (diff) | |
scripts/documentation-file-ref-check: rewrite it in perl with auto-fix mode
The original shell script works, but:
1) it is too slow;
2) it is hard to exclude rejex patterns
Convert it to perl.
Here, the new version is able to check the entire tree in
less than a second (after cached):
real 0m0,284s
user 0m0,668s
sys 0m0,778s
The old version takes more than a minute to complete (also
after cached):
real 1m17,905s
user 0m25,583s
sys 0m55,334s
It also produce less false-positives (if any).
The new script also contains an auto-fix mode.
Usually, file references get lost when they're moved to some other
place and/or renamed to .rst.
Add an experimental mode to auto-fix those.
Signed-off-by: Mauro Carvalho Chehab <mchehab+samsung@kernel.org>
Signed-off-by: Jonathan Corbet <corbet@lwn.net>
Diffstat (limited to 'scripts/documentation-file-ref-check')
| -rwxr-xr-x | scripts/documentation-file-ref-check | 125 |
1 files changed, 113 insertions, 12 deletions
diff --git a/scripts/documentation-file-ref-check b/scripts/documentation-file-ref-check index bc1659900e89..2520bc14ffac 100755 --- a/scripts/documentation-file-ref-check +++ b/scripts/documentation-file-ref-check | |||
| @@ -1,15 +1,116 @@ | |||
| 1 | #!/bin/sh | 1 | #!/usr/bin/env perl |
| 2 | # SPDX-License-Identifier: GPL-2.0 | ||
| 3 | # | ||
| 2 | # Treewide grep for references to files under Documentation, and report | 4 | # Treewide grep for references to files under Documentation, and report |
| 3 | # non-existing files in stderr. | 5 | # non-existing files in stderr. |
| 4 | 6 | ||
| 5 | for f in $(git ls-files); do | 7 | use warnings; |
| 6 | for ref in $(grep -ho "Documentation/[A-Za-z0-9_.,~/*+-]*" "$f"); do | 8 | use strict; |
| 7 | # presume trailing . and , are not part of the name | 9 | use Getopt::Long qw(:config no_auto_abbrev); |
| 8 | ref=${ref%%[.,]} | 10 | |
| 9 | 11 | my $scriptname = $0; | |
| 10 | # use ls to handle wildcards | 12 | $scriptname =~ s,.*/([^/]+/),$1,; |
| 11 | if ! ls $ref >/dev/null 2>&1; then | 13 | |
| 12 | echo "$f: $ref" >&2 | 14 | # Parse arguments |
| 13 | fi | 15 | my $help = 0; |
| 14 | done | 16 | my $fix = 0; |
| 15 | done | 17 | |
| 18 | GetOptions( | ||
| 19 | 'fix' => \$fix, | ||
| 20 | 'h|help|usage' => \$help, | ||
| 21 | ); | ||
| 22 | |||
| 23 | if ($help != 0) { | ||
| 24 | print "$scriptname [--help] [--fix-rst]\n"; | ||
| 25 | exit -1; | ||
| 26 | } | ||
| 27 | |||
| 28 | # Step 1: find broken references | ||
| 29 | print "Finding broken references. This may take a while... " if ($fix); | ||
| 30 | |||
| 31 | my %broken_ref; | ||
| 32 | |||
| 33 | open IN, "git grep 'Documentation/'|" | ||
| 34 | or die "Failed to run git grep"; | ||
| 35 | while (<IN>) { | ||
| 36 | next if (!m/^([^:]+):(.*)/); | ||
| 37 | |||
| 38 | my $f = $1; | ||
| 39 | my $ln = $2; | ||
| 40 | |||
| 41 | # Makefiles contain nasty expressions to parse docs | ||
| 42 | next if ($f =~ m/Makefile/); | ||
| 43 | # Skip this script | ||
| 44 | next if ($f eq $scriptname); | ||
| 45 | |||
| 46 | if ($ln =~ m,\b(\S*)(Documentation/[A-Za-z0-9\_\.\,\~/\*+-]*),) { | ||
| 47 | my $prefix = $1; | ||
| 48 | my $ref = $2; | ||
| 49 | my $base = $2; | ||
| 50 | |||
| 51 | $ref =~ s/[\,\.]+$//; | ||
| 52 | |||
| 53 | my $fulref = "$prefix$ref"; | ||
| 54 | |||
| 55 | $fulref =~ s/^(\<file|ref)://; | ||
| 56 | $fulref =~ s/^[\'\`]+//; | ||
| 57 | $fulref =~ s,^\$\(.*\)/,,; | ||
| 58 | $base =~ s,.*/,,; | ||
| 59 | |||
| 60 | # Remove URL false-positives | ||
| 61 | next if ($fulref =~ m/^http/); | ||
| 62 | |||
| 63 | # Check if exists, evaluating wildcards | ||
| 64 | next if (grep -e, glob("$ref $fulref")); | ||
| 65 | |||
| 66 | if ($fix) { | ||
| 67 | if (!($ref =~ m/(devicetree|scripts|Kconfig|Kbuild)/)) { | ||
| 68 | $broken_ref{$ref}++; | ||
| 69 | } | ||
| 70 | } else { | ||
| 71 | print STDERR "$f: $fulref\n"; | ||
| 72 | } | ||
| 73 | } | ||
| 74 | } | ||
| 75 | |||
| 76 | exit 0 if (!$fix); | ||
| 77 | |||
| 78 | # Step 2: Seek for file name alternatives | ||
| 79 | print "Auto-fixing broken references. Please double-check the results\n"; | ||
| 80 | |||
| 81 | foreach my $ref (keys %broken_ref) { | ||
| 82 | my $new =$ref; | ||
| 83 | |||
| 84 | # get just the basename | ||
| 85 | $new =~ s,.*/,,; | ||
| 86 | |||
| 87 | # Seek for the same name on another place, as it may have been moved | ||
| 88 | my $f=""; | ||
| 89 | |||
| 90 | $f = qx(find . -iname $new) if ($new); | ||
| 91 | |||
| 92 | # usual reason for breakage: file renamed to .rst | ||
| 93 | if (!$f) { | ||
| 94 | $new =~ s/\.txt$/.rst/; | ||
| 95 | $f=qx(find . -iname $new) if ($new); | ||
| 96 | } | ||
| 97 | |||
| 98 | my @find = split /\s+/, $f; | ||
| 99 | |||
| 100 | if (!$f) { | ||
| 101 | print STDERR "ERROR: Didn't find a replacement for $ref\n"; | ||
| 102 | } elsif (scalar(@find) > 1) { | ||
| 103 | print STDERR "WARNING: Won't auto-replace, as found multiple files close to $ref:\n"; | ||
| 104 | foreach my $j (@find) { | ||
| 105 | $j =~ s,^./,,; | ||
| 106 | print STDERR " $j\n"; | ||
| 107 | } | ||
| 108 | } else { | ||
| 109 | $f = $find[0]; | ||
| 110 | $f =~ s,^./,,; | ||
| 111 | print "INFO: Replacing $ref to $f\n"; | ||
| 112 | foreach my $j (qx(git grep -l $ref)) { | ||
| 113 | qx(sed "s\@$ref\@$f\@g" -i $j); | ||
| 114 | } | ||
| 115 | } | ||
| 116 | } | ||
