Re: Add "--show-all" revision walker flag for debugging

!MAILaRCHIVE_VOTE_RePLACE
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
To: Johannes Schindelin <Johannes.Schindelin@...>
Cc: Linus Torvalds <torvalds@...>, Junio C Hamano <gitster@...>, Git Mailing List <git@...>
Date: Saturday, February 9, 2008 - 9:28 pm

On Sun, 10 Feb 2008, Johannes Schindelin wrote:


Absolutely.

I, too, have some reservations about adding any kind of generation 
header to commit objects.  First because it can be generated and 
maintained locally, just like the pack index.  But also because its 
usefulness has not been proven in all possible graph topologies, and 
adding it to the commit header pretty much deny any further 
modifications/improvements on it, if for example some other kind of 
generation notation becomes advantageous to use.

So please don't put it into the commit object format.  The object 
database should ultimately contain only data that cannot be regenerated.


Nicolas
-
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
Add "--show-all" revision walker flag for debugging, Linus Torvalds, (Sat Feb 9, 6:02 pm)
Re: Add "--show-all" revision walker flag for debugging, Johannes Schindelin, (Sat Feb 9, 9:12 pm)
Re: Add "--show-all" revision walker flag for debugging, Nicolas Pitre, (Sat Feb 9, 9:28 pm)
Re: Add "--show-all" revision walker flag for debugging, Johannes Schindelin, (Sat Feb 9, 9:30 pm)
Re: Add "--show-all" revision walker flag for debugging, Jakub Narebski, (Sun Feb 10, 4:17 pm)
Re: Add "--show-all" revision walker flag for debugging, Linus Torvalds, (Sun Feb 10, 4:50 pm)
Re: Add "--show-all" revision walker flag for debugging, Jakub Narebski, (Sun Feb 10, 6:53 pm)
Re: Add "--show-all" revision walker flag for debugging, Linus Torvalds, (Sun Feb 10, 7:11 pm)
Re: Add "--show-all" revision walker flag for debugging, Jakub Narebski, (Sun Feb 10, 9:24 pm)
Re: Add "--show-all" revision walker flag for debugging, Linus Torvalds, (Mon Feb 11, 11:59 am)
Re: Add "--show-all" revision walker flag for debugging, Nicolas Pitre, (Mon Feb 11, 12:39 pm)
Re: Add "--show-all" revision walker flag for debugging, Jakub Narebski, (Mon Feb 11, 12:26 pm)
Re: Add "--show-all" revision walker flag for debugging, Nicolas Pitre, (Sun Feb 10, 9:59 pm)
Re: Add "--show-all" revision walker flag for debugging, Nicolas Pitre, (Sun Feb 10, 5:04 pm)
Re: Add "--show-all" revision walker flag for debugging, Linus Torvalds, (Sat Feb 9, 9:22 pm)
Re: Add "--show-all" revision walker flag for debugging, Junio C Hamano, (Sun Feb 10, 12:09 am)
Re: Add "--show-all" revision walker flag for debugging, Junio C Hamano, (Sun Feb 10, 12:21 am)
Re: Add "--show-all" revision walker flag for debugging, Johannes Schindelin, (Sat Feb 9, 9:29 pm)
Re: Add "--show-all" revision walker flag for debugging, Linus Torvalds, (Sat Feb 9, 7:52 pm)
Re: Add "--show-all" revision walker flag for debugging, Junio C Hamano, (Sun Feb 10, 12:44 am)