Linus Torvalds 批评 Paragon Software 并在 GitHub 的过程中创建了不必要的合并

Linus Torvalds 一直在等待 很长一段时间 让 Paragon Software 发布其驱动程序 NTFS 将其添加到 Linux 内核 这已经完成了 Torvalds 终于将新驱动程序与 Linux 5.15 内核源代码合并。

但在此之前他抱怨使用 GitHub 的合并请求功能 在帖子中,说 GitHub “创建了绝对不必要的合并”。 显然,Linux 内核的创建者并不真正喜欢 GitHub 合并,如果他们这样做的话。 此外,编译器警告现在被视为内核构建中的默认错误。

八月初, Torvalds 为 Paragon Software 提供了一个推动请求的方法» 将与内核源代码合并的实际代码发送,以便其 NTFS 读/写驱动程序可以包含在下一个版本 5.15 中,该版本当前为其打开了合并窗口。

Paragon 正式提交了“撤回请求”,称:

“当前版本适用于普通/压缩/spars 文件,并支持 acl 和 NTFS 日志播放。” 也就是说,该公司仍在熟悉提交过程,Torvalds 有很多评论要发表,目的是改进未来的拉取请求。 首先,Linux 内核的创建者说,pull request 应该已经签名了。 “在一个完美的世界里,这将是一个 PGP 签名,我可以通过信任链直接追踪到你,但我从来没有真正要求过它,”他说。

然后 注意到拉取请求中的代码包括使用 GitHub Web UI 进行的合并提交.

“这是我 * 真的 * 不想看到的另一件事:GitHub 创建了不必要的合并,你永远不应该使用 GitHub 接口来合并任何东西。” 合并不是这些事情之一。 过去,尤其是 2012 年,Torvalds 曾抱怨 GitHub 的某些方面。

“我不做 GitHub 拉取请求。 GitHub 会丢弃所有相关信息,例如即使是发出拉取请求的人的有效电子邮件地址。 diffstat 也缺乏和不必要的,“他当时说。 请注意, git request-pull 命令与来自 GitHub 的拉取请求功能不同。 此外,Paragon 存储库具有缺少信息的提交消息,例如 »Merge branch 'torvalds: master' into master«。 谈到这一点,Torvalds 周六表示“Linux 内核合并必须*正确*”。

除了添加控制器 Paragon NTFS 到 Linux 内核 5.15, Torvalds 还启用了“-Werror”构建选项, 这是所有内核构建的默认值。

编译标志 »-Werror«将所有警告视为编译错误. 通过将所有警告提升为错误,Torvalds 强制开发人员确保在其他情况下可能会被开发人员忽视或仅受到开发人员松散影响的构建警告现在被优先考虑,因为它们会中断构建过程。

Torvalds 评论 关于变化 我们真的应该总是有一个干净的构建 如果我们无法修复它们,我们将在必要时禁用过度热心的特定警告。 但是,虽然我在自己的树上非常虔诚地执行了这一点,但不一定报告警告的各种建筑机器人却没有。

确保处理警告,但是这些项目大部分都没有Linux内核那么大。 另一方面,WERROR 作为 Kconfig 开关。 如果较新版本的编译器引入了内核无法立即纠正的新警告或无法在没有警告的情况下创建内核的其他选择性问题,这将禁用 »-Werror» 标志。 WERROR 选项默认为所有内核构建启用。

Linus 发完消息 补丁的:

“希望这意味着我将收到更少的拉取请求,其中包含我们实施的各种自动化没有注意到的新警告。” 我敲木头。


发表您的评论

您的电子邮件地址将不会被发表。 必填字段标有 *

*

*

  1. 负责数据:MiguelÁngelGatón
  2. 数据用途:控制垃圾邮件,注释管理。
  3. 合法性:您的同意
  4. 数据通讯:除非有法律义务,否则不会将数据传达给第三方。
  5. 数据存储:Occentus Networks(EU)托管的数据库
  6. 权利:您可以随时限制,恢复和删除您的信息。