On Tue, May 06, 2014 at 12:17:14AM +0000, Eric Wong wrote:
> Users may already store sensitive data such as imap.pass in
> .git/config; making the file world-readable when "git config"
> is called to edit means their password would be compromised
> on a shared system.
Makes sense, and the patch looks good to me.
> +test_expect_success POSIXPERM,PERL 'preserves existing permissions' '
> + chmod 0600 .git/config &&
> + git config imap.pass Hunter2 &&
> + perl -e \
> + "die q(badset) if ((stat(q(.git/config))) & 07777) != 0600" &&
I don't think we usually bother with a PERL prereq for running
one-liners like this from the test script, though I don't think it hurts
anything to do so.
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html