Re: [PATCH v2 1/2] Add for_each_remote() function, and extend remote_find_tracking()
- From
Junio C Hamano <gitster@pobox.com>
- Date
- Jul 10, 2007, 18:38 UTC
- Message-ID
- <7vtzscqf2r.fsf@assigned-by-dhcp.cox.net>
- In-Reply-To
- <Pine.LNX.4.64.0707101848050.4047@racer.site>
Johannes Schindelin <Johannes.Schindelin@gmx.de> writes:
Show 5 quoted lines
> The function for_each_remote() does exactly what the name suggests. > > The function remote_find_tracking() was extended to be able to search > remote refs for a given local ref. You have to set the parameter > "reverse" to true for that behavior.
The updated patch does not use "reverse" but the old description is still there.
Daniel, one thing I fear about your "I want to store the message in the object store so that I can reuse even after I re-polish the series" desire on the cover letter topic is this kind of gotcha, and that is why I suggested "*** BLURB GOES HERE ***". Both the summary (diffstat and shortlog) part and the description part should be kept fresh in the updated 0/N; while we can automate the summary part whenever we re-generate 0/N, you cannot automate the description part.