oops-- forgot to "reply all"; so it only went to Junio... ---------- Forwarded message ---------- From: Sitaram Chamarty <sitaramc@xxxxxxxxx> Date: Mon, Mar 22, 2010 at 3:14 PM Subject: Re: running "git init --shared" on an existing repo To: Junio C Hamano <gitster@xxxxxxxxx> On Mon, Mar 22, 2010 at 12:36 PM, Junio C Hamano <gitster@xxxxxxxxx> wrote: > Sitaram Chamarty <sitaramc@xxxxxxxxx> writes: > >> And it seems there was a beginning made. builtin/init-db.c at line 220 says: >> >> /* >> * We would have created the above under user's umask -- under >> * shared-repository settings, we would need to fix them up. >> */ > > That "the above" refers to "in this process that created the repository > directory and populated it halfway by calling copy_templates()", and never > "some time ago by some other process that created the repository, in which > this process is now being run", so it is not fair to say "a beginning". Possible, but the fact remains that when you do that "some time ago", and then come back and do this, it *does* fix up the perms of refs, refs/heads, and refs/tags. > You would need to run "find $GIT_DIR -print0 | xargs -0 chown/mod ..." or > a moral equivalent of it, I guess. Shouldn't be a rocket surgery ;-) true... the toruble with using perl for years is that it sorta spoils you; I need to actually *think* now, despite it not being rocket surgery :-) I will send a patch by and by... I wonder if cheating and calling system("find <whatever> -type d -exec chmod g+s") will be acceptable ;-) -- Sitaram -- Sitaram -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html