Thread (6 messages) flat view 6 messages, 4 authors, 2016-06-15

Re: [JGIT PATCH 2/2] Make Repository.isValidRefName compatible with Git 1.6.3

From: Linus Torvalds <torvalds@linux-foundation.org>
Date: 2016-06-15 22:46:43


On Fri, 8 May 2009, Robin Rosenberg wrote:
In 3e262b95c509 I taught C Git to disallow refs whose names end in
".lock".
Btw, I think we should revert that, and instead change our naming for 
lock-files.

It's a silly limitation, and for a stupid reason, I think. I can well 
imagine people using a branch naming policy of using '.' instead of 
spaces, and then It makes perfect sense to call a branch "fix.vm.lock" 
when you have a VM locking issue you want to fix.

So I think we should remove that stupid "*.lock" rule from the C version 
(and then obviously from the JGIT one too).

Now, that leaves what we _should_ call the lock-files, and the only sane 
thing to do is to pick a sequence that is already invalid for other 
reasons. We could make the lockfile end in "..lock", for example (".." is 
not legal in names due to the confusion with the "a..b" range notation), 
or just end or begin with a "." (same issue, except "..." now).

Or just make it end in '~' which is also invalid for similar reasons.

		Linus
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help