Re: gitk feature request..

From: Petr Baudis <pasky@suse.cz>
Date: Nov 11, 2006, 17:42 UTC
Message-ID: <20061111174242.GB7201@pasky.or.cz>
In-Reply-To: <7vslgu28do.fsf@assigned-by-dhcp.cox.net>

On Tue, Nov 07, 2006 at 11:34:59PM CET, Junio C Hamano wrote:

Show 23 quoted lines

> Paul Mackerras <paulus@samba.org> writes:
> 
> > Good idea.  Junio, is there a canonical place under .git where gitk
> > should put such things?
> 
> Well, we do not design things in advance but tend to let things
> evolve, which probably is a bad habit but I am not sure how else
> we can avoid overdesigning before knowing the needs.
> 
> The existing state-keeping programs seem to keep their stuff
> immediately under $GIT_DIR.  Examples are:
> 
> 	.git/description (gitweb)
>         .git/cvs-authors (cvsimport)
>         .git/gitcvs.<branch>.sqlite (cvsserver)
> 
> So, .git/gitk-<foo> (or .git/gitk/<bar>) would be in line with
> others.  We _might_ want to have a standard plan to keep
> Porcelains stepping on each other's toes, and probably migrating
> everybody to .git/aux/{common,gitcvs,gitk,...}/<foo> would be a
> sane thing to do.  description and cvs-authors could probably be
> shared across Porcelains, so I do not think we mind leaving them
> in the current place or throw them in .git/aux/common/

I think the porcelains tend to use pretty specific names for their stuff and a risk of conflict is pretty small, so I don't think it's worth the compatibility problems. Also, over time it's bound that qgit will something like start to peek and poke at some gitk stuff and learn about CVS imports so it'll peek at gitcvs as well, etc., so I'm not sure if the net benefit would really be that large or rather give the porcelains a false illusion of safety and privacy.

Except for cg-merge which is quite a pig when it comes to its state and .git files; I guess it should use a .git/cg-merge-state/ directory instead. And the hooks space where cg sort of hoped that git would reuse its hooks names but it didn't play out.

-- 
				Petr "Pasky" Baudis
Stuff: http://pasky.or.cz/
#!/bin/perl -sp0777i<X+d*lMLa^*lN%0]dsXx++lMlN/dsM0<j]dsj
$/=unpack('H*',$_);$_=`echo 16dio\U$k"SK$/SM$n\EsN0p[lN*1