合并后文件的 git 提交中的更改消失答案

【问题标题】：Change in a git commit of a file disappears after merge合并后文件的 git 提交中的更改消失
【发布时间】：2017-05-26 07:14:53
【问题描述】：

我正在开发一个功能分支。那时我做了一些其他更改，这些更改已经通过 github 中的 PR 合并到开发分支中，我需要在我的功能分支中进行这些更改。所以我决定将开发分支合并到我的功能分支中。我删除了文件中的一些代码并在功能分支中提交。合并develop到feature分支后，发现文件commit中删除代码的变化消失了，删除的部分由于合并操作又加回来了。请问：在什么情况下会出现这种情况？谢谢！

更新：

我使用 github 来查看历史。这是示例：这是功能分支OWL-1697 中提交80d82ee 中agent.go 的history。这是功能分支OWL-1697 中此文件的整个history。合并操作后，80d82ee对该文件的更改消失了。

【问题讨论】：

git merge 从不删除任何提交。为什么您认为它确实如此——或者更具体地说，您使用什么来查看存储库中的提交？为了不向您展示“无聊”的提交，您用来查看它们的任何内容都可能对您撒谎。至于问题的另一半，合并可能会删除 code （甚至整个文件），这取决于您要合并的内容以及您请求合并的方式。无论如何，请考虑尝试提供minimal reproducible example。
我使用github查看历史。这是示例：这是功能分支OWL-1697 中提交80d82ee 中agent.go 的history。这是功能分支OWL-1697 中此文件的整个history。合并操作后，80d82ee对该文件的更改消失了。

标签： git github merge rebase

【解决方案1】：

我不建议使用 GitHub 查看提交和文件历史记录。（GitHub 可能有一些不错和/或奇特的方式来查看这些，但作为一个非常随意的 GitHub 用户，我不知道有哪一种。命令行工具有很多标志，尽管你必须是 Git 专家才能知道使用哪个，何时使用。）

TL;DR 总结了合并出错的原因

阅读下面的文本后，请特别注意git log 输出的最后几行：

* | | | | | 80d82eef [OWL-1730][common][nqm-mng] Code refactoring
* | | | | | 2438360a [OWL-1667][common][nqm-mng] From `hbs` to `nqm-mng`
| |_|/ / /  
|/| | | |   
* | | | | b598349a Merge pull request #295 from masato25/OWL-1724

在提交2438360a 中，您添加了一些行，然后在提交80d82eef 中，您再次将它们取出。

但是，您通过提交 d04492d9:

添加了 same 行

| | * | d04492d9 [OWL-1667][common][nqm-mng] From `hbs` to `nqm-mng`

有点高。您在 2438360a 中添加的行的同一天执行此操作。你在80d82eef 中取出了线条，很久以后......但是在不同的分支。在拓扑顺序中，您在放入更改后几乎立即将其取出......同时还将更改留在单独的分支中。然后，您通过e1d369d0 将它们带入另一个分支，然后将它们带入另一个分支，依此类推，直到您将它们直接带回您认为已将它们带出的分支。

查看提交图

如果我克隆您的存储库并使用 git log --all --decorate --oneline --graph 查看它，我会看到部分内容，其中包括提交 80d82eef（在底部）：

* aeecad13 (origin/OWL-1697) [OWL-1730][nqm-mng] Update debug message
* f0797f8e [OWL-1730][nqm-mng] Fix the proble from merge
* 5aaba93b [OWL-1730][nqm-mng] Drop gin's "gopkg.in" paths

[mass snip]

* | | | d2d4debd [OWL-1730][nqm-mng] Tests
* | | | e1037c28 [OWL-1730][nqm-mng] Queue service
* | | |   a847d783 Merge branch 'OWL-1677' into OWL-1697
|\ \ \ \  
| * \ \ \   ea916618 Merge branch 'develop' into OWL-1677
| |\ \ \ \  
| | |/ / /  
| * | | | 056a7442 [OWL-1677] Refactoring testing to usage of Ginkgo
| * | | | 800c3f09 [OWL-1677] Add utilities for testing by Ginkgo framework
| * | | |   85deb914 Merge branch 'develop' into OWL-1677
| |\ \ \ \  
| * | | | | c830da0f [OWL-1677] Fix tests
| * | | | | 14d8571c [OWL-1677] Refactoring test to Ginkgo
| * | | | | 8538306f [OWL-1677] Re-write testing to Ginkgo framework
* | | | | | 906741cd [OWL-1730][nqm-mng] Rename types and methods
* | | | | | 80d82eef [OWL-1730][common][nqm-mng] Code refactoring

