繁体   English   中英

合并后文件的git commit中的更改消失

[英]Change in a git commit of a file disappears after merge

我在功能分支中进行开发。 那时,我做了一些其他更改,这些更改已通过github中的PR合并到developer分支中,而我需要在我的功能分支中进行这些更改。 因此,我决定将开发分支合并到功能分支中。 我已删除文件中的一些代码并提交到功能分支中。 将development合并到feature分支后,我发现文件提交中删除代码的更改消失了,并且由于合并操作而将被删除的部分重新添加回去。 我想问:在什么情况下会发生这种情况? 谢谢!

更新:

我使用github查看历史记录。 这里是例子:这是历史agent.go在提交80d82ee在特征分支OWL-1697 这是功能分支OWL-1697中此文件的全部历史记录 合并操作后,此文件的80d82ee更改消失。

我不建议使用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相同的添加行d04492d9

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

高一点。 您与2438360a添加的行在同一天进行了此2438360a 您后来在80d82eef中将线路拔出了……但是在另一个分支中 按照拓扑顺序 ,将更改放入后几乎立即将其取出...,同时还将更改留在单独的分支中。 然后,您通过e1d369d0将它们带入另一个分支, e1d369d0其带入另一个分支,依此类推,直到将它们带回到您认为已将它们取出的分支中。

查看提交图

如果我克隆您的存储库并使用git log --all --decorate --oneline --graph进行查看,我会部分地看到这一点,其中包括commit 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

现在,您在此处关心的文件在commit 80d82eef被命名为modules/nqm-mng/restful/agent.go 80d82eef --decorate otuput中可以看到,在--decorate领先的分支名称是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 (相对于其第二个父级da0dcde6 ),它还会重新添加您不想要的行;
  • dfa34ab98170d62e1bb6d624c2d09f9e4e7e0b57 (与其第二个父代5571dc2c );
  • 219347a6f1649ccfee8e69b481e3166d8d310fe0 (与其第二个父级1a60e4aa );
  • b11d41be75700c450280a4a5e0edb12381045303 (与其第二个父级28176808 );
  • 8f5a0e242fd7c7ded6cdcc4517c33c793b96f8ea (与其第二个父级5178acdb );
  • e1d369d06a49846f39661642bdfb7c0c81a86b8e (与其第一个父级8b1d4f0f );
  • d04492d986fdf1257cb93a5524f2501e767bd564 :这是您在5月2日所做的普通提交,在其中添加了不需要的行;
  • 80d82eef9606800c094858ac0d60f27ef9ad1307 :这是您在5月19日所做的普通提交,在其中删除了不需要的行;
  • 2438360a653e5c158b366f313bf771db4f294147 :这是您在5月2日所做的普通提交,在其中添加了不需要的行;
  • 然后许多不相关的提交都不会影响您关心的行,而这些提交都早于这些更改。

将这些放在上下文中

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

查看合并带来了不需要的更改的合并基础也是一个好主意。 为此,我们需要在该git merge-base的两个父级的两个哈希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而言,在commit 2438360a80d82eef添加然后立即删除这些行是完全无关的。 重要的是在d04492d9对这些相同的行进行了独立且从未取消的添加。 这就是由一系列合并带来的更改的源头,并最终由合并7e00e309带来了。 Git只能说这些行应该在那儿,因为它们是在合并的“一侧”添加的,而另一侧对文件没有任何作用!

由于您尚未在功能分支中进行的文件更改尚未提交,因此合并时可能将其保存了。 因此,在合并操作时,这些更改会重新添加。

暂无
暂无

声明:本站的技术帖子网页,遵循CC BY-SA 4.0协议,如果您需要转载,请注明本站网址或者原文地址。任何问题请咨询:yoyou2525@163.com.

 
粤ICP备18138465号  © 2020-2024 STACKOOM.COM