File history lists unrelated merge commits

[minj]

• GitLens Version: 7.5.6
• VSCode Version: 1.19.2
• OS Version: Manjaro Linux

Steps to Reproduce:

1. https://github.com/minj/foxtrick/blob/master/content/core.js
2. toggle file history
3. compare with https://github.com/minj/foxtrick/commits/master/content/core.js

This might be related to #238

However, compare these:

git log -7 --oneline --decorate -- content/core.js
aca790215 Merge pull request #1495 from minj/web-ext-1339
4249db6bd rewrite the clipboard copying to use the stackoverflow hack
6b885f7c4 Merge pull request #1469 from minj/epic-new-live-1434
1cb90b6a7 adjust shortTeamName mechanism to HT changes
785705895 (clear-cache-1378) automatically clear cache when season changes
873a80e5d move HT_TIME detection before jsLink
ce0636bfd (origin/pulls/1389) put a final touch on things

git log -7 --oneline --decorate --simplify-merges -- content/core.js
aca790215 Merge pull request #1495 from minj/web-ext-1339
4249db6bd rewrite the clipboard copying to use the stackoverflow hack
6b885f7c4 Merge pull request #1469 from minj/epic-new-live-1434
1cb90b6a7 adjust shortTeamName mechanism to HT changes
785705895 (clear-cache-1378) automatically clear cache when season changes
873a80e5d move HT_TIME detection before jsLink
ce0636bfd (origin/pulls/1389) put a final touch on things

git log -7 --oneline --full-history -- content/core.js
4855c9cc0 (HEAD -> master, origin/master, origin/HEAD) Merge remote-tracking branch 'origin/cdn'
656c70157 (origin/pulls/1538) Merge branch 'minor-fixes'
58b73809e Merge branch 'promotion-diff-fix'
ea9f6613e Merge branch 'cws-login-fix'
5ca175474 Merge remote-tracking branch 'origin/cdn'
14ed47efe Merge remote-tracking branch 'origin/cdn'
b5fa6bc1b Merge remote-tracking branch 'origin/cdn'

Evidently, --simplify-merges should not change anything here. GitLens works in --full-history mode for some reason though

[minj]

That was fast, thanks

[ian-h-chamberlain]

I'm still seeing this behavior with GitLens 7.5.9 (or 8.0.0 beta); the list for "Show File History" appears to match the output of git log -7 --oneline --full-history.

@eamodio @minj can either of you confirm that you're no longer seeing it? I'm wondering if I'm missing a setting or something...

[minj]

I haven't tried beta, the commit looked "reasonable" so I figured I would wait for a stable release.

[eamodio]

I definitely removed --full-history from the file log calls. If you run git log -7 --oneline --full-history vs git log -7 --oneline do you see a difference?

[ian-h-chamberlain]

I agree that the commit looks correct, but this is what I'm getting:

$ git log -7 --oneline -- src/lib/app/Application.cpp
f01bb7162a Modify cpp files based on clang-tidy
b7730b83b4 I95-14446 : Formatted C++ codebase
a2feb303e6 Create auditdReceiver
85ba17e106 Use the appropriate log category per PollerThread and ConnectionManager
828775605e Use the appropriate log category per application
1f11506e25 Handle Application state changes more carefully
54add446e8 Prevent shutdown before Application is fully constructed and run()

$ git log -7 --oneline --full-history --  src/lib/app/Application.cpp
15502a9f15 (origin/I95-17770-merge-dp-3.1.11-13) Merge PR#5495 (athompson/I95-18130-remove-128t-salt-minion-dep=>release/devilspurse) to develop
922208ea6a Merge PR#5512 (acormier/TOOL-1608-migrate-validatedb=>release/devilspurse) to develop
86e7b7bf83 Merge PR#5533 (I95-18013-initializer-daemon-reload=>release/devilspurse) to develop
591f142491 Merge PR#5534 (dmon/I95-18113-pcapinjectiontest-devilspurse=>release/devilspurse) to develop
68b153b24d (origin/I95-19770-merge-dp-pr5485) Merge PR#5485 (I95-16608-router-register-with-node-name=>release/devilspurse) to develop
b6323a07fa Merge PR#5516 (I95-18160-copy-cert-every-time=>release/devilspurse) to develop
be9b542d47 Merge PR#5473 (dmon/I95-18007-fib-only-supports-7000-entries=>release/devilspurse) to develop

[eamodio]

GitLens runs something along the lines of the following:
git log -7 --name-status -M --oneline --follow -m -- src/lib/app/Application.cpp

Can you send the output of that and also:
git log -7 --name-status -M --oneline --follow -- src/lib/app/Application.cpp

FYI, if you set "gitlens.debug": true" in your settings, there will be a new GitLens (Git) output channel in the output pane and you can see every git command GitLens runs

[ian-h-chamberlain]

Thanks for the tip about gitlens.debug.
I realized that the history of that particular file was actually not the best example for this, but the same problem seems to apply with another file, which has some merge commits in its recent history.

