Re: [msysGit] Re: [PATCH 12/12] [TODO] setup: bring changes from 4msysgit/next to next
- From
Johannes Sixt <johannes.sixt@telecom.at>
- Date
- Jul 2, 2008, 19:32 UTC
- Message-ID
- <200807022132.27751.johannes.sixt@telecom.at>
- In-Reply-To
- <D35A2542-3943-4BDB-AEDA-0F8B7052EF7D@zib.de>
On Mittwoch, 2. Juli 2008, Steffen Prohaska wrote:
Show 19 quoted lines
> On Jul 2, 2008, at 6:17 PM, Johannes Schindelin wrote: > > On Wed, 2 Jul 2008, Steffen Prohaska wrote: > >> From: Johannes Sixt <johannes.sixt@telecom.at> > >> > >> Hannes, > >> You introduced "minoffset" in 861429a7c37c7. > > > > AFAICT it was redone differently in 'next', because 'next' has this > > ceiling dir thingie, which allows a different (much smaller) patch. > > > > It might be more sensible to base your patch series on 'next'... > > Hmm.. it is based on next. But obviously I needed to merge > mingw's master to 4msysgit's master and resolve conflicts. > Maybe I made the wrong decisions then. > > Hannes, > If you believe that your setup.c is good, then I'll copy your version > to 4msysgit's master.
The setup.c in mingw.git (and soon Junio's master) and Junio's next are _different_, but both are correct. If you reverse-apply the patch you presented here, then you get the version from Junio's next, which is a good state.
[ Of course, the result will work only as long as you do not set GIT_CEILING_DIRECTORIES, because we haven't taken care of the helper functions that this feature uses, longest_ancestor_length() and normalize_path(). ]
We have debated about set_git_dir(".") in the past. mingw.git doesn't have it, and it works (for me). I don't know what it is needed for.
-- Hannes