我们可以看到提交实际上仍然存在，并没有被合并删除。

查找影响文件的提交：`--full-history`

现在，您在这里关心的文件在提交80d82eef 中被命名为modules/nqm-mng/restful/agent.go。正如我们从--decorate otuput 中看到的，这里的分支名称是origin/OWL-1697。然而，修改这个特定文件的提交有点难以找到。

首先要知道的一点是在将git log 限制为路径名时使用--full-history（我怀疑没有办法让GitHub 这样做）。原因是，如果合并撤销了您想要完成的事情，Git 的“历史简化”会删除不对源代码的最终版本贡献的提交，使得无法找出哪个合并删除了该贡献。

要知道的第二点魔法是，在查看合并提交时，我们可能希望 -m 使 Git 将合并“拆分”成两半。

我还使用--topo-order 强制 Git 以拓扑合理的顺序显示提交（通常不是必需的，但在这里通常是个好主意）。

谁做了什么

我们现在可以看看80d82eef 对感兴趣的文件做了什么：

$ git show 80d82eef -- modules/nqm-mng/restful/agent.go
commit 80d82eef9606800c094858ac0d60f27ef9ad1307
Author: chyeh <chyeh@cepave.com>
Date:   Fri May 19 11:34:09 2017 +0800

    [OWL-1730][common][nqm-mng] Code refactoring

    Move some code from `common/` to `modules/nqm-mng/` as the preparation
    of the following development.

diff --git a/modules/nqm-mng/restful/agent.go b/modules/nqm-mng/restful/agent.go
index 3ba34901..f21afd05 100644
--- a/modules/nqm-mng/restful/agent.go
+++ b/modules/nqm-mng/restful/agent.go
@@ -124,10 +124,3 @@ func clearCachedTargetsOfAgentById(
        r := commonNqmDb.DeleteCachedTargetsOfAgentById(q.AgentID)
        return mvc.JsonOutputOrNotFound(r)
 }
-
-func nqmAgentHeartbeat(
-       req *commonNqmModel.AgentHeartbeatRequest,
-) mvc.OutputBody {
-       r := commonNqmDb.AgentHeartbeat(req)
-       return mvc.JsonOutputBody(r)
-}

现在我们可以运行这个相当长的命令了（为了显示的目的，我把它分成了两行）：

$ git log --full-history -m -p --topo-order \
    origin/OWL-1697 -- modules/nqm-mng/restful/agent.go

注意三个额外选项（完整历史记录、合并拆分和拓扑顺序）。

输出很长，所以我不会全部引用，但显示的提交是：

f0797f8e1e2e5a41db30225f4e72dc987b055ac8，在其中重新删除失败合并带回的行；
7e00e309f0ed5750f6b7b052e77431ec4797d601（与它的第一个父级 d2d4debd 相比），它重新添加您不想要的行）；
648a324e1333814968a09e4b0277fc0774b4fce6（与其第一个父级 447ac685 相比），它只影响 import 行；
ccbad82029c50040250a48180caeacfcc57044bd（与其 second 父级 da0dcde6），也重新添加了您不想要的行；
dfa34ab98170d62e1bb6d624c2d09f9e4e7e0b57（与其第二个父级 5571dc2c）；
219347a6f1649ccfee8e69b481e3166d8d310fe0（与其第二个父级 1a60e4aa）；
b11d41be75700c450280a4a5e0edb12381045303（与其第二个父级 28176808）；
8f5a0e242fd7c7ded6cdcc4517c33c793b96f8ea（与其第二个父级 5178acdb）；
e1d369d06a49846f39661642bdfb7c0c81a86b8e（与其第一个父级 8b1d4f0f）；
d04492d986fdf1257cb93a5524f2501e767bd564：这是您在 5 月 2 日进行的一次普通提交，其中添加了您不想要的行；
80d82eef9606800c094858ac0d60f27ef9ad1307：这是您在 5 月 19 日所做的普通提交，您在其中删除了您不想要的行；
2438360a653e5c158b366f313bf771db4f294147：这是您在 5 月 2 日进行的一次普通提交，其中添加了您不想要的行；
然后是许多不影响您关心的行的不相关提交，所有这些都早于这些更改。

将这些放在上下文中

根据您进行合并的方式，“最有趣”的往往是第一个父级。上面就是这种情况。