The actual command that gets run when I "Show file history":
git log --name-status -M --format=%x3c%x2ff%x3e%n%x3cr%x3e %H%n%x3ca%x3e %an%n%x3ce%x3e %ae%n%x3cd%x3e %at%n%x3cp%x3e %P%n%x3cs%x3e%n%B%n%x3c%x2fs%x3e%n%x3cf%x3e --follow -n200 -m -- src/lib/protobuf/state.proto

And here's the output of those two commands, it looks like the -m flag is the one pulling in these extra merges:

$ git log -7 --name-status -M --oneline --follow -m -- src/lib/protobuf/state.proto
6315a70d11 (from 6071a2e4ec) Merge branch 'develop' into ichamberlain/I95-17783-disable-auto-entitlement
M       src/lib/protobuf/state.proto
e28beb5f76 (from 346e980a78) (origin/rkoss/I95-18237-build-error-pserviceagent, origin/develop, origin/chase/I95-18219-ipv4-to-v6-df-bit-lost, origin/bob/I95-16139-warn-tenant-prefix, origin/ST-353-performance-collection-1210-basic, origin/HEAD, develop) Merge pull request #5518 in T128/i95 from jbogdanov/I95-18149-show-config-version-timeout to develop
M       src/lib/protobuf/state.proto
2602eaffb7 (from 9324915589) Merge pull request #5399 in T128/i95 from I95-17982/amenon/end-ha-test-2 to develop
M       src/lib/protobuf/state.proto
2cfeefc1e5 (from 2e77111a27) Merge pull request #5527 in T128/i95 from aware/I95-16866-source-tenant-ui to develop
M       src/lib/protobuf/state.proto
660fbe52cc (from 0827cfc292) (origin/rpatterson/I95-18210-cleanup-dnsManager, origin/chao/I95-18218-auto-generated-pppoe-hostname-robot) Merge pull request #5543 in T128/i95 from dbuchanan/stop-inactivity-query to develop
M       src/lib/protobuf/state.proto
e64041267d (from aa753e0488) (origin/gjordan/I95-18017-resolve-merge-conflict) Merge branch 'develop' into gjordan/I95-18017-resolve-merge-conflict
M       src/lib/protobuf/state.proto
bb9be224ca (from 07f20d16ae) (origin/athompson/I95-18223-installer-dep-downgrade-issues) Merge pull request #5523 in T128/i95 from athompson/I95-17796-installer-log-yum-output to develop
M       src/lib/protobuf/state.proto

vs

$ git log -7 --name-status -M --oneline --follow -- src/lib/protobuf/state.proto
5bff46cd1a *) Changing prefix-length type *) Adding C++ tests *) Making prefix-length non-mandatory *) Adding revision information
M       src/lib/protobuf/state.proto
97591e20b0 *) show_network_interface changes *) dhcp info prefix propagation to state
M       src/lib/protobuf/state.proto
0cc554d238 Add host-name backend support for show peers command
M       src/lib/protobuf/state.proto
606a883fbd Add filter for interface physical address in state monitor
M       src/lib/protobuf/state.proto
4cecf9a80b Reserve public_ip_address in state.proto
M       src/lib/protobuf/state.proto
d6f378f8cc Revert "Merge pull request #4372 in T128/i95 from ichamberlain/I95-16536-show-asset-ip-datamodel to release/devilspurse"
M       src/lib/protobuf/state.proto
ce575242ea Add duplicate ids to asset state service response
M       src/lib/protobuf/state.proto

I'm not sure what the desired output is, based on looking at the actual history of the file. The best result seems for me to be this, although the merge commits show as empty because of other commits (the ones that were merged in) that makes those changes.

$ git log -7 --name-status -M --oneline --stat  -- src/lib/protobuf/state.proto
65a35b02d5 Merge PR#5451 (rherrero/I95-17489-show-network-interface-prefix=>release/devilspurse) to develop
5bff46cd1a *) Changing prefix-length type *) Adding C++ tests *) Making prefix-length non-mandatory *) Adding revision information
M       src/lib/protobuf/state.proto
97591e20b0 *) show_network_interface changes *) dhcp info prefix propagation to state
M       src/lib/protobuf/state.proto
0cc554d238 Add host-name backend support for show peers command
M       src/lib/protobuf/state.proto
60be503e2d Merge PR#5185 (tjb/I95-16468-pcli-mac-address-unavailable=>release/devilspurse) to develop
606a883fbd Add filter for interface physical address in state monitor
M       src/lib/protobuf/state.proto
40c7abdc3c Merge PR#4826 (ichamberlain/I95-17077-detect-duplicate-minion-ids=>release/devilspurse) to develop

I guess it comes down to how you want to implement the "File History" list. Certainly, I don't think it makes sense to show merge commits that didn't affect the file, but it may be difficult to show all merge commits that did since some of them will be "empty" according to git.

Hope this helps!

[eamodio]

Thank you so much for the details! Seems like the -m flag is doing more than it is supposed to. Because I do want the file information for the merge commit IF it affected the file. I'll dig into this a bit more.

For the file you posted could you also check that file history on GitHub and let me know which one it matches? Thanks!