From: Junio C Hamano <gitster@pobox.com>
Date: Sat, 20 Feb 2016 21:32:17 GMT
Subject: Re: [PATCH v5 25/27] refs: add LMDB refs storage backend
Message-ID: <xmqqr3g79ify.fsf@gitster.mtv.corp.google.com>
In-Reply-To: <1455827001.7528.87.camel@twopensource.com>

David Turner <dturner@twopensource.com> writes:

> So just add this after every mkdir?
>
> 	if (shared_repository)
> 		adjust_shared_perm(db_path);

The function itself checks shared_repository configuration, so the
caller does not have to bother.  You should rather treat it as a
declaration and a documentation that says "this path in $GIT_DIR
should be writable".

You should check its exit value for errors, though.