Re: [PATCH 0/2] log/ format-patch improvements

2 messages, 2 authors, 2016-06-15 · open the first message on its own page

Re: [PATCH 0/2] log/ format-patch improvements

From: Junio C Hamano <hidden>
Date: 2016-06-15 22:49:23

Jonathan Nieder [off-list ref] writes:
Ramkumar Ramachandra wrote:
...
quoted
Good idea. I'll write a patch. Do we also want people to be able to
turn off `--no-merges`?
I don't see a need for it.

However, if you can think of good names for --undo-no-merges and
--undo-merges options to "git log", that might be a nice independent
change for the revision option parser.
Wait a bit.  How would you represent a merge in a patch form that can be
read by "git apply" (and "patch") in the first place?

Re: [PATCH 0/2] log/ format-patch improvements

From: Ramkumar Ramachandra <hidden>
Date: 2016-06-15 22:49:23

Hi Junio,

Junio C Hamano writes:
Jonathan Nieder [off-list ref] writes:
quoted
Ramkumar Ramachandra wrote:
...
quoted
Good idea. I'll write a patch. Do we also want people to be able to
turn off `--no-merges`?
I don't see a need for it.

However, if you can think of good names for --undo-no-merges and
--undo-merges options to "git log", that might be a nice independent
change for the revision option parser.
Wait a bit.  How would you represent a merge in a patch form that can be
read by "git apply" (and "patch") in the first place?
Oh, I'm not attempting that. I merely wanted people to be able to turn
off `--no-merges` so they can get the current functionality.

As far as represeting merge commits go, I'm still thinking about it. I
talked to Thomas about it briefly on IRC. The main issues:
1. What's the point? When there's no conflict resolution, a merge
   commit would be empty.
2. How do we uniquely specify what to merge? We can't use branch names
   or commit SHA1s, because they can change.

-- Ram
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help