[xen staging] scripts/add_maintainers.pl: Don't call get_maintainers.pl with -f

classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view
|

[xen staging] scripts/add_maintainers.pl: Don't call get_maintainers.pl with -f

patchbot
commit 41339ecb5f18ca7ec7b0c914c952a0e1715ae511
Author:     Julien Grall <[hidden email]>
AuthorDate: Tue Jun 5 17:39:38 2018 +0100
Commit:     Julien Grall <[hidden email]>
CommitDate: Tue Jun 12 12:08:15 2018 +0100

    scripts/add_maintainers.pl: Don't call get_maintainers.pl with -f
   
    The option -f of scripts/get_maintainers.pl will return the maintainers
    of a given file, *not* the list of maintainers if the file was a patch.
   
    The output expected of add_maintainers is the latter, so drop the option
    -f.
   
    Signed-off-by: Julien Grall <[hidden email]>
    Release-acked-by: Juergen Gross <[hidden email]>
    Reviewed-by: Lars Kurth <[hidden email]>
    Acked-by: Ian Jackson <[hidden email]>
---
 scripts/add_maintainers.pl | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/scripts/add_maintainers.pl b/scripts/add_maintainers.pl
index 99e4724112..09e9f6609f 100755
--- a/scripts/add_maintainers.pl
+++ b/scripts/add_maintainers.pl
@@ -420,7 +420,7 @@ sub ismailinglist ($) {
 sub getmaintainers ($$$) {
     my ($file, $rto, $rcc) = @_;
     my $fh;
-    open($fh, "-|", $get_maintainer, @get_maintainer_args, '-f', $file)
+    open($fh, "-|", $get_maintainer, @get_maintainer_args, $file)
         or die "Failed to open '$get_maintainer'\n";
     while(my $line = <$fh>) {
         chomp $line;
--
generated by git-patchbot for /home/xen/git/xen.git#staging

_______________________________________________
Xen-changelog mailing list
[hidden email]
https://lists.xenproject.org/xen-changelog