查看合并的合并基础也是一个好主意，该合并会带回您不想要的更改。为此，我们需要在该合并的两个父级的两个哈希 ID 上运行 git merge-base。提醒一下，该合并的哈希 ID 是 7e00e309（或完整的 7e00e309f0ed5750f6b7b052e77431ec4797d601）。所以让我们对两个父母运行git merge-base：

$ git merge-base --all 7e00e309^1 7e00e309^2
b598349a0ba6d6901ef812440746e2dc633c4cdc

提交“低于”这一点往往不感兴趣。

我们现在可以使用：

git log --decorate --oneline --graph origin/OWL-1697

在完整的上下文中查看所有这些，尽管匹配哈希 ID 非常困难和痛苦。或者，我们可以运行：

git log --decorate --oneline --graph origin/OWL-1697 \
    --full-history -- modules/nqm-mng/restful/agent.go

当与它们的任何父提交相比时，将该显示剥离为仅触及该文件的提交。结果仍然很长，但现在可以遵循。让我们跟随它到合并基础，即以b598349a开头的提交：

* f0797f8e [OWL-1730][nqm-mng] Fix the proble from merge
*   7e00e309 Merge branch 'develop' into OWL-1697
|\  
| *   648a324e Merge pull request #306 from Cepave/OWL-1771
| |\  
| | * 4d722f8d [OWL-1771] Change `gin`'s import paths
| * |   447ac685 Merge branch 'roby-testing' into develop
| |\ \  
| * | | ee1daba9 Merge pull request #305 from masato25/OWL-1674_ma
| | |/  
| |/|   
| * |   ccbad820 Merge pull request #303 from masato25/OWL-1740
| |\ \  
| * \ \   69a9ee6e Merge pull request #302 from masato25/OWL-1765
| |\ \ \  
| | * \ \   9e9d1a55 Merge branch 'develop' into OWL-1765
| | |\ \ \  
| | |/ / /  
| |/| / /   
| | |/ /    
| * | |   dfa34ab9 Merge pull request #300 from masato25/OWL-1740
| |\ \ \  
| | |/ /  
| * | |   219347a6 Merge pull request #301 from masato25/OWL-1755
| |\ \ \  
| | |/ /  
| * | |   b11d41be Merge pull request #299 from masato25/OWL-1755
| |\ \ \  
| | |/ /  
| | | /   
| | |/    
| |/|     
| * |   8f5a0e24 Merge pull request #298 from humorless/OWL-1644-c
| |\ \  
| * \ \   e1d369d0 Merge pull request #286 from Cepave/OWL-1667
| |\ \ \  
| | |/ /  
| |/| |   
| | * | d04492d9 [OWL-1667][common][nqm-mng] From `hbs` to `nqm-mng`
| * | |   8b1d4f0f Merge pull request #297 from hitripod/develop
| |\ \ \  
| * | | | 05e45dc8 Merge pull request #296 from masato25/OWL-1765
| | |_|/  
| |/| |   
* | | |   a847d783 Merge branch 'OWL-1677' into OWL-1697
|\ \ \ \  
| * \ \ \   ea916618 Merge branch 'develop' into OWL-1677
| |\ \ \ \  
| | |/ / /  
| |\ \ \ \  
| | |/ / /  
| * | | |   85deb914 Merge branch 'develop' into OWL-1677
| |\ \ \ \  
* | | | | | 80d82eef [OWL-1730][common][nqm-mng] Code refactoring
* | | | | | 2438360a [OWL-1667][common][nqm-mng] From `hbs` to `nqm-mng`
| |_|/ / /  
|/| | | |   
* | | | | b598349a Merge pull request #295 from masato25/OWL-1724
| |_|/ /  
[snip]

此输出包含合并出错的所有原因。就 Git 而言，在提交 2438360a 和 80d82eef 中添加然后立即删除这些行是完全不相关的。重要的是在d04492d9 中独立添加这些相同行。这就是一系列合并带来的变化的来源，最终由合并7e00e309带来。 Git 所能告诉的只是这些行应该在那里，因为它们是由合并的一个“边”添加的，而另一边对文件没有任何作用！

【讨论】：

非常感谢！根据你的回答，我找到了原因。那是因为我rebase'd OWL-1667，这导致了不必要的提交2438360a 。

【解决方案2】：

由于您在功能分支中所做的文件更改尚未提交，因此在合并时您可能会将它们隐藏起来。因此，在合并操作时，这些更改会添加回来。