PatchReviewParty

来自 PostgreSQL wiki
导航至搜索

既是 UG 会议又是黑客马拉松:它是 Patch Review Party,它很快要到您的城市了 - 因为您将举办一场!您的目标是让人们了解补丁审查流程,让他们加入 RRReviewers,并审查一些补丁。

组织者(就是您!)在事前需要做的事情

找一些人参加

让参与者提前注册。PDXPUG 将参与者限制在 10-12 人,通常是因为空间限制。

确定时间

3 小时似乎很合适 - 2 小时太短,4 小时则会占用人们过多的时间。对我们的团队来说,工作日晚上(星期日到星期四)最合适。

提供用户支持

  • 饮料和食物(如果您在周末这样做,聚餐会比较合适)
  • 拼车设置/停车/方便近使用公共交通
  • 座位
  • 方便的卫生间
  • 网络
  • 在 USB 存储设备中备上最新的 Pg 和补丁(因为通常有人会忘记下载)
  • 电源
  • 某种信息共享方式,如白板、幻灯片或您的团队的 IRC 频道。在我们的第一次 PRP 中,一些非本地 Pg 专家以及当前提交负责人自愿坐在我们团队的 IRC 频道中,并回答问题。这帮了很大的忙。

最后,将他们需要在事前做的事情通知与会者(下一章节)。

与会者在事前需要做的事情

设置当前版本的 Pg

(仅此一步就能节省大量时间)

git clone git://git.postgresql.org/git/postgresql.git

如果您已克隆代码库数天,请在 PRP 之前用 `git pull origin` 更新代码。

configure && make && make check

确保您在远离任何您不想弄乱的 Pg 安装的地方执行此操作。

查看 Wiki 上的相关文档

https://wiki.postgresql.ac.cn/wiki/RRReviewers

https://wiki.postgresql.ac.cn/wiki/Reviewing_a_Patch

获取补丁

对于我们的第一个 PRP,我们让 CommitFest 经理给我们的组分配一些容易审查的补丁。现在,我们通常会从 https://commitfest.postgresql.org/action/commitfest_view/inprogress 中的当前补丁列表中选择我们自己的补丁。

参会人员可以下载自己的补丁(如果可能的话,提前下载!),或者组织者可以通过 USB 提供补丁。

在 PRP 中

进行单人和配对审查的效果不错。选择最适合您的方式;在我们的最后一个 PRP 中,我们使用了混合方式并且效果很好。如果这是你的第一个 PRP,你可能希望提前将人员配对,这样新人员就可以与更有经验的人员一起工作。

如果你以前从未修补过,如何修补(老式方法)

patch -b -p0 < [patchfile]

或者,使用 git 更简单便捷

git apply [patchfile]

...然后再次运行 `configure && make check && make install`。

一些提示

  • 如果你希望保持更改的独立性,可以在应用补丁之前执行“git branch”。(git branch [branch_name])
  • https://commitfest.postgresql.org/action/commitfest_view/inprogress 获取补丁:单击补丁名称,然后单击备注下的“Patch”链接转至 -hackers 邮件列表,你应该在那里找到补丁作为附件 - 确保你获取到最新版本。
  • 首先检查补丁是否采用正确的格式
  • 你可能需要处理 Windows 风格的车**riage**返回;尝试
    • dos2unix
    • perl -pi -e 's/\r\n/\n/;' [patchfile]
  • 如果你正在使用 patch,你可能需要
    • 更改补丁文件顶部**的文件路径,以匹配 git 仓库中的路径**
    • 调整补丁级别(patch 命令的 -p 参数)

总结和后续

如果并非每个人都在 PRP 中完成他们的审查,不必担心;一些补丁对于一个晚上来说太复杂了。在第一个 PRP 中,当你有一些新人员时,你可能需要了解审查的状态,并确保每个人都正确地进行了后续。 (这样可以省去 CommitFest 经理的麻烦。)