git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: [PATCH] filter-branch: do not consider diverging submodules a 'dirty worktree'

From
Johannes Schindelin <johannes.schindelin@gmx.de>
Date
Feb 4, 2009, 17:24 UTC
Message-ID
<alpine.DEB.1.00.0902041823330.22763@intel-tinevez-2-302>
In-Reply-To
<7vvdrqunog.fsf@gitster.siamese.dyndns.org>
Hi,
On Wed, 4 Feb 2009, Junio C Hamano wrote:
Show 13 quoted lines
> Johannes Schindelin <johannes.schindelin@gmx.de> writes:
> 
> > As filter-branch could not care less about submodules' actual 
> > contents, it does not make sense to check if the checked-out 
> > submodules are up-to-date before running filter-branch.  So do not do 
> > it.
> 
> Sorry, but I am confused.  Is that because even the tree-filter does not 
> use the actual work tree but works in the temporary area .git-rewrite, 
> and a diverged submodule cannot possibly matter (and index-filter works 
> solely on the index anyway)?
> 
> If so, why do we even check dirtiness of anything at all?

I guess it is because we could update the working directory with read-tree -u -m HEAD at the end. Actually, that is exactly what we do.

Submodules do not matter much here, as nothing will be overwritten, really, but a dirty working directory matters.

Ciao, Dscho

Previous: Junio C HamanoNext: Johannes Sixt
Message 3 of 9 in “filter-branch: do not consider diverging submodules a 'dirty worktree'”
  1. filter-branch: do not consider diverging submodules a 'dirty worktree'Johannes Schindelin, Feb 4, 2009
  2. Junio C HamanoFeb 4, 2009
  3. Johannes SchindelinFeb 4, 2009
  4. Johannes SixtFeb 4, 2009
  5. Junio C HamanoFeb 4, 2009
  6. Johannes SchindelinFeb 4, 2009
  7. Junio C HamanoFeb 4, 2009
  8. Johannes SchindelinFeb 5, 2009
  9. Johannes SixtFeb 5, 2009

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.