I saw this set of comments in a merge review and it struck me as a little off:
- There are outstanding bugs for [pkg] please address them.
This is not relevant to the packaging review process.
I see two problems with the above statements: 1) The reviewer is not specifying a specific problem that they consider a blocker for approval of the package. 2) The packager is not acknowledging that potential problems with the runtime of the package are relevant to the review.
Perhaps we need to have some statement of reviewers' and packagers' responsibilities at the top of the Guidelines:
"It is the reviewer's responsibility to point out specific problems with a package and a packager's responsibility to deal with those issues. The reviewer and packager work together to determine the severity of the issues (whether they block a package or can be worked on after the package is in the repository.) The Packaging Guidelines are a collection of common issues and the severity that should be placed on them. If a reviewer or packager believe their package reveals a flaw in the Guidelines, examples of how the rule is failing should be brought to the attention of the Packaging Committee so they can consider whether the rule has unanticipated consequences and modify it to better fit all situations." [1]_
Let me know what you think of having such a statement in general and this wording in particular.
[1]_: http://fedoraproject.org/wiki/PackagingDrafts/OverallReviewGoals
-Toshio
Toshio Kuratomi wrote:
I saw this set of comments in a merge review and it struck me as a little off:
- There are outstanding bugs for [pkg] please address them.
This is not relevant to the packaging review process.
As stated, I agree, it's just too vague.
What is relevant, however, are reviewers mentioning *specific* current/outstanding issues/bugs that ought to be addressed.
-- Rex
On Mon, 2007-04-09 at 08:19 -0500, Rex Dieter wrote:
Toshio Kuratomi wrote:
I saw this set of comments in a merge review and it struck me as a little off:
- There are outstanding bugs for [pkg] please address them.
This is not relevant to the packaging review process.
As stated, I agree, it's just too vague.
What is relevant, however, are reviewers mentioning *specific* current/outstanding issues/bugs that ought to be addressed.
Exactly. Do you think my proposal makes it more obvious that this is what we want or less? (Or is it just verbiage that people will skip over because it's not part of a checklist?)
-Toshio
packaging@lists.fedoraproject.org