login
Login / Register
Header Space

 
 

Home » Mailing list archives » git » 2007 » August » 28

Re: [PATCH] name-rev: Fix non-shortest description

Score:
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
[view in full thread]
From: Johannes Schindelin <Johannes.Schindelin@...>
To: Jeff King <peff@...>
Cc: Uwe <ukleinek@...>, <git@...>, <gitster@...>
Subject: Re: [PATCH] name-rev: Fix non-shortest description
Date: Tuesday, August 28, 2007 - 5:15 am

Hi,

On Tue, 28 Aug 2007, Jeff King wrote:


This is exactly what I tried to avoid, and why I did _not_ do the 
"correct" fix.  In some real-life (i.e. non-OpenSource) repositories you 
do get a mess, and you do get quite a lot of messy merges.  So I am not at 
all convinced that this holds together in such setups.

Besides, name-rev is already a memory hog.  Your patch makes it worse.

IOW I think that my patch is a good trade off between correct and working.

Ciao,
Dscho

-
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:
name-rev does not show the shortest path, Uwe , (Thu Aug 23, 6:38 am)
[PATCH] name-rev: Fix non-shortest description, Johannes Schindelin, (Mon Aug 27, 7:37 am)
Re: [PATCH] name-rev: Fix non-shortest description, Jeff King, (Tue Aug 28, 4:20 am)
Re: [PATCH] name-rev: Fix non-shortest description, Johannes Schindelin, (Tue Aug 28, 5:15 am)
Re: [PATCH] name-rev: Fix non-shortest description, Jeff King, (Tue Aug 28, 5:25 am)
Re: [PATCH] name-rev: Fix non-shortest description, Johannes Schindelin, (Tue Aug 28, 6:03 am)
Re: [PATCH] name-rev: Fix non-shortest description, Jeff King, (Tue Aug 28, 6:26 am)
Re: [PATCH] name-rev: Fix non-shortest description, Johannes Schindelin, (Tue Aug 28, 7:02 am)
Re: [PATCH] name-rev: Fix non-shortest description, Jeff King, (Tue Aug 28, 7:11 am)
Re: [PATCH] name-rev: Fix non-shortest description, Johannes Schindelin, (Tue Aug 28, 7:28 am)
Re: [PATCH] name-rev: Fix non-shortest description, Jeff King, (Tue Aug 28, 7:34 am)
Re: [PATCH] name-rev: Fix non-shortest description, Uwe , (Mon Aug 27, 3:27 pm)
Re: [PATCH] name-rev: Fix non-shortest description, Johannes Schindelin, (Tue Aug 28, 4:39 am)
Re: [PATCH] name-rev: Fix non-shortest description, Uwe , (Tue Aug 28, 4:54 am)
Re: [PATCH] name-rev: Fix non-shortest description, Jeff King, (Tue Aug 28, 5:00 am)
Re: [PATCH] name-rev: Fix non-shortest description, Uwe , (Tue Aug 28, 5:17 am)
Re: [PATCH] name-rev: Fix non-shortest description, Jeff King, (Tue Aug 28, 5:26 am)
Re: name-rev does not show the shortest path, Julian Phillips, (Fri Aug 24, 7:55 am)
Re: name-rev does not show the shortest path, Uwe , (Fri Aug 24, 8:52 am)
Re: name-rev does not show the shortest path, Julian Phillips, (Fri Aug 24, 11:21 am)
Re: name-rev does not show the shortest path, Junio C Hamano, (Fri Aug 24, 2:33 pm)
Re: name-rev does not show the shortest path, Johannes Schindelin, (Sat Aug 25, 11:04 am)
Re: name-rev does not show the shortest path, Jeff King, (Sun Aug 26, 5:23 am)
Re: name-rev does not show the shortest path, Johannes Schindelin, (Sun Aug 26, 11:38 am)
Re: name-rev does not show the shortest path, Jeff King, (Mon Aug 27, 5:24 am)
Re: name-rev does not show the shortest path, Johannes Schindelin, (Mon Aug 27, 5:57 am)
Re: name-rev does not show the shortest path, Johannes Schindelin, (Mon Aug 27, 7:18 am)

Mail archive search
Enter your search terms.
Optionally limit your search to a specific mailing list.
advanced
Colocation donated by:
Who's online
There are currently 3 users and 1310 guests online.

Online users

Syndicate
Syndicate content
© 2007 KernelTrap.  |  Powered by Drupal.  |  Legal statement.
speck-geostationary