Add MANIFEST.SKIP file to skip t/GITPERLLIB

This was pointed out by Salve J. Nilsen via RT.

Bug: RT-83620
Signed-off-by: Gustavo L. de M. Chaves <gnustavo@cpan.org>
Make Git::More::get_current_branch return the branch full name

Previously, Git::More::get_current_branch was implemented via "git
branch" and returned the current branch short-name (i.e., sans the
refs/head prefix). Now it's reimplemented using the "git symbolic-ref
HEAD" command, which is faster and returns the branch full name
instead.

Signed-off-by: Gustavo L. de M. Chaves <gnustavo@cpan.org>
fixup: gerrit hook names

Signed-off-by: Gustavo L. de M. Chaves <gnustavo@cpan.org>
fixup: userenv

Signed-off-by: Gustavo L. de M. Chaves <gnustavo@cpan.org>
Fix treatment of close pipe exit value

I was treating the result of 'close' the same way as the result of
'system' but this is wrong. Close returns a boolean value while system
returns $?.

This problem was reported to me by Marcelo Bissaro.

Signed-off-by: Gustavo L. de M. Chaves <gnustavo@cpan.org>
Add missing dependency

The sub Git::Hooks::grok_groups invokes read_file but the module
wasn't being use'd before.

Signed-off-by: Gustavo L. de M. Chaves <gnustavo@cpan.org>