about summary refs log tree commit homepage
path: root/lib/PublicInbox/Config.pm
diff options
context:
space:
mode:
authorEric Wong <e@yhbt.net>2020-01-10 09:14:19 +0000
committerEric Wong <e@yhbt.net>2020-01-11 06:32:08 +0000
commit9f5a583694396f84056b9d92255bba0197b52bc8 (patch)
treeefece7a24babf0bfa9ad0f31abab5a119ce384ff /lib/PublicInbox/Config.pm
parent6d576feb27625abc36675b7b466265d4f06b2f26 (diff)
downloadpublic-inbox-9f5a583694396f84056b9d92255bba0197b52bc8.tar.gz
Most spawn and popen_rd callers die on failure to spawn,
anyways, and some are missing checks entirely.  This saves
us a bunch of verbose error-checking code in callers.

This also makes popen_rd more consistent, since it already
dies on pipe creation failures.
Diffstat (limited to 'lib/PublicInbox/Config.pm')
-rw-r--r--lib/PublicInbox/Config.pm2
1 files changed, 1 insertions, 1 deletions
diff --git a/lib/PublicInbox/Config.pm b/lib/PublicInbox/Config.pm
index cc8c1eaf..56d146c2 100644
--- a/lib/PublicInbox/Config.pm
+++ b/lib/PublicInbox/Config.pm
@@ -158,7 +158,7 @@ sub git_config_dump {
         return {} unless -e $file;
         my @cmd = (qw/git config -z -l/, "--file=$file");
         my $cmd = join(' ', @cmd);
-        my $fh = popen_rd(\@cmd) or die "popen_rd failed for $file: $!\n";
+        my $fh = popen_rd(\@cmd);
         my $rv = config_fh_parse($fh, "\0", "\n");
         close $fh or die "failed to close ($cmd) pipe: $?";
         $rv;