Re: [PATCH 1/3] Move bundle specific stuff into bundle.[ch]

!MAILaRCHIVE_VOTE_RePLACE
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
To: Shawn O. Pearce <spearce@...>
Cc: Daniel Barkalow <barkalow@...>, Johannes Schindelin <Johannes.Schindelin@...>, <git@...>
Date: Wednesday, July 18, 2007 - 3:26 am

"Shawn O. Pearce" <spearce@spearce.org> writes:


I have to disgree.

We on the git list *live* in pretty much git-only world (ok, I
am ignoring git-svn people for now ;-)), so it might feel it
beneficial to add a new output format to git-diff.

But many users of git generated patch needs to interact with
patches that are not generated by git.  I think an additional
postprocessor in patchutils/interdiff toolchest would probably
make much more sense.  Then a person who reviews a patch that is
supposed to be line movement can use the filter to verify the
parts that should be only line movements are indeed are
movements and nothing else.

Speaking of "diff" output, what I would like to see is a support
of 'copied context' (i.e. traditional 'context diff' format you
would get from "diff -c"), in addition to the 'unified context'
we support.  Generating copied context may help people who need
to give patches to others that accept only copied context format
patches (are there important projects that take only cpied
context format patches?).  Being able to accept copied context
format patches in 'git-apply' would also be handy.

Of course, we could use interdiff to mangle copied context
format into unified context format, so 'git-apply' is not a big
deal, though.

-
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
[PATCH 1/3] Move bundle specific stuff into bundle.[ch], Johannes Schindelin, (Tue Jul 17, 6:49 pm)
Re: [PATCH 1/3] Move bundle specific stuff into bundle.[ch], Daniel Barkalow, (Tue Jul 17, 10:48 pm)
Re: [PATCH 1/3] Move bundle specific stuff into bundle.[ch], Johannes Schindelin, (Wed Jul 18, 5:56 am)
Re: [PATCH 1/3] Move bundle specific stuff into bundle.[ch], Daniel Barkalow, (Wed Jul 18, 12:25 pm)
Re: [PATCH 1/3] Move bundle specific stuff into bundle.[ch], Johannes Schindelin, (Wed Jul 18, 12:33 pm)
Re: [PATCH 1/3] Move bundle specific stuff into bundle.[ch], Daniel Barkalow, (Tue Jul 17, 11:23 pm)
Re: [PATCH 1/3] Move bundle specific stuff into bundle.[ch], Johannes Schindelin, (Wed Jul 18, 6:09 am)
Re: [PATCH 1/3] Move bundle specific stuff into bundle.[ch], Shawn O. Pearce, (Tue Jul 17, 11:29 pm)
Re: [PATCH 1/3] Move bundle specific stuff into bundle.[ch], Shawn O. Pearce, (Wed Jul 18, 2:19 am)
Re: [PATCH 1/3] Move bundle specific stuff into bundle.[ch], Junio C Hamano, (Wed Jul 18, 3:26 am)