Re: [PATCH 3/2] format-patch: use clear_commit_marks() instead of some adhocery

On 6/27/06, Johannes Schindelin [off-list ref] wrote:

Hi,

On Tue, 27 Jun 2006, Martin Langhoff wrote:

quoted

And it errors out with ignore-if-in-upstream:

 $ ./git format-patch --ignore-if-in-upstream -o .patches origin master
 fatal: Not a range.

Could you test with "origin..master" instead of "origin master"?

Funny you mention that! Now it works ;-) and it even produces the
patches I would expect. There is something strange though. I have a
repo with ~150 pending patches to push, of which git-cherry spots ~100
as already merged upstream. So the old git-format-patch.sh would spit
50 patches, and the initial C version would do 150.

Now this version gives me 50 patches, regardless of
--ignore-if-in-upstream. Is that expected?

Also, if the two heads are identical, it still says 'Fatal: Not a
range", but that isn't so important.

cheers,


martin