diff options
| author | Aleksandr Nogikh <nogikh@google.com> | 2026-01-19 15:55:38 +0100 |
|---|---|---|
| committer | Aleksandr Nogikh <nogikh@google.com> | 2026-01-20 08:35:12 +0000 |
| commit | c3117fc5b6ef9751fabc4522a4e9183c82c13b0a (patch) | |
| tree | 0006285340068899dde8be288f19116b6b6992ac /pkg/subsystem/linux/maintainers.go | |
| parent | 75769ae79e1f7c200dfc52bd7711b2b43f88f28c (diff) | |
pkg/subsystem: adjust the wildcard matching rules
Contrary to the description on top of MAINTAINERS, many F: records that
point to folders actually don't end with / or /*.
The get_maintainer.pl script already tolerates this, so let's do the
same.
Diffstat (limited to 'pkg/subsystem/linux/maintainers.go')
| -rw-r--r-- | pkg/subsystem/linux/maintainers.go | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/pkg/subsystem/linux/maintainers.go b/pkg/subsystem/linux/maintainers.go index 00aedda0e..07c72fdad 100644 --- a/pkg/subsystem/linux/maintainers.go +++ b/pkg/subsystem/linux/maintainers.go @@ -209,7 +209,6 @@ var ( func wildcardToRegexp(wildcard string, store *strings.Builder) { store.WriteByte('^') - // We diverge a bit from the standard MAINTAINERS rule semantics. // path/* corresponds to the files belonging to the `path` folder, // but, since we also infer the parent-child relationship, it's @@ -231,7 +230,9 @@ func wildcardToRegexp(wildcard string, store *strings.Builder) { if tokenStart < len(wildcard) { store.WriteString(regexp.QuoteMeta(wildcard[tokenStart:])) } + // get_maintainers.pl script tolerates the absence of / when the wildcard + // points to a folder. Let's do the same. if wildcard == "" || wildcard[len(wildcard)-1] != '/' { - store.WriteByte('$') + store.WriteString("(?:" + escapedSeparator + "|$)") } } |