Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

addons: git: pass --no-find-copies --no-find-copies-harder --find-ren… #618

Closed
wants to merge 1 commit into from

Conversation

thesamesam
Copy link
Member

…ames to git log

I currently have a local git hack to allow configuring git to default to --find-copies-harder because it's extremely useful when working on ebuild repositories (prompted by a discussion with Eli Schwartz).

Unfortunately, this can confuse pkgcheck's git intergration because it'll call git log like:

git log --name-status --diff-filter=ARMD -z --pretty=tformat:%n%h%n%ct cc5b3b9f134a070c548faa4e3de17d615497d0b3..origin/HEAD

and get nothing back because (I think) git is interpreting some changes as copies rather than renames or new files.

Explicitly pass options to disable finding copies, even though normally this isn't necessary, to keep things working.

…ames to git log

I currently have a local git hack to allow configuring git to default to
--find-copies-harder because it's *extremely* useful when working on ebuild
repositories (prompted by a discussion with Eli Schwartz).

Unfortunately, this can confuse pkgcheck's git intergration because it'll
call `git log` like:
```
git log --name-status --diff-filter=ARMD -z --pretty=tformat:%n%h%n%ct cc5b3b9f134a070c548faa4e3de17d615497d0b3..origin/HEAD
```
and get nothing back because (I think) git is interpreting some changes as copies
rather than renames or new files.

Explicitly pass options to disable finding copies, even though normally this
isn't necessary, to keep things working.

Signed-off-by: Sam James <sam@gentoo.org>
@gentoo-bot gentoo-bot closed this in e688357 Sep 8, 2023
@thesamesam thesamesam deleted the git-hax branch September 8, 2023 12:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants