[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 1/4] get_maintainer.pl: exit with status 1 if no
From: |
Paolo Bonzini |
Subject: |
Re: [Qemu-devel] [PATCH 1/4] get_maintainer.pl: exit with status 1 if no maintainer found |
Date: |
Wed, 22 Oct 2014 14:32:15 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.2.0 |
> Thus, the patch makes the script exit with status 1 when none of the
> above output() calls produces output.
>
> Maybe it exits with status 1 when it fails to produce output? Obvious
> ouput elsewhere: printing the version (okay),
That exits with 0.
> and a print in get_maintainers(). Can't tell what the latter does. Hmm.
That is for --sections, and you're right, this is missing:
diff --git a/scripts/get_maintainer.pl b/scripts/get_maintainer.pl
index f4fee27..e682d1f 100755
--- a/scripts/get_maintainer.pl
+++ b/scripts/get_maintainer.pl
@@ -617,6 +617,7 @@ sub get_maintainers {
print("$line\n");
}
print("\n");
+ $exit = 0;
}
}
}
Paolo
- [Qemu-devel] [PATCH 0/4] small improvements to get_maintainer.pl, Paolo Bonzini, 2014/10/22
- [Qemu-devel] [PATCH 1/4] get_maintainer.pl: exit with status 1 if no maintainer found, Paolo Bonzini, 2014/10/22
- [Qemu-devel] [PATCH 3/4] get_maintainer.pl: move git loop under "if ($email) {", Paolo Bonzini, 2014/10/22
- [Qemu-devel] [PATCH 4/4] get_maintainer.pl: point at --git-fallback instead of enabling it automatically, Paolo Bonzini, 2014/10/22
- Re: [Qemu-devel] [PATCH 4/4] get_maintainer.pl: point at --git-fallback instead of enabling it automatically, Michael S. Tsirkin, 2014/10/22
- Re: [Qemu-devel] [PATCH 4/4] get_maintainer.pl: point at --git-fallback instead of enabling it automatically, Paolo Bonzini, 2014/10/22
- Re: [Qemu-devel] [PATCH 4/4] get_maintainer.pl: point at --git-fallback instead of enabling it automatically, Thomas Huth, 2014/10/22
- Re: [Qemu-devel] [PATCH 4/4] get_maintainer.pl: point at --git-fallback instead of enabling it automatically, Michael S. Tsirkin, 2014/10/22
- Re: [Qemu-devel] [PATCH 4/4] get_maintainer.pl: point at --git-fallback instead of enabling it automatically, Paolo Bonzini, 2014/10/22
- Re: [Qemu-devel] [PATCH 4/4] get_maintainer.pl: point at --git-fallback instead of enabling it automatically, Markus Armbruster, 2014/10/22
[Qemu-devel] [PATCH 2/4] get_maintainer.pl: treat all M entries the same, Paolo Bonzini, 2014/10/22