Bug 34929 - [FR] %e2k вместо e2k
Summary: [FR] %e2k вместо e2k
Status: CLOSED FIXED
Alias: None
Product: Sisyphus
Classification: Development
Component: php7 (show other bugs)
Version: unstable
Hardware: e2k Linux
: P3 enhancement
Assignee: Anton Farygin
QA Contact: qa-sisyphus
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2018-05-21 15:06 MSK by Michael Shigorin
Modified: 2018-05-22 10:49 MSK (History)
3 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Michael Shigorin 2018-05-21 15:06:41 MSK
Прошу поправить спек:

-%ifarch e2k
+%ifarch %e2k

...для поддержки e2kv4 и в дальнейшем -- других субархитектур; сизифный rpm этот макрос уже умеет (вот за p8 не уверен, но если вдруг нет -- это баг).
Comment 1 Grigory Ustinov 2018-05-21 15:09:12 MSK
(В ответ на комментарий №0)
> Прошу поправить спек:
> 
> -%ifarch e2k
> +%ifarch %e2k

А почему сразу мне не написать? В acl стоит @everybody же.
Comment 2 Anton Farygin 2018-05-21 15:12:40 MSK
я не хочу править ACL на php, но не надо собирать этот пакет из-за опечатки
Comment 3 Anton Farygin 2018-05-21 15:13:46 MSK
но вообще да, вы правы - не надо кому попало собирать пакет php7
Comment 4 Anton Farygin 2018-05-21 15:16:34 MSK
7.2.5-alt1%ubt-1-gef79280

в пакеты попадёт при выходе следующей версии
Comment 5 Grigory Ustinov 2018-05-21 15:19:04 MSK
(В ответ на комментарий №2)
> я не хочу править ACL на php, но не надо собирать этот пакет из-за опечатки

Это не опечатка, а реальная проблема. Семейство e2k-архитектур объединяется
макросом %e2k, который включает в себя на текущий момент две архитектуры.
Оставляя ifarch e2k мы получаем разную сборку под две похожие архитектуры. Это
совершенно неправильно. Надо исправить. Если ты не хочешь это делать, сделаю я.

А пока проблема есть, пусть лучше бага будет открыта. Закроешь её строчкой (Closes: #34929).
Comment 6 Anton Farygin 2018-05-21 15:25:05 MSK
проблемы уже нет.
Comment 7 Anton Farygin 2018-05-21 15:27:03 MSK
для сборки для e2k берите последний коммит из моего гита. А в репозиторий x86 это попадёт при выходе новой версии.

Я снял ACL с php7, что бы вы не начали делать ошибки при его обновлении.
Comment 8 Michael Shigorin 2018-05-22 10:49:38 MSK
(В ответ на комментарий №1)
> А почему сразу мне не написать? В acl стоит @everybody же.
Потому что у rider@ там довольно хитрая схема для php\d-*.

(В ответ на комментарий №7)
> Я снял ACL с php7, что бы вы не начали делать ошибки при его обновлении.
Резонно.

(В ответ на комментарий №4)
> 7.2.5-alt1%ubt-1-gef79280
> в пакеты попадёт при выходе следующей версии
Спасибо!