import: clobber $? if global init.defaultBranch fails

We don't want $? influencing further error checking downstream,
such as thw _wq_done_wait call in public-inbox-clone when
git-config(1) invocations which set $?=0 are replaced with
in-process libgit2 config function calls.

diff --git a/lib/PublicInbox/Import.pm b/lib/PublicInbox/Import.pm
index e6d725fc5563e9da731e8252bc345f846f86d7fd..35985f74f60cd8b8b3a7fbcb16a3bc7fbd227561 100644 (file)
--- a/lib/PublicInbox/Import.pm
+++ b/lib/PublicInbox/Import.pm
@@ -29,6 +29,7 @@ sub default_branch () {
                my $h = run_qx([git_exe,qw(config --global init.defaultBranch)],
                                 { GIT_CONFIG => undef });
                chomp $h;
+               $? = 0;
                $h eq '' ? 'refs/heads/master' : "refs/heads/$h";
        }
 }