Re: Suggestion: make --left-right work with --merge

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Paul Mackerras <paulus@xxxxxxxxx> writes:

> Junio C Hamano writes:
>
>> Try it on user inputs like "master..next", "next...master".  You
>> wanted to grab only the positive ones, no?
>
> No... gitk passes the IDs it gets from git rev-parse (both positive
> and negative) to git log, rather than the original arguments.

I am not sure if I was answering the right question, then.

I thought the issue you were addressing was this.

 (1) the user says "gitk master...next" to start you.

 (2) you run "git log master...next" (or whatever the equivalent of what
     the user gave you) and draw.

 (3) the user does things outside your control to modify refs, and says
     "Update".

 (4) you could re-run "git log master...next" again, but that would show
     mostly what you already know.

     If you saved all the positive ones you used in (2), you could instead
     run "git log master...next --not <positives in 2>" to ask only the
     incremental changes (as long as the branches do not rewind, but you
     do not discard nodes from the already drawn graph anyway).

     <positives in 2> in this example sequence would be the object names
     for master and next when (2) was run.

I think your "gitk passes the IDs" part is about the master...next part in
the above example.  I do not think it really matters if they are converted
to object names or kept symbolic when given to "git log".  I was talking
about the part you add after --not in the second round, and that was why
my answer was only about positive ones.

Come to think of it, when you are told to "Update", you already know the
positive tips you can use to optimize (4), don't you?  They are the
commits you drew in (3) that do not have children.
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]

  Powered by Linux