Subject: Re: [PATCH v5 25/27] refs: add LMDB refs storage backend

From: Junio C Hamano <[email-masked]>
Date: Sat, 20 Feb 2016 13:32:17 -0800

David Turner <[email-masked]> 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.