Package Review Process/ru: Difference between revisions

Автор: Tom 'spot' Callaway и другие
Редакция: 0.06
Изначальный черновик: 2007-03-12
Последняя версия: 2010-11-13
Сверил с оригиналом: Mamasun 08:34, 27 December 2011 (UTC)

In other languages: English (en) español (es) русский (ru) 中文（中国大陆） (zh-cn) [edit]

Терминология
Некоторые важные термины даны в английском варианте.

Цель проверки (Review)

При добавлении нового пакета в Fedora он в первую очередь должен пройти формальную процедуру проверки (review). Цель данной процедуры (review) заключается в попытке убедиться, что пакет удовлетворяет требованиям качества Fedora. Это не значит, что пакет идеален, но что он удовлетворяет хотя бы минимальным требованиям качества.

Процедуру проверки (review) проходят полностью новые пакеты, переименованные пакеты и пакеты из старых репозиториев Fedora Core.

Процесс проверки (review)

В данном процессе участвуют два человека: автор (contributor) и рецензент (reviewer). В данном документе мы опишем обе роли.

Автор (Contributor)

Автор — это человек, который хочет добавить(submit) и поддерживать (maintain) новый пакте в Fedora. Чтобы стать автором необходимо следовать инструкциям, описанным здесь.

Как автор, вы должны создать пакет в соответствии с Package Naming Guidelines и Packaging Guidelines. Существует ряд пакетов, которые не могут быть включены в Fedora, поэтому чтобы убедиться, что ваш пакет не является таковым, проверьте не содержит ли он Forbidden items.

Когда вы полностью довольны своим spec-файлом, вам необходимо представить (submit) SRPM на проуедуру проверки (package review). Для этого следуйте инструкциям:

Опубликуйте ваш spec-файл и SRPM где-нибудь в интернете.
Заполните запрос на проверку (request for review) в багзилле. В качестве руководства используйте скриншоты образцов заполнения запросов на проверку.
File:PackageReviewProcess review.png
Если у вас до сих пор не було пакетов в Fedora, то вам необходим поручитель (sponsor), поэтому добавьте FE-NEEDSPONSOR к багам, которые блокируются выполнением вашего запроса (review request). Для получения более подробной информации прочитайте How to get sponsored into the packager group.
Дождитесь пока кто-то возьмется проверить ваш запрос! В этот момент флаг fedora-review flag будет белым, что означет, что рецензент (reviewer) назначен.

Review Swaps
If nobody comments on your review request, you might want to mail to a mailing list (devel@lists.fedoraproject.org, for example) asking for a "review swap". This is an offer to do a review of someone else's package in exchange for them reviewing your package. This is usually one-for-one, or can be some other private arrangement depending on the difficulty of the respective packages.
Могут появляться комментарии от людей, которые формально не являются реуензентами пакета, они могут добавлять пометки NotReady в поле Whiteboard, означающие, что запрос на проверку (review request) еще не прочтен по каким-то причинам, о которых они сообщают. После того, как вы напишите им, пожалуйста, опубликуйте ссылки на обновленные SPEC и SRPM файлы и удалите их из Whiteboard. Ожидается, что вы будете реагировать на комментарии, включая обновление вашего запроса в ответ; если же вы этого не делаете, то ваш тикет будет закрыт.
Рецензент (reviewer) берется за проверку вашего пакета. Флаг fedora-review flag изменится на ?
Рецензент (reviewer) проверит ваш пакет. Вы должны будете устранить все недочеты, которые он найдет. Как только рецензент доволен пакетом, флаг fedora-review примет значение +, указывающее на то, что пакет прошел процедуру проверки(package has passed review).
В этот момент вам необходимо сделать запрос SCM admin request для вашего только что одобренного пакета.
Когда это сделано, вы можете импортировать ваш пакет в SCM.
Checkout the package using "fedpkg clone <package-name>" do a final check of spec file tags, etc.
Request a build by running "fedpkg build".
Repeat the process for other branches you may have requested.
Request updates for Fedora release branches, if necessary, using "fedpkg update" or another Bodhi interface as detailed in Bodhi Guide.
Вы должны убедиться, что ваш тикет (review ticket) закрыт. You are welcome to close it once the package has been built on the requested branches, or if you built for one of the Fedora release branches you can ask Bodhi to close the ticket for you when it completes the process. If you close the ticket yourself, use NEXTRELEASE as the resolution.

Вам не нужно проходить процесс проверки снова при внесении последующих изменений в ваш пакет.

Рецензент (Reviewer)

Рецензент — это человек, который выбран для проверки пакетов.

Комментарии от посторонних
Комментарии от посторонних людей так же приветствуются. Особенно от людей, которые ищут поручителей, что демонстрирует их знание Packaging Guidelines.

Рецензентом (Reviewer) может быть любой зарегистрированный участник Fedora, состоящий в группе packager group. Есть одно исключение: если пакет является первым для автора (Contributor), рецензент (Reviewer) должен быть поручителем (Sponsor). Вы можете проверить есть ли у автора поручитель, поискав его e-mail в packager group of the account system.

Поиск запросов на проверку: http://fedoraproject.org/PackageReviewStatus/ (флаг fedora-review flag пустойили баг подписан на nobody@fedoraproject.org)
If you notice some issues that need to be solved before you want to start a formal review, add these issues in a comment and set the Whiteboard of the bug to contain NotReady. This helps other possible reviewers to notice that the review request is not yet ready for further review action.
if you want to formally review the package, set the fedora-review flag to ? and assign the bug to yourself.

Stepping back from a Review
If you want to step back from the review for any reason, reset the fedora-review flag to be blank and reassign the bug to the default owner of the component, which is nobody@fedoraproject.org
Review the package ...
- Go through the MUST items listed in Review Guidelines .
- Go through the SHOULD items in Review Guidelines .
Include the text of your review in a comment in the ticket. For easy readability, simply use a regular comment instead of an attachment.
Take one of the following actions:
- ACCEPT - If the package is good, set the fedora-review flag to +
  
  Time to sponsor?
  If the Reviewer is also acting as Sponsor for the Contributor, then this is the time to sponsor the Contributor in the account system
- FAIL, LEGAL - If the package is legally risky for whatever reason (known patent or copyright infringement, trademark concerns) close the bug WONTFIX and leave an appropriate comment (i.e. we don't ship mp3, so stop submitting it). Set the fedora-review flag to -, and have the review ticket block FE-Legal.
- FAIL, OTHER - If the package is just way off or unsuitable for some other reason, and there is no simple fix, then close the bug WONTFIX and leave an appropriate comment (i.e. we don't package pornography for redistribution, sorry. Or, this isn't a specfile, it's a McDonald's menu, sorry.) Set the fedora-review flag to -.
- NEEDSWORK - Anything that isn't explicitly failed should be left open while the submitter and reviewer work together to fix any potential issues. Mark the bug as NEEDINFO while waiting for the reviewer to respond to improvement requests; this makes it easier for reviewers to find open reviews which require their input.
Once a package is flagged as fedora-review + (or -), the Reviewer's job is done although they may be called upon to assist the Contributor with the import/build/update process and to sure that the Contributor closes the ticket out when the process is complete.

Значения флага fedora-review

fedora-review	(BLANK)	Пакет нуждается в проверке
fedora-review	?	Процедура проверки начата
fedora-review	-	Пакет не прошел процедуру проверки по юридическим или иным причинам (Package Failed Review)
fedora-review	+	Пакет одобрен (Package Approved)

Специальные блокирующие тикеты (Special blocker tickets)

Существует ряд тикетов, которые могут быть помещены в поле "Blocks" для указания на определенный статус запроса:

FE-NEEDSPONSOR	Автору необходим поручитель(sponsor); проверка (review) должна быть выполнена поручителем (sponsor).
FE-DEADREVIEW	Запрос закрыт из-за того, что автор пропал; подобные "мертвые" запросы могут быть полезными людям, которые ищут пакет, чтобы стать его автором.
FE-Legal	Пакет ожидает проверки командой правовой командой.

Поле Whiteboard

Чтобы сэкономить время рецензента (reviewer), на странице http://fedoraproject.org/PackageReviewStatus/NEW.html не будут отображаться запросы, которые не могут быть рассмотрены. Поле Whiteboard может быть использовано для дополнительных пометок о состоянии тикета, которые могут сделать тикет неотображаемым или отображаемым иначе.

NotReady	Пакет еще не готов к проверке (review). Можно открывать запрос на проверку (review ticket) с пометкой NotReady и работать над пакетом до тех пор, пока он не будет готов к просмотру рецензентом (reviewer).
BuildFails	Пакет не может быть собран (The package fails to build).
StalledSubmitter	Процесс проверки застрял, потому что автор не отвечает.

@@ Line 58: / Line 58: @@
 Рецензент &mdash; это человек, который выбран для проверки пакетов.
-{{admon/note|Comments by other people|
+{{admon/note|Комментарии от посторонних|
-Other people are encouraged to comment on the review request as well. Especially people searching for sponsorship should comment other review requests to show, that they know the [[Packaging:Guidelines|Packaging Guidelines]].
+Комментарии от посторонних людей так же приветствуются. Особенно от людей, которые ищут поручителей, что демонстрирует их знание [[Packaging:Guidelines|Packaging Guidelines]].
 }}
-The Reviewer can be any Fedora account holder, who is a member of the [https://admin.fedoraproject.org/accounts/group/members/packager/* packager group]. There is one exception: If it is the first package of a Contributor, the Reviewer must be a [https://admin.fedoraproject.org/accounts/group/members/packager/*/sponsor Sponsor]. You can check if a Contributor has already been sponsored by looking the e-mail address up in the [https://admin.fedoraproject.org/accounts/group/members/packager/* packager group of the account system].
+Рецензентом (Reviewer) может быть любой зарегистрированный участник Fedora, состоящий в группе [https://admin.fedoraproject.org/accounts/group/members/packager/* packager group]. Есть одно исключение: если пакет является первым для автора (Contributor), рецензент (Reviewer) должен быть [https://admin.fedoraproject.org/accounts/group/members/packager/*/sponsor поручителем (Sponsor)]. Вы можете проверить есть ли у автора поручитель, поискав его e-mail в [https://admin.fedoraproject.org/accounts/group/members/packager/* packager group of the account system].
 <ol>
-<li> Search for a review request that needs a reviewer: http://fedoraproject.org/PackageReviewStatus/ ('''fedora-review flag''' is blank or the bug is assigned to nobody@fedoraproject.org)
+<li> Поиск запросов на проверку: http://fedoraproject.org/PackageReviewStatus/ (флаг '''fedora-review flag''' пустойили баг подписан на  nobody@fedoraproject.org)
 <li> If you notice some issues that need to be solved before you want to start a formal review, add these issues in a comment and set the Whiteboard of the bug to contain NotReady. This helps other possible reviewers to notice that the review request is not yet ready for further review action.
 <li> if you want to formally review the package, set the '''fedora-review''' flag to '''?''' and assign the bug to yourself.

Search