[devel] new check for sisyphus_check

[devel] new check for sisyphus_check

[Alexey Tourbin]

[-- Attachment #1: Type: text/plain, Size: 253 bytes --]

Предлагаю добавить ещё одну проверку в sisyphus_check: все патчи,
перечисленные с помощью тегов, должны впоследствии накладываться с
помощью макросов %patch.  Это можно легко сделать через rpm -bE.

Забыли наложить патч -- это достаточно частая ошибка.

[-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --]

Re: [devel] Re: new check for sisyphus_check

[Alexey Voinov]

[-- Attachment #1: Type: text/plain, Size: 708 bytes --]

Alexey Tourbin wrote
> On Fri, Jun 27, 2003 at 08:33:08PM +0300, Alexander Bokovoy wrote:
> > Иногда патч не прикладывается по серьезным причинам, но держится в пакете
> > для того, чтобы, например, переделать его позже.
> Да.  У меня тоже так.
> Ну тогда warning, но его скорее всего не увидят...
Не надо, пожалуйста такого варнинга.
"Оставленый" патч, это вполне обычная ситуация.
Варнинг будет очень напрягать, особенно тех, кто любит -Wall -Werror :)


-- 
Best Regards!           | Когда вам платят за работу, надо по крайней мере
Alexey Voinov           | делать вид, что вы работаете...
                        |                         Б.Виан "Осень в Пекине"
voins@voins.program.ru
vns@altlinux.ru


[-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --]

Re: [devel] new check for sisyphus_check

[Denis Ovsienko]

> Предлагаю добавить ещё одну проверку в sisyphus_check: все патчи,
> перечисленные с помощью тегов, должны впоследствии накладываться с
> помощью макросов %patch.  Это можно легко сделать через rpm -bE.
>
> Забыли наложить патч -- это достаточно частая ошибка.
Данная проверка не позволит сохранить в пакете патч, к которому позже есть
вероятность вернуться. Кроме того, в пакетах, из которых строится более
одного бинарного пакета с индивидуальными патчами, либо в спеке, в котором
стоят %define по нужным опциям и в зависимости от них прикладываются
патчи, ситуация только осложнится.
Можно делать warning, а потом посмотреть, даст это что-нибудь или нет.

--
    DO4-UANIC