On 11/5/26 11:38, Daniel P. Berrangé wrote:
The following two uses of get_maintainer.pl should return the
same results, but do not:
$ ./scripts/get_maintainer.pl -f ./hw/net/vmxnet3.c
get_maintainer.pl: No maintainers found, printing recent contributors.
get_maintainer.pl: Do not blindly cc: them on patches! Use common sense.
"Philippe Mathieu-Daudé" <[email protected]> (commit_signer:4/7=57%)
"Michael S. Tsirkin" <[email protected]> (commit_signer:4/7=57%)
Xiaoyao Li <[email protected]> (commit_signer:3/7=43%)
Thomas Huth <[email protected]> (commit_signer:3/7=43%)
Zhao Liu <[email protected]> (commit_signer:3/7=43%)
[email protected] (open list:All patches CC here)
$ ./scripts/get_maintainer.pl -f hw/net/vmxnet3.c
Dmitry Fleytman <[email protected]> (maintainer:VMware)
Jason Wang <[email protected]> (odd fixer:Network devices)
[email protected] (open list:All patches CC here)
In the former case, the leading "./" needs to be removed before
trying to find a filename match.
Blindly stripping the "./" is valid because the script already
enforces that it is run from the QEMU git root directory, so
canonicalizing the filename vs $CWD is not required.
Signed-off-by: Daniel P. Berrangé <[email protected]>
---
scripts/get_maintainer.pl | 1 +
1 file changed, 1 insertion(+)
Reviewed-by: Philippe Mathieu-Daudé <[email protected]>
