how to most effectively cherry pick by selective patch hunk?

  (i'm looking for a solution not just for current git but, sadly,
going back to git-2.9.2, which is installed on my current contract
build system, and i have little authority to bump it up.)

  summary: made a couple dozen commits on branch, call it "oldb",
where i was relatively undisciplined about enforcing clean, modular
commits so i want to go back and clean things up -- refactor by
changing order, combining some trivial commits into one, breaking
large, unwieldy commits into smaller pieces, better commit messages
and so on, so i start a new branch "newb" at the same origin, and
here's the problem.

  every old commit consisted of adding a new patch to an existing
openembedded recipe, so every commit had two components:

  * a brand new patch file to be placed under "files/", and
  * adding a new line to SRC_URI variable, as in:

    SRC_URI += " \
	first.patch \
	second.patch \
	third.patch \
	... etc etc ...
    "

  i think you see the problem. a commit adding a brand new file will
never create a merge conflict, as it's a new file. but if i start
reordering commits, then the addition of that line to the .bbappend
file will *certainly* conflict as the patches will almost certainly be
renamed and in a different order.

  what would be great is some sort of "-p" (patch selection) option
with cherry-pick, but i don't see that.

  what would work for me is to auto-get the addition of the patch file
from the old branch, at which point i am more than happy to manually
fix the .bbappend file and manually do another commit. i'm thinking i
can just "git checkout" the new patch file from the old branch, and
take it from there.

  thoughts? am i overthinking this?

rday

On Tue, Feb 09, 2021 at 08:58:06AM -0500, Robert P. J. Day wrote:

  what would be great is some sort of "-p" (patch selection) option
with cherry-pick, but i don't see that.

We have "checkout -p", but of course the problem there is that it's
picking out of the whole state of that commit. So you might see other
changes not introduced by that commit.

Conceptually, adding "cherry-pick -p" would be pretty easy. The strategy
for all of the "-p" options is to generate a diff, then feed that diff
to the patch-selection code, then apply whatever the user selects. For
"checkout -p $commit", that diff is the diff between $commit and our
current state. But for "cherry-pick -p", it would be the diff between
$commit^ and $commit.

Of course that involves a change to Git, and you were looking for
something you could do with existing versions. :) You can emulate it by
making the commit's parent equivalent to your current state. I.e.:

  git checkout --detach ;# detached HEAD for temporary commit
  git cherry-pick $commit ;# maybe deal with conflicts
  commit=$(git rev-parse --verify HEAD) ;# remember the temp commit
  git checkout - ;# back to your branch
  git checkout -p $commit

-Peff

On Tue, 9 Feb 2021, Jeff King wrote:

On Tue, Feb 09, 2021 at 08:58:06AM -0500, Robert P. J. Day wrote:

quoted

  what would be great is some sort of "-p" (patch selection) option
with cherry-pick, but i don't see that.

We have "checkout -p", but of course the problem there is that it's
picking out of the whole state of that commit. So you might see
other changes not introduced by that commit.

  this is probably what i'll use since i can use <pathspec> to grab
only the patch file from that commit, then add and commit it manually
from there.

rday