ANDROID: Improve OWNERS file definitions

- Extend subdirectory expressions to also include recursively
- Combine expressions with common owners
- Minor cleanups

Note, the way expressions are evaluated in the code-owners plugin (as
of now and for compatibility with the `find-owners` plugin), means we
match slightly more than expected, even with just the single asterisk.
As expressions like `net*` are implicitly treated like `{**/,}net*`,
we would not only match net_something.h and net_filter/header.h, but
also (unexpectedly) platform/net_something.h.
This change does not make things better in this regards, but for all the
above mentioned points, this is an improvement.

Syntax Documentation:
https://gerrit.googlesource.com/plugins/code-owners/+/refs/heads/master/resources/Documentation/path-expressions.md

Fixes: 1d1197098c ("ANDROID: add initial set of OWNERS")
Signed-off-by: Matthias Maennich <maennich@google.com>
Change-Id: I7e5503382c7c6247a77c13c5c9f2ffd3cc6dd7e6
This commit is contained in:
Matthias Maennich 2021-05-06 17:33:15 +01:00
parent 4e307150ee
commit fa6ac567ed
9 changed files with 21 additions and 26 deletions

View File

@ -1 +1 @@
per-file f2fs*=file:/fs/f2fs/OWNERS
per-file f2fs**=file:/fs/f2fs/OWNERS

View File

@ -1,7 +1,4 @@
per-file crypto/*=file:/crypto/OWNERS
per-file include/*=mzyngier@google.com,willdeacon@google.com
per-file kernel/*=mzyngier@google.com,willdeacon@google.com
per-file kvm/*=mzyngier@google.com,willdeacon@google.com
per-file lib/*=mzyngier@google.com,willdeacon@google.com
per-file mm/*=file:/mm/OWNERS
per-file net/*=file:/net/OWNERS
per-file crypto/**=file:/crypto/OWNERS
per-file {include,kernel,kvm,lib}/**=mzyngier@google.com,willdeacon@google.com
per-file mm/**=file:/mm/OWNERS
per-file net/**=file:/net/OWNERS

View File

@ -1,3 +1,3 @@
per-file crypto/*=file:/crypto/OWNERS
per-file mm/*=file:/mm/OWNERS
per-file net/*=file:/net/OWNERS
per-file crypto/**=file:/crypto/OWNERS
per-file mm/**=file:/mm/OWNERS
per-file net/**=file:/net/OWNERS

View File

@ -1,7 +1,6 @@
per-file base/*=gregkh@google.com,saravanak@google.com
per-file block/*=akailash@google.com
per-file md/*=akailash@google.com,palmerdabbelt@google.com,paullawrence@google.com
per-file net/*=file:/net/OWNERS
per-file base/**=gregkh@google.com,saravanak@google.com
per-file block/**=akailash@google.com
per-file md/**=akailash@google.com,paullawrence@google.com
per-file net/**=file:/net/OWNERS
per-file scsi/**=bvanassche@google.com,jaegeuk@google.com
per-file tty/*=gregkh@google.com
per-file usb/*=gregkh@google.com
per-file {tty,usb}/**=gregkh@google.com

View File

@ -1,2 +1 @@
per-file crypto/*=ebiggers@google.com
per-file verity/*=ebiggers@google.com
per-file {crypto,verity}/**=ebiggers@google.com

View File

@ -1 +1 @@
per-file net=file:/net/OWNERS
per-file net/**=file:/net/OWNERS

View File

@ -1,4 +1,4 @@
per-file bio.h=file:/block/OWNERS
per-file blk*.h=file:/block/OWNERS
per-file f2fs*=file:/fs/f2fs/OWNERS
per-file net*=file:/net/OWNERS
per-file f2fs**=file:/fs/f2fs/OWNERS
per-file net**=file:/net/OWNERS

View File

@ -1 +1 @@
per-file f2fs*=file:/fs/f2fs/OWNERS
per-file f2fs**=file:/fs/f2fs/OWNERS

View File

@ -1,3 +1,3 @@
per-file f2fs*=file:/fs/f2fs/OWNERS
per-file fuse*=file:/fs/fuse/OWNERS
per-file net*=file:/net/OWNERS
per-file f2fs**=file:/fs/f2fs/OWNERS
per-file fuse**=file:/fs/fuse/OWNERS
per-file net**=file:/net/OWNERS