system2() call "git" leads to issue on Windows

[psychelzh]

Prework

• Read and agree to the Contributor Code of Conduct and contributing guidelines.
• Confirm that your issue is a genuine bug in the gittargets package itself and not a user error, known limitation, or issue from another package that gittargets depends on. For example, if you get errors running tar_make_clustermq(), try isolating the problem in a reproducible example that runs clustermq and not gittargets. And for miscellaneous troubleshooting, please post to discussions instead of issues.
• If there is already a relevant issue, whether open or closed, comment on the existing thread instead of posting a new issue.
• Post a minimal reproducible example like this one so the maintainer can troubleshoot the problems you identify. A reproducible example is:
  • Runnable: post enough R code and data so any onlooker can create the error on their own computer.
  • Minimal: reduce runtime wherever possible and remove complicated details that are irrelevant to the issue at hand.
  • Readable: format your code according to the tidyverse style guide.

Description

This is just the same as yihui/crandalf#24. On Windows OS, tar_git_ok() complains even if the user name and email are configured globally. This is simply because R for Windows sets HOME environment as Documents folder. Hopefully this is taken care of here.

Reproducible example

gittargets::tar_git_ok()
#> ✔ Git binary: 'C:\PROGRA~1\Git\cmd\git.exe'
#> ✖ No Git config global user email. See details in ?tar_git_ok().
#> ✖ No Git config global user email. See details in ?tar_git_ok().
#> [1] FALSE

^{Created on 2022-08-20 with reprex v2.0.2}

[psychelzh]

Maybe an alternative is to use "gert" package?

[wlandau]

Thanks for letting me know. Not sure why I forgot to use processx::run() on gittargets:::tar_git_binary() instead of system2("git", ...), but the latest commits should fix it.

Unfortunately I can't use gert because libgit2 is incompatible with Git LFS.

[wlandau]

Seems I misread the docs of processx::run(). Look like we'll need to temporarily set HOME after all.

[wlandau]

Looks to be fixed this time: https://github.com/ropensci/gittargets/runs/7939003978