如何在代码审查中审查 InstallShield ism 文件答案

【问题标题】：How to review InstallShield ism files in code review如何在代码审查中审查 InstallShield ism 文件
【发布时间】：2020-01-22 08:09:04
【问题描述】：

我们最近对签入我们的源代码控制 (Helix Perforce) 的文件实施了强制审查 (Helix Swarm)。目前，这也适用于工具生成的文件，尤其是 InstallShield .ism 文件。由于它们是文本文件，因此可以查看经典差异。这种审查的质量当然很差，因为这些文件不是为了人类可读的。另一种方法是在 InstallShield 编辑器中打开文件，但我的团队中只有少数人拥有所需的许可证，这被证明是审查速度的一个重大瓶颈。

由于上述问题，我们正在讨论取消对 InstallShield 的审查。我想知道是否有可行的方法来查看 InstallShield 文件，以便两个人可以查看安装例程中的更改。

【问题讨论】：

如果您的 .ism 文件是保存的纯文本而不是二进制文件，我想您可以使用 XML 解析器
嗯，这至少可以消除所有视觉上的混乱。我不确定这是否有助于读者了解对文件的更改是否有效。
我会为此目的指向 InstallShield MSI Diff 工具，但我认为它仅适用于二进制 ISM 文件。而且它仍然是表级的，所以它只比 XML 更易读。而且我忘记了它的许可条款是否对您想要的内容足够友好。在实践中，除非您的整个团队在安装方面足够专业，否则无论如何他们都很难审查 ISM 更改。

标签： installshield pull-request

【解决方案1】：

现在我们决定只做简单的合理性检查。将拉取请求中描述的内容与可以从 XML 文件的呈现差异中轻松掌握的信息进行比较。

例子：

说明：添加/更新库 xy

检查添加/更改/删除的行是否包含库名称和可能的版本。如果看似不相关的文件行发生更改，则需要报告可能存在的缺陷。

这可能不是对设置最终会做什么的完美质量控制，但至少我们可以及早发现明显的错误，而无需进一步的工具。

【讨论】：

【解决方案2】：

您可以在 Perforce 中通过扩展名将“MsiDiff.exe”添加为差异应用程序，以查看 InstallShield ism 文件。

MsiDiff.exe 可以在 [IS-INSTALLDIR]\System 中找到。

在 Perforce 中添加它的方法如下： Perforce diff application by extension

【讨论】：