openSUSE:审查工具

跳转到：导航，搜索
Factory 开发
开发模型 - Factory 提交 - 维护更新流程 - 法律审查 - Factory 审查 - Check-in - openSUSE 审查团队 - 工具 - 发布流程


使用 osc 进行交互式审查

这取决于刚刚发布的 osc 版本 0.135（从 openSUSE:Tools 仓库获取）。

打开一个大的终端窗口，输入 "osc request list -U <您的ID> -i" 并浏览列表。它将显示提交请求并提供以下选项（括号中的字母是要键入的字母，例如 i 表示 diff）

 "d(i)ff/(a)ccept/(d)ecline/(r)evoke/(b)uildstatus/c(l)one/(e)dit/(s)kip/(c)ancel"

重要的选项是

  • diff: 在您的编辑器中打开更改的差异
  • accept: 接受审查/请求（您可以使用 "-m 注释" 提供一些注释）
  • decline: 拒绝审查/请求。如果您之前请求了差异，它将再次在您的编辑器中打开差异，并允许您轻松地复制和粘贴一些行到您的拒绝消息中，该消息将添加到差异的顶部。
  • revoke: 撤销请求
  • buildstatus: 显示软件包是否已构建。
  • clone: 从正在审查的 SR 克隆一个新的 SR。
  • skip: 暂时忽略审查/请求，转到下一个。
  • cancel: 停止审查。
请注意用于选择“diff”和“decline”的键。您可能会想按“d”来显示源差异，然后发现按“d”后，您面临的是拒绝 SR 的过程。幸运的是，“decline”操作需要确认，因此如果您不小心按了“d”，可以中止“decline”过程。

一些示例

  • osc review list -G opensuse-review-team openSUSE:Factory -i: 交互式地审查所有具有 opensuse-review-team 组的开放审查以及 openSUSE:Factory 作为目标项目的软件包。
  • osc request list -U a_jaeger -i: 交互式地审查所有具有开放提交请求且我（我的用户在 openSUSE 构建服务中是 a_jaeger）被允许批准/拒绝的软件包。

如果您是某个组的成员，还应将选项 "review_inherit_group = 1" 添加到您的 ~/.oscrc 中。这样，它就不会询问您要审查哪个组，而是采用您在命令行（"-G opensuse-review-team"）中使用的组。

使用 Web UI 进行交互式审查

访问 OBS 项目列表 显示顶部的“主项目”列表。这些主项目通常是受审查流程约束的项目。单击您将要审查的项目链接。

生成的页面是项目摘要页面。在“信息”部分，您将看到指向“开放事件”的链接，该链接还显示当前开放事件的数量。单击“开放事件”链接。将显示事件列表。

通常，作为审查员，您应该仅审查那些处于“审查”状态的项目。所有处于“审查”状态的事件在“信息”列中都标有黄色标志。

对于您审查的每个 SR，首先要验证表中的所有数据表明 SR 准备好进行审查。尤其重要的是

  • 验证 SR 是否具有合理的摘要
  • 验证 patchinfo bug 引用和 CVE 引用是否与软件包中列出的引用匹配。（请参阅“问题”列。）
  • 确保“构建”列中有一个绿色复选标记，表明软件包已在维护者的项目中成功构建。
审查的细节——如何审查、哪些内容被接受以及哪些内容应该被拒绝——在这里没有描述，而是在此门户的其他页面中讨论。

要进行审查，请单击黄色标志。将显示“请求 <SR#> (审查)”页面。

每个软件包的源差异显示在屏幕的主要部分。您可以使用顶部选项卡切换到每个软件包。至少有一个选项卡也将是 patchinfo。

您还可以查看右侧窗格中的“构建结果”和“提及的问题”（bugzilla 数据）。

作为审查团队的成员，您可以在顶部添加评论，然后按“接受审查”或“拒绝审查”。如果您想跳过此审查并在以后进行或允许其他人进行审查，只需按浏览器的“后退”按钮即可。

模板

缺少补丁提及

Please see
https://en.opensuse.net.cn/openSUSE:Packaging_Patches_guidelines#Patch_live_cycle

=> Added / removed patches need to be mentioned by name in .changes, in
order to get a full trail of life cycles.

缺少或错误的 bug 编号缩写

Please see 
https://en.opensuse.net.cn/openSUSE:Howto_write_good_changes#Bug_fix.2C_feature_implementation

=> Anytime you have fixed a bug (or implemented the feature), you have to mention the number of bug in changes. As fix should be reported in upstream bugzilla, also add a prefix before the number, so people will know where to find an information. 

The full list of prefixes is available on https://en.opensuse.net.cn/openSUSE:Packaging_Patches_guidelines#Current_set_of_abbreviations

在 %install 中清理 buildroot

Please read the thread about "cleaning the buildroot correctly" on
 http://lists.opensuse.org/opensuse-packaging/2007-02/msg00006.html
and fix your package ...   Thanks!

更改中仅有版本号

Please be a bit more verbose in the changes entry ... (not just version number).

Please read https://en.opensuse.net.cn/openSUSE:Howto_write_good_changes for more details about a good changelog entry.
从 "https://en.opensuse.net.cn/index.php?title=openSUSE:Review_tools&oldid=73500" 检索