Re: [PATCH] Pay attention to GIT_DIR when searching the git directory

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Jakub,

Jakub Narebski schrieb am Mon 05. May, 07:10 (-0700):
> Jörg Sommer <joerg@xxxxxxxxxxxx> writes:
> > Johannes Schindelin schrieb am Mon 05. May, 12:36 (+0100):
> 
> > > If you do not say _what_ is affected by the change, you only confuse
> > > the readers.
> > 
> > Do you want something like “initialization: Pay attention to GIT_DIR when
> > searching the git directory”?
> 
> I think the idea was to use
>   "contrib/hooks/setgitperms: Use GIT_DIR if set"
> or something like that.

Well, if this is the prefered form, I'll use it. But I don't like it,
because the affected file is a piece of information that's already in the
commit object. And it makes the :/ syntax useless, because in a series of
commits all commits start with the same string: the filename. Thirdly, in
gitk I've the problem that the space is very limited (because of my
screen) and it is very annoying to see the same string at the begin of
every commit message, especially when I've filtered for one file.

> > Is there a document with guidelines for commit messages, e.g. write the
> > first letter upper or lower case?
> 
> There are a few unwritten conventions.

Are there any plans to write them down?

Should I resend the patch or can Junio fix the commit message? I would
take your suggestion.

Bye, Jörg.
-- 
Geld allein macht nicht glücklich, aber es ist besser in einem Taxi zu
weinen, als in der Straßenbahn.
	       	   	       			(Marcel Reich‐Ranicki)

Attachment: signature.asc
Description: Digital signature http://en.wikipedia.org/wiki/OpenPGP


[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]

  Powered by Linux