Bug 27709 - no check arch in check-firmware
Summary: no check arch in check-firmware
Status: CLOSED FIXED
Alias: None
Product: Sisyphus
Classification: Development
Component: sisyphus_check (show other bugs)
Version: unstable
Hardware: all Linux
: P3 normal
Assignee: Dmitry V. Levin
QA Contact: qa-sisyphus
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-09-07 04:11 MSK by led
Modified: 2012-09-15 03:44 MSK (History)
8 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description led 2012-09-07 04:11:03 MSK
Прошу убрать следующую проверку из check-firmware:

if [ "$rpm_arch" != noarch ]; then
        FileError "package architecture should be 'noarch'" "$f"
        rc=1
fi

Обоснование: набор файлов в пакетах firmware-kernel для разных архитектур может отличаться (даже для x86 и x86_64, не говоря уже о не-x86 архитектурах)
Comment 1 Michael Shigorin 2012-09-07 13:02:29 MSK
Видимо, стоит смягчить до warning и оставить на рассуждение майнтейнера.
Comment 2 Dmitry V. Levin 2012-09-12 14:56:32 MSK
У нас есть действующее правило про noarch в http://www.altlinux.org/FirmwarePolicy

Будем отменять?
Comment 3 led 2012-09-12 15:59:24 MSK
(В ответ на комментарий №2)
> У нас есть действующее правило про 
> oa
> ch в http://www.altliux.og/FimwaePolicy

>Будем отменять?

Не знаю. Но, как минимум третий пункт этого правила не имеет под собой никаких оснований и не соответствует (и не может соответствовать) реальной действительности. К тому же, IMHO, его отмена не влечё никаких негативных последствий.
Comment 4 led 2012-09-12 16:13:42 MSK
(В ответ на комментарий №3)
>третий пункт этого правила не имеет под собой никаких оснований

Точнее, не имеет оснований как обязательный. Как рекомендательный - может быть.
Comment 5 Anton Farygin 2012-09-12 16:27:17 MSK
Дима, надо смягчить. Согласен с тем, что firmware могут быть архитектурно-зависимыми, и более того - разными для разных архитектур и одного устройства.
Comment 6 Dmitry V. Levin 2012-09-12 17:39:09 MSK
(In reply to comment #3)
> (В ответ на комментарий №2)
> > У нас есть действующее правило про 
> > oa
> > ch в http://www.altliux.og/FimwaePolicy
> 
> >Будем отменять?
> 
> Не знаю. Но, как минимум третий пункт этого правила не имеет под собой никаких
> оснований и не соответствует (и не может соответствовать) реальной
> действительности.

Вопрос был только по третьему пункту.  Чрезмерность требования noarch сейчас кажется очевидной, но 3.5 года назад это требование, кажется, не вызывало вопросов.

> К тому же, IMHO, его отмена не влечёт никаких негативных последствий.

OK
Comment 7 Repository Robot 2012-09-13 01:32:10 MSK
sisyphus_check-0.8.35-alt1 -> sisyphus:

* Wed Sep 12 2012 Dmitry V. Levin <ldv@altlinux> 0.8.35-alt1
- 211-check-firmware: removed "noarch" check (closes: #27709).