git.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Johannes Schindelin <Johannes.Schindelin@gmx.de>
To: Junio C Hamano <gitster@pobox.com>
Cc: git@vger.kernel.org
Subject: Re: [PATCH] Teach filter-branch about subdirectory filtering
Date: Fri, 8 Jun 2007 06:04:58 +0100 (BST)	[thread overview]
Message-ID: <Pine.LNX.4.64.0706080603520.4059@racer.site> (raw)
In-Reply-To: <7vzm3bmfeq.fsf@assigned-by-dhcp.cox.net>

Hi,

On Thu, 7 Jun 2007, Junio C Hamano wrote:

> Good thing you folded git-split into this, which is an obviously right 
> thing to do.

So my secritt evvil plann did not stay secritt that long.

> But it makes me wonder, if it is common to want to split a combined 
> project into two (or more) sets, taking more than one directories at a 
> time.

It's just a guess, but now that we come really, really close to having a 
concise implementation of git-subproject which will probably soon 
propagate to master, and then maint, I gather that more and more people 
come and want to split their projects (which they maintained as one big 
project) into several subprojects (which they should have been from the 
start, but the tool did not easily allow for that).

> We might however want to make sure that we detect an attempt to give 
> more than one --subdirectory-filter parameter.

Yeah. But then, I think that rev-list actually does not like to be passed 
a file/directory which does not exist currently, so that is probably a 
sensible safeguard against that usage.

Unless you think -- as I fear you do -- that people could think 
git-filter-branch (its name not withstanding) filters _multiple_ branches 
_at once_.

That case I did not foresee, and you're right, I should output a big fat 
error for the case when somebody says "filter-branch --subdirectory-filter 
a/ --subdirectory-filter b/". It does not make any sense. But humans are 
known for their unique feeling that things should work, even if they make 
no sense at all, and their brain should have warned them (that it is 
unlikely, at best, to work), but failed to do so.

Ciao,
Dscho

  reply	other threads:[~2007-06-08  5:08 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-06-08  0:30 [PATCH] Teach filter-branch about subdirectory filtering Johannes Schindelin
2007-06-08  0:57 ` Junio C Hamano
2007-06-08  5:04   ` Johannes Schindelin [this message]
2007-06-08  7:19     ` Johannes Sixt
2007-06-08 13:03       ` Johannes Schindelin
2007-06-08 13:19 ` Johannes Sixt
2007-06-08 21:05 ` Alex Riesen

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=Pine.LNX.4.64.0706080603520.4059@racer.site \
    --to=johannes.schindelin@gmx.de \
    --cc=git@vger.kernel.org \
    --cc=gitster@pobox.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).