Set the git identity before patching

[mrexodia]

When using sleigh as a dependency it fails in the GitHub Actions pipeline with an error like this:

HEAD is now at 9c724c1a Merge remote-tracking branch 'origin/GP-2355_AARCH64_thunk_pattern' into patch
[2/9] Performing update step for 'ghidrasource-populate'
[3/9] Performing patch step for 'ghidrasource-populate'

Committer identity unknown

*** Please tell me who you are.

Run

  git config --global user.email "you@example.com"
  git config --global user.name "Your Name"

to set your account's default identity.
Omit --global to set the identity only in this repository.

fatal: empty ident name (for <runner@xxx>) not allowed
FAILED: ghidrasource-populate-prefix/src/ghidrasource-populate-stamp/ghidrasource-populate-patch

For now I just set it the same as in main.yml, but let me know if there should be some logic there. Perhaps it should only set the identity if there is none set globally or something...

[tetsuo-cpp]

For now I just set it the same as in main.yml, but let me know if there should be some logic there. Perhaps it should only set the identity if there is none set globally or something...

I don't see any harm in doing it unconditionally. What do you think @ekilmer?

[ekilmer]

LGTM! Thank you!

I have also been setting this in my CI yaml, but I've also run into this issue with Dockerfiles and other temporary build environments that don't have it set up.

I also agree that setting it unconditionally won't hurt or change anything